serverReadyAction doesn't launch browser on VS Code for Mac in latest insider build

VSCode Version: 1.45.0-insider
OS Version: macOS Catalina 10.15.3 (19D76)

Steps to Reproduce:

Point VS Code to the project folder, allow it to add the necessary C# files for the project
Run it

Does this issue occur when all extensions are disabled?: Yes
Here's the launch.json file:
{
   // Use IntelliSense to find out which attributes exist for C# debugging
   // Use hover for the description of the existing attributes
   // For further information visit https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md
   "version": "0.2.0",
   "configurations": [
        {
            "name": ".NET Core Launch (web)",
            "type": "coreclr",
            "request": "launch",
            "preLaunchTask": "build",
            // If you have changed target frameworks, make sure to update the program path.
            "program": "${workspaceFolder}/projectname/bin/Debug/netcoreapp3.1/projectname.dll",
            "args": [],
            "cwd": "${workspaceFolder}/projectname",
            "stopAtEntry": false,
            // Enable launching a web browser when ASP.NET Core starts. For more information: https://aka.ms/VSCode-CS-LaunchJson-WebBrowser
            "serverReadyAction": {
                "action": "openExternally",
                "pattern": "^\\s*Now listening on:\\s+(https?://\\S+)"
            },
            "env": {
                "ASPNETCORE_ENVIRONMENT": "Development"
            },
            "sourceFileMap": {
                "/Views": "${workspaceFolder}/Views"
            }
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach",
            "processId": "${command:pickProcess}"
        }
    ]
}

Here's the output from the debug console:
-------------------------------------------------------------------
You may only use the Microsoft .NET Core Debugger (vsdbg) with
Visual Studio Code, Visual Studio or Visual Studio for Mac software
to help you develop and test your applications.
-------------------------------------------------------------------
Using launch settings from '/Users/me/Projects/project/projectname/Properties/launchSettings.json' [Profile 'Production']...
[14:25:46 INF] Application Starting Up
[14:25:47 DBG] Hosting starting
[14:25:47 INF] User profile is available. Using '/Users/me/.aspnet/DataProtection-Keys' as key repository; keys will not be encrypted at rest.
[14:25:47 DBG] Reading data from file '/Users/me/.aspnet/DataProtection-Keys/key-59a27b16-f86b-4e55-8b3a-6346714aae22.xml'.
[14:25:47 DBG] Reading data from file '/Users/me/.aspnet/DataProtection-Keys/key-ce84825d-06af-42c7-9c05-acb038469135.xml'.
[14:25:47 DBG] Found key {59a27b16-f86b-4e55-8b3a-6346714aae22}.
[14:25:47 DBG] Found key {ce84825d-06af-42c7-9c05-acb038469135}.
[14:25:47 DBG] Considering key {59a27b16-f86b-4e55-8b3a-6346714aae22} with expiration date 2020-06-19 18:02:29Z as default key.
[14:25:47 DBG] Forwarded activator type request from Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.ConfigurationModel.AuthenticatedEncryptorDescriptorDeserializer, Microsoft.AspNetCore.DataProtection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60 to Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.ConfigurationModel.AuthenticatedEncryptorDescriptorDeserializer, Microsoft.AspNetCore.DataProtection, Culture=neutral, PublicKeyToken=adb9793829ddae60
[14:25:47 DBG] Using managed symmetric algorithm 'System.Security.Cryptography.Aes'.
[14:25:47 DBG] Using managed keyed hash algorithm 'System.Security.Cryptography.HMACSHA256'.
[14:25:47 DBG] Using key {59a27b16-f86b-4e55-8b3a-6346714aae22} as the default key.
[14:25:47 DBG] Key ring with default key {59a27b16-f86b-4e55-8b3a-6346714aae22} was loaded during application startup.
[14:25:47 DBG] Failed to locate the development https certificate at 'null'.
[14:25:47 DBG] Using development certificate: CN=localhost (Thumbprint: 0232F2D929A1C96493D8D82169236B116021F40B)
[14:25:47 INF] Now listening on: https://localhost:5001
[14:25:47 INF] Now listening on: http://localhost:5000
[14:25:47 DBG] Loaded hosting startup assembly projectname
[14:25:47 INF] Application started. Press Ctrl+C to shut down.
[14:25:47 INF] Hosting environment: Development
[14:25:47 INF] Content root path: /Users/me/Projects/project/projectname
[14:25:47 DBG] Hosting started

Am I missing something? I normally work in Rider with the /Properties/launchSettings.json file, so I don't know if I'm supposed to be doing something else or if this feature just doesn't work on Mac.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

serverReadyAction isn't launching default browser on macOS Catalina (#93371)
Mac: Browser won't launch using serverReadyAction or launchBrowser for ASP.NET Core projects (#93517)

Those issue weren't resolved at the time. Don't know if this is a related or an entirely new issue, because now the appropriate information is now being sent to the debug console, whereas before it was not.

