[firebase_crashlytics] 65 key/value pairs will be allowed

I didn't test this, but reading the code I think I found a bug in this method:
void _setValue(String key, dynamic value) {
    // Check that only 64 keys are set.
    if (_keys.containsKey(key) || _keys.length <= 64) {
      _keys[key] = value;
    }
  }

So in the line if (_keys.containsKey(key) || _keys.length <= 64) when we want to add a new key/value pair, and there are already 64 key/value pairs, a 65th will be added. It should be _keys.length < 64 or _keys.length <= 63, thus blocking at 64 and not at 65, since if (_keys.length <= 64) is true  for _keys.length == 64, which allows one more addition.

Thanks for this report, as you are aware Crashlytics has since had a major rework and we now defer custom key counting/limits to the validators that already exist in the native SDKs (no need to duplicate logic there).
Going through a bit of issue cleanup today so will close this now.

No problem, happy cleaning :)


Added option to create/update an API using its name.

This is to allow automatic creation/update and deployment of an API.
This allows us to:

Create an API if it doesn't exist.
Update an API if it already exists.
Not maintain the API ID. Since, having multiple APIs with same name is a scenario, I am not sure, how many people will want.
Only need a Swagger file.
Easy to automate in CI.

Fixed the issue that was present in the previous pull request.


Fix unable to get entries which are not in a folder

I was unable to get BitWarden entries which are not in any folder because get_data() always adds the --folder argument, even if no folder was given.

It seems like there already is a pull request mentioning this problem, closing!

Thanks, I'm still waiting for the changes by @schuppentier. Hopefully, it will soon be fixed :)

It should now be fixed on main 🙂


Add PM6_VeryStrong criteria code modifier to pull-down for PM6

Request for a PM6_VeryStrong option to be added to the drop-down menu for PM6.
It is a valid option per SVI guidance: https://www.clinicalgenome.org/site/assets/files/8490/recommendation_ps2_and_pm6_acmgamp_critiera_version_1_0.pdf

Replaces https://github.com/ClinGen/clincoded/issues/1787

If rolled out we'd probably need an OR between these criteria.
This ticket requires input from the SVI.

Retiring this in favor of #41


Data Grid scrolling fixes

closes #2064
closes #2156

Can we add a test?

have been trying to write a test that fails on the old version and passes on this but haven't been successful (on making it fail for old) so far - will keep trying

@jpmorganchase/salt-developer after 2+ days I'm struggling to write a test that works (current one in files is one attempt of many..) does anyone have any ideas on how this can definitely work in cypress?


latency reduction

This reduces the latency of the first include("examples/2d/elixir_advection_basic.jl") on my system between five and ten percent while using Trixi does not change significantly. I've measured that by running
julia --threads=1 -e '@time using Trixi; @time include("examples/2d/elixir_advection_basic.jl")'

multiple times. It's basically a proof of concept and we might invest more work to reduce the latency further. I'm also looking forward to Julia v1.6 in this respect.

Again, only examples/2d/elixir_mhd_rotor_shockcapturing_amr.jl  fails...

However, I feel like there should be more documentation, especially about the way precompile.jl was generated. I don't know if it would be better in the source code or in the docs, but right now it's basically you or nobody who can work with it.

I've added some documentation in the source code in 1fa7b1c. You can think of this PR as a basic proof of concept and a learning experience for me. Performing more of this work, I would expect to be able to gain additional five to ten percent by adding more precompile statements. However, a lot of time seems to be spent on actually compiling machine code, which isn't cached right now. I think it's also useful for us to know this kind of information. It's also worth noting that Trixi can benefit if other packages (we depend on) use (more) precompile statements.

I think it's great that you're looking into this! Since 1.6 will presumably considerably change the performance in this metric, I guess we should wait until it is released and then look into it again.

Yes, I think so, too. A preliminary check using the current nightly build of Julia on my system shows that the time of using Trixi decreases significantly while the latency of trixi_include(default_example()) increases a bit. Nevertheless, the combined latency of both statements is reduced by ca. 15 to 20 percent compared to Julia v1.5.2.


Ability to aggregate features of dependencies

After it is possible to declare dependencies to other modules, it would be a great benefit to support the automated aggregation of their features into the own features, sites and products.

Automated aggregation of dependency features into own features, sites and products can be enabled by setting a corresponding filter property, e.g.
-Db2.aggregate.feature.sdk.filter = **.sdk.**
-Db2.aggregate.category.sdk.filter = **.sdk.**
-Db2.aggregate.product.sdk.filter = **.sdk.**

<properties>
  <b2.aggregate.feature.filter>!**.sdk.**,!**.tests.**,!**.sources.**</b2.aggregate.feature.filter>
  <b2.aggregate.feature.sdk.filter>**.sdk.**</b2.aggregate.feature.sdk.filter>
</properties>


Change beaconlog's date to a timestamp

Currently we're saving just a 2017/01/20 style date. Changing this to a unix timestamp would allow us to format it later and allow us to include time in our display (if needed). Go has some built in formats that look pretty nice. https://golang.org/pkg/time/#pkg-constants

Great - we looked into this but couldn't figure out datetimes and how to handle them yet for Go. Let's get into it when we fix up writing to file #4

Sure. I have a solution for writing to the file. It's based on the work @benjmarshall has in his branch.

Great! we just merged #24 and this should be good to go next!

Closed in #30


Not correctly parsing Typescript output?

Hi there, first of all - thanks for putting this together!
I'm having an issue getting it to work in my TypeScript React project.
I'm using the example script, and I get empty objects back for all my files.
I added some console.log statements to your code to see what's going on.
The first is in findFirstJsxOpeningLikeElementWithName:
[ NodeObject {
    pos: 4020,
    end: 4147,
    flags: 0,
    transformFlags: undefined,
    parent: undefined,
    kind: 247,
    tagName:
     IdentifierObject {
       pos: 4021,
       end: 4037,
       flags: 0,
       parent: undefined,
       text: 'FormattedMessage' },
    attributes: [ [Object], [Object], pos: 4037, end: 4140 ] } ]
[]

The next is in main in the elements.attributes.forEach function:
attr:
NodeObject {
  pos: 4037,
  end: 4075,
  flags: 0,
  transformFlags: undefined,
  parent: undefined,
  kind: 250,
  name: IdentifierObject { pos: 4037, end: 4046, flags: 0, parent: undefined, text: 'id' },
  initializer:
   NodeObject {
     pos: 4047,
     end: 4075,
     flags: 0,
     transformFlags: undefined,
     parent: undefined,
     kind: 252,
     expression:
      TokenObject {
        pos: 4048,
        end: 4073,
        flags: 0,
        parent: undefined,
        kind: 9,
        text: 'Magikcraft.NotLoggedIn' } } }
attr:
NodeObject {
  pos: 4075,
  end: 4140,
  flags: 0,
  transformFlags: undefined,
  parent: undefined,
  kind: 250,
  name:
   IdentifierObject {
     pos: 4075,
     end: 4095,
     flags: 0,
     parent: undefined,
     text: 'defaultMessage' },
  initializer:
   NodeObject {
     pos: 4097,
     end: 4140,
     flags: 0,
     transformFlags: undefined,
     parent: undefined,
     kind: 252,
     expression:
      TokenObject {
        pos: 4098,
        end: 4138,
        flags: 0,
        parent: undefined,
        kind: 9,
        text: 'Go to Minecraft and type \'/spellbook\'' } } }

And finally in main just before return res.concat(dm);. This is res:
[ { id: undefined, defaultMessage: undefined } ]

It looks like elements.attributes.forEach is parsing the wrong properties of the returned nodes. Could this be a TypeScript compiler version issue?
You are requiring typescript, but the package.json doesn't specify a version. The version in my project is 2.1.4.

Changing line 71 of lib/index.js to:
        element.attributes.forEach(function (attr) { return attr.name && (msg[attr.name.text] = attr.initializer.expression.text); });

Ping @jwulf  I just merged your PR and publish a new version to npm. Thanks


Boolean values in Docker API

While implementing isolated tests I have noticed some issues.
Docker API supports boolean values as 1/true/True or 0/false/False only (as well as doc says). Part of current implementation doesn't follows requirements, false becomes empty string in request ?force=&noprune=.
I offer to cast these values to 0 or 1 explicitly.
https://github.com/r0b1n/docker-php/commit/aa51198bc6d110b187897a198dc558653384597d

@r0b1n I agree that this should be fixed. They might prefer type casting though..see https://github.com/stage1/docker-php/blob/master/src/Docker/Docker.php#L133

Yup, these values should be type casted, don't hesitate to submit a PR :)

closed in #147


Extensions in portable mode are broken

From @keck9939
If you follow exactly the instructions at https://code.visualstudio.com/docs/editor/portable#_migrate-to-portable-mode to migrate to portable mode, the extensions vanish. (I'm using Windows 10, VSCode 1.74.0). I did move rather than copy the data and code directories. This may be important, otherwise it might find things in the original location.
In addition, if you re-install the extensions in the portable installation after following the referenced instructions, and then move the whole portable directory from say d:\vscode to e:\vscode, the extensions will vanish again.
Steps to Repro:

Create a portable VS Code and install some extensions
Copy this set up to another machine
🐛 No extensions are shown

Here is the workaround until this fix is available

Open the extensions folder and delete following files - extensions.json and .init-default-profile-extensions

Hello, Thank you for sharing this workaround. Will this be fixed in a future version? (I use a USB Stick, and when the drive letter changes, it has to be repeated again)? Thanks!

Will this be fixed in a future version?

Yes, it is and will be available in next release 1.75.0

There are other files that contain absolute paths too:

./data/user-data/languagepacks.json
./data/user-data/User/globalStorage/storage.json

Those absolute paths will be updated on VSCode start, thus they are not causing any breakages. But, to reduce file change for those portable mode users on the go, is it possible to share the new toRelativePath() function and apply the same fix to the code related to those files?


Use POPF

Hello,
I’m using the PDDL extension in vscode, and I want to use the POPF planner, only it keeps giving me this error:
zsh: exec format error: /Users/rodrigobernardo/Desktop/PDDL_CODE/popf.exe
rodrigobernardo@MacBook-Pro-de-Rodrigo PDDL_CODE %

