Remove commas between endpoints in grid

Description
The commas between endpoints are redundant as the underline of the anchor visually separates multiple entries. The comma visually collides with the underline, and looks a bit strange when there's only one item and the rest in overflow.
Subjective, but I think it looks a little sharper like this.
Before

After

Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

The commas between endpoints are redundant as the underline of the anchor visually separates multiple entries.

This won't be true for tcp endpoints. They're plain text.
However, maybe the space can be the delimiter. That shouldn't be a situation where an endpoint contains spaces.

This won't be true for tcp endpoints. They're plain text.

True, some URLs won't be hyperlinked or underlined, but the space should still be clear.

Yeah I like this change

/azp run

/azp run

/azp run

@adamint the test failure here seems related to health checks. Did we commit something that breaks tests?
Assert.Null() Failure: Value of type 'Nullable<HealthStatus>' has a value
Expected: null
Actual:   Healthy


Timeout on update script silently removes all reviews of a community

docker-compose --file .docker/docker-compose.yml --file .docker/docker-compose.dev.yml run -T app npx ts-node scripts/find-reviews-from-crossref-via-biorxiv 10.7717 10.7287 \
| tee ./data/reviews/53ed5364-a016-11ea-bb37-0242ac130002.csv
Date,Article DOI,Review ID
Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1145:16)

Exit code of $ make update-event-data was 0.

Discovered that the exit code is from tee, not the script.

Rather than looking at improvements of writing files only on success or retrying requests, we've moved the update script task to the top of the retro list so it can be hopefully mitigated with a human process of checking the diff in a pair/mob before committing.


Fix std se conversion

Correct the formula/function used to convert the standard error in the tear sheet to a one-period standard error. This is done by creating a new function in the utils module.
Issue #155

Thank you @MichaelJMath


Handle when target is not an object.

@jonschlinkert Do you think it makes sense to do it this way where the target is returned, or does it make sense to merge the rest of the values and return the results?
The scenario that came up is that one of the tests in templates is using engine. The pages and layouts in the test have a layout property on the data object. There is also the built-in layout helper because of the layout template type being used in the test.
engine was attempting to merge an object that looked similar to this:
var ctx = { layout: 'a' };
var imports = { layout: function() { ... } };
ctx = assign({}, ctx, imports);

Since the function in the imports object had a property on it, the value of the layout in ctx ('a') became the target and tried to assign properties from the function to the string.
This change will result in ctx looking like { layout: 'a' }.
The other way we can do it is so ctx would look like this: { layout: function() { ... } }.

Although this is really not related to assign-deep, it seems like the function should be on the context always. accidentally overwriting helpers with non-function values has been a problem for a while.

does it make sense to merge the rest of the values and return the results?

IMHO, this is how it should be done

does it make sense to merge the rest of the values and return the results?

IMHO, this is how it should be done

Sounds good. I think this makes more sense for what assign-deep should be doing. In the case of templates, the layout value would have already been used at this point to determine the layout, then the helper can still be used if needed.
I've already made the change locally and tested in templates and everything works so I'll get it pushed up here.

I've already made the change locally and tested in templates and everything works so I'll get it pushed up here

sweet, thx


keep getting "You have been signed out because reading stored authentication information failed."

Does this issue occur when all extensions are disabled?: Yes/No Yes

VS Code Version: 1.65.2
OS Version: Windows_NT x64 10.0.19044

I get 3 devices sharing the same configuration, one day, I changed my Microsoft account password on one of my devices, after then I keep getting this message every time opening the vscode on the other two devices.

Steps to Reproduce:

Change the MS account password
Open vscode.

I tried to reinstall the vscode but it didn't help, maybe there are some cache exists but don't know how to clean up.
thought re-install completely might help on this.

haha, Guess what.. I tried to update my password on the device with this popup, now all my 3 devices will have this popup..

Can you follow these steps so that we can get more information on what's going on?

Wait for the issue to happen...
Open the Output pane (ctrl+shift+u I think on Windows)
drop in this issue he contents of the following:

Microsoft Authentication Output channel
Log (Settings Sync) Output channel
Log (Window) Output channel

Microsoft Authentication
[Info  - 01:28:50.209] Reading sessions from secret storage...
[Info  - 01:28:50.354] Got 2 stored sessions
[Trace  - 01:28:50.354] Read the following stored session with scopes: email offline_access openid profile
[Info  - 01:28:50.354] Refreshing token for scopes: email offline_access openid profile
[Trace  - 01:28:50.355] Read the following stored session with scopes: email offline_access openid profile
[Info  - 01:28:50.355] Refreshing token for scopes: email offline_access openid profile
[Error  - 01:28:51.316] Refreshing token failed (for scopes: email offline_access openid profile): {"error":"invalid_grant","error_description":"AADSTS50132: The session is not valid due the following reasons: password expiration or recent password change, SSO Artifact is invalid or expired, session is not fresh enough for application, or a silent sign-in request was sent but the user's session with Azure AD is invalid or has expired.\r\nTrace ID: a6eae0ef-54a1-4b8f-b6dd-530327321e00\r\nCorrelation ID: 92363cb6-5fe3-431a-a360-d6c52528db92\r\nTimestamp: 2022-03-17 01:28:51Z","error_codes":[50132],"timestamp":"2022-03-17 01:28:51Z","trace_id":"a6eae0ef-54a1-4b8f-b6dd-530327321e00","correlation_id":"92363cb6-5fe3-431a-a360-d6c52528db92","error_uri":"https://login.microsoftonline.com/error?code=50132"}
[Error  - 01:28:51.316] Error: Refreshing token failed
[Info  - 01:28:51.316] Logging out of session 'f8cdef31-a31e-4b4a-93e4-5f571e91255a/1:live.com:00037FFE0647EDAC/4d5aa5cf-560c-475a-be2c-dfca3c7d6e82'
[Info  - 01:28:51.316] Session not found 'f8cdef31-a31e-4b4a-93e4-5f571e91255a/1:live.com:00037FFE0647EDAC/4d5aa5cf-560c-475a-be2c-dfca3c7d6e82'
[Info  - 01:28:51.542] Setting token for scopes: email offline_access openid profile
[Info  - 01:28:51.542] Token refresh success for scopes: email offline_access openid profile
[Info  - 01:28:51.544] Getting sessions for the following scopes: email offline_access openid profile
[Info  - 01:28:51.544] Got 1 sessions for scopes: email offline_access openid profile
[Info  - 01:28:51.544] Token available from cache (for scopes email offline_access openid profile), expires in 5361998 milliseconds

Log (Settings Sync)
[2022-03-17 09:28:48.881] [settingssync] [info] Using settings sync service https://vscode-sync.trafficmanager.net/
[2022-03-17 09:28:48.882] [settingssync] [info] Auto Sync is enabled.
[2022-03-17 09:28:48.882] [settingssync] [info] Auto Sync: Suspended until auth token is available.
[2022-03-17 09:28:51.555] [settingssync] [info] Auto Sync: Started
[2022-03-17 09:28:51.555] [settingssync] [info] Auto Sync: Triggered by Interval
[2022-03-17 09:28:51.812] [settingssync] [info] Deleting from backup /C:/Users/perseusw/AppData/Roaming/Code/User/sync/settings/20220214T205428.json
[2022-03-17 09:28:52.639] [settingssync] [info] Settings: No changes found during synchronizing settings.
[2022-03-17 09:28:52.653] [settingssync] [info] Keybindings: No changes found during synchronizing keybindings.
[2022-03-17 09:28:52.658] [settingssync] [info] Snippets: No changes found during synchronizing snippets.
[2022-03-17 09:28:52.661] [settingssync] [info] Tasks: No changes found during synchronizing tasks.
[2022-03-17 09:28:55.534] [settingssync] [info] GlobalState: Updated remote ui state
[2022-03-17 09:28:55.540] [settingssync] [info] GlobalState: Updated last synchronized ui state
[2022-03-17 09:28:55.618] [settingssync] [info] Extensions: No changes found during synchronizing extensions.
[2022-03-17 09:28:55.621] [settingssync] [info] Sync done. Took 3026ms
[2022-03-17 09:30:29.811] [settingssync] [info] Auto Sync: Triggered by Activity
[2022-03-17 09:30:29.968] [settingssync] [info] Settings: No changes found during synchronizing settings.
[2022-03-17 09:30:29.973] [settingssync] [info] Keybindings: No changes found during synchronizing keybindings.
[2022-03-17 09:30:29.978] [settingssync] [info] Snippets: No changes found during synchronizing snippets.
[2022-03-17 09:30:29.983] [settingssync] [info] Tasks: No changes found during synchronizing tasks.
[2022-03-17 09:30:29.989] [settingssync] [info] GlobalState: No changes found during synchronizing ui state.
[2022-03-17 09:30:30.075] [settingssync] [info] Extensions: No changes found during synchronizing extensions.
[2022-03-17 09:30:30.076] [settingssync] [info] Sync done. Took 147ms