@AlphaComposite the problem is in the regular expression that you are using:
^\\s*Now listening on:\\s+(https?://\\S+)

The caret '^' at the beginning prevents the pattern from matching the output on the debug console:
[14:25:47 INF] Now listening on: https://localhost:5001

With the carat you are expecting whitespace at the beginning of a line, but there is a
actually a time stamp in square brackets.
Just remove the caret and it will work.


Emoji Section added in Feedback Section

Description
Emoji Section added in Feedback Section
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #804

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:

@varshith257  Kindly add suitable level to the pr.

Thankyou for your contribution


Update grpc and protobufs dep

This also regenerates the proto file for containerd so that everything builds correctly.
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

LGTM

Squash the commits before merging?


Prefix added to commit message when escape pressed

Replication Steps:
Scenario 1:

Click the extension button or use the Ctrl+Alt+Enter shortcut
Press the escape key twice to cancel
A semicolon ( : ) will be entered into the commit message

Scenario 2:

Click the extension button or use the Ctrl+Alt+Enter shortcut
Select a prefix
Press the escape key to cancel
The previously selected prefix will be entered into the commit message

Hmmm yeah i don't handle cancel at this time, I'll add it in v1.1.2  in weeks :wink:

I Handle it, you can upgrade your extension version 😉


Fix Arabic compilation error

Trying to run this as is with Ruby 2.3 gives this error:
root@79a926ee95f2:/usr/src/app# bundle exec thin -e dev -R config.ru -p 7143 start
onetime[7]: (<unknown>): did not find expected key while parsing a block mapping at line 108 column 5
Error loading locale: /usr/src/app/etc/locale/ar ((<unknown>): did not find expected key while parsing a block mapping at line 108 column 5)
onetime[7]: Error loading locale: /usr/src/app/etc/locale/ar ((<unknown>): did not find expected key while parsing a block mapping at line 108 column 5)

I don't know a character of Arabic(!) but I can see the extra quote characters, so I've replaced with single quotes, which fixes the runtime error.

Hey, thanks for this 🙏


A more generic and practical output

I guess that people want to use it like this way:
<script type="text/php">
    // php code here
</script>

so, make it as generic as possible and register it to packagist.org.
Rather than instruct people how to build it, it is more reasonable to provide the output. If my output is the same output as yours, why do i have to build my self while we have the same needs: a common php engine. Let your output be the "official" one.

My changes have been pulled into the main branch.

I'd like this functionality as well, especially since <?php makes IDEs and servers go crazy trying to render PHP.
Unfortunately, I get mixed messages from the points above - is this functionality implemented in oraoto/pib or not?
Either way, neither original repo nor the fork show an example without the PHP open tag (and, while at it, they also incorrectly show the 'equal' (in the html tag) surrounded with space.


Added a Dockerfile

Resolves #11
This pr adds a Dockerfile which is a 2 stage build with the final image based on a distroless image with a debian 12 base. It also updates the README.md with a build command and adds a Makefile to make the build more ergonomic.

The output of the make docker-build command should maybe also include a .env volume mount since that is required at runtime.
What do you think?

The output of the make docker-build command should maybe also include a .env volume mount since that is required at runtime.
What do you think?

How about instead of injecting the .env directly in to the docker file can you instead create a docker compose and lets handle the envs there? We're planning on deploying this to some sort of serverless hosting service so it will most likely end up handling the env vars.

It requires the .env injection at runtime as is, so a docker-compose file would still work with this PR. I kind of assumed the make command would only be ran in dev anyway.

A simple check for if the environment variables exist already would also work right? If any of them don't exist the program can then try and load the .env like it does now to read the rest maybe? It would make the .env useful for overriding variables too but I'm not sure how useful that would be.

@MitchellBerend https://github.com/devanbenz/bits-and-bytes-bot/pull/16 please check this pr and merge
will require you to run ENV=local go run .  to run locally and load up .env

will create a seperate issue to make a docker compose cause I wanna get the datastore setup so it can be added to the compose 👍


localize stringcommands to allow servers to use localised messages

allows for cool localised messages, including ui images within the messages

where test mod

one of the single greatest PRs to ever exist

where test mod

havent made yet

also doesnt rly need a test mod per say, a command is enough

script NSSendAnnouncementMessageToPlayer(player, "%%$rui/pilot_loadout/mods/gunrunner%% %%$r2_ui/menus/loadout_icons/attachments/suppressor%% %%$r2_ui/menus/loadout_icons/primary_weapon/primary_kraber%%", "death", <255, 0, 0>, 1, 0)
should work

code looks good and works in testing


[JOSS] Implement automated testing via CI

I think SNUB is a great tool, and has the potential to be used by quite some people!
That said, and especially given this is the case, I would strongly recommend implementing automated testing via CI (using something like travis-ci, for example). I think this will make keeping track of potential issues much easier as the package grows.
This issue is part of a paper review for JOSS.

Thanks for the suggestion! I created a github action that runs pytest and checks black formatting on pushes/PRs to main. I also created protections for main so that all merges require a PR.
Here's the github action config: https://github.com/calebweinreb/SNUB/blob/main/.github/workflows/ci.yml
Here's a log of the last completed action: https://github.com/calebweinreb/SNUB/actions/runs/8125132061/job/22207389372


Add Total Track Field

To support:
{track #} of { total track # }

https://github.com/NickvisionApps/Tagger/releases/tag/2023.8.0-rc1


[FLINK-3082] Fixed confusing error about an interface that no longer exists

The ManualTimestampSourceFunction interface does not exist.
Yet there are error messages that thell you to take a look at it.
This simply fixes these error messages.

Thanks for catching, this.
Will merge this...


Invalid logging of markers destroy

The following lines should be moved to ~180.
https://github.com/tombatossals/angular-leaflet-directive/blob/master/src/directives/markers.js#L155-L158

This issue was moved to angular-ui/ui-leaflet#98

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


Add PCA-based compression operator

Rel #180

this looks ugly...

Would it make sense to have a PCA operator which is then used by a coil-compression function?

One of the goals is to be able to first construct the compression matrix, then apply it to different data.
So we could move the application to kdata into a function

pre-commi.ci autofix

Does anybody want to take this PR?
The signature should be changed, instead of separate_dims as an argument and only working on dim -4, there should be a dim argument which dimension to compress, defaulting to -4.
And a shared_dims (tuple of int or ellipsis) with the opposite meaning of separate dims. A single compression matrix should be constructed for these dimensions, with ellipsis (default) meaning all dimensions. For dimensions not specified as shared, normal natxhing shall be used.
The data operator part should just be removed here.

@ckolbPTB are you interested?

To me separate dims seems to more intuitive parameter and easier to define of the shape of data used for calculating the compression matrix is different to the input data to forward.

I would then add a compress_coils algorithm in a separate PR

To me separate dims seems to more intuitive parameter and easier to define if the shape of data used for calculating the compression matrix is different to the input data to forward.

yes, the implementation would be easier, but using it would be a bit inconsistent, as we assume all leading dimensions are always batch dimensions.
otherwise adding another leading dimensions needs a code change to batch along that dimensions. this is not what we want.
This is why I deliberately suggested having the inverse, a joint_dim argument -- and why I tried to find somebody else to tackle this ;)

yes, the implementation would be easier, but using it would be a bit inconsistent, as we assume all leading dimensions are always batch dimensions.

I did not mean the implementation but the usage. One example woud be PCA-based compression for qMRI. Let's assume we have one dimension a which is supposed to be treated separately. We would calculate N signal curves in the shape (a,t,N) and then calculate the compression matrix for this. This compression matrix would then be applied to the data of shape (a,t,z,y,x). If I used joint_dim then I would have to specific joint_dim=-1 for the init but then change it when applying to the data. If I used separate_dims it would be 0 for both.
Also for coil compression I would like to define the separate_dim as the slice dimension and not care if I have additional leading dimensions due to dynamics or echos or something like this.

If I am not mistaken we have to specify three dimensions:

the dimension in which to compress (coils or time)  when creating the compression
dimension to compress when applying the data
dimensions to batch over vs dimensions to consider as samples on the data matrix when creating the compression (this is where we disagree) - do you want to use the same compression for different slices, subjects etc or the same?

Either we allow to specify all three, or certain applications require either unsqueezes or reaaranges.
I used the code to construct one coil compression operator based on one sample and applied it to differently shaped inputs with different batch dimensions
So it would be creation_dim=coil, joint_dim=z y x, application_dim=coil. At application this should now broadcast in the leading dimensions of the data and the same compression is applied for rotation,slice,....
In a qmri example, you would want to create the matrix with (t,N)-shaped samples, and compress in time. You want to apply it to data
( t, slice, rotation, coil,zyx)
Here the creation dim is 0, the application dim is 0 and the there is no joint_dim.
If the data were
( Subject, t, slice, rotation, coil,zyx) you would have to specify an application dimension or create the matrix with (subject or 1,t,N) samples.
Specifying separate dimensions instead of joint dimensions would not help here.

Also for coil compression I would like to define the separate_dim as the slice dimension and not care if I have additional leading dimensions due to dynamics or echos or something like this.

This breaks both pytorch and somewhat mrpro conventions: leading dimensions are considered batch dimensions, thus should be treated as separate.
If you want to share compression along dynamics, for example, you should specify that. If you then use batches of different subjects, it should treat the new 0tj dim as a separate batch dim. So the logic is exactly the opposite.

If I am not mistaken we have to specify three dimensions:

the dimension in which to compress (coils or time)  when creating the compression
dimension to compress when applying the data
dimensions to batch over vs dimensions to consider as samples on the data matrix when creating the compression (this is where we disagree) - do you want to use the same compression for different slices, subjects etc or the same?

Either we allow to specify all three, or certain applications require either unsqueezes or reaaranges.
I used the code to construct one coil compression operator based on one sample and applied it to differently shaped inputs with different batch dimensions
So it would be creation_dim=coil, joint_dim=z y x, application_dim=coil. At application this should now broadcast in the leading dimensions of the data and the same compression is applied for rotation,slice,....
In a qmri example, you would want to create the matrix with (t,N)-shaped samples, and compress in time. You want to apply it to data
( t, slice, rotation, coil,zyx)
Here the creation dim is 0, the application dim is 0 and the there is no joint_dim.
If the data were
( Subject, t, slice, rotation, coil,zyx) you would have to specify an application dimension or create the matrix with (subject or 1,t,N) samples.
Specifying separate dimensions instead of joint dimensions would not help here.

Also, thinking about it, maybe the differences between use in qmri and coil compression are greater than the common parts. And itigjt ne easier to just have to things that do a bit of (different) reshaping before applying a PCA ..?

We could just keep the separate&dim logic for know, although it is kind of a foot gun that adding a batch dimension in front of the data will make this by default a shared dimension.

But isn't this the definition of a batch dimension?

We could just keep the separate&dim logic for know, although it is kind of a foot gun that adding a batch dimension in front of the data will make this by default a shared dimension.

But isn't this the definition of a batch dimension?

It would usually be separate dimension, i.e. having a batch dimension is equivalent to having a for loop over the different batch samples. So no joint operations connecting the batch samples (beside batch norm as THE special case)
L

Example:  having a batch of three subjects should do separate coil compressions for each of the subjects.

@fzimmermann89 could you have a look at this please?

@fzimmermann89 could you merge this or is anything still missing?

I will check later and merge it.

Does anybody know if it is possible to make RearrangeOp a link in the documentation?

Does anybody know if it is possible to make RearrangeOp a link in the documentation?

I don't think I understand the question....

In the type hints etc class names are linked in the documentation to other sites.
I was trying to make the reference to RearrangeOp a clickable link in the documentation -- but failed.

vs


Replace internal get_storage_var_address function implementation with starknet_rs. Issue  #221

Usage related changes
It's transparent for users
Development related changes
Replace internal get_storage_var_address function implementation with starknet_rs. Issue  #221
Checklist:

[X ] Applied formatting - ./scripts/format.sh
[X] No linter errors - ./scripts/clippy_check.sh
[X ] No unused dependencies - ./scripts/check_unused_deps.sh
[X] Performed code self-review
[X] Rebased to the last commit of the target branch (or merged it into my branch)
[N/A] Documented the changes
[X] Linked the issues which this PR resolves
[N/A] Checked the TODO section in README.md if this PR resolves it
[N/A] Updated the tests
[X] All tests are passing - cargo test

This PR is a candidate for closing because it introduces no changes.

i think i made a mistake 😬


edits in response to review comments (issue 24) merged with checkpoint changes

There we go! The pull request looks great!


INSTALLATION.md: Fixed spelling errors in documentation

Just a small fix for two typos that I noticed in INSTALLATION.md

Nice catch, thanks for the fixes! 💯


invalid character ']' looking for beginning of value

After referencing my ServerGrid.Json I received the following error:
2019/06/17 16:56:08 invalid character ']' looking for beginning of value
This turned out to be due to a trailing ',' at the end of an array list in the sublevels section of one of the grids. Removing the trailing comma fixes the problem, though the parser shouldn't crash on this issue as the trailing symbol doesn't cause issues with the game server or other parsers.

This isn't exactly a high priority item I'm certain, but for others having issues with the new viewer this provides both the fix and the reason which is why I opened it.

Yea im getting something similar but can't locate it in mu json :(
https://github.com/GrapeshotGames/AtlasMapViewer/issues/18

You can use an online json validator to help find the error if you're having trouble. I've seen most people I help have errors around their quest section.

Paul M.

From: Doublee<EMAIL_ADDRESS>Sent: Saturday, June 22, 2019 11:01:00 AM
To: GrapeshotGames/AtlasMapViewer
Cc: Paul McDowell; Author
Subject: Re: [GrapeshotGames/AtlasMapViewer] invalid character ']' looking for beginning of value (#14)
Yea im getting something similar but can't locate it in mu json :(
#18https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGrapeshotGames%2FAtlasMapViewer%2Fissues%2F18&data=02|01||4ff8de06fd2c44935b9d08d6f72ad2ff|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636968160626068926&sdata=DEI9cIJsGm83PcYCqYEotdIyPyDEzw1txJjHNL9%2B4QM%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGrapeshotGames%2FAtlasMapViewer%2Fissues%2F14%3Femail_source%3Dnotifications%26email_token%3DABEMTQLLW3XCQTODWARDIPLP3ZEDZA5CNFSM4HY2T4WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYKMI2Q%23issuecomment-504677482&data=02|01||4ff8de06fd2c44935b9d08d6f72ad2ff|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636968160626078937&sdata=h4gixWk7014tZnMOZt9z%2Fl037xh1pNXI6nRvvZSZPpM%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABEMTQL6JLDBR55GOQFTYV3P3ZEDZANCNFSM4HY2T4WA&data=02|01||4ff8de06fd2c44935b9d08d6f72ad2ff|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636968160626098954&sdata=R80c13xgKzFDkbaTnRkjl8guFtrRnFZdeSeYhaRDr%2B4%3D&reserved=0.


Thanks 😊

Not sure how else to say thank you but open an issue. I love this website. It follows closely the spirit of Go By Example by being straight and to the point. We need more sites like it 👍

I personally like the density and conciseness of the website. I know Terraform has a lot more to offer but I like how you've captured the core and essentials. Someone asked for workspaces but honestly, a few engineers will directly ever touch workspaces. I will send suggestions your way when I think of any. Have an amazing day! 😊


Issue setting cookie

To avoid going through login screen and entering all the details, I decided to login user via our API. However, the below code that works with chrome driver, doesn't work with ghost driver.
self.driver.get(self.api_url)

if not len(self.driver.get_cookies()):
    self.driver.add_cookie(cookie)

Example of cookie dict:
{'domain': '.hub.adcat.com', 'name': 'session-version-1', 'expires':<PHONE_NUMBER>, 'value': 'AYd...', 'path': '/api/', 'port': None}

I verified that I am on the following URL: https://hub.adcat.com/api/session, but during runtime I get Can only set Cookies for the current domain (got the same - as expected - for google.com).
I tried modifying cookie['domain'] to hub.adcat.com but received Unable to set Cookie.
At first I was running with phantomjs 1.9.1 but updating to 1.9.7 didn't resolve the issue, selenium 2.40.0.
Any ideas how to proceed with debug/issue resolution?

Bumping this old thread. Having this issue with latest selenium and phantomjs 2.0

Same with PhatomJS 2.1.1 and Selenium 2.49.1

I'm having the same issue here. was that fixed in https://github.com/qa/arquillian-phantom-driver/pull/22? if so how do I get that into my production setup?

I've run into this too with PhantomJS 2.1.1 - it looks like it may be related to ariya/phantomjs#14047

Hi @akanieski, sorry for my late response.
If I remember correctly, the fix was that I had to specify a domain and some additional parameters the new cookie should be set for. See: https://github.com/arquillian/arquillian-extension-drone/commit/f403362a406c220bc9910ff1f19d62f4a80197a6

Based on https://github.com/SeleniumHQ/selenium/wiki/JsonWireProtocol#cookie-json-object
expiry and domain should be optional, and should not cause any issues, but ghostdriver is using them for checking https://github.com/ariya/phantomjs/blob/master/src/ghostdriver/request_handlers/session_request_handler.js#L769

This seems to only allow me to set expired cookies, did anyone find a solution?

Is there any solution? I'm using phantomjs 2.1.1 (win32) with chimp. It's not possible to set a valid cookie for localhost. Name, value, domain and expiry are set but the result is always "unable to set cookie".

Still a issue in PhantomJS 2.1.12

Any update with this issue?

@franziz this is a known issue.  it is fixed in commit d038837 upstream.
you need to compile phantomjs 2.1.1 from source with commit d0388371cfda166d97165c1b68182857b954cc83 to fix this issue.


Check missing mappings for Shopify categories

What are you trying to accomplish?
Checking which Shopify categories lack corresponding channel mappings
Testing result

The test seems to be working as expected — though again we cannot merge this until we have the found conflicts resolved (and likely have tooling to keep these up to do date easily enough) (cc @jeanrick per our chats)

This PR was merged automatically as part of this PR: Excluded given taxonomy for mappings in checking missing mappings


UnboundLocalError

Hello,
I am running CITE-seq-count pipeline and it gives me UnboundLocalError.  Any suggestions?  Here is the code:
python /CITE-seq-Count-master/CITE-seq-count.py -R1 P-xxxx1.fastq.gz -R2 xxxx2.fastq.gz -t TAG_List.csv -cbf 1 -cbl 16 -umif 17 -umil 26 -tr "^[ATGC]{15}[TGC][A]{6,}" -hd 1 -o citeseqcount.tsv -cells 5000
Output:
Traceback (most recent call last):
File "/CITE-seq-Count-master/CITE-seq-count.py", line 232, in
main()
File "/CITE-seq-Count-master/CITE-seq-count.py", line 151, in main
tag_length = check_tags(ab_map, args.hamming_thresh)
File "/CITE-seq-Count-master/CITE-seq-count.py", line 136, in check_tags
return(len(a))
UnboundLocalError: local variable 'a' referenced before assignment

Yep, that is correct. Will be fixed in the next release.


BUG: Estimator.eval() runs feature_engineering_fn more than once.

The PR is aimed to fix #12205.
What changes were proposed in this pull request?
eliminate the second call of feature_engineering_fn in evaluate method.
How was this patch tested?

[x] add an unit test.

Can one of the admins verify this patch?

By the way, @ispirmustafa . I'm a newbie. If I want to run only one unit test, say the test case I added, could you tell me how to do it? Thanks.

I have found it: bazel test -c opt //tensorflow/contrib/learn:estimators_test
Could you let @tensorflow-jenkins  test the PR? Thanks.

Hi, @ispirmustafa . The patch has been modified as suggested, and could you review it? Thanks.

Jenkins, test this please

Thanks, @ispirmustafa and @jhseu .


language independence ...

Hey there,
again : this is not an issue.
The changes recently done are totally fine but let me explain why I make (made or am planning to make) which changes in the fork https://github.com/redaktor/nlp_compromise
As a European I would love this project to be as multilingual as possible ;)
The changes made have these goals :
• for contributing be totally explanative and readable
• for transport be browser-friendly and thus very small
• completely separate data / language logic / project logic
Three new files in src/data
: dictionary.js
The file where we can contribute multilingual words in the categories like in the readme.
: dictionary_rules.js (tba)
The file where we can contribute multilingual rules.
: _build.js
To build the data modules for one/some/all languages.
This could also be the first grunt step.
It will generate or overwrite a folder like 'en'.
Check it out node _build -l
Basically I am planning to let the build script generate a customized client side file and additional AMD browser modules.
See for instance the module.exportslines, there are more than 30 but they are useless in the browser and apart from that I'd optimize the compressing for browser a bit further.
I do also try to avoid duplicates further. For example in phrasal verbs : Some verbs are already in the verb data module and some adjectives are already in the adj. module ...
When it is complete:
• each module e.g. in /parent should only be a littlebit 'project logic'.
• our database can autotranslate
• I could attach our web interface to encourage translators even more ;)

hey man. Love the ideas.
I'm not ready to take the plunge into other languages yet, though kudos to the work you're doing.
amd support, etc is great, if you keep the PRs small.
The /adverbs_decline and stuff is also cool, have you added any more english data?
my guess is that any new languages will require a brand new pos.js file, conjugation logic, etc.. then it begins to feel like they should have seperate repos. I really don't know.
I know some nlp libraries generalise into all languages, but they are probably more clever than me ;)
Lots of good data in the wordnet european language forks. That's where I would go first.
cheers man,