The domain and the problem are fine because I already tested with the default planner.
The POPF executable, I took from the ROSPLAN page on GitHub
Could the file extension be wrong? What could I be doing wrong?

I tried using it on linux and it works. Where can I get the extension to use on Mac?

As discussed in the Slack group: there is only one build of the PDDL extension for VS Code. It runs cross platform as it depends on node.js. But the planner executable runs in a process spawned by node.js on your local operating system. On Windows, you need a windows build of the planner. On Mac, you need a MacOS/Darwin build of the planner.
No, the popf.exe would not run on Linux.


Bump version

I wonder if the GHA which updates the data could be updated to bump the version as well (IIRC bsb_nz does that).

Thanks Zubin. Will be updating the GHA soon to handle the version bump.


Edge case in Fall Mountain if you grab the crown at the same time than someone else.

If you grab the crown at the same time than someone else and it shows your celebration the game will try to go to the reward screen but it won't be able to do it and after a few seconds the client will disconnect from the server, the tracker won't register it.
Here are the logs:
Player-prev.log
Player.log
At the end of Player-prev.log (I restarted the game to get the crown after I got disconnected) it shows that I won the round and also it shows the crash so I'm guessing you should be able to catch this edge case.
Here are the db files, keep in mind I played one game before I realized that I should submit a report and I won the previous final on Fall Mountain (if you look at the time you will notice that it's not the same game).

It's actually quite hard to account for that, it's actually why a bunch of people had messed up stats about 2 weeks ago, cause I was checking for the results from the server but wasn't verifying when they came. It can come at any point when playing and it would screw up the logic and cause a lot of problems, so I made it so you had to get the results at the end of the show or I would ignore it. It might be possible to do something for your case, but I wouldn't want to mess with it much without possibly screwing up stats again for normal games.
I can move things around in your log file to make it record that game for you if that's something you'd want. You'd then just make sure the game is closed and make it Player.log and open the tracker and it should add just that one show.

I appreciate that you took the time to fix the log, thanks, it was added correctly after I did what you said, anyways it's not really an issue for me I just thought it was worth reporting it, for now on I'll just add one to the previous game counter if I encounter this issue like I did for this round before you fixed the log.

I think Ill close this now. Not going try and mess with adding special logic to account for this.


Non-Microsoft developer unable to build PTVS

From gitter chat:
Seems quiet, but here we go. I'm trying to build load in VS2022 a cloned repo of PTVS with no success, following instructions there and other hints. My (non-corporate) MS account login required to get the creds to access the NuGet repo is rejected. Any help? Here's some CLI things I've tried:
C:\xxx\xxxxx\PTVS>vsts-npm-auth -config .npmrc -force

vsts-npm-auth v<IP_ADDRESS>
-----------------------
Getting new credentials for source:https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/, scope:vso.packaging_write vso.drop_write
Couldn't get an authentication token for https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/.
Couldn't get an authentication token for https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/.

C:\xxx\xxxxx\PTVS>c:\users\xxxxxxxxx\.nuget\plugins\netfx\CredentialProvider.Microsoft\CredentialProvider.Microsoft.exe -I -V Verbose -U "https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json"
[Verbose] [CredentialProvider]Running in stand-alone mode
[Verbose] [CredentialProvider]Command-line v1.0.0+94af8126cf520805dd7e213b92cdecd9d8728d1f: c:\users\xxxxxxxxx\.nuget\plugins\netfx\CredentialProvider.Microsoft\CredentialProvider.Microsoft.exe  -I -V Verbose -U "https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json"
[Verbose] [CredentialProvider]Handling auth request, Uri: https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json, IsRetry: True, IsNonInteractive: False, CanShowDialog: False
[Verbose] [CredentialProvider]URI: https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json
[Verbose] [CredentialProvider]VstsBuildTaskServiceEndpointCredentialProvider - This credential provider must be run under the Team Build tasks for NuGet with external endpoint credentials. Appropriate environment variable needs to be set.
[Verbose] [CredentialProvider]Skipping NuGetCredentialProvider.CredentialProviders.VstsBuildTaskServiceEndpoint.VstsBuildTaskServiceEndpointCredentialProvider, cannot provide credentials for https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json
[Verbose] [CredentialProvider]VstsBuildTaskCredentialProvider - This credential provider must be run under the Team Build tasks for NuGet. Appropriate environment variables must be set.
[Verbose] [CredentialProvider]Skipping NuGetCredentialProvider.CredentialProviders.VstsBuildTask.VstsBuildTaskCredentialProvider, cannot provide credentials for https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json
[Verbose] [CredentialProvider]GET https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json
[Verbose] [CredentialProvider]VstsCredentialProvider - Detected an on premise Azure DevOps Server.
[Verbose] [CredentialProvider]Skipping NuGetCredentialProvider.CredentialProviders.Vsts.VstsCredentialProvider, cannot provide credentials for https://pkgs.dev.azure.com/devdiv/_packaging/Pylance/npm/registry/nuget/v3/index.json
[Verbose] [CredentialProvider]Unable to acquire credentials.

We need to do the following:

[x] Create a public azure artifacts feed for pylance
* This feed is available at https://dev.azure.com/azure-public/vside/_artifacts/feed/pylance
* The npm registry URL for pulling from this feed is https://pkgs.dev.azure.com/azure-public/vside/_packaging/pylance/npm/registry/
[x] Add a step in the pylance release process to push STABLE releases to this new feed (Erik is doing this)
[x] Manually push the latest pylance to the public feed for testing (Erik is doing this)
[x] Update the pylance feed in .npmrc
* The public feed we'll be using is the super public feed for microsoft, which is https://pkgs.dev.azure.com/azure-public/vside/_packaging/msft_consumption/nuget/v3/index.json
* This feed has our new pylance feed as an upstream source
[x] Update the nuget feed in nuget.config to something public facing, like https://pkgs.dev.azure.com/azure-public/vside/_packaging/msft_consumption/nuget/v3/index.json
* Run PreBuild.ps1 to make sure it works. Update missing package versions as appropriate.
* Report any failures to Andrew Arnott, he will add the appropriate upstreams needed
[x] Test on a fresh VM to make sure everything works as it should
* On the VM, with no authentication setup done, try to run PreBuild.ps1. This should fail because the public feed doesn't have packages we need.
 This SHOULD pull packages from upstream sources into the public feed. Once this is done, the public can pull from the public feed, no further authentication should be required unless new package versions are released.
* On the VM again, try to run PreBuild.ps1. The package restore for both npm and nuget should now succeed.
[x] Add a step in the pylance release process to populate the public feed by pulling the just released pylance version with authentication.
* This will automate the previous step and it won't need to be done manually anymore except in rare circumstances when we upgrade nuget packages.
[x] Update the build docs at https://github.com/microsoft/PTVS/wiki/Build-and-Debug-Instructions-for-PTVS.
* All the authentication steps for the public will go away once we are using public package sources.
* Keep auth docs there for OUR team so we can pull them to the public feed once new versions are released

Thanks, wasn't sure if this was appropriate here.

Definitely yes :)
The repo is public so the public should be able to build. Let me investigate the best way to handle this and I’ll reply back

@walterpg I'm working on this now, there are a few infra tasks we need to do first to enable this. This is my current task :)

@walterpg https://github.com/microsoft/PTVS/pull/7178 fixes this problem. Just pull latest from main once this issue is closed. I've updated the build instructions to indicate that the authentication stuff should only be run by microsoft employees. The public should never have to do it.

Thanks, finally got back to this. The PythonTools.sln Debug/Any CPU build works in VS 17.4 pre2.1 with some massaging.


Short solution needed: "How to install ufw in ubuntu" (ufw)

Please help us write most modern and shortest code solution for this issue:
How to install ufw in ubuntu (technology: ufw)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to install ufw in Ubuntu


Two questions concerning create_mask_rcnn_tf_record_multi.py

First, great job and congrats.
Question1: From the following code, I see that if a bounding box of one big object contains multiple other smaller objects of the same class (for example, if the objects are diagonal), then the mask of these smaller objects are included in the bounding box of this big object. I just want to make sure that it is an intended behavior.
code:
mask_np_black = mask_np*0
mask_np_black[int(ymin):int(ymax), int(xmin):int(xmax)] = mask_np[int(ymin):int(ymax), int(xmin):int(xmax)]
mask_remapped = (mask_np_black == mask_pixel[class_name]).astype(np.uint8)
masks.append(mask_remapped)

example:

Question2: With the help of some 3rd party tools, for example, openCV, the create_mask_rcnn_tf_record.py could be modified without much difficulty to suit the case where multiple objects within the same class could be automatically handled without using LabelImg tool. If you want, I could make a PR on this.
Many thanks again.

Hi,
Thank you for pointing it out, mentioned behavior is not intended. It will great if you could fix this and make a PR, otherwise I will do it as soon as I get time.
Thank you for your proposal to contribute. It will be great if you could work on removing the dependency on LabelImg tool. I also have an idea (not mature enough to cover all possible use cases though) to remove the dependency on LabelImg tool but I didn't get time to work with it. I would like to share my idea  with you (in case it could help you while you will be working with the idea you have). My idea is to use connected component algorithm for each class (or pixel color) in order to distinguish between different objects of same class (or pixel color) but drawback is the case where objects of same class have overlapping mask. It will be great if you could share your approach too and we discuss to finalize the approach.

Hi there,
Thank you for your quick reply.
Concerning question 1, I have some doubt since actually I myself still do not quite understand the relationships between masks and bounding boxes. Do you mean that in the above example, the bigger bounding box should only contain the mask of the bigger rotated red rectangular, leaving out the smaller rotated rectangular?
Concerning question 2, I am thinking of using the thresholding and findcontour by openCV library. I will try to make a small test first to verify if this approach works. Then I'll make a PR for your verification. But the behavior of my first question will also affect the implementation.

yes, the bigger bounding box should only contain the mask of the bigger rotated red rectangular, leaving out the smaller rotated rectangular because here we are drawing bounding box and mask for each individual object.
Good Idea! Just find contour show also do, its just that it will also have same limitation as the one I mentioned in my last comment (fails in the case where we have overlapping mask). Unless until we find the solution for issue with overlapping masks of objects, it will not be good idea to take in the solution.