Log (Window)
[2022-03-17 09:28:48.828] [renderer1] [error] [Extension Host] (node:26284) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use Code --trace-deprecation ... to show where the warning was created)

@GreyElec do you see the account you changed here in the list of accounts?

If so, if you sign out and sign back in does that help?:

It's somewhat expected that you see this notification because the error message says it all:

The session is not valid due the following reasons: password expiration or recent password change, SSO Artifact is invalid or expired, session is not fresh enough for application, or a silent sign-in request was sent but the user's session with Azure AD is invalid or has expired.

But I would expect that you would see this notifications on any existing open windows across your devices and then never again.

Hi @TylerLeonhardt , Yes, I changed the password, I do understand I should see the notification but my issue is, the message shows up every time I opened the VSCODE on any of the devices after the password change. this doesn't looks right to me.
it seems there is an outdated cache and it was never replaced.

Can you do this and let me know what you see:
https://github.com/microsoft/vscode/issues/145189#issuecomment-1073077050

tried dozens of times but I am OK to do it one more time, as shown in the figure,  after logout, login, and restarting VSCODE, the message shows up and tells about this error.

Me abian robado el dispositivo me podrian dejar ingresar otra ves
Get Outlook para Androidhttps://aka.ms/AAb9ysg

From: Yanchen Wang @.>
Sent: Monday, March 21, 2022 7:29:54 PM
To: microsoft/vscode @.>
Cc: Subscribed @.***>
Subject: Re: [microsoft/vscode] keep getting "You have been signed out because reading stored authentication information failed." (Issue #145189)
[image]https://user-images.githubusercontent.com/33427038/159388673-689eaba2-2310-42f2-9d31-cab955e3397c.png
tried dozens of times but I am OK to do it one more time, as shown in the figure, after logout, login, and restarting VSCODE, the message shows up and tells about this error.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode/issues/145189#issuecomment-1074588030, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYF6QHEVOJBLUIWLMCJZRTTVBEPBFANCNFSM5Q27KWMA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I am having this issue too, after changing my Microsoft account password. I tried above to sign out and re-launch but notification keeps showing. "You have been signed out because reading stored authentication information failed." I then choose to sign in with Microsoft account. This launched browser and response is "You are signed in now and can close this page." But does not resolve the issue. I will try to re-install VSCode.

Version: 1.65.2 (user setup)
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
Date: 2022-03-10T14:33:55.248Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19044

This link helped me.
How to fix:

Exit Visual Studio Code application
Open: Credential Manager ("Start → Settings" → then search for "Credential Manager")
Click on the Windows Credentials tab.
Remove all credentials start with vscode.vscode, git:, ada:
Open Visual Studio Code application and login again. Done

https://xenvn.com/threads/vscode-fix-you-have-been-signed-out-because-reading-stored-authentication-information-failed.2584/#post-3139

Wow, really nice tips. this workaround helped on my side.

This link helped me.
How to fix:

Exit Visual Studio Code application
Open: Credential Manager ("Start → Settings" → then search for "Credential Manager")
Click on the Windows Credentials tab.
Remove all credentials start with vscode.vscode, git:, ada:
Open Visual Studio Code application and login again. Done

https://xenvn.com/threads/vscode-fix-you-have-been-signed-out-because-reading-stored-authentication-information-failed.2584/#post-3139

This sorted it out for Windows. Also interesting to see how many tokesn/password I needed to cleanup. Now the question is, what is the equivalent in macos, has I still have the issue there

This sorted it out for Windows. Also interesting to see how many tokesn/password I needed to cleanup. Now the question is, what is the equivalent in macos, has I still have the issue there

Answering my own question, it is in Keychain Access. In Keychain Access, select 'login' on the left side, then you can filter on the top right search box for 'vscode' and just delete all entries. Sorted.

I got same issue on Linux .so how to deal with it ?

I got same issue on Linux .so how to deal with it ?

That may depend on the distro used. Gnome Wallet? KDE Wallet? Just guessing here, will give a look next time I boot into Linux

This is all great information, folks. I'm curious to know if you all get into a bad state again if you change your password once more.
When I tried, I was unable to reproduce.. which leads me to believe that you all got into a bad state from a previous change I made, and that new customers won't be affected by this.
If my theory is true, I will at least document these steps to clear the vscode specific items out of the keychain.

@jpixy on Ubuntu with gnome, there's an app called "Passwords & Keys" and from there the steps are similar to what others have shown on macOS and Windows.

as an FYI, in my case I was running on multiple computers with different OS (a Win11, a Win10, a MacOS an Ubuntu machine). Once I changed the password issue was on all machines. I did notice I had multiple entries for vscode, I believe one for each time I logged out and logged in again.

Finally a fix! Thank you for the workaround @capegreg! I had been dealing with this issue for about as long as @GreyElec  😅

This link helped me.
How to fix:

Exit Visual Studio Code application
Open: Credential Manager ("Start → Settings" → then search for "Credential Manager")
Click on the Windows Credentials tab.
Remove all credentials start with vscode.vscode, git:, ada:
Open Visual Studio Code application and login again. Done
...

I got the same issue and solved it in the same way.
Note that maybe it's only me: I left Visual Studio Code UI on while deleting Windows Credentials. After reopening the UI, Sync says it is enabled and syncing, but not. It "synced" 0 profiles. You have to turn off sync and turn it back on.

This sorted it out for Windows. Also interesting to see how many tokesn/password I needed to cleanup. Now the question is, what is the equivalent in macos, has I still have the issue there

Answering my own question, it is in Keychain Access. In Keychain Access, select 'login' on the left side, then you can filter on the top right search box for 'vscode' and just delete all entries. Sorted.

Thank you so much this worked for me

Ahh I see what I was doing wrong. Those sessions from your password change weren't being removed properly. Now they should be! This will be in VS Code Insiders tomorrow and VS Code stable early next month.

This sorted it out for Windows. Also interesting to see how many tokesn/password I needed to cleanup. Now the question is, what is the equivalent in macos, has I still have the issue there

Answering my own question, it is in Keychain Access. In Keychain Access, select 'login' on the left side, then you can filter on the top right search box for 'vscode' and just delete all entries. Sorted.

Thank you @rodrigoespinozadev, that solve the problem.

I'll mark this issue as verified as part of https://github.com/microsoft/vscode/issues/172064. Please file a new issue if the bug continues to occur on the latest Stable and/or Insiders versions of VS Code.


Make Faucet on Testnet adaptive

The faucet feature allows people to run btcli w faucet to get funds on the testnet. The POW difficulty is really small and causes issues where people abuse the faucet for testnet TAO making it impossible for new people. Lets make the POW faucet on testnet adaptive based on a max rate of new tokens per day/block with an adaptive moving average method.

closing as no longer valid


Refactoring Master for Labs19

Description

Projects List View
Project View
Sidebar
Moved Routes to Authorization Root
Implement Layout Component

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Change Status

[x] Complete, but not tested (may need new tests)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] My changes generate no new warnings
[x] There are no merge conflicts

Looks good, reviewed code collectively as we were combining the views into the Layout view.
Outstanding issues that need discussing:

Solution for state management for component-level functions

Comments to outline the relatively confusing authentication flow

Code was reviewed collectively as we were composing the views.
Looks good, everything is lining up nicely with the UX files.


Try to generate moq in module mode again

https://github.com/matryer/moq/issues/84 was recently closed, we should update the version and try to generate without the GOPATH again.

Closed in #75


[Misc][Quark] Upstream Quark format to VLLM

This PR is related to the issue discussed in this https://github.com/vllm-project/vllm/issues/10294.

Its a bit unfortunate that:

We will have all this duplicated code
We will have multiple model formats to maintain

However, I understand that AMD wants to push forward with Quark, so I think this approach is very reasonable. I would like to take on this effort myself and do two things:

Factor out some of the utility functions such that we do not have too much copy / paste code between the backends
Leverage the Kernel abstraction such that compressed-tensors models can run on RoCM backend

To do this, it would be very helpful if the Quark team could provide:

A sample W8A8-int8 model
A sample W8A8-fp8 model

If this is done, I can take on the effort of finalizing the integration. Thanks!

Its a bit unfortunate that:

We will have all this duplicated code
We will have multiple model formats to maintain

However, I understand that AMD wants to push forward with Quark, so I think this approach is very reasonable. I would like to take on this effort myself and do two things:

Factor out some of the utility functions such that we do not have too much copy / paste code between the backends
Leverage the Kernel abstraction such that compressed-tensors models can run on RoCM backend

To do this, it would be very helpful if the Quark team could provide:

A sample W8A8-int8 model
A sample W8A8-fp8 model

If this is done, I can take on the effort of finalizing the integration. Thanks!