But why should it be in other repos?
The rewritten modules in the fork load languages from the /en /de whatever folders in runtime.
It falls back to english.
But let's say nlp.noun('person', 'de').pluralize() should return 'personen' and not 'persons' ...
This can either be solved elegant in

node.js
or
browser / AMD
(use cached when loaded already like 'en' [or perhaps an option.defaultLang] and load other dictionaries in runtime. E.g. dojo's lang.mixin and lang... might come in handy

In general that would be ultimate - nlp, dynamically multilanguage in the browser ...
For AMD haters the _build.js could generate a one file with the -l en,de,es option ...

yeah, i understand what you're thinking. my concern here is that in most lines of code there are assumptions about english - noun.inflect() for example, doesn't make sense to korean nouns.. I'm afraid it's not been built to be very generic, so it will involve more than just swapping the lexicon.
So my recommendation would be to get data for one language, and see how it behaves before designing an api it for all european languages. It'd be sweet to see a javascript nlp in the browser for another language.. I bet there's not much in nlp_compromise that will be helpful for it, in the long run..
... is that pessimistic of me?
I think in N America we have a less .. cool and more ... terrified  view of other languages ;)
Make a spanish demo!
cheers

or nlp_kompromiss ?
does german have similar parts of speech even? I know there's a third gender.. or gendered verbs or something. I really know nothing.
let's make a repo and have fun.