Thanks again.
By "overlapping mask", do you imply that the same region of an image could belong to different objects? Personally I have not met such a situation and I do not know if mask-rcnn support this case. But anyway, I don't think this could not be solved by using LabelImg tool.

Yes, I meant [B]. Looking forward to see the solution.

Hi Vijendra,
I have attached a mini-sample for your reference.
I made myself a sample mask png with 3 circles: red, green, and blue, each overlapping with each other, like this:

This small sample will individual each partial body of the circle, and show all 3 partial bodies using cv2.imshow(). The rest of the work should be simple enough.
Please let me know if this is the behavior you were expecting. Cheers.
overlapping_test.zip

Hi,
I think working with the case shown in the image below should be better because it also covers the case where the object of the same class is overlapping.

Hi Vijendra,
I think now I understand your concern. I have to admit that I haven't met this case before, though it comes now as an obvious question.
I am not sure how exactly the mask-rcnn learns to segment these two red circles during training. Probably by using the bounding box data supplied with the mask? In this case, if we don't use the LabelImage, the machine has to learn to annotate the bounding box in autonomy.
One way I would think of is to use watershed algorithm that comes freely by openCV library, assuming that the boundary between objects of the same class is clear, which could be a problem ...
Tom

Hi Tom,
One easy case which I could tell you is the balloon image you had attached above. There if we have balloon as a class then we will have to label all balloons as the same class and it will be overlapping.
Good question regarding Mask-RCNN and because of how mask-RCNN architecture is built on top of Faster-RCNN, I agree with your opinion.
I think it's not safe to assume that the boundary between objects of the same class is clear.
Your other approach of annotating border will not be user-friendly but it will work well so I will say worth a shot.
If we are using a pre-trained model given in Tensorflow API or any other pre-trained model which is trained with a rectangle as bounding-box then we could use rectangle only. If you want to use any other polygon then you will have to train your own model with the kind of polygon you want as a bounding box after doing some modifications in architecture.

Hi Vijendra,
I came across this article written by a matterport engineer which gives a good walk through over mask-rcnn.
This article did not mention the case you illustrated, but I am quite sure that his also used masks to generate bounding boxes for overlapped objects (ballon), as he wrote:

And, bounding boxes are generated dynamically from the masks.

Looking at his example of annotation, it seems that he used different colors to mask balloons that overlap each other:
annotation image
I will look into the dataset that he created to have a better understanding.
Tom

Hi Vijendra,
I looked into the balloon sample from matterport mask rcnn project. Practically it uses polygons to save the mask boundary for each object in the image. I think that's why there's no problem to generate bounding boxes in this case. Other than bounding boxes, the boundary of connected objects is already set by hand.
So in my personal opinion, since PixelAnnotationTool is great and ease to use and worth to stick to, it could be a work-around to ask user to manually add a boundary between the same objects overlapping with each other.
Tom

Hi Tom,
I agree that right now the only option we could see is to ask the user for drawing the boundary, it will be great if you could implement this in a separate file and make a PR. I will review it.

Hi Vijendra,
I have tried the "drawing boundary manully" approach and it seems that PixelAnnotationTool lacks some native support for it. See question I raised _https://github.com/abreheret/PixelAnnotationTool/issues/39_
So I have been re-thinking this issue. there might be another approach, to better make use of the capability of PixelAnnotationTool.
For the overlapped objects, no matter if they belong to the same type or no, use different color to mark them. And in your create_mask_rcnn_tf_record.py file, simply modify your color dictionary like this:
mask_pixel = {'redcircle':[76,149], 'bluecircle':[29]}
So instead of assigning one color to one object, use an array with multiple colors for one object. In the above example, I use 76 and 149 to define "redcircle" and 29 for "bluecircle". In this case, as long as the color assignments are consistant throughout the whole dataset, it would be fine.
And it requires some minimum modifications to the original code. What do you think ?

Hi Tom,
Sorry for the late reply because of unavailability. I agree with your idea,  keeping different color for all overlapping objects will be good in order to avoid the need of boundary whereas the choice of using different colors for all or keeping enough different color to avoid overlapping objects of same class having the same color could be left on the user (same code will work for either choice made by user). I reviewed the PR you made, looks good. Please add comments for the changes you have made, I will merge it as soon as comments will be added.


Update the package file and contribution instructions

I had quite a few issues getting the tests to run after I pulled down the repository.  After playing with package.json for a while, I found a set of packages that would run on a clean download for tests and hoped it would be useful for others.  I also added instructions for installing grunt-cli and running bower and npm.

Thanks for this PR.
These packages are not necessary:

karma-chrome-launcher (you included twice but none of them needed)
karma-firefox-launcher
karma-html2js-preprocessor
karma-requirejs
requirejs
karma-coffee-preprocessor

Try these and let me know if it works or not.

I'll check into this on Wednesday.  I think I removed them and it stopped building until I readded karma-chrome-launcher (which is probably how I got it twice), but I'll certainly try.  Any thoughts on #373?  That's the one that's more important to me since it solves a problem we're having at the office with using the autocomplete inside each row of a table.

It appears that it does indeed still work with the reduced number of packages so you can see if this works better.


Is circular_queue::safe_reset necessary?

Clang seems to want to optimize the zeroing in circular_queue::safe_reset by a call to bzero, even when using #pragma clang optimize off. I'm not completely clear as to whether bzero is async-safe (this function is defined directly in assembly, at least under FreeBSD, and although it certainly does not call malloc I couldn't tell whether clobbering is an issue). I assume that there is probably a problem with bzero, given that under FreeBSD and Mac OS X the program appears to get stuck with a number of threads in libsystem_platform.dylib`_platform_bzero$VARIANT$Merom.
I tried just removing the code in circular_queue::safe_reset, and nothing terribly bad seemed to happen (I did not check that the stack traces were all correct).
Can circular_queue::safe_reset be removed? If not, how can the problem be addressed?
Edit: the use of a volatile variable as below obviously prevents the bzero optimization (and allows the program to complete), but it's probably very wasteful:
volatile int zero = 0;

void safe_reset(void *start, size_t size) {
    char *base = reinterpret_cast<char *>(start);
    char *end = base + size;
    for (char *p = base; p < end; p++) {
        *p = zero;
    }
}

Thanks for investigating. Safe reset, or something like it, has to exist because memset isn't async-safe. The circular buffer design that I'm using here requires zeroing out of things.
GCC was trying to optimise it away to memset before, and we used a specific flag to disable the optimisation. AFAIK bzero is a legacy function and use of memset is preferred.
I think the best solution to this problem would be to find out what the flag is to clang to disable the "optimise reset functions to bzero" optimisation and then apply that.

Right, but I don't know what the right flag would be for Clang.
Would you mind walking me through why zeroing is necessary at all? My analysis is as follows:

safe_reset is only ever called from Profiler::handle to zero out JVMPI_CallFrame frames[kMaxFramesToCapture], which is a local variable
frames is used as a field in trace passed as an argument to AsyncGetCallTrace, which fills in frames
after the call to AsyncGetCallTrace, frames is passed to CircularQueue::push
frames is not read at all in CircularQueue::push, but is passed to CircularQueue::write
copies of lineno and method_id from each element of frames are made into frame_buffer_[slot]; there is a comment in CircularQueue::write that says "Padding already set to 0 by the consumer", but I'm not sure what that means

Unless AsyncGetCallTrace needs frames to have been pre-zeroed by the caller, as far as I can tell zeroing frames has no functional consequence: all the fields in frames that are read should have been written to by AsyncGetCallTrace, overwriting their previous value. I had a quick look at the Hotspot source and I don't see mention of a requirement for pre-zeroing before calling AsyncGetCallTrace (and a cursory examination of the AsyncGetCallTrace code also suggests it does not rely on pre-zeroing, as far as I can tell).
I'm not familiar with the history of why things are set up in this way, so please let me know if I'm missing something.

Sure, finding the correct flag will need a bit of research if we go down that route.
For further explanation of the padding comment, see issue #77. For more information on struct padding see this SO question - http://stackoverflow.com/questions/5397447/struct-padding-in-c.
The safe_reset function's declaration actually includes a comment that explains the zeroing out issue and has been in the codebase ever since I took Jeremy Manson's original prototype.
// We have to set every byte to 0 instead of just initializing the
// individual fields, because the structs might be padded, and we
// use memcmp on it later.  We can't use memset, because it isn't
// async-safe.

Now an interesting observation is that we actually don't memcpy the structs anymore and haven't for an absolute age. So we can possibly just initialize the fields. We do need to initialize the fields though because otherwise we're passing a pointer to uninitialized data to a function. If we do make that change though - we'll /still/ have to check that clang and GCC don't try to optimise/breakify everything to memset or bzero.

Overall my point is that AsyncGetCallTrace does not seem to need its JVMPI_CallTrace argument zeroed out. It seems to me that it's fine to pass  to a function a pointer to uninitialized memory, as long as the function only ever writes to that memory.
If one wants to make extra sure that no uninitialized data are written out because of struct padding, it does make sense to zero out records in the queue. This comment in issue #77 suggested having the zeroing be done by the consumer instead of the producer, which I think makes good sense.
To summarize, removing the zeroing in Profiler::handle should not affect the output of the profiler, but if anything it should improve performance (which is especially welcome in the signal handler context), and it would do away with the async safety problem.

@cinquin I think you are correct and that safe_reset is not required.

Also, given that we currently ALWAYS publish a trace, we can avoid copying all together and use the queue allocated call trace. This will end up being a claim/release kind of interface rather than the current 'push'.

@nitsanw I agree that a claim/release interface would be nice since it would minimize unnecessary copying. It would require changes in the CircularQueue code though. To avoid making things more complex, for now I submitted pull request #110 that just removes safe_reset. I did not touch the GCC compile option in case it has effects in other places.

Thanks for the discussion gents.
re: "This comment in issue #77 suggested having the zeroing be done by the consumer instead of the producer, which I think makes good sense." - it actually already does, that refers to zeroing out the data structure in the queue itself though.
I'm happy with @nitsanw's suggestion of just passing a pointer to the queue directly with a claim/release API, makes sense from an efficiency point of view.
I'll take a look into the uninitialized data situation. I was under the impression that passing uninitialized data to a function had a boatload of undefined behaviour. If that's not correct then I'm happy to accept #110, otherwise I think the pointer still need initializing.

