Look Rotation Viewing Vector is Zero Error on third Prefab when starting VR

Hello,
I got a strange issue, when loading a third Prefab into my scene and starting VR afterwards.
Up to 2 of those Prefabs the VR view starts without issues, but as soon as there is a third Prefab in the Scene the console throws this Error: Look Rotation Viewing Vector is Zero - (thrown from Runtime.Math)
It is strange, because the scene works without issues with much more of those Prefabs but only breaks when staring VR Mode.
Does somebody knows how this issue can be related to the WebXR Plugin?
Testing it with a HTC Vive in Firefox on Windows 10 (Unity 2019.4.13f1).
Thanks.

I guess that it's related to the WebAssembly memory growth. When that happens, typed arrays on the page that refers to the WASM memory would lose the reference.
Like SharedArray here https://github.com/MozillaReality/unity-webxr-export/blob/master/Assets/WebXR/Plugins/WebGL/webxr.jslib#L20
To fix it, you need to store the byteOffset, then you can check if it's byteLength == 0.
Like done here:
https://github.com/De-Panther/unity-webxr-export/blob/master/Packages/webxr/Runtime/Plugins/WebGL/webxr.jslib#L41

Hey thanks for your reply. Yet I do not completely understand where I would have to implement that. If I understand you correctly, those shared arrays are built as soon as something is changed live, during runtime.
But I do not use those SharedArrays in my code.
But still I think the issue is related to that.
Since I was removing everything from my prefab, until the error disappeared.
I think it might be related to the dynamic creation of RenderTextures or VideoPlayer, which render to a RenderTexture.
Still strange that this only happens when starting VR Mode.
Do you have any clue why this happens, or how to solve it?
Thank you.

But I do not use those SharedArrays in my code.

The exporter use them, so if you use the exporter, you use them :)
As I wrote, the wasm memory grow, and then the variable named SharedArray here https://github.com/MozillaReality/unity-webxr-export/blob/39b22eb851a1c4462a3ccb5d7c9f63a137e54b0f/Assets/WebXR/Plugins/WebGL/webxr.jslib#L20
Lose it's reference.
So do the same thing that was done in this fork for Module.XRSharedArray https://github.com/De-Panther/unity-webxr-export/blob/b788ee930f0b05cdaedeaffc9f35f13563e12000/Packages/webxr/Runtime/Plugins/WebGL/webxr.jslib#L41


Accept single filepath

This change allows you to run a single test file. I've added the filePath property to the config options as the documentation seems to suggest it would be the right place:
Options:
  --config, -c
      The path to a jest config file specifying how to find and execute tests.

The purpose of this change is to make the Jest CLI easier to use with a grunt / gulp style work flow:
Component.js is changed -> grunt / gulp watch notices -> runs spec/component-spec.js

Going to close this out since, as pointed out, it should already work.
It's true that the default is to accept a regexp pattern -- but I believe that also covers the case where you pass a specific path (it would just be a very specific "pattern").
Feel free to comment further if you think this is still strictly necessary.


Freewheel SSP adapter doc is missing

Prebid.js PR is here: https://github.com/prebid/Prebid.js/pull/1793

On this page: http://prebid.org/dev-docs/bidders.html
FreewheelSSP is referenced as a video bidder, next to StickyAdsTV, a native one.
The truth is:  freewheel-ssp is an update to stickyadstv. They are actually the same bidder adapter referenced with those two aliases.
=> on the doc page StickyAdsTV can be removed,  (stickyadstv is the former company name)
FreewheelSSP supports both video and native and supports the same parameters as the former bidder adapter.
Thanks.

Confirmed this is done. http://prebid.org/dev-docs/bidders.html#freewheel-ssp


Please update release in npm repository to 0.2.0

The current version in npm is 0.1.0. Update please to 0.2.0. Many thanks in advance!

Done.


bump ui to version 0.4.2

Codecov Report

Merging #339 (a3aa43a) into main (e3f38d0) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a3aa43a differs from pull request most recent head 74bbfd7. Consider uploading reports for the commit 74bbfd7 to get more accurate results

@@            Coverage Diff            @@
##              main      #339   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          234       234
  Lines        12793     12793
=========================================
  Hits         12793     12793

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 e3f38d0...74bbfd7. Read the comment docs.


Implement Reflect for State<S> and NextState<S>

Objective

Make it possible to snapshot/save states
Useful for re-using parts of the state system for rollback safe states
Or to save states with scenes/savegames

Solution

Conditionally add the derive if the bevy_reflect is enabled

Changelog

NextState<S> and State<S> now implement Reflect as long as S does.

LGTM. Should we register these types when App::add_state is called too?

LGTM. Should we register these types when App::add_state is called too?

This would be great, but unfortunately I don't think we can do that without requiring all state types (i.e. S) implement Reflect


memory_limiter cannot be configured in the config yaml

The otelcol binary expects that memory_limiter can be configured only with env vars. If it's configured solely in config yaml, for example:
      memory_limiter:
        ballast_size_mib: "4096"
        check_interval: 1s
        limit_mib: "8192"
        spike_limit_mib: "2560"

the collector is failing to start with the following message
main.go:283: Memory limit (460) is less than 2x ballast (4096). Increase memory limit or decrease ballast size.

If --mem-ballast-size-mib is not provided, the collector will start but misconfigured:
memory_limiter is likely incorrectly configured. ballast_size_mib must be set equal to --mem-ballast-size-mib command line option.	{"kind": "processor", "name": "memory_limiter"}

This is no longer occurring.
Following reproduction steps, I now get:
2023/02/24 15:29:41 settings.go:384: Set ballast to 168 MiB
2023/02/24 15:29:41 settings.go:400: Set memory limit to 460 MiB

Closing this as complete.


Add limit.wait

This adds a new function property to limit, wait().  This will wait until all active or pending calls have completed (rather than just discarding them as clearQueue() does).

How is this different/better than just using Promise.all, per the readme example?

How is this different/better than just using Promise.all, per the readme example?

That requires tracking all of the promises.  The use-case I have is to define an async function of which at most one invocation is active at any time.  That's fairly easy to accomplish with something like this:
let _p = Promise.resolve();

const _f = async () => {..};

const f = () => {
  _p = _p.then(_f);
  return _p;
}

const close = async () => {
  await _p;
}

We use this for async functions that "reconcile" things -- for example, reconciling a list of message queues to listen to with an array of listener objects.  Whenever the list of message queues change, we just call this.reconcile(), and that queues a reconciliation to begin as soon as the current reconciliation is done.  For our use-cases, the fact that multiple calls can be queued doesn't matter.
The code structure above doesn't read very well in a larger code structure, and is easy to mess up, so I'm looking for a more readable solution.  To be honest, limit isn't much more readable, so maybe this isn't the right tool.  I'm open to other ideas for a tool to use..

So on reflection, maybe this isn't the right tool.  I wrote https://github.com/djmitche/p-sync that does precisely what I want.  And I built it with ava and xo and an MIT license :)
Are you interested in adopting that into the promise-fun family?

(renamed to p-synchronize)
I think I prefer that library.  If you'd like to add it to promise-fun, let me know.

Had you seen https://github.com/sindresorhus/p-series

I had - it doesn't do the same thing (since you can't add promises to it)


Need to Remove Singletons from ballerina-http Module

There are some singleton classes in ballerina-http module which need to be removed. This causes problems in the long run since they are also shared when running test cases in ballerina and might cause intermittent test case failures too.

This is separated into 2 issues to ease the implementation tracking

https://github.com/ballerinalang/ballerina/issues/4247
https://github.com/ballerinalang/ballerina/issues/4248

Hence closing the issue.


[Doc enhancement][TOC placement change][Issue 3922]

Please review,

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 6ea5ff4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 3534c0e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

msteams-platform/samples/integrate-web-apps-overview.md
83
24
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

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

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/samples/integrate-web-apps-overview.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/people-picker-capability.md
81
22
link
:white_check_mark:

msteams-platform/samples/integrate-web-apps-overview.md
83
24
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

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

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/people-picker-capability.md
:white_check_mark:Succeeded
View

msteams-platform/samples/integrate-web-apps-overview.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/location-capability.md
82
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/mobile-camera-image-permissions.md
87
21
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/native-device-permissions.md
89
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/people-picker-capability.md
81
22
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
89
16
link
:white_check_mark:

msteams-platform/samples/integrate-web-apps-overview.md
83
24
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 043d0a6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/how-to/enable-SSO-auth-me.md
:warning:Warning
View
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/location-capability.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/mobile-camera-image-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/native-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/people-picker-capability.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
:white_check_mark:Succeeded
View

msteams-platform/samples/integrate-web-apps-overview.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/enable-SSO-auth-me.md