...wanna? I couldn't do it without a fluent language-speaker.
I speak some french, but badly. How's yours??

so it will involve more than just swapping the lexicon.
Yes - I am coding as fast as I can ;)
I' ll do german and we have translators for spanish and maybe one for danish/swedish.
But however. First we need to make translation easygoing, I thought.
Regarding the rules - I find multiple similar rules in german which need just to be slightly modificated, e.g. the substitution value changes...
My french is badder than yours ;)

have you added any more english data?

just a few, like
// things that are often named after people
{ en: 'club', meta: {personBlacklist: ['en']} },
{ en: 'museum', meta: {personBlacklist: ['en']} },
{ en: 'hall', meta: {personBlacklist: ['en']} },
{ en: 'arena', meta: {personBlacklist: ['en']} },
{ en: 'stadium', meta: {personBlacklist: ['en']} },


Breakdown of payment from each GSC campaign.

I'd like to see a breakdown of the payment per cpk for each daily super block.
If we know the line items that make up the total, it is more clear and easier to troubleshoot if there are any GSC campaign payout issues.
Maybe this exists already?

It seems CameroonONE is based on current status not the specific point in time.
Height 150080 of 34k BBP should show CameroonONE details, but since all my sponsorships are due, this is not being displayed.
Height: 150080
Date: 10/8/2019 19:05
Source: Smart-Contract-Payment
Credit: 34<PHONE_NUMBER>0 BBP (matures in 97 more blocks)
Transaction ID: 523519680f6efa2b482aadaad54a12e0d0582f6167059912f8383cde3914fa2d