Hi @robertgshaw2-neuralmagic. Thank you for your comments.
Your proposed work is highly valuable and will benefit both VLLM contributors and users. Factoring out utility functions to reduce code duplication and leveraging the Kernel abstraction for compatibility with the RoCM backend are excellent steps forward.
Here is a config file of W8A8-fp8:
config.json
And this is a config file of W8A8-int8:
config.json
key names and data types of Quark safetensors
model.layers.1.self_attn.k_proj.input_scale, 	torch.float16
model.layers.1.self_attn.k_proj.weight, 	torch.float8_e4m3fn
model.layers.1.self_attn.k_proj.weight_scale, 	torch.float16
model.layers.1.self_attn.o_proj.input_scale, 	torch.float16
model.layers.1.self_attn.o_proj.weight, 	torch.float8_e4m3fn
model.layers.1.self_attn.o_proj.weight_scale, 	torch.float16
model.layers.1.self_attn.q_proj.input_scale, 	torch.float16
model.layers.1.self_attn.q_proj.weight, 	torch.float8_e4m3fn
model.layers.1.self_attn.q_proj.weight_scale, 	torch.float16
model.layers.1.self_attn.v_proj.input_scale, 	torch.float16
model.layers.1.self_attn.v_proj.weight, 	torch.float8_e4m3fn
model.layers.1.self_attn.v_proj.weight_scale, 	torch.float16

KV scale format if kv cache used
model.layers.1.self_attn.k_proj.output_scale, 	torch.float16
model.layers.1.self_attn.v_proj.output_scale, 	torch.float16

And it’s important that the integration of the Quark format into VLLM is a priority. To ensure long-term maintainability, we would like to keep the responsibility for maintaining the Quark format within AMD’s team. Our team will closely monitor the integration to minimize any maintenance overhead for the vLLM project.
Let us know if you need any further support during the process. Thanks again for your collaboration!

KV scale format if kv cache used
model.layers.1.self_attn.k_proj.output_scale, 	torch.float16
model.layers.1.self_attn.v_proj.output_scale, 	torch.float16

Hi @kewang-xlnx can you describe what happens in the case of a model with fused QKV proj, such as Phi-3? In several cases models do not have separate k_proj/v_proj linear layers with which to register output scales on. This is why we register explicit self_attn.k_scale and self_attn.v_scale in compressed-tensors

KV scale format if kv cache used
model.layers.1.self_attn.k_proj.output_scale, 	torch.float16
model.layers.1.self_attn.v_proj.output_scale, 	torch.float16

Hi @kewang-xlnx can you describe what happens in the case of a model with fused QKV proj, such as Phi-3? In several cases models do not have separate k_proj/v_proj linear layers with which to register output scales on. This is why we register explicit self_attn.k_scale and self_attn.v_scale in compressed-tensors. We could likely reuse the same utility if you choose the same place to register these (on the Attention module)

Hi @mgoin. If the kv_scale of fused  QKV proj in quark is as below, take Phi-3 for example.
model.layers.22.self_attn.qkv_proj.output_scale
This is why we need to remap one kv_scale to both attn.k_scale and attn.v_scale.


closes Issue #170

Issue Title:
resolve issue: #170
Type of change ☑
What sort of change have you made:
I have added dropdown for shapes

Additionally, I would recommend you to follow guidelines for creating PR.
You need to properly name the Issue Title with actual Title, please don't leave it unchanged.

Additionally, I would recommend you to follow guidelines for creating PR.
You need to properly name the Issue Title with actual Title, please don't leave it unchanged.

Sorry for that I have changed the Title and I will make sure not to change that in future.

We don't need separate state for setting imagePath
Instead you can create an object which points to the images, For example:
const imagePaths = {
  'rectangle': rectImg,
  'circle': circleImg,
  'triangle': triangleImg
}

You can utilize this object from img like,
<img src={imagePaths[selectedTool]}/>

Sure I will do that but just for learning part.....what I did is it a bad practice??

Yeah, using state for too much stuff is a bad practice. I agree that you're learning but you need to learn in a right way by following good practices.
Update the code as per my request, then tag me here in the comments.

@0xabdulkhalid  I have made the requested changes please check

@AMISH-KHAN You haven't merged the latest changes, kindly fix it.
Actual One:

Yours One:

hey I have just noticed that some one has made the dropdown what should I do
https://github.com/singodiyashubham87/Draw-it-out/assets/123164129/321cbdcd-dd08-443b-b4dd-46c9e471fd62

@AMISH-KHAN Do one thing then reposition the toolbar to centre, they are not aligned.

I will add my changes if that's okay?

@AMISH-KHAN What you wanna add in your changes?

@AMISH-KHAN Seems good, you can resolve all the conflicts & make a fresh PR, I'll merge it.

@AMISH-KHAN Conflicts are still there, advised to create a fresh PR.

@AMISH-KHAN Any updates?

Just give me a day or two I will do it soon

@AMISH-KHAN Are you working on this?

Closing due to inactivity.


Flutter Gallery doesn't launch on device

It also has the wrong name flutter_gallery.  Presumably it hasn't been updated after the iOS project changes from @abarth.
FYI @tvolkert @chinmaygarde

I'm trying:
flutter build ios --release
flutter install

And then clicking on the resulting flutter_gallery binary.  It just flashes black and then sends me back to the home screen.

The flutter run logs are just
vm-service: Isolate creation error: (null)
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered

repeated 4 times.  I assume the 4 times is due to https://github.com/flutter/flutter/issues/4874

I can reproduce this issue.

I'm able to run the gallery, from master branch, onto my iPhone 6S. Maybe this was fixed?

I'm going to close, since it's been working for a while. Please reopen if you experience this issue.


Modification to hs tracking fields

Modified user and resource logging fields to ensure logstash fingerprints are unique=, see #2082

@Castronova - Just as an FYI, this PR actually merged the contents of develop into the release candidate... probably not what we wanted to do here.

Oh no. Can we revert it back easily? Should I pull/rebase with the rc branch and then create a new pr?


[BUG] coredns looses customization after cluster or docker restart

What did you do
How was the cluster created?
k3d cluster create --config cluster.yaml
# cluster.yaml (the network exists, is created externally)
# info: https://k3d.io/v5.4.4/usage/configfile/
apiVersion: k3d.io/v1alpha4 # this will change in the future as we make everything more stable
kind: Simple # internally, we also have a Cluster config, which is not yet available externally
metadata:
  name: $CLUSTER_NAME # name that you want to give to your cluster (will still be prefixed with `k3d-`)
kubeAPI: # same as `--api-port myhost.my.domain:6445` (where the name would resolve to <IP_ADDRESS>)
  host: $LOCALDOMAIN # important for the `server` setting in the kubeconfig
  hostIP: "<IP_ADDRESS>" # where the Kubernetes API will be listening on
  hostPort: "6445" # where the Kubernetes API listening port will be mapped to on your host system
image: rancher/$IMAGE # same as `--image rancher/k3s:v1.20.4-k3s1`
network: $DOCKER_NETWORK # same as `--network my-custom-net`
volumes: # repeatable flags are represented as YAML lists
  - volume: $DOMAIN_STORAGE/rwo:/var/lib/rancher/k3s/storage # same as `--volume '/my/host/path:/path/in/node@server:0;agent:*'`
    nodeFilters:
      - all
  - volume: $DOMAIN_STORAGE/rwx:/var/lib/csi-local-hostpath # :shared # same as `--volume '/my/host/path:/path/in/node@server:0;agent:*'`
    nodeFilters:
      - all
  - volume: $HOME/.local/share/mkcert/rootCA.pem:/etc/ssl/certs/rootCA.pem
    nodeFilters:
      - all
ports:
  - port: 80:80 # same as `--port '80:80@loadbalancer'`
    nodeFilters:
      - loadbalancer
  - port: 443:443 # same as `--port '443:443@loadbalancer'`
    nodeFilters:
      - loadbalancer
registries:
  config: |
    mirrors:
      "local-registry":
        endpoint:
          - https://$LOCALDOMAIN:5555
    configs:
      "local-registry":
        tls:
          ca_file: "/etc/ssl/certs/rootCA.pem"
hostAliases: # /etc/hosts style entries to be injected into /etc/hosts in the node containers and in the NodeHosts section in CoreDNS
  - ip: $PUBLICIP
    hostnames:
      - $LOCALDOMAIN
options:
  k3d: # k3d runtime settings
    wait: true # wait for cluster to be usable before returining; same as `--wait` (default: true)
    timeout: "60s" # wait timeout before aborting; same as `--timeout 60s`
  k3s: # options passed on to K3s itself
    extraArgs: # additional arguments passed to the `k3s server|agent` command; same as `--k3s-arg`
      - arg: --tls-san=$LOCALDOMAIN
        nodeFilters:
          - server:0
      - arg: --disable=traefik
        nodeFilters:
          - server:0
  kubeconfig:
    updateDefaultKubeconfig: true # add new cluster to your default Kubeconfig; same as `--kubeconfig-update-default` (default: true)
    switchCurrentContext: true # also set current-context to the new cluster's context; same as `--kubeconfig-switch-context` (default: true)

