Documentation is very unclear. It lacks formulas in many APIs.

For example, in SparseCategoricalAccuracy(), the words in this API does not help to give a clear picture to understand what it is doing.  A formula, associated with an example, is clear enough for this API.

@FrankYinXF
Can you please provide the links you are referring to.Thanks!

Can you please mention which formula you need exactly,I might probably be able to help you with that.

@FrankYinXF
I can relate to you on this ... Let me clarfiy it for you.
The SparseCategoricalEntropy is similar to the CategoricalCrossentropy Function.
The only difference being that the y_true need not be one-hot columns instead they are integer values denoting class number.
Let's try it with an example:
You have a 3 class prediction example namely cat, dog and monkey.
cat is assigned as your first label        --> 0
dog is assigned as your second label --> 1
monkey as your third label                  --> 2
Now while predicting through any network the output logits must have a shape of 3 with one for each of the above mentioned class. When you keep the parameter logits=True in the SparseCategoricalCrossEntropy function we feed this layer directly in the loss function as y_pred.
Here the y_true which the loss function expects is not a one-hot column of shape (3,) instead a single integer value corresponding to the class i.e.
0-> cat
1-> dog
2-> monkey
What the above function does is that it first applies a softmax function(if logits=True) and computes class probailities and on these probabilites it applies a crossentropy function. Now the crossentropy here expects inputs of the same shape so sparsecategoricalcrossEntropy makes a one-hot column in a optimized way to compute the loss instead of feeding a one-hot ourself.
It saves us 2 things , first the burden of making a one-hot column every time and also reduces memory usage as we don't need to save labels as one-hot columns. This is especially useful when the number of classes is extremely high say >1000.
Now if logits=False the function expects softmax probabilities instead of the logits as input.

@FrankYinXF
Can you please provide the links you are referring to.Thanks!

Any updates regarding this issue? Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Empty Dashboard "Add View" menu should show view submenu only

An empty DashCanvas/Container displays an "Add View" button, which shows the standard menu.
Somewhat non-ideally, this means the user gets this weird experience, where they must navigate into a submenu to add items:

In many cases, instances might have other menu items - notably a "reset to defaults" option.  I can see how those might be nice to make available here, but the tradeoff in the default case seems unfortunate. Could we build a dedicated menu for this button that skips right to the available widgets, with perhaps an API to add any other arbitrary buttons (or other content) to the empty placeholder?  That would let an app with a reset function highlight that even more clearly.

perhaps an API to add any other arbitrary buttons (or other content) to the empty placeholder?

Would love to discuss this bit - have some ideas but would be good to hear how everyone might want to customize the empty canvas / how much flexibility to allot for here


Misspelled [About] on github

Hi.
It says "Alphine JS" on the About sidebar.
I believe it should be "Alpine JS".

Do you mean, in README? it's fixed now. Any other places?

on the sidebar, to the right

Fixed. Thanks


ShaderMaterial Base64 Encoded Vertex And Fragment Programs

Added support for ShaderMaterial Base64 Encoded Vertex And Fragment
Programs:
{
vertextElement: “base64:xxxxxxxx”,
fragmentElement: “base64:xxxxxxxx”
}