Line 19, Column 20: [Warning-hard-coded-locale] Link 'https://azure.microsoft.com/en-us/free/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/location-capability.md
82
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/mobile-camera-image-permissions.md
87
21
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/native-device-permissions.md
89
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/people-picker-capability.md
81
22
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
89
16
link
:white_check_mark:

msteams-platform/samples/integrate-web-apps-overview.md
83
24
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 053c25b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/concepts/app-fundamentals-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/location-capability.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/mobile-camera-image-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/native-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/people-picker-capability.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/enable-SSO-auth-me.md
:white_check_mark:Succeeded
View

msteams-platform/samples/integrate-web-apps-overview.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
96
5
link
:white_check_mark:

msteams-platform/concepts/app-fundamentals-overview.md
83
10
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
91
5
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/location-capability.md
82
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/mobile-camera-image-permissions.md
87
21
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/native-device-permissions.md
89
17
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/people-picker-capability.md
82
22
link
:white_check_mark:

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
89
16
link
:white_check_mark:

msteams-platform/messaging-extensions/how-to/enable-SSO-auth-me.md
87
6
link
:white_check_mark:

msteams-platform/samples/integrate-web-apps-overview.md
83
25
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


cleanup(sinsp): add output format option to sinsp-example

Signed-off-by: Melissa Kilby<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
@Andreagit97 and @Molter73 follow up PR that exposes option to customize output JSON fields for easier debugging. Should not break existing usages of sinsp-example binary.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: incertum
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval by writing /assign @mstemm in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Very nice PR, thanks @incertum !

LGTM label has been added.  Git tree hash: c54b2e2abd61daf0ad8d42ef22d13783b0203e17

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 708680b3df1775d5652a5521c06c0c8d9ad5090b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,Molter73]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,Molter73]

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


nbdev_new does not work

I first tried to use nbdev_new according to the tutorial which says it will create a new project and init a git repo. Sadly the tutorial does not say if it will create a new dir or expect to be inside an empty dir, nor does --help provide any more information.
This gave me the following exception:
Traceback (most recent call last):
  File "/home/johann/software/anaconda3/envs/python3.8/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/nbdev/cli.py", line 85, in nbdev_new
    url = run('git config --get remote.origin.url')
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/fastcore/xtras.py", line 173, in run
    if res.returncode: raise IOError("{} ;; {}".format(res.stdout, res.stderr))
OSError: b'' ;; b''

I then found isse #307 so I cloned some existing repo, changed into the directory and ran nbdev_new in there.
This gave me the following exception:
Traceback (most recent call last):
  File "/home/johann/software/anaconda3/envs/python3.8/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/nbdev/cli.py", line 99, in nbdev_new
    else: urlsave(f'{FILES_URL}00_core.ipynb')
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/site-packages/fastcore/net.py", line 145, in urlsave
    Path(name).write_bytes(res)
  File "/home/johann/software/anaconda3/envs/python3.8/lib/python3.8/pathlib.py", line 1240, in write_bytes
    view = memoryview(data)
TypeError: memoryview: a bytes-like object is required, not 'str'

So I could not find any way at all to initialize a nbdev repo locally.
Installed nbdev (got 1.1.6) using pip under anaconda with python 3.8 on Ubuntu 20.04.

conda create --name nbdev python=3.8
conda activate nbdev
conda install -c fastai nbdev
nbdev_new nbdev_example

Creating project as above works for me.
Also:
❯ nbdev_new --help
usage: nbdev_new [-h] [--template_git_repo TEMPLATE_GIT_REPO] [--xtra XTRA] name

Create a new nbdev project with a given name.

positional arguments:
  name                  A directory to create the project in