I've accepted the pull request.

Great; thanks.
I've updated my other pull request #108  per your comments. With these two pull requests, honest profiler should work pretty well both under Mac OS X and FreeBSD.


have no access to homework-link

Continutul enuntului il puteti gasi la urmatorul link:
link enunt
===>>
404 Not Found
The requested URL /ui/TODO was not found on this server.
Apache/2.2.22 (Debian) Server at vmchecker.cs.pub.ro Port 443

Hello.  This link not work. The corect link is : https://ocw.cs.pub.ro/courses/alf/teme/tema_en_1
Or if you want the link in French : https://ocw.cs.pub.ro/courses/alf/teme/tema1
Success at homework.


Introducing Copier Guru on Gurubase.io

Hello Copier team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the Copier Guru to Gurubase. Copier Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "Copier Guru" badge, which highlights that Copier now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable Copier Guru in Gurubase, just let me know that's totally fine.

Interesting, thanks for the heads up. I asked a few questions. The results ranged from good to not bad but incomplete :slightly_smiling_face:

We have an exciting release coming up in a few days that will significantly improve answer accuracy. I would love to hear your thoughts after it's out—would that work for you?

the most frequently asked ones

Nice, this would be valuable to maintainers. We can already get similar results from search engines, but I suppose questions on a service like Gurubase will be much more detailed.


"Sites you've probably visited" Returns Inaccurate Results

I landed on the site using Chrome 46.0.2490.80 and it did not load a single site that I've actually visited with the exception of a few like Reddit and Quizlet which I feel it would be incorrect to assume are accurate results simply because of the scale/traffic those sites get.
I'm running Chrome with the following extensions:

Application Launcher for Drive (by Google) 3.2
Devtools Terminal 0.4.0
Evernote Web Clipper 6.7
Ghostery <IP_ADDRESS>
Google Art Project 1.0.37
HTTPS Everywhere 2015.8.13
LastPass: Free Password Manager 3.2.41
Mailvelope 1.2.2
Pushbullet 278
Stylish 1.4.0
Tampermonkey 3.11
uBlock Origin 1.2.0

Having also tried loading the site with an exception from HTTPS Everywhere, the result was the same.

an exception from HTTPS Everywhere

Have you tried disabling HTTPS Everywhere completely?

Disabled HTTPS Everywhere, the returned list was 12 sites that I've never visited.

I suspect this is due to interference with your addons (which is noted as a limitation in the README, fwiw); try opening an incognito window and seeing whether the results change. Addons are disabled by default in incognito but you can check by going to chrome://extensions.

Disabled HTTPS Everywhere <-- LOOL if this fix the issue i won't disable it.

It doesn't fix the issue in general, though it might prevent it for some cases.

How is this a "crappy test"? It's a privacy issue that can be taken advantage of by users that don't have their browsers properly configured.


TypeError: (0 , _prettyFormat.default) is not a function with TypeScript + Babel

🐛 Bug Report
There looks like there may already be a fix for this issue https://github.com/facebook/jest/pull/11360, but it was merged a while ago, not sure if this change has made it into a release yet, or perhaps this change is actually the cause of the problem?
jest-circus calls pretty-format.format but when pretty-format is transformed by babel via preset-env, there is no format function attached.
Logging the value of prettyFormat in node_modules/jest-circus/build/formatNodeAssertErrors.js gives:
[Function: prettyFormat] {
      plugins: {
        AsymmetricMatcher: { serialize: [Function: serialize], test: [Function: test] },
        ConvertAnsi: { serialize: [Function: serialize], test: [Function: test] },
        DOMCollection: { serialize: [Function: serialize], test: [Function: test] },
        DOMElement: { serialize: [Function: serialize], test: [Function: test] },
        Immutable: { serialize: [Function: serialize], test: [Function: test] },
        ReactElement: { serialize: [Function: serialize], test: [Function: test] },
        ReactTestComponent: { serialize: [Function: serialize], test: [Function: test] }
      }
    }

I'd love to be able to monkeypatch a workaround in jest.setup.js like so:
import prettyFormat from './node_modules/jest-circus/node_modules/pretty-format'

prettyFormat.format = prettyFormat

But this doesn't work, seems to be a different pretty-format instance.
envinfo
npx envinfo --preset jest

  System:
    OS: macOS 11.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Binaries:
    Node: 14.17.0 - ~/.nvm/versions/node/v14.17.0/bin/node
    npm: 6.14.13 - ~/.nvm/versions/node/v14.17.0/bin/npm
  npmPackages:
    jest: ^27.0.4 => 27.0.4

FWIW I'm investigating a similar issue. I see in the yarn.lock that there is an older (26.x) version of pretty-format pulled in by other dependencies.  I haven't found a solution yet, but maybe this could help you debug on your end @timoxley
The project I'm on uses yarn workspaces, so I'm trying to wrap my head around how come an older version of pretty-format is finding it's way to one of the sub-packages while the latest is in the top-level node_modules...

@jmarek-sky I believe it'll still be installed if different versions are required. I think to avoid that you need overrides set in package.json (I'm not familiar with yarn workspaces, but I've been using npm workspaces and lerna in separate projects).


/home/ubuntu/torch/install/bin/luajit: cannot open <> in mode  w at /home/ubuntu/torch/pkg/torch/lib/TH/THDiskFile.c:649

ubuntu@ip-Address:~/cnn-benchmarks$ th convert_model.lua -input_caffemodel VGG16_SOD_finetune.caffemodel -input_prototxt VGG16_SOD_finetune_deploy.prototxt -backend nn
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 538683157
Successfully loaded VGG16_SOD_finetune.caffemodel
conv1_1: 64 3 3 3
conv1_2: 64 64 3 3
conv2_1: 128 64 3 3
conv2_2: 128 128 3 3
conv3_1: 256 128 3 3
conv3_2: 256 256 3 3
conv3_3: 256 256 3 3
conv4_1: 512 256 3 3
conv4_2: 512 512 3 3
conv4_3: 512 512 3 3
conv5_1: 512 512 3 3
conv5_2: 512 512 3 3
conv5_3: 512 512 3 3
fc6: 1 1 25088 4096
fc7: 1 1 4096 4096
fc8-SOD100: 1 1 4096 100
/home/ubuntu/torch/install/bin/luajit: cannot open <> in mode  w at /home/ubuntu/torch/pkg/torch/lib/TH/THDiskFile.c:649
stack traceback:
        [C]: at 0x7f1d7a3567e0
        [C]: in function 'DiskFile'
        /home/ubuntu/torch/install/share/lua/5.1/torch/File.lua:385: in function 'save'
        convert_model.lua:36: in main chunk
        [C]: in function 'dofile'
        ...untu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
        [C]: at 0x00405d50
ubuntu@ip-Address:~/cnn-benchmarks$

I get the same error with or without -backend nn. Not sure how to resolve it.

You need to add the flag -output_t7 giving the path where you want the output file written.
I probably should put better error handling in this script.

It works now! Thanks for the help!

@ProGamerGov @jcjohnson . I faced the same error. Could u tell me where should I add the flag -output_t7?

@ProGamerGov . where should I add the flag, please?


Increment Badge

Send { "badge": "Increment" } while submitting to Parse, increments badge icons on iOS.

PR https://github.com/bulletind/khabar/pull/81


images with links

Although the content on this page is useful, the screenshots are tiny and they are not linked to a full size screenshots. Any chance that you could fix it please?

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

ID: b58981cc-c653-509e-2bbf-3d2d0231b18c
Version Independent ID: 7aa12d46-4f12-e06a-407d-7916e1748b69
Content: Threat Trackers - New and Noteworthy - Office 365
Content Source: microsoft-365/security/office-365-security/threat-trackers.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@MSFTTracyP  Please add your comment and help us in resolving this issue. Thanks.

@MSFTTracyP gentle reminder please help us resolving this issue. Thanks.

@hetzbh Thanks for your feedback on this. I'll look into scheduling a change to this article, time and workload permitting. My apologies for delays.

@hetzbh Thanks for your feedback on this. I'll look into scheduling a change to this article, time and workload permitting. My apologies for delays.

The images in the article have been updated and can be expanded.
Closing this issue.
FYI: @yogkumgit


Support for Python 3.6+

Hi, @DeanWay .
First of all, thanks for the initiative!
Would it be possible to also support Python 3.6? In addition to being a personal need, this would be better aligned with FastAPI's requirements.
Cheers,
Igor.

version 0.7.0 officially supports python 3.6


add source for errors in generated services

This currently prints:
PS C:\Users\cataggar\io\azure-sdk-for-rust\services\svc\batch> cargo run --package azure_svc_batch --example create_task_thiserror -- a b c d e
    Finished dev [unoptimized + debuginfo] target(s) in 0.52s
     Running `C:\Users\cataggar\io\azure-sdk-for-rust\services\target\debug\examples\create_task_thiserror.exe a b c d e`
creating job
- batch error
- Failed to execute request: policy error
error: process didn't exit successfully: `C:\Users\cataggar\io\azure-sdk-for-rust\services\target\debug\examples\create_task_thiserror.exe a b c d e` (exit code: 1)

You can see policy error printed, but it doesn't have a source. That is fixed in #613. After that is merged:
PS C:\Users\cataggar\io\azure-sdk-for-rust\services\svc\batch> cargo run --package azure_svc_batch --example create_task_thiserror -- a b c d e
    Finished dev [unoptimized + debuginfo] target(s) in 0.50s
     Running `C:\Users\cataggar\io\azure-sdk-for-rust\services\target\debug\examples\create_task_thiserror.exe a b c d e`