What did you do afterwards?
once created the cluster, i checked the coredns configmap, and it's ok, it contains:

NodeHosts:
----
<IP_ADDRESS> e4t.local
<IP_ADDRESS> host.k3d.internal
<IP_ADDRESS> k3d-dom1-serverlb
<IP_ADDRESS> k3d-dom1-tools
<IP_ADDRESS> local-registry
<IP_ADDRESS> k3d-dom1-server-0

i then restarted docker (desktop, on mac), container started but the configmap was completely emptied, it now contains just this:
NodeHosts:
----
<IP_ADDRESS> k3d-dom1-server-0

What did you expect to happen
configmap survives a simple docker restart... i confirm this happens even on k3s container stop and restart...
Screenshots or terminal output

Which OS & Architecture
arch: x86_64
cgroupdriver: cgroupfs
cgroupversion: "2"
endpoint: /var/run/docker.sock
filesystem: extfs
name: docker
os: Docker Desktop
ostype: linux
version: 20.10.17

Which version of k3d
k3d version v5.4.4
k3s version v1.23.8-k3s1 (default)

PLEASE note that even the k3s version is wrong, as the cluster config has this ENV...
K3SVERSION=v1.21.14-k3s1
export IMAGE=k3s:"${K3SVERSION}"
Which version of docker
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc., v0.8.2)
  compose: Docker Compose (Docker Inc., v2.6.1)
  extension: Manages Docker extensions (Docker Inc., v0.2.7)
  sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
  scan: Docker Scan (Docker Inc., v0.17.0)

Server:
 Containers: 44
  Running: 9
  Paused: 0
  Stopped: 35
 Images: 70
 Server Version: 20.10.17
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 2
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runtime.v1.linux runc io.containerd.runc.v2
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1
 runc version: v1.1.2-0-ga916309
 init version: de40ad0
 Security Options:
  seccomp
   Profile: default
  cgroupns
 Kernel Version: 5.10.104-linuxkit
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 4
 Total Memory: 2.921GiB
 Name: docker-desktop
 ID: UIUY:OMDB:ECAX:Q4QW:PPCY:U566:SK3J:CWXV:42GG:S7ZX:SJEJ:OVFB
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 HTTP Proxy: http.docker.internal:3128
 HTTPS Proxy: http.docker.internal:3128
 No Proxy: hubproxy.docker.internal
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  e4t.local:5555
  hubproxy.docker.internal:5000
  <IP_ADDRESS>/8
 Live Restore Enabled: false

Any way to implement in k3d the official "coredns-custom" cm, to be created when in config.yaml you define custom hosts or other coredns related mods, instead of the actual method that patches directly the default cm?
https://docs.digitalocean.com/products/kubernetes/how-to/customize-coredns/

@fragolinux see https://github.com/k3d-io/k3d/issues/816

@fragolinux Unfortunately this is expected behavior with regard to Rancher K3s and is not a k3d issue per se.  Rancher K3s rewrites the manifest files whenever it is started and the documentation states that they shouldn't be altered:

Manifests for packaged components are managed by K3s, and should not be altered. The files are re-written to disk whenever K3s is started, in order to ensure their integrity.