optional arguments:
  -h, --help            show this help message and exit
  --template_git_repo TEMPLATE_GIT_REPO
                        url to template repo (default: https://github.com/fastai/nbdev_template.git)
  --xtra XTRA           Parse for additional args

Hope it helps.

conda create --name nbdev python=3.8
conda activate nbdev
conda install -c fastai nbdev
nbdev_new nbdev_example

Creating project as above works for me.
Also:
❯ nbdev_new --help
usage: nbdev_new [-h] [--template_git_repo TEMPLATE_GIT_REPO] [--xtra XTRA] name

Create a new nbdev project with a given name.

positional arguments:
  name                  A directory to create the project in

optional arguments:
  -h, --help            show this help message and exit
  --template_git_repo TEMPLATE_GIT_REPO
                        url to template repo (default: https://github.com/fastai/nbdev_template.git)
  --xtra XTRA           Parse for additional args

Hope it helps.

As I said I installed nbdev using pip as the README suggests should be possible.
I did this again now using a new conda environment with Python 3.8 and after pip install nbdev I got nbdev version 1.1.11 which no does not work in a different way:
nbdev_new mynbdev1

Gives me:
usage: nbdev_new [-h]
nbdev_new: error: unrecognized arguments: mynbdev1

nbdev_new --help gives:
usage: nbdev_new [-h]

Create a new nbdev project from the current git repo

optional arguments:
  -h, --help  show this help message and exit

nbdev_new without any argument gives inside a git repo that has a remote URL defined gives:
Traceback (most recent call last):
  File "/home/johann/software/anaconda/envs/nbdev/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/nbdev/cli.py", line 99, in nbdev_new
    else: urlsave(f'{FILES_URL}00_core.ipynb')
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/net.py", line 145, in urlsave
    Path(name).write_bytes(res)
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/pathlib.py", line 1240, in write_bytes
    view = memoryview(data)
TypeError: memoryview: a bytes-like object is required, not 'str'

nbdev_new without any arguments inside  a git repo without a remote URL gives:
Traceback (most recent call last):
  File "/home/johann/software/anaconda/envs/nbdev/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/nbdev/cli.py", line 85, in nbdev_new
    url = run('git config --get remote.origin.url')
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/xtras.py", line 174, in run
    if res.returncode: raise IOError(stdout)
OSError

When instead installing nbdev using conda, I get version 0.2.40 and  things seem to work.
So the main problem is that in various places the documentation/README says one can install nbdev using pip when in fact the pip package seems to be very outdated.
The pip package should get updated or the documentation should explicitly point out that pip does not contain the most recent version / should be avoided.
Since I often prefer to install all packages using pip, rather than deal with conda packages from all kinds of different conda channels plus stuff that is only available via pip, I would prefer the first solution.

As I said I installed nbdev using pip as the README suggests should be possible.
I did this again now using a new conda environment with Python 3.8 and after pip install nbdev I got nbdev version 1.1.11 which no does not work in a different way:
nbdev_new mynbdev1

Gives me:
usage: nbdev_new [-h]
nbdev_new: error: unrecognized arguments: mynbdev1

nbdev_new --help gives:
usage: nbdev_new [-h]

Create a new nbdev project from the current git repo

optional arguments:
  -h, --help  show this help message and exit

nbdev_new without any argument gives inside a git repo that has a remote URL defined gives:
Traceback (most recent call last):
  File "/home/johann/software/anaconda/envs/nbdev/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/nbdev/cli.py", line 99, in nbdev_new
    else: urlsave(f'{FILES_URL}00_core.ipynb')
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/net.py", line 145, in urlsave
    Path(name).write_bytes(res)
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/pathlib.py", line 1240, in write_bytes
    view = memoryview(data)
TypeError: memoryview: a bytes-like object is required, not 'str'

nbdev_new without any arguments inside  a git repo without a remote URL gives:
Traceback (most recent call last):
  File "/home/johann/software/anaconda/envs/nbdev/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/nbdev/cli.py", line 85, in nbdev_new
    url = run('git config --get remote.origin.url')
  File "/home/johann/software/anaconda/envs/nbdev/lib/python3.8/site-packages/fastcore/xtras.py", line 174, in run
    if res.returncode: raise IOError(stdout)
OSError

When instead installing nbdev using conda, I get version 0.2.40 and  things seem to work.
So the main problem is that in various places the documentation/README says one can install nbdev using pip when in fact the pip package seems to be very outdated.
The pip package should get updated or the documentation should explicitly point out that pip does not contain the most recent version / should be avoided.
Since I often prefer to install all packages using pip, rather than deal with conda packages from all kinds of different conda channels plus stuff that is only available via pip, I would prefer the first solution.

@jph00 We have to change nbdev_new to use the theme as well.   Will  look into this today also

@jph00 We have to change nbdev_new to use the theme as well.   Will  look into this today also

I was able to reproduce this issue with the following steps:
gh repo create <user-name>/new-test-repo
cd new-test-repo
nbdev_new

Got the following error message
Traceback (most recent call last):
  File "/Users/hamelsmu/anaconda3/bin/nbdev_new", line 33, in <module>
    sys.exit(load_entry_point('nbdev', 'console_scripts', 'nbdev_new')())
  File "/Users/hamelsmu/github/fastcore/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/Users/hamelsmu/github/nbdev/nbdev/cli.py", line 99, in nbdev_new
    else: urlsave(f'{FILES_URL}00_core.ipynb')
  File "/Users/hamelsmu/github/fastcore/fastcore/net.py", line 145, in urlsave
    Path(name).write_bytes(res)
  File "/Users/hamelsmu/anaconda3/lib/python3.8/pathlib.py", line 1240, in write_bytes
    view = memoryview(data)
TypeError: memoryview: a bytes-like object is required, not 'str'

I was able to reproduce this issue with the following steps:
gh repo create <user-name>/new-test-repo
cd new-test-repo
nbdev_new

Got the following error message
Traceback (most recent call last):
  File "/Users/hamelsmu/anaconda3/bin/nbdev_new", line 33, in <module>
    sys.exit(load_entry_point('nbdev', 'console_scripts', 'nbdev_new')())
  File "/Users/hamelsmu/github/fastcore/fastcore/script.py", line 105, in _f
    tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/Users/hamelsmu/github/nbdev/nbdev/cli.py", line 99, in nbdev_new
    else: urlsave(f'{FILES_URL}00_core.ipynb')
  File "/Users/hamelsmu/github/fastcore/fastcore/net.py", line 145, in urlsave
    Path(name).write_bytes(res)
  File "/Users/hamelsmu/anaconda3/lib/python3.8/pathlib.py", line 1240, in write_bytes
    view = memoryview(data)
TypeError: memoryview: a bytes-like object is required, not 'str'

cc: @jph00 this is probably a high priority thing to work on, will try to address this on Monday

cc: @jph00 this is probably a high priority thing to work on, will try to address this on Monday

Should be fixed now if you install the latest fastcore

Should be fixed now if you install the latest fastcore


Shortcut for real-time screen layout customization

Description of the new feature / enhancement
I came from Linux and did not realize the fact that Windows does not have the shortcut alt+click to resize or move windows natively. I think that in this repository we can implement this functionality together with [FancyZone] (https://github.com/microsoft/PowerToys#fancyzones). ⭐
It would be an implementation similar to: Allow customization of the layout in real time, according to the user's preferences at the time.
Scenario when this would be used?
It helps and a lot in productivity, especially for people who use several windows at the same time and have little space on the monitor. 😄
Other Advantages of This Implementation: 👍

No time is wasted trying to hit the edges of the screen with the mouse to make a simple resize, avoiding loss of productivity;
Familiarity and agility to use the system, honestly I can't adapt without this type of functionality.

Supporting information
In the Linux world, these settings are available in the key 💻 :
/org/gnome/desktop/wm/preferences/mouse-button-modifier

therefore, it can be better understood by researching it. 😄
Or it can be observed the use of it by the community in some forums:
how-to-get-altright-mouse-to-resize-windows-again
how-to-reset-all-gnome-tweak-tool-settings-to-defaults
This GIF demonstrates a little of what I'm suggesting 🎥 :

This repository is one of the example methods for the schema of this suggestion: gsettings-desktop-schemas

Thank you for the suggestion! We'll investigate the possibility of adding this into Fancy Zones or making it a new Power Toy and set prioritization following our general stability efforts.

Not a bad idea. Seems related to issues #1823, #269, #39.

Wouldn't this be alt-drag which is issue #269

Wouldn't this be alt-drag which is issue #269

Alt-drag only moves, this one resizes too.

Wouldn't this be alt-drag which is issue #269

If it will include both moving and resizing (and send-to-back) I think all three issues are included.


Build Ruby 3.1.2, 3.0.4, and 2.7.6

See:

https://www.ruby-lang.org/en/news/2022/04/12/ruby-3-1-2-released/
https://www.ruby-lang.org/en/news/2022/04/12/ruby-3-0-4-released/
https://www.ruby-lang.org/en/news/2022/04/12/ruby-2-7-6-released/

What I did here:

Bumped patch versions and package revisions in config.yml
Let pre-commit hook to do everything else

Cool! thanks for creating the PR :)

Hi @FooBarWidget , would be great to have this merged due to the two CVEs.

Yes on it. Sorry had a busy weekend.

Release done. Thanks @Envek


attestation: specialize error when gh is old

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This isn't really ideal (I'll look further into the bootstrap cycle issues that led us to try version sniffing in the first place), but it'll help more users in the current beta resolve issues they're seeing.
See https://github.com/Homebrew/homebrew-core/issues/177384#issuecomment-2263195832.

Honestly, it might just be easier to only check brewed gh version in Attestation.enabled? e.g.
gh_version = Formula["gh"].any_installed_version
return false if gh_version.nil? || gh_version < "2.49"

Especially given we hope to phase out gh entirely.

Honestly, it might just be easier to only check brewed gh version in Attestation.enabled? e.g.

Yeah, seems right. I remember we had some bootstrap cycle issue with accessing Formula[...] in that context, but I think as long as it's below no_verify_attestations? that shouldn't be an issue.

Honestly, it might just be easier to only check brewed gh version in Attestation.enabled? e.g.

Agreed.

I've updated this so that we now only enable attestations if gh is already installed and sufficiently new (unless the user explicitly enables them with HOMEBREW_VERIFY_ATTESTATIONS=1). This has the side effect of curbing the current beta significantly, since most people currently in it are probably getting it from having developer mode enabled. But per @Bo98's points that seems fine to me.

Hey! This looks like it might be the cause of #18028

Hey! This looks like it might be the cause of #18028

Looks like it, thanks. Reverting


@Import extensionless file throws a System.UriFormatException

Attempting to import a partial file that does not have an extension causes an error to be thrown by the DartSassHost code which is NOT handled with a SassException. At first I thought this might be a Dart Sass issue, but now it appears to be a DartSassHost bug:
Using a virtual filesystem that allows files without extensions and file structure like this:

/bootstrap5/scss/bootstrap.scss

@import "mixins/banner";

/bootstrap5/scss/mixins/_banner

... contents of this file do not matter for this issue ...

Attempting to compile bootstrap.scss results in the following error message:
System.UriFormatException: Invalid URI: The format of the URI could not be determined.
   at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
   at DartSassHost.Helpers.PathHelpers.MakeRelativePath(String relativeTo, String path)
   at DartSassHost.Helpers.PathHelpers.PrettifyPath(String currentDirectory, String path)
   at DartSassHost.JsonConverters.CompilationResultConverter.ReadStackFrame(Utf8JsonReader& reader, String currentDirectory)
   at DartSassHost.JsonConverters.CompilationResultConverter.ReadStackFrames(Utf8JsonReader& reader, String currentDirectory)
   at DartSassHost.JsonConverters.CompilationResultConverter.ReadError(Utf8JsonReader& reader)
   at DartSassHost.JsonConverters.CompilationResultConverter.ReadFirstError(Utf8JsonReader& reader)
   at DartSassHost.JsonConverters.CompilationResultConverter.ReadResult(Utf8JsonReader& reader)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonConverter`1.ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)
   at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan`1 utf8Json, JsonTypeInfo jsonTypeInfo, Nullable`1 actualByteCount)
   at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan`1 json, JsonTypeInfo jsonTypeInfo)
   at System.Text.Json.JsonSerializer.Deserialize[TValue](String json, JsonSerializerOptions options)
   at DartSassHost.SassCompiler.InnerCompile(String content, Boolean indentedSyntax, String inputPath, String outputPath, String sourceMapPath, CompilationOptions options)
   at DartSassHost.SassCompiler.Compile(String content, String inputPath, String outputPath, String sourceMapPath, CompilationOptions options)
   at ...

Aside from the fact that in most system having files without extensions is generally not a good idea (let's just say I don't have any control over that), this code seems to be supported by Dart Sass and can be compiled successfully using other tools (including the LibSassHost library).

Hello, Levi!
Show me the source code of the CustomVirtualFileManager class.

Hello, Levi!
It is quite possible that the error you describe has been fixed in version 1.0.0.


10 course functionality

For page.tsx files, please use "export default function function name, like Page or Home {}" syntax instead of fat arrow syntax "page: any = () => {}". Fat arrow can be used for everything except page.tsx.

This raises an error though. I think typescript requires you to specify the type and that requires a fat arrow function?

This raises an error though. I think typescript requires you to specify the type and that requires a fat arrow function?

Strange, it doesn't raise an error for me. I'll change it to mine and we can look at it in person.


update newly passing tests

Probably started working with https://github.com/pytorch/pytorch/pull/68565

Closing in favour of https://github.com/pytorch/functorch/pull/285


