Add an example for model building with fluent API and Actions

This page currently has an example that looks like this:
protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    modelBuilder.Entity<Blog>()
        .Property(b => b.Url)
        .IsRequired();
}

It would be nice if it also mentioned the alternative syntax, which helps better organizing code:
protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    builder.Entity<Blog>(ConfigureBlogEntity);
}

private void ConfigureBlogEntity(EntityTypeBuilder<Blog> builder)
{
    builder.Entity<Blog>()
        .Property(b => b.Url)
        .IsRequired();
}

I couldn't actually find this way of configuring entities in the docs at all (I might have missed it), so this seems a good place to mention it.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3437af67-630b-d28f-7fbd-3eedf647f22a
Version Independent ID: 6c44f987-19a7-c256-e8fa-10378e61cf6e
Content: Creating and configuring a model - EF Core
Content Source: entity-framework/core/modeling/index.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @rowanmiller
Microsoft Alias: divega

@matteocontrini
I think this is a great feature and the command dotnet ef dbcontext scaffold must have an option to scaffold a DbContext and entity types in this way. Actually, after scaffolding a real database you end with OnModelCreating method with high cyclomatic complexity, a low maintenance index and a massive number of lines.

Duplicate of #411. Also, see https://github.com/aspnet/EntityFrameworkCore/issues/8434


376 keep current folder names

Fixes #376

I see we are still using the "tidy" country, not the "original" country for the output folder:

(Britain1 vs Britain)


Swap out existing kafka testing jar

In order to reduce our coding footprint
Swap out existing kafka testing scaffolding (zipkin-kafka-receiver) with https://github.com/chbatey/kafka-unit

fixed by https://github.com/openzipkin/zipkin/pull/476


https://github.com/rafaqz/Extents.jl/pull/10 breaks Tyler

https://github.com/rafaqz/Extents.jl/pull/10 breaks Tyler
Extents.intersect need to be changed to Extents.intersection and comp for Extents needs to be >= Extents v0.1.2

Issue is with MapTiles not Tyler


Create PRs instead of issues

When the cherry-pick does not apply correctly, try creating a PR instead of an issue.

@ybettan @yevgeny-shnaidman WDYT about this? To me the current flow works pretty well. Shall we close?

I also like the current way. The faulty PRs are due to conflicts, and it is much easier to create your own cherry-pick, resolve everything and create a new PR. I say lets close. @ybettan ?

Agree.
/close


Invalid Y axis in container statistics

The following bug has been reported:
When restricting a container's memory, the Y-axis of the container statistics chart is invalid.