19:37:52 leaderboard Sunk 150080
19:37:52
{
  "Prominence v1.1": "Details",
  "Block Range": "149670-149875",
  "HEALING: BLENRYgVPwzfsrJoJNbvC537qMHsCVyeFU [Sunk], Pts: 8341395.00": "1.32%",
  "POG: BLENRYgVPwzfsrJoJNbvC537qMHsCVyeFU [Sunk], Pts: 0.00": "0.00%",
  "Healing": "Diary Entries",
  "Sunk": "prayer",
  "Prominence": "Totals",
  "ALL: BLENRYgVPwzfsrJoJNbvC537qMHsCVyeFU [Sunk], Pts: 8341395.00, Reward: 4415.160": "1.320%"
}


Spurious unresolved-import for items generated by proc-macro attributes

rust-analyzer extension version 0.2.465 reports unresolved-import for items generated by proc-macro attributes, even though both rust-analyzer.cargo.loadOutDirsFromCheck and rust-analyzer.procMacro.enable are enabled. The code compiles without problem.
Reproduction example
In a proc-macro crate using quote:
use proc_macro::TokenStream;

#[proc_macro_attribute]
pub fn foo(attr: TokenStream, item: TokenStream) -> TokenStream {
    (quote::quote! {
        pub struct GeneratedStruct;
        pub const GENERATED_CONST: i32 = 42;
        pub fn generated_fn() {}
    }).into()
}