creating job
- batch error
- Failed to execute request
- policy error
- failed to execute request
- error sending request for url (https://a.b.batch.azure.com/jobs?api-version=2022-01-01.15.0): error trying to connect: dns error: No such host is known. (os error 11001)
- error trying to connect: dns error: No such host is known. (os error 11001)
- dns error: No such host is known. (os error 11001)
- No such host is known. (os error 11001)
error: process didn't exit successfully: `C:\Users\cataggar\io\azure-sdk-for-rust\services\target\debug\examples\create_task_thiserror.exe a b c d e` (exit code: 1)

You can see the reqwest::Error is printing is printing the sources in its error message: :/
- error sending request for url (https://a.b.batch.azure.com/jobs?api-version=2022-01-01.15.0): error trying to connect: dns error: No such host is known. (os error 11001)

I'm guessing here: https://github.com/seanmonstar/reqwest/blob/c6eb2c4fcbc3934b119772e0997d91671ff68dc7/src/error.rs#L192-L194


fixed wrong semaphore version

semaphore v2.19.10 do not exist in the docker registry, it was a typo that was mistakenly published by the developer (https://hub.docker.com/r/semaphoreui/semaphore/tags?page=&page_size=&ordering=&name=v2.19.10)
Published versions available: https://hub.docker.com/r/semaphoreui/semaphore/tags
ref: https://github.com/semaphoreui/semaphore/issues/2154

Summary by CodeRabbit

Chores

Downgraded the Semaphore service image version, which may affect application performance and reliability.

Thank you for helping out and fixing this. The CI is failing because the config.json needs to have the same version.

@dragonfire1119 whoops! sorry, fixed that 😄


Add: landing page starter & personal website

Hi,
just added a landing page starter and my personal site to the lists - hope that's cool.

https://awesome11ty.ttntm.me
https://ttntm.me

Thank you!


the new rangeOfOnes implementation reveals some bugs

when roaringarray.go's func rangeOfOnes(start, last int) container is implemented purely with bitmaps, arrays, or rle16 containers, I see test suite failure. Here are the 3 options:
For context: this
https://github.com/RoaringBitmap/roaring/blob/master/roaringarray.go#L42
is being replaced by:
 func rangeOfOnes(start, last int) container {
       return newBitmapContainerwithRange(start, last) // option 1
       //return newArrayContainerRange(start, last) // option 2
       //return newRunContainer16Range(uint64(start), uint64(last)) // option3
 }

and here are the suite fails, which are different in each case:
bitmaps for runs, test failures:

func TestDoubleAndNotBug01(t *testing.T)  fails, line 1681

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 1681:
  Expected: true
  Actual:   false

----------

arrays for runs: 2 distinct fails:

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 1616:
  Expected: 'true'
  Actual:   'false'
  (Should be equal)

--- FAIL: TestDoubleAdd2 (0.01s)

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 1629:
  Expected: 'true'
  Actual:   'false'
  (Should be equal)

--- FAIL: TestDoubleAdd3 (0.01s)

----------
with rle16:

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 787:
  Expected: '96617'
  Actual:   '9'
  (Should be equal)

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 865:
  Expected: '35392'
  Actual:   '1928'
  (Should be equal)

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 886:
  Expected: '131999'
  Actual:   '131351'
  (Should be equal)

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 933:
  Expected: 'true'
  Actual:   'false'
  (Should be equal)

Failures:

  * /Users/jaten/go/src/github.com/glycerine/roaring/roaring_test.go
  Line 1589:
  Expected: 'true'
  Actual:   'false'
  (Should be equal)

--- FAIL: TestFlipBigA (0.12s)

@lemire: it seems quite strange that your patch https://github.com/glycerine/roaring/pull/1 would induce these failures, so I suspect that they were latent all along. I also suspect that they have to do with container conversion rules and expectations, since that's really all that would be changing here.  Any clues would help.
Any other thoughts?

Opened https://github.com/RoaringBitmap/roaring/issues/75 instead; better to keep things centralized.


new propmon SecurityCouncilManagerTimelockStage

L2TimelockExecutionSingleStage doesn't pick these up, so we're good to make another ProposalStage for the manager

is working: https://arbiscan.io/tx/0xaef387c63698d071411c7ea0cc33f3ec9140e23d7f8bf66736b6ce1c6da87db8
it only executed the last update message in the L2 timelock

some weird bugs actually, moving back to draft


UnserInfo button should be renamed

It should display the proper term; "User Information"

decided to rename as "Export"


New websites and metadata issue

Hi, and thank you for this amazing work. Could I ask you to add these other streaming websites please?

altadefinizione.democrat
filmsenzalimiti.food

Since they added some eurostreaming.delivery captchas, what do you think about using selenium for scraping? Because that site is really the primary resource in Italy for TV series. The other sites don't have series unless they're just released. I don't know how to program well yet, otherwise I would have gladly contributed. Thank you very much.
ps. I have a question: does MammaMia use imdb to search for metadata? I ask because it happened that after searching on trakt or tmdb from the stremio search bar, MammaMia opened another film. Actually, it happens for many films, unfortunately. Take for example the animated Beauty and the beast of 1991. Via streamingcommunity it finds me another film, while from lordchannel it takes the right cartoon. What's the matter?

Both sites are just clones of GuardaHD which is added by Stremify. For eurostreaming I won't use Selenium:
1 It is heavy, and not ideal on  a server that needs to serve contents to a lot of people
2 It won't help, even if you render javascript you have to complete a captcha
Yes I use tmdb/imdb for fetching metadata. Usually that thing with StreamingCOmmunity only happens when StreamingCommunity doesn't have that movie but it has one with similar name and the same release year

Hi,

I invite you to try opening Beauty and the Beast, the 1991 animated film. It opens up something completely different and the other existing Beauty and the Beast films are all of different dates. And trust me, it's not the only case, I wouldn't have reported it to you otherwise. There are also cases of movies or TV series, already present on StreamingCommunity, that have a name but open up a completely different movie, and there are quite a few cases unfortunately. It's just that it gets confused. Once you've searched for a movie, you should try to make it do the search differently by working with the movie id itself, maybe.
Could you also add TV2000 to Italian TV?
When could you implement the possibility of hiding the TVChannels section from the stremio home?
In fact Eurostreaming is also on Stremify, unfortunately it never works nor finds out anything, even with the latest fix: https://github.com/stremify/stremify/tree/main/server /routes
With seleniumbase, a lighter version, I can accept captchas from heavy sites like unknowncheats after logging in with chrome windowless mode.

Thank you anyway, appreciate your availability.

1 I will look into it if it's actually on Streaming Community website. I can not work directly with movie IDs cause the site doesn't support them.
2 I don't know, maybe if I find a source easy enough to add
3 this is in WIP
4 the eurostreaming on stremify is just a clone of guardaserie, not the real one.
And still seleniumbase is not really suitable

Tried Beauty and the Beast, and it correctly opens the one that is on Streaming Community Website. Be sure to disable Fast Search, that is almost certainly why it doesn't get the right one.


upgrade hoard to latest & add better conf

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>What this PR does / why we need it:
Updates the hoard chart to point to the most recent release. It also includes better configuration of the supported backends.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @compleatang

All the above are documentation requests. this is fine to test.
/ok-to-test

/lgtm

/retest

/test all

/check-cla

/cc @k8s-ci-robot

/ok-to-test

/ok-to-test

/test all

/retest


Sidekiq::Scheduler.schedules

Hi
I am using
gem 'rails', '4.2.6'
gem 'sidekiq', '4.1.1'
gem 'sidekiq-scheduler', '~> 2.0'
I am trying to see all the schedules jobs.
Sidekiq::Scheduler.schedules
NoMethodError: undefined method schedules' for Sidekiq::Scheduler:Class Did you mean?  scheduled_jobs 	from (irb):23 	from/.rvm/gems/ruby-2.3.0@wms/gems/railties-4.2.6/lib/rails/commands/console.rb:110:in start'
from .rvm/gems/ruby-2.3.0@wms/gems/railties-4.2.6/lib/rails/commands/console.rb:9:in `start'
from .rvm/gems/ruby-2.3.0@wms/gems/railties-

Hi @sahilchopra ,
Use Sidekiq.get_all_schedules for that. It returns a Hash containing the schedules configuration

Thnx


Help

OS/Web Information

Web Browser:
Local OS:
Remote OS:
Remote Architecture:
code-server --version:

Steps to Reproduce

Expected

Actual

Logs

Screenshot

Notes

This issue can be reproduced in VS Code: Yes/No

Who is hacking me and how do I hack them back? I don't have an administrator but it says I do for box and Google


Xiaomi Sensor Door and Window sensor v2

Home Assistant release with the issue:

arch
armv7l

dev
false

docker
false

hassio
false

os_name
Linux

python_version
3.7.5

timezone
Europe/Samara

version
0.101.2

virtualenv
true

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Raspberry 3B+
Integration:

https://www.home-assistant.io/integrations/xiaomi_aqara/
Description of problem:
Mi Door and Window sensor v2 - lumi.sensor_magnet.v2 is working good in Mi Home App - journal is filled with events of opening and closing.
Sensor is identified by HASS as binary_sensor.door_window_sensor
Hass is identified switches and lights ok - they are turning on and off
But automation for door is not working and state of door sensor is stuck by off
- id: '1572979445285'
  alias: Fridge Door Open
  description: ''
  trigger:
  - entity_id: binary_sensor.door_window_sensor_158d00027b3ba7
    from: 'off'
    platform: state
    to: 'on'
  condition: []
  action:
  - alias: ''
    data:
      message: Door is opened
      title: Fired
    service: persistent_notification.create

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
xiaomi_aqara:
  discovery_retry: 5
  gateways:
    - host: 'xxxxx'
      mac: 'xxxxx'
      key: 'xxxxxx'

Traceback (if applicable):

Additional information:

@Danielhiversen @syssi
I tried Domoticz one year ago and this particular sensor worked there
https://www.domoticz.com/wiki/Xiaomi_Gateway_(Aqara)
How I can compare them, where to look?

Switched to Domoticz - it works fine.

@bongiozzo Which version of the Xiaomi Aqara gateway do you own? Could you capture the open/close event via tcpdump?

Capturing the traffic between Domoticz and the Xiaomi Gateway would be sufficient.

Can be closed as it works ok on new fresh install.


refactor: fix resource names.  Alias old names

:tada: This PR is included in version 1.9.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Upgrade for GO results in segfault

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[ ] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)

I did brew update and then brew upgrade

What happened (include command output)

/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.7/lib/ruby/2.3.0/pathname.rb:388: [BUG] Segmentation fault at 0x0000000000000000
ruby 2.3.7p456 (2018-03-28 revision 63024) [universal.x86_64-darwin9.0]
See also the crash report. While doing brew upgrade the bug arose at
ruby_2018-09-21-155044_andwen-5.crash.txt
ruby_2018-09-21-160243_andwen-5.crash.txt
==> Upgrading go
==> Downloading https://homebrew.bintray.com/bottles/go-1.11.high_sierra.bottle.tar.gz
==> Downloading from https://akamai.bintray.com/d2/d2dfd9846bc03498dd69c01eaccc0a3018a66d00864d7433d70f07645aac6a40?__gda__=exp=1537538495~hmac=7c1b9617b9ebe14ca0d68f3b649a76803116c07cf28829ebc40809670c84254c&response-content-disposition
######################################################################## 100.0%
==> Pouring go-1.11.high_sierra.bottle.tar.gz
==> Caveats
A valid GOPATH is required to use the `go get` command.
If $GOPATH is not specified, $HOME/go will be used by default:
  https://golang.org/doc/code.html#GOPATH

You may wish to add the GOROOT-based install location to your PATH:
  export PATH=$PATH:/usr/local/opt/go/libexec/bin
==> Summary
🍺  /usr/local/Cellar/go/1.11: 9,273 files, 403.9MB

What you expected to happen

hat GO will be upgraded

Step-by-step reproduction instructions (by running brew commands)

as mentioned above

[ ]  ran brew config and brew doctor and included their output with your issue?

We need this.

why are you closing this issue? It took me half an hour to get all info and I hoped it would help you.  Telling me that I missed sth is ok. But closing it because of that is simply demotivating.

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information.

That's why; we can't help you if you don't provide the information. You're asking us for help and we can only do that if you provide the information. We can consider reopening this once we have the information that we need and if it's not a duplicate of an existing issue.

In this case it's almost certainly a duplicate of https://github.com/Homebrew/brew/issues/4923

thanks for coming back to me. I followed the instructions in #4923 and ran brew doctor, brew update and brew upgrade again without an error. Thanks @MikeMcQuaid


Release 3.7.0

Codecov Report

Merging #955 into master will increase coverage by 0.05%.
The diff coverage is 97.42%.

@@            Coverage Diff             @@
##           master     #955      +/-   ##
==========================================
+ Coverage   96.61%   96.66%   +0.05%
==========================================
  Files         178      188      +10
  Lines        4964     5314     +350
  Branches      473      500      +27
==========================================
+ Hits         4796     5137     +341
- Misses        112      117       +5
- Partials       56       60       +4

Impacted Files
Coverage Δ

datahub/search/serializers.py
93.33% <ø> (-1.12%)
:arrow_down:

datahub/core/constants.py
100% <ø> (ø)
:arrow_up:

...ub/core/management/commands/loadinitialmetadata.py
96.96% <ø> (+3.03%)
:arrow_up:

datahub/investment/proposition/serializers.py
100% <100%> (ø)

datahub/company/timeline/client.py
100% <100%> (ø)

datahub/investment/proposition/urls.py
100% <100%> (ø)

datahub/omis/order/validators.py
95.32% <100%> (ø)
:arrow_up:

datahub/omis/order/admin.py
94.78% <100%> (ø)
:arrow_up:

datahub/company/urls.py
100% <100%> (ø)
:arrow_up:

datahub/investment/proposition/constants.py
100% <100%> (ø)

... and 33 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 e0eeaf0...cc4376d. Read the comment docs.


OrderExpiryTime not being set when creating an order

OrderExpiryTime is not being set when creating an order. Instead the maximum time specified in the documentation is being set.
request := cashfreeSDK.CFOrderRequest{
			OrderId:       &oid,
			OrderAmount:   sku.Price,
			OrderCurrency: "INR",
			//TODO : Notify URL
			CustomerDetails: cashfreeSDK.CFCustomerDetails{
				CustomerId:    data.UID,
				CustomerEmail: user.Email,
				CustomerPhone: data.Phone,
				CustomerName:  user.Name,
			},
			OrderTags:       &oTags,
			OrderExpiryTime: &expiryTime,
		}

Hi @TUGTEN Can you share a sample expiry time you are using

expiryTime := time.Now().Add(time.Minute*15 + time.Second*10).Format(time.RFC3339) produces 2023-10-09T10:48:23+05:30.
In the sdk, implementation/client.go:641, the openapi generated file seems to be setting an incorrect key: expiry_time instead of order_expiry_time
createOrder := map[string]interface{} {
				"order_id": orderCreate.OrderId,
				"order_amount": orderCreate.OrderAmount,
				"order_currency": orderCreate.OrderCurrency,
				"customer_details": map[string]interface{} {
					"customer_id": orderCreate.CustomerDetails.CustomerId,
					"customer_email": orderCreate.CustomerDetails.CustomerEmail,
					"customer_phone": orderCreate.CustomerDetails.CustomerPhone,
					"customer_bank_account_number": orderCreate.CustomerDetails.CustomerBankAccountNumber,
					"customer_bank_ifsc": orderCreate.CustomerDetails.CustomerBankIfsc,
					"customer_bank_code": orderCreate.CustomerDetails.CustomerBankCode,
					"customer_name":orderCreate.CustomerDetails.CustomerName,
				},
				"expiry_time": orderCreate.OrderExpiryTime,
				"order_note": orderCreate.OrderNote,
				"order_tags": orderCreate.OrderTags,
				"order_splits": orderCreate.OrderSplits,
			}

Fix that works:
createOrder := map[string]interface{} {
				"order_id": orderCreate.OrderId,
				"order_amount": orderCreate.OrderAmount,
				"order_currency": orderCreate.OrderCurrency,
				"customer_details": map[string]interface{} {
					"customer_id": orderCreate.CustomerDetails.CustomerId,
					"customer_email": orderCreate.CustomerDetails.CustomerEmail,
					"customer_phone": orderCreate.CustomerDetails.CustomerPhone,
					"customer_bank_account_number": orderCreate.CustomerDetails.CustomerBankAccountNumber,
					"customer_bank_ifsc": orderCreate.CustomerDetails.CustomerBankIfsc,
					"customer_bank_code": orderCreate.CustomerDetails.CustomerBankCode,
					"customer_name":orderCreate.CustomerDetails.CustomerName,
				},
		                "order_expiry_time": orderCreate.OrderExpiryTime,      // + + + FIX
				"order_note": orderCreate.OrderNote,
				"order_tags": orderCreate.OrderTags,
				"order_splits": orderCreate.OrderSplits,
			}

Please use the latest version (v2.1.3) and try
Closing the ticket for now. Please raise a new ticket if the issue still persists


Upgrade to ww 0.3.1

Pull Request Description
Fixes #2035
Fixes #2019
Fixes #2285
Fixes #1955
Feature branch tracking the woodwork upgrade. Creating it now so that I can track if we ever fall behind main.
Roadmap:

[x] Demo datasets, utils, and preprocessing
[x] Data Checks
[x] Objectives (this will require updating the confusion matrix impl)
[x] Components (will skip stacked ensembler unit tests since they take in pipelines)
[x] Pipelines (will add stacked ensembler unit tests back in)
[x] automl
[x] model_understanding
[x] Docs

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

Just as a heads up (I think i've brought this up before), in infer_feature_types there is a copy call. If a user passes a 2GB dataset to AutoML, will this take up ~4GB of memory while running AutoML Search?

@gsheni I totally get what you mean!
Our components already copy the data (I believe this is because we have a convention that we should not modify the user's data) so I don't think the copy in infer_feature_types is "new behavior". This change just makes it so that the copy happens in one place, so we can get rid of the copy in our components:
Imputer

LSA

I think you have a point that our convention may cause problems for large datasets. But I don't think that's specific to the ww upgrade. I'd be happy to continue that conversation in a different issue!

    👀
      👀
🤯 😁

 👀
 👀
🤯 😁

I think it's nuts that the overall net line count is just "-2"


Tekken 7 (Nvidia to AMD)

Is it possible to Spoof all Nvidia Cards to AMD through DXVK
Cause im dying to Play Tekken 7 without getting 30fps and 40fps
some reason Spoofing Card does not work for me
(DXVK_CONFIG_FILE=/home/arnold/.local/share/Steam/steamapps/common/TEKKEN 7/TekkenGame/Binaries/Win64/dxvk.conf %command%)
When i enter this and try to launch the Conf with the 2 lines inside the game wont load
So is it possible to Spoof all Nvidia card from your Side
and if possible please give us a side version.

This is already being done in DXVK 0.93.

dang guess my GTX 960 cant pull of a 60fps on Tekken 7 on Ubuntu
Nooooooooo
thx for the quick reply

try tweaking the nvidia control panel and your system, there is an old video of a guy playing at 60fps with a GT 1030, it may be possible to get better performance with that gpu

I'm under the impression that Pascal GPUs run much better than the older ones in general, I have a GTX 670 and it doesn't do nearly as well as it should compare to my RX 480.

my RX 480 570 and 580 works soooooo sweet but my 960 makes me wanna cry

i Notice in some DX9 Games or Unreal Engine 3 Games
When i turn off Rendering Threads i get so much FPS boost Maybe wine cant do it properly

i got 3-4 more FPS in TEKKEN 7
Keep it up Slow and steady and Slow and Steady

Sweet Sweet Another 3-5 FPS increase
Im getting 37-40 fps in Tekken 7 Arctic Stage
Come on Come ON a lil more upgraders Proton 3.16-6


Editing the Localization.TextResources.resx file causes namespace for TextResources to change

If you edit the TextResources.resx files, than any code that refers to TextResources fails to compile with:
Error CS0103: The name 'TextResources' does not exist in the current context (CS0103) (XamarinCRM)
I noted that the namespace in TextResources.Designer.cs file is XamarinCRM.Localization but this namespace is in no "using" statement anywhere where TextResources is referenced in code.
Adding "using XamarinCRM.Localization" to all files that reference TextResources resolves the issue.

I see this is because the generated TextResources.designer.cs file does have namespace XaarinCRM initially, but this gets changed to XamarinCRM.Localization on  build.
I suspect this might have been that the TextResources.resx file was moved into the Localization folder and project was not built before committing to source control. So when the TextResources.designer.cs is generated at build time the namespace is based on the location of the TextResources.resx file, thus getting the new namespace XamarinCRM.Localization.

#117


Improve hooks in nuxt.config.js

Related docs PR: https://github.com/nuxt/docs/pull/716
Currently they're set as follows:
hooks: {
  'build:done' (ctx) { }

With this change, they will also be able to set using hierarchical syntax:
hooks: {
  build: {
    done(ctx){ }
  }
}

I've already submitted a PR do nuxt/docs with a new section about it.

Codecov Report

Merging #3766 into dev will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #3766      +/-   ##
==========================================
+ Coverage   97.91%   97.92%   +0.01%
==========================================
  Files          18       18
  Lines        1149     1158       +9
  Branches      310      311       +1
==========================================
+ Hits         1125     1134       +9
  Misses         23       23
  Partials        1        1

Impacted Files
Coverage Δ

lib/core/nuxt.js
95.04% <100%> (+0.48%)
: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 eb114a4...50a62fb. Read the comment docs.

Good to me, could you please make a PR to nuxt/docs and update the release note as well @galvez ?


Add suggested improvements to Data Bias interactive.

[x] Fix line spacing (thanks @eAlasdair)
[x] On first level, fade to grey instead of snapping
[x] Try Black or White for initial background - so it's not immediately obvious there's a trick going on
[x] Use a border
[ ] Potentially alter wording - waiting on @caitlind

Decision yet to be made on whether we start with black or white or a random colour.
The coloured box still jumps around on mobile at the moment.
Resolves #1190

I've added my changes to the text, opinions on what I've written?

First stage: "Forced perspective like this can influence what data you collect, and can bias overall results."
Slider stage: "Changing your perspective can decrease data bias."

I had to change to formatting a tiny bit on the second stage so that the text didn't shift the display downwards. Does that still look ok to everyone?
Oops, just realised I left the old text in there commented out, I'll go delete that!

I had to change to formatting a tiny bit on the second stage so that the text didn't shift the display downwards. Does that still look ok to everyone?

Looks fine, I don't think what you described should have happened though - all the text sits in a box big enough for all of it


Group sequentially numbered files

When there are many files in a same folder, it is hard to find a specific files. It might be useful if we can group certain files. Say, group
file_1.txt
file_2.txt
file_3.txt

as
file_*.txt

and allows to click into it to access specific file.
An example:

That's a neat idea, but NERDTree is already bloated beyond reason. Adding another level of grouping would be both difficult and slow.


x/tools/gopls: remote server with docker

What version of Go are you using (go version)?
go1.12.17 darwin/amd64
Does this issue reproduce with the latest release?
YES recent version of gopls 0.4.4
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/ninja/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/ninja/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/ninja/Documents/<name>/<project>/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/65/7_p4zdgn2llfqf6fvn1zymjw0000gn/T/go-build716194075=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I am trying to setup gopls for neovim with the builtin lsp. But I have a dependency on linux to build the source I cannot build it on mac. So I tried setting up docker gopls server to listen at 7050.
Dockerfile
FROM golang:1.12.16-stretch
RUN apt-get update && apt-get install -y libudev-dev

ARG GOPATH
ENV GOPATH $GOPATH

ARG MY_PWD
ENV MY_PWD ${MY_PWD}

RUN mkdir -p $GOPATH
RUN mkdir -p $MY_PWD

ENV PATH="/usr/local/go/bin:$GOPATH/bin:${PATH}"
RUN GOPROXY=https://proxy.golang.org GO111MODULE=on go get golang.org/x/tools/gopls@latest
WORKDIR $MY_PWD

CMD /go/bin/gopls  -listen=":7050"

I opened nvim and it throws the same build error as the one I got on mac.
Neovim config
:lua << EOF
require'nvim_lsp'.gopls.setup{
cmd = {"gopls","-remote","localhost:7050"};
}
EOF

I initially tried this docker setup for a simple hello world example and that worked. But now I am not sure whether it actually compiled inside docker and communicated the results outside or it just compiled in mac itself.
Gopls listen inside docker doesn't give any logs other than connected and exited (atleast when I run docker logs ).
PS: I execed into docker and built the source and it worked as expected.
What did you expect to see?
I expect gopls -remote to build the source inside docker and give back the results to the client.
What did you see instead?
I see the same build error which I got on mac, which might mean gopls is not forwarding the commands to the remote and doing it locally.

/cc @findleyr

If you're seeing connection logging from the daemon inside docker, then you are indeed connecting to the daemon and gopls should be forwarding the LSP.
What is the nature of your build error?  My guess is that this may be related to the fact that the gopls forwarder also forwards go env, meaning it forwards GOOS.  If this is likely to be the problem, you can try running neovim with GOOS=linux to confirm.
Once we figure this out, we should update the env forwarding logic: what you're trying to do seems like a valuable use-case to support.
I'll also note that there may be issues using gopls on go1.12.  Our support for Go versions older than the last two releases is best-effort, though we do make sure gopls compiles and tests pass (this is a recent addition to our CI).

BTW, I would expect there to be other problems running the gopls daemon in docker. Namely that the file URIs would be incorrect.  However, for darwin->linux it could probably be made to work.  I assume you've mounted your workspace at the exact same absolute path.
Links to outside your workspace (in GOROOT or GOPATH) will probably break.

Thanks for the help! GOOS=linux worked.
Yeah on the paths I have created absolute paths inside docker to make it work and also in docker go is installed in /usr/local/go and the GOROOT=/usr/local/go  but in my mac since I have to install a older go it is installed in<EMAIL_ADDRESS>Right now I have symlinked to /usr/local/go and changed GOROOT in mac so that both work. But it might be helpful if the go  env of the docker is getting used inside except the GOPATH.

Great, thanks for confirming. You've done something I had wanted to experiment with for a while :)
Repurposing this issue to track documenting and streamlining this use-case for the daemon.


Parameter types are incorrectly confused for parameter identifiers and color highlighted as such.

Checklist

[x] This problem exists even with the setting "C_Cpp.enhancedColorization": "Disabled"
[x] This problem exists even with all the other extensions disabled.
[x] The problem doesn't exist for C++.

The code with a problem is (A function declaration without the paramter identifier):
void fn(unsigned int);

void fn(const char[])

It looks like:

It should look like:
As you can see, in the code above with the C programming language, both int and char should be colored dark blue instead of light blue (which is used for parameter identifiers in VSC). The problem is only persistent with C and not C++. int and char being keywords should be highlighted with dark blue as all other keywords are but that isn't the case.

Another issue I noticed, unlike most IDEs, in VSC most keywords are dark blue but not all. For example the return keyword is pinkish. I don't think that's a good idea. All keywords should be the same color or at least we should be able to make them all the same color.

Looks very similar to jeff-hykin/better-cpp-syntax#200.
jeff-hykin/better-cpp-syntax@ad31cfa only fixed its cpp grammar, and indeed the issue doesn't happen when set to cpp mode.
 (c)
 (cpp)

Copied my reply from https://github.com/microsoft/vscode/issues/138695#issuecomment-990025943

Another issue I noticed, unlike most IDEs, in VSC most keywords are dark blue but not all. For example the return keyword is pinkish. I don't think that's a good idea. All keywords should be the same color or at least we should be able to make them all the same color.

@sbeve72 you can customize any theme for any language to have exactly the colors you want just by editing your settings. Short tutorial here
(I don't control the default theme btw)
Keyword is pretty broad. There's operators, control flow, entity declarations, built in primitives, built-in modifiers, built in pointers, built in macros, built in functions, type casting,  access specifiers, and more. It would be a major loss of information if the parser didn't distinguish between them. If you want to make them all the same color you're welcome to find or make a theme that does, but the capability will exist for theme designers who want to deeply color different code functionality.

Can I do anything about the bug that I mentioned in the original comment? Is there a way to customize the theme to fix that?

You can change the color, but the original thing is a legitimate bug that is broken on the parser side. Right now the C parser needs to be rewritten, and that coloring is happening because it thinks int and char are parameters instead of types


Add filter function in orde to achieve the exact opposite of filter_except

Hi,
To keep some cookies, we need to use filter_except() function on a list, but to delete cookies we need to iterate on indivudual delete() function. Addind a filter() function that is the exact opposite of filter_except() allows for use of the same logic (and eases templating VCLs when needed).
Best regards,
Thierry

Would be great to get this functionality. Any chance of rebasing this @tmagnien so that it might be possible to merge. @lkarsten is this a changeset that interests you?

Hi, if @lkarsten is interested, I can update my code, yes.

+1


WIP: Convert projects to use .NET Standard 1.1

A couple of thoughts/issues with this at the moment:

As part of this conversion, support for Windows Phone 8.0 was dropped as .NET Standard 1.1 is needed for the core functionality and Windows Phone 8.0 only supports .NET Standard 1.0. All other targets remain the same. This can be fixed, but would make the project a bit more complicated - is this a target that is desired going forward?
Support for building E2E tests with local projects or package reference is not supported with NuGet v3.0
This will require at least VS 2015 to open/edit
This updates Newtonsoft.Json to 9.0.1 as it supports .NET Standard natively. There are some obsolete methods that are being used
This specifies Xamarin.Android to us SDK 5.1 as none had been specified.
There are some binding redirect issues with iOS and Android builds. This can be fixed, I just didn't have the time at the moment.

Fixes #190

Hi @twsouthwick, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;


fix: add MetaMask Provider Detection and Initialization for MetaMaskSnapWallet

This PR fixes #268
It enhances the MetaMaskSnapWallet implementation by adding support for detecting and initializing a MetaMask provider. Key changes include:

Introducing a MetaMaskProvider interface and related utility functions (isMetaMaskProvider, getProvider, detectMetaMaskProvider, and waitForMetaMaskProvider) to handle MetaMask provider detection and initialization.
Modifying the MetaMaskSnapWallet class to utilize the new provider detection logic, ensuring the wallet initializes with a valid MetaMask provider.
Updating the init method in MetaMaskSnapWallet to asynchronously fetch and assign the MetaMask provider.
Adjusting tests to reflect these changes.

These improvements ensure that the MetaMaskSnapWallet can robustly detect and initialize MetaMask providers, enhancing the reliability and usability of the MetaMask integration. This update makes the get-starknet integration as minimalist as possible and facilitates further interaction with the starknet/get-starknet repo.

Can't happen. Provider detection has to happen before remoteEntry.js is loaded.


Store Mount Table in VoltDB is very slow

I found the proxy layer's mount table stored in VoltDB is too slow to forwarding traffic. Even worse, we cannot cache the mount table in the stateless name-nodes that will cause the cache coherence problem when rebalancing happens.
Instead, I have a very interesting and extremely fast solution to solve that. We can store the mount table in VoltDB's Zookeeper!!!! We don't need to deploy a new distributed key-value component since VoltDB already included one!
Zookeeper's port is 7181. See here: https://docs.voltdb.com/AdminGuide/HostConfigPortOpts.php
Why choose Zookeeper? Because Zookeeper enables us to deploy a Node Cache in each proxy instance (router) to cache the mount table. At the same time, Node Cache can be used to watch a Zookeeper Node. Whenever the data is modified, the Node Cache will change its state to contain the current data. https://curator.apache.org/curator-recipes/node-cache.html

echo "status" | nc  localhost 7181 | head -n 1
Zookeeper version: 3.3.3-1073969, built on 02/24/2011 03:33 GMT
VoltDB's ZooKeeper is 3.3.3.


Required validator bug with parantheses object accessing

Versions:

VueJs: 2.5.13
Vee-Validate: 2.0.5
bootstrap-vue: 1.5.1

Description:
When using a bootstrap form radio group  (https://bootstrap-vue.js.org/docs/components/form-radio)
and paranthese object accessing vee-validate sets a required field to invalid althought it's valid.
Steps To Reproduce:
By using  a radio-group field from Bootstrap
e.g.:
<b-form-radio-group                                         name="someName"                                         v-model="form['salutation']"                                         :options="['Mr.', 'Mrs.']"                                         v-validate="'required'"> </b-form-radio-group>
vee-validate set the field validation state to invalid althought form.salutation object has a value and required validation should be fullfilled.
When v-model is given without paranthese object accessing notation, validator behavior is like it should be.
e.g.:
<b-form-radio-group                                         name="someName"                                         v-model="form.salutation"                                         :options="['Mr.', 'Mrs.']"                                         v-validate="'required'"> </b-form-radio-group>

This is sadly a limitation. the validator uses the watch API under the hood which only works for simple paths, like dot delimited paths but not the [] one.
You could help the validator by providing a data-vv-value-path which should point to the component internal value state.


GRE Vocabulary Extension

Hello @harshita214 👋🏻 , as a GSSOC'22 Contributor  i would like to give my contribution to this repo by adding the GRE Vocabulary Extension .

What this Extension Do ??
These Extension help the GRE aspirants get all vocab   Word + Definition   at one place and that too in one click , sounds cool 😎right

Tech Stack  📚
HTML 🏷
CSS 🎨
ReactJS 🎁

Note  :    If you like my idea , pls assign these issue to me and comment down below so that i can start working on this COOL Extension .

@RiyaBhandari-2811, There is an extension Meaning of Word already present for the same, are you also trying to build like this. If yes then you can enhance it , if not then kindly explain your features in brief

@RiyaBhandari-2811, There is an extension Meaning of Word already present for the same, are you also trying to build like this. If yes then you can enhance it , if not then kindly explain your features in brief

My extension is specific to GRE vocab , also it contains features like :

It gives you the Synonyms + Examples + Pronunciation features .
It has clear search also , so basically my div will not just keep growing if i search for more word at single time .

The main feature is the Pronunciation one which will help user to get the pronunciation for search word correctly .

@RiyaBhandari-2811, There is an extension Meaning of Word already present for the same, are you also trying to build like this. If yes then you can enhance it , if not then kindly explain your features in brief

Nooo mine is very different , what mine extension does is when ever you click on extension  it will give you new word with its definition and example .  The one your talking about is the word search one mine is not search it is suggesting new word every time .

What happen @harshita214 ???

@RiyaBhandari-2811 then yours is similar to Word of the day But it shows constant word for time being, would you like to enhance this one and transform it by adding the above features you are suggesting and changing word each time a button is clicked

@RiyaBhandari-2811 ?

@RiyaBhandari-2811 ?

I was thinking of doing of my own , but it will get repeated . I'm working on new extension idea .
I'm trying to make it rough if it works I'll raise the new issue (I'm figuring out the code/ tech stack) .

@RiyaBhandari-2811 ?

I was thinking of doing of my own , but it will get repeated . I'm working on new extension idea .(Started my work) . I'm trying to make it rough if it works I'll raise the new issue (I'm figuring out the code/ tech stack) .

update?

@whiteHatpro I'm not working on these issue anymore other wise issues will get repeated and that's wrong .

okay @RiyaBhandari-2811 closing this issue


Eliminate dead code by using pytests.raises instead of try/except/else

This PR just includes the simplest cases to make review simple as this is my first PR to the jinja project.

Thanks, cleaning up the tests and using pytest features is really helpful, keep going if you want to. 😀

Getting everything merged that quick was a welcome surprise. Just wanted to make sure the changes were something that made sense for the repo. I'll definitely work on a few of the similar cases when I get the chance then.
Thanks for quick turnaround and the warm welcome 😀


Reworking Multicasting: share, connect, and makeConnectable

I'd like to do this instead of what I was doing over at #5432 ..
Overview

Make share configurable, such that it can be used to create things like shareReplay or people can configure the behavior however they like. If they want a shareBehavior or shareLast, they can easily make it work however they like.
Add a newconnect operator that does the "selector" version of multicast, publish, et al. (this also fixes a broken error behavior in publishReplay).
Add a new makeConnectable function that creates a ConnectableObservableLike (instead of ConnectableObservable).

What this sets us up for:
We'd have

share (NOTE: I'll leave shareReplay as just a wrapper around this, as seen in this PR)
connect
makeConnectable

Instead of

share
multicast
publish
publishLast
publishReplay
publishBehavior
refCount
shareReplay
ConnectableObservable

All "operators" at that point would actually only return Observable and never ConnectableObservable, nor would any require ConnectableObservable as a source (such as with refCount).
TODO:

[ ] Additional documentation
[ ] Additional tests
[ ] Add deprecation details to deprecated operators and paths

Other thoughts
It's possible that a better API for makeConnectable might be to return a tuple with a connect function. Like so:
const [ result$, connect ] = makeConnectable(source);
result$.subscribe(console.log);
connect();

// instead of
const result$ = makeConnectable(source);
result$.subscribe(console.log);
result$.connect();

How this "replaces" certain things
Well, the I find the prevalence of publishReplay(1), refCount() very disturbing. For one thing, I doubt many folks using this are aware of what this does completely. It's not retryable, it's not replayable, etc.
a$.pipe(publishReplay(1), refCount())

// is

a$.pipe(
  share({
    connector: () => new ReplaySubject(1),
    resetOnError: false,
    resetOnComplete: false,
    resetOnUnsubscribe: false
  });
)

"But Ben, that's more code!"... yes, and it's explicit code that tells you exactly what you're doing there. That's not retryable, it's not repeatable... it will never reset internally. It also allows the author more control without having to know implementation details of publishReplay. Needing to know implementation details of our multicasting operators has been a sore point of this library, IMO.
NO WORRIES: a$.pipe(share()) will work exactly as it does now. (Notice those tests didn't change)

Related: #3833

Rebased

Core team to review... revisit next meeting.

This is blocked on #5729

It's possible that a better API for makeConnectable might be to return a tuple with a connect function. Like so:

just throwing in other alternatives:
const notifier = new Subject();

makeConnectable(source, notifier);
notifier.subscribe(connects(source));

NOTE: Lots of work being done here, still a WIP.

Okay, I've made a few changes to this.
Per @cartant's request, makeConnectable is now connectable.
OTHER VERY IMPORTANT THINGS:

I've added a new TestScheduler helper:  expectObservable(a$).toEqual(b$). I needed this so I could test that the deprecation advice I was giving people in the deprecation messages was proven, not just theoretical. It's publicly exposed, and I think we should ship it. It's useful for this exact scenario and it follows the jest and jasmine pattern of toBe and toEqual, IMO.
I've fixed the Observer interface to remove the optional closed property, which shouldn't have been there. (That may be breaking for TypeScript users).
I've fixed the Subscribable interface to just be what is valid as a return value of [Symbol.observable](). That's because I was using with SubjectLike.
I'm now exporting a new type SubjectLike, it's exactly what it sounds like.

TODO

We need to discuss SubjectLike. We might want it to be SubjectLike<In, Out> and not just SubjectLike<T>. This relates to things like WebSocketSubject, etc.
We need to bikeshed the setup function as discussed above.

cc @cartant @kwonoj

Requesting rereview.
Note that the change to make Subscribable accept Partial<Observer<T>> also meant that the type for Observable subscribe needed to take Partial<Observer<T>> instead of PartialObserver<T>, the latter being a type that forced at least one callback (for better or worse). I'm totally fine with this, and I'm not sure that bit was helpful anyhow. If anything, I'd bet it was confusing for users at times.

🎉


Upgrade to Spring Boot 3

Upgrade to Spring Boot 3 and Spring Framework 6
javax ->jakarta migration
In addition, upgrade dependencies and adjust demos to latest spring configuration properties.
add missing maven wrapper to webflux demo

Both demo apps are working properly.

@torstenmandry Hey Torsten, I gave this a quick sanity check, looks good from my perspective. But I am personally not really involved in this project. I'm happy to merge it if someone else who is using the starter might have a look, too. In case you aren't, you maybe know someone who is. Thanks.

Thanks @rvullriede for your time and effort, appreciated!

Hi @michael-simons , thanks for your review and the comments. I've updated the PR with the following changes:

removed unneeded wrapper binaries
updated github actions to latest versions (v3) and to Java 17
updated npm dependencies used in the demos to their latest versions.

Excellent catch with the outdated actions 😅 me copy and pasting things in the night.

Thanks @rvullriede I merged this.

thx @michael-simons :-) Is there a chance to also publish a release (incl. maven central) so it's easy for people to pick it up?
I'm happy to provide a README update so it's clear which version works for which Spring Boot version.

@rvullriede https://repo.maven.apache.org/maven2/org/faucet-pipeline/faucet-pipeline-spring-boot-starter/2.0.0/
;)
And yes, I would love to have an update to the readme along the lines "1.3.0 is for Spring Boot 2.x, 2.x for Spring Boot 3.x"