JPEG XL

Hello,
do you plan to add support for JPEG XL format into imgproxy?
It is supported by libvips. It's in testing in Chromium and in Firefox nightly.

Update: JXL is currently not supported any more by Chrome and Firefox, but Safari will ship with support for it. The situation might change with Firefox/Chrome.

another vote, especially since apple announced system wide support for iOS17 / macOS14.

Safari 17 with JPEG XL support has been shipped
https://webkit.org/blog/14445/webkit-features-in-safari-17-0/

Another bump. I would really love to move to JPEG XL for our iOS app. Keep in mind that JPEG XL supports lossless transcoding from JPEG to JPEG XL.

Hey everyone!
We're working on JPEG XL support: https://github.com/libvips/libvips/pull/3712.
Be aware though that the JPEG XL encoder needs a lot of memory: https://twitter.com/darth_sim/status/1711794478778814844

@DarthSim I see that libvips PR has been merged; does that mean imgproxy should support JPEG XL soon?

@chendo The PR was merged to vips 8.16 which is not released yet.
And honestly, I don't quite understand why you are still hyped up about JXL. It is still supported only by Safari and it still requires an enormous amount of RAM for encoding. So nowadays it looks like a good way to shoot yourself in the foot for a single browser's sake.

@chendo The PR was merged to vips 8.16 which is not released yet.
And honestly, I don't quite understand why you are still hyped up about JXL. It is still supported only by Safari and it still requires an enormous amount of RAM for encoding. So nowadays it looks like a good way to shoot yourself in the foot for a single browser's sake.

regarding the memory comment — is that still true? see https://github.com/libjxl/libjxl/issues/1106#issuecomment-2060730300

@sbaildon They indeed improved things, and the compression of a 7360x4912 image now consumes 664M instead of 1.8G, but this is still huge. On the other hand, compression to AVIF consumes more than that.

@DarthSim ahh, I see. In our case, we're mostly wanting JXL decoding support so we can have our source files be smaller and thus take less time for imgproxy to fetch. We're not looking to implement JXL for end users just yet, but our analytics indicate about 50% of our users are on Safari versions that support JXL.

@chendo Nah, the case is that libvips with proper JXL support has not been released yet. The changes from my PR should go to v8.16 but it seems like it's delayed. I sometimes do patch vips right in the Docker image but I don't think this is a good idea in this exact case.

Libvips 8.16 released yesterday with animation and much better metadata support (ICC profiles, EXIF, ect). The only thing missing is HDR although that may not be required at this time anyway

JPEG XL support was added to the latest build

Just gave it a try and works so far great in iOS. Thanks! ❤

This might not be the right place to say it, but I'd recommend setting the default effort to 5 instead 4, which enables the Variable blocks of VarDCT. 4 and below use fixed 8x8 like the old JPEG. If possible, you could also add a separate effort setting for lossless, to maintain speed for both use cases.

@jonnyawsom3 Effort 5 dramatically slows down encoding compared to effort 4. We did some tests with SSIMULACRA2 and found that effort 4 still shows surprisingly good results. It's not as good as effort 5+, but still good.
When we added AVIF support, the most common issue here on GH was "imgproxy saves to AVIF too slowly" or "imgproxy times out when I use AVIF." I had to reply to every single one of them that this is because AVIF is slow, we can do nothing about this, but here's the config that you can use to change the speed, etc. In the end, I gave up and increased the default AVIF speed.
Considering that most people will adopt JPEG XL just because it's a new "best in the world" image format and because Apple added its support to Safari, the situation will be exactly the same if not worse.
So, we tried to find a sweet spot between good encoding speed and good results. Our defaults should work well for those who just want JPEG XL. For the rest, the IMGPROXY_JXL_EFFORT config is at their service.