The consumer crate in the same workspace:
#[macro_use]
extern crate my_proc_macro;

use bar::{GeneratedStruct, GENERATED_CONST, generated_fn}; // unresolved-import

mod bar {
    #[foo]
    const A: () = (); // Doesn't matter. Just a random item to throw at the macro.
}

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/6029.

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/6029.

But anyway, thanks for the issue report, it's lovely ❤ .

But anyway, thanks for the issue report, it's lovely ❤ .

Sorry for the duplicate. I only searched for unresolved-import and didn't realize that this is actually a more general problem. Thanks for pointing to the existing issue.

Sorry for the duplicate. I only searched for unresolved-import and didn't realize that this is actually a more general problem. Thanks for pointing to the existing issue.


Is it possible to provide a detailed tutorial for upgrading

Every time a new version is released, the database must be rebuilt. I can only empty the data directory and start syncing from 0, which took more than 50 hours. I am very happy to upgrade to the new version. At the same time, I am also very upset that it takes a lot of time to upgrade. I hope to publish some tutorials so that I can upgrade from the fully synchronized old version to the new version as soon as possible. Thank you.

@rhyslbw @CyberCyclone Hi, Can you help me？

You can now use the cardano-db-sync DB snapshot to expedite the upgrade, and I've just updated the Wiki to direct you back to the Getting Started section of the README after removing the volumes. Please try this procedure first, then let me know if you require further assistance.

 volumes:
      - ./config/network/${NETWORK:-mainnet}:/config
      - ./data/db-sync-data:/var/lib/cdbsync
      - ./data/node-ipc:/node-ipc