When k3d creates a cluster (k3d cluster create), it injects entries into the CoreDNS config map by updating the manifest file /var/lib/rancher/k3s/server/manifests/coredns.yaml, however, when the container is restarted, K3s will rewrite the manifest file (as seen in the log entry: level=info msg="Writing manifest: /var/lib/rancher/k3s/server/manifests/coredns.yaml) and so all the NodeHosts entries in the ConfigMap are removed.  You can't, despite the k3d documentation, even map a volume with a custom coredns.yaml file to this path as K3s will still overwrite it (I've proved this myself).
A workaround for this issue is to run k3d cluster stop <cluster name> and then 'k3d cluster start ` to ensure that the NodeHosts entries are re-injected by k3d.
It is possible to disable the Rancher K3s CoreDNS manifests and replace with your own but, unfortunately, k3d doesn't have an option for injecting the NodeHosts entries into a different manifest file (e.g., /var/lib/rancher/k3s/server/manifests/custom/mycoredns.yaml).

Solved 2 years ago 😅
I just save the coredns cm after 1st start, and reapply on restart
Thanks BTW!

Host entries to the CoreDNS config are now managed via the coredns-custom configmap as per  https://github.com/k3d-io/k3d/pull/1453 so they survive restarts of the cluster and host system.
This is released in https://github.com/k3d-io/k3d/releases/tag/v5.7.0


Update ADOPTERS.md

Updating Adopters List with World Fuel Services
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

You'll need to sign your commit to make the DCO task happy. When I forgot to sign a commit I usually just do it like this:
git commit --amend --sign

and then force push to your branch

@anirudhkurapati I've fixed up this PR in #11103. Please add a comment if you think it looks good! Closing this PR as it's in a less mergeable stage. If you rather feel like comitting yourself just open a new PR that's signed :)


[kafka-consumer] Consumer metrics should have a tag with topic name

Which problem is this PR solving?
Resolves #4777
Description of the changes
Add tag with topic name to Kafka consumer metrics.
How was this change tested?
Unit tests
Signed-off-by: Alex Bublichenko<EMAIL_ADDRESS>

please make sure all commits are signed

please make sure all commits are signed

My bad, sorry, amended with signature.


Fixes #925: add Containerfile test that runs skstat -l

I am not knowledgeable in Rust. If the skstat -l command fails for some reason, do you think the container build should fail as well ?
Besides that, feel free to commit.

I am not knowledgeable in Rust.

Me neither. When you see me type Rust, it is just the last tutorial I gulped trying to get out, all this mediated through IntelliSense and IDE auto fixes.
And writing code is actually easier than reading it, in some cases.

If the skstat -l command fails for some reason, do you think the container build should fail as well ?

I guess. If the image is going to be broken, better  not push it. That should be a good enhancement, yes.

https://github.com/skupperproject/skupper-router/issues/1063


Add validation for auctions, special classification model, status workflow, deleted validation , rectificationPeriod

depends on https://github.com/openprocurement/openregistry.lots.core/pull/28
https://github.com/openprocurement/openprocurement.api/pull/294

Fix travis


Schema editing

Im doing a fully integrated npm library, that combines:
restify,
generate-schema,
jquery,
brutus-json-forms,
datatables,
diskdb
In node, instance restify, and do a init call to microdash, and you
get a full dashboard for your microservice.
The library includes all the assets, serves them, and lets u use json files
for configuration, loggins, kpi etc.
When first using a table,, a default schema is generated.
So I'd love to be able to "edit" the schema.
So it would be lovely to have a full schema definition for "schema".
Have u looked at this?
It would allow me to create arbitratialy complex forms.
Some of the functionality you have, should make this doable.
https://github.com/glennswest/microdash

Hi @glennswest, sounds interesting!
How exactly can I help you?

I would love a "schema" for the "schema"
As you already support variant records, it should be possible to create a "schema" that would allow
the creation of the schema for a form.
Self editing


Change mandatory attribute to customer

Reference: https://start.exactonline.nl/docs/HlpRestAPIResourcesDetails.aspx?name=SalesEntrySalesEntries

thanks @Bramjetten, could you fix the spec too?

Fixed it.


Integration tests & build system enhancements

This is work in progress.
See detailed commits for more.

So far looks good!  Thanks!

@dmp42 Is this ready to go?

@nathanleclaire @ehazlett this is ready to go
The only point I'm not confident with is the release script.
@ehazlett can we look into this later?

LGTM


Add N-Queen in DP

🚀 Feature
The N Queen is the problem of placing N chess queens on an N×N chessboard so that no two queens attack each other.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
N-Queen is one of the most famous DP Implementation. I just saw that there is no N-Queen problem listed on the repository and that's why this issue is made.
Pitch
...

I would like to solve this issue in C++

I want to work on this in both C and Python

I would like to work on this issue in java and C#.

@HarshCasper Can I make another PR for the Python version of N-Queen until this PR is merged?

If the implementation in java is open, please assign it to me

Hi, can I add the implementation in Java for N-Queen problem?


Fix Power overlay

This PR includes two fixes:

Removing --smallfiles command line option since it has removed from latest mongodb - removed-mmapv1-options

Creating project for sock-shop application.

It's a-ok


Spring Tool Suite: Automate (un)installation + add license

Shouldn't we rename the cask to spring-tool-suite based on the filename?
License source: https://spring.io/tools

The Spring Tool Suite as well as the Groovy & Grails Tool Suite are built on top of the Eclipse platform, which is provided by the Eclipse Foundation under the Eclipse Public License.

Shouldn't we rename the cask to spring-tool-suite based on the filename?

Yes.

@vitorgalvao PR updated.

@radeksimko Care to update rebase so we can merge?

Seems to be no longer applicable.


Storage [#904]: Add support for uploading files from folder

Please add the issue number to the description.

Added a new commit based on the specified notes.


Update Seqhash algorithm

Is your feature request related to a problem? Please describe.
Update the seqhash algorithm to give versioned output. For example, pUC19 would be v1_DCD_4b0616d1b3fc632e42d78521deb38b44fba95cca9fde159e01cd567fa996ceb9, with the DCD standing for "DNA-Circular-Double stranded".
Describe the solution you'd like
https://git.sr.ht/~koeng/python-seqhash/tree/master/seqhash/__init__.py similar output to the reference code.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This has been added as of Pr #82. Closing.


Code error:RuntimeError: stack expects each tensor to be equal size, but got [512, 512, 3] at entry 0 and [1, 512, 512] at entry 2

hello, When I run the following command: "bash tools/dist_train.sh configs/samba/samba_upernet-15k_loveda-512x512_6e4.py 2 --work-dir work_dirs/samba_upernet-15k_loveda-512x512_6e4 --amp".
The following error occurs :
RuntimeError: stack expects each tensor to be equal size, but got [512, 512, 3] at entry 0 and [1, 512, 512] at entry 2
seg_label = self._stack_batch_gt(batch_data_samples)
File "/data_sdc/liuTest/Samba/mmseg/models/decode_heads/decode_head.py", line 289, in _stack_batch_gt
return torch.stack(gt_semantic_segs, dim=0)
RuntimeError:  stack expects each tensor to be equal size, but got [1, 512, 512] at entry 0 and [409, 512, 106] at entry 2

Hello! Thank you very much for your interest in our work.
This error occurs during the network's image input handling phase, and our dataloader is able to correctly load the LoveDA inputs. Could you please check if you have applied any additional cropping during data loading? Or if the LoveDA training and validation data you downloaded are correct?
Please feel free to contack us.

I trained with my own dataset and had the following problem:


[question]How to trust the minio server that uses the self-signed certificate?

1.When I use mc client to connect to a server that uses a self-signed certificate, it prompts mc: Unable to initialize new alias from the provided credentials. Get "https://192.168.xx.xx:9000": tls: failed to verify certificate: x509: certificate signed by unknown authority.
2.Trust the self-signed certificate using environmental variables export SSL_CERT_FILE=/home/test/hbd/minioserver.crt
3.I can connect to the server using the MC  client but using the C++ SDK client can not connect the server.
How to trust the minio server that uses the self-signed certificate?

Use SetSslCertFile()


engine_rocks: Add extension method for creating RocksEngine references

What have you changed?
This adds an extension trait, EngineCompat, to engine_rocks, that contains a single method, c, for creating a &RocksEngine type from &Arc<DB>.
The intent here is to create a conversion from the world of the concrete RocksDB engine to the world of the abstract KvEngine with as little syntactic noise as possible. Presently, this conversion is written as RocksEngine::from_ref, and the more the work of abstraction progresses, the more syntactic noise I see it imposing on the tikv codebase. With this change, there will just be a 4-character inline expression added anywhere the conversion happens.
This is a temporary tool, and once tikv is fully converted to use engine traits it will go away.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?
cargo check
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/4184
Benchmark result if necessary (optional)
Any examples? (optional)

I'm finding I need to keep adding to this while working on snapshot abstraction, so I'll hold onto this for now.


How to support usage with open

This is no support for using open and cacheing with an async filesystem. Support this correctly.
This may require wrappign async buffered file, or we just use non async workflow for this. I am not sure of the right way to support this flow


[Feature Request] Pre-Combat Intro Track

During the phase when combat is starting, but before the first turn begins, Combat Music Master does not start playing music.
This is a period when starting a degree of hype music would be welcome.  Something to get players alert as they roll initiative and as the bosses are introduced.
Ideally, it would be both a specific playlist, and something specific to each token in the combat.  It would start the moment combat goes into the rolling initiative phase, and transitions to normal behavior once the GM hits "begin encounter."

That’s a very good idea.
On Sat, 26 Aug 2023 at 3:35, Michael Amygdalidis @.***>
wrote:

During the phase when combat is starting, but before the first turn
begins, Combat Music Master does not start playing music.
This is a period when starting a degree of hype music would be welcome.
Something to get players alert as they roll initiative and as the bosses
are introduced.
Ideally, it would be both a specific playlist, and something specific to
each token in the combat. It would start the moment combat goes into the
rolling initiative phase, and transitions to normal behavior once the GM
hits "begin encounter."
—
Reply to this email directly, view it on GitHub
https://github.com/elizeuangelo/fvtt-module-combat-music-master/issues/15,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7FAZCYLYDHGSUHZNJRMXDXXFAFVANCNFSM6AAAAAA37E56UU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Kind regards,
Apostol Apostolov
Producer, Game Designer @ Gameloft


Add import check

Proposed changes
This PR add a clean installation check to avoid missing dependency issues from the framework
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes
[ ] Public-facing documentation has been updated with the changes affected by this PR. Even if the provided contents are not in their final form, all significant information must be included.
[ ] Any backwards-incompatible/breaking change has been clearly documented in the upgrading document.

Made a temporary commit to check if it works or not and the check works - https://github.com/valory-xyz/open-autonomy/actions/runs/7622558730/job/20760831758?pr=2146

Made a temporary commit to check if it works or not and the check works - https://github.com/valory-xyz/open-autonomy/actions/runs/7622558730/job/20760831758?pr=2146

The check failed.

Made a temporary commit to check if it works or not and the check works - https://github.com/valory-xyz/open-autonomy/actions/runs/7622558730/job/20760831758?pr=2146

The check failed.

It was supposed to, I made it intentional

Made a temporary commit to check if it works or not and the check works - https://github.com/valory-xyz/open-autonomy/actions/runs/7622558730/job/20760831758?pr=2146

The check failed.

It was supposed to

Ok, so you tested that it fails using f092e0f3ac0c0e5a42b9bcc454bf03786a10567f.


[QUESTION] Connection Pool - How to find that a CallableStatement uses connection pool (specifically HikariCP)? SQL Profiler or PerfMon counters?

How to set up a profiler trace to find that my CallableStatement uses connection pool?
I set a trace to capture Audit:Login , Audit:Logout, RPC:Completed. But all I see is 1-Non-Pooled (image below)
Is there any configuration in Hikari/JDBC driver to get the pool to work? I would expect my second call to the service be 2-Pooled in profiler
Hikari configuration (I used both SQLServerDataSource and SQLServerConnectionPoolDataSource for datasourceclassname, both same result ):
public HikariDataSource createHikariDataSource()  {
		HikariDataSource dbDatasource = new HikariDataSource();
		dbDatasource.setDataSourceClassName("com.microsoft.sqlserver.jdbc.SQLServerConnectionPoolDataSource");
		dbDatasource.addDataSourceProperty("databaseName", "tempdb");
		dbDatasource.setPoolName("myJDBCTestConnectionPool");
		dbDatasource.setUsername("someuser");
		dbDatasource.setPassword("totallysafepassword");
		dbDatasource.setMinimumIdle(5);
		dbDatasource.setMaximumPoolSize(10);
		dbDatasource.setConnectionTimeout(300);
		dbDatasource.setIdleTimeout(1000);
		dbDatasource.setMaxLifetime(1000);
		return dbDatasource;
    }

Proc execution code (redacted)
public void execMyProc() {
	HikariDataSource hkds = createHikariDataSource();
	Connection conn = hkds.getConnection();
	CallableStatement stmt3 = conn.prepareCall({? = CALL dbo.connectionpooltest @InputParam1=?});
	stmt3.registerOutParameter(1,Types.INTEGER);
	stmt3.setInt(2,1);
	ResultSet rs1 = stmt3.executeQuery();
        rs1.close();
	conn.close();
}

HikariCP and mssql-jdbc maven pom.xml entries
<dependency>
	<groupId>com.zaxxer</groupId>
	<artifactId>HikariCP</artifactId>
	<version>3.4.5</version>
</dependency>
<dependency>
	<groupId>com.microsoft.sqlserver</groupId>
	<artifactId>mssql-jdbc</artifactId>
	<version>8.4.0.jre8</version>
</dependency>

SQL Code
use tempdb;
go
create or alter proc connectionpooltest (@InputParam1 int)
as
begin
	return @InputParam1;
end;
go

Profiler screenshot:
Java Version : 1.8.281 (Language level 11 or 8)
SQL Server 2017 Developer Edition (same behavior on 2014 and 2019)
Is there anyway to reliably say that my HikariCP configuration is using a pool ? How do I find it using profiler/prefmon/other tools?

hi @praskutti
Although the mssql-jdbc driver should be compatible with Hikari pools we do not directly support it. Your questions relating to HikariCP are best directed to the HikariCP support team.

@lilgreenbird , thanks for the response. Is there a list of directly supported connection pool libraries? Does JDBC directly support connection pooling (like ADO.NET SqlConnection) ?

the JDBC driver does support connection pool please see Using connection pooling for more info. You can also see tests for that in PoolingTests.java.

Hi @praskutti, you can also try turning on logging in the driver.

@lilgreenbird  thank you, we went through it. I am unable to see the "2-Pooled" in the SQL Profiler. But I will dig further.
@rene-ye  , yeah, saw that! Thank you. I have asked my devs to check if we can enable logging for Hikari.

BTW a comment here, there was some confusion in the past because SQL Events (AuditLogoff and XEvent) shows if a connection pool is used or not. However it bases this decision only on the fact if the "reset session" (TDS 7.1 RESETCONNECTION flag) is used.
So if a pool uses normal "rollback" style resets it does not display as pooled even when it was.

Closing the issue due to no actions required from the JDBC team. Please feel free to re-open if any other questions/problems arise.


hierarchical table rows

this is achieved by displaying the table row group header as a table row itself via a toggle-able option in GroupingConfig, and indenting it (and all its child rows) accordingly.
grouped rows can also be collapsed now! i've added a column at the very left that simply contains a collapse/expand button.

closed in favor of #58


Replace magic wand with dice

Thank you for your contribution to the KodaDot NFT gallery.
👇  _ Do a quick check before the merge.
PR type

[ ] Bugfix
[ ] Feature
[X] Refactoring

Before submitting Pull Request, please make sure:

[X] My contribution builds clean without any errors or warnings
[X] I've merged recent default branch -- main and I've no conflicts
[X] I've tried respect high code quality standards
[X] I've didn't break any original functionality
[X] I've posted screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything works
[ ] I found edge cases

What's new?

[X] PR closes #1602

Had issue bounty label ?

[ ] Fill up your KSM address: Payout

Community participation

[X] Are you at KodaDot Discord?

Screenshot

[X] My fix has changed something on UI, a screenshot for others, is best to understand changes.

What are other "dices" as options? Was thinking of something easier recognizable on a small icon. If there is nothing else good around, let's go with this one then

What are other "dices" as options? Was thinking of something easier recognizable on a small icon. If there is nothing else good around, let's go with this one then

@yangwao you have other options listed in the issue #1602

@lohba let's post your KSM address in template

Perfect, I’ve sent the payout https://kusama.subscan.io/extrinsic/0x00915c8de0bdc4a8b91dab001bf9b3c538e397d80139e88fdebd1f177c6eaae0
Let’s grab another issue and get rewarded!
https://github.com/kodadot/nft-gallery/issues/


make account creation work

I am trying to create an account on cloud.neurodata.io/nd/accounts/login/ and recieved the following confirmation email.

Welcome to NeuroData!
To finalize your account, just visit the address below. (If it's not a link, you can copy and paste it into your web browser's address bar.)
http://example.com/nd/accounts/activate/53e747cbf578742244ab072ac669cdaff900b944/
If you didn't sign up for an account, just ignore this message:

The link within this email did not activate my account and therefore I could not login and use your service.
(I know the administrator login, but the whole point is that an external user is currently not able to use the service without us giving them our password, which seems like a bad thing)

I believe this should be a trivial amount of work, as when I replace example.com with cloud.neurodata.io I am able to activate my account successfully.

@gkiar cloud.neurodata.io is off-limits and is under beta. Please do not create accounts/tokens on it until I say so. You are free to read data off it but I cannot guarantee uptime or any tokens you create will be carried over.
Having said that, yes that is the issue and I think @Aeusman was working on registration last. I will co-ordinate with him and get this fixed.

Thank you.
Where do you suggest I test, then? I am not storing any unique (or even large) datasets, but rather am trying to test our service as if I were an outside user.

I view what I am doing as "beta testing," so perhaps upon clarification of my goal this is the right instance for this, but please do let me know :)

@gkiar This is great. Just delete all datasets/projects/tokens you have created after you are done. Also, please don't upload any new data for now. You can beta-test the GUI which is what we need. Some other services aka propagate, stats don't run in the cloud yet.

@gkiar This is resolved now.
@Aeusman @randalburns django by default inserts example.com as the site name when it creates the project. One can modify this manually by using the sites-admin interface. The automated way to do this is to use migrations(django recommends this and not fixtures. django1.7> they moved away from having this as a parameter in the settings file) I wrote a migration in fixtures.migrations which cleans out the django_sites table and inserts cloud.neurodata.io as the site_name. Currently this name is hard-coded in the migration file, we can find a way to load this from custom settings variable at a later stage.


Adding KubeAPILatency alerts for v3

Adding KubeAPILatency alerts as per https://issues.redhat.com/browse/OSD-3037.
For now, adding the alerts with "warn" and "avg" severities which would be reviewed for clusters for some time and then appropriate severity will be set.
Right now, the logic is following:

WARN - Average API Latency for 99 percentile pods with threshold value greater than 1000ms (1 second) for last 10 minutes.
AVG - Average API Latency for 90 percentile pods with threshold value greater than 1000ms (1 second) for last 20 minutes.

TODO: Intention is to identify genuine API latency alerts which are actionable by SRE. The severity will be adjusted after review of above alerts for a month.
SOP: https://github.com/openshift/ops-sop/blob/master/v3/alerts/openshift_master.asciidoc#210-openshift-master-api-call-latency

/lgtm


Build shaded jar for all record readers

Build shaded jar for all record readers by default
Output shaded jar to directory: target/plugins/record-reader/${file-format}-record-reader-package

Codecov Report

Merging #4896 into master will decrease coverage by 20.44%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #4896       +/-   ##
=============================================
- Coverage     56.62%   36.17%   -20.45%
=============================================
  Files          1175     1175
  Lines         62792    62792
  Branches       9208     9208
=============================================
- Hits          35554    22714    -12840
- Misses        24602    38055    +13453
+ Partials       2636     2023      -613

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/pinot/common/http/MultiGetRequest.java
0% <0%> (-100%)
0% <0%> (ø)

...nullvalue/RealtimeNullValueVectorReaderWriter.java
0% <0%> (-100%)
0% <0%> (ø)

...che/pinot/common/restlet/resources/TablesList.java
0% <0%> (-100%)
0% <0%> (ø)

...che/pinot/pql/parsers/pql2/ast/OptionsAstNode.java
0% <0%> (-100%)
0% <0%> (ø)

...ot/core/query/scheduler/TableBasedGroupMapper.java
0% <0%> (-100%)
0% <0%> (ø)

...t/core/segment/index/readers/OnHeapDictionary.java
0% <0%> (-100%)
0% <0%> (ø)

...ommon/utils/retry/RetriableOperationException.java
0% <0%> (-100%)
0% <0%> (ø)

...t/core/query/scheduler/OutOfCapacityException.java
0% <0%> (-100%)
0% <0%> (ø)

...ot/core/query/scheduler/SchedulerQueryContext.java
0% <0%> (-100%)
0% <0%> (ø)

...pinot/common/exception/TableNotFoundException.java
0% <0%> (-100%)
0% <0%> (ø)

... and 573 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 bc87b48...7c8b701. Read the comment docs.


improve multiple workspaces

Thanks for developing this awesome extension
But, The current extension does not perform well in multiple workspaces
Prettier configuration file is not read correctly when multiple workspaces

Should be fixed in 1.1.17 :)