Change Result<T, impl miette::Diagnostic> to Result<T, E>

Category
User level API features/changes
Describe the feature you'd like to request
Currently some api returns Result<T, impl miette::Diagnostic> instead of Result<T, E> convert all methods for better consistency as well as it is easier to work with Result<T, E>
Currently 4 methods returns Result<T, impl miette::Diagnostic>

https://docs.rs/cedar-policy/latest/cedar_policy/struct.Entity.html#method.attr
https://docs.rs/cedar-policy/latest/cedar_policy/struct.EntityUid.html#method.from_json
https://docs.rs/cedar-policy/latest/cedar_policy/struct.Template.html#method.to_json
https://docs.rs/cedar-policy/latest/cedar_policy/struct.Policy.html#method.to_json

Describe alternatives you've considered
None
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Somewhat related to #695. I think the comment there is relevant.
Basically, we're looking to rework error types in a 4.0 release, but we can consider non-breaking changes before that.

Related: #745


Dubious but working code in the plugins section of build.gradle.kts, that is correctly marked as invalid in IntelliJ

Current Behavior
We have code in the plugins section of the root build.gradle.kts in the Kotlin repo, that contains an if statement accessing API on the Project. Building the Kotlin project does not fail at this place and even seemingly works correctly if the accessed property is changed.
But IntelliJ displays an error on this call, as it is seen on the screenshot below.
It looks like IntelliJ's interpretation is correct, according to the Kotlin language rules, because the receiver on plugins function is marked with a DSL marker GradleDsl, so the DSL on the Project should not be accessible there.

Expected Behavior
I would expect the build to fail on the access to kotlinBuildProperties, similarly to the IntelliJ behavior.
Alternatively, the API could be changed to allow the access there, but this could be a dangerous change.
Context (optional)
No response
Steps to Reproduce
The code in the Kotlin repo demonstrates it.
Gradle version
8+
Build scan URL (optional)
No response
Your Environment (optional)
No response

The issue is in the backlog of the relevant team and is prioritized by them.


Problem uploading to ESP32

I have a Daikin Altherma heatpump from 2008, EKHB007AGT1 and ERYQ006ACV3. PCB DB-G57-101. I have installed and maintained my heatpump for many years. I have also done some modification for relay to the water pump, after fault code AA, relay KPR was defect and I mounted a new relay. Then mounted a new relay unit outside the PCB to omit new fault for KPR relay in the future. The X10 connection have 8 pins.
I'm new to platformio and started yesterday. I have an ESP32 devkit v1, from e-bay. Testet with some hello wold and test internal blink led. But with loading your code to ESP32 I got an error message:

Executing task: C:.platformio\penv\Scripts\platformio.exe run --target upload <

Processing esp32 (platform: espressif32; board: esp32doit-devkit-v1; framework: arduino)
Verbose mode can be enabled via -v, --verbose option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/esp32doit-devkit-v1.html
PLATFORM: Espressif 32 (3.1.1) > DOIT ESP32 DEVKIT V1
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (esp-prog) External (esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:

framework-arduinoespressif32 3.10005.210308 (1.0.5)
tool-esptoolpy 1.30000.201119 (3.0.0)
tool-mkspiffs 2.230.0 (2.30)
toolchain-xtensa32 2.50200.97 (5.2.0)
LDF: Library Dependency Finder -> http://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 29 compatible libraries
Scanning dependencies...
Dependency Graph
|--  2.8.0
|--  1.0
|   |--  1.0
|   |--  1.0
|   |--  1.0
|   |   |--  1.0
|--  1.0
|--  1.0.3
Building in release mode
Retrieving maximum program size .pio\build\esp32\firmware.elf
Checking size .pio\build\esp32\firmware.elf
Advanced Memory Usage is available via "PlatformIO Home > Project Inspect"
RAM:   [=         ]  14.5% (used 47364 bytes from 327680 bytes)
Flash: [======    ]  59.0% (used 772677 bytes from 1310720 bytes)
Configuring upload protocol...
AVAILABLE: esp-prog, espota, esptool, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa
CURRENT: upload_protocol = esptool
Looking for upload port...
Auto-detected: COM6
Uploading .pio\build\esp32\firmware.bin
esptool.py v3.0
Serial port COM6
Connecting........_
Chip is ESP32-D0WDQ6 (revision 1)
Features: WiFi, BT, Dual Core, 240MHz, VRef calibration in efuse, Coding Scheme None
Crystal is 40MHz
MAC: 3c:61:05:2a:6d:74
Uploading stub...
Running stub...
Stub running...
Changing baud rate to 215200
Changed.
Configuring flash size...

A fatal error occurred: Timed out waiting for packet header
*** [upload] Error 2
==================================================================== [FAILED] Took 9.17 seconds ====================================================================
Environment    Status    Duration

esp32          FAILED    00:00:09.175
=============================================================== 1 failed, 0 succeeded in 00:00:09.175 ===============================================================
The terminal process "C:.platformio\penv\Scripts\platformio.exe 'run', '--target', 'upload'" terminated with exit code: 1.
Terminal will be reused by tasks, press any key to close it.

Hello @Oystein-github and Welcome!
A fatal error occurred: Timed out waiting for packet header this is your error.

Make sure that nothing is connected on the board when you upload.
Try to lower your upload speed upload_speed = 115200 in your platformio.ini
Press the reset button and release just before the upload starts.

Thank you, it works with # 2, change the upload_speed to 115200. Then I can go forward to test it.

Glad to hear this :)


Add compression support

Adding accept-encoding to the request will make Vespa compress the response, which can reduce latency for cross-region query traffic:
Accept-Encoding: gzip

This is the default behavior for both httpx and requests.
See
They will also be decompressed automatically.
See #883

From httpx docs:

Any gzip and deflate HTTP response encodings will automatically be decoded for you.

From requests FAQ:

Requests automatically decompresses gzip-encoded responses, and does its best to decode response content to unicode when possible.


[1.6.x][KOGITO-4940] move KieBase creation in generated code into a static f…