root:/node/cardano/v6.1.0/cardano-graphql/data# ls
db-sync-data  node-db  node-ipc  postgres-data

I designated the volume as the data directory under the current directory, how should I delete the volume? Delete the directory directly?

volumes:
  db-sync-data:
  node-db:
  node-ipc:
  postgres-data:

https://github.com/input-output-hk/cardano-graphql/blob/2202252d2a36e493f00ac0434063f75a363f8020/docker-compose.yml?_pjax=%23js-repo-pjax-container%2C div[itemtype%3D"http%3A%2F%2Fschema.org%2FSoftwareSourceCode"] main%2C [data-pjax-container]#L133
By the way, how should I make a local directory in this place？

 volumes:
      - ./config/network/${NETWORK:-mainnet}:/config
      - ./data/db-sync-data:/var/lib/cdbsync
      - ./data/node-ipc:/node-ipc

root:/node/cardano/v6.1.0/cardano-graphql/data# ls
db-sync-data  node-db  node-ipc  postgres-data

I designated the volume as the data directory under the current directory, how should I delete the volume? Delete the directory directly?

Deleting the directories for _postgres-data and _db-sync-data would have the same result, but you could also rename the folder as a quick backup to the previous version.

It's probably better to not store the data in the current directory though as it makes it difficult to do Git updates as it will complain about updated files not being committed.

By the way, how should I specify the local directory in this place？

You can just omit any mounted host paths from this section. See
https://docs.docker.com/compose/compose-file/compose-file-v3/#volumes

Thank you, if I start syncing from 0, how do I use snapshot sync?

You just need to add the snapshot URL variable (make sure you have the right snapshot associated with the right db-sync version).
If you only remove db-sync and postgres volumes and the rest is in sync then it will import the snapshot and then sync up once it's imported. This is insanely faster then the db following the chain and inserting block by block.
If you're starting everything from scratch, using the snapshot is still fast then following the chain. As in, cardano-node can sync a lot quicker then db-sync takes to populate. So an import will be faster regardless.

If you're starting everything from scratch, using the snapshot is still faster than following the chain. As in, cardano-node can sync a lot quicker than db-sync takes to populate. So an import will be faster regardless.