Error when executing commands in PowerShell

[Enter feedback here]
These commands, when entered into PowerShell, return an error:
At line:2 char:74
+ ...  trace -n usbtrace -p Microsoft-Windows-USB-USBXHCI (Default,PartialD ...
+                                                                 ~
Missing argument in parameter list.
At line:3 char:70
+ ... date trace -n usbtrace -p Microsoft-Windows-USB-UCX (Default,PartialD ...
+                                                                 ~
Missing argument in parameter list.
At line:4 char:74
+ ...  trace -n usbtrace -p Microsoft-Windows-USB-USBHUB3 (Default,PartialD ...
+                                                                 ~
Missing argument in parameter list.
    + CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : MissingArgument

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

ID: 9064a690-442c-8ee0-0b29-34022148d9c9
Version Independent ID: b7dc2f55-342b-d846-6617-b06fd9a35021
Content: How to capture a USB event trace with Logman - Windows drivers
Content Source: windows-driver-docs-pr/usbcon/how-to-capture-a-usb-event-trace.md
Product: windows-hardware
Technology: usbcon
GitHub Login: @mhopkins-msft
Microsoft Alias: mhopkins

Hi @MadDenker,
Thanks for reaching out to us.
This article uses the Windows command processor, cmd, as opposed to PowerShell. The first two steps in the instructions say:

Open a command-prompt window that has administrative privileges. To do so, select Start, type cmd in the search box, Select and hold (or right-click) cmd.exe, and then select Run as administrator.
In the command-prompt window, enter these commands to start a capture session:
...

When I run the commands from an elevated command prompt (aka "Run as administrator"), as the indicated in the article, they work as expected.
Microsoft Windows [Version 10.0.22621.1105]
(c) Microsoft Corporation. All rights reserved.

C:\Windows\System32>logman create trace -n usbtrace -o %SystemRoot%\Tracing\usbtrace.etl -nb 128 640 -bs 128
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-USB-USBXHCI (Default,PartialDataBusTrace)
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-USB-UCX (Default,PartialDataBusTrace)
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-USB-USBHUB3 (Default,PartialDataBusTrace)
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-USB-USBPORT
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-USB-USBHUB
The command completed successfully.

C:\Windows\System32>logman update trace -n usbtrace -p Microsoft-Windows-Kernel-IoTrace 0 2
The command completed successfully.

C:\Windows\System32>logman start -n usbtrace
The command completed successfully.

C:\Windows\System32>

Please use the Windows command processor to execute the commands in this article.
Thanks,
~ Mark


Add Version Logging

Adresses #280.
When xcbeautify starts, the semantic version is logged to the console. It uses the semantic version found in Version.swift. It is suppressible via the CLI argument --disable-version-logging.
Should I also make it suppressible via an environment variable?

Yes. I hope, I did not forget anything this time.

This is now available, starting in 2.4.0. Thanks again @ikelax for the contribution!

Curious as to the reasoning for making this the default behavior? In our CI we try to minimize unnecessary output as much as possible. Of course we can do that with the flag, but now we have to remember to put it everywhere lol


integrating with itflow

any chance this can be integrated with itflow certificates page?
https://github.com/johnnyq/itflow/blob/master/client_certificates.php
atm, the validity dates have to be inserted manually, but will be nice to get the sslcheck automatically from the domains page
https://github.com/johnnyq/itflow/blob/master/client_domains.php
any chance can you have a look at this please?
Thank you!

Hi @aftechro,
Feel free to use this code in this repo as a basis for implementation to help that project. There's only about 100 lines of code in this, most of which is the command-line output. The SSL work is only a couple of lines.
I won't be offering an implementation myself since I'm not familiar with ITFlow nor do I have much desire to work on that project to be perfectly honest (Nothing against ITFlow but I have enough projects to distract myself right now).
As a side note, it's not clear if ITFlow is open-source (Outside of source being accessible), Can't see any license in the repo or site.
Will therefore close this off.

Hey, apologies for delay reply.
Thank you for the permission of trying to implement this :D i`ll give it a try and see how that works.
Thank you!


Add support for AVIF

This PR adds support for the AVIF image format.
Intervention/Image added support for AVIF for the Imagick driver in 2.6.0 (https://github.com/Intervention/image/pull/1056) and for the GD driver in 2.7.0 (https://github.com/Intervention/image/pull/1113).

If this PR gets accepted, should I create a separate PR for Glide 1.x? Or is Glide 1.x in maintenance only mode?


why separate usages of bool and other types in flag package?

As been described in https://godoc.org/flag:

The following forms are permitted:
-flag
-flag=x
-flag x  // non-boolean flags only

One or two minus signs may be used; they are equivalent. The last form is not permitted for boolean flags because the meaning of the command
cmd -x *

where * is a Unix shell wildcard, will change if there is a file called 0, false, etc. You must use the -flag=false form to turn off a boolean flag.

But I found not-boolean types also have the "*" problem(eg. A file named as numbers and been passed to a number flag using ), why treat bool as special case?
And in my view the "" problem can be a problem of users' behavior, not a bug of program itself.
Howerver, this optimize can cause some problems when user not familiar with these considerations, as the inconsistent usage may cause confusion.
Can this design be a over-optimization case?

We don't use the issue tracker to ask questions.  Please use a forum instead.  See https://golang.org/wiki/Questions.  Thanks.
The point that the documentation is trying to make is that non-boolean flags always take an argument.  So writing cmd -x * where -x is a numeric flag will always take the first first as the argument, and give an error if the first file is not a number.  With boolean flags we want -x to be the same as -x=true.  If we permit -x false, that means that a command line like -x * will sometimes enable -x and sometimes not, depending on the name of the first file.  That ambiguity means that we cannot accept -x false, but only -x=false.


Fix typo in 0.21.0 release post

ACK 94d0fa84390891a9e63cdc5c501ebe226547d619 , checked the modified text with Google Translate.


Typo isAccessable in Microsoft.Web AppServiceEnvironments.json

Typo in Azure/azure-rest-api-specs/specification/web/resource-manager/Microsoft.Web/stable/2018-02-01/AppServiceEnvironments.json.
Property "isAccessable" should be isAccessible

If you're doing a new version to fix this api, there are 4 others in the same file in propery es

"Defintions",
"Availablility",
"Availabilily",
"diagnosics"


Removed all private modifiers as per review comment

There were many private modifiers, reason why I didnt notice the code avoided it's use. I just went ahead a remove all of them in this PR.
Removed all private modifiers as per review comment at https://github.com/Microsoft/bond/pull/93

Reworded the commit message to follow length line limits and merged from command line. Thx for fixing this.


[integration] Document working with AWS Lambda Functions

Document (on artifacthub) the process and ability to both:

Trigger Keptn sequences from AWS Lambdas
Have Keptn trigger Lambdas in response to task events

https://github.com/keptn-contrib/artifacthub/pull/122


jquery bootstrap wrong loading order?

Dear peoples of the Interwebs!
I've manage to load jQuery and Bootstrap using require.js.
My main.js:
`'use strict';
var $ = require('jquery');
var Link = require('../_modules/link/link');
require('bootstrap-sass');
$(function() {
new Link(); // Activate Link modules logic
console.log('Welcome to Yeogurt!');
});`
Both of them load but bootstrap is constantly spiting out an error: "Error: Bootstrap's JavaScript requires jQuery".
I'm not sure but i think that they load in an invalid order. Is there a way to modify it?

Hey @inspmoore,
Looks like this issue has come up before. Try taking a look Here and see if it helps.
Cheers :beers: !


Add a "Translate this Doc" link to docs footer

At the bottom of each doc page, we have these links:

We should add another that says "Translate this Doc".
If the visitor's locale is the default English (US, Canadian, or otherwise), one of the following should happen:
a) link to Crowdin project landing page. This is very easy to implement but is not the best UX, as the person has to then pick a target language and re-navigate to the file in question:
b) display a modal (or a new page, or a <select> element) that prompts for desired target langauge, then redirects to the Crowdin page for that language/filename pair on Crowdin.
If the visitor's locale is not the default, clicking "Translate this Doc" should take them straight to their language/filename pair on Crowdin. 🎯
I've created a module get-crowdin-file-ids that we weave into the output of the electron-i18n module, so we generate these "straight to the right translation page" URLs from the website.

cc @pierreneter who is working on this.

I moved the get-crowdin-file-ids repo to the @electron org, made it work with the new electronjs.org crowdin proxy, and set up semantic releases for it.
@pierreneter this should unblock you. 🚀


add daskhub example

What does this PR do?
Adds an example composition for daskhub deployment.

I will add a README file for this.

Added readme and removed hardcoded password and api tokens.


Allow setting a specific minute intervals on time picker

Is your feature request related to a problem? Please describe.
My team is building a timesheet application and we have some use cases where we don't want to allow our users to pick precise minutes. In certain cases, we need to use 30 minutes increments and others 5 minutes and others will full precision. At the moment, it does seems like we have a way to control this.
Describe the solution you'd like
An optional prop such as minuteInterval?: number where the number would be used to display the time increments on the clock and only allow the user to select within that range. Should default to 1 as it currently is.
Describe alternatives you've considered
I haven't found any prop that could control this. It doesn't seems like we can override or extend the component to achieve this. If there is, please point me to the right direction!

PRs are welcome!


\addfontfeature should cache fonts

Description
The \addfontfeature command does not cache fonts at global scope.  Why is this important?  It turns out that users don't seem to be aware that \addfontfeature reloads the whole font from disk with adjusted features and that is absolutely detrimental for typesetting performance.  Even worse, commands that seem innocuous, like \textsuperscript can become real monsters, when, in this case by loading the realscripts package, adding \addfontfeature to the definition of \textsuperscript and thus reloads the whole font on every invocation!
However, the usage of \addfontfeature seems to me mostly to be done programmatically, i.e. inside a macro with always the exact same set of features.  It seems, therefore, advantageous to cache fonts at the macro level.  It could also be done in the font loader, which might be possible for LuaTeX, but since XeTeX development has basically been abandoned, there is not much chance there.
Check/indicate

[x] Relevant for XeTeX
[x] Relevant for LuaTeX
[x] Issue tracker has been searched for similar issues?
[x] Links to <tex.stackexchange.com> discussion if appropriate

Minimal example demonstrating the issue
An example for this issue can't be minimal because the problem really only manifests when \addfontfeature is called many times.  However, we can have a minimal example generator, which is only a few lines of Lua.  Here I just read a dictionary from disk.  On Linux /usr/share/dict/words is a file with one word per line.
-- save as make_big.lua
local words = {}
do
    local file = io.open("/usr/share/dict/words", "r")
    for word in file:lines() do
        words[#words + 1] = word
    end
    file:close()
end

local preamble = [[
\documentclass{article}
\usepackage{fontspec}
\setmainfont{SourceSerifPro-Regular.otf}
\begin{document}
]]

do
    local file = io.open("big.tex", "w")
    file:write(preamble)
    for i = 1, math.min(#words, 8000) do
        file:write(string.format("{\\addfontfeature{Style=Alternate}%s}\n\n", words[i]))
    end
    file:write([[\end{document}]])
    file:close()
end

Running
$ texlua make_big.lua

will generate a file, similar to this:
\documentclass{article}
\usepackage{fontspec}
\setmainfont{SourceSerifPro-Regular.otf}
\begin{document}
{\addfontfeature{Style=Alternate}A}

% ... thousands more

\end{document}

Note that I deliberately chose Style=Alternate as a font feature, because Source Serif Pro actually does not have this feature, which worsens the problem because for each time the font is loaded from disk, a multiline warning message has to be written to the log, increasing the amount of disk I/O and slowing down the process further.
Running this in LuaTeX generates the following timing on my machine
$ time lualatex --interaction=batchmode big.tex
This is LuaTeX, Version 1.10.0 (TeX Live 2019)
 restricted system commands enabled.

luaotfload | main : initialization completed in 0.142 seconds
[1]+  Done                    emacs test.py

real	0m39.038s
user	0m39.631s
sys	0m2.421s

However, just adding
\renewcommand\addfontfeature[1]{}

after \begin{document}, the timing reduces to
$ time lualatex --interaction=batchmode big.tex
This is LuaTeX, Version 1.10.0 (TeX Live 2019)
 restricted system commands enabled.

luaotfload | main : initialization completed in 0.141 seconds

real	0m2.532s
user	0m2.406s
sys	0m0.124s

That is a whopping 20 times speedup!
Further details
In fact, if you keep scaling this up, the huge number local macro definitions from expl3 falls on your feet.  In this TeX.SX question, the user actually ran out of strings: https://tex.stackexchange.com/questions/517505
A related topic is Release space in the string pool.

Here is a simple plain TeX reproducer of the problem.
\input luaotfload.sty

\def\mainfont{SourceSerifPro-Regular.otf}

\long\def\onum#1{%
    \begingroup
        \expandafter\font\csname\mainfont:+onum\endcsname="\mainfont:+onum" at 10pt
        \csname\mainfont:+onum\endcsname#1%
    \endgroup
}

\newcount\counter
\counter=0
\loop\ifnum\counter<10000
    \onum{123}\endgraf
    \advance\counter by 1
\repeat

\bye

On my machine this takes 7.968s to typeset.  However, when I replace the definition of \onum by
\long\def\onum#1{%
    \begingroup
        \ifcsname\mainfont:+onum\endcsname\else
            \global\expandafter\font\csname\mainfont:+onum\endcsname="\mainfont:+onum" at 10pt
        \fi
        \csname\mainfont:+onum\endcsname#1%
    \endgroup
}

it suddenly only takes 0.796s!

Thanks for the detailed analysis! I am embarrassed to admit that I thought at some point I did eliminate the inefficiency here — but either I didn't do it correctly or it is a false memory.
I'll do what I can to correct this quickly; I have a few fontspec issues piling up at the moment...

If possible you should add a switch to disable the caching.  It might be that LuaTeX garbage-collects unused \fonts, so if you are using like a thousand different fonts in your document, the caching might actually hurt you.  Not a common use-case for sure, but maybe something to keep in mind.

Has there been any progress on this? Sorry for poking. I am facing this with documents where I use old style num except for siunitx  numbers and in tables where lining numbers are used. However, with this configuration, it adds at least 50%  to the compilation time. It would be great to have some update.

@wspr Apologies for poking as well, but has there been any progress on this since it's been almost a year and a half since the last reply?
Would appreciate it if a reply could be made here, even if there has been no progress yet.

@kwand — apologies, I'm having a hard time juggling family and work at the moment and TeX work has slipped behind. I couldn't promise a timeframe for an update on this, I'm sorry.

I know you guys are busy and have lives, but I am hoping to give this a bump in case it's been forgotten. I'm the asker of the original question on tex.stackexchange that prompted @hmenke to figure out this issue and report it. The performance hit I see on real documents is annoying but not unlivable, but it is annoying.


providers/bitmovin: hls manifest files not pointing to the right sources

When running a HLS job with Bitmovin, the manifests created are pointing to invalid segment URL's.
Example:
job.json
{
  "outputs": [
    {"preset": "240p", "fileName": "manifest_240p.m3u8"},
    {"preset": "360p", "fileName": "manifest_360p.m3u8"},
    {"preset": "480p", "fileName": "manifest_480p.m3u8"},
    {"preset": "720p", "fileName": "manifest_720p.m3u8"},
    {"preset": "1080p", "fileName": "manifest_1080p.m3u8"}
  ],
  "provider": "bitmovin",
  "source": "http://flv.io/KailuaBeach.mp4",
  "streamingParams": {
    "playlistFileName": "manifest.m3u8",
    "segmentDuration": 6,
    "protocol": "hls"
  }
}

Output: https://s3.amazonaws.com/cbsi-encode-outputs/f1a54dbd56cdf180/manifest.m3u8
Level manifests are concatenated with job ID's prefix, I'm working on a patch  to fix this one.

Fixed via #199.


chore: test formData

Add ability to test formData for easier reproduction of issues
test-formData.webm

resolves #382

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Web] Very slow fetching of 40MB js file in debug mode

First sorry to add to a lot of performance related issues, such as #76009.
If this seems to be already covered in a different issue, please close this one.
Description
I am using the fhir package. It includes a generated code file with 17K LOC which results in ~40MB javascript file in web debug mode.
For some reason these 40MB take up to 10min to load in web debug mode.
Even after all other files are fetched in still takes 5min for the remaining 20MB.
Expected behavior
Locally generated file should be fetched much faster.
Reproducible steps
Clone repo: https://github.com/hpi-studyu/studyu
Run study_designer:
cd study_designer
flutter run -d chrome

Image of Chrome DevTools Network tab

Exported HAR file from Chrome DevTools Network tab
https://drive.google.com/file/d/1HYB1IzG8FGe3bKp02JKrAFCjKr4tmBeB/view?usp=sharing
flutter doctor -v
[✓] Flutter (Channel beta, 2.2.0-10.1.pre, on Microsoft Windows [Version 10.0.21364.1], locale de-DE)
    • Flutter version 2.2.0-10.1.pre at C:\Users\nilss\flutter
    • Framework revision<PHONE_NUMBER> (7 days ago), 2021-04-15 12:01:02 -0700
    • Engine revision d2a2e93510
    • Dart version 2.13.0 (build 2.13.0-211.6.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\nilss\AppData\Local\Android\Sdk
    • Platform android-S, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\nilss\AppData\Local\Android\Sdk
    • Java binary at: C:\Users\nilss\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\202.7231092\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    • All Android licenses accepted.

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

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.9.31205.134
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio
    • Android Studio at C:\Users\nilss\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\202.7231092
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 202.8346
    • android-studio-dir = C:\Users\nilss\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\202.7231092
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 11 (API 30) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.21364.1]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 90.0.4430.85
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 90.0.818.42

• No issues found!

I was able to reproduce this and identified a performance problem in our dev server that causes it. Fix pending

@nstrelow I really appreciate the detailed bug with the example that reproduces the problem attached. It makes it so much easier to improve the framework and the tooling

@jonahwilliams Thanks so much for looking into this so quickly.
I am really excited, that this problem is fixable.
Could I already try the improvement by dependency_overriding shelf?
You mention a required flutter change. Is that just an sdk/engine/... roll or an actual code change?
Is this something I could help with?
Sorry for being so overly enthusiastic 😅

Thanks a lot for the explanation, I was able to get it to work.
Now it never takes longer than 5sec. That's a 120x improvement.
I am just blown away, this is pure magic 🧙♂

This has now rolled into the framework and is available on the master channel. There are a few more places we could speed up though, I noticed we spend a lot of time copying the chrome profile and re-writing identical JavaScript. I'll file new bugs for those:
https://github.com/flutter/flutter/issues/81160
https://github.com/flutter/flutter/issues/81161