@deltakosh @MackeyK24 in the method _loadVertexShader you added the following code:
            // Base64 encoded ?
            if (vertex.startsWith("base64:")) {
            	var vertexBinary = window.atob(vertex.substr(7));
            	callback(vertexBinary);
            	return;
            }```

Unfortunately the "startWidth" function is not available for the version of TS/JS I compile... so it crashes.

You're definitely right. Reverting

indexOf could be used instead


fixed the auth services : - Add validation and Guard to router links on app route.ts

also Add auth 0 from google


Update shows menu even without child permission

This is  a bug.

Prerequisites
yes
Description

Menu checking somehow shows menu entries that the user doesn't have permission for now
Steps to Reproduce

Have a menu structure with a child that a role doesn't have access to (f.e. foo.bar.index), being a child of a has_children foo menu
Log in as that user, the api/core/home request still returns this menu, even if the user doesn't have the permissions for that route

Expected behavior

TreeBuilder to only return menu items that the user(role) has access to
Actual behavior

Seemingly returning all menu items (from a brief glance system & admin is also returned)

Note: I have an overridden AppState (residing under App\Http\Responses) however I took over the updated AppState from the 3.3 upgrade and haven't changed any BE code related to the menutree; so that should be unrelated

Trying to reproduce I removed the system.tutorials.index permission from the admin role and reloaded the window and everything looks ok.

Admin wouldn't cause it indeed...
I just found the culprit
in TreeBuilder.php's permissions() you check the permissions of the first user, instead of the current user

so the permissions (menu structure) would only be correct for users with the same role as the first user in the DB

Oh wow....


docs/guide-pt-BR/security-authentication.md - translate [ci skip]

3 files:
docs/guide-pt-BR/output-data-providers.md
docs/guide-pt-BR/output-theming.md
docs/guide-pt-BR/security-authentication.md
Thanks

Thank you!


Remove Cuda feature (superceeded by OpenCL)

This pr removes Cuda support as it will be supported with OpenCL implementation.

One more thing: would be nice if you went through docs and updated them clarifying that CUDA is only used from within OpenCL now, it still mentions separate CUDA support


Use evergreen Claude versions.

Updates Claude model names to claude-v1 and claude-instant-v1 so that both pickup the latest compatible models. E.g. claude-v1.3 is available as of today.

Thank you!

Thank you!


Added a default to notAfter to make it work.

Current snippet won't work - if an user copy from the document and run in PowerShell prompt - as-is. Because the $notAfter variable is not defined, as it did for password. I have added a default value (6 months) for the $notAfter while declaring it.

Docs Build status updates of commit f8e35a0:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azureadps-2.0/signing-in-service-principal.md
:warning:Warning

Details

docs-conceptual/azureadps-2.0/signing-in-service-principal.md

[Warning] Missing attribute: title. Add a title string to show in search engine results.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Browsers render "top" textBaseline inconsistently

I know you changed the default textBaseline from "alphabetic" to "top" in version 0.5, however FF positions the text differently than Chrome/IE9 (Latest stable builds on Windows). This can also be seen in the Inheritance tutorial that comes with the latest build of easeljs. Maybe setting the default back to "alphabetic" would be better until this issue is resolved, or adding a warning in the documentation.

It seems most browsers are pretty consistent with the baseline these days, but in CreateJS the difference seem to have increased. Is there something in the code to compensate for the differences which is now basically overcompensating?
It seems only 'middle' and 'alphabetic' are consitent in CreateJS.
To compare different browsers I used: https://www.w3schools.com/TAgs/playcanvas.asp?filename=playcanvas_textbaseline&preval=top

@arnovanoordt There is no compensation in CreateJS.


Huge increase in install size

https://packagephobia.now.sh/result?p=netlify-cli
v2.12.0: 79.7 MB
v2.13.0: 117 MB
Also, you guys even have husky listed in dependencies which results to errors when installing via npm. Not sure if there are even more devDependencies listed wrongfully in dependencies.

There's definitely more devDependencies wrongfully listed in dependencies after v2.12.0.
https://github.com/netlify/cli/compare/v2.12.0...v2.13.0#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R61-R134
lint-staged, mocha, chai probably too.

As of 2.30.0 it seems to be 151.3M, the largest dependency in our project by a factor of 5. Any plans on shrinking it down?

v2.41.0 was 219MB 😛<EMAIL_ADDRESS>TBH this is a very serious situation security-wise also. The more packages one depends on, the bigger the risk all people who use the netlify-cli package are put into.

We want to be minimal with dependencies and package size but compared to other issues this one is not a priority right now. We understand that this is not optimal. And since this is an open-source project, we would invite you (the community) to help us fix it. If you can create a PR to optimize dependencies or reduce package size in general. We would gladly accept it.

An increase happened again recently :/
https://packagephobia.com/result?p=netlify-cli
There are plenty of unmet dependencies, vulnerable ones etc. This issue should be higher priority for sure.

Thank @XhmikosR, the latest bump is probably due to https://github.com/netlify/cli/pull/1469.
Don't think that's a major concern at the moment as the binary files (which are contributing to the size increase) are optionalDependencies so you'd only get the one that matches your platform.
For vulnerable ones we have https://github.com/netlify/cli/issues/1497 which we are currently looking into as a part of https://github.com/netlify/netlify-plugin-edge-handlers/issues/97.
If you know of more can you please open specific issues for those?
I opened 2 issues regarding unmet and deprecated ones:
https://github.com/netlify/cli/issues/1527
https://github.com/netlify/cli/issues/1528

TBH at this point I care about the size. 231MB and counting. With so many dependencies, the chances of something going wrong are too many. It's just so many bytes wasted, it takes minutes to install on my VM. It's just not a a good sign for the package.

https://packagephobia.com/result?p=netlify-cli
https://npm.anvaka.com/#/view/2d/netlify-cli

Also, https://lgtm.com/projects/g/netlify/cli?mode=list. Personally I'd add CodeQL too.

3.4.3 | Publish Size: 575 kB | Install Size: 236 MB | Publish Date: 2021-01-26 | Publish Files: 287 | Install Files: 22761

And the size keeps increasing. With every release, more packages are added, the bigger the size and the risk something goes wrong in one of the dependencies...

3.4.3 | Publish Size: 575 kB | Install Size: 236 MB | Publish Date: 2021-01-26 | Publish Files: 287 | Install Files: 22761

And the size keeps increasing. With every release, more packages are added, the bigger the size and the risk something goes wrong in one of the dependencies...

Hi @XhmikosR. Can you share some more details about any limitation you're experiencing? We haven't added any new dependencies in a while, and when we do, we're very rigorous about the vetting process to ensure code quality and good security practices.
If you have any additional concerns, or you have suggestions about a specific aspect of the codebase or any dependency we're using, feel free to reopen the issue or create a new one. Until then, I'll close this one.
Thanks for your input!

Hi @XhmikosR. Can you share some more details about any limitation you're experiencing? We haven't added any new dependencies in a while, and when we do, we're very rigorous about the vetting process to ensure code quality and good security practices.
If you have any additional concerns, or you have suggestions about a specific aspect of the codebase or any dependency we're using, feel free to reopen the issue or create a new one. Until then, I'll close this one.
Thanks for your input!

Well, I can't reopen the issue myself since you closed it...
There's nothing more to add to the issue itself; netlify-cli is ~236MB, needs ~22.7K files to install, it takes a long time to do so (more than 90s on a Windows 10 machine with an NVMe driver), and poses a pretty big security risk due to the number of dependencies.
Now, you may choose to close the issue, but the gist remains the same.

Well, I can't reopen the issue myself since you closed it...
There's nothing more to add to the issue itself; netlify-cli is ~236MB, needs ~22.7K files to install, it takes a long time to do so (more than 90s on a Windows 10 machine with an NVMe driver), and poses a pretty big security risk due to the number of dependencies.
Now, you may choose to close the issue, but the gist remains the same.

In November 2023 it's 281 MB and 48,028 files


Implement def-use chains with value semantics

The def-use chain of a value is currently stored in that value. The issue with that approach is that it compels values to have reference semantics, so that their def-use chain can be updated whenever a new instruction is created.
If def-use chains were all stored in an external data structure owned by the module, we could give them pure value semantics. Further, the builder would no longer to allocate them in the shared buffer to guarantee uniqueness.

Implemented in #760252e.


Add a comment to TempRValue optimization.

A recent fix added some code that is inconsistent with the design of
the pass, creates an implicit coupling between separate parts of the
optimization, and relies on incredibly subtle reasoning to ensure
correctness.


Append (not replace) default encodings to custom encodings

Same as #351 but for the latest and greatest.
From #351:
Before, any custom "accept-encoding" header via the wrap-accept-encoding middleware would be replaced by the wrap-decompression middleware. The new wrap-decompression middleware tries to merge any existing values in the "accept-encoding" header.

@atominthegrass would you be able to add a unit test for this?

Sure thing -- the added tests will fail on master and pass with this PR

Merged, thanks!


can you make this use yolov8 pose

can you make this use yolov8 pose https://docs.ultralytics.com/tasks/pose

Sadly, not really.
Yolov8 is only a 2D pose estimator, meaning all it does is find where certain joints are on an image. There is no depth information, or how these keypoints are positioned in 3d space, so we cannot use it for FBT.
If a model existed that output a 3D root relative skeleton same as mediapipe, then it would be possible to convert the code to use it. However, I have yet to find a model that would do that both as accurately as mediapipe, and also as fast (it runs realtime on a cpu, something that even the smallest yolov8 pose cannot do). Googles data and compute power is hard to beat...

I found this repository:
https://github.com/SkalskiP/sport

Yeah sport would be cool to have work as full body track but that requires 2 cameras


Reports

pip 22 has a --report  option that outputs a JSON record of what happened when installing - seems useful

The report option is nice, but I didn't find the verbose option in the docs.

Try it on command line - good way to see what to expect for verbose + report options
report may only be on install?


Resolves #2725

Add note for proper node cleanup.
https://github.com/MicrosoftDocs/windows-powershell-docs/issues/2725

Docs Build status updates of commit f2c4f5d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2016-ps)
Details

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2019-ps)
Details

docset/winserver2012r2-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2022-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2016-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2019-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

I was a bit hesitant about using 'However' but couldn't think of anything better.

I was a bit hesitant about using 'However' but couldn't think of anything better.

@JohanFreelancer9 any ideas to improve the sentence? or leave as is?

Docs Build status updates of commit 56581aa:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2016-ps)
Details

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2019-ps)
Details

docset/winserver2012r2-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2022-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2016-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2019-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Cleared assignees by mistake with this one. The PR is ready to be merged. Thanks. @tiburd

Docs Build status updates of commit 3244081:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2016-ps)
Details

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2019-ps)
Details

docset/winserver2012r2-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2022-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2016-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2019-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 67e99d3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md
:warning:Warning
View (WindowsServer2019-ps)
Details

docset/winserver2012r2-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2016-ps)

docset/winserver2022-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md

Line 0, Column 0: [Warning: link-out-of-scope] File 'winserver2019-ps/Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' referenced by link '../Microsoft.Windows.ServerManager.Migration/Uninstall-WindowsFeature.md' will not be built because it is not included in build scope.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit c22ee4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2016-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2016-ps)

docset/winserver2019-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

docset/winserver2022-ps/adfs/Remove-AdfsFarmNode.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Use interrobang for default prompt

Bit of a playful suggestion!  When unicode is supported by the terminal, use an interrobang (‽) for the prompt.  This makes sense because while the focus of intero is adding more queries (?), the prompt is also used to execute commands (!).  The two together is the interrobang‽‽‽

I like this, it's cute and unique!
On 22 May 2016 03:14, "Michael Sloan"<EMAIL_ADDRESS>wrote:

Bit of a playful suggestion! When unicode is supported by the terminal,
use an interrobang (‽) for the prompt. This makes sense because while the
focus of intero is adding more queries (?), the prompt is also used to
execute commands (!). The two together is the interrobang‽‽‽
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/chrisdone/intero/issues/40

\o/ I didn't realize this would be an emacs change, but I suppose that's better than assuming a unicode terminal.
I was going to suggest borrowing the unicode detection code from stack: https://github.com/commercialhaskell/stack/blob/master/src/Stack/Types/StackT.hs#L111
But it turns out that stack borrowed it form GHC!  So it should be available.  Anyway, no big deal :)


CI: Pull latest image before rebuilding

Because we rebuild from scratch every time, every docker image is unrelated, even though there may be no difference in the docker file. The primary goal of this PR is to reduce redundant downloads for users of the poldracklab/fmriprep docker images on new releases.
It has a futher advantage of nearly halving the image creation stage on Circle:
Before

After

The failure on saving the image is a little weird. I'll see if that keeps happening.

Okay, that seems to be working. Is there any reason this is a bad idea?
There is the issue that if a major change happens near the top of the Docker file, the time savings are going to be negated and have the additional cost of the pull; under those conditions, it would probably be a good policy to comment out the line until the next release updates :latest.
apt packages will get frozen for a while, though that might be a feature, rather than a bug.
And it may still be worth rebuilding from scratch every so often. That could be done by pushing a clean-built poldracklab/fmriprep:latest just before a release, so the release image is based on the fresh build.

Tests passing. Thoughts?

So in principle this should speed up builds and make them less dependant on say MGH servers?
If this works we could turn off saving and restoring the docker image using .tar (to speed things up a little more) - right?

Right. This should only hit MGH if either docker hub goes down or the Dockerfile has changed to the extent of needing it.
I don't want to kill the cache just yet. I'm curious if after a couple rounds through, this will actually make the cache work; the pull might give the needed context to the layers that are saved in the .tar file.

Merging. If we don't see a speed-up after a couple more PRs, we can remove the caching.


[Enhancement] Add the ability to modify belt hints

Using this mod, it is possible to change the inputs/outputs of logistics stations and the recipes used by buildings, but it is not possible to change belt hints.  For example, changing my smelters from iron to copper, I would like to change the iron ore and iron bar belt hints to copper ore and copper bar hints.

This feature was added in version 1.3.1


Ensure thread safety for go interpreter

See https://github.com/dop251/goja#is-it-goroutine-safe

Is it goroutine-safe?
No. An instance of goja.Runtime can only be used by a single goroutine at a time. You can create as many instances of Runtime > as you like but it's not possible to pass object values between runtimes.

This is not an issue. EM always creates a new goja Runtime when loading an extension and it is only used for a single request.


Suggestion: useHead

Hey, nice work on this!
I'm planning out a possible refactor of how we handle stuff in <head> and a possibly nice addition to this library struck me. Several apps I've worked with in the past has represented all this data in an object, building that object up in a Redux selector or by some other method, then passing that head-object to a specialised component that maps it to Helmet.
Maybe there is an opportunity here to get rid of boilerplate in userland in these cases by providing a useHead-hook that is a combination of all the others and takes an object, or array, as an argument?
This was also inspired by Remix, where returning a { meta: {} } from preload-functions will be supported out of the box.

Hey @Ephem
Welcome and thanks!
I do like the idea behind this but it feels pretty hard to keep performant. I guess most of the time this array would be fresh on every render meaning we’d have to traverse this and check it with the DOM. In our plain useMeta, useLink,... we can take advantage of the flat structure of the argument to prevent this check from being needed.
There are ways to possibly circumvent this but I fear these might contradict the “rules of hooks” for instance iterating over said array and just using the hooks but if the array becomes larger/smaller this will induce errors in React.
Lastly I think this might also become a significant increase in size if we’d do the first approach (call dispatcher for meta/title/titleTemplate) and manual for links (since these can be duped).
What are your thoughts on this?

Those are valid points!
I guess it's one of those cumbersome "you should memoize this in userland" kind of scenarios, which wont work because people don't read the docs or like to memoize things. :P Then again, if the component where hooked head is used does not re-render often, it's probably not a problem to start with, but it does push that complexity to userland either way.
Without being familiar with the implementation I guess one way to at least avoid DOM access would be to keep track of the last applied value for this particular tag?
While I do feel this probably has the same complexity regardless of being implemented in a library or directly in apps (like we do now), I'm definitely emphatic to the payload increase. This library is very lean right now which would be a shame to increase if this is functionality that few people use and I'm guessing this would be hard to build in a way that it minifies away if unused. :)

I've been giving this more thought and this might be the ideal use case for a Babel Macro this could allow us to export a hook called useHead which spawns a function that keeps the length of the previous execution and compares it to the current execution. When it sees we're adding a new meta tag it will throw and suggest an empty placeholder for it.
const macro = (arguments) => {
  arguments.forEach(() => {
    if (meta) useMeta()
  });
}

To avoid breaking the rule of hooks? I think the problem with that is that metadata is often generated from dynamic data not known at build time, so you can't always predict which placeholders to add.

Since this is a development tool it only warns the user, it does not attempt to predict what the new values will be.
It only transforms useHead into a series of the native hooks.

But it does this at build time right?
We have a component that stays mounted between page transitions which could theoretically use this hook. We fetch the metadata for each page from an API at runtime so arguments are not known ahead of time, and they change in length between different pages. Some pages have og-ratings and some pages do not for example, but in reality a whole bunch of things is removed and added.

Hmm, the goal for me with this library was to allow for granularity as opposed to, how I perceived, massive blocks.
<helm>
  <meta />
  ...
</helm>

Where here it would be a useMeta/... on a subcomponent for granular control.

I'm not quite sure I understood your last comment fully, but I totally get that you want the API lean and focused on granularity as opposed to what might very well be a pretty niche usecase, that can also be pretty easily built on top. :smile:
I wasn't sure if it was a good fit myself and just thought I'd spin the idea by you, thanks for taking the time to discuss!

@Ephem no need to close this, I do agree on it being a nice feature, the rules of hooks however make this a very tall feat because, every render should have the same amount of hooks.

Oh sorry, I misunderstood, I thought you were saying those hindrances made it hard to fit this feature into this library. I'll reopen. 😄
I think it would be very hard to have this as a grouped hook for that reason. I've thought some more about this, and maybe leaving memoization up to the user isn't such a bad approach after all? In the bigger picture, I think memoization needs to become the standard when passing objects to custom hooks or every single custom hook out there will have this problem. Doing deep compare by default because users might forget to memoize will probably hurt more in the long run.
If you think that is too much responsibility to put on the user, I think either keeping track of the last values that was queued to the dispatcher in the useHead hook and avoiding queuing them if they are the same OR keeping track of the last values that were added/updated to the DOM inside the dispatcher itself and avoiding adding/updating them if they are still the same is probably the best way to go. If you add that logic to the useHead hook itself and that is properly tree-shakeable, that might mean the payload doesn't have to go up for users not using it?

Implemented in<EMAIL_ADDRESS>


Upgrade Hystrix

Hystrix 1.5.11 was re-released as 1.5.18 because this is the version that is internally used at Netflix and is considered more stable, s. https://github.com/Netflix/Hystrix/issues/1891 and https://github.com/Netflix/Hystrix/releases/tag/v1.5.18

Codecov Report

Merging #3281 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3281      +/-   ##
============================================
+ Coverage     65.28%   65.35%   +0.07%
- Complexity     1477     1481       +4
============================================
  Files           188      188
  Lines          6965     6965
  Branches        847      847
============================================
+ Hits           4547     4552       +5
+ Misses         2104     2103       -1
+ Partials        314      310       -4

Impacted Files
Coverage Δ
Complexity Δ

...oud/netflix/zuul/filters/post/SendErrorFilter.java
76.36% <0%> (+3.63%)
17% <0%> (+1%)
:arrow_up:

...x/security/SecurityContextConcurrencyStrategy.java
77.77% <0%> (+16.66%)
8% <0%> (+3%)
:arrow_up:

Continue to review full report at Codecov.

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

@ryanjbaxter Sure!

I created a new PR (#3288) against the 2.0.x branch. Closing this one.


labgraph monitor

Thanks for the PR @Zina-Kamel! Do you have any more details on specifically what this PR accomplishes that can be added to the description?

Please use the following LabGraph template for pull request(s): https://github.com/facebookresearch/labgraph/blob/main/.github/PULL_REQUEST_TEMPLATE.md


Slider inside Slider??

Hi,
I need to build a Slider inside another Slider... it is a bit strange but have someone done this before?
Thanks!

Please read the contribution guidelines :)


[AD-1880] JWPlayer Bid Adapter

This PR adds the JW Player Bid Adapter.
The adapter modifies the incoming request to be formatted in accordance to AppNexus' expected oRTB params.
Pending Questions
Should the endpoint be written in config.go ?

I am still getting through the rest of the code, but here is a file checklist from the documentation. It seems like you still need to modify config.go, create a jwplayer/params_test.go file, and the test jsons.

#19 122.4 --- FAIL: TestDefaults (0.10s)
#19 122.4     config_test.go:986:
#19 122.4         	Error Trace:	config_test.go:986
#19 122.4         	            				config_test.go:125
#19 122.4         	Error:      	Received unexpected error:
#19 122.4         	            	validation errors (1 error):
#19 122.4         	            	  1: There's no default endpoint available for jwplayer. Calls to this bidder/exchange will fail. Please set adapters.jwplayer.endpoint in your app config
#19 122.4         	Test:       	TestDefaults
#19 122.4         	Messages:   	Setting up config should work but it doesn't

@dshore I believe I resolved this. make dev-local ran fine for me with these changes.
@jorgeluisrocha  the files you mention are optional. I added them. The **json ones are examples. I think it's great for documentation and onboarding. We can add more examples  as we add features


Dependencies need updating: flutter_inappwebview

Please update flutter_inappwebview to version 5. As you are using version 4, this relies on an old version of uuid (2.0) which is causing problems with newer dart packages.

Any workaround at present?

Not that I have found, I tried dependency overrides but had no luck due to breaking changes introduced in newer versions.
Really hoping @sarbagyastha will update the package soon.

Found a PR with the update: https://github.com/sarbagyastha/youtube_player_flutter/pull/449 - I'm going to test this now

@jimmyff when do you plan to implement it?

There is a fork for youtube_player_iframe which fixes your problem.
https://pub.dev/packages/youtube_plyr_iframe

@jonatadashi that was my plan, then realized I am using youtube_player_flutter too. So there a need for an upgrade in youtube_player_flutter too :(

@MeAtPros I'm not implementing the change (I've previously sent PRs but @sarbagyastha doesn't tend to accept them). I'm just utilising the fix PR sent by filipedfs here: https://github.com/sarbagyastha/youtube_player_flutter/pull/449. I've forked his change and using it like so (I suggest you fork your own rather than using mine):

dependency_overrides:
  youtube_player_iframe:
    git:
      url: https://github.com/jimmyff/youtube_player_flutter.git
      path: packages/youtube_player_iframe

This will tide me over until @sarbagyastha accepts the PRs or updates the package himself.

fwiw, I recommend using @popshootjapan 's fork (master branch) for now. @filipedfs 's fork (develop branch) does not include updating inappwebview for the youtube_player_flutter package (only the iframe package), and when I added that update to its pubspec.yaml, I got build-time errors.
@popshootjapan 's fork ran fine, and includes a bunch of (hopefully sound!?) changes to support null safety.
dependency_overrides:
  youtube_player_flutter:
    git:
      url: https://github.com/popshootjapan/youtube_player_flutter.git
      path: packages/youtube_player_flutter

@sarbagyastha, with the last commit being 3 months ago, and no apparent movement here, what's the story on maintenance for this repo? Are you looking for community support? I need something that I can count on in production.

I was occupied with other stuffs for last couple of months. I'm planning to update the package within this week or next.

Just wondering - any plans to update the dependencies?

This seems to be resolved. @jimmyff @sarbagyastha


ReactNativeModuleBase not resolved in App

Hi guys!
I'm having a problem building my test application
I followed your instructions from the readme, made a clean Expo module, made a Kotlin multiplatform module, added annotations, generated classes
The problem occurred on Step 4
I'm trying to register a generated class but I'm getting a class incompatibility error.

Am I missing some step?

The generated class RN module class will be called {MyClassName}Android. So it looks right if your RN module class is called Greeting.
However you can also skip step 4 and use auto generated Module Providers instead, they are available in common code and are called {MyClassName}Provider you can call getModule in android and ios and get the respective platform implementation.

Yes, I checked the option with the provider, but got the same error

Ok from the error message I would suspect that you are missing the de.voize:reakt-native-toolkit dependency in your project. Can you please follow the Installation instruction from the Readme.

The dependency was definitely added to the shared_library project, otherwise the classes would not have been generated. Do I still need to connect reakt-native-toolkit to my Android application?

What do you mean with "connect" the "android" project will depend on the "shared" project so you can access the shared code including the generated RN module classes. In which project is the MyRNPackage class?
Also, react native 0.20.1 looks odd to me, this is a very old version?

If the MyRNPackage is in you android project you also need to add the toolkit dependency to you android project.

@kcn1-71 do you still need help with the project setup?

Yeah, sorry, just forgot to close the issue


Investigate how to use filtered Vector Search to improve search results

Vector Search - specifically k nearest neighbor - is very sensitive to outliers. Filtered vector search gives the ability to reduce the search space prior to performing vector search. This is being advertised in Pinecone's marketing materials significantly, so we need to figure out how to perform this. Tooling such as langchain actually don't make it possible to do filtered vector search.

@hemidactylus new metadata search I think can help - maybe its the first version of it


test-suite: Remove references to NHS number from the "no-nhs-number" scenario

The inclusion of an NHS Number within the XML sample data file appears to be a mistake as logically there shouldn't be an NHS Number specified according to the PatientRole.id field of the ITK Spec: https://nhsdigital.github.io/111-ITK/Domains/Templates/Tabular View/COCD_TP145201GB01-NoEdit.html#PatientRole
The inclusion of a no NHS Number scenario within the TestPack also appears to be a mistake given that you can't specify the NHS Number for this scenario.

Keen to replace the spreadsheet with something more useful, but that's going to be a bigger chunk of work.

The inclusion of an NHS Number within the XML sample data file appears to be a mistake as logically there shouldn't be an NHS Number specified according to the PatientRole.id field of the ITK Spec: https://nhsdigital.github.io/111-ITK/Domains/Templates/Tabular View/COCD_TP145201GB01-NoEdit.html#PatientRole
The inclusion of a no NHS Number scenario within the TestPack also appears to be a mistake given that you can't specify the NHS Number for this scenario.

On this page I see: "The HL7 attribute id allows identification of the patient using the NHS number or a local identifier if the NHS number is not available." Also, there can be a scenario where there is no Patient ID.
However, the test data spreadsheet still looks wrong (assuming we're talking about row 2 - Jira ID NIAD-2127). They're testing having no patient NHS number, and the test data has an NHS number. If this was to test a scenario with no patient id, I would expect an OID of 2.16.840.1.1138<IP_ADDRESS>.<IP_ADDRESS> and also to see some local identifier data in the test data, which I don't see.
I'm not sure I should approve this PR, however, as it removes OID 2.16.840.1.1138<IP_ADDRESS>.<IP_ADDRESS>, which according to the documentation, is for unverified NHS numbers... is there a reason why this needs removed?


ListenForAttributeChange crashes editor when changing attribute values

Steps to reproduce:

Open the project in Unreal Engine 4.25
In UI_HUD, fiddle the Attribute value in ListenForAttributeChange. Alternatively, change the value of the array for ListenForAttributesChange.
Saving the blueprint causes an editor crash.

It's an engine bug in 4.25 - all BP nodes (GetFloatAttribute for example) with Attributes as parameters cause this crash. I submitted a bug report to Epic. I'll comment again with the issue # when they confirm it. Good find!

In the meantime, you can convert all of the Attribute pins to variables. That does not crash.

Bug report has been accepted. Issue UE-92787
https://issues.unrealengine.com/issue/UE-92787

Great, thanks for posting the UE bug and especially for providing the workaround. I couldn't decipher what exactly was going wrong; I'm glad you were able to isolate it more carefully.

Do we have news about this ? It still crashes for me :<

4.25.1, released today, fixes the crash.


The Always on Top  enhancement of FancyZones

Description of the new feature / enhancement
Implying window jacking in FancyZones
Scenario when this would be used?
When user using FancyZones,they will ba able to use Layout Editor to create a zones which is always on the top.
Supporting information
Uesr needs to enable  Always on Top first.

/dup https://github.com/microsoft/PowerToys/issues/919


Not all Starstations have LocTypeStarStation tag

Many of the star stations do not have the correct location tags on their Location record.

Details
No examples yet, please comment with the names/locations of an stations so they can be checked.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Check star station location keywords

The following are missing LocTypeStarStation. Some are also missing LocTypeStarstationInterior and LocTypeStarstationExterior depeding on location. Another keyword some have and others don't is LocTypeSettlement.
All are Editor IDs
RL083StaryardSpaceLocation
RL083StaryardLocation
StationTheKeyDefensiveBatteryAlphaSpaceCellLocation (unsure, added anyway)
StationTheKeyDefensiveBatteryBetaSpaceCellLocation (unsure, added anyway)
StationTheKeyDefensiveBatteryGammaSpaceCellLocation (unsure, added anyway)
SettleDeimosStaryardSpaceCellLocation
SettleDeimosStaryardLocation
LC099NovaStaryardInteriorLocation
LC099NovaStaryardSpaceLocation
LC099NovaStaryardLocation
PolvoStationLocationSpaceCellLocation
DR019SpaceCellLocation
DR019Location
LC175StationRE939Location
LC175StationRE939InteriorLocation
LC175StationRE939SpaceLocation
LC122InteriorLocation
LC122SpaceCellLocation
LC122Location
SettleStroudEklundStaryardSpaceLocation
SettleStroudEklundStaryardLocation
DR033SpaceCellLocation
DR033Location
StationTheClinicSecureWingLocation
StationTheClinicSpaceCellLocation
StationTheClinicLocation
StationTheDenSpaceLocation
StationTheDenLocation
StarStationTheEyeSpaceCellLocation
StarStationTheEyeLocation
DR032DerelictStationSpaceCellLocation
DR032DerelictStationLocation
SettleTridentLuxuryStaryardSpaceCellLocation
SettleTridentLuxuryStaryardLocation

Oh god that formatting. Posted on the discord as well.
https://discord.com/channels/916686954421157930/916692282021916702/1175535709713608746

After actually playing some main quests (:D) I found some more missing keywords.
SY-920
LC043SYSpaceCellLocation, needs 0001674D and 0008A8F9
LC043SY03Location, needs 0001674D and 0001674E
LC043SY02Location, needs 0001674D and 0001674E
LC043SY01Location, needs 0001674D and 0001674E


Markdown syntax highlighting?

I don't seem to be able to view any syntax hightlighting for markdown text. Maybe I've got  something configured wrong? Is there default syntax highlighting for .md files that's just not showing up for me?

Syntax Highlighting is not an option for some languages yet

Its available


Custom Speech: add en-US structured text sample

Purpose

...

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ X ] Documentation content changes
[ ] Other... Please describe:

@BrianMouncer Could you please help review and approve? :) (This is for CLM public preview samples)

Could you help merge this to PROD?

@andreasjl i saw this PR has been approved for a month but haven't merged (i'm not authorized). Would you proceed?

@ArcherAZ I am not authorized, either. Maybe @mahilleb-msft can merge it?


Memory issue

Great work, thanks.
I have an issue about the memory.
In the example, the features have small dimension, eg, 3.
When I try larger dimension, I will face out of memory issue.
e.g., I tried
data = np.random.rand(100, 300)
it takes more than 100G memory.
Do you have any idea about this issue ?
Thanks.

Hi @daodaofr,
I am aware of this issue. It has something to do with the way high numbers of dimensions are handled internally. Consider it a bug for now. I am looking into it. For now, however, it is simply not possible to process this or you have to buy a machine with a lot of RAM :P.
Cheers


Fixes some of the type hints

Adds some more strict type comparisons

@sleekslush @louisgv  It seems like after this change the following code is now typed as boolean | undefined instead of just boolean for the value variable, is this expected?
const [value] = useStorage("some-key, true)

@goenning I think that reflects the actual type more closely, albeit it diverged from the sync useState API
Since storage is an async API, the initial value can be undefined.

Isn’t the second param meant to be the default though?
It seems to work that way

Yea, np. I didn't spend a ton of time on this originally so I'll try to take a deeper look this weekend and fix that issue


fix(explorer): fix amend tx view not to show old price

Related issues 🔗
Closes #5823
Description ℹ
The amend order view could show the old price in some circumstances (depending on the shape of the amend). Now it doesn't.

I'm not sure how to test this ... but there is some suboptimal icon stacking going on in this view.
Whether that is specific to this view / PR or is a broader symptom across various places I'm not sure.

Yeah - I'll rebase the branch and I believe I'll get @MadalinaRaicu's improved layout for free


⚠ PostgreSQL has degraded performance

In 124b13c, PostgreSQL (https://news.ycombinator.com) experienced degraded performance:

HTTP code: 200
Response time: 623 ms

Resolved: PostgreSQL performance has improved in c8473aa.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-24-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1859

Questions ?
Please refer to the Backport tool documentation


Can't connect to docker container running inside WSL2

Your Windows build number:  Microsoft Windows [Version 10.0.19041.153]

I'm running a couchbase server inside WSL2 with multiple port-forwarding :
(from Windows) ❯ wsl -l -v
  NAME      STATE           VERSION
* Ubuntu    Running         2

(inside WSL) ❯ docker ps
CONTAINER ID        IMAGE                 COMMAND                  CREATED             STATUS              PORTS
                                                                                NAMES
aaad5f11ad16        couchbase:community   "/entrypoint.sh couc…"   17 minutes ago      Up 4 minutes        <IP_ADDRESS>:8091-8096->8091-8096/tcp, <IP_ADDRESS>:11207->11207/tcp, <IP_ADDRESS>:11210-11211->11210-11211/tcp, 18091-18096/tcp   couchbase

Everything seems to work, I can connect to localhost:8091 from the Windows Host, I can setup the Couchbase and if I try to connect to the different ports exposed by the container, things look nice and clear:
(from Windows) ❯ Invoke-RestMethod -Uri http://localhost:8092

couchdb version couchbase
------- ------- ---------
Welcome         6.5.0-4966-community

(from Windows) ❯ Invoke-RestMethod -Uri http://localhost:8093
Invoke-RestMethod: 404 page not found

(from Windows) ❯ Invoke-RestMethod -Uri http://localhost:8094
Invoke-RestMethod: {"error":"Page not found","status":"fail"}

The problem is that if I try to connect to the same ports using a Java server that I'm building, I get a Timeout, 100% of times.
It worked until yesterday if I put the IP address of the WSL VM instead of localhost in the configuration file of my server (not the best solution, but at least it worked), then I updated the kernel of the WSL - because I was forced to - and it stopped working.
Even after restarting the lxssservice the problem persists.
The only thing that works is if I start the docker container with --net=host but then I can't start multiple instances of the same container because of the ports binding..
I don't know if it's related but when I run a netstat on the WSL machine it seems like only tcp6 bindings have been created but it seems to work if I force a tcp4 connection:
(from WSL) ❯ sudo netstat -tlnp
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp6       0      0 :::11207                :::*                    LISTEN      1510/docker-proxy
tcp6       0      0 :::11210                :::*                    LISTEN      1496/docker-proxy
tcp6       0      0 :::11211                :::*                    LISTEN      1481/docker-proxy
tcp6       0      0 :::8091                 :::*                    LISTEN      1593/docker-proxy
tcp6       0      0 :::8092                 :::*                    LISTEN      1579/docker-proxy
tcp6       0      0 :::8093                 :::*                    LISTEN      1566/docker-proxy
tcp6       0      0 :::8094                 :::*                    LISTEN      1552/docker-proxy
tcp6       0      0 :::8095                 :::*                    LISTEN      1538/docker-proxy
tcp6       0      0 :::8096                 :::*                    LISTEN      1524/docker-proxy
(from WSL) ❯ telnet -4 localhost 8091
Trying <IP_ADDRESS>...
Connected to localhost.
Escape character is '^]'.
^]
telnet>

If someone has a better solution than the --net=host I'm all ears..
Thank you everyone in advance, small bugs aside, WSL (and WSL 2 in particular) is what made me get back to Windows as my daily driver.

I had the same problem, and it turned out that WSL2 only binds to ipv6 localhost ([::1]) and not ipv4 localhost (<IP_ADDRESS>). It took me a while to figure out, because some applications default to ipv6 resolve first, and other apps (like node.js) defaults to ipv4 lookup. Hence, the host localhost works in your browser, but not in, lets say, node.js.
I resorted to using the ipv6 localhost url, http://[::1]:8091 and it just worked.
When you test in WSL2 and connect to localhost with -4 flag, it works because from the WSL side of things it is acessible both via ipv4 and ipv6. From my experience, even if it lists only ipv6 listings in netstat, it does still listen on ipv4, but maybe this is actually the problem, the Windows bridge only forward traffic for the netstat listing and errorneously only binds ipv6 from the host into the WSL subsystem.
I hope this helps.

As doing a search on a similar issue brought me here, the workaround is to use the WSL2 subsystem ip adress instead, i.e. the adress returned by doing a ifconfig inside wsl.

$ ifconfig
eth0: flags=4163  mtu 1500
[...]
        inet <IP_ADDRESS> netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
[...]

Still quite inconvenient as I think this address changes at every restart of WSL.

@koying The subsystem IP did not work for me, but using the IPv6 loopback address like @jishi recommended works perfectly!

Oh sure. My workaround is only for non-ipv6 enabled apps.

Ah, that makes sense

It worked for me @jishi thanks!

I have tried ip address of both "ip from ifconfig inside wsl" and "http://[::1]:"  and neither worked for "Ubuntu-20.04" Window 10 Wsl2

If you're using MySQL Workbench, make sure to leave out the brackets and use ::1 as the host

I resorted to using the ipv6 localhost url, http://[::1]:8091 and it just worked.

In my case, MySQL Workbench is the only SQL Viwer app that works with localhost. Beekeeper Studio was not working, even with WSL IP 172.X.X.X.
Using IPV6 localhost ::1 works like a charm!

As doing a search on a similar issue brought me here
The workaround [::1] works for ipv6-enabled app, but if you need ipv4, an option is to use the WSL2 subsystem ip adress instead, i.e. the adress returned by doing a ifconfig inside wsl.

$ ifconfig
eth0: flags=4163  mtu 1500
[...]
        inet **<IP_ADDRESS>** netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
[...]

Still quite inconvenient as I think this address changes at every restart of WSL.

thank you this was helpful.....

Has anybody make this work on Ubuntu-20.04?

Has anybody make this work on Ubuntu-20.04?

Same issue as well for me on Ubuntu-20.04 can't connect to docker with any of the suggestions above

The IPv6 workaround (::1) works fine for me on Ubuntu 20.04 LTS.

@massix
run the command on ubuntu:
ip addr | grep -E "\binet.global" | sed -E "s/inet(.)/./\1/"
result example:
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
get the first ->  <IP_ADDRESS>
run the command in the shell windows as an administrator (the port, for example, is 3333):
netsh interface portproxy add v4tov4 listenport=3333 listenaddress=<IP_ADDRESS> connectport=3333 connectaddress=<IP_ADDRESS>
listenport   is the windows port
listenaddress  is the windows address, usually <IP_ADDRESS>
connectport is the ubuntu port
connectaddress is the int ip ubuntu

Why cannot I run curl <IP_ADDRESS>:2375 in wsl2 ubuntu with docker desktop for windows?
I got <IP_ADDRESS> IP in this file /etc/resolv.conf on wsl2/ubuntu.
I tried to create a interface portproxy like this netsh interface portproxy add v4tov4 listenport=2375 listenaddress=<IP_ADDRESS> connectport=2375 connectaddress=<IP_ADDRESS>, but I got error curl: (52) Empty reply from server.
if I execute the command docker -H <IP_ADDRESS>:2375 ps, displays this error during connect: Get http://<IP_ADDRESS>:2375/v1.24/containers/json: EOF
My problem began when I download code and executed on it the  fabric8io/docker-maven-plugin. From here, I looked at all forums and google search results, but without success.
Is there any idea how could I proceed with this problem?
best regards,
efcjunior

More than one year after and the problem still exist.
I used the upload diagnostic tools: 53A83A43-5D21-4978-846F-3FD11662A256/20211206133132

Me too, this is a serious problem

I stopped hoping for a fix. As I fear MS will only offer this "premium" feature only over its "freemium" docker desktop :(.

Same problem.
Windows 11, WSL2 Ubuntu 20.04, Docker for windows WSL2 backend (https://docs.docker.com/desktop/windows/wsl/)
I can curl localhost:<port> inside the container. But I cannot access it in either WSL2 using curl or browser of the windows host.
I tried the following things:

Add the port to Inbound rules on Windows firewall
Use the address of WSL2 instead of localhost or <IP_ADDRESS>
Also tried the ipv6 address

Still having the problem. I was able to make it work on my Ubuntu work laptop, but it's not working on my personal computer. Seriously considering switching to Linux. Works on Linux.

WSL 2
Windows 10
Docker app with React & NestJS. The content from React is served, but the port <IP_ADDRESS>. is not working

This is still happening.

Explicitly bind the port of the server in the docker container to the port of the WSL2.
So in docker run file add
-p 80xx:80xx  =>  (port in WLS2 binds to port in docker container).
Now your docker is visible in Windows.
This seems to work for me. For some reason the professional edition works effortlessly with
--network=host
in docker run

Crimini:
The desire here is to directly access the docker internal network, as can be done on linux and MacOS.  Binding the port with -p connects the host port to the container's port, but we wish to directly access the container's port. For example, some applications do not have predictable port numbers, but we do not wish to use --host=network because that might conflict with host ports.

containers should have availability to be accecible from outside w/o any port redirections.
As it done in linux , where user defined or host docker bridge networks are exposed to host via
docker network ls
NETWORK ID     NAME              DRIVER    SCOPE
bf01bc4d9244   docker_user_net   bridge    local
=>
ifconfig
br-bf01bc4d9244: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
ether 02:42:33:a6:20:c4  txqueuelen 0  (Ethernet)
RX packets 0  bytes 0 (0.0 B)
RX errors 0  dropped 0  overruns 0  frame 0
TX packets 0  bytes 0 (0.0 B)
TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

Still happening, we are having issues using localhost to communicate to our docker container on port 9001.
WSL2 Ubuntu Version 20.04.
We can get the container up and running via WSL2, but curling that container is currently impossible.

In my case I was trying to run a docker container on Ubuntu 20.04 with a web interface I wanted to access from a browser on my desktop. What worked was using the -p flag to publish the container's port(s) to the host:
docker run -p 8000:8000 -it whatever/image

In my case I was trying to run a docker container on Ubuntu 20.04 with a web interface I wanted to access from a browser on my desktop. What worked was using the -p flag to publish the container's port(s) to the host:
docker run -p 8000:8000 -it whatever/image

Yes, but that is not the issue under discussion. Rather, when one has a set of containers all interacting on the Docker private network, it is often useful to look in on that assembly of containers and connect to the ports they use.  Explicitly adding the port is only possible by stopping the entire container system and restarting, and knowing which container publishes that specific port and going in and adding that command.  Moreover, the port may conflict and changing the client port number may not always be easy.
Most importantly, the system operates differently under WSL than in other operating systems, limiting WSL's utility.

This is happening to me, still finding ways

In my case, I was trying to connect to Postgres running in docker.
Initially, I was using this command to run Postgres from WSL (Ubuntu) 👇
docker run -e POSTGRES_PASSWORD=password postgres

after running the above docker command, the docker container started but was not accessible in WSL i-e <IP_ADDRESS>:5432 in WSL was not accessible for connection.
What solved my issue 👇
I had to specify the port binding while running the docker container. Here is the command 👇
 docker run -p 5432:5432 -e POSTGRES_PASSWORD=password postgres

after this, my server running inside wsl was able to connect to postgres

Really annoying that we cannot just use localhost. I have tried all kinds of suggestions, but nothing really worked for me. Finally I figured out that docker run arg --net=host in combination with [::1] is working from my spring application. Before I thought that those would be two separate solution suggestions. So you have to use both for the Docker container which should be able to connect to other Docker containers (e.g., call http://[::1]:8080/container2 from container1). --net=host might not be perfect as already described in this thread, but at least it works.
Not related, but in case you want to connect to your windows host machine from the docker container, you have to use the address host.docker.internal, e.g., host.docker.internal:5000 to access a service with port 5000 running directly on windows.

Not sure if there will negative side effect. But it works if we run container with -p <IP_ADDRESS>:80:80.
docker run -d -p <IP_ADDRESS>:80:80 IMAGE_NAME

I'm able to access the container from Windows by running this command docker run -p <IP_ADDRESS>:80:80.

The whole point of Docker is to shield all this complexity from the container users, so having them do complex flags on every run does not seem like a viable work around. How freakin' hard can it be to also bind WSL2 to IPv4. MS made a lot of noise when they released WSL2 about how much better it was than WSL1 w/hyper-v, which is likely true except for the one thing most people will actually be trying to do with it not working.  Sad they have let this sit here for over a year with not even so much as a response.

@genyded My response was to switch all my computers to Linux. No regrets.

I wasted 2 days trying with Docker on Windows 11 Pro with WSL. WSLv2 is a complete mistake. It's been good for years and it's such a zonk. On MacOS, it flashes and smokes from the first launch. Save your time...

We're using Laravel and this is a MAJOR issue because any work-around screws up all the URL's for links and routing. Is anyone form Microsoft even looking into this? How hard can it be to bind to IPv4 and then IPv6? They hack together WSL so we can even begin to use Docker on THIER OS and then leave us stranded unable to develop and test apps using it. I am trying to convince our CEO to move to mac for dev solely because of this issue, but he is hung up on the $$ difference. Guess it's true though, you do get what you pay for. 3 YEARS LATER AND NOT A PEEP FROM MICROSOFT ON THIS ISSUE.

this problem still exist on Ubuntu 20.04

Hi Yueyi, I also encountered this issue. But I have fixed it through updating the latest version. Maybe you can go to  https://github.com/microsoft/WSL/releases and download and install the latest msixhundle file under the Asset attachment.

Hi Yueyi, I also encountered this issue. But I have fixed it through updating the latest version. Maybe you can go to https://github.com/microsoft/WSL/releases and download and install the latest msixhundle file under the Asset attachment.

oh,thank you, I will have a try.

Cant believe this is still open. What a waste the windows hardware is proving to be for devs

Just bumped into this problem.
Impossible to start my Docker container on Windows:
C:\Users\gusta> docker start 37ad37d59367b1978d7c8e823be19dbf2a2b48f9d798868bc09d59971085b1ec
Error response from daemon: Ports are not available: exposing port TCP <IP_ADDRESS>:50922 -> <IP_ADDRESS>:0: listen tcp <IP_ADDRESS>:50922: bind: An attempt was made to access a socket in a way forbidden by its access permissions.
Error: failed to start containers: 37ad37d59367b1978d7c8e823be19dbf2a2b48f9d798868bc09d59971085b1ec

Any updates on this? I've tried the pre-release (1.3.14) to no avail.   This complicates using WSL for development with Docker Desktop.

Just bumped into this problem.
Impossible to start my Docker container on Windows:
C:\Users\gusta> docker start 37ad37d59367b1978d7c8e823be19dbf2a2b48f9d798868bc09d59971085b1ec
Error response from daemon: Ports are not available: exposing port TCP <IP_ADDRESS>:50922 -> <IP_ADDRESS>:0: listen tcp <IP_ADDRESS>:50922: bind: An attempt was made to access a socket in a way forbidden by its access permissions.
Error: failed to start containers: 37ad37d59367b1978d7c8e823be19dbf2a2b48f9d798868bc09d59971085b1ec

I had the same problem.
Solution for me was to check and disable some "unplugged" virtual Network Adapters.
Details.

I had Windscribe VPN on the host. When installed, it creates several virtual adapters (Windscribe VPN, Windscribe Windtun420). I've disabled both, and problem was sovled.
On the other hand, I have several OpenVPN adapters. They don't affect this problem.

@nervousapps Thank you so much, I knew there had to be a firewall issue, they don't mention the two rules you added.
I had your first command as per their docs, but it suddenly worked once I added the rules.
Perhaps this needs updating: https://learn.microsoft.com/en-us/windows/wsl/networking#accessing-a-wsl-2-distribution-from-your-local-area-network-lan

After a whole day of pulling hair out, solved my issue (in my case it was a Vite project) by switching from this:
"scripts": {
    "dev": "vite",
    ...
},

To this:
"scripts": {
    "dev": "vite --host <IP_ADDRESS>",
    ...
},

Now I can access the project in the browser using http://localhost:5173 as normal.
Docker command:
docker run -d -p 5173:5173 --name vue-app-container vue-app

After a whole day of pulling hair out, solved my issue (in my case it was a Vite project) by switching from this:
"scripts": {
    "dev": "vite",
    ...
},

To this:
"scripts": {
    "dev": "vite --host <IP_ADDRESS>",
    ...
},

Now I can access the project in the browser using http://localhost:5173 as normal.
Docker command:
docker run -d -p 5173:5173 --name vue-app-container vue-app

This worked like a charm. Change the app host to bound to <IP_ADDRESS> instead of the default localhost or <IP_ADDRESS>.

bind your docker app host to <IP_ADDRESS>, other than <IP_ADDRESS> works to me.
wsl: ubuntu
windows: 11
docker image tested: mlflow  docker run -it --rm -p 5000:5000 ghcr.io/mlflow/mlflow mlflow ui -h <IP_ADDRESS>
result: from windows browser, I can access locahost:5000

this worked for me. why is microsoft so useless in trying to fix such a basic issue?
its been 4 years guys. common now.

Saw the same issue with Windows 10 with wsl Ubuntu-20.04. This doc solved for me. it boils down to the first command of @nervousapps answer

Much of the networking problems I've encountered with WSL2 and Docker for Windows over the years went away when I switched to Mirrored mode networking.
[wsl2]
networkingMode=mirrored
dnsTunneling=true
autoProxy=true

Then, you can, for instance, access your Windows host from Linux using <IP_ADDRESS>. Additionally, dnsTunneling and autoProxy can also help if you're behind a corporate proxy.

This bug is really annoying! 😠 Common, it's only Windows the only who do this. It should imitate Linux but it doesn't in this way.
Anyway, for the folks who are developing on python i suggest the following:

There is library that you can use to detect when you are in docker called detect: https://pypi.org/project/detect/
In the standard python library there is a library called platform
If you want to detect when using docker in WSL2 you can the following:

import platform
import detect

if detect.docker and "WSL2" in platform.release():
   # Bind to <IP_ADDRESS> instead of <IP_ADDRESS>

Please @microsoft solve this bug. A lot of people is waiting for you and we don't like to be fooled.

I am using windows docker desktop v4.33.1. There's a feature in settings >  features in development > beta features >enable host networking. It works for me with the feature enabled.

@massix run the command on ubuntu: ip addr | grep -E "\b_inet.global" | sed -E "s/inet(.)/._/\1/"
result example: <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>
get the first -> <IP_ADDRESS>
run the command in the shell windows as an administrator (the port, for example, is 3333): netsh interface portproxy add v4tov4 listenport=3333 listenaddress=<IP_ADDRESS> connectport=3333 connectaddress=<IP_ADDRESS>
listenport is the windows port listenaddress is the windows address, usually <IP_ADDRESS> connectport is the ubuntu port connectaddress is the int ip ubuntu

I know this is super old lol, but this fixed my problem!
My problem is I have been trying to run a Plex server in a docker container, inside WSL Ubuntu. I wasn't able to enable remote access because my router obviously can only route traffic to my Windows host, but then Windows couldn't route the traffic to the docker container. (at least that's how I see it in my head, I don't know if that's accurate :sweat_smile:)
This seems like port forwarding, but between Windows and the docker container. Which makes sense. I figured it would be something like that but didn't know how to actually do it.


using gitpath to access repository root

Stack from ghstack:

-> #869
#867
#866

Codecov Report

Merging #869 (815753f) into gh/anuragprat1k/11/base (e1777df) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##           gh/anuragprat1k/11/base     #869      +/-   ##
===========================================================
- Coverage                    33.02%   32.95%   -0.07%
===========================================================
  Files                          341      341
  Lines                        33397    33389       -8
===========================================================
- Hits                         11028    11004      -24
- Misses                       22369    22385      +16

Impacted Files
Coverage Δ

agents/droidlet_agent.py
44.58% <0.00%> (-7.82%)
:arrow_down:

interpreter/craftassist/tasks.py
74.45% <0.00%> (-0.55%)
:arrow_down:

perception/robot/tests/test_perception.py
98.08% <0.00%> (+0.02%)
:arrow_up:

interpreter/robot/tests/world.py
89.05% <0.00%> (+0.72%)
:arrow_up:

REPO_ROOT = os.path.abspath(os.path.dirname(file))

This is much neater. Done, thanks.


Bug: Pre-commit hook fails with PackageNotFoundError

Before You Begin
Before proceeding, please make sure to follow these steps:

[x] I have checked for similar issues in the project's issue tracker.
[x] I have searched closed issues to see if a similar problem was reported
before.

Issue Details
Running the pre-commit-hook fails with a PackageNotFoundError even though running licensecheck from the command line works fine.
Description
I created a new project with poetry resulting the the following pyproject.toml:
[tool.poetry]
name = "test"
version = "0.1.0"
description = ""
authors = [""]
readme = "README.md"
license = "Apache 2.0"

[tool.poetry.dependencies]
python = "^3.10"

[tool.poetry.group.docs.dependencies]
licensecheck = "^2024.1.4"

[tool.poetry.group.dev.dependencies]
ipykernel = "^6.29.4"
pre-commit = "^3.7.0"

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

[tool.licensecheck]
using = "poetry:dev;docs"
format = "markdown"
file = "docs/licenses.md"

Expected Behavior
The hook should run and generate the report file.
Actual Behavior
(test-py3.10) excidion@thinkbook-14-gen-6:~/dev/lic$ pre-commit run licensecheck
licensecheck.............................................................Failed
- hook id: licensecheck
- exit code: 1

Traceback (most recent call last):
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 35, in getPackageInfoLocal
    pkgMetadata = metadata.metadata(requirement)
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 986, in metadata
    return Distribution.from_name(distribution_name).metadata
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 548, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: No package metadata was found for IPYKERNEL

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 134, in getPackages
    packageinfo.add(getPackageInfoLocal(requirement))
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 59, in getPackageInfoLocal
    raise ModuleNotFoundError from error
ModuleNotFoundError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/bin/licensecheck", line 8, in <module>
    sys.exit(cli())
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/cli.py", line 86, in cli
    sysexit(main(args))
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/cli.py", line 120, in main
    myLice, depsWithLicenses = get_deps.getDepsWithLicenses(
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/get_deps.py", line 232, in getDepsWithLicenses
    packages = packageinfo.getPackages(reqs)
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 137, in getPackages
    packageinfo.add(getPackageInfoPypi(requirement))
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 84, in getPackageInfoPypi
    homePage=_pkgMetadataGet(info, "home_page"),
  File "/home/excidion/.cache/pre-commit/repop8ow40km/py_env-python3.10/lib/python3.10/site-packages/licensecheck/packageinfo.py", line 21, in _pkgMetadataGet
    value = JOINS.join(value)
TypeError: can only join an iterable

The weirdest thing is, every time i run it the first error message No package metadata was found for IPYKERNEL references a different package name.
System Information
Please provide the following additional information about your system or
environment:

Operating System (OS): Zorin OS
OS Version: 16.3
Browser (if applicable): -
Browser Version (if applicable): -
Project Version (if applicable): 2024.1.4

Hey thanks for this! The issue is a super misleading bug but just released a fix for this now. Basically calling "".joins() on a non iterable type (like an int, or None) is what leads to this. It's likely this is caused by some missing metadata which is why I didn't pick it up with the existing tests. Added tests and fixed in https://github.com/FHPythonUtils/LicenseCheck/releases/tag/2024.1.5

That fixed it, thank you for the quick response! :)

Perfect! Really pleased that's sorted. And yeah no worries at all! Must admit it was very much a face palm moment that bug!


portal experience of http redirect setting is different from doc

It is mentioned in below doc that, when configuring http to https redirect, user needs to select the endpoint name from dropdown
https://learn.microsoft.com/en-us/azure/cdn/cdn-storage-custom-domain-https#http-to-https-redirection

In the above rule, Cdn-endpoint-name refers to the name that you configured for your CDN endpoint, which you can select from the drop-down list.

However, in current Rule Engine portal, instead of a drop down, a textbox is used as endpoint input. The description is quite confusing and I'm not able to configure "source", as per doc

Please help update this doc to align with current portal behavior

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

ID: 518f4eaa-297f-5981-97c5-274a4103fa5d
Version Independent ID: 0dbdc5d6-d448-544d-a862-3f7f5204be22
Content: Tutorial: Access storage blobs using an Azure CDN custom domain over HTTPS
Content Source: articles/cdn/cdn-storage-custom-domain-https.md
Service: azure-cdn
GitHub Login: @duongau
Microsoft Alias: duau

@bangdu-ms
Thanks for your feedback! We will investigate and update as appropriate.

@duongau
Can you please check and add your comments on this doc update request as applicable.

@bangdu-ms
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Hi @bangdu-ms, thank you for bringing this to our attention. I've removed the mention of the drop down list. Thank you. #please-close


Missing topic?

Came into this repository from: https://github.com/topics/mud-server
Oddly I can’t see a topic listed there, I wanted to update the topic with relevant links.

Github ogrencem

That is because "mud-server" is not curated yet. GitHub shows custom topics when it is not curated.

What @R4356th said!


[material-ui][TextField] slotProps.htmlInput.component does not replace input html element

Steps to reproduce
The following works and replaces the wrapper div of the input element:
<TextField slotProps={{ input: { component: () =>
Hello world
 } }} />

The following does not work:
<TextField slotProps={{ htmlInput: { component: () =>
Hello world
 } }} />

Current behavior
Does not replace the  element in the rendered component.
Expected behavior
Replaces the <input element in the rendered component.
Context
I would like to put a link instead of the  html element in the component, so I can keep other e.g.: shrinked label, so the link will fit with the other fields in the form.

  Output from `npx @mui/envinfo` goes here.

Search keywords: htmlinput

@balazs-zsoldos use slots.htmlInput to replace wrapper around htmlInput
Paste below code here for working demo
import * as React from 'react';
import Box from '@mui/material/Box';
import TextField from '@mui/material/TextField';

export default function BasicTextFields() {
  return (
    <Box
      component="form"
      sx={{ '& > :not(style)': { m: 1, width: '25ch' } }}
      noValidate
      autoComplete="off"
    >
      <TextField slots={{htmlInput:()=>
hello
}} id="outlined-basic" label="Outlined" variant="outlined" />

    </Box>
  );
}

component prop is not supported in htmlInput. I believe the reason TypeScript doesn't throw a type error is that, currently, InputBaseComponentProps accepts any string.
current: https://github.com/mui/material-ui/blob/e720f3b2905b9b3679238ffb5853770c65e86816/packages/mui-material/src/InputBase/InputBase.d.ts#L245-L250
Updating the InputBaseComponentProps type as shown below will cause TypeScript to throw an error when the component prop is passed to htmlInput. cc @DiegoAndai @ZeeshanTamboli  wdyt, should i make this change?
Post change:
export interface InputBaseComponentProps
extends React.HTMLAttributes<HTMLInputElement | HTMLTextAreaElement> {
// Accommodate arbitrary additional props coming from the inputProps prop
[arbitrary: string]: any;
component?: never;
}

Hey, @balazs-zsoldos, thanks for the report. As @sai6855 explained, this is expected as the HTML input element does not accept a component prop/attribute. Did the provided example work for you?
@sai6855 I wouldn't change this type at this moment.

@DiegoAndai Thanks for the clarification. All clear now.


RuntimeError: Argument #8: Padding size should be less than the corresponding input dimension, but got: padding (5, 5) at dimension 2 of input 5

Hey friends!
Ran into a really frustrating issue that I can't figure out for the life of me. I've uninstalled and reinstalled dependencies, tried multiple videos and multiple models. I'm running the code on an M2 MAC which was working flawlessly until I closed and reopened VSCode. I'm very frustrated with myself for not being able to figure it out. Please help. Thank you in advance.
File ~/Desktop/AI/Practical-RIFE/interpolator_helper.py:244, in interpolate_inference_video(args)
242 I0_small = F.interpolate(I0, (32, 32), mode='bilinear', align_corners=False)
243 I1_small = F.interpolate(I1, (32, 32), mode='bilinear', align_corners=False)
--> 244 ssim = ssim_matlab(I0_small[:, :3], I1_small[:, :3])
246 break_flag = False
247 if ssim > 0.996:
File ~/Desktop/AI/Practical-RIFE/model/pytorch_msssim/init.py:107, in ssim_matlab(img1, img2, window_size, window, size_average, full, val_range)
104 img1 = img1.unsqueeze(1)
105 img2 = img2.unsqueeze(1)
--> 107 mu1 = F.conv3d(F.pad(img1, (5, 5, 5, 5, 5, 5), mode='replicate'), window, padding=padd, groups=1)
108 mu2 = F.conv3d(F.pad(img2, (5, 5, 5, 5, 5, 5), mode='replicate'), window, padding=padd, groups=1)
110 mu1_sq = mu1.pow(2)
RuntimeError: Argument #8: Padding size should be less than the corresponding input dimension, but got: padding (5, 5) at dimension 2 of input 5

You can firstly change
ssim = ssim_matlab(I0_small[:, :3], I1_small[:, :3])
=> ssim = 0.995
to work around.
I guess there is something wrong with the pytorch version.

Yes this worked! Thank you very much friend!  Since I have your attention, can you point me to a script or something to run inference with 2.3 model? I have the model weights, just not really sure how to inference because of the dependencies for inference_video.py.

You may look up the history: https://github.com/hzwer/Practical-RIFE/blob/327f6ee4766a6bf0da7d115e50e1ce2c1c5a49a0/inference_video.py


AssertionError - type inference issue on recursive function

Here is a link to a gist I have posted that attempts to port the KD-tree implementation from scipy to numba which removes the object-oriented structure in the implementation.  Since full reproducibility requires the code for building the tree as well, I decided to post as a gist rather than embed the code in the issue. Please let me know if I should do the latter as well.
As a result, the 'traverse' function is a rather complicated recursive function which the type inference engine in numba appears to be having difficulty with. If I remove the 'numba.jit' decorators (or even remove the 'nopython' parameter), the function executes as expected.
Below is the error I am receiving:
assert self.type is not None (AssertionError)
At this point, I am unaware if how I have written the function adheres to the current functionality available within numba or if this is a bug within numba. Posting an issue here in hopes to receive some direction on which it may be.
Thanks,
David

@dpatschke, I have tracked down the issue.  It's not a recursion problem.  It is because np.array([]).astype(np.int32) has a untyped list usage.  Change that line to np.empty(0, dtype=np.int32) and it will work.
The minimal reproducer:

@sklam Thanks so much for looking into this! It works for me now.
I would never have thought there was a difference between the two.
Thanks again!


Adjust warning host count calculation

Include 'pending-for-input' and 'insufficient-unbound' agent states
when calculating warning hosts count.
Signed-off-by: Jiri Tomasek<EMAIL_ADDRESS>

/retest


Conduit and conduwuit has a rare chance of sending duplicate events over /sync

This is not really confirmed because there hasn't been a clear reproduction method, no one's provided any /sync responses showcasing the issue or where the events are being erroneously sent twice, and the reports of this issue are from only a couple users who generally have issues with their homeserver that aren't affecting anyone else.
But it's supposedly a problem in some things like maubots.

For the record, this appears to be an upstream issue too.
https://gitlab.com/famedly/conduit/-/issues/180

It's been confirmed so not really sure, someone needs to work on solving it.

I wrote a gist with my findings and a script to repro it, in case it helps anyone with debugging.

I can confirm that @jevolk's patch (https://github.com/jevolk/conduwuit/commit/b81211c0b6fc7c332052cae35e3dd61ab390b54f) resolves this issue.

Fixed/merged in main branch


blocks are converted incorrectly

Example:
Markdown:
Hello <World>
HTML conversion:

Hello &lt;World&gt;

& gets converted to & <-- Not Expected
If we remove the code block, same gets converted to this:

Hello <World>

As per documentation there is no option that would disable this conversion happening only for code blocks. Please fix this.

That is the expected behavior.
<, > and & need to be escaped in HTML,  tags are not an exception.

You always represent an ampersand in HTML content as & or &, never a bare &. This includes both text between an element's tags, as well as attribute values. No exceptions.
This is because & itself denotes the start of an HTML entity reference (as seen above) and can often cause problems when appearing in the middle of text that could be interpreted as an entity reference.

Hello Tivie,
I want to display HTML text within code block. Example Markdown:

This is some HTML<\div>

Now as per functionality I think it is fine. I can simply write the HTML text as above (without escaping <,>, and &, and it will render correctly. However, if you open the same markdown on Github, it interprets it as proper HTML and it will display the text as follows:
This is some HTML
No wrapping div displayed.
I will have to escape it separately on github to make it work. For example, same code above need to be written as follows to make it work:

\
This is some HTML\

Now showdown is interpreting it incorrectly.
Please suggest how to achieve it as there is no way to do it in showdown. Before closing the ticket kindly understand the problem. Just that it has been implemented that way doesn't mean it is correct.

Ah, I see. Well, you seem to have found a related bug in the code.

input:

This is some HTML<\div>

expected output;

This is some HTML<\div>

actual output:

This is some HTML<\div>

Actually, there is no bug.
If you want a span element to be parsed as is, you can wrap it in a block (a div for instance)

This is some HTML

This seem to be a workaround for the problem. Anyways, thanks.

Well, it's not a problem "per se".
There are two kinds of HTML elements in showdown, semantically speaking):

span (inline html elements)
block

Block elements are not parsed unless you add markdown=1 attribute to it.
Span elements are always parsed, unless they are inside a block
This happens for a number of reasons:

historical: the original parser, written by John Gruber, worked like that so showdown replicated it.

pratical: almost all parsers behave like that, so people expect this kind of behavior

tehcnical: since span elements can appear interpolated in text, it would be hard to parse them

Rather than creating a new one, I might piggy-back off this issue because it's sort of the same issue.
Why is it un-reasonable to expect...

 This code snippet

```
```
To be rendered as  elements with HTML inside it?
I need a library that can handle this as all of my markdown content is related to code, so there's a lot of css, javascript and html floating around.
If a Markdown parsing library can't handle this, then I legitimately don't know what I can do.

Because almost all use cases expect code inside
 tags to be a respresentation of said code.
For instance, if the code is not escaped...
this:

foobar

is interpreted as this:

foobar


feat(_validate_start_time): Adding a if statment to the to start_time…

… to avoid validating when there is a dault value, since there is no needance to to with the value is 0
Hi, open this PR to add new validator in the start_time_valdiator,
Did this becouse of a bug with mutagen at ->
def media_duration(file_path: str):
file = mutagen.File(file_path)
duration = file.info.length
return duration
the script  is crashing due to duration returning none,
i noticed that we can avoid this bug just to avoid validitng when the start time is 0

Hi, is that possible to release a new version of the package?
so in this way we can download new version of the package with modifications,
we wanna because in this, will only need to update the version of the package inside poetry instead of put the package inside our projects

yeah of course i will be updating the pypi package too

Hi, is that possible to release a new version of the package?
so in this way we can download new version of the package with modifications,
we wanna because in this, will only need to update the version of the package inside poetry instead of put the package inside our projects

it has been updated: https://pypi.org/project/audio-extract/0.7.0/


My input obj file is different from demo obj file

Hi, thanks for your great work. I put my obj file on stage 1 like this:
python pipeline_paint3d_stage1.py  --sd_config controlnet/config/depth_based_inpaint_template.yaml  --render_config paint3d/config/train_config_paint3d.py  --mesh_path /public/omniteam/yejr/Paint3D-main/input_data/penguin/face_blue.obj --outdir outputs/penguin/stage1
And I get the following error:
Traceback (most recent call last):
File "/public/omniteam/yejr/Paint3D-main/paint3d/models/mesh.py", line 30, in init
mesh = kal.io.obj.import_mesh(mesh_path, with_normals=True, with_materials=True)
File "/public/omniteam/yejr/conda_env/paint3d/lib/python3.9/site-packages/kaolin/io/obj.py", line 187, in import_mesh
face_normals_idx.append([int(d[2]) for d in data])
File "/public/omniteam/yejr/conda_env/paint3d/lib/python3.9/site-packages/kaolin/io/obj.py", line 187, in
face_normals_idx.append([int(d[2]) for d in data])
ValueError: invalid literal for int() with base 10: ''
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/public/omniteam/yejr/Paint3D-main/pipeline_paint3d_stage1.py", line 287, in
main()
File "/public/omniteam/yejr/Paint3D-main/pipeline_paint3d_stage1.py", line 193, in main
mesh_model = TexturedMeshModel(cfg=render_cfg, device=device,).to(device)
File "/public/omniteam/yejr/Paint3D-main/paint3d/models/textured_mesh.py", line 31, in init
self.mesh = Mesh(self.cfg.guide.shape_path, self.device, target_scale=self.cfg.guide.shape_scale,
File "/public/omniteam/yejr/Paint3D-main/paint3d/models/mesh.py", line 32, in init
mesh = kal.io.obj.import_mesh(mesh_path, with_normals=True, with_materials=False)
File "/public/omniteam/yejr/conda_env/paint3d/lib/python3.9/site-packages/kaolin/io/obj.py", line 187, in import_mesh
face_normals_idx.append([int(d[2]) for d in data])
File "/public/omniteam/yejr/conda_env/paint3d/lib/python3.9/site-packages/kaolin/io/obj.py", line 187, in
face_normals_idx.append([int(d[2]) for d in data])
ValueError: invalid literal for int() with base 10: ''
I try to copy the material.mtl file from demo and still can't run, how to deal with it?

Seems like my obj file lacks of 'normal'?


Restrict read-only sessions

Restrict read-only sessions to one per (project + tool version).
Fixes #421.

I've planned the review for today or tomorrow.


LumisPerJob doesnt belong to this template

keeping it open in case there are further minor updates.

Merging these templates. @ticoann it won't break, but you may want to clone WMCore again before injecting your tests. Also a minor update to the inject-test script.


ociodisplay inconsistency between tetrahedral interpolation and linear sampling

With tetrahedral interpolation, the gpu processor generates manual interpolation code in Lut3DOpGPU.cpp and specifies the use of GL_NEAREST.
However, the SetTextureParameters function used by ociodisplay still sets the texture to GL_LINEAR in this case. Shouldn't this if-statement also set the texture to GL_NEAREST if the incoming interpolation is tetrahedral as well?

Thanks for raising that @num3ric, I think this does indeed look suspicious. It should probably be the Lut3DOpGPU.cpp code that need fixing to set the LUT interpolation scheme to NEAREST when TETRAHEDRAL is used on the GPU? It sounds like a serious issue?

Still, even if sampling on a grid point, an implementation might do extra work (reads and blending math) in GL_LINEAR mode so it could be a performance hit.


Make the gtihub URL configurable to allow support for enterprise installs

Description
Rather than only use GitHub.com, it would be great if we could replace the base URL host to our internal enterprise GitHub.  All the paths and api calls should remain the same, so I imagine the change is fairly trivial to implement.

So you mean like instead of github.com it would be company.com? Or company.github.com?
Do you have a public enterprise domain we could test against to ensure the feature works as intended? If not, I may need help testing any PR I end up making lol

Any update on support for GitHub Enterprise (configuration of the URL)?

Sorry for the delay, life has been really busy. I can try to take a look at this soon.
I'd rather not have to use my free trial up on this issue if I can help it 😅, so I'll try to ask my questions here.
So for a company like Acme:

Could we expect a URL like https://acme.io?
Would we link to repos with the same path as we do GitHub (e.g. https://acme.io/foo-repo)?
Would we link to tickets with the same path as we do GitHub (e.g. https://acme.io/foo-repo/issue/1, https://acme.io/foo-repo/pull/2)?
Will requests made to https://api.github.com (REST) and https://api.github.com/graphql (GraphQL) still work?
Are images and other kinds of media content still accessible via https://raw.githubusercontent.com?

Sorry for the delay, life has been really busy. I can try to take a look at this soon.
I'd rather not have to use my free trial up on this issue if I can help it 😅, so I'll try to ask my questions here.
So for a company like Acme:

Could we expect a URL like https://acme.io?

Yes, custom URL

Would we link to repos with the same path as we do GitHub (e.g. https://acme.io/foo-repo)?

Yes, after the main domain name, the rest of the URL paths would be the same as github.com

Would we link to tickets with the same path as we do GitHub (e.g. https://acme.io/foo-repo/issue/1, https://acme.io/foo-repo/pull/2)?

Same answer

Will requests made to https://api.github.com (REST) and https://api.github.com/graphql (GraphQL) still work?

No.  Everything is behind the domain name for security.  And likely behind the companys' firewalls.

Are images and other kinds of media content still accessible via https://raw.githubusercontent.com?

No, same answer,.


[Time series Forecasting] Continuous Ranked Probablity Score (CRPS) loss for probablity network ouput type

We could consider implementing CPRS an alternative training loss for the probabilistic network output type.
INFO: https://www.lokad.com/continuous-ranked-probability-score

Hey @dengdifan !
Within a university project, my team and I would like to contribute to this issue. If you have any more detailed specifications or hints, we would be glad to get them from you.
Thanks for your work !

Hi @omeurer,
sorry for the late reply, I am currently working on some other projects these days.
Basically, you could check if you can add a new loss type under this py module: https://github.com/automl/Auto-PyTorch/blob/master/autoPyTorch/pipeline/components/training/losses.py
Then you should attach the new loss to this dict: https://github.com/automl/Auto-PyTorch/blob/master/autoPyTorch/pipeline/components/training/losses.py#L128
Finally, some constraints might be required for this function: https://github.com/automl/Auto-PyTorch/blob/master/autoPyTorch/pipeline/time_series_forecasting.py#L207
if you have any further questions, please let me know.