[db-sync-node.Handshake:Info:49] [2021-12-22 12:57:01.59 UTC] WithMuxBearer (ConnectionId {localAddress = LocalAddress "", remoteAddress = LocalAddress "/ipc/node.socket"}) Send (ClientAgency TokPropose,MsgProposeVersions (fromList [(NodeToClientV_1,TInt 764824073),(NodeToClientV_2,TInt 764824073),(NodeToClientV_3,TInt 764824073),(NodeToClientV_4,TInt 764824073),(NodeToClientV_5,TInt 764824073),(NodeToClientV_6,TInt 764824073),(NodeToClientV_7,TInt 764824073),(NodeToClientV_8,TInt 764824073),(NodeToClientV_9,TInt 764824073),(NodeToClientV_10,TInt 764824073)]))
[db-sync-node.Subscription:Notice:48] [2021-12-22 12:57:01.62 UTC] Identity Required subscriptions started
[db-sync-node.Handshake:Info:49] [2021-12-22 12:57:01.63 UTC] WithMuxBearer (ConnectionId {localAddress = LocalAddress "", remoteAddress = LocalAddress "/ipc/node.socket"}) Recv (ServerAgency TokConfirm,MsgAcceptVersion NodeToClientV_10 (TInt 764824073))
[db-sync-node.Mux:Info:49] [2021-12-22 12:57:01.63 UTC] Bearer on ConnectionId {localAddress = LocalAddress "", remoteAddress = LocalAddress "/ipc/node.socket"} event: Handshake Client end, duration 0.035786967s
[db-sync-node.Mux:Info:50] [2021-12-22 12:57:01.63 UTC] Bearer on ConnectionId {localAddress = LocalAddress "", remoteAddress = LocalAddress "/ipc/node.socket"} event: State: Mature
[db-sync-node:Info:53] [2021-12-22 12:57:01.63 UTC] Starting chainSyncClient
[db-sync-node:Info:53] [2021-12-22 12:57:01.64 UTC] Cardano.Db tip is at slot 48360823, block 6647999
[db-sync-node:Info:58] [2021-12-22 12:57:01.64 UTC] Running DB thread
[db-sync-node:Info:58] [2021-12-22 12:57:01.95 UTC] Rolling back to genesis
[db-sync-node:Info:58] [2021-12-22 12:57:30.20 UTC] Deleting 6647999 blocks up to slot 48360823

I use snapshots to sync from a new server, it seems Rolling back to genesis

@rhyslbw
https://github.com/input-output-hk/cardano-graphql/wiki/Docker#rebuild-cardano-db
Should execute docker-compose down instead of docker-compose stop

Initial synchronization, the snapshot cannot be used, you must wait for the cardano-node-ogmios synchronization to complete before you can use the snapshot

@rhyslbw @CyberCyclone Thank you, I now have the knack for using snapshots


Always assigning to an un-initialized variable in an exhaustive switch

TypeScript Version: 3.4.0-dev.20190220

Search Terms:
switch case exhaustive assignment assign undefined TS2322
Code
Compile with the flags --noImplicitReturns --noImplicitAny --strictNullChecks
enum Test {
    A,
    B,
    C
}

// this example correctly understands that all possible values of t are handled
function foo(t: Test): number {
    switch (t) {
        case Test.A:
            return 1;
        case Test.B:
            return 2;
        case Test.C:
            return 3;
    }
}

// this should be identical to foo, since a is always assigned as the switch is exhaustive
function bar(t: Test): number {
    let a;
    switch (t) {
        case Test.A:
            a = 1;
        case Test.B:
            a = 2;
        case Test.C:
            a = 3;
    }
    return a; // I don't expect an error here
}

Expected behavior:
Code compiles with no errors. When a is returned in bar, its type is number. Typescript understands that an exhaustive switch with each case assigning to a type-inferred variable means that the variable's type is the union of each assigned value's type. I'd expect this to work because returns within an exhaustive switch are understood in a similar manner.
Actual behavior:
When a is returned in bar, its type is number | undefined
TS2322: Type 'number | undefined' is not assignable to type 'number'.
  Type 'undefined' is not assignable to type 'number'.

Playground Link:
link here
Turn on noImplicitReturns, noImplicitAny, and strictNullChecks.
Related Issues:

https://github.com/Microsoft/TypeScript/issues/22470 (different, since the default doesn't assign)
https://github.com/Microsoft/TypeScript/issues/18362 (this is pretty old, from TS 2.4.2. Since foo in my example works I think the closure reason might be invalid)

Comments in #18362 still apply. We have some special-casing we're capable of doing for return, but it doesn't generalize to assignments.

how is it not covered by officially recommended:
declare function noMore(never: never): never;

// this should be identical to foo, since a is always assigned as the switch is exhaustive
function bar(t: Test): number {
  let a;
  switch (t) {
    case Test.A:
      a = 1;
      break;
    case Test.B:
      a = 2;
      break;
    case Test.C:
      a = 3;
      break;
    default: return noMore(t);
  }
  return a; // I don't expect an error here
}

?

http://www.typescriptlang.org/docs/handbook/advanced-types.html

duplicate of #8545?


Visualize your Robot's actual state and sensor Data in Unity3D

Hi, I am a beginner in programming and I am working in the way to link RVIZ/GAZEBO with Unity.
I found in your ROS# wiki a link to a Youtube video about “Visualize your Robot's actual state and sensor Data in Unity3D.” and that is exactly what I want to do with a UR5 robot arm.
Problem, I cannot find the joint State Patcher/Kinematic Patcher and Gravity Patcher in your package.
Is it because these data are not public? Can you please help me to understand how I can make it possible?
Cordially,
Bertrand

Hi @Bertrand64300,
That video is meant as a showcase only. It uses an old version of ROS#, and some custom scripts that are not included in the base ROS# package.
It is absolutely possible to model a robot's actual state in Unity, and so far we have a tutorial showing how to do this, using the Turtlebot as an example. Please follow this instead of the showcase video.
I will now close this issue, since there several other issues open which are closely related to this topic.
Suzannah