It seems like portainer is showing the memory including cache. At the moment, we use memory_stats.usage the includes memory_stats.stats.cache.
"memory_stats":{
      "usage":1536000,
      "max_usage":2859008,
      "stats":{
         "active_anon":8192,
         "active_file":450560,
         "cache":708608,
         "dirty":0,
         "hierarchical_memory_limit":52428800,
         "hierarchical_memsw_limit":0,
         "inactive_anon":61440,
         "inactive_file":258048,
         "mapped_file":536576,
         "pgfault":1115,
         "pgmajfault":8,
         "pgpgin":894,
         "pgpgout":704,
         "rss":69632,
         "rss_huge":0,
         "total_active_anon":8192,
         "total_active_file":450560,
         "total_cache":708608,
         "total_dirty":0,
         "total_inactive_anon":61440,
         "total_inactive_file":258048,
         "total_mapped_file":536576,
         "total_pgfault":1115,
         "total_pgmajfault":8,
         "total_pgpgin":894,
         "total_pgpgout":704,
         "total_rss":69632,
         "total_rss_huge":0,
         "total_unevictable":0,
         "total_writeback":0,
         "unevictable":0,
         "writeback":0
      },
      "limit":52428800
   },`

I will create a PR to fix this.


Waypoint Lesson requires the removal of Monospace even though it stated the removal of a dead link in the lesson information.

Challenge http://www.freecodecamp.com/challenges/waypoint-ditch-custom-css-for-bootstrap has an issue. Please describe how to reproduce it, and include links to screen shots if possible.

go to challenge question.
Look at the requirements needed to go to the next lesson. You will notice that one of the requirements require the removal of Monospace from all paragraphs.

Read the lesson information. You will notice that it makes no mention of removing the Monospace font from all paragraphs but instead requires the removal of a dead link.

I think for consistency purposes, this should be fixed. I am using Google Chrome.

Hi @maugus33 thank you for submitting this issue. The directions say to delete the "p" declaration from your style element, and that is where the monospace font family is assigned. This said, if you misunderstood this, there's a good chance other people will misunderstand this, too, so I will update the wording. Thanks and happy coding! :)

Hi @QuincyLarson In the instruction, it still is not clear that it is the "p" declaration from the style element. New coders are still learning how to "talk" code (and digest instructions), including how to differentiate between where (style vs body vs...) something is located. If "delete the p element" means anything related to "p", this wording may be clear to advanced coders, less clear to others:

The waypoint can be completed by [also] deleting the "p" declaration from the body element, though the instructions do not require this explicitly. In the following waypoint, both declarations are absent.
Updating the instructions to "Delete the p body element that contains a dead link, then delete the p style element."  might clear up any confusion. Cheers!


Collapse/Accordion - CSS issue with only one card

Hello
I've just used the accordion with cards and on some pages I need only one card. As you can see on the screenshot the CSS messes up the borders. I think it's because the one card is both the first and last child of the accordion.
Regards
Michael

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen/JS Bin or Stackblitz and report back with your link, Bootstrap version, and specific browser and OS details.
This is an automated reply

Bottom border already fixed in #28365, but top border isn't rounded.
Fix could be: .accordion > .card:last-of-type --> .accordion > .card:last-of-type:not(:first-of-type) in _card.scss

Fixed in the meanwhile: https://jsbin.com/weresipeje/edit?html,output


Unicode codepoint truncation

Unicode codepoints above 65535 are truncated with pygame and sdl2 text providers (but not pil). For example, U+1F3A8 is rendered as U+F3A8, and U+102FB is rendered as U+02FB. This happens in Python 2 and Python 3.
Thanks to LogicalDash for bringing attention to this issue. (https://botbot.me/freenode/kivy/2014-12-19/?msg=27980189&page=3)
Tested using the Symbola font: http://users.teilar.gr/~g1951d/Symbola.zip
Test script:
import kivy
kivy.require('1.8.1')

from kivy.app import App
from kivy.lang import Builder
from kivy.core.text import Label as CoreLabel

CoreLabel.register('Symbola', 'Symbola.ttf')

root = Builder.load_string('''
GridLayout:
    cols: 2

    Label:
        font_size: sp(64)
        text: '{high}'
    Label:
        font_size: sp(64)
        text: '{low}'
    Label:
        font_name: 'Symbola'
        font_size: sp(256)
        text: u'{high}'
    Label:
        font_name: 'Symbola'
        font_size: sp(256)
        text: u'{low}'
'''.format(high='\U000102fb', low='\u02fb'))

class TestApp(App):
    def build(self):
        return root

if __name__ == '__main__':
    TestApp().run()

Output with KIVY_TEXT=pygame (reportedly the same as sdl2 output):

Output with KIVY_TEXT=pil:

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

This seems to be an upstream issue, I’ve been able to reproduce using PyTouhou, which is using SDL_ttf directly to do the rendering.

Any updates on this?


Some questions.

1.How players become co_owner
2.I tried kick but it didn't work, so I want to know how to move the player out of member
3.Visitors cannot open the menu，Is it because of [inventory] permission that visitors cannot open the menu?
menu is zmenu，but command can work，item click dont work

Kick screenshot

	[worlds.sky-overworld.portal-nether]
		enabled = false

skyllia promote player
I will check together
Yes it's possible
For the portal, create a new issues

Try with this version : https://github.com/Euphillya/Skyllia/actions/runs/9093608607

Still unable to open the menu

Promote doesn't seem to be working properly.

Still unable to open the menu

Try to contact developer to Zmenu

Promote doesn't seem to be working properly.

Strange, can you check in database if the player is correctly register

Players can't open boxes and so on in town, but the menu is normal.I don't think it's the menu. I believe the author will ignore me with such feedback.

In town ? My plugin only cancels if the player is on a skyblock island.

I mean, it works in cities and towns, but not in empty islands.

https://www.spigotmc.org/resources/zmenu-ultra-complete-menu-plugin.110402/

@404forever1 So after checking, the problem does not come from the Skyblock plugin. He should read his inventory instance last and if it is canceled, open it anyway. Adding a configuration to each inventory would be a very good idea!

What am I supposed to do?

contact the developer of your plugin.


rcon web not enabled, can login but cant' send say command

result to "say" command is "response for another request" and rcon prompt is closing
"status" or "players" command is working.

I'm new to the Rust, but what I saw on the test server tells me that the game does not give an answer to the say command in any way.
Maybe something wrong with my server, I'll check one more time.

The error message has been removed. Rust responds with an empty line to the say "message" command, and rcon-cli does the same now. Try v0.8.0-beta.2

beta 2 seems to be fully working.
I'll test some more commands and report potential bugs.

@gdepre Thanks for your feedback!


Update VMSS instances with updated VMSS model for Trusted Launch features

Related command
az vmss create
az vmss update-instances
Is your feature request related to a problem? Please describe.
After including the Guest Attestation by default feature in the latest version, I discovered a miss from the original request specification. For the VMSS scenario, the current behavior is that the VMSS model reflects the MSI enablement and Guest Attestation Extension installation, but the instances are not actually updated with the model. This is because by default, VMSS has the upgrade policy set to manual. This blocks the actual VM instances of VMSS to actually get the Trusted Launch features.
Describe the solution you'd like
In order to apply the updated VMSS model to the instances, we need to accomplish the equivalent of the following command AFTER MSI and the extension is updated in the VMSS model in the convenience layer:
az vmss update-instances --resource-group "rgname" --name "vmssName" --instance-ids *
Additional context
This was a miss from the original request

Compute

Would it be possible to leverage Get-AzVmssVM to run the following test: Using the current version of the PS modules where only the VMSS model is updated, created a VMSS with trusted launch configuration, confirm that the individual instances do not have the extensions, then compare with using the new version of the PS modules that contains the requested changes and confirm that the individual instances actually has the extension and MSI


Countdown problem

What steps will reproduce the problem?

Download all needed plugins (updated to latest as well)
Start the server
Made an arena
Joined the game with few other friends

What were you expecting to happen? What happened instead?
The game to actually start, but the countdown continued counting down even in the negatives
What version of the plugin are you using? Type /version <Plugin Name>
Latest for 1.11.2
What Spigot version are you using? Type /version
1.11.2
What plugins are you using? Type /plugins
All of the dependency plugins (I am testing it on a test server with nothing but those plugins in it)
Do you have an error log? Use pastebin.com. If you're not sure, upload your whole server log
I do not have an error log
Did your client crash? Upload errors in .minecraft/logs/latest.log as well
It did not crash
Additional information?  (Are you using Bungeecord? Did it work in previous versions? etc.)
I am using Bungeecord and it did work like in the lobby and stuff, but when it was counting down, it continued to count down even when it went on the negatives.

Same problem here :c


Update KnnContainer.cs

Overcomplication

Oh sorry didn't see this at first! Yeah I think that's the result of some weird code porting, thanks for fixing that!


Test more cipher modes

[x] tests are added or updated

Add a test case that uses the evp cipher list to validate successful encryption and decryption of all cipher modes accepted by the enc command (i.e. not AEAD, XTS and wrappers).
The nearest existing test case to this is test_enc that uses the list -cipher-commands to get a list of ciphers to validate.  This omits about half of the available cipher modes.  This test, instead uses enc -ciphers for the list.
Unlike test_enc, this test makes no attempt to validate both binary and base64 encoded output.

I can understand why you named the test like you did.  Considering its close with relationship with test_enc, I would however prefer if it was in the same group, i.e. something like test/recipes/20-test_enc_more.t

Test script renamed to suggestion.

Test scripted updated to use the functionality from #2716.
The tests will not run properly until that is merged.

I just tested this with the #2716 commit on top.  Seems to run fine.

Okie, #2716 has been merged, it's time to rebase this.

Merged.  master: 227a44b1f639ea02d1a278ed07fa5f2dc6d286c6


Feature/use models

Start taking advantage of swagger models and types.  There should be many more changes like this at some later date.
This also uses the swagger DockstoreTool.ModeEnum.

Codecov Report

Merging #105 into develop will decrease coverage by 0.41%.
The diff coverage is 60%.

@@             Coverage Diff             @@
##           develop     #105      +/-   ##
===========================================
- Coverage    65.41%   64.99%   -0.42%
===========================================
  Files           93       93
  Lines         2478     2574      +96
  Branches       312      345      +33
===========================================
+ Hits          1621     1673      +52
- Misses         690      734      +44
  Partials       167      167

Impacted Files
Coverage Δ

src/app/shared/container.service.ts
90.24% <60%> (-9.76%)
:arrow_down:

src/app/search/search.service.ts
36.29% <0%> (+1.29%)
:arrow_up:

src/app/listentry/listentry.component.ts
50.76% <0%> (+2.76%)
:arrow_up:

src/app/search/search.component.ts
43.5% <0%> (+3.89%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 54101f9...a984a80. Read the comment docs.

FYI, definitely an improvement, but I think we should be switching gears to focus on testing for the next little while as opposed to this kind of improvement which can wait for 1.4


Automatic variable transformation with less changes

Adaptation of PR #75 requiring less changes in the model module.

Everything looks good and cleaner than my version, I just simplified make_transformed_distribution

@GianmarcoCallegher, I think 192ea292b76ee22bf60344bc8dc3c5b5ffd3ae31 is not equivalent to the old version: If the default event space bijector depends on the arguments of the TFP distribution (which is unlikely but not impossible), the old version used the bijector given the current model state, while the new version uses the bijector given the model state from when transform() was called...


DNM: testing swift-installer-scripts

Please test with following PRs:
apple/swift-installer-scripts#150
apple/swift-installer-scripts#151
@swift-ci please build toolchain Windows platform

Please test with following PRs:
apple/llvm-project#5792
https://github.com/apple/swift-installer-scripts/pull/150
https://github.com/apple/swift-installer-scripts/pull/151
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/llvm-project/pull/5792
https://github.com/apple/swift-installer-scripts/pull/150
https://github.com/apple/swift-installer-scripts/pull/151
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/llvm-project/pull/5792
https://github.com/apple/swift-installer-scripts/pull/150
https://github.com/apple/swift-installer-scripts/pull/151
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/llvm-project/pull/5792
https://github.com/apple/swift-installer-scripts/pull/150
https://github.com/apple/swift-installer-scripts/pull/151
https://github.com/apple/swift-installer-scripts/pull/152
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/152
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/152
https://github.com/apple/swift-package-manager/pull/5975
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/152
https://github.com/apple/swift-package-manager/pull/5975
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-package-manager/pull/5978
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-package-manager/pull/5980
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/163
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/159
@swift-ci please build toolchain Windows platform


Home Assistant Companion (Android, iOS app)

Hi,
You did a great job with this plugin, but currently tunnel with freenom domain cannot be used with android, ios app (companion) because you cannot entry freenom domain. Works great with ngrok url but not with freenom domain.
URL what phone app want is: http:// or https://homeassistant.myhome.com
I love your idea with domain instead of ngrok hard url. :-)
Thanks!

Hi,
I have the same issue and didn't find a solution yet.
Freenom URL forwarding forwards your domain and all subdomains, so you're reaching your instance through http://homeassistant.tk as well as through http://test.homeassistant.tk, but changing the external URL in the app doesn't solve the problem because the companion app doesn't like cloaking (the URL forwarding) and that's why can't display the webpage in-app. I'm also looking for a way to fix this.

Hi,
Thanks for response. Hope you will solve this (maybe with developer itself?).
Can you remove require fields for domain to don't use it? Freenom domain registration is difficult, because if you fill domain name as ondrashek and you want to choose domain as .tk you get Not available message. You need to fill domain name as ondrashek.tk to register a new domain.


chore: bdd-spring-1593138633 to 0.0.1

chore: Promote bdd-spring-1593138633 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Cannot load RAVEN on lemhi1

Thank you for the defect report

[X] I am using the latest version of RAVEN.
[X] I have read the Wiki.
[X] I have created a minimum, reproducible example
that demonstrates the defect.

Defect Description
Cannot load RAVEN on lemhi1

Steps to Reproduce
ssh'd into lemhi1 since sawtooth1 is down. Tried the command that is provided on the RAVEN wiki for hpc
module load pbs python MVAPICH2
and then got the error (see attachment)
Expected Behavior
Be able to run RAVEN on lemhi, and eventually run HERON tests cases on the HPC.
Screenshots and Input Files
No response
OS
Windows
OS Version
No response
Dependency Manager
CONDA
For Change Control Board: Issue Review

[x] Is it tagged with a type: defect or task?
[x] Is it tagged with a priority: critical, normal or minor?
[x] If it will impact requirements or requirements tests, is it tagged with requirements?
[x] If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure

[x] If the issue is a defect, is the defect fixed?
[x] If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

@shigrekar1 We have removed the requirement of pbs. Please let us know if you still have the issue.


Fix issue in --ip6 validation for docker create

- What I did
This fix tries to address the issue raised in comment:
https://github.com/docker/docker/pull/25943#discussion_r76843081
Previously, the validation for ip6 is done by checking ParseIP().To16(). However, in case an IPv4 address or an IPv4-mapped Ipv6 address has been provided, the validation will pass (should fail).
- How I did it
This fix first check if --ip6 is passed with a valid IP address and returns error for invalid IP addresses. It then check if an IPv4 or IPv4-mapped IPv6 address is passed, and return error accordingly.
- How to verify it
This fix adds two more cases in the tests. One for IPv4 address passed to --ip6 and another for IPv4-mapped IPv6 address passed to --ip6. In both cases, without this fix the validation will pass through.
cc @aboch please take a look.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

LGTM 🐸
/cc @aboch @mavenugo @mrjana

looks good to me

LGTM


Fix EmptyMetadata for splat kwargs

Fixes #2004, when metadata is detected as empty when it's actually a kwargs splat.
cc @tylerhunt

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[-] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

Thanks, @pirj!


Add ARM compatibility

This PR does some refactoring to add ARM compatibility to the library.
First, I added a PREFETCH Macro that maps to the correct prefetch function for the architecture used for compilation. For x86 and x64, it will map to _mm_prefetch. For aarch64, it will use gcc's __builtin_prefetch. For this, I was following a recommendation in this post. I think it would be possible to embed assembly, but this seems like it may get a little complex. So, for a first version, I just used __builtin_prefetch, but can change if you think another implementation may be better. If the architecture is neither x86 nor aarch64, PREFETCH will default to a no-op. After making this change, I was able to compile and run ./release/bunit.
Second, I refactored similarity_search/include/method/hnsw_distfunc_opt_impl_inline.h. Instead of having preprocessor directives inside the functions, I created an implementation of L2Sqr16Ext, L2SqrExt, ScalarProduct for PORTABLE_SSE2, PORTABLE_AVX, PORTABLE_NEON, and a default one for architectures without SIMD support.
Third, I added NEON intrinisics to the PORTABLE_NEON functions. For this, I referenced the ARM Intrinsics guide and faiss's ARM support.
Fourth, I changed a few naming conventions in similarity_search/include/method/hnsw_distfunc_opt_impl_inline.h in order to improve readibility with SIMD instructions (at least from my own beginner's perspective).
I ran ./release/bunit on both a m5.4xlarge instance for x86 and m6g.4xlarge instance for ARM. Both passed.

Awesome, I am gonna look into this soon.

Hi @jmazanec15 many thanks! One thing about PREFETCH though. On Windows, it needs to be _mm_prefetch I think.

PS: @jmazanec15 we also seem to have the issue with includes under Windows: https://github.com/nmslib/nmslib/issues/399
VCPKG in fact, includes, a patch: https://github.com/microsoft/vcpkg/blob/256d8a497a77f000168fe5a5b7c90a1962fa76ff/ports/nmslib/fix-headers.patch

@searchivarius I added a "portable_prefetch" header file, and included <intrin.h> for Microsoft as specified here: https://docs.microsoft.com/en-us/cpp/intrinsics/x86-intrinsics-list?view=msvc-160. I will test on a Windows machine.
In regards to https://github.com/microsoft/vcpkg/blob/256d8a497a77f000168fe5a5b7c90a1962fa76ff/ports/nmslib/fix-headers.patch, I believe this should be solved by the code change. I removed #include <sys/time.h> from test_overlap.cc as well.


Add support for params generation

This changes the top level cli to have “index” and “params” as the two main commands to either generate or dump

according to the gha docs the last couple versions of ubuntu runners have all included jq in the pre-installed tools so a cheap json input file validation we could add to the new human-readable json params files could be as simple as
for file in $params_files
do
    cat $file | jq empty
done

which just parses the file and then applies an empty filter to it (which is a no-op) but the file command will blow up with a line and character number if there are any issues parsing the file


register fail

skywalking 6.1
skyAPM-php-sdk
git clone last version 20190520
php -i
skywalking Support => disabled (register fail)

@goerzh

Retry last release


add clipboard on about page

now versions are clickable and will add the version to the clipboard, for easier copy and pasting when making issues

I think there's much better way to make text copiable, if I even aim for that… I don't think actually to be there any reason, unless you need to save these values outside of the GitHub issues, as there's a Report a bug menu option which actually generates a pre-filled issue with the template which could be further edited for any reason (like if you actually don't want to report it or sth).

I never knew about the report a bug menu option :skull:, accustomed of going directly to GitHub.
Can you change the template to mention the button then?

I think there's much better way to make text copiable

How? execCommand? I still like the idea of using a class so you can make anything "copiable"

Um… user-select in CSS (or it's -webkit equivalent)? At least it's much more simple IMO.
I've personally made it non-copyable by the design. After all, text in native apps is usually non-copyable by default when it isn't meant to be.
I guess I could use that with some kind of selector if really needed, to make it possible to select the text and give a user an option to copy it whole or part of it…
I could implement this on my own… There's no need to do a PR for such small change to be done :wink:.


Polish ProblemDetails

Add ability to set extended members on ProblemDetails
Skip empty valued properties when serializing ProblemDetails

Fixes #8296
Fixes #8317

Check in with @rynowak about the Newtonsoft.Json dependency before merging

@JamesNK spoke to @rynowak and @glennc about it

If you want to use Newtonsoft.Json here, consider explicitly referencing Newtonsoft.Json in the Mvc.Core the project with the ASP.NET version.

Considering we're only using it get to attributes (and not executing code), do you think there's value in adding an explicit reference?

A version of Json.NET eventually needs to do the serialization. If the app only references Mvc.Core, and no other package references Newtonsoft.Json then 9.0.1 will be the version of the library included with the website and 9.0.1 will be used to serialize the type.
Now if a dev raises an issue with serializing this type you will need to consider whether Newtonsoft.Json 9.0.1 was being used, or whether it is 11.0.1(2?) that every other part of ASP.NET uses.

If the app only references Mvc.Core, and no other package references Newtonsoft.Json then 9.0.1 will be the version of the library included with the website and 9.0.1 will be used to serialize the type.

That's kinda already a problem today (regardless of the type). However, we wouldn't use the 9.0.1 version with our formatters would use the correct version which would be the scenario we primarily care about.


Martin H. Contributing Guidelines

Besides the normal ReadMe, a Contributing Guideline should provide a better introduction for new contributors.

notes for creating the contribution guide:
https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/setting-guidelines-for-repository-contributors
https://gist.github.com/briandk/3d2e8b3ec8daf5a27a62

@evilru
i created a first version for our cla assistant #62
https://github.com/1DSAG/UI5-Best-Practice/blob/main/contributing.md
feel free to edit this guideline


suggestion for docs: collapse markdown link URLs

I didn't see this in the documentation, but the extension is very useful for collapsing markdown link URLs:
Regex: \[.*\]\((.*)\)
Regex Flags: g
Then links will look like this:

Hi @tecunningham
Users are allowed to document where the extension can help in different cases.
I will be happy to merge any use case you can suggest such this one 👍 it looks really useful

OK cool I will play around with it a bit more. The one downside is that when I have word-wrap on, and the URL is long, then the blank lines get rendered. I think there's no workaround for that right?

@tecunningham Yeah we noticed that, unfortunately the only workaround is to disable the word-wrap 😑.
Regarding any code formatting where it will actually break lines, I've thought of hooking a code editor with the extension to force some prettifying rules. But that will be an overhead since it must be language aware.
I must add limitations note in the home page until VS Code support horizontal folding.

@moalamri , can we have support for regex per language as well ?, instead of one regex fits all solution

@tecunningham added #75

Note that this is an improvement: it makes the regexp non-greedy, so will match the first closing parantheses (")") that it finds:
"inlineFold.regex": "\[.\]\(([^\)])\)",
E.g. suppose you have this:
[one](one) (asdfjkl)

Then the previous regexp will fold everything up to the last parantheses.

Hey @tecunningham
Urls can contain parentheses as well, I guess being greedy is the better option. That being said we can tweak the Regex to consider the ending valid if is ")" followed by a space "\s", that way we can at least narrow down mismatches?

Oh good point. OK working on an improved regexp, and I think I've solved the "multiple matches on one line" issue too. Will update later today.

OK I suggest this as new recommendation for markdown links:
   "inlineFold.regexGroup": "2",
   "inlineFold.regex": "\\[([\\w\\s\\d]+)\\]\\(([^\\s]+)\\)[,.!\\?]*\\s",
   "inlineFold.regexFlags": "g",

Here are a bunch of test cases:
**Should fold:**

- [filename link](../abc.md)

- [filename link with parantheses](../Twitter/Twitter_Private%202022-10-29,%2008_53%20PM%20(22%20items).csv)

- [HTTP link](http://asdf.com)

- [HTTPS link](https://en.m.wikipedia.org/wiki/Wikipedia:Wikipedia_Signpost/2022-10-31/News_and_notes#abc)

- [HTTP link with parentheses](http://asdfj.com/asdf(asdf))

- [HTTP link with punctuation afterwards](http://google.com)...!

- Two HTTP links on one line: [one](http://google.com/), [two](http://google.com/)

- Three HTTP links on one line: [one](http://google.com/), [two](http://google.com/), [three](http://google.com/)

- Four HTTP links: [one](http://google.com/) [two](http://google.com/) [three](http://google.com/) [four](http://google.com/)

- Four relative links: [one](one) [two](two) [three](three) [four](four)

**Should not fold:**

- [HTTP with space](http://asdjfkl .com)

- [link with text straight afterwards](http://google.com/)asdf

Awesome 👍

... I must add limitations note in the home page until VS Code support horizontal folding.

For those interested (as I was) this required feature is tracked in microsoft/vscode#50840.
Thanks @moalamri for working around this limitation as far as possible!


Ready Check (Request)

A feature requested by one of my clan members - a ready check system.
Simply put, someone uses a command to start a ready check and everyone else has to respond either in the affirmative or negative. I've attached a screenshot (set up using .acr's) to better demonstrate what I mean.
If this could be limited to only the people in the voice channel of the person who started the check and with an optional variable for a timer, that would be awesome.
Not sure how doable this is, or how sought after it is, but it would be nice. :)

Too specific, i don't see a wide use for it.


fix release workflow - take 2

We have to set persist-credentials: false in the GitHub Action workflow.
Needed to allow Semantics Release to manage auth.
But this prevents the action from having permission to do a push.
Hence we will get Semantic Release to do it.

This PR was included in release version 1.1.0-beta.2 :tada:

This PR was included in release version 1.1.0 :tada:


[incubator/fluentd-cloudwatch] Be able to specify busybox image registry

Currently the daemonset attempts to pull a busybox image from dockerhub - this can be an issue if we are deploying this chart in worker nodes that have no egress access to the internet or rely completely on private registries, such as AWS ECR, due to compliance or security issue.
It would be nice to be able to specify the repo for the busybox image in the values file.

#15353 would fix this


Bump elixir-ls to 0.14.0

Also: Interpolate the VERSION into the URL; renamed SERVER_HASH to SERVER_SHA256.

Please wait with the merge. The server doesn't launch and it seems to crash.
Sublime Console:
LSP: lsp-elixir crashed (1 / 5 times in the last 180.0 seconds), exit code 1, exception: None
...
LSP: lsp-elixir crashed (5 / 5 times in the last 180.0 seconds), exit code 1, exception: None

I don't know the internals but I tried to start manually from a project folder with a mix.exs:
'~/.cache/sublime-text/Package Storage/LSP-elixir/server/0.14.0/language_server.sh'

Content-Length: 655

{"jsonrpc":"2.0","method":"window/logMessage","params":{"message":"** (exit) exited in: GenServer.call(Mix.State, {:update, {Map, :put, [:env, :dev]}}, :infinity)\n    ** (EXIT) no process: the process is not alive or there's no process currently associated with the given name, possibly because its application isn't started\n    (elixir 1.13.4) lib/gen_server.ex:1019: GenServer.call/3\n    lib/launch.ex:7: ElixirLS.Utils.Launch.start_mix/0\n    lib/language_server/cli.ex:23: ElixirLS.LanguageServer.CLI.main/0\n    (stdlib <IP_ADDRESS>) erl_eval.erl:685: :erl_eval.do_apply/6\n    (elixir 1.13.4) lib/code.ex:404: Code.validated_eval_string/3","type":2}}

** (exit) exited in: GenServer.call(Mix.State, {:update, {Map, :put, [:env, :dev]}}, :infinity)
    ** (EXIT) no process: the process is not alive or there's no process currently associated with the given name, possibly because its application isn't started
    (elixir 1.13.4) lib/gen_server.ex:1019: GenServer.call/3
    lib/launch.ex:7: ElixirLS.Utils.Launch.start_mix/0
    lib/language_server/cli.ex:23: ElixirLS.LanguageServer.CLI.main/0
    (stdlib <IP_ADDRESS>) erl_eval.erl:685: :erl_eval.do_apply/6
    (elixir 1.13.4) lib/code.ex:404: Code.validated_eval_string/3

Works again now! :+1:

💜


MSVC Complie Error

In VS2022
it's Release version can't be complied correctly
There is a very strange error, only in the line that has 'log2', check it please

bug fixed


Setup is not one step

Should have a simple way for setup of the environment. Perhaps a Grunt task?

It's been partially addressed in the merge commit of f0c6821, however I feel like until the configuration issues get solved this issue really isn't solved.


Shadow artifact

Noticed a strange artifact in both Editor and Launcher (Win 11, Chrome)
https://user-images.githubusercontent.com/5677782/155851344-a68c51a6-bdfd-4b62-ab97-e0146b349168.mp4

Try adjusting the bias on the light - it makes it go away.

We adjusted it a while back .. but little bit is still left there .. it depends on geometry mostly, and as its mostly only visible on the sphere, its not considered a priority. Too much bias creates other issues so this is acceptable for now I think.


fix: suffix version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.542%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
927

Relevant Lines:
927

💛  - Coveralls


LSP notifications UI

@nheuillet do you think displaying showMessage is a good enough MVP?

@bugadani yes I think displaying it nicely would be great already

nicely

That's not me, probably :D But I'll do my best


Add metadata endpoint for healthcheck purposes

I don't see this endpoint documented anywhere in the Confluent Schema Registry API documentation. Do you have a reference?

I don't see this endpoint documented anywhere in the Confluent Schema Registry API documentation. Do you have a reference?

Took it from https://github.com/confluentinc/schema-registry#openapi-spec

Took it from https://github.com/confluentinc/schema-registry#openapi-spec

Such endpoint in Official java client https://github.com/confluentinc/schema-registry/blob/7258349ebe11d5d283faf8e06aff287907ed8b6f/client/src/main/java/io/confluent/kafka/schemaregistry/client/rest/RestService.java#L970

Such endpoint in Official java client https://github.com/confluentinc/schema-registry/blob/7258349ebe11d5d283faf8e06aff287907ed8b6f/client/src/main/java/io/confluent/kafka/schemaregistry/client/rest/RestService.java#L970

Also I have this https://gist.github.com/Sujimoshi/79fa52d93df6de2c42d740b222c36cc9
This is TypeScript client generated with swagger-codegen from OpenAPI spec of schema registry

Also I have this https://gist.github.com/Sujimoshi/79fa52d93df6de2c42d740b222c36cc9
This is TypeScript client generated with swagger-codegen from OpenAPI spec of schema registry

This one works only with confluentinc/cp-schema-registry:6.0.0

This one works only with confluentinc/cp-schema-registry:6.0.0

I'm not sure that we should introduce this. My reasoning is that, at least the way I see it, this library is not a generic Confluent Schema Registry API client. If it were, it wouldn't have the ability to encode/decode messages - just fetching/registering schemas. Getting cluster metadata and other stuff doesn't seem to belong, and it becomes unclear to me what the scope is.
@erikengervall: You're the main developer of this library, what do you think?

I'm not sure that we should introduce this. My reasoning is that, at least the way I see it, this library is not a generic Confluent Schema Registry API client. If it were, it wouldn't have the ability to encode/decode messages - just fetching/registering schemas. Getting cluster metadata and other stuff doesn't seem to belong, and it becomes unclear to me what the scope is.
@erikengervall: You're the main developer of this library, what do you think?

I'm not sure that we should introduce this. My reasoning is that, at least the way I see it, this library is not a generic Confluent Schema Registry API client. If it were, it wouldn't have the ability to encode/decode messages - just fetching/registering schemas. Getting cluster metadata and other stuff doesn't seem to belong, and it becomes unclear to me what the scope is.
@erikengervall: You're the main developer of this library, what do you think?

I agree that it isn't exactly within the scope of this library, I'd recommend setting up health checking independently from this library.

Thanks for your initiative @Sujimoshi, but I'm going to close this PR as a wontfix based on the discussion between myself and Erik. If you want to health check the schema registry, our recommendation is to do that outside of this library.


Holding page is all squishy

Set browser height to 720px
Visit https://belowtheline.org.au
SQUISHY

Everyone's a critic.

It's fixable. Is the source for this page in this repo?

https://github.com/belowtheline/btl2016/blob/master/app/assets/index.html


Remove use of FnvHashMap/Set for node IDs, index IDs, queue IDs, client IDs

It's only good for really short strings (< 20 bytes), but our strings are not that short:

node IDs are not that short:quickwit-indexer-prod-us-east-1a
index UIDs are not that short: logline.c06d8f1aee.2024-02-01.10:01HNGZE70MS3YBV8FVSMVTA910
queue IDs are not that short: <index_uid>/<source_id>/<shard_uid>

Let's revert to the default HashMap/Set.

This is not a bug, so I removed the tag.
Also this bench suggests the limit where HashMap overperforms FnvHashMap is at around 60bytes.
https://cglab.ca/~abeinges/blah/hash-rs/
I'd rather stick to FnvHashMap for the moment.


onChangeAction event is not triggered when multiple beacons are nearby.

onEnterAction is triggered when two beacons are nearby. but only one beacon data is passed as an event object . I went through docs and found out that onChangeAction  has data about multiple beacons in the range. but it doesn't trigger at all. I have a use case where i need to display multiple beaons list. how can i achieve this? is there any other alternative to onChangeAction?

Are you sure that both beacons are tagged appropriately? Last time I was testing, I believe I saw the onChangeAction trigger properly. I'll re-test when I have a moment.

This is affecting our release schedule. Please look into this as soon as possible . Thanks.

I can confirm that it works for me:
Running application "example" with appParams: {"rootTag":1}. __DEV__ === true, development-level warning are ON, performance optimizations are OFF
location permission: always
'zone1 onEnter', { tag: 'proximity-one-cet',
  attachments:
   { 'proximity-one-lkn/title': 'blueberry',
     'proximity-one-cet/title': 'blueberry',
     'proximityone-ci2/title': 'blueberry',
     'proximity-bxf/title': 'blueberry' },
  deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' }
'zone1 onChange', [ { tag: 'proximity-one-cet',
    attachments:
     { 'proximity-one-lkn/title': 'blueberry',
       'proximity-one-cet/title': 'blueberry',
       'proximityone-ci2/title': 'blueberry',
       'proximity-bxf/title': 'blueberry' },
    deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' } ]
'zone1 onChange', [ { tag: 'proximity-one-cet',
    attachments:
     { 'proximity-one-lkn/title': 'blueberry',
       'proximity-one-cet/title': 'blueberry',
       'proximityone-ci2/title': 'blueberry',
       'proximity-bxf/title': 'blueberry' },
    deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' },
  { tag: 'proximity-one-cet',
    attachments: {},
    deviceIdentifier: 'fd10f449c05ee38c26edc723bb502e2a' } ]

You can see first both onEnter and onChange triggered for 7d3a2f6c, then followed by onChange with both 7d3a2f6c and fd10f449.
I tested with the latest example app, on Android.
Maybe double-check your onChangeAction, your zones setup (tags/range), and the Estimote Cloud tags?

each beacon is detected individually on onEnter but onChangeAction never gets triggered. How can i help you to replicate the issue? how to get the logs like you did?

Can you show me your code, and write down your tags setup? (i.e., which beacon has what tags)

This is my beacon monitoring code
const beaconList = [
            {
                bec_id: 1001,
                bec_uuid: <BEACON_IDENTIFIER>,
                str_id: 4005,
                bec_name: 'mint',
                loc_id: 2001,
                id: 25,
                tag: 'store1'
            },
            {
                bec_id: 1002,
                bec_uuid:<BEACON_IDENTIFIER>,
                str_id: 1002,
                bec_name: 'blueberry',
                loc_id: 2010,
                id: 26,
                tag: 'store11'
            },
            {
                bec_id: 1003,
                bec_uuid:<BEACON_IDENTIFIER>,
                str_id: 4006,
                bec_name: 'ice',
                loc_id: 2005,
                id: 27,
                tag: 'store3'
            },
            {
                bec_id: 1004,
                bec_uuid:<BEACON_IDENTIFIER>,
                str_id: 4006,
                bec_name: 'coconut',
                loc_id: 2004,
                id: 28,
                tag: 'store2'
            }
        ];
        if (beaconList.length) {
            RNEP.locationPermission.request().then(
                permission => {
                    if (permission !== RNEP.locationPermission.DENIED) {
                        const credentials = new RNEP.CloudCredentials(ESTIMOTE_APP_ID, ESTIMOTE_APP_TOKEN);
                        const config = {
                            notification: {
                                title: 'Exploration mode is on',
                                text: "We'll notify you when you're next to something interesting.",
                                channel: {
                                    id: 'exploration-mode',
                                    name: 'Exploration Mode'
                                }
                            }
                        };
                        var zones = [];
                        Reactotron.log({ beaconList });
                        for (var i = 0; i < beaconList.length; i++) {
                            var zone = new RNEP.ProximityZone(5, beaconList[i].tag);
                            zone.onEnterAction = this.onZoneEntered;
                            zone.onExitAction = this.onZoneExit;
                            zone.onChangeAction = this.onZoneChanged;
                            zones.push(zone);
                        }
                        RNEP.proximityObserver.initialize(credentials, config);
                        if (zones.length) {
                            RNEP.proximityObserver.startObservingZones(zones);
                        }
                    }
                },
                error => {
                    Alert.alert('Error', JSON.stringify(error));
                    console.error('Error when trying to obtain location permission', error);
                }
            );
        }

    onZoneEntered = context => {
        Alert.alert(`beacon Zone Entered`, JSON.stringify(context));
        console.log('zone onEnter', context);
    };

    onZoneExit = context => {
        console.log('zone onExit', context);
    };

    onZoneChanged = contexts => {
        console.log('zone onChange', contexts);
        Alert.alert('zoneChanged', JSON.stringify(contexts));
    };

Thanks! So you have one beacon per each zone, is that right? In this case, you should only ever see onEnter and onChange trigger together (when you enter range of that one beacon), and onExit and onChange trigger together too (when you exit range of that one beacon). You'll never see onChange trigger on its own, because that only makes sense for a zone with multiple beacons, whereas you have zones each with a single beacon.
Does that make sense now? Is the behavior I've described what you're seeing?

how to get the logs like you did?

Sorry, I missed that question earlier. My logs were simply output from the console, you can see that via react-native log-android or via Chrome/React Developer Tools (https://facebook.github.io/react-native/docs/debugging#chrome-developer-tools).

No I do not have one beacon per zone. I placed two beacons nearby my phone. and opened the app, only onEnter is triggered.

Your beaconList has exactly 4 beacons, and each of them has different tag. Are there any other beacons that are on not on the beaconList, but which share tags with these 4 beacons?
In any case, this looks like a problem with your setup and/or understanding how Proximity Zones work, and the plugin itself works as intended, so I'll close this bug report. This is essentially a Proximity SDK conversation, and the react-native-proximity is just a bridge between the Proximity SDK and React Native, so it'd be better to continue this conversation on our forums (https://forums.estimote.com), in the Proximity category.

I just need to know one thing. two beacons with tags store1 and store2 respectively. I place those beacons near my phone. i open the app. How am i supposed to know there are multiple beacons in range when only onEnter is triggered.?

You should see two onEnter events. One for the "store1" zone, and one for the "store2" zone.

Ok thank you

Can you suggest a better approach for my use case? if the phone is inside range of multiple beacons we need to display those list of beacon data. since onEnter has data of only one beacon. how can i achieve it?

Assign the same tag to all the beacons (e.g., "store", instead of "store1", "store2", etc.), and create just one Proximity Zone for that tag.
Then the onChange handler will be called every time the list of "store" beacons currently in range changes.

You have said that Then the onChange handler will be called every time the list of "store" beacons currently in range changes..
onChange will be called only when beacon range changes ?. Doesn't it trigger initially with data of all beacons in range?

Perhaps it's best we look at the logs I posted previously to understand this better: (I've reformatted them slightly for better readability)
'zone1 onEnter', {
  tag: 'proximity-one-cet',
  attachments: { ... },
  deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' }
'zone1 onChange', [ {
    tag: 'proximity-one-cet',
    attachments: { ... },
    deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' } ]

'zone1 onChange', [ {
    tag: 'proximity-one-cet',
    attachments: { ... },
    deviceIdentifier: '7d3a2f6ce0eaeffc05f939c5a8b4943a' },
  { tag: 'proximity-one-cet',
    attachments: {},
    deviceIdentifier: 'fd10f449c05ee38c26edc723bb502e2a' } ]

Here you can see I have two beacons both tagged as "proximity-one-cet".
First, I entered range of 7d3a, which triggered both onEnter, and onChange with an array with 7d3a.
Then, I entered range of fd10, which triggered onChange with array with both 7d3a and fd10.
Say, if I were now to exit range of 7d3a, but remain in range of fd10, there would be onChange with an array with fd10.
Finally, if I were now to exit range of fd10, there would be onChange with an empty array, and an onExit with fd10.

I get what you are saying. But Please consider this scenario where the app starts scanning on click of a button. Your two beacons 7d3a  and fd10 are placed close together and they have the same tag. you go near them and click the button to start monitoring. my question is will the onChange event get triggered with array of both beacon data?

Since Bluetooth scanning is sequential (radio discovers packets one by one), first you'll get an onChange event with one beacon (whichever the radio managed to scan first), and almost immediately after (specifically, after the radio scans the second beacon) you'll get onChange with both beacons.

can you tell me approx the time taken for subsequent scans?
so that i can wait for multiple beacons detection and complete my functionality.
Does the distance between the beacons have an impact in time delay?

The way this works is, the Bluetooth stack in the smartphone will periodically start scanning, and during that time, it should detect any Bluetooth devices that broadcast nearby. Beacons in Estimote Monitoring broadcast every 200 or 300 ms, so if the smartphone is listening and processing this data, it should take < 200/300 ms between beacon detections.
In practice this is more complicated, because the Bluetooth stack in smartphones doesn't necessarily scan all the time, it also needs to share time/resources between other apps, and things like Bluetooth headphones, etc.
Distance is generally negligible, since Bluetooth radio waves travel at the speed of light. Distance however does increase the error rates, i.e., the probability that the signal/data will get corrupted.


Nsf core fw impl

NSF multi process framework implementation.

Thank you for contributing to openstack/group-based-policy!
openstack/group-based-policy uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Toggle fullscreen on pinch to zoom

This patch makes webtorrent to do fullscreen (maximize/unmaximize) when pinch to zoom event (two finger zoom in/out gesture) happened on macOS instead probably less meaningful zoom. Have a look here for more info about the approach.
To test this, open webtorrent on mac and do two finger zoom on webtorrent window with and without this patch. The intended gesture matches VLC on pinch to zoom action.
I hope you find this useful. Thanks!

Thanks, looks good!


Web zlib.encode

zlib.encode does not working in web application
import 'dart:io';
import 'dart:convert';

void main() {

  var str = "123";

 print(utf8.decode(zlib.decode(zlib.encode(utf8.encode(str)))));

}

Exception: Uncaught (in promise) Error: Unsupported operation: _newZLibDeflateFilter
C:\Flutter\flutter_zlib>flutter doctor -v
[√] Flutter (Channel beta, 1.18.0-11.1.pre, on Microsoft Windows [Version 10.0.18363.836], locale ru-RU)
    • Flutter version 1.18.0-11.1.pre at d:\android\flutter
    • Framework revision 2738a1148b (3 weeks ago), 2020-05-13 15:24:36 -0700
    • Engine revision ef9215ceb2
    • Dart version 2.9.0 (build 2.9.0-8.2.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at d:\android\sdk\
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = d:\android\sdk\
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 46.0.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] Connected device (2 available)
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 83.0.4103.97

• No issues found!

Hi @NickNevzorov
at the moment dart:io   supports only DartVM AND Flutter on mobile (Android and iOS)
Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments, possibly providing a minimal reproducible code sample that does not use 3rd party plugins, and I will reopen it.
Thank you

Hi, then, how can I use zlib in flutter web? Thanks!

Same question here, is there any alternative for Zlib in Flutter Web?

I finally use the archive package.


document how to create a "list of references" field type

document how to create a "list of references" field type.
example markdown frontmatter:
---
authors:
  - people/me.yml
  - people/you.yml
---

what i tried:
import { defineDocumentType, makeSource } from "contentlayer/source-files";

const Person = defineDocumentType(() => {
  return {
    name: "Person",
    contentType: "data",
    filePathPattern: "persons/**/*.yml",
    fields: {
      name: {
        type: "string",
        required: true,
      },
    },
  };
});

const Post = defineDocumentType(() => {
  return {
    name: "Post",
    contentType: "mdx",
    filePathPattern: "posts/**/*.mdx",
    fields: {
      title: {
        type: "string",
        required: true,
      },
      authors: {
        type: "list",
        of: { type: "reference", of: Person },
        required: true,
      },
    },
  };
});

export default makeSource({
  contentDirPath: "content",
  documentTypes: [Person, Post],
});

this throws: "Error: A case was not handled for value: {"type":"reference","of":{"type":"document"}}"

Same issue for me

Thanks a lot for opening this issue and creating a reproduction. I've just looked into it and I believe this should already work using the following code:
const Post = defineDocumentType(() => {
  return {
    name: "Post",
    contentType: "mdx",
    filePathPattern: "posts/**/*.mdx",
    fields: {
      title: {
        type: "string",
        required: true,
      },
      authors: {
        type: "list",
//        of: { type: "reference", of: Person },
        of: Person,
        required: true,
      },
    },
  };
});

I guess you've landed on the code snippet you shared above based on the current docs (see below)?

(P.S. Cases of "wrong configuration" like these should be caught by #214)


Implement Error.Is() method

Go 1.13 introduced wrapped errors, as well as errors.Is(x, y), which will automatically unwrap errors and either do a comparison, or check to see if an error's Is() method returns true.  This would basically be:
func (err Error) Is(target Error) bool {
		switch v := target.(type) {
		case ErrNo:
			return err.Code == v
		case ErrNoExtended:
			return err.ExtendedCode == v
		}
		return false
}

Then errors.Is(err, sqlite3.ErrConstraint) should work as expected.
I can send a PR in a couple of weeks if nobody objects and/or nobody gets around to it.

Seems reasonable. Perhaps we should add a case for syscall.Errno as well?

For syscall.Errno, it seems like having an Unwrap() method which would return that value if present would be better.  That way any code that relied on unwrapping would work, even if it didn't know anything about sqlite3.Error.  I was going to include a patch to do something like that in my series (if I got to it).

Not sure I follow. Wouldn't you want to be able to do something like errors.Is(err, syscall.ENOENT)?
I don't think implementing the Unwrap method would be straightforward, since there are three candidates that could all be considered the underlying error - err.Code, err.ExtendedCode and err.SystemErrno. Instead, we should probably implement As in a similar manner to Is.


Documentation: Drawer Natigation Example Error

Describe the bug
From the Documentation, the example Drawer Natigation displays an error on the website itself:
https://docs.nativebase.io/building-drawer-navigation
To Reproduce
Steps to reproduce the behaviour:

Go to https://docs.nativebase.io/building-drawer-navigation
Click on Example
Scroll down to Example
See error

Expected behaviour
To see the example.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform

NativeBase version: Last

This issue is related to https://github.com/GeekyAnts/nativebase-docs closing it here and creating the issue on nativebase-docs Repo. Please follow this https://github.com/GeekyAnts/nativebase-docs/issues/132.


Linking against shared library

Perhaps this is out of scope for this "hello world " nanobind repo, but a use case I am struggling with that is not that unconventional is that I have an external library (GEOS in this case) that I HAVE to link against dynamically due its license (LGPL v 2.1). When I was linking statically everything was peachy but now I'm in DLL hell with editable installation (which is key for quick development).  The @rpath doesn't seem to be getting set properly and it is not entirely clear to me how I should be handling this. I've tried a number of incantations of CMAKE rpath messing about but so far I still can't load both my nanobind main repo and the dependent library.

These things are just hard, period. But they aren't really related to nanobind. I will close the ticket because I lack the time to provide this level of support. If you wish to contribute documentation yourself, then feel free to open a PR.

Fair enough


Jio bug 1460051

Closes-bug: https://bugs.launchpad.net/jio/+bug/1460051

I think this should be part of puppet-rjil, the code going in puppet-neutron should be reusable everywhere, so any rjil/jiocloud specific code should be in puppet-rjil.

@hkumarmk Patch moved to https://github.com/JioCloud/puppet-rjil/pull/625

@sajuptpm so we can close this PR right?

Yes, we can close it.


Publish 7.6-20220822144040+0000

Publish 7.6-20220822144040+0000

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build cancelled.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


[DOC] Update README with libseccomp installation instructions

Looks good to me, thanks.

LGTM

I take that back, can you make this Pull Request to the develop branch instead of master? Thanks :)


Win10Home 10.0.19013.vb

termsrv-win10home-10.0.19013.1.zip

mine is 18950, I'm tried to find the offsets, followed the operation in stas'M's video uploaded 5 years ago. But " not listening". Not really understanding what exactly is going on, I give up and wait for another stable version, updated with windows.

Forget what I just said, I figured it out. Not tested ,here is your offsets:
[10.0.19013.1]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=89FE1
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=10624
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=20C95
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=28EBC
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19013.1-SLInit]
bInitialized.x64      =105DF8
bServerSku.x64        =105DFC
lMaxUserSessions.x64  =105E00
bAppServerAllowed.x64 =105E08
bRemoteConnAllowed.x64=105E10
bMultimonAllowed.x64  =105E14
ulMaxDebugSessions.x64=105E18
bFUSEnabled.x64       =105E1C
by the way, I share my version:
[10.0.18950.1000]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=88461
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=197C4
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=20F05
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=25BCC
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.18950.1000-SLInit]
bInitialized.x64      =102DB0
bServerSku.x64        =102DB4
lMaxUserSessions.x64  =102DB8
bAppServerAllowed.x64 =102DC0
bRemoteConnAllowed.x64=102DC8
bMultimonAllowed.x64  =102DCC
ulMaxDebugSessions.x64=102DD0
bFUSEnabled.x64       =102DD4


Improvement: create an index on created_at

As most read queries will be in the form of
select * from `activity_log` order by `created_at` desc limit 50

creating an index on the "created_at" field can speed things up

I'll add the index in the next major version of this package which will be released soon.


W2ui 1.4

I see only one commit that is uniqu in this request: https://github.com/vitmalina/w2ui/commit/273d69ee2e63131c1dbc099ef37a763b59cf00aa and this has already bin fixed in the branch. Please pull all updates from master branch before submitting pull requests.


Add documentation for require function to associative arrays

Documentation for https://github.com/dlang/druntime/pull/2162

I recommend that the documentation for both of these functions mentions the fact that they avoid the need for multiple lookups.  IMO, that will often be the deciding factor when a user chooses to use these functions.


Activate site generation and deploy to gh-pages

Site gen during release is sort of important.

Work has begun and seems to be going OK.
Currently, it requires special settings.xml setup.

Works OK for standard items, but multi-module builds are a nightmare.
I think using the properties for <url> and <site> stuff might be a mistake when using github pages.


GetUpdates failure on Restricted chatMember

No value found for 'Restricted'
com.bot4s.telegram.models.MemberStatus doesn't have "Restricted" MemberStatus. It was added in 5.3:
https://core.telegram.org/bots/api#chatmember

Thanks for reporting that, I have created a fix, do you have a quick Snippet I could use to test this behavior ? I don't have a super group ready to tests that right now.
Otherwise could you try to use a snaptshot version ?

Sorry, I cannot check how code is working. My bot tried to get update, failed each time and tried again many times. I fixed this by writing primitive bot in python, which consumed this update. After that scala bot is working fine.

Ok, thanks for reporting that, I've tested it and it should not be an issue anymore. I'll do a release this weekend


Feature Request: Abstract Search API

I'd like to swap out Tavily with my own search API. If you're interested, I can make abstract my work and contribute it so it's easier to add additional search backends.

To implement this, I would

Rename search_tavily to search_configured_client.
Add an environment variable SEARCH_BACKEND; it would default to tavily.
Write an if-else block in search_configured_client that would route to a specific search backend

(this is very simple, may be subject to change.)

Any additional search backends would be configurable here.
Ideally, I'd implement a dynamic plugin approach; any python module reference (e.g. my_third_party.search:search_method) that returns a SearchResponse could be use.

This is great, I definitely want to support this. What other search providers are you working with?
Feel free to make a PR!

Hey, I just implemented an abstract search API. Farfalle now supports both tavily and searxng. Let me know if there are other search providers you'd like me to add support for.

Awesome! Searx was the one I would've suggested. Nice work


How to use order_by with a jsonfield.

class Test(models.Model):
rating = JSONField()
for example, rating is like this {'a':1, 'b':2}
I tried Test.objects.order_by("rating__a"), it didn't work.
It said "Join on field 'rating' not permitted. Did you misspell 'a' for the lookup type?"

interested in this same functionality. any updates here?


[✨FEAT✨]: Cone mapping

Is there an existing enhancement request for this?

[X] I have searched the existing feature requests

Description
Map of all cones detected
Additional Information
No response

Plan:
1: Assuming perfect sensor and assume that the cone detection team provides the distances and angle from the front for all of the cones. Complete the mapping project. (Due date likely June)
2: Improve the existing mapping algorithm with perfect sensor by considering the sensor imperfectness (kalman filter or other thing)
(Due date likely July)

For the plan I would like somthing more spesific and a plan of actions an example might be :
We are going to first make 3 test data set by hand then we will try to see if x algo will be effective and then compare it to y, we are going to do this by making a node called z that is in package a yada yada
this is a good basic outline but a plan should be a plan of action so that the team and you guys know what are the steps you have to do in order to compleate this

Use python script to generate test data, use publisher to simulate cone detection output and use subscriber to receive output
Create a map of the track using a mapping algorithm such as occupancy grid mapping or SLAM.
Use the 3D world coordinates of the cone to determine its position in relation to the robot's current pose and output a ROS PoseWithCovariance message with the cone's position and orientation.
Continuously update the map with new cone detections and publish the updated map to a separate topic.
Attach the color (int) of the cone to the ROS message.
Publish the ROS message to a designated topic.
Test the algorithm on a real-world scenario to ensure it is robust and accurate enough for deployment. Monitor the performance of the algorithm, tweak parameters as necessary, add filtering method for noises, and optimize for speed and accuracy (Iterative process).

Merged Kalman filter cone mapping version, now exploring particle filter version


test: Update cloud CI to use known_good.json

Updates to other repos can cause errors to propagate
from changes that have nothing to do with PRs on
Vulkan-Tools.
This commit changes cloud CI systems to build with
known_good.json in Vulkan-Tools so that issues
affecting other repos do not prevent PRs in Vulkan-Tools
from getting merged.

This is passing CI.  Will merge after lunch if there are no comments by then.


conda-build hangs forever on Mac OS -- scipy seems to be the culprit

I created this minimal example of what appears to be the same problem affecting the build of climlab, e.g. https://github.com/brian-rose/climlab/issues/88, https://github.com/conda/conda-build/issues/3395
This package will build and test properly when you execute
conda build conda-recipe

from the root directory.
However if scipy is included in the run dependencies (remove the comment in meta.yaml), then conda-build will hang forever as described in the above issues.

As discussed in https://github.com/brian-rose/climlab/issues/88, the problem goes away with a particular conda configuration:
conda config --add channels conda-forge
conda config --add channels defaults
conda install pycryptosat conda-build
conda config --set sat_solver pycryptosat
conda config --set channel_priority strict

so that the .condarc file looks like
channels:
  - defaults
  - conda-forge
sat_solver: pycryptosat
channel_priority: strict

With this configuration the minimal package builds and tests successfully even when including scipy in the run dependencies.


Change color of Social media icons for better UX

Change color of Social media icons for better UX and make it look more legit

/assign

Fixed in #16


[ML][Pipelines] Remove serverless from compute schema

Description
This PR targets to remove value "serverless" from compute schema to avoid incorrect prompt.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Upload with postage stamps

As an uploader of content, I want to upload my content without being bothered by the details on purchasing stamps and attaching them, such that uploading content is not much more complicated that what it used to be before postage stamps were required.
Obvious integrations are:

swarm-cli
gateway
bee-status

This issue is a first step, the scope should be minimal and ideally, it should be released at the same time as when the postage stamps are released (or shortly thereafter).

So I have started working on this as part of the https://github.com/ethersphere/bee-js/pull/290 but I was expecting that stamps will be optional and hence choose some way how to design API around that. Now it seems that stamps will be mandatory and then it is a question if how to design the API around that. I see two ways.
Mandatory parameter
Having all the upload methods like bee.uploadData(), bee.uploadFiles() required parameter batchId that needs to be passed when uploading data. This is somewhat not great DX as the user's of bee-js has to do the stamps management across their whole app.
Example:
uploadData(data: string | Uint8Array, stamp: string | Stamp, options?: UploadOptions): Promise<Reference>

Instance's default batch
To my understanding, it is not really expected for most of the use cases to have a lot of (small) different batches as it requires on-chain transactions that are expensive, so most probably the users will create a batch that they will be employing for a longer time. @Eknir is this a valid assumption?
If so, then I would suggest a similar design like with signer, where default signer can be specified for the Bee class instance and can be specified in options where it is needed if users needs to override the default.
Example:
class Bee{
//...
	constructor(url: string, {batchId, ...}: BeeOptions) {
		//...
	}

//...

uploadData(data: string | Uint8Array, options?: UploadOptions): Promise<Reference>
}

Where options support batchId as well and at least one has to be configured.

I am still not sure if the postage stamps are mandatory or optional. @Eknir or @acud can you please share more information about this?
If it's optional I would make it part of the options. If it's mandatory it's possible to implement in the two ways that is outlined. I am not a huge fan of the way we did the signer, because if it's optional in both the constructor and in the upload then we can no longer rely on the type system to enforce correct invocations and we have to do runtime checks instead. However since we already have this pattern with the signer it may be more consistent to do the same.

We went through this on the public demo session - right now postage stamps are mandatory, this is also reflected in the API design - if you try to upload something without a batch ID in the header, the upload will fail on a 400 Bad Request.

...no longer rely on the type system to enforce correct invocations and we have to do runtime checks instead.

Well, I mean TypeScript is not giving you any guarantees. I actually think this is the "dark side" of TS when a lot of people start to "feel safe" because they have types check and hence are not doing runtimes checks and then when some JS dev uses your library he is surprised what it can do with it, usually leading with very errors that do not make sense. So I don't see it as a negative thing.

We went through this on the public demo session - right now postage stamps are mandatory, this is also reflected in the API design - if you try to upload something without a batch ID in the header, the upload will fail on a 400 Bad Request.

Thanks for the input. Well, I will move forward then with the "signer approach".

Well, I mean TypeScript is not giving you any guarantees. I actually think this is the "dark side" of TS when a lot of people start to "feel safe" because they have types check and hence are not doing runtimes checks and then when some JS dev uses your library he is surprised what it can do with it, usually leading with very errors that do not make sense. So I don't see it as a negative thing.

This is not really what I meant (and we are doing input validation anyhow). What I meant is that by making a mandatory argument optional and also optional in the constructor it is no longer possible to evaluate and therefore understand the code with an upload call without a postage stamp argument if the Bee instance creation is not in the same scope.
For example, imagine if you only see this in the code somewhere:
// ...
await bee.uploadData(data)
// ...

It may be a valid or invalid call, depending on how the bee object was instantiated. It may also be valid (in the sense that there is a batchId provided in the constructor) but incorrect in the context when using the wrong bee instance. But because the mandatory argument is decoupled from the actual call you have to verify the instance creation calls and all the different ways the bee object can be passed until it arrives to the actual invocation and that may be not always trivial.
I feel that this style makes the code slightly easier to write but harder to read ultimately. So this may be better for hackathon style of proof-of-concept projects but harder to maintain on the long term.

This was resolved in #290


The type field of perf_event_attr.config must not be used when the raw_type field is 1

When the raw_type field of perf_event_attr.config is 1, all other 63 bits are used to specify the value to be set to the IA32_PERFEVTSELx MSR, as the kernel documentation explains.
The 'config' field specifies what the counter should count.  It
is divided into 3 bit-fields:

raw_type: 1 bit   (most significant bit)	0x8000_0000_0000_0000
type:	  7 bits  (next most significant)	0x7f00_0000_0000_0000
event_id: 56 bits (least significant)		0x00ff_ffff_ffff_ffff

If 'raw_type' is 1, then the counter will count a hardware event
specified by the remaining 63 bits of event_config.  The encoding is
machine-specific.

The current version still works because the upper bits of the IA32_PERFEVTSELx MSR are just reserved and never actually used.

thanks!


GOAL: Automate AAA ingestion and calculation

We need to scrape the AAA data daily and calculate AAA gasoilne and AAA Other fuels based on the following specs - https://docs.google.com/document/d/1kvTzZEHKAFb9dSb5J55NVzTsLjrWchfVdkYS7CFfNYA/edit

Duplicate of: https://github.com/truflation/truflation-ingestors/issues/14


Lower CI Java Version to Java 11 for all plugins and run integration tests on 8

(Copy of https://github.com/opensearch-project/opensearch-plugins/issues/64)
We lowered the version of Java to 11 for OpenSearch in opensearch-project/opensearch-build#74. In the documentation we say:

We recommend Java 11 (LTS), but OpenSearch also works with Java 8.

[ ] Lower the Java version for CI to 11 (use LTS)
[ ] Run integration tests on at least Java 8 and 11 (ensure broad compatibility)
[ ] Document that the preferred version of Java for development is 14 (do development on latest and greatest)

Will require newer OpenSearch, currently JDK 11 build would fail with this in integration tests:
Error: LinkageError occurred while loading main class org.opensearch.gradle.reaper.Reaper
        java.lang.UnsupportedClassVersionError: org/opensearch/gradle/reaper/Reaper has been compiled by a more recent version of the Java Runtime (class file version 58.0), this version of the Java Runtime only recognizes class file versions up to 55.0

In that case ill close this:)

awaiting a newer OpenSearch. keeping this open for later:)

Let's scrap this and just move forward to 17 and OpenSearch 2.0.


Add yarn as a valid value for pkg_managers

Codecov Report

Merging #67 (58f86cc) into master (66fe1d7) will decrease coverage by 3.30%.
The diff coverage is 51.87%.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   69.01%   65.70%   -3.31%
==========================================
  Files          13       14       +1
  Lines         284      347      +63
  Branches       38       57      +19
==========================================
+ Hits          196      228      +32
- Misses         82      112      +30
- Partials        6        7       +1

Impacted Files
Coverage Δ

validator/__main__.py
0.00% <0.00%> (ø)

validator/schema/image_schema.py
94.44% <ø> (ø)

validator/schema/modification_schema.py
100.00% <ø> (ø)

validator/schema/rpm_schema.py
100.00% <ø> (ø)

validator/schema/streams_schema.py
100.00% <ø> (ø)

validator/schema/releases_schema.py
21.62% <22.22%> (+21.62%)
:arrow_up:

validator/distgit.py
51.28% <48.64%> (-48.72%)
:arrow_down:

validator/support.py
84.61% <86.95%> (+11.03%)
:arrow_up:

validator/cgit.py
100.00% <100.00%> (ø)

validator/format.py
100.00% <100.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6bfc90...58f86cc. Read the comment docs.

/lgtm

This has been included in https://github.com/openshift/ocp-build-data-validator/pull/71.


Fixed Possible Typo and Listed Points for Legibility

Tweaked the wording of the Context title, and listed the reasons to NOT use context for better readability.

Context has since become a stable API. Thanks for the PR!


How to use it without npm?

Is it possible to use chartjs-chart-wordcloud without npm? For exemple by downloading it (or CDN) and then adding it to my project manually?

take a look at the codepen mentioned in the README: https://codepen.io/sgratzl/pen/WNwzYgy If you download that you should have a good start


Don't enforce Javascript style guide

I found that it's really frustrating being forced to use a JS style guide. I believe linting errors and warnings should only pop up on real code issues and not for using a different number of spaces for indentation.
I like using semi colons to end statements, this might be a very uncommon example, but the JS engine will interpret the following …
return
{
    prop: 'val'
}

as
return;
{
    prop: 'val'
};

By automatically adding its own semi colons and it would actually return nothing instead of the object I was expecting. Having semi colons helps avoiding these kind of issues (although the example I'm using is anything but common)
TL:DR; Don't stop me from using the style that I'm most comfortable with. Ship the template with sane ESLint rules that won't stop me from actually writing code.

@silvestreh I think what's "sane" is very subjective in this case. We're shipping with rules from a community-developed standard, so it's not arbitrary. The good news is that you can use your own fork with vue-cli, with eslint rules that fit better with the coding style you'd like to enforce. :smiley: Or you can turn off eslint completely.
In the future, we're planning on making linting and some other features optional when generating a new project.

Yes, it's kind of subjective. That's why I was hoping to have rules that don't enforce a style, but rather print errors and warnings for actual errors and warnings. Is having a different setting for indentation an error? I don't think so.
I believe that, at least for now, the fork option is the best one. Thanks!


Ability to open JSON files that are a big dictionary

e.g. this one:
https://cdn.jsdelivr.net/npm/web-features/index.json
Which looks like this (truncated):
{
  "anchor-positioning": {
    "spec": "https://drafts.csswg.org/css-anchor-position-1/#anchoring"
  },
  "array-at": {
    "compat_features": [
      "javascript.builtins.Array.at"
    ],
    "spec": "https://tc39.es/ecma262/multipage/indexed-collections.html#sec-array.prototype.at"
  },
  "array-flat": {
    "caniuse": "array-flat",
    "compat_features": [
      "javascript.builtins.Array.flat",
      "javascript.builtins.Array.flatMap"
    ],
    "spec": "https://tc39.es/ecma262/multipage/indexed-collections.html#sec-array.prototype.flat"
  }
}

Currently have logic that looks for dictionaries and sees if they have a key that's a list of items and treats that as the data:
https://github.com/simonw/datasette-lite/blob/f334d39d50fcc3b8adb52e92b373b780134b153f/webworker.js#L143-L147
I'm going to change that so if the dictionary has more than ten items AND they are all themselves dictionaries it transforms them into a list of dicts and adds a _key primary key to each one.

Actually no, I'm going to say it does that if it's a dictionary where each value is itself a dictionary (not a list).

Updated documentation: https://github.com/simonw/datasette-lite/blob/3392020b36e1495d6e1ca0b222748ebc814fe5f1/README.md#loading-json-data

Demo: https://lite.datasette.io/?json=https://cdn.jsdelivr.net/npm/web-features/index.json#/data/index

TIL: https://til.simonwillison.net/datasette/baseline


Feature: Documentation

Expand docs to show all commands with usage examples.

Updated README.md on branch main.


protocol running with web version.

for the web version, it could be cool to see it run protocols for programs, like opening a new tab for html, and literally just opening the file with python for python files, could be cool :)

I'm not sure I understand this feature request. Can you elaborate a bit?


Neovides uses cpu while idle

Looks like the most recent rewrite broke the idle pausing of the renderer. I'll need to track down why that is and fix it.

On windows it shows using ~95% of the GPU in addition to ~1.5% of CPU when idle and in the background. If I lower the framerate to 20, it goes down to ~40%. The very high GPU usage seems more concerning than the CPU usage.

On windows it shows using ~95% of the GPU in addition to ~1.5% of CPU when idle and in the background. If I lower the framerate to 20, it goes down to ~40%. The very high GPU usage seems more concerning than the CPU usage.

Yeah I need to track down what exactly is going on

Yeah I need to track down what exactly is going on

It also appears to increase over time indicating a leak of some sort

It also appears to increase over time indicating a leak of some sort

I don't really understand the codebase, but it looks like idle events are created, but never handled.
https://github.com/Kethku/neovide/blob/68582e05516fd3f6744752d8198ff3016d2190b9/src/editor/window.rs#L381-L383
https://github.com/Kethku/neovide/blob/68582e05516fd3f6744752d8198ff3016d2190b9/src/renderer/rendered_window.rs#L551
https://github.com/Kethku/neovide/blob/68582e05516fd3f6744752d8198ff3016d2190b9/src/window/winit/mod.rs#L252-L254

It seems to be related to multigrid. Scrolling down, as in to render the next line, with it enabled triggers it. On another note, neovide seems to using quite a bit of GPU power compared to something like Chrome. It's fairly negligible for me, topping out at ~5.5% GPU, but something I noticed.


Enhancement: configurable decimal places in r2 and r stats

Sometimes it's useful to display more decimal places in r2 and r than the hard-coded 2 decimal places currently available:

This is easily passed as a configuration option (e.g. decimalPlaces) in the regressionSettings object and can be defaulted to 2, so that existing users will see no difference.
A related issue is that the value of regression.rSquared is rounded to two decimal places before it is use to calculate regression.rValue - this can easily lead to rounding errors in the rValue, so it probably makes sense to calculate the rValue first with the unrounded rSquared and then round rSquared to the desired number of decimal places afterwards.
I'll create a pull request with these changes/enhancements - let me know if you are OK with merging it. Thanks for a most useful library!

yeah ! sounds great, send me the pull request.
Thanks for contributing to this project - I'm a bit busy at this moment,
but I will find a time to add more improvements and fixes
2015-06-11 17:43 GMT-03:00 Seamus Mac Conaonaigh<EMAIL_ADDRESS>
Sometimes it's useful to display more decimal places in r2 and r than the
hard-coded 2 decimal places currently available:
[image: image]
https://cloud.githubusercontent.com/assets/394952/8117769/d2314abc-1058-11e5-8b6f-a4d3921f26ee.png
This is easily passed as a configuration option (e.g. decimalPlaces) in
the regressionSettings object and can be defaulted to 2, so that existing
users will see no difference.
A related issue is that the value of regression.rSquared is rounded to
two decimal places before it is use to calculate regression.rValue - this
can easily lead to rounding errors in the rValue, so it probably makes
sense to calculate the rValue first with the unrounded rSquared and then
round rSquared to the desired number of decimal places afterwards.
I'll create a pull request with these changes/enhancements - let me know
if you are OK with merging it. Thanks for a most useful library!
—
Reply to this email directly or view it on GitHub
https://github.com/streamlinesocial/highcharts-regression/issues/17.


Fix typing / quickly. Fixes #2077

Basically if anything is typed immediately before the slash (not just a slash), then it's OK to drill down (I think).
I also got rid of one if() for the case where / is long pressed. I didn't like the fact that it would immediately drill down into 'Macintosh HD' if you held / for long enough. I'm happy for this to be changed back though if there's a case for it

I didn't like the fact that it would immediately drill down into 'Macintosh HD' if you held / for long enough.

Same here. I’ll give this a look.


Child entities not persisting when using AddRangeAsync and SaveChangesAsync

Attempting to add a graph of entities where there is a 1:m relationship (parent: children) using the AddRangeAsync and Savechanges Async method will prevent the children from being persisted to the DB.
In the code below, the a collection of StoreProductEntity (Parent) are being persisted in the DBContext . Each StoreProductEntity has an IList (child).  After calling AddRangeAsync() and SaveChangesAsync(), only the parent entities are persisted, the children are not.
This is different behavior than the corresponding synchronous methods, AddRange() and SaveChanges(), where in the entire graph (parent and children) are persisted to the DB.
[Table("StoreProduct")]
    public class StoreProductEntity
    {
        public int Id { get; set; }
        public string Name { get; set; }

        public virtual IList<StoreProductPriceEntity> Prices { get; set; }

    }
    [Table("StoreProductPrice")]
    public class StoreProductPriceEntity
    {
        public int Id { get; set; }
        public decimal SalesPrice { get; set; }
        public virtual StoreProductEntity StoreProduct { get; set; }
        public int StoreProductId { get; set; }
    }

    public class Provider
    {
        private readonly DataContext dataContext;

        public Provider(DataContext dataContext)
        {
            this.dataContext = dataContext;
        }
        public async Task<int> AddAndSaveAsync(IEnumerable<StoreProductEntity> entities)
        {
            await this.dataContext.StoreProducts.AddRangeAsync(entities);
            var numAdded = await this.dataContext.SaveChangesAsync();
            // StoreProductEntities are saved, the children StoreProductPrice entities are not saving.
            return numAdded;
        }
    }

    public class Sample
    {
        private readonly Provider provider;

        public Sample(Provider provider)
        {
            this.provider = provider;
        }
        public async Task<int> SaveEntitiesSample()
        {
            var entities = new List<StoreProductEntity>
            {
                new StoreProductEntity
                {
                    Name = "A",
                    Prices = new List<StoreProductPriceEntity>
                    {
                        new StoreProductPriceEntity {SalesPrice = 1m}
                    }
                },
                new StoreProductEntity
                {
                    Name = "B",
                    Prices = new List<StoreProductPriceEntity>
                    {
                        new StoreProductPriceEntity {SalesPrice = 2m}
                    }
                }
            };

            // This call will persist the StoreProducts, but not the nested storeproductPrices.
            return await this.provider.AddAndSaveAsync(entities);

            //Also Tried.
            foreach (var entity in entities)
            {
                foreach (var price in entity.Prices)
                {
                    price.StoreProduct = entity;
                }
            }
            // same result.  Store Products are persisted, but not the nested/child storeproductprices.
            return await this.provider.AddAndSaveAsync(entities);

        }
    }

Further technical details
EF Core version: 1.1
Database Provider:  Microsoft.EntityFrameworkCore.SqlServer
Operating system: Win 10
IDE:  Visual Studio 2015

I too face same issue while adding enity with child collection using AddRangeAsync with  SaveChangesAsync

Note for triage: looks like async Add methods are, at least sometimes, only acting on a single object.
@cdarrigo @mkprajeen You almost certainly don't need to use the async methods for Add/AddRange. They provide no benefit over the sync versions except when using the specialized sequence value generator. (It would be great to know if you are actually using that value generator.)

It works fine when I changed to AddRange instead of AddRangeAsync.  thanks ajcvickers

@divega @rowanmiller Should this also go in 1.1.1? AddAsync first shipped in 1.1.

To me it seems like a serious enough issue to warrant a patch.

Talked to @divega and moving.

Justification: The behavior of Add and AddAsync should be the same. The async versions not doing graph behavior is a copy/paste error.
Risk: Low-risk; copy/paste error.

This patch bug is approved. Please use the normal code review process w/ a PR and make sure the fix is in the correct branch, then close the bug and mark it as done.

✅ Verified


⚠ Bofrost.fr has degraded performance

In 78b0993, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1187 ms

Resolved: Bofrost.fr performance has improved in 8a27990 after 14 minutes.


Keeps triggering SELinux for trying to access execheap

Type: Bug
This just keeps repeatedly happening...
I had the same issue with VSCodium.
Really not sure if this is about SELinux or about VSCode?
Would love some advice on how to behave here. Like does VSCode need this access?
Here's the message:
SELinux is preventing code from using the execheap access on a process.
*****  Plugin allow_execheap (53.1 confidence) suggests   ********************
If you do not think code should need to map heap memory that is both writable and executable.
Then you need to report a bug. This is a potentially dangerous access.
Do
contact your security administrator and report this issue.
*****  Plugin catchall_boolean (42.6 confidence) suggests   ******************
If you want to allow selinuxuser to execheap
Then you must tell SELinux about this by enabling the 'selinuxuser_execheap' boolean.
Do
setsebool -P selinuxuser_execheap 1
*****  Plugin catchall (5.76 confidence) suggests   **************************
If you believe that code should be allowed execheap access on processes labeled unconfined_t by default.
Then you should report this as a bug.
You can generate a local policy module to allow this access.
Do
allow this access for now by executing:
ausearch -c 'code' --raw | audit2allow -M my-code
semodule -X 300 -i my-code.pp
Additional Information:
Source Context                unconfined_u:unconfined_r:unconfined_t:s0-
s0:c0.c1023
Target Context                unconfined_u:unconfined_r:unconfined_t:s0-
s0:c0.c1023
Target Objects                Unknown [ process ]
Source                        code
Source Path                   code
Port
Host                          home
Source RPM Packages
Target RPM Packages
SELinux Policy RPM            selinux-policy-targeted-40.24-1.fc40.noarch
Local Policy RPM              selinux-policy-targeted-40.24-1.fc40.noarch
Selinux Enabled               True
Policy Type                   targeted
Enforcing Mode                Enforcing
Host Name                     home
Platform                      Linux home 6.9.10-200.fc40.x86_64 #1 SMP
PREEMPT_DYNAMIC Thu Jul 18 21:39:30 UTC 2024
x86_64
Alert Count                   5308
First Seen                    2024-07-26 22:57:24 CEST
Last Seen                     2024-07-27 21:09:10 CEST
Local ID                      68a0f966-91d0-4948-8ed8-522e3456b749
Raw Audit Messages
type=AVC msg=audit(1722107350.878:1108): avc:  denied  { execheap } for  pid=5126 comm="code" scontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tcontext=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023 tclass=process permissive=0
Hash: code,unconfined_t,unconfined_t,process,execheap
VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:08:12.169Z)
OS version: Linux x64 6.9.10-200.fc40.x86_64
Modes:
Extensions (10)

Extension
Author (truncated)
Version

catppuccin-vsc
Cat
3.15.0

catppuccin-vsc-icons
Cat
1.13.0

es7-react-js-snippets
dsz
4.4.3

prettier-vscode
esb
10.4.0

copilot
Git
1.219.0

copilot-chat
Git
0.17.1

debugpy
ms-
2024.8.0

python
ms-
2024.10.0

vscode-pylance
ms-
2024.7.1

es7-react-js-snippets
rod
1.9.3

/duplicate https://github.com/microsoft/vscode/issues/224673


Donate link is not working

Describe the bug
https://salvo.rs/donate.md = 404
To Reproduce
Steps to reproduce the behavior:

Go to 'salvo.rs
Click on 'Donate'
See error

Expected behavior
See how to donate to this project

Thank you, I fixed it, I feel like I missed out on a billion :)


Fix encounter card summary component

Description

Right now the EncounterSummary component crashes the app when hovered, and has styling issues.

Acceptance Criteria

[ ] Component doesn't crash app
[ ] Fix component styling
[ ] Rename to EncounterCardSummary for consistency
[ ] Add tests if missing

Implementation details

n/a
Testing notes

n/a

LGTM!