…… (#1236)

[KOGITO-4940] move KieBase creation in generated code into a static field

wip

wip

wip

Update kogito-codegen-modules/kogito-codegen-rules/src/main/java/org/kie/kogito/codegen/rules/ProjectSourceClass.java

Co-authored-by: Daniele Zonca<EMAIL_ADDRESS>
Update kogito-codegen-modules/kogito-codegen-rules/src/main/java/org/kie/kogito/codegen/rules/ProjectSourceClass.java

Co-authored-by: Daniele Zonca<EMAIL_ADDRESS>
Update kogito-codegen-modules/kogito-codegen-rules/src/main/java/org/kie/kogito/codegen/rules/ProjectSourceClass.java

Co-authored-by: Daniele Zonca<EMAIL_ADDRESS>

wip

wip

Co-authored-by: Daniele Zonca<EMAIL_ADDRESS>(cherry picked from commit 8d55e448f67317679cc661048844444bbef4af54)
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Pull Request
Please add comment: Jenkins retest this

Quarkus LTS checks
Please add comment: Jenkins run LTS

Native checks
Please add comment: Jenkins run native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

The Linux check has failed. Please check the logs.


[proxima-platform] shade io packages to shield conflicting dependencies

I dont like this one, every io should have separate jar with a child first classloader. It would be much easier for other people to add and maintain io modules and will prevent lot of potential issues in the future..

consider io modules as a user code, you have to execute. there is an analogy to spark executors running udfs

I understand your point, but I think that has to be done in separate issue. Would you mind filling one please? I'm currently not in a position to be able to implement it.


LiveDeck.liveCards needs review

Description
LiveDeck.liveCards currently refers only one party (5 cards, presumed to be allies), leading to opponent party exists no where.
Purpose
Makes opponent party to be contained inside Live object.

Currently ideas
Solution 1
Modifies LiveDeck.liveCards make it contains 2 parties (10 cards).
Solution 2
Creates another object named OpponentLiveCards or anything else inside of LiveDeck.


The image and gif files shown on PyPI need to be corrected.

Describe the bug
A clear and concise description of what the bug is.
The image and gif files shown on PyPI need to be corrected.

To Reproduce
Steps to reproduce the behavior:

Code/Link to Repo:

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Specifics (please complete the following information):

Python Version:
Reflex Version:
OS:
Browser (Optional):

Additional context
Add any other context about the problem here.

Research
I find examples that can show images and .gif on PyPI correctly.
Case of showing the image
FastAPI
FastAPI on PyPI -> https://pypi.org/project/fastapi/
FastAPI README.md on GitHub  -> https://github.com/milochen0418/fastapi/blob/master/README.md?plain=1
Case of showing the .gif
streamlit-keyup
streamlit-keyup on PyPI -> https://pypi.org/project/streamlit-keyup/
streamlit-keyup README.md on GitHub -> https://github.com/blackary/streamlit-keyup/blob/main/README.md?plain=1
Conclusion:
From the observation, the way to fix this issue is to put the full-path hyperlink on README.md
But How to solve it?
We can find the out answer here.
https://github.com/pypi/warehouse/issues/5246
When you have the path,
https://github.com/kirankotari/shconfparser/blob/master/asserts/img/sh_run.png
You can use the following to replace it.
https://github.com/kirankotari/shconfparser/raw/master/asserts/img/sh_run.png
All you need to do is replace blob into raw and you can get the raw full-path hyperlink.
Then put it into your README.md
For example, the following is the such solution
shconfparser on PyPI -> https://github.com/network-tools/shconfparser/blob/master/README.md?plain=1
shconfparser README.md on GitHub -> https://github.com/network-tools/shconfparser/blob/master/README.md?plain=1


Default sorting on "gained" tab

Suggestion:
Change the default sorting for the gained tab on an individual player to EHP/EHB, rank, or "gained" (I'm not sure which one would be best) instead of alphabetically.
Most likely, a user is going to be looking at that tab to see the skills/bosses/activities they have been working on recently, so it would make sense (to me) for one of those metrics to show up at the top.
Current default sorting:

More relevant sorting:

Although I recognize that showing relevant data first is useful, I personally prefer the default order to be consistent, aka each line be where you expect it to be, and in this case that means keeping the same order as the OSRS hiscores and having them ordered alphabetically, which makes it easier to find.
Also, you can do that with just one click on the column header, and that's why it's there, to allow for customization.
But i'm open to other opinions.

No updates on this, no one seems to hace an alternate opinion, closing for now.


⚠ Fortnite has degraded performance

In 516d6ee, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1513 ms

Resolved: Fortnite performance has improved in 34458f7.


🐛 [Bug] Docker build fails

Bug Description
Sending build context to Docker daemon  29.57MB
Step 1/34 : ARG BASE=22.01
Step 2/34 : ARG BASE_IMG=nvcr.io/nvidia/pytorch:${BASE}-py3
Step 3/34 : FROM ${BASE_IMG} as base
 ---> 3f290bb26216
Step 4/34 : FROM base as torch-tensorrt-builder-base
 ---> 3f290bb26216
Step 5/34 : RUN rm -rf /opt/pytorch/torch_tensorrt /usr/bin/bazel
 ---> Using cache
 ---> 6a5e47ade331
Step 6/34 : ARG ARCH="x86_64"
 ---> Using cache
 ---> 94acf4d3746c
Step 7/34 : ARG TARGETARCH="amd64"
 ---> Using cache
 ---> 6e159a1f5635
Step 8/34 : ARG BAZEL_VERSION=4.2.1
 ---> Using cache
 ---> 981ce2761707
Step 9/34 : RUN [[ "$TARGETARCH" == "amd64" ]] && ARCH="x86_64" || ARCH="${TARGETARCH}"  && wget -q https://github.com/bazelbuild/bazel/releases/download/${BAZEL_VERSION}/bazel-${BAZEL_VERSION}-linux-${ARCH} -O /usr/bin/bazel  && chmod a+x /usr/bin/bazel
 ---> Using cache
 ---> 3387313ebde0
Step 10/34 : RUN touch /usr/lib/$HOSTTYPE-linux-gnu/libnvinfer_static.a
 ---> Using cache
 ---> 2be1d77d6422
Step 11/34 : RUN rm -rf /usr/local/cuda/lib* /usr/local/cuda/include   && ln -sf /usr/local/cuda/targets/$HOSTTYPE-linux/lib /usr/local/cuda/lib64   && ln -sf /usr/local/cuda/targets/$HOSTTYPE-linux/include /usr/local/cuda/include
 ---> Using cache
 ---> 97185e01fe2f
Step 12/34 : RUN apt-get update && apt-get install -y --no-install-recommends locales ninja-build && rm -rf /var/lib/apt/lists/* && locale-gen en_US.UTF-8
 ---> Using cache
 ---> 85a8c0dd0f06
Step 13/34 : FROM torch-tensorrt-builder-base as torch-tensorrt-builder
 ---> 85a8c0dd0f06
Step 14/34 : RUN rm -rf /opt/pytorch/torch_tensorrt
 ---> Using cache
 ---> 05475178cb1e
Step 15/34 : COPY . /workspace/torch_tensorrt/src
 ---> Using cache
 ---> 5c2b45977e72
Step 16/34 : WORKDIR /workspace/torch_tensorrt/src
 ---> Using cache
 ---> 080d93900192
Step 17/34 : RUN cp ./docker/WORKSPACE.docker WORKSPACE
 ---> Using cache
 ---> f2a6f1878089
Step 18/34 : RUN ./docker/dist-build.sh
 ---> Running in fb5b50c7fca3
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
running bdist_wheel
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Loading:
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Analyzing: target //:libtorchtrt (1 packages loaded, 0 targets configured)
INFO: Analyzed target //:libtorchtrt (43 packages loaded, 3021 targets configured).
INFO: Found 1 target...
[0 / 81] [Prepa] Writing file cpp/lib/libtorchtrt.so-2.params ... (2 actions, 0 running)
ERROR: /workspace/torch_tensorrt/src/core/lowering/passes/BUILD:10:11: Compiling core/lowering/passes/linear_to_addmm.cpp failed: (Exit 1): gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 61 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
core/lowering/passes/linear_to_addmm.cpp: In function 'void torch_tensorrt::core::lowering::passes::replaceLinearWithBiasNonePattern(std::shared_ptr<torch::jit::Graph>)':
core/lowering/passes/linear_to_addmm.cpp:37:93: error: 'struct torch::jit::Function' has no member named 'graph'
   37 |         std::shared_ptr<torch::jit::Graph> d_graph = decompose_funcs.get_function("linear").graph();
      |                                                                                ^~~~~
Target //:libtorchtrt failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 166.072s, Critical Path: 17.92s
INFO: 1243 processes: 1196 internal, 47 processwrapper-sandbox.
FAILED: Build did NOT complete successfully
FAILED: Build did NOT complete successfully
using CXX11 ABI build
building libtorchtrt
The command '/bin/sh -c ./docker/dist-build.sh' returned a non-zero code: 1

To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/NVIDIA/Torch-TensorRT
cd Torch-TensorRT
docker build --build-arg BASE=22.01 -f docker/Dockerfile -t torch_tensorrt:latest .

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version (e.g. 1.0.0):
PyTorch Version (e.g. 1.0):
CPU Architecture:
OS (e.g., Linux):
How you installed PyTorch (conda, pip, libtorch, source):
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version:
CUDA version:
GPU models and configuration:
Any other relevant information:

Additional context

@Biaocsu: Which branch are you using?
Please use release/ngc/22.01 branch if you want to build the source inside the NGC container.

@Biaocsu: Which branch are you using? Please use release/ngc/22.01 branch if you want to build the source inside the NGC container.

I used the latest master branch. OK, I will use "release/ngc/22.01" branch

@andi4191 can this issue be solved, as it has been so long.
I need to pull the master branch for many issue solved where other branch do not support.
please, help solve and verify this problem

Hi @chophilip21 @Biaocsu,
It seems the error is related to this API change updated here in NGC 22.02 release.
https://github.com/NVIDIA/Torch-TensorRT/commit/449723ef014fc839cbed5fda0824419a902bd403#
However, I am able to build release/ngc/22.01 branch with BASE=22.01. Can you please check if your are using the correct checked out version?


The hotkey FN using for fast forward in Retroarch by default

The hotkey FN (button 16) using for fast forward in Retroarch by default.
It conflict with retroarch menu button
Please remove it.

What unit are you referring too?  I don't know of any supported devices where a Function key is set up as a particular hotkey in Retroarch by default.

r36s

I don't support the R36S.  You'll need to go here for support for the R36s using ArkOS. https://github.com/AeolusUX/ArkOS-R3XS

Aeolus says that its from you image
https://github.com/AeolusUX/ArkOS-R3XS/issues/25

That's not what he's saying at all.  He's saying he will not make that change globally in his fork.  You will have to make that change in Retroarch directly for your use case.  Just go to the Retroarch settings menu>Input>Hotkeys>Fast-Forward (Toggle or Hold) then press Y.  Then go back to the root retroarch menu and go to configuration file then do Save Current Configuration.


Add pattern to fold tensor.empty used by extract in transform dialect

This allows generating smaller allocations for reduction code generation. Without that we may generate allocation based on the original size rather than the tiled size.

I think this already exists upstream.

ah ok, I looked for it and couldn't find it. Do you have a pointer?

I think this already exists upstream.

ah ok, I looked for it and couldn't find it. Do you have a pointer?

never mind, I found it. Replaced.

I asked in offline chat if we really want a blanket application vs more targeted control as it can have non trivial effects on bufferization in general and could be a footgun. @matthias-springer  is the most up-to-date on patterns interference with bufferization and what foldings have "fusion-like" effects.


Cannot Figure Out How to Use without CLI, Docs Do Not Help

🐛 Bug
Not a bug, but the library just doesn't seem to be able to be used outside the CLI out of the box. I'm trying to import a separator as specified in the documentation, but if you import the class through models, then you need to specify a Model (nn.Model), which you don't have access to out of the box. Using one of the util methods, load_separator, also does not work, as it leads to a 403 Forbidden Error.
To Reproduce
Steps to reproduce the behavior:

install openunmix
try to import and use the Separator class
try to use the utils.load_separator class with any arguments

Expected behavior

I can load a separator in code and use it to create stems programmatically without calling from the CLI or using subprocess.
Environment
Please add some information about your environment

PyTorch Version (e.g., 1.2): 2.2.2
OS (e.g., Linux): Sonoma 14.1
torchaudio loader (y/n): n
Python version: 3.11.7
CUDA/cuDNN version:
Any other relevant information:

If unsure you can paste the output from the pytorch environment collection script
(or fill out the checklist below manually).
You can get that script and run it with:
wget https://raw.githubusercontent.com/pytorch/pytorch/master/torch/utils/collect_env.py
# For security purposes, please check the contents of collect_env.py before running it.
python collect_env.py

Additional context

@bgenchel seems that the model checkpoint on zenodo is down
import torch
separator = torch.hub.load('sigsep/open-unmix-pytorch', 'umxhq')

produces the same error. I will investigate

fixed in newest release v1.3.0


[$250] Settings - Legal first and last name fields aren't accepting umlaut characters

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.13-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Navigate to https://staging.new.expensify.com/
Log in with any annyount
Navigate to Account settings - Legal name
Input "Ádám" as first name
Input "Horváth" as last name
Click on the "Save" button

Expected Result:
I should be able to save a legal name with umlauts
Actual Result:
Legal first and last name fields aren't accepting umlaut characters
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/5f4a44c6-f9cf-4748-b28a-cb4332a2cd31
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015c756a4a661d9b30
Upwork Job ID:<PHONE_NUMBER>226587925
Last Price Increase: 2024-08-05

Looks like bank account personal name rejected accent chars in https://github.com/Expensify/App/pull/37052 and then reverted in https://github.com/Expensify/App/pull/44689 to accept back.
Similarly, should we now support accent chars in legal name?
cc: @johncschuster

Proposal
Please re-state the problem that we are trying to solve in this issue.
Legal first and last name fields aren't accepting umlaut characters
What is the root cause of that problem?
In the code below, the validation of the legal name requires that it has no accented characters.
https://github.com/Expensify/App/blob/5e6527613d0f234fcb22507b878697a2bec356c5/src/libs/ValidationUtils.ts#L355-L356
What changes do you think we should make in order to solve the problem?
To accept accented characters, remove the hasAccentedChars check from the code below so we take all Latin characters.
https://github.com/Expensify/App/blob/5e6527613d0f234fcb22507b878697a2bec356c5/src/libs/ValidationUtils.ts#L355-L356
After that, remove the ACCENT_LATIN_CHARS regex from the code below as it is no longer required.
https://github.com/Expensify/App/blob/5e6527613d0f234fcb22507b878697a2bec356c5/src/CONST.ts#L2292
Below is a POC after applying the change:
https://github.com/user-attachments/assets/2a7bb2ea-902b-4b80-9cfe-2ee362331999
What alternative solutions did you explore? (Optional)

Thanks for raising your question above, @situchan!
I'm not 100% sure if we've elected to accept special characters in the legal first and last name sections of the product. I've raised a discussion in #product to figure that out.

Similarly, should we now support accent chars in legal name?

Yes, I think so. If we're going to accept accent characters in the personal name fields, we should accept them in all fields in which a user can enter their legal name.

Bump to remove the Overdue label

We can go with @daledah's alternative solution as we now accept accent characters in legal name.
🎀👀🎀 C+ reviewed

Let's roll with @daledah's approach 👍

Looks like the changes are out- @johncschuster would you be able to handle the next steps here? 🙏

bump @johncschuster on next steps 🙏

Sorry for missing your ping, @robertjchen! Yep! I'll take care of next steps (let me catch up on those now)

@daledah, can you please apply to the Upwork Job?
@situchan, do we need a set of regression test steps?

I've adjusted the title of the issue to reflect the date this should have been paid out. This will also help me come back to this since I action all of my "Hold for payment" issues first thing in the morning and won't miss it.

The automated tests already cover regression test so I think we're all good.

Sweet. Thanks!

I've issued payment to @situchan. All that's left is to issue payment to @daledah 👍

Sure thing, @daledah! I've just sent you an offer. Can you accept that?

thanks, all!


Notification job only update payment status

Describe the bug
Hi, we have some orders on production which have been correctly paid in Adyen but they finally have been failed in SFCC.
In the Business Manager, we can see in the history that the order was in status created before the redirection to payment page (can be Oney, CB with 3DS), but the order was never placed.
After a few hours, based on the SFCC configuration, the order is automatically failed.
To Reproduce
Steps to reproduce the behavior:

Make a payment with external redirection, like Oney
When the payment is validated, close the page before redirection to the confirmation page
Check the order status after the notification process has been executed for your order

Expected behavior
We would expect a check on the order status in the notification process to prevent this kind of errors.
If the payment is confirmed but the order is still with status created, the order should be placed by the job.
Placing the order only on a front process is not 100% secure, some client can close the page, your phone can turn off etc...

Merchant account : PacificPecheECOM
PSP examples :<PHONE_NUMBER>492673 or<PHONE_NUMBER>434404

Hi @aquinzain ,
Thanks for creating this issue.
I will investigate this and create an internal ticket for it.
I will keep you updated.
Kind regards,
Bas Maassen
Adyen

Hi @aquinzain ,
We have fixed this issue with PR #426.
Kind regards,
Bas Maassen
Adyen

Hi @maassenbas,
The fix has been approved on our side, but we had to fix two little things in adyenService.js on line 42 and 47
`    if (order.status === Order.ORDER_STATUS_CREATED) {
if (order.status**.value** === Order.ORDER_STATUS_CREATED) {

  // custom fraudDetection

  const fraudDetectionStatus = { status: 'success' };

  const placeOrderResult = COHelpers.placeOrder(

    order,

    fraudDetectionStatus,

    fraudDetectionStatus

  );`

Thanks @aquinzain.
Will close the issue now.

Hi @maassenbas
Found this issue and I think we might be having the same issue. I would like to implement this fix, but I have a question.
What are the reason for using a custom sendEmail function to generate the confirmation e-mail?
Why not use the COHelpers.sendConfirmationEmail() function? As you do in the Adyen.js Controller?
That would include any customization created for the order confirmation e-mail.
Best regards
Søren


Core-node-docs: read file

Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

Need to add meta title and description.


Not upgraded to WebSocket

I get this error when trying to connect to a socket.io server from a Flutter app.
WebSocketException: Connection to '..' was not upgraded to websocket
I get this library from "Flutter" section of dart libraries.

This library is WAMP protocol client.
I don't know any relationship with socket.io.
Thanks


Fill for all shapes doesn't work

The fill options show only once you pick a non-transparent background color.
Not sure how you ended up with the screenshot 1 (could be a multiselection where at least one element had a non-transparent bg, but in that case the color input should be empty 🤔)

Not sure if this was intended, but the fill option will appear once you select a background color for a shape. It looks like by default it won't show the fill options because it's transparent by default

Thanks @dwelle, I didn't notice that there was transparent background enabled.
This fixed my problem


fix: LSDV-5167: Ranker undo/redo

Store original List items in result to track their changes too, this allows to rollback any changes made to Ranker. Full Ranker state is serialized now.
PR fulfills these requirements

[ ] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[ ] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

Codecov Report
Patch coverage: 16.66% and project coverage change: +0.02 :tada:

Comparison is base (5f853c3) 65.20% compared to head (9b74279) 65.23%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1416      +/-   ##
==========================================
+ Coverage   65.20%   65.23%   +0.02%
==========================================
  Files         436      436
  Lines       27252    27249       -3
  Branches     7214     7214
==========================================
+ Hits        17771    17775       +4
+ Misses       8139     8133       -6
+ Partials     1342     1341       -1

Impacted Files
Coverage Δ

src/components/Ranker/Item.tsx
11.11% <0.00%> (ø)

src/tags/control/Ranker.js
11.59% <20.00%> (+0.48%)
:arrow_up:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/git merge master


1.0.14_crab version of Use system environment for gfal2 stageout

@mmascher can you please make a reference here to the master PR?

it's https://github.com/dmwm/WMCore/pull/6808

As per Emilis request, I've created a new tag:
1.0.14_crab_7


Ldap User are still existing after deleting them in GUI

After removing demo user and tenants in GUI they still exist in Ldap. To recreate use the command
sudo ldapsearch -H ldaps://<IP_ADDRESS> -D cn=Manager,ou=systemuser,ou=user,dc=fworch,dc=internal -y /usr/local/fworch/etc/secrets/ldap_manager_pw.txt -x | grep demo

users, groups could be deleted in ldap, but there was a bug in tenant deletion


Add Ord implementation for BoundedVec

interestingly other bounded types already had it, this one was forgotten.

can we derive PartialOrd here?

bot merge

bot merge


Support for pgdiskann client

This PR adds support for pgdiskann - a recent postgres extension that adds support for DiskANN index type to postgres. Following pgdiskann specific parameters are configurable for pgdiskann:
l_value_is
l_value_ib
max_neighbors
Note
The pgdiskann extension is currently in private preview on Azure PostgreSQL and requires special permission to access. For more details, see the preview announcement and fill out this form to request access.

/assign @alwayslove2013


GitHub Action

Any Relevant Actions To Be Used can be mentioned here

please add lint test

can I set-up eslint for this project ?

@ashishnagpal2498 I would like to work on this issue

still working on the issue @AltCtrlDel1999 ?

Can you suggest any link to know about github Actions
I tried one but getting Error

I will complete it by tomorrow

can I work on this issue?

Sure @rajatgoel47 . Assigning it to you.


Adapt section components to new style

See https://www.figma.com/file/mibA3VIpBm4nyJTsIKtY8o/React---UI-Library?node-id=99%3A3068

Is this the proposed style and color of Tobi for the error screen?

We don't have a proposal for this, it's the "old" message screen. I'll have Tobi take look.

Is this the proposed style and color of Tobi for the error screen?

We don't have a proposal for this, it's the "old" error message. I'll have Tobi take a look.

Tobi says it's ok for him like that.

@alexlanz @mmalfertheiner I suggest to squash and merge this PR to avoid 46 commits being merged into the main branch (and we have some fixes and reverts here), with commit message: "adapt section components to new design". Wdyt?

For me it's fine. I think it makes sense in this case.


Thanks to Polka

Hi All,
I have created a universal router based on Trouter and Polka. Kindly check and let me know your feedback.
urout - https://www.npmjs.com/package/urout - Universal Router.
userve - https://www.npmjs.com/package/userve - Express like router framework.
ubrow - https://www.npmjs.com/package/ubrow - Express like browser router.
Thanks

Very cool! 😄 Glad to give some inspiration 👏
Just so you know (you probably do!), Trouter can actually be run in the browser as is – you just have to attach event listener(s) to do the matching. And then a more browser-based solution is navaid.
Congratulations to you, good suite of libs 🎉


[WIP] Add dictionary variables and related blocks

See also https://github.com/FBDY/bb-vm/issues/50.

Now, there is the long work of adding related blocks one by one... Then comes the VM work for their execution at https://github.com/FBDY/bb-vm/pull/51.

At this point it seems like a good idea to work on https://github.com/FBDY/bb-vm/pull/51 in parallel to this. At least testing the add/get blocks would be nice.


DragonFly - Change PTHREAD_STACK_MIN from 1 to 16k

This is probably related to cargo failing on DragonFly when running with
multiple jobs.
The minimum pthread stack size has been increase as shown here 1,
which was needed to fix nodejs.

@bors: r+

:pushpin: Commit 9fe906c has been approved by alexcrichton

:hourglass: Testing commit 9fe906c72ec90282f98ebfde500d5f19a03df478 with merge 50bc03daa4cbb24fe3b8854d44d5d8f4366145a0...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 50bc03daa4cbb24fe3b8854d44d5d8f4366145a0 to master...


Importing flows from a JSON with umlauts doesn't work

What are the steps to reproduce?
Import flows from a JSON-File on my computer. I can click on the file and it puts the content in the text area.
What happens?
The problem is that all umlauts ( ä,ö,ü) aren't displayed correctly.
What do you expect to happen?
All the umlauts should be displayed as they look.
Please tell us about your environment:

[x ] Node-RED version: 1.0.3
[ ] Node.js version:
[ ] npm version:
[ x] Platform/OS: Windows
[ x] Browser: Chrome

Hi @laquesisa
can you share an example file that demonstrates the issue?
The Import dialog allows you to upload a file, rather than paste the contents in. Can you check that method works for you? We added that mode specifically in response to this type of problem (with Japanese characters in that instance).

Closing due to inactivity. If this is still an issue, please reopen with the requested information.


[Enhancement] Support For Configuring Filters by Dragging Items Directly From NEI

Please add support for enderio Filters to recieve configuration information directly from NEI Currently to configure filters on item conduits the player must have the item they wish to set in their inventory and  drag it over the filter. There are several mods that allow item filters to be populated by dragging items directly from NEI which makes it much easier to initially set up item filters.

That definitely sounds convenient. What other mods support that feature?

Logi pipes does this Generally for complex crafting operations using machines i just the crafting pipe hooked to an output buffer and a satellite to an input buffer with enderio conduits providing the back-end item transport between the machine array and the output buffer.


[#142128979] Add DWP Carer's Allowance IP to the whitelist

What
As part of the #142128979, we'd like to add the DWP Carer's Allowance IP to our whitelist. This should be followed by creation of the organisation and users, for them to try out the PaaS.
How to review
Make sure the correct IP address has been added.
Who can review
Not @paroxp

LGTM


Need content for Privacy Statement

@matthewcarleton Matthew, I've added a PR based on kubevirt if it gets merged as it is, will require linking "/privacy" from the relevant pages so that we can close this issue
@ptrnull ^


Load the network data for the first time, pictures showed not to come out, after the refresh can display, how to solve？

Load the network data for the first time, pictures showed not to come out, after the refresh can display, how to solve？
code：
 CGFloat width = (SCREENWIDTH - 30)/2;         return [collectionView ar_sizeForCellWithIdentifier:@"RecommendedCell"                                                   indexPath:indexPath                                                  fixedWidth:width configuration:^(id cell) {                                                      RecommendedListResultModel *listData =self.m_newArr[indexPath.row];                                                      [cell setCellData:listData];

What did you use to download images? it's strange.

Is to load the network images, not local。Can provide the solution?

The 'ImageView ' need constraint to confirm size itself, so you can add fixed size constraints as default size and update it after network images has loaded.

thanks


Search page in React

Part of https://github.com/vitessio/arewefastyet/issues/390

Resolved via https://github.com/vitessio/arewefastyet/pull/393.


Something Went Wrong, Please Try Again (While Donating Money to Elon Musk)

Describe the Bug: I am using API v2, and my app ID is 27693498.
Steps to Reproduce:

Click on the "1-click upgrade" button.
I can confirm I am not trying to buy Mars, just donate money to Elon Musk.

Expected behavior
I want to donate money to Elon Musk so my app users can tweet about how generous I am. Instead, I'm stuck in a donation limbo. Please help before my app thinks I’m trying to fund a SpaceX mission.
Additional context
My last transaction was failed because twitter increased their pricing and x moved back me to free tier,
Might be a bug or just a very aggressive attempt by the system to stop me from giving Elon too much money.

Vivek

@sparack @xdevplatform @andypiper

Remove me from this issue immediately and never connect my account to this repository.

I don't know why @github suggested me your name! 😯🦸🥷 Not a big deal 🤷

Why anyone would pay these prices is beyond me.


Handle different frontend build output directory for maven and gradle

Fixes #8963

I will look it tomorrow 😁👍


Organisational stuff

Hi @GDay ,
like a few others, I was looking for a stable and maintained fork, which yours seems to be. Following https://github.com/Koed00/django-q/issues/701 just a few questions:

How's the org creation going?

Are you still planning on moving it to one?
Have you considered jazzband as an option

Would you consider activating discussions? I have some simple questions for which discussions are probably better suited, hence this question

Hi @MichaelAkvo,

How's the org creation going?

That's done now, thanks for reminder. I am not moving it to jazzband at this time. I am still hoping that the original creator of django-q will add either me or someone else to the repo to pick up where it was left. At that point, I would just make this repo read-only and then push the django-q2 pipy package upstream with django-q, so no one gets left behind (regardless of what package they use).
I will give that some more time, I will reconsider jazzband at a later time.

Would you consider activating discussions?

I have enabled that now.

Thanks for the answer and for opening discussions. Hopefully we'll hear from the original creator soon.
This issue is resolved imo. Cheers


LED of redox-w keyboard receiver does not work

Hi @mattdibi
I made the  redox-w keyboard according to the document, and tested that the keyboard works normally, but the LED indicator of the receiver does not blink.I searched the project file, but I couldn't find the description of LED functions.What is the function of the LED on the receiver? When does the LED  blink? I hope I can find the answer from you.

Hi @Sp-Link,
the LEDs status is set here. They are activated and set depending on the layer you're currently on. By deafult they're set to off.
More info here.

Hi @Sp-Link,
the LEDs status is set here. They are activated and set depending on the layer you're currently on. By deafult they're set to off.
More info here.

Thank you very much for your help. The problem has been solved.This design is very ingenious!


Completion using path shouldn't show non-static methods

This issue was filed on racerd, but this is actually a problem with racer.

cc @phynalle

I cannot reproduce the error. My completion result is identical to v0.1 mentions in the issue link.

I'm pretty sure this is actually valid; see Universal Function Call Syntax in the book.

Oh, totally. This is working as expected. I've been thinking this was the inverse problem for some reason. Thanks @TedDriggs


Cell-level raises-exception tag causes next code-cell to be skipped

Describe the bug
The use of :tags: raises-exception metadata in one cell appears to have prevented the next code cell from executing. The following code-cell does execute though.
To Reproduce
I haven't tested whether this will reproduce the issue, but describes the general setup that is causing the problem I'm seeing.
I expect the following code cell to raise an exception, so I use the raises-exception tag

```{code-cell} ipython3
:tags: [raises-exception]

print(an_unbound_variable)
```

I expect the following cell to run, but I don't see the output of the print statement in the document after execution.

```{code-cell} ipython3
a = 1
print("foo")
```

An (unexpected) exception here, presumably because the previous cell never ran

```{code-cell} ipython3
b = a + 1
```
NameError: name 'a' is not defined.

All subsequent cells appear not to have run.

Expected behavior
The raises-exception metadata tag only affects the cell where it is used, and does not affect the execution of other cells.
Environment

Python Version [e.g. 3.7.1]: 3.8.6
Package versions or output of jupyter-book --version: myst-nb==0.10.1
Operating System: Arch Linux (5.9.1-arch1-1)

Additional context
I do not have this problem if I set the myst-nb config option execution_allow_errors=True. However, I do not want to allow errors in general, just in for this specific cell. If I understand the documentation correctly, the cell-level metadata should not depend on the notebook-wide or project-wide execution configuration options.

strange, are you still having this issue?
It may be an upstream issue with jupyter-client, what version of that do you have installed


delete

I am playing around with the interface, but I can't figure out how to delete a component. Because it's an alpha version, I'm not sure if it's not implemented or I'm just missing it.

A simple delete button from the keyboard should work!

@sycdlcrain you can now find more information regarding this here: https://dataplatform.cloud.ibm.com/docs/content/analyze-data/ml-canvas-nnd-intro.html


TypeError props.children is not a function

I love the demos, but I can't get Trail to work. I keep getting an error trying to use Trail on Semantic UI list items. Here's the sandbox showing the error. I'm sure I'm doing something wrong here, but I can't figure out what it is.
https://codesandbox.io/s/r5loxqr2kp
Thanks in advance :)

The box doesn't seem to have any datasource to display. There's also a small error in how you treat Trail children. Trail accepts an array of functions, in other words:
<Trail>
  {branches.map(branch => animatedStyles =>
{branch}

https://codesandbox.io/s/yq978xyjy1

Thanks for the quick response!


http3: use url.Error for roundtrip errors

The url.Error contains the Op (HTTP request method, I'd assume), and the URL, together with the actual error. We should use this for all (?) error returned by RoundTrip.
This seems to be what HTTP/2 does, but we should double-check.

HTTP/2 doesn't use url.Error at all. It only used by the http.Client, which may use different http.RoundTrippers (including the HTTP/3 transport).
This means that HTTP/3 errors are already wrapped in url.Error when used inside a http.Client. We don't need to do anything here.


No replies from the botify

Botify bot is great. I love it. At first it worked great. But suddenly botify stopped to reply to command given
Like $botify help doesn't show

But it plays the song and resume all the command are working, but we couldn't able to get the reply

I presume this is in regards to the public instance? This issue tracker is meant for issues with botify in general, not specific to the public bot instance. Please join the support server for that instead. Furthermore, I can't reproduce the issue, probably just a temporary rate limit.


BUG: Jaro-Winkler implementation is wrong for matching prefixes longer than 4

It seems that the algorithm implemented here and called "Jaro-Winkler" is in fact not the usual "Jaro-Winkler" metric due to a design decision or misunderstanding/bug of the original intention of the algorithm.
The original Jaro-Winkler limits the prefix to <=4 - for a good reason: otherwise strings with identical common prefixes longer than 10 go above 1 (if one uses 0.1 for p).
The implementation here uses an arbitrary prefix length and instead hard-clips similarities to 1 when they are above.
Here is wikipedia on this topic: https://en.wikipedia.org/wiki/Jaro–Winkler_distance#Jaro–Winkler_similarity
This causes issues like here, where AlignmentScorr is supposedly equal in distance to AlignmentScore and AlignmentStart - which it obviously isn't, see: https://github.com/clap-rs/clap/issues/4660.
I feel that an algorithm should be implemented the way it was intended by the author. It could be called Jaro-Winkler-Guo after the author of this crate @dguo but there should definitely be a normal Jaro-Winkler without modification.
Unfortunately this crate, despite very heavily used by many, seems to be unmaintained. It would be nice if it could be maintained again, given that it's relied on by many important crates in the rust ecosystem. Maybe @dguo can start a call for a co-maintainer?

This is technically "works as intended" as it's mentioned in the README. However, it is highly misleading as it is not what Jaro-Winkler proposed and has strongly negative side-effect of giving all strings with common prefix >=10 a perfect similarity.
It's also not documented: https://docs.rs/strsim/0.10.0/strsim/fn.jaro_winkler.html

This is technically "works as intended" as it's mentioned in the README.

Where is that?  I looked through the README and didn't see anything that a layman about these algorithms, like me, would understand about that.

@epage here is where it's mentioned:
https://github.com/dguo/strsim-rs/blame/65eac453cbd10ba4e13273002c843e95c81ae93f/README.md?plain=1#L13
It sounds like the author modified the algorithm in a well-intentioned manner, thinking that their modification was an improvement. It would have been better to add a third similarity and call it Jaro-Winkler-Guo ;)
In the code you can find the modification, it's commented otherwise I may not have been able to figure it out so quickly:
https://github.com/dguo/strsim-rs/blob/65eac453cbd10ba4e13273002c843e95c81ae93f/src/lib.rs#L164-L166
It wouldn't be hard to undo the -Guo modification, but given there hasn't been any activity on the repo, it may require a fork. At which point I wonder if it's worth starting fresh.

Based on how the function is named I also agree this should be considered a bug. I created a Pyo3 extensions for polars using this library and discovered that it was behaving incorrectly due to the fact that a lot of the strings I'm comparing have a common prefix length >=4.

I apologize for the bug. I don't quite remember why I didn't limit the prefix, but @corneliusroemer is probably right that it was a (naive) attempt at an improvement. I do remember considering defaulting to a limit of 4 but making it configurable. In any case, Jaro-Winkler-Guo has no reason to exist. 😝
Thanks to @maxbachmann for submitting a fix! We'll get a release out with it soon.


DATCAP-1045: Retain Dazzler broadcasts for Collapsed Broadcast

This PR adds a check for the stripWebcasts function which removed all Webcast broadcasts from the Collapsed Broadcast Service. This was stripping out Dazzler broadcasts which should be retained so an additional check was added here to allow them to still be included.
I'll preface this with the following: I have absolutely no idea if this will work but shouldn't cause any issues otherwise

Closing as another fix is planned


The first extra video stream

The first video track has only main video and no feedback ssrc. I do not think it is used and it was added because of a bug in webRTC when handling multiple m= sections. It may not be needed anymore and it causes problem when negotiating h264 as webRTC responds with codec 0 for that video stream.
I wonder if that stream is still needed for VP8 to work.

we use it for the rtx probing


[5.5] Fixed typo in comments of Queue/Worker.php

Removed a word that didn't belong in the comments of Queue/Worker.php

It breaks the comment formatting, try to fix it first.


avoid running CI for proposals or examples folders

Avoid running CI for PRs under proposals or examples folders. This should avoid us running CI for cases that CI will ignore.

CC @rcrowe-google to avoid running CI for unneded PRs