Update Github Migration Tests (#8938)

Update all Github migration tests to use a new repo created just for these tests that won't accidentally be modified by regular users interacting with issues.
Fixes #8895
back port to release/v1.10

Thanks for this @mrsdizzie


Mistakes in training

Hello,First of all, thank you very much for this research work! However, when I try to train with the 3DMatch dataset you provided, I get an error message：
(torch) yy@yy:~/se3-equi-graph-registration/src$ python train_eval_egnn.py --mode train --base_dir /media/yy/Elements/registration_data/3DMatch_FCGF_Feature --num_epochs 50 --batch_size 2
Epoch 1/50
Traceback (most recent call last):
File "train_eval_egnn.py", line 1201, in
train_model(cross_attention_model, train_loader, val_loader, num_epochs=num_epochs,
File "train_eval_egnn.py", line 969, in train_model
train_loss = train_one_epoch(model, train_loader, optimizer, device, epoch, writer, use_pointnet, log_interval, beta)
File "train_eval_egnn.py", line 696, in train_one_epoch
graph_idx_0 = knn_graph(xyz_0, k=k, loop=False)
File "/home/yy/anaconda3/envs/torch/lib/python3.8/site-packages/torch_cluster/knn.py", line 132, in knn_graph
edge_index = knn(x, x, k if loop else k + 1, batch, batch, cosine,
File "/home/yy/anaconda3/envs/torch/lib/python3.8/site-packages/torch_cluster/knn.py", line 81, in knn
return torch.ops.torch_cluster.knn(x, y, ptr_x, ptr_y, k, cosine,
File "/home/yy/.local/lib/python3.8/site-packages/torch/ops.py", line 854, in call
return self._op(*args, **(kwargs or {}))
RuntimeError: x.dim() == 2 INTERNAL ASSERT FAILED at "csrc/cuda/knn_cuda.cu":93, please report a bug to PyTorch. Input mismatch
My pytorch version is 2.3.0 and CUDA is 12.1, is the error reason that my version is different from yours?
Thank you very much.

Are u using batch size bigger than one? The current knn graph cluster has some bug to use batch size > 1, so it is not supporting batch size bigger than one. If not, try Cosine Distance Option: If cosine=True or cosine=False is not explicitly provided, try specifying it to avoid potential issues:
"
edge_index = knn(x, x, k, batch=batch, cosine=False)
"
Or try update PyTorch Geometric and Dependencies: This could also be a version compatibility issue. Make sure your torch, torch-cluster, and torch-geometric libraries are up-to-date. You can update them using:
"
pip install --upgrade torch torch-cluster torch-geometric
"
We will commit the batch size bug fix version by the end of this month, as we found the larger batch can help accumulated gradient for a more stable training. The recommended pytorch geometry version is pyg (Pytorch-Geometric) 2.4.0, knn api may be adapted or remapped in new version.

Hello, I have a question to bother you again. The following warning occurred during my training.  Where should I modify this ratio？Thank you very much.
Warning: Not enough sample points for the fixed number, sampling with repetitions.
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!
Not enough positive or negative points to satisfy the 0.60-0.40 ratio. so repeating samplinf will be used!

These warnings are only used for helping debugging, it is normal, you can disable it in the dataloader file, as the scan pair sometimes are not enough to finad enough point pairs, thus repeating sampling of point pairs will be used.


replace "com.google.android.gms:play-services-ads" to "com.google.firebase:firebase-ads:10.2.6"

my cordova app using this "cordova-admob-pro"  plugin.
this plugin is using "com.google.android.gms:play-services-ads".
affter add another  plugin "cordova-plugin-fcm".
"cordova-plugin-fcm" is  include "framework com.google.firebase".
build error
`* What went wrong:
Execution failed for task ':processDebugGoogleServices'.

Please fix the version conflict either by updating the version of the google-services plugin (information about the latest version is available at https://bintray.com/android/android-tools/com.google.gms.google-services/) or updating the version of com.google.android.gms to 9.0.0.`

my situation is  replace "com.google.android.gms:play-services-ads" to "com.google.firebase:firebase-ads:10.2.6"  on the plugins.xml define.
Is there any good method for this situation?

instead of cordova-plugin-admobpro, use this one: cordova-plugin-admobpro-firebase

@floatinghotpot
Thank you.


Fix broken link

Signed-off-by: Usha Mandya<EMAIL_ADDRESS>Fixes https://github.com/docker/docker.github.io/issues/11916
Updated info on opting out of usage statistics

@StefanScherer Could you PTAL?


Linux kernel dropping packets from the UPF to gNB

Hi @herlesupreeth
First of all thanks for the repository, i've used it for a while and has helped me a lot!
After trying out your eupf branch i've notice that i couldnt ping anything from the UEs, both with simulated srsUE and real smartphones. After going through the traces i concluded that the eUPF implementation was doing what it was supposed to do and that the packets to the gNB were being dropped by the linux kernel.
Following this forum, it suggests that Linux kernel drops incoming packets which source address match the local interface address. And to solve this issue you can enable accept_local in ipv4 configurations.
So i used your eupf-routes container to reflect this change on the host machine, its working for me without any other modifications. Let me know if you've seen this issue on your side and if so, if this fixes it for you!

Awesome!! I could figure out this thats why I ended up keeping eupf integration a separate branch ..thanks a lot for your contribution


Cleanup color variables

We have too many different color variables in different packages. Clean them up.

Closing due to conversion to sass and denali


Create fr-fr.js

Woops sorry I forgot to espace a few apostrophes, mind them before merging my request.


Allow oci_push to use plain HTTP for a specific host.

Right now, we can only push to HTTPS, which is creating some problems for a project I'm working on where we'd really like to push to a local registry container. We can in theory add a self-signed cert to that registry container, but the hoops you have to jump through to make that work on Docker Desktop for Mac are considerable and definitely require manual intervention, so it'd be really, really nice if we could instead optionally push to a http host.
This adds a new attribute to oci_push, plain_http_host, which is a string value for a Docker registry host, including port if needed. This is an optional attribute, defaulting to the empty string. It'll be passed to the ocitool call as --plain-http-host="<value>", and onwards from there to ResolverWithHeaders and on to credhelper.RegistryHostsFromDockerConfig. If it's non-empty, the returned docker.RegistryHosts function will check if the host parameter matches plainHTTPHost, and if so, it'll change the Scheme for the host from https to `http.

Soooo this doesn't work in a bunch of ways, as it turns out. The next problem is that dockerRegPusher.Mount has a hardcoded https - I did manage to work around that by creating a new resolver in CopyContent to use for that Mount call with the original ref for the parent, rather than a relative one, but that's probably a bad idea in a few ways.
And even once that part "worked", I was still unable to actually push - I think that's because the parent image and its layers aren't in the registry I'm trying to push to. I think that oci_push doesn't actually work at all if the "parent" image (based on the org.opencontainers.image.base.name annotation on parent layers) isn't in the registry you're pushing to, regardless of http vs https etc. Maybe. In any case, I'm giving up on this. =)


Link doesn't point to the correct place

I think the link here called install Singularity

Should point here, (instead of linking the same section it's already in).
That would probably help with many of the users complaints we've seen recently that the install docs are hard to understand.

I think it's supposed to be pointing at the remote page... but because there's a:
------------
Installation
------------

header, it's instead linking to that internal link it makes for the header, instead of the page named installation ...
@devprincess ... For this, what if we changed the Header to be named:  Quick Installation Steps instead of Installation? I think that link should point to the correct installation page then.

That change would also be useful because it would be a hint to the user that there is more complete installation documentation elsewhere.

Got it, checking this right now thank you :+1: @GodloveD @jmstover

Is this issue open? I can work on it.

Hey @arshadkazmi42 this issue has been fixed, thanks anyway!


Add RootIssuer appsetting to prevent having to dive into the db...

... to add the root issuer when working with AzureAd
Also added some consts and used them throughout (this might need some more attention but probably better in a separate pr)

don't merge yet please

All is well... ready to merge, together with the other one on blazor.


navigator.usb.requestDevice immediately returns "DOMException: No device selected." without prompting for device selection in custom editor

VSCode Version: 1.53.0
OS Version: Window 10 Version 10.0.19041 Build 19041

Steps to Reproduce:

Create an extension that exposes a custom editor
In the custom editor setup an event listener for some user action (button click, keypress, etc) and in response request access to a USB device attached to your system

document.addEventListener('keydown', event => {
    const filters = [{vendorId: 0x046D, productId: 0xC52B}] // Swap Ids to those of a USB device attached to your system
    navigator.usb.requestDevice({filters}).then(dev => console.log(dev))
}

Initiate the user action and observe the error "DOMException: No device selected." reported in console without the user being prompted to select a USB device.

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

Relates to electron/electron#14545
You can use the usb package in your main extension code, and communicate with your custom editors via messages

Relates to electron/electron#14545
You can use the usb package in your main extension code, and communicate with your custom editors via messages

Fantastic, thank you. Apologies for not seeing the existing issue in my search 🙏

Fantastic, thank you. Apologies for not seeing the existing issue in my search 🙏


Release 2.1.0

[x] Create a release ticket in github. Copy this checklist.
[x] Make a release branch

VERSION="..."
VERSIONALPHA="$VERSION-alpha.0"
ISSUE="..."

git checkout -b release/$VERSION

[x] Update package.json version to $VERSIONALPHA and commit

git commit -m "release: Update package version to $VERSIONALPHA #$ISSUE"

[x] Tag with the alpha version

git tag $VERSIONALPHA

[x] Publish test npm version $VERSIONALPHA

npm publish

[x] Check package internals - contains all required files (lib/, README.md, ./djv.js, package.json, ...), no extra files
[x] Check with updated json-schema test suite - check generated report
[ ] Check speed metrics
[ ] Update package.json version to $VERSION and commit

git commit -m "release: Update package version to $VERSION #$ISSUE"

[ ] Create a changelog

npm run changelog

[ ] Update Readme
[ ] Tag $VERSION

git tag $VERSION

[ ] Merge release branch to master (via github)
[ ] Publish version $VERSION

npm publish

[ ] Send update to gitter

Validator
Relative speed
Number of test runs per second

djv
100%
48052 (± 2.88%)

ajv
98%
47068 (± 0.74%)

json-schema-validator-generator
72.1%
34623 (± 3.8%)

jsen
66.8%
32110 (± 4.38%)

is-my-json-valid
50.1%
24057 (± 4.16%)

themis
15.1%
7244 (± 11.03%)

z-schema
6.1%
2949 (± 2.13%)

jsck
3.8%
1835 (± 1.9%)

skeemas
1.2%
563 (± 1.71%)

json-schema-library
0.7%
338 (± 1.02%)

tv4
0.2%
90 (± 2.63%)

Validator
Relative speed
Number of test runs per second

djv
100%
20383 (± 2.16%)

ajv
78.3%
15961 (± 5.5%)

json-schema-validator-generator
67.2%
13698 (± 4.37%)

is-my-json-valid
63.2%
12874 (± 3.53%)

jsen
63.1%
12863 (± 3.58%)

themis
19%
3868 (± 6.17%)

jsck
6.8%
1376 (± 3.56%)

z-schema
6.6%
1341 (± 4.64%)

skeemas
1%
196 (± 2.49%)

json-schema-library
0.7%
149 (± 2.04%)

jsonschema
0.7%
138 (± 3.82%)

tv4
0.4%
82 (± 2.03%)


update docs with new status output

#1242  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @BelSasha and the rest of your teammates on  Graphite


On initial load of the OS Hub homepage the compact map and sidebar are small before resizing

This results in

A redraw with a lot of page motion once the content is ready
Potential map rendering issues

This was most easily repeatable in an incognito window. I was able to repeat it in both Firefox and Chrome

@obrienad @jwalgran, can you provide more details about what part of this is repeatable? or what conditions are required for repeatability? I am not able to see this behavior in chrome.
also a point estimate would be helpful in deciding how much of a priority this is

We will be having a planning meeting shortly so we can provide more detail and an estimate within the hour.

It is a 1 (point, but also hour). Ability to reproduce it might be affected by data being loaded or the map is changing - those things might make it easier to see the problem.


fix #13

The test files in this pull request are courtesy of @DavidDudson and his previous pull request in #22.

This is now established in Git history via https://github.com/scalameta/paradise/pull/32.

Awesome! I thought the whole mod removal logic was a bit strange/had changed. I'd attributed to the mTree commits but presumed filter mods was still necessary. That's probably why the only time it was actually removing annotation was erroneous (eg. When duplicate annotations were around) I'm glad you sorted it :)


Shorten FCP length

Shortens the current FCP length for all triage categories. The current length is quite excessive, and just delays updates.

t: new

Releases: 4 -> 2 days

t: refactor

Snapshots: 4 -> 3 days
Releases: 8 -> 6 days

t: documentation

Snapshots: 3 -> 2 days
Releases: 7 -> 4 days

t: toolchain

Releases: 3 -> 2 days

Will merge before the Quilt Meeting on 10/8


Rename the function len to bit_len for Bitmap

Signed-off-by: remzi<EMAIL_ADDRESS>Which issue does this PR close?

Closes #1233 .
Rationale for this change

The name bit_len is more clear because the function return the length of bits (not bytes).
What changes are included in this PR?

Are there any user-facing changes?

The function name is changed.

Codecov Report

Merging #1242 (cbdf4c3) into master (731e132) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1242      +/-   ##
==========================================
- Coverage   82.96%   82.95%   -0.01%
==========================================
  Files         178      178
  Lines       51522    51522
==========================================
- Hits        42743    42742       -1
- Misses       8779     8780       +1

Impacted Files
Coverage Δ

arrow/src/bitmap.rs
88.13% <100.00%> (ø)

...rquet/src/arrow/record_reader/definition_levels.rs
86.25% <100.00%> (ø)

arrow/src/array/transform/mod.rs
84.64% <0.00%> (-0.13%)
:arrow_down:

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 731e132...cbdf4c3. Read the comment docs.

Thanks @HaoYang670


Doesn't work if not named Index.html

I am set on this theme, but it causes issues when added to an existing site. If the index.html is named anything other than index.html, for example blog.html it would display a 404 error where the posts should be. I nailed this down after fiddling with directories and the config file, just renamed it to index.html and it worked. I do not have the Jekyll knowledge to track down and fix this issue that's why I'm submitting this here.

Thanks for reporting this, @Vitallity. I'll do a little digging to see what's going on.

Looks like this is an issue with Jekyll itself rather than Pixyll.


Bad documentation

Your readme.md shows the package name for a layou as:
<top.defaults.view.ColorPickerView />
It really is:
<top.defaults.colorpicker.ColorPickerView />
You should fix that.

My fault, fixed :)


Clickhouse client working with bun js

Bun js is fasted speed js runtime runner
https://bun.sh/
But client not working. Difference with node in this file
client/dist/connection/node_http_connection.js:19
May be difference in http client
Please fix this in next versions. Or tell please how I may fix this myself

I am not familiar with this tool. However, I see fetch usage there. Does it work with @clickhouse/client-web?

I am not familiar with this tool. However, I see fetch usage there. Does it work with @clickhouse/client-web?

With web version works fine! Thanks!

Would love to see this issue being reopened. It's working with the client-web library, but that means we lose the ability to work with the connection pool and keepAlive feature which is only available in the client library.
Would be awesome if the client library would also be compatible with Bun, thank you! :)

@dsumer, I suspect the Node.js version will not work with Bun unless we rewrite it entirely to the fetch API. I think the right approach is to make keep-alive configurable in the web version instead - it will be there after https://github.com/ClickHouse/clickhouse-js/pull/220.

@dsumer, as 0.2.8 is out, it is now possible to modify Keep-Alive setting for the web version (and it is enabled by default). See the release notes.


Don't treat "Fixing" as a commit prefix

Some examples from changelogs in the wild:
2017-04-26 1ff6f3f1 ing doArchive (John Milmine)
2017-04-07 41eddfcc ing cms page history controller to use new page id param (Tim Kung)
2017-05-02 2187c160b ing pagination api doc typo (3Dgoo)

This happens when the commit messages start with Fixing ....
Pretty low priority, but ideally the regex would match the words it looks for with a space or : afterwards, but not another letter.
The same thing would happen with Fixes ...
2017-02-09 6e2797ffc es for using dblib PDO driver. (Andrew O'Neil)
2016-12-08 dc530c0 es #529 (torleif)

Haha, well. Maybe only remove the prefix IFF the whole prefix is a whole word? It's a problem for prefixes other than just "fixing"

fixing bug => FIX Fixing bug
fix bug => FIX bug

PR at #110
Now we only strip commit prefixes if they match the case, i.e.FIX Foo should be stripped but Fix foo would not. I've assumed that people who write FIX foo know how cow works, but people who write Fix foo don't so in that case Fix would be part of their description.
Also adds spaces to the end of each regex rule, so Fixed bar wouldn't end up as ed bar in the changelog, even though the case check wouldn't match it anyway


Install Klotho with scoop package manager

I noticed that the README doesn't have any directions to install Klotho on Windows.
I also noticed that there is a Windows binary for Klotho, in the Releases section on GitHub.
Most developer tools on Windows are packaged up for distribution with the Scoop package manager.
Could you guys add a Scoop package manifest file, to install Klotho via Scoop CLI?
It's basically like MacOS Homebrew, but for Windows.
https://github.com/ScoopInstaller/Scoop

Hey @pcgeek86 - that's likely an oversight on our end, we'll likely remove the .exe's for now - while we made sure klotho compiles on Windows, it's not a well tested route and for the time being suggest folks use WSL2 and the linux binaries for maximum compatibility. We are planning on support vanilla Windows down the line, but it's not on the hi-pri list at the moment.


Custom validation

In addition to syntax validation, the extension could:

[x] #41
[ ] #42
[x] Check for the release entry to be defined. 6ce8db6aa6a55c726257e6fde38ca207a6e93abf

Custom validation rules #35 and profiles #46 are now supported. The complete list of custom validation rules will be tracked in #45


Ctrl + Tab not working

Steps to reproduce
Hit and hold Ctrl then hit Tab.
Ctrl is not on when Tab down event is received.
2023-07-01 17:09:50.354 14253-14253 BrowserActivity         net....fulguris.full.download.debug  D  dispatchKeyEvent KeyEvent { action=ACTION_DOWN, keyCode=KEYCODE_CTRL_LEFT, scanCode=29, metaState=META_CTRL_ON|META_CTRL_LEFT_ON|META_NUM_LOCK_ON, flags=0x0, repeatCount=109, eventTime=30313692000000, downTime=30313692000000, deviceId=-1, source=0x101, displayId=-1 }
2023-07-01 17:09:50.386 14253-14253 BrowserActivity         net....fulguris.full.download.debug  D  dispatchKeyEvent KeyEvent { action=ACTION_DOWN, keyCode=KEYCODE_CTRL_LEFT, scanCode=29, metaState=META_CTRL_ON|META_CTRL_LEFT_ON|META_NUM_LOCK_ON, flags=0x0, repeatCount=110, eventTime=30313723000000, downTime=30313723000000, deviceId=-1, source=0x101, displayId=-1 }
2023-07-01 17:09:50.398 14253-14253 BrowserActivity         net....fulguris.full.download.debug  D  dispatchKeyEvent KeyEvent { action=ACTION_DOWN, keyCode=KEYCODE_TAB, scanCode=15, metaState=META_META_ON|META_META_LEFT_ON|META_NUM_LOCK_ON, flags=0x0, repeatCount=0, eventTime=30313735000000, downTime=30313735000000, deviceId=-1, source=0x101, displayId=-1 }
2023-07-01 17:09:50.420 14253-14433 EGL_emulation           net....fulguris.full.download.debug  D  app_time_stats: avg=56089.21ms min=56089.21ms max=56089.21ms count=1
2023-07-01 17:09:50.505 14253-14253 BrowserActivity         net....fulguris.full.download.debug  D  dispatchKeyEvent KeyEvent { action=ACTION_UP, keyCode=KEYCODE_TAB, scanCode=15, metaState=META_META_ON|META_META_LEFT_ON|META_NUM_LOCK_ON, flags=0x0, repeatCount=0, eventTime=30313842000000, downTime=30313842000000, deviceId=-1, source=0x101, displayId=-1 }
2023-07-01 17:09:50.885 14253-14253 BrowserActivity         net....fulguris.full.download.debug  D  dispatchKeyEvent KeyEvent { action=ACTION_UP, keyCode=KEYCODE_CTRL_LEFT, scanCode=29, metaState=META_NUM_LOCK_ON, flags=0x0, repeatCount=0, eventTime=30314222000000, downTime=30314222000000, deviceId=-1, source=0x101, displayId=-1 }

✔ Expected Behavior
App should be able to handle Ctrl + Tab.
Ctrl flag should be on when Tab is received and Ctrl is still pushed.
❌ Actual Behavior
Since Ctrl flag is not set application behaves as if only Tab was hit without Ctrl.
Other Software
No response
Please specify the version of Windows Subsystem for Android
2305.40000.6.0

The issue is only with Ctrl + Tab. Ctrl + F works as expected for instance.

Converting CTRL+TAB to META+TAB is necessary for accessibility compliance.  META+TAB is Android's keyboard shortcut to navigate to header menu elements (such as three-dots menus), and the ability to navigate to all interactable UI elements of an app via keyboard is an accessibility requirement.  Unfortunately, Windows keyboards don't have a META key, and we can't use WINDOWS+TAB because Windows always consumes the WINDOWS key rather than passing it to apps.  CTRL+TAB was the best alternative to map to META+TAB since we needed to pick a modifier key and CTRL+TAB isn't typically used in Android.  The CTRL modifier key will work for all keys other than TAB.  Is there a reason you need CTRL+TAB?

I use Ctrl+Tab to cycle between browser tabs which is a common use case. It was easy enough to workaround this issue by keeping track of the Ctrls up and down events. Still I don't think such tricks are a good idea. Many apps don't even use the standard header menu and so won't handle that shortcut. Even Chrome does not comply there. That is a really obscure feature. It's not even listed in the shortcuts help dialog META + ?. I would not do tricks like this if I were you. Users and developers know they don't have meta on Windows so they would not expect it to work. However they would expect Ctrl to work. If devs want to enforce such overrides on WSA they can easily implement that at the application level.
There must be a better way to emulate meta. What about using the context menu button? That guy is not very much used on Android. Whatever solution you go for it would be nice if it could be deactivate from the WSA settings.

We discussed this and decided to revert the CTRL+TAB to META+TAB conversion since META+TAB is really obscure.  I'm not sure where our accessibility team discovered it because I couldn't find any Google documentation about it.  We'll ask our accessibility team to come up with an alternative solution, such as working with Google to make TAB reach the header menus without needing a modifier key.  The revert will be in our 2307 build.

Thanks for following up on that. Great outcome.


pull request for e2e test at 12/27/2016 00:26:12

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


usecols keyword argument of pd.read_csv says it expects list[str] but the documentation says otherwise

Describe the bug
usecols keyword argument of pd.read_csv says it expects list[str] but the documentation says otherwise:

Documentation (https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.read_csv.html):

usecols list-like or callable, optional Return a subset of the columns. If list-like, all elements must either be positional (i.e. integer indices into the document columns) or strings that correspond to column names provided either by the user in names or inferred from the document header row(s). If names are given, the document header row(s) are not taken into account. For example, a valid list-like usecols parameter would be [0, 1, 2] or ['foo', 'bar', 'baz']. Element order is ignored, so usecols=[0, 1] is the same as [1, 0]. To instantiate a DataFrame from data with element order preserved use pd.read_csv(data, usecols=['foo', 'bar'])[['foo', 'bar']] for columns in ['foo', 'bar'] order or pd.read_csv(data, usecols=['foo', 'bar'])[['bar', 'foo']] for ['bar', 'foo'] order.
If callable, the callable function will be evaluated against the column names, returning names where the callable function evaluates to True. An example of a valid callable argument would be lambda x: x.upper() in ['AAA', 'BBB', 'DDD']. Using this parameter results in much faster parsing time and lower memory usage.

To Reproduce

Provide a minimal runnable pandas example that is not properly checked by the stubs.
import pandas as pd; path_to_csv = "mycsv.csv"; df_db = pd.read_csv(path_to_csv, usecols=[0])
Indicate which type checker you are using (mypy or  pyright). PyCharm default type-checker, I haven't checked mypy.
Show the error message received from that type checker while checking your example.

Please complete the following information:

OS: Windows
OS Version 11
python version 3.11.1
version of type checker mypy 1.1.1
version of installed pandas-stubs pandas-stubs <IP_ADDRESS>321

Additional context
Realted to this SO post.

Need to change https://github.com/pandas-dev/pandas-stubs/blob/2e3bbe81dbc269504e84b035621938dc193406bb/pandas-stubs/io/parsers/readers.pyi#L45-L52
to be usecols: list[HashableT] | Callable[[Hashable], bool] | None = ...  which is what is used in the pandas source.
Also change other places in that file that have the usecols argument.
PR with tests welcome.

When will the changes be made public? meaning I can do pip install -U pandas-stubs?

When will the changes be made public? meaning I can do pip install -U pandas-stubs?

Unsure at the moment.  I would like the next release to support the 2.0 features, but there is work described in #624 that needs to get done.
If you can't wait for that work to get done, I believe that you could just clone the repo, switch to the main branch, set up the dev environment, do a poetry build, then you will get a wheel file in dist and you can then install from the wheel via pip install -U dist/name_of_wheel_file.whl .


Detailed job description panel

Provide a detailed version with culprits and test results.

Added culprits - needs configuration in build wall config panel

at javafx.scene.control.Labeled.setText(Labeled.java:145)
at buildwall.panel.description.FailureDetail.updateCulpritText(FailureDetail.java:148)
at buildwall.panel.description.FailureDetail.lambda$3(FailureDetail.java:110)
at buildwall.panel.description.FailureDetail$$Lambda$572/327839098.accept(Unknown Source)
at utility.observable.FunctionListChangeListenerImpl$$Lambda$537/1054625189.accept(Unknown Source)
Not on JavaFx thread - fix


Module deprecated warning when running CLI with NodeJS v23

I did not seet this warning before I upgraded NodeJS to the latest version (v23). Now when I run the CLI, I get the warning:

(node:23596) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)

The tests run fine, so it's just a warning but may be worth looking at. There is a possibility that it's just my setup, but I uninstalled and reinstalled the latest version (23.1.0) just in case.

already addressed last year: https://github.com/lupomontero/psl/issues/315 but tough-cookie seems to have fixed the issue. I will take a look

I updated tough-cookie, which should remove this warning in the next release.


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Added CSS file for formatting of the HTML.


Using Zulu builds of OpenJDK in GH actions

The AdoptOpenJDK has been discontinued Since July 2021. When using Zulu you get all the latest updated builds for all versions of OpenJDK. I've also added the up incoming LTS JDK version 17-ea. The build and test ran successfully. 🥇

Thanks for contributing!

Glad to help!


Adding validation

Hello!
I would like to enhance the project.
I have a suggestion - what if we add a validation?
We can have several default validations:

only numbers
check if valid email
...

And then we can have a custom validation in which the user can add his own regex as a parameter to function.
What do you guys think about idea?
Any suggestions/ideas are more than welcome.

This is a great idea, I'm sure most of those who implemented SkyFloatingLabelTextField are already running few validations on their code.
I think that the regex approached is also a good idea.
there are few projects for more complex validations such as credit-card or phone numbers where simple regex won't be enough but for the simpler cases it will work.

That is what I do now. I have validation code in my project.
But when you use SkyFloatingLabelTextField in many View Controllers you have to make some sort of manager that will have validation logic or you must implement validation logic in any View Controller.
So I was thinking why not put that logic in SkyFloatingLabelTextField. Then the validation will be like one liner in your code.

Yeah this is definitely something that could be worth considering. We need to keep ability to use UITextFieldDelegate directly in combintation with validations however.
Something like this with a few built in and the ability to add new ones yourself maybe?
protocol Validator {
  bool isValidForValue(value)
}


Revisit the logic for showing Posix ACL warning for files

Storage Explorer Version: 1.18.1
Bug Description
When creating new files in a HNS directory, the inherited Mask and Named User/Group permission may lead to a warning.
Steps to Reproduce

On a parent directory, set the Mask permission to be RX. Set the Group permission to be RX. There will be no warning on the parent's ACL.
Go into that directory, upload a file.
View the ACL of the uploaded file. On the file, the Mask have R permission while the Group have RX permission and we show a warning.

Additional Context
We need to figure out

If the permission inherited may have issues suggested by the warning
If not, we should hide the warning in this type of scenarios

Have reached out to someone internally about discussing this....

Had a great talk with an ACL expert on the service side! Some recommendations for us:

Move away from a warning symbol -> use an (i) instead. When these situations happen it is usually a user choice/by design behavior. Warning is therefore too strong.
Have better linking to documentation about what to do in this situation.

Let's take these recommendations in 1.21!


[Dev] Updates the makefile for shipping new RN deps

Adds the new deps - https://github.com/artsy/Specs/commit/9d5378347300bade19a4b30ec39ec91832ab3eee
#trivial

if there's any more this should probably get turned into a ruby/js script, it's a bit unmanageable

They aren't needed

Spotted this too:
https://github.com/facebook/react-native/blob/1c534719115fc4e99da9a414cff8638a6059a948/ReactCommon/yoga/yoga.podspec#L42

We should just take some time to move the specs in the RN repo into an actual Spec repo and never deal with this stuff again. I’ll do so during this sprint, because this is just a time hog.


F score over boundaries

🚀 Feature
An F score over the boundaries.
It will requires boundaries extraction on multiclass tensors.
Motivation
It is used in many Video dataset and their related challenges (Davis, Yotube VOS, etc.).
Pitch
Generally we have opencv/script/numpy implementations but it could be very useful to have a metrics directly to plug in the eval step.
Alternatives
Implementing BoundaryIOU
https://bowenc0221.github.io/boundary-iou/
Additional context
F-boundary Reference implementation:
https://github.com/davisvideochallenge/davis2017-evaluation/blob/master/davis2017/metrics.py

Hi @bhack, thanks for proposing this metric to be a part of torchmetrics.
One question: what would typical examples for such a metric look like?

Check Section 4.1 in
https://www.cv-foundation.org/openaccess/content_cvpr_2016/papers/Perazzi_A_Benchmark_Dataset_CVPR_2016_paper.pdf
You will find also a comparative analysis where the alternative BoundaryIOU was proposed at:
https://arxiv.org/pdf/2103.16562.pdf
But F is still the popular one for boundaries measurement in dedicated conference challenges/workshop.
I suppose that probably we could reuse the F1Score multiclass metric already available in the library but it need to be integrated by a boundary extraction function.
Let me know if you need any other info.

@vkosolapov Has a torchmetric for BoundaryIOU but still relaying on OpenCV for extracting the boundarties:
https://github.com/vkosolapov/anchor_free/blob/master/anchor_free/model/metric.py#L8-L74

@Borda Can  the mask to boundary util contributed somewhere in the meantime? I don't see a generic util file in the image folder.

This is a very useful metric for semantic segmentation since it correlates nicely with human perception of IoU.
Any news on whether it will be added?


Document how to change environment variables

Document how to view, change and apply environment variables on the live GPaaS environments.
We make a minor addition to local environment management to include reference to our Dotenv initialiser for use with critical variables whose absense would prevent the app starting.

2 extra commits that I fixed as CI was failing intermittently (after turning tests back on in CI presumably). Piggy backing on this PR.


Revert "ha: Clean up expected vip failed actions"

Reverts SUSE-Cloud/automation#1802

My apologies, this probably isn't necessary like I originally thought.


RequiresRoles for Modules in Nancy 2.x not present

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of Nancy
[x] I have verified if the problem exist in both DEBUG and RELEASE mode
[x] I have searched open and closed issues to ensure it has not already been reported

Description
Former RequiresClaims(...) could be used for a list of roles of a user.
In Nancy 2.x,, this function checks the claims of the Security.ClaimsPrincipal.
However, this ClaimsPrincipal also offers a check for roles directly (user.IsInRole). This property of a user should be leveraged for Nancy Modules: this.requiresRole(string role), or similar.
Steps to Reproduce

System Configuration

Nancy version: 2.0
Nancy host

[x] Nancy.Hosting.Aspnet
[ ] Nancy.Hosting.Self
[ ] Nancy.Owin ()
[ ] Other:

Other Nancy packages and versions:
Environment (Operating system, version and so on):
.NET Framework version: 4.7.2
Additional information:

Closed by #2982


Consider retry not only all read commands with DoMulti on error MOVED / ASK / TRYAGAIN / ...

Recently we experienced a redis cluster scaling, involve with cluster slot migrating, master-slave switching.
We got some errors return by DoMulti like:

MOVED {slot} {ip:port}
ASK {slot} {ip:port}

We are using rueidis v1.0.49. After checking driver source, we believe that behavior is associated with method clusterClient.doMulti
https://github.com/redis/rueidis/blob/main/cluster.go#L733-L769
We used go-redis/v8 before and switch to rueidis recently. Checked go-redis and it does do retry not only read command.
https://github.com/redis/go-redis/blob/v8/cluster.go#L771-L852
Wondering if rueidis can support these situations ?
If we add some retry with invoking DoMulti, we may can not recycle the Completed Object cause we need to Pin that for protentially retry. Also we have to do much more code to send command to NODE with ip:port the error return

Hi @wyxloading,
Thank you for raising this issue and looking into the source.
Yes, I think we should support the situation. Actually the doMulti you mentioned is optional. It is an optional optimization if all your commands go to a single connection, but unfortunately, it doesn't handle the complex MOVED and ASK situations.
So the solution I think is just to delete the doMulti and the entry of the optimization and let the rest normal path handle the retry correctly. Delete the following: https://github.com/redis/rueidis/blob/2246f91cf874bc3131eead363d0ad239a1df5903/cluster.go#L678-L683
Since you already looked into the source, would you also like to contribute?

ok, i would test that ASAP

Hi @wyxloading,
Thanks! Let me know if you have any problems. And I just noticed that _pickMulti function also needs to be updated to:
func (c *clusterClient) _pickMulti(multi []Completed) (retries *connretry, last uint16, toReplica bool) {
	last = cmds.InitSlot
	init := false

	for _, cmd := range multi {
		if cmd.Slot() == cmds.InitSlot {
			init = true
			break
		}
	}

	c.mu.RLock()
	defer c.mu.RUnlock()

	count := conncountp.Get(len(c.conns), len(c.conns))

	if !init && c.rslots != nil && c.opt.SendToReplicas != nil {
		for _, cmd := range multi {
			var p conn
			if c.opt.SendToReplicas(cmd) {
				p = c.rslots[cmd.Slot()]
			} else {
				p = c.pslots[cmd.Slot()]
			}
			if p == nil {
				return nil, 0, false
			}
			count.m[p]++
		}

		retries = connretryp.Get(len(count.m), len(count.m))
		for cc, n := range count.m {
			retries.m[cc] = retryp.Get(0, n)
		}
		conncountp.Put(count)

		for i, cmd := range multi {
			last = cmd.Slot()

			var cc conn
			if c.opt.SendToReplicas(cmd) {
				toReplica = true
				cc = c.rslots[cmd.Slot()]
			} else {
				cc = c.pslots[cmd.Slot()]
			}

			re := retries.m[cc]
			re.commands = append(re.commands, cmd)
			re.cIndexes = append(re.cIndexes, i)
		}
		return retries, last, toReplica
	}

	inits := 0
	for _, cmd := range multi {
		if cmd.Slot() == cmds.InitSlot {
			inits++
			continue
		}
		if last == cmds.InitSlot {
			last = cmd.Slot()
		} else if init && last != cmd.Slot() {
			panic(panicMixCxSlot)
		}
		p := c.pslots[cmd.Slot()]
		if p == nil {
			return nil, 0, false
		}
		count.m[p]++
	}

	if last == cmds.InitSlot {
		// if all commands have no slots, such as INFO, we pick a non-nil slot.
		for i, p := range c.pslots {
			if p != nil {
				last = uint16(i)
				count.m[p] = inits
				break
			}
		}
	}

	retries = connretryp.Get(len(count.m), len(count.m))
	for cc, n := range count.m {
		retries.m[cc] = retryp.Get(0, n)
	}
	conncountp.Put(count)

	for i, cmd := range multi {
		var cc conn
		if cmd.Slot() != cmds.InitSlot {
			cc = c.pslots[cmd.Slot()]
		} else {
			cc = c.pslots[last]
		}
		re := retries.m[cc]
		re.commands = append(re.commands, cmd)
		re.cIndexes = append(re.cIndexes, i)
	}
	return retries, last, false
}

Hope this can help you.

File a PR, check #697
Maybe another question:

wondering should driver auto recover from error failed to fetch the cache because EXEC was aborted by redis or connection closed by DoMultiCache during redis cluster scaling ?


Feature/852 key generation

:tada: This PR is included in version 0.15.0-development.9 :tada:
The release is available on:

npm package (@development dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.16.0-development.1 :tada:
The release is available on:

npm package (@development dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing the document.uri.query field when running extension in Codespaces

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

VS Code Version: 1.57.1
OS Version: Darwin x64 19.6.0
Codespaces Version: 1.57.1

We come across a behavioral inconsistency on VSCode v.s CodeSpaces when developing our extension. We have a class that implements the vscode.CommentingRangeProvider. In its provideCommentingRanges(document: vscode.TextDocument) method, we are trying to access thedocument.uri.query field, but it is missed if the uri scheme is file and when we are in CodeSpaces. This never happened when we are developing in VSCode. Is it a bug or an intended behavior?
For your convenience, here is an extension we prepared to demo this issue:
https://github.com/tianzerun/vscode-extension-helloworld/tree/main
VSCode:

Notes: there is a query field
Codespaces:

Notes: there is no query field

we are trying to access thedocument.uri.query field, but it is missed if the uri scheme is file and when we are in CodeSpaces.

Query parameters for file uris are bogous because the file system doesn't define any meaning for those, e.g file:///foo/bar.txt?aaa is the same as file:///foo/bar.txt?bbb. For custom schemes that's a different story and up to the implementor of the file system.
@bpasero for failing/checking in the file system provider when this happens and @alexdima for why this is different in codespaces. Maybe due to uri transforming?

Let's track in https://github.com/microsoft/vscode/issues/110377


Neovim client: fix semantic tokens

Neovim 0.9 add support for semantic tokens.
When opening a file and editing I get the following error: E5248: Invalid character in group name
I saw that the server registers tokens that are not in the specification: support-type-primitive, jsx-tag and jsx-lowercase.
This PR change name of tokens:

support-type-primitive -> type
jsx-tag -> class (class was used as module name, now it's namespace)
jsx-lowercase -> interface (div, h1)

One question: Why emit a token for jsx-tag, the < and > in <div>? Couldn't this be done via regex?
Forum post: https://forum.rescript-lang.org/t/neovim-vim-rescript-lsp-semantic-tokens-issue-and-maintenance/4387

Can semantic tokens be turned off in neovim until all works?

Can semantic tokens be turned off in neovim until all works?

yes

I'm afraid this PR will (from memory but I could be wrong) break some themes.

It seems not, because semanticTokenScopes maps to the same TextMate Scope.
This changes the name of the token the server emit, but the semanticTokenScopes field didn't change the TextMate scope target.
Example: previously the token jsx-lowercase was mapped to entity.name.tag, I changed jsx-lowercase to interface but it still gets mapped to entity.name.tag
VSCode Semantic token map

I'm going to do some tests.

Dark (default dark)
Before:

After:

Light (default light)
Before:

After:

Solarized Dark
Before:

After:

Solarized Light
Before:

After:

Monokai Dimmed ⚠
Before:

After:

Tomorrow Night Blue ⚠
Before:

After:

One Dark Pro ⚠
Before:

After:

Three themes have changed: Monokai Dimmed, Tomorrow Night Blue and One Dark Pro.
The changes were in the token < and >

Three themes have changed: Monokai Dimmed, Tomorrow Night Blue and One Dark Pro.
The changes were in the token < and >
Curious that the token </ is not emited.

The token </, if I remember correctly, is handled via regexp, as it's easy to recognise. While < and > are really overloaded and need the help of semantic highlighting.
So based on these findings, is it possible to handle the issue on the neovim side?

In general, I just don't know what the constraints are with neovim. One could also do everything via grammar, which is less precise but avoids breaking things. As long as that extra complexity does not affect the main extension.

So based on these findings, is it possible to handle the issue on the neovim side?

No, because it is necessary to change the name of the token emited by the server. Ideally, the specification would have a name for the token < and >, but it doesn't.
The best approach is to not emit < and > as jsx_tag.

So based on these findings, is it possible to handle the issue on the neovim side?

No, because it is necessary to change the name of the token emited by the server. Ideally, the specification would have a name for the token < and >, but it doesn't.
The best approach is to not emit < and > as jsx_tag.

That breaks things for 98% of users to benefit 2%? Or maybe I don't understand the suggestion.

Yes, you are correct.
What I'm saying is that the LSP specification should have a token to jsx.
I will continue emitting the token, but I will change the token type.

I changed the token from class to modifier.
One Dark Pro
After:

Tomorrow Night Blue
After:

So this seems to solve the issue.
Anything else pending, or can this go ahead?

I guess a quick check is needed that the other styles still work.

Yes, I tested it on all themes supported by the extension.

Yes, I tested it on all themes supported by the extension.

Fantastic. Thank you so much for sorting this out. Pretty subtle stuff.


feat: new doc stack

This PR:

Adds the required files to build the new doc stack / website
closes #33

Check out this PR (or open this PR in a codespace)
Run pip install -r requirements.txt then mkdocs serve -a localhost:8000

Point your browser at http://localhost:8000

I'm ignoring the failing YAML lint as this is failing on files that I haven't touched naturally during this work and so don't want to change those files

Hi @agardnerIT ! Thank you for starting this rework. 🙏
I just went quickly through the files pushed ( as I am on my phone at PromCon) and I have noticed you have pushed the Perses docs currently living in the upstream repo.
I would prefer these files be still in the upstream repo as much as possible.
The initial idea was that each repos will bring their own documentation and the website will aggregate  the documentations of each repos.
For example, I believe it will make sense to have a documentation relative to the perses-operator and if the doc is living in the perses-operator repo, that would be weird from my opinion.
Also having the doc living in the repo will allow us to version it like it the case today

closing this in favor of #35


CRI merge master

Just opened for CI tests.

I think that @s-urbaniak wants to take this separately in a further iteration of https://github.com/coreos/rkt/pull/3318.

This is the opposite direction - merging master in to CRI.

@s-urbaniak approved this out of band.


Problem in multi-gpu training

Bug description
Hi, my problem is even if my environments have multiple gpus, it only runs on one GPU. Could you help me?
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/1
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 1 processes
----------------------------------------------------------------------------------------------------

You are using a CUDA device ('NVIDIA RTX A6000') that has Tensor Cores. To properly utilize them, you should set `torch.set_flo
at32_matmul_precision('medium' | 'high')` which will trade-off precision for performance. For more details, read https://pytorc
h.org/docs/stable/generated/torch.set_float32_matmul_precision.html#torch.set_float32_matmul_precision
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [1,2]

What version are you seeing the problem on?
v2.1
How to reproduce the bug
The running scripts:
    trainer = pl.Trainer(
        accelerator="gpu",
        devices=2,
        max_epochs=100,
        strategy="ddp"

    trainer.fit(model, data_module)

Error messages and logs
# Error messages and logs here please

Environment

Current environment
python                    3.9.19               h955ad1f_1
pytorch-lightning         2.0.0                    pypi_0    pypi
pyyaml                    6.0.2                    pypi_0    pypi
readline                  8.2                  h5eee18b_0
requests                  2.32.3                   pypi_0    pypi
setuptools                72.1.0           py39h06a4308_0
sqlite                    3.45.3               h5eee18b_0
sympy                     1.13.2                   pypi_0    pypi
tk                        8.6.14               h39e8969_0
torch                     2.0.0+cu118              pypi_0    pypi
torchaudio                2.0.0+cu118              pypi_0    pypi
torchmetrics              1.4.1                    pypi_0    pypi
torchvision               0.15.0+cu118             pypi_0    pypi

More info
No response

I encountered the same problem, how did you solve it?


Automatic PR from RestAPI-PR414

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/414

Coverage decreased (-0.001%) to 57.864% when pulling 053e393a2a9d03afbd1eaf0346b5876d9d4ab0a1 on AutorestCI:RestAPI-PR414 into 06499603a16830e022becf77fa535f3ddb8b1ccf on Azure:dev.


Can't find cufft.h

Hello, I was trying to install AreTomo2, but encountered an error:
$make exe -f makefile
In file included from Util/CNextItem.cpp:1:
Util/CUtilInc.h:3:10: fatal error: cufft.h: No such file or directory
#include <cufft.h>
^~~~~~~~~
compilation terminated.
I searched the files, and in Util/CNextItem.cpp, includes "CUtilInc.h". In Util/CUtilInc.h, it includes cufft.h.
But there is no cufft.h in the folder.
Would you please help?
Thanks.
Liguo

makefile11.txt
Maybe you can check my makefile, I added a few more environment variables.

Maybe you should change $CUDA_HOME to the directory for cuda on your machine. cufft.h is located at $CUDA_HOME/include aka $CUDA_INC in my makefile.


Change a element based on the theme mode

How can we change a element based on the theme mode?
For example:
An image should change based on the theme. If it is dark mode, show a dark image else show default image.

You can create a variable like thisbool isLightTheme  = Theme.of(context).brightness == Brightness.light; to get the current theme. Base on the value of isLightTheme you can show the image.


chore: add .gitattributes

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

2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Allows nil values to be passed in from rails

Looks like this fixes issue #50
Rails passes in nil from hash as a blank and this converts that to a null that can be used in Cassandra query

Add tests for the fix and we can merge the PR. Thanks.

@bsbodden tests added!


@nxext/sveltekit has dependency on @angular-devkit/schematics

Not sure why, but if I create a blank workspace and add a sveltekit app it makes me install angular schematics as a workspace dependency before working, but after that it's ok...

NX  Generating @nxext/sveltekit:application

Cannot find module '@angular-devkit/schematics'
Require stack:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>/Users/dmcqueeney/Websites/nx-sveltekit/node_modules/nx/src/config/workspaces.js
/Users/dmcqueeney/Websites/nx-sveltekit/node_modules/nx/src/command-line/generate.js
/Users/dmcqueeney/Websites/nx-sveltekit/node_modules/nx/src/command-line/nx-commands.js
/Users/dmcqueeney/Websites/nx-sveltekit/node_modules/nx/bin/init-local.js
/Users/dmcqueeney/Websites/nx-sveltekit/node_modules/nx/bin/nx.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@nxext/sveltekit does not work for me too -> Several errors. Tried also to install @angular-devkit/schematics

It seems @angular-devkit/schematics is used by the @nrwl/devkit which is the basis for nx plugins. Interestingly it is not automatically added as a dependency. Adding this dependency manually does solve this issue, but adding it automatically would be nice.
A current pull request does switch to using a custom devkit instead of Narwhal's, which might also address this issue?


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


alacritty: 2018-08-30 -> 2018-09-29

Motivation for this change
Updating alacritty to master as of 2018-09-29.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

cc @Mic92

See https://github.com/NixOS/nixpkgs/issues/46807

I will do an upgrade here: https://github.com/NixOS/nixpkgs/pull/47591 (also we already have the same version without my fork in staging).
Let me know, if you still want to be in the maintainer list.

The other pull request is merged. You can add yourself as maintainer if you want.

Thanks! I appreciate your offer but no need since my branch will not be
used. Feel free to close this PR (or I can do it later tonight).
On Mon, Oct 1, 2018, 7:02 AM Jörg Thalheim<EMAIL_ADDRESS>wrote:

The other pull request is merged. You can add yourself as maintainer if
you want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/47579#issuecomment-425868405, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdgiejGySx9oM_30yJh0B4vAnGHud2Kks5ugfY_gaJpZM4XBW9J
.

closing, superseded


Add Typescript Types for Chart.js

Description
Provide a more detailed introduction to the issue itself

Add Typescript Types for Charting (plot.ly/chart.js)

Coming in with ChartJs 3 no longer needed for current version


Us#52

Pull Request Test Coverage Report for Build 1482

115 of 134 (85.82%)  changed or added relevant lines in 10 files are covered.
54 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-6.4%) to 73.032%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/io/github/kaiso/relmongo/events/callback/PersistentPropertyConvertingCallback.java
2
3
66.67%

src/main/java/io/github/kaiso/relmongo/mongo/DatabaseOperations.java
9
10
90.0%

src/main/java/io/github/kaiso/relmongo/mongo/PersistentRelationResolver.java
0
1
0.0%

src/main/java/io/github/kaiso/relmongo/events/callback/PersistentPropertyPostLoadingCallback.java
31
33
93.94%

src/main/java/io/github/kaiso/relmongo/events/processor/MappedByProcessor.java
5
7
71.43%

src/main/java/io/github/kaiso/relmongo/util/AnnotationsUtils.java
35
47
74.47%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/io/github/kaiso/relmongo/events/callback/PersistentPropertyPostLoadingCallback.java
1
93.48%

src/main/java/io/github/kaiso/relmongo/mongo/PersistentRelationResolver.java
13
41.67%

src/main/java/io/github/kaiso/relmongo/model/LoadableObjectsMetadata.java
15
0%

src/main/java/io/github/kaiso/relmongo/events/callback/PersistentPropertyLoadingCallback.java
25
0%

Totals

Change from base Build 1407:
-6.4%

Covered Lines:
566

Relevant Lines:
775

💛  - Coveralls


Add command storage tests.

Current coverage is 78.09%

Merging #116 into master will increase coverage by +0.62% as of fee676f

@@            master    #116   diff @@
======================================
  Files           77      77
  Stmts         2344    2347     +3
  Branches       267     267
  Methods
======================================
+ Hit           1816    1833    +17
  Partial         87      87
+ Missed         441     427    -14

Review entire Coverage Diff as of fee676f

Powered by Codecov. Updated on successful CI builds.

@alexander-yevsyukov, PTAL.

@alexander-yevsyukov, PTAL.

Please adjust CommandStorageRecord so that it works with target rather than aggregate. So, we’d have target_id and target_id_type fields.
Also, as we agreed, assume that a command to a ProcessManager would be the first field of a command message similarly to aggregate commands.

@AlexanderLitus, any update on this?

@alexander-yevsyukov, PTAL. I've also made some additional changes.

LGTM


Create Cabeçalho “header”

Automatically generated by Netlify CMS

👷 Deploy Preview for driveclean-teste processing.

Name
Link

🔨 Latest commit
d6efc0d4b9a97152e21ec64af8666e8e092e6e25

🔍 Latest deploy log
https://app.netlify.com/sites/driveclean-teste/deploys/65856b9ab418e40008357dfc


cmd/govim: only update quickfix when user is not busy

FYI @leitzler - this should make the quickfix updates less distracting.


Move from Flask to Quart

[x] Basic web server migration
[x] Fix base path middleware
[x] Fix directory walker to be async
[ ] Fix image/video thumbnailing to be async
[ ] Fix header scripts being async
[ ] Fix page thumbnailing

#1 fixed in 34b2a725f0158f34ff50e0b3aead97c1b86fc332, 800129161b39b41267484e30039dd5ff2feb920b, and f438331f2cabfc038888856f128c193bdf6ef99d

#\3 fixed in e82800f7f53a979b935f326832f816b4fcc329c8

#\2 fixed in e4b6ae27cb06686823d4f64d22afa7f230dce5cf

#\4 needs to be tested some more before re-closing.

#\4 fixed in 88d2fed5aa040a260ae6125ac4ffe0ea67054b58


HIV dataset

Where can I download the HIV dataset from you?

The HIV dataset is available in the repository.


Doc: State that Request.ParseForm() does not parse JSON payloads

What did you expect to see?
I spent a good deal of time trying to determine why my XHR-style "forms" (actually JSON) were coming in blank.
What did you see instead?
After much debugging, head-desking, and searching, I discovered that this doesn't work that way. I've learned, but other people who have used other server frameworks may make the same mistake I did. In fact, I discovered it on Stack Overflow through someone else having the same problem.
I realize that a change to ParseForm would need to go through the language change proposal process. Obviously, nothing needs to be added to the language to be able to parse JSON payloads. However, I believe that a short note in this comment would help people understand that they'll need to parse JSON bodies themselves.
I'd be happy to update the comment to that effect, but before I go through that effort, I wanted to ask the team if that sort of thing would be welcome. So - let me know. :) (and thank you for your time)

Doesn't the doc already talk about this?

For other HTTP methods, or when the Content-Type is not application/x-www-form-urlencoded, the request Body is not read, and r.PostForm is initialized to a non-nil, empty value.

"Content-Type is not application/x-www-form-urlencoded". I assume you're talking about application/json.
Am I misunderstanding your case or maybe the doc not spell it out well enough?

You're right that it does say that; I guess the negative implications of that statement didn't register with me.
I also probably ran down a rabbit trail because the router I was using at the time (Vestigo) puts its parameters in the form. I actually changed routers, thinking that Vestigo was whacking my posted values. Then, the form was always empty! :) (As one would expect, but that was the point where I got really confused.)
When I found the solution on Stack Overflow, I thought "Oh - of course." But, seeing the activity on that question, I thought it might be nice to call out this implication explicitly.

We generally want to keeps docs as succinct as possible since verbose docs leads to people ignoring them completely. Currently the doc is concise and correct on this matter.
It is staggering how popular that SO question is - 150k views. So maybe including something short about json being an example of unsupported content type could be helpful.

Which kind of HTTP method is your request? (https://tools.ietf.org/html/rfc2616#section-5.1.1)

Mine was a POST, as was the one being discussed in the Stack Overflow thread.

via PR, or without action?


[NamelessMC]: Cannot connect to your website, got an unexpected API error: API error nameless:invalid_post_contents (meta: {"field":"tps"}).

Bungee server
I have this error in the console (spam)
[NamelessMC]: Cannot connect to your website, got an unexpected API error: API error nameless:invalid_post_contents (meta: {"field":"tps"}). Run /nlpl last_error to retrieve the full stack trace.
Last_error :
[NamelessMC]: com.namelessmc.plugin.lib.nameless-api.exception.ApiException: API error nameless:invalid_post_contents (meta: {"field":"tps"})
        at com.namelessmc.plugin.lib.nameless-api.RequestHandler.makeConnection(RequestHandler.java:222)
        at com.namelessmc.plugin.lib.nameless-api.RequestHandler.post(RequestHandler.java:59)
        at com.namelessmc.plugin.lib.nameless-api.NamelessAPI.submitServerInfo(NamelessAPI.java:86)
        at com.namelessmc.plugin.common.AbstractDataSender.lambda$run$3(AbstractDataSender.java:158)
        at net.md_5.bungee.scheduler.BungeeTask.run(BungeeTask.java:66)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
        at java.base/java.lang.Thread.run(Thread.java:833)

Debug info :
[NamelessMC]: [Nameless-Java-API Debug] Making connection POST to https://domaine.com/index.php?route=/api/v2/minecraft/server-info
[NamelessMC]: [Nameless-Java-API Debug] POST request body:
{"server-id":1,"time":xxx8876849xxx,"free-memory":xx38817xx,"max-memory":xx68709xx,"allocated-memory":536870912,"verify_command":"verify","players":{"xxxxx611be4048xxxfb642f330xxxxx":{"name":"USERNAME","login-time":xxx8876848xxx,"address":"/xxx.xxx.xxx.xxx:x546x"}}}
[23:07:29 INFO] [NamelessMC]: [Nameless-Java-API Debug] Website response body, after 116ms:
{
    "error": "nameless:invalid_post_contents",
    "meta": {
        "field": "tps"
    }
}

All commands work, user verification works. The problem comes from the minecraft server sending information to the site but it still does it even if I set "send-data-server" to false.

Replace the contents of modules/core/includes/endpoints/ServerInfoEndpoint.php with this https://paste.rkslot.nl/raw/xayotubuwe

Thanks :)


add missing generators

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 62.90%. Comparing base (e5b8d44) to head (83fe18e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #185   +/-   ##
=======================================
  Coverage   62.90%   62.90%
=======================================
  Files          22       22
  Lines        6375     6375
  Branches     1208     1208
=======================================
  Hits         4010     4010
  Misses       1993     1993
  Partials      372      372

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


receive input on change

Hi, first this node works very well.
i can confirm for modell RX-V777.
is it possible to get somehow the value if its changed by other devices, like the normal remote.
i would change the volume slider in my dashboard if someone change it maybe via normal remote.
ist the avr sending commands, or i have to use the incomming node with an 1-2 second sheduler? so each second the sheduler grab the volume state from the avr ?
wich is the way to go ?
thanks in advance.

Happy to hear, that the nodes are working with your model.
The "AVR Yamaha Get" node needs to be triggererd/polled to read settings from your receiver, but the "AVR Yamaha In" node emits a message whenever certain settings are changed in the receiver (e.g. volume). The messages are emitted regardless of why the setting was changed (remote, manual, nodered, ...).
I think this is what you want.
Have a look at Example 3 in the readme: https://github.com/krauskopf/node-red-contrib-avr-yamaha

Ok i tried, seems this doesnt work.
i already used example 4 , there is no debug in my console.
i tried to change the volumne and the scene modes with my orginal remote control.
what else could i try ?

The AVR Yamaha uses UPnP events and does not work on every system . Most of the time, this is because there is already another UPnP service running and blocking port 1900 for other multicast listeners. Also check, that the Topic System.Misc.Event.Notice is written to On.
Where are you running node-red? A raspberry pi? Is there any other software running on the device which might already block the UPnP port?
The other chance that you have is polling for any state changes.

its rasberry pi, with redmatic addon. i think someone else is using this port already. is it possible to change the port ?

No, the port 1900 is part of the UPnP standard. Applications which want to access the port have to open the port in REUSE mode. Otherwise, the application is blocking other applications.
Seems like there is already a bugfix for this in homematic: https://homematic-forum.de/forum/viewtopic.php?f=65&t=41452
Maybe upgrade to the newest version, or try to deactivate in homeatic, as stated in the thread.

this is 1 year old thread, i have the newest version.
there is an hint:
each  UPnP-Announcer should have: SO_REUSEPORT/SO_REUSEADDR

This is what I'm doing. Can you check if the node is working if UPnP is temporary disabled in  /etc/init.d/S50eq3configd as written in the thread? Or maybe homematic is stopped?

Hi, i installed nodered fresh by itself on windows 10. installed only your node.
doesnt work. ip of the receiver is not dynamic. post is 8080.
could this be a receiver problem ?
attached a screen, seems it works somehow, but the ip is <IP_ADDRESS> ist this normal ?
the above section, there is upnp events received.. but the nodered debug console, shows nothing.

Hi,
ich wechsel mal auf Deutsch. Das macht es einfacher. Ja das mit <IP_ADDRESS>:1900 passt.
Auch, dass UPnP Events empfangen werden sieht gut aus. Interessant zu wissen wäre, was empfangen wird.
Leider ist die Ausgabe in der aktuellen Version deaktiviert.
Aber dazu habe ich mal einen Patch angehängt.  Diese befindet sich unter <.node-red user Verzeichns>\node_modules\node-red-contrib-avr-yamaha. Das User Verzeichnis ist i.d.r. unter C:\user\<username>\.node-red.
Wird auch beim Starten in der Console angezeigt.
avr-yamaha.zip
Danach bitte nochmal das Debug-Log von oben zeigen.

Noch eine Ergänzung.
Ist <IP_ADDRESS> die IP des Receivers?
    Danke.

Cannot reproduce and the latest version should have all correct options set. Might be also already fixed in redmatic.

Cannot reproduce and the latest version should have all correct options set. Might be also already fixed in redmatic.


rename RaftMetrics.leader_metrics to replication

Refactor: remove ClientRequestEntry;
It does not need to cache the entry, only the response tx.
Log entry can be fetched from storage by index.
Only tx channels for sending back result to client are stored and are
indexed by log-index.

This change is

LGTM


Clarify that PhysicsShapeQueryParameters3D.exclude is copied

Fixes #93895

Added the line Note that [code]exclude[/code] is a replace-don't-edit type property and can only be modified by replacing the entire array. to the PhysicsShapeQueryParameters3D.exclude docs

This should probably be done to all the cases of this, both 2D and 3D as well as the other parameters such as point and ray queries

Your own PR does this but it is done for every Packed Array property. To handles arrays like these would require a solution that allows Array properties to add the note automatically given a certain criteria

That would be pretty complex as there's no way to detect it, so it'd be a matter of adding a new property to the XML class, not sure it's worth it, but might look into checking how many cases there are for this

Not this case specifically, but there are many times where information is repeated that may benefit from such a system, similar to the keywords.

Absolutely, and I'll look at if there's enough cases to justify a system, but since this isn't automatable, and is likely not very common (i.e. cases that copy and aren't Packed*Arrays which require a different message), it might not be worth it, but I'll take a look at adding an option to manually tag with is_copied or something to make the output text automated

Please add to the other cases mentioned to get this to cover all the related cases, then this should be ready!

You also merged your branch instead of using a force push, please use git push --force instead, and when you're done please squash your commits into one, see here

I think I accidentally created a branch from some other local branch instead of master, is it okay if I open new pr with the above changes?

You can just rebase it, see the link, it shouldn't matter as long as you rebase on master
Also please make the final commit message something clear, like the PR title

Thanks!


Update device-update-ubuntu-agent.md

Adds clarity to the Add a tag to your device step
Resolves issue MicrosoftDocs/111836

There is an open item waiting to merge which contains the image file referenced in this commit. Here is the link to the request https://github.com/MicrosoftDocs/azure-docs/pull/112046

@vimeht
Can you review the proposed changes? Note that the image is added in https://github.com/MicrosoftDocs/azure-docs/pull/112046, which will need to be merged first to clear the validation warning in this PR.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

emailed author

approved by email from @andrewbrownmsft


Tweak sampled resource initialization

Tickets addressed: N/A
Review: By commit
Merge strategy: Merge (no squash)

Description

This PR is a minor adjustment to #956 that slipped my mind when reviewing yesterday. Cell allocation and updates are not allowed to occur during the same phase of simulation (Allocation must occur during initialization, updates must occur during task execution).  This PR moves the allocation into the SampledResource constructor - using the user-provided period as the initial value for the register, removing the need for an update during initialization.
Verification

I added a simple SampledResourceTest to exercise this code a little bit.
Documentation

See documentation section of the original PR #956
Future work

Sampled resources, while a good example of a daemon tasks, are pretty inefficient. @Twisol has some ideas for some better tools we can provide out of the box to address some use cases that currently require daemon tasks.

Just some random thoughts around this situation:

Allocation must occur during initialization

Although it wouldn't solve things here (since the problem is with the event emission at initialization time), I'd like to make it possible to allocate cells during the simulation, too. I think @JoelCourtney may have had some concerns around other features this might complicate though 🤔

updates must occur during task execution

Emitting events during initialization doesn't seem impossible to allow, and I don't think it would cause any new problems. The fact that we don't allow it right now comes down to the fact that we didn't know all the ramifications of the current design, and stayed somewhat conservative to keep our options open. But I think this is something that could be pursued. (We originally didn't allow cells to be queried during initialization, either -- we'd need to make sure those queries see any initialization-time events id we went in this direction.)

Looks good! Thanks for having my back @mattdailis - next time I will test like a good dev should. :)


feat(CLI): Support image rm command

Description
envd image rm should be supported to remove unused envd images.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Ref https://github.com/tensorchord/envd/issues/621#issuecomment-1189753075
To avoid docker rmi

Implement in envd destroy command


geocoder needs to support geoip2 from Maxmind

geoip is no longer available for purchase from maxmind and it is the version that geocoder supports.
http://dev.maxmind.com/geoip/geoip2/whats-new-in-geoip2/

This was added by #800.


Loading issues when using with a chrome extension

Hi Polymer team,
I have been developing a chrome extension using Polymer for some time now. Thanks to Polymer, I was was able to create a very intricate extension that is sheltered from the host's scripts and CSS. :+1:
I do however have a few of concerns about releasing it in it's current state. I would like to find out about some strategies for preventing the following problems I have been facing:

Loading Polymer into the page leaks into the global namespace. Polymer does not come bundled into JS files but rather, it comes in the form of an html page and requires the user to use an HTML import to load it into the page. AFAIK, content scripts only allow for CSS and JS but not HTML. To resolve this issue, I am including it by dynamically generating a link element and adding it as such into the page:

function loadUrl(url) {
  return new Promise(
    function(resolve, reject) {
      var link = document.createElement('link');
      link.setAttribute('rel', 'import');
      link.setAttribute('href', url);
      link.onload = function() {
        resolve(url);
      };
      document.head.appendChild(link);
    });
}

loadUrl(chrome.extension.getURL("polymer/polymer.html")).
then( loadUrl(chrome.extension.getURL("my/component.html")) )

Once loaded into the host page, it does not run in isolation like content scripts and can cause namespace conflicts if the page already has Polymer loaded.

Polymer does not tell you when it is loaded and ready to use. Polymer does not (currently) fire an event when it has loaded and as result, my components sometimes load before Polymer does (see bullet point 1) and break.

To mitigate this, I am triggering a custom event at the end of polymer-micro.html (which is where Polymer) is defined as such:
var ev = new CustomEvent('polymer-loaded');
document.dispatchEvent(ev);

The Polybuild tool does not generate proper code for a chrome extension. As useful as it is, it generates a separate javascript file outside of the dom-module causing a namespace leak into the global window object. For example, if was importing jQuery in my module, Polybuild would generate a JS file containing jQuery outside of the DOM module causing it to be attached to the host window object - a big no-no in a chrome extension.

Thanks for the feedback.
I have posted this in StackOverflow but my Polymer questions typically don't get many views there so I decided to post it here as well.

There is a bounty open in StackOverflow

Cricket cricket cricket..

Can you expand more on (3), or file an issue on polybuild and link to this issue?

It seems that the StackOverflow question has been answered. As such, I am closing this issue.


UWP: Incorporate toolchain for building ANGLE using Ninja

Microsoft ANGLE sources are outdated as noted in the documentation for compiling for UWP.
This PR aims to add a front-end to https://chromium.googlesource.com/angle/angle project. You are expected to setup necessary toolchains for building ANGLE itself: https://chromium.googlesource.com/angle/angle/+/HEAD/doc/DevSetup.md. Once this is done, SCons will call autoninja to compile ANGLE along with Godot.
Have to compile with scons angle_toolchain=ninja. I've preserved the previous build toolchain using MSVC, but I don't guarantee it to still work (not being able to compile ANGLE using MSVC is one of the motivations behind making this PR).
Even though I've managed to build ANGLE and some Godot export templates that way, I have no way to actually test this due to export/signing issues, see #30558, even with official builds of Godot, so I'm losing motivation in working on this further. Seems like there's a way at the end of the day: https://github.com/godotengine/godot/issues/30558#issuecomment-797831794.
arm does not seem to be available in ANGLE master branch, only arm64.
Note that this was done purely out of personal interest/educational purpose, there's a lot of thing I don't even understand, but since there's no active UWP maintainers, I decided to submit this PR. Perhaps @vnen could find something useful from this. 🙂
Some useful snippets:
set ANGLE_SRC_PATH=d:/src/angle
vcvarsall.bat x64
scons platform=uwp tools=no target=release_debug angle_toolchain=ninja

To-do

[ ] Figure out how to compile for arm64 (host/target args).
[ ] Add concurrent job option to ANGLE.

I've finally figured how to export to UWP with workaround (https://github.com/godotengine/godot/issues/30558#issuecomment-797831794), so reporting that I've managed to build ANGLE libs and successfully exported project on x64, debug with a custom export template. Should also work for x86.
The remaining problem is being able to build for arm:
Traceback (most recent call last):
  File "d:/src/angle/build/vs_toolchain.py", line 573, in <module>
    sys.exit(main())
  File "d:/src/angle/build/vs_toolchain.py", line 569, in main
    return commands[sys.argv[1]](*sys.argv[2:])
  File "d:/src/angle/build/vs_toolchain.py", line 396, in CopyDlls
    raise Exception('Unknown target_cpu: ' + target_cpu)
Exception: Unknown target_cpu: arm
ERROR at //build/toolchain/win/BUILD.gn:54:3: Script returned non-zero exit code.
  exec_script("../../vs_toolchain.py",
  ^----------

and arm64:
Traceback (most recent call last):
  File "d:/src/angle/build/vs_toolchain.py", line 573, in <module>
    sys.exit(main())
  File "d:/src/angle/build/vs_toolchain.py", line 569, in main
    return commands[sys.argv[1]](*sys.argv[2:])
  File "d:/src/angle/build/vs_toolchain.py", line 397, in CopyDlls
    _CopyRuntime(target_dir, runtime_dir, target_cpu, debug=False)
  File "d:/src/angle/build/vs_toolchain.py", line 372, in _CopyRuntime
    _CopyUCRTRuntime(target_dir, source_dir, target_cpu, suffix)
  File "d:/src/angle/build/vs_toolchain.py", line 291, in _CopyUCRTRuntime
    _CopyRuntimeImpl(target, source)
  File "d:/src/angle/build/vs_toolchain.py", line 234, in _CopyRuntimeImpl
    abs(os.stat(target).st_mtime - os.stat(source).st_mtime) >= 0.01)):
WindowsError: [Error 3] The system cannot find the path specified: 'C:\\Program Files (x86)/Microsoft Visual Studio/2019/Community\\VC\\Redist\\MSVC\\14.28.29325\\arm64\\Microsoft.VC142.CRT\\msvcp140.dll'
ERROR at //build/toolchain/win/BUILD.gn:54:3: Script returned non-zero exit code.
  exec_script("../../vs_toolchain.py",
  ^----------

Owh cool, I didn't realize Googles version of Angle actually runs on UWP. They don't mention it by name which I always found weird.

Owh cool, I didn't realize Googles version of Angle actually runs on UWP. They don't mention it by name which I always found weird.

It did in the beginning, then they switched buildsystems and removed it, then they started supporting it (for quite a while actually). It's not mentioned in the Readme but it is on the build instructions page.

Owh cool, I didn't realize Googles version of Angle actually runs on UWP. They don't mention it by name which I always found weird.

It did in the beginning, then they switched buildsystems and removed it, then they started supporting it again (for quite a while actually). It's not mentioned in the Readme but it is on the build instructions page.

Ok, sounds like I need to give this a try too soon

Got this to build and it runs fine as x64 on PC, but unfortunately it crashes instantly when ran on Xbox. It doesn't crash with the current stable build when ran on Xbox. I'll try looking further into it and hopefully will find a fix for it. This PR could very well be the fix for #9168 that we need as well as a few other UWP port bugs. For the arm64 build problem it looks like you're just missing a DLL. I'll try at that one this weekend. Not sure what to do about the arm problem though...seems like that is the main thing that'll be a showstopper about this.

I got angle with arm64 to compile but keep running into problems with a handful of modules due to what seems to be similar issues as this one #45687 where it is saying "emmintrin.h(20): fatal error C1189: #error:  This header is specific to X86 and X64 targets". Disabling some of these modules, specifically vorbis, is just getting me LNK2019 errors so unfortunately the compilation isn't finishing. BUT the angle part is compiling fine at least. Someone who has more experience with compiling for ARM/ARM64 might know some workarounds for these module specific issues.

Got this to build and it runs fine as x64 on PC, but unfortunately it crashes instantly when ran on Xbox. It doesn't crash with the current stable build when ran on Xbox.

I've forgot to ask: how do you setup the environment? There seems to be some UWP-specific options when running vcvarsall.bat, perhaps they can help:

Disabling some of these modules, specifically vorbis, is just getting me LNK2019 errors so unfortunately the compilation isn't finishing.

There was indeed some issues around this in the past: #39219, likely not everything can be disabled safely: #39196.

Hmm yeah I can give it a go using the store parameter and see if that helps. Before I just used the 'x64' parameter. Just connected to my xbox from Visual Studio to debug remotely and go this before it crashed:

Calling dumpbin /DEPENDENTS on the godot.uwp.exe spit out quite a few dependencies, and more than I remember seeing when checking for dependencies of my uwp builds on 3.2.x. So it's likely there is just a DLL missing...now to find which one of these it is...

Okay running vcvarsall.bat x64 store is the fix. Runs fine on xbox now. Thanks! Also now that I got it running I can confirm that this fixes #9168. No input lag at all on xbox! :smiley:

By the way, I wanted to discuss the current state of GLES3 support in Godot UWP, probably it is the right place to ask :)
Our game I was trying to port on Xbox worked with GLES3 renderer. The game works on Godot 3.1.2.
I've changed the following line:
https://github.com/godotengine/godot/blob/e16f264a7e0bec6acd07284a1fc9c256fc483368/platform/uwp/os_uwp.cpp#L189
to the following
ContextEGL_UWP::Driver opengl_api_type = ContextEGL_UWP::GLES_3_0;
then I rebuilt Godot with libEGL.dll and libGLESv2.dll from Google Angle and Voila, our GLES3 Godot game works as UWP on our laptop. It is not working on Xbox though, but it is a different story.
Right now this line still holds GLES_2_0
https://github.com/godotengine/godot/blob/80fc90e82a57e8111cb7577b05bb2de021000e75/platform/uwp/os_uwp.cpp#L167
with the following comment
// FIXME: Hardcoded for now, add Vulkan support.
So, my questions are:

Will it be possible to run GLES3 games in Godot 3.x UWP without a need to tweak the code?
Will Vulkan renderer work well with UWP on Xbox one (probably no, I guess Xbox supports only DirectX)

Sorry for such a long comment. I just really want our game to work on Xbox via UWP, but it seems that right now it is not possible :)

Will Vulkan renderer work well with UWP on Xbox one (probably no, I guess Xbox supports only DirectX)

This is likely possible using a Vulkan to DirectX 12 translation layer.

Wow, glad to hear that, I hope Angle has or will have such support!
Also, there is some kind of translation layer here, although the project seems abandoned...
https://github.com/twinaphex/VulkanOnD3D12

The "official" in 2022 for  Vulkan over D3D12 is Dozen

Closing as this contributor can no longer update this further.


Cross portal container block interaction broken?

I can confirm this in 1.18.2-1.19.2. Didn't test 1.19.3/1.19.4, so not sure about those versions.

Can confirm in 1.19.4 Quilt with latest versions of Quilt, QAPI, Immersive Portals

Probably fixed in the next version (MC 1.19.4)


add order type to commerce schema

Issue #1534

Closing the PR as the PR at https://github.com/adobe/xdm/pull/1537 is approved.


 Re-linked the repositories but now we lost our signatures. (we're using cla-assistant.io) Any way we can get them back?

Thanks for pointing it out. CLA assistant doesn't support repository transfer yet. But it should be for sure implemented. I've done some preparation for that. Could you check the list of signed CLAs in CLA assistant again, please?

It looks good. Thanks for that!

The workflow is now to add a renamed/ transferred repository again to CLA assistant.


delete ertra slash

Signed-off-by: yuxiaobo<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Delete ertra slash
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Don't need
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

repeat.


Update Center: Network Error has occurred

Mostly looks like an issue because the files are not signed or anything, need to check.
The index file is downloaded successfully but downloading a selected nbm did not work.

Should be fixed in https://github.com/jMonkeyEngine/sdk-update-center/commit/12b95b67efe4411f9b42740f1f422ab8628ea635


Question: Setting Options

Please provide greater details on setting options. The Settings pane doesn't include any options for your package. I need to add transpilation.

Hi, that's odd. The settings pane should contain all options mentioned in the README. Do you mind posting a screenshot?

Note: if you're transpiling from CoffeeScript for example, it's good practice to add that in test/mocha.opts, which is automatically picked-up by Mocha.
--compilers coffee:coffee-script/register

@rprieto I reinstalled it and was able to see the settings.
Thanks for the tip on mocha.opts. That works well.

No worries, happy it's fixed!


Query logs from slurm job

Asked that question in k8s community slack and got response from one of operator-sdk developer, look like this is not possible (at least yet). She suggested grabbing logs based on labels, that is almost the same approach we use now. Currently logs may be retrieved by running kubectl logs <slurm job name>-job which is acceptable. Closing this issue.


feat(client): add LogStreamAppender

What this PR does
This PR adds LogStreamAppender to the client, and it is a client to append asynchronously only a particular log stream.
Which issue(s) this PR resolves
Resolves #433

Current dependencies on/for this PR:

main

PR #444

PR #411

PR #412

PR #413

PR #414

PR #445

PR #446

PR #447

Current dependencies on/for this PR:

main

PR #444

PR #411

PR #412

PR #413

PR #414

PR #445

PR #446

PR #447

Current dependencies on/for this PR:

main

PR #444

PR #411

PR #412

PR #413

PR #414

PR #445

PR #446

PR #447

Codecov Report
Patch coverage has no change and project coverage change: -0.09 :warning:

Comparison is base (80abf51) 63.15% compared to head (ab11263) 63.07%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                     Coverage Diff                      @@
##           pipelined_append_handler     #459      +/-   ##
============================================================
- Coverage                     63.15%   63.07%   -0.09%
============================================================
  Files                           132      132
  Lines                         18145    18149       +4
============================================================
- Hits                          11459    11447      -12
- Misses                         6120     6135      +15
- Partials                        566      567       +1

Impacted Files
Coverage Δ

pkg/varlog/config.go
91.42% <ø> (ø)

pkg/varlog/log.go
0.00% <0.00%> (ø)

pkg/varlogtest/log.go
72.20% <0.00%> (-0.47%)
:arrow_down:

... and 8 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.

@ijsong started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@ijsong merged this pull request with Graphite.


Ansible 2.8

Install Ansible 2.8 as the default version.
Require minimum Ansible version 2.7.
Allow setting the default Ansible version with MACHINE_ANSIBLE_VERSION environment variable.
Test Ansible 2.7, 2.8 and 2.9 on Travis.

Build is falling due to https://github.com/markosamuli/ansible-aws-tools/issues/10


When updating pip via puppet-python, an error occurs.

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 6.4.2
Ruby: ruby 2.5.3p105 (2018-10-18 revision 65156) [x86_64-linux]
Distribution: CentOS Linux release 7.6.1810 (Core)
Module version: master

How to reproduce (e.g Puppet code you use)
What are you seeing
$ grep -A5 python: data/app/pup.yaml
python::python_pips:
  pip:        { ensure: 'latest' }
  pre-commit: { ensure: 'latest' }
  setuptools: { ensure: 'latest' }
  wheel:      { ensure: 'latest' }

$ /opt/puppetlabs/bin/puppet agent -t
Info: Using configured environment 'dev'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Retrieving locales
Info: Loading facts
Error: Facter: error while resolving custom fact "pip_version": undefined method `[]' for nil:NilClass
Info: Caching catalog for cusppupent01.internal.cnngad.com
Info: Applying configuration version '1558463440'
Notice: System Reboot Required Because:
  audit => modified
Notice: Applied catalog in 25.40 seconds
$ which pre-commit
/usr/bin/which: no pre-commit in (/usr/local/openssl/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/opt/puppetlabs/bin:/opt/puppetlabs/puppet/bin)

What behaviour did you expect instead
Successful installation of pip, setuptools, wheel, and pre-commit
Output log
See above.
Any additional information you'd like to impart
Possibly related: https://github.com/pre-commit/pre-commit/issues/1039

Apparently, I must install the python-setuptools rpm before installing the setuptools pip, else it will fail silently.


PluginView layout is overlapped with PluginSourceBrowser

Hi! I just downloaded xbar v2.0.32-beta from latest release and seems like PluginView layout is broken on my machine. PluginSourceBrowser is hover the rest of PluginView area and I can't tap the install button 😂 .
Some details about my machine

MacBook Pro (15-inch, 2018)
macOS Big Sur 11.1
xbar v2.0.32-beta

Hi @kajornsakp have you tried frisbeeing your laptop into the sea?

@matryer Strongly agree. He should try to do that.

But seriously, this bug gets fixed if you upgrade Big Sur to the latest. It was happening in Catalina too, but we fixed it.
@leaanthony What was the fix you did?

It was a CSS rule. It made no sense at the time, but does it ever? There's a one line commit in the old repo as it happened before merge.

@kajornsakp Does this consistently happen for you? Does the Refresh button help at all?

@matryer yes, refresh button didn't help. Even force quit and restart the app still give the same result

@kajornsakp I know upgrading to the latest Big Sur will fix this, but I'd love to try and figure out how to fix it without forcing people to do that.

@kajornsakp What version of Safari you have installed? Thanks.

This was the fix for Catalina:

@leaanthony Safari version : Version 14.0.2 (166<IP_ADDRESS>.9)
@matryer Yeah, I wish I could update to latest version of Big Sur too. But this mac is controlled by MDM service from my company 😅

Thanks. I'm wondering if we can get an archive version of safari at that version

This should either be fixed by now (depending on OS version) or will be fixed by #705