Problem in position embedding

https://github.com/mlfoundations/open_lm/blob/619a8b3afcb4e0b0dda7fb58ed271a26662fc10f/open_lm/model.py#L129
It seems to me that the rotary position embedding is being applied on the head dimension (dim -2) of the vectors q, k instead of the sequence dimension (dim 1).
I think the head and sequence dimensions should be swapped before calling  position embedding .
(see https://github.com/facebookresearch/xformers/blob/748c159096d4f9fcfe3eaf22801e5aed4777210b/xformers/components/positional_embedding/rotary.py#L85)
What I'm proposing is simply to re-write RotaryWithCast as follow:
class RotaryWithCast(RotaryEmbedding):
    def forward(self, q, k, v):
        q, k = super().forward(q.permute(0, 2, 1, 3), k.permute(0, 2, 1, 3))
        q = q.permute(0, 2, 1, 3)
        k = k.permute(0, 2, 1, 3)
        return q.to(v.dtype), k.to(v.dtype), v

Here is the runs I made with a custom subset of starcoder data. The original 11m training is in brown. My implementation using a different positional encoding is in orange.

Good catch! The blow up curves your are seeing are similar to the ones we were seeing before we introduced qk norm for the smaller models. Will do some testing with this fix on my end as well. Would you like to open a PR?

Wow, amazing catch! We really appreciate this.

We've added your name to the README because this is a very substantial bug catch. It's pretty interesting that our first 1B/7B runs do pretty well even without proper posembeds, but we should fix this going forward.

Great code base by the way. It's a pleasure to read.
Thanks for proposing to include me. I could open a PR but it's probably simpler for you to just include what I wrote (or a better version... I haven't tested if calling contiguous would make a difference).

actually moving that line before the call to view would be enough...
https://github.com/mlfoundations/open_lm/blob/9b3ca532bab801e7035ed7a3ff2f9771ed30ecd2/open_lm/model.py#L129

The problem actually seems to be upstream in xformers. Opened an issue here: https://github.com/facebookresearch/xformers/issues/841


QPACK: make header encoding concurrency possible

Change qpack::encode  and it's underlying functions to make it possible for simultaneous calls.
Make DynamicTableEncoder::insert accept a Arc<Mutex<DynamicTableEncoder>> rather than a mutable reference and  return a Result<(), BlockedHeaderEncoding> or a similar type to express the encoding is waiting on a shared resource. So polling the unblocking condition becomes possible.

DynamicTableEncoder::insert doesn't need to be aware of any locking/refcounting strategy itself; it can just take a &mut self and let the caller wrap it in Arc<Mutex<_>> (or something else) as needed.

As Ralith stated, the concurrency handling actually happens at the connection level (Arc<Mutex<ConnectionRef>>).
Closing.


Is there any way to know the saved camera locations people use like on f1,f2,3 etc

Looking for a way to get the camera location saved events, Would be nice too to know when they are used, however, I can figure that one out outside the scope of sc2reader, but really need to know when a camera location is saved.

Sorry, I'm not familiar enough with the contents of the replays to know how to do this, but maybe someone else can jump in!

Search this repo for camera:
https://github.com/search?q=repo%3Aggtracker%2Fsc2reader camera&type=code

I see, So right now looks like sc2reader just has camera movement events and not camera save events.
Looks like the s2protocol has the camera save event defined here and nothing for using these camera locations: https://github.com/Blizzard/s2protocol/blob/e6dfb2cea7ef0d34199d3df2f2cbebd7ddcd7960/json/protocol92440.json#L5760
Just adding this for context:
To try figure out if a camera location hotkey is used I guess I would need to

Save the saved location coordinates
Check whenever a camera moved to any of the saved coordinates, when it does, make sure all the following are true:

The user did not double tab a hotkey to center at that units coordinates (like double tap 4)
The user didn't pan to the coordinate
The user didn't just click the minimap
If all are true, then the user tapped the hotkey for that saved location.

I am not super familiar with the internals of sc2reader and how to add new events but I can probably just use mpyq directly for my use case.
Thank you


attempt fix for devrelease helm move

Signed-off-by: jonahjon<EMAIL_ADDRESS>Currently if we are doing a dev release we skip helm chart.yaml fix.
We should only do the packages chart upload skip if it's not dev release too, otherwise it will fail on dev releases.

/override eks-anywhere-e2e-presubmit


Do not check only unlocked gems for pruning

We're seeing an issue with bundler updates not correctly pruning all the old cached dependencies that have been removed.
This PR removes a bit of code that digs into bundler internals to find gems that have been "unlocked" (meaning to have their constraints removed so they may be updated by bundler), and only remove .gems that match this list. This code existed in an attempt to support cases where you had a conditional dependency in your Gemfile (for example in order to test against multiple versions of some dependency). However, this logic seems flawed, as it does not take into account any transitive dependencies, which is also the issue that we've observed.
Since this code seems to cause more problems than it tries to solve, I think removing it is the best step forward for now. We could potentially update the check to take all the old deps, compare them with the new ones, and only prune .gems from the cache that were actively changed in this PR, but that has us deviating from bundlers native behavior, which I'm not sure is worth the hassle.
This also removes a bit of temporary code that was in place in order to support an internal project at GitHub, which is no longer needed. Since these were entangled, removing them together is easiest, but if there's any objections I'm happy to split this up.

@jurre I got curious about this and experimented with the fix I had in mind. Can you check whether this patch fixes the issues with some gems not being pruned?
diff --git a/bundler/helpers/v2/lib/functions/lockfile_updater.rb b/bundler/helpers/v2/lib/functions/lockfile_updater.rb
index ca619ccbb..c49603c75 100644
--- a/bundler/helpers/v2/lib/functions/lockfile_updater.rb
+++ b/bundler/helpers/v2/lib/functions/lockfile_updater.rb
@@ -64,8 +64,12 @@ module Functions
     def cache_vendored_gems(definition)
       # Dependencies that have been unlocked for the update (including
       # sub-dependencies)
-      unlocked_gems = definition.instance_variable_get(:@unlock).
-                      fetch(:gems).reject { |gem| __keep_on_prune?(gem) }
+      resolve = definition.resolve
+
+      updated_gems = resolve.reject do |spec|
+        lockfile_specs.include?(spec) || __keep_on_prune?(spec.name)
+      end.map(&:name).uniq
+
       bundler_opts = {
         cache_all: true,
         cache_all_platforms: true,
@@ -79,8 +83,7 @@ module Functions
         # Only prune unlocked gems (the original implementation is in
         # Bundler::Runtime)
         cache_path = Bundler.app_cache
-        resolve = definition.resolve
-        prune_gem_cache(resolve, cache_path, unlocked_gems)
+        prune_gem_cache(resolve, cache_path, updated_gems)
         prune_git_and_path_cache(resolve, cache_path)
       end
     end
@@ -95,14 +98,14 @@ module Functions
     end

     # Copied from Bundler::Runtime: Modified to only prune gems that have
-    # been unlocked
-    def prune_gem_cache(resolve, cache_path, unlocked_gems)
+    # been updated
+    def prune_gem_cache(resolve, cache_path, updated_gems)
       cached_gems = Dir["#{cache_path}/*.gem"]

       outdated_gems = cached_gems.reject do |path|
         spec = Bundler.rubygems.spec_from_gem path

-        !unlocked_gems.include?(spec.name) || resolve.any? do |s|
+        !updated_gems.include?(spec.name) || resolve.any? do |s|
           s.name == spec.name && s.version == spec.version &&
             !s.source.is_a?(Bundler::Source::Git)
         end
@@ -147,8 +150,7 @@ module Functions
     end

     def unlock_blocking_subdeps(dependencies_to_unlock, error)
-      all_deps =  Bundler::LockfileParser.new(lockfile).
-                  specs.map(&:name).map(&:to_s)
+      all_deps = lockfile_specs.map(&:name).map(&:to_s)
       top_level = build_definition([]).dependencies.
                   map(&:name).map(&:to_s)
       allowed_new_unlocks = all_deps - top_level - dependencies_to_unlock
@@ -224,6 +226,10 @@ module Functions
       sources.all? { |s| s&.fetch("type", nil) == "git" }
     end

+    def lockfile_specs
+      @lockfile_specs ||= ::Bundler::LockfileParser.new(lockfile).specs
+    end
+
     def lockfile
       @lockfile ||= File.read(lockfile_name)
     end

I think it should not break any existing specs.

@deivid-rodriguez that does not seem to pass the test-case we had in place for this unfortunately

Oh too bad! You mean any existing test cases in our test suite or the realworld example where we found that not all previous .gem files were being pruned?

I was just running the tests cases that we have here around conditional deps, I assume it will work as expected on the real-world scenario that doesn't have these

@jurre Just to make sure I pushed the change to trigger CI and it seems to be passing: https://github.com/dependabot/dependabot-core/pull/7295.

@jurre Just to make sure I pushed the change to trigger CI and it seems to be passing: #7295.

Nice! My dev env must have not picked up the changes somehow, that's brilliant, let me pull those changes in here, I agree keeping the not updated gems in the cache is better, thanks @deivid-rodriguez!


How to pass a list of words in a str_detect

Hello,
I need to understand if it is possible to pass a list of words in a str_detect with sparklyr.
I did it with R like this :
datasetml$common_ent_word<-str_detect(datasetml$NAME_NORM_LONG, "CORP| INC|&|LLC|L.L|LIMITED|GROUP|SERVICE|'|FARM|RESORT|
BEAUTY|MASONRY|BUILD|FRAMING|TRUST|HOUSE |DIESEL|PAINTING|LANDSCAPE|ACADEMY|GARAGE|TECH")

But when I try to do the same in sparklyr:
X<-X1 %>% mutate(common_Ent_Corp=str_detect(NAME_NORM_LONG, 'CORPINC|&|LLC|L.L|LIMITED|GROUP|SERVICE|'|FARM|RESORT|
BEAUTY|MASONRY|BUILD|FRAMING|TRUST|HOUSE |DIESEL|PAINTING|LANDSCAPE|ACADEMY|GARAGE|TECH'))

Seems that it is not working.
Any advice of this?
Thank you
Regards

@Scotturbina not supported, you would have to use regexp_extract or regexp_replace instead.

@javierluraschi  do you have some example I can follow?
Thank you
Regards

I use the Spark SQL function like in place of str_detect when working with sparklyr and Spark tables rather than dplyr and local data frames.
Ex:
#local data frame with dplyr
iris_tbl <- sdf_copy_to(sc, iris)
filter(iris, str_detect(Species, "ersicol")) %>% head()

Sepal.Length Sepal.Width Petal.Length Petal.Width    Species
1          7.0         3.2          4.7         1.4 versicolor
2          6.4         3.2          4.5         1.5 versicolor
3          6.9         3.1          4.9         1.5 versicolor
4          5.5         2.3          4.0         1.3 versicolor
5          6.5         2.8          4.6         1.5 versicolor
6          5.7         2.8          4.5         1.3 versicolor

#spark table with sparklyr
filter(iris_tbl, like(Species, "%ersicol%"))

# Source: spark<?> [?? x 5]
   Sepal_Length Sepal_Width Petal_Length Petal_Width Species
          <dbl>       <dbl>        <dbl>       <dbl> <chr>
 1          7           3.2          4.7         1.4 versicolor
 2          6.4         3.2          4.5         1.5 versicolor
 3          6.9         3.1          4.9         1.5 versicolor
 4          5.5         2.3          4           1.3 versicolor
 5          6.5         2.8          4.6         1.5 versicolor
 6          5.7         2.8          4.5         1.3 versicolor
 7          6.3         3.3          4.7         1.6 versicolor
 8          4.9         2.4          3.3         1   versicolor
 9          6.6         2.9          4.6         1.3 versicolor
10          5.2         2.7          3.9         1.4 versicolor
# … with more rows

Note that you need the wildcard symbols in there. There are a ton of resources on regular expressions if you want to do more complicated string manipulations.
Also a reference on dplyr-SQL parsing, which explains why functions like str_detect() don't work inside dplyr function calls on databases/Spark.

@benmwhite  Hello, and it is possible to use like with more than one word?

@Scotturbina It doesn't, but depending on your situation you can either use a clever regular expression, or just use OR with multiple like calls inside filter.
filter(iris_tbl, like(Species, "%ersicolor%") | like(Species, "%etos%"))


[Storage] Delete Blob API responds not include header "x-ms-delete-type-permanent", which is not aligned with rest API doc

In Rest API doc, Delete Blob API responds will have "x-ms-delete-type-permanent" header for version 2017-07-29 and later.
Per my test, server will also return this header after 2017-07-29, no matter the account enable/disable soft delete.
But in storage swagger, the delete blob API responds not define the header "x-ms-delete-type-permanent".
This is not aligned with rest API doc and server behavior.
Currently , Azurite (storage rest API emulator) API is generated from swagger, since in swagger Delete Blob API responds not include header "x-ms-delete-type-permanent", Azurite also won't return it.
But Azure Storage Rust SDK is expecting the x-ms-deleted-type-permanent header and fails when this header is missing in the Azureite response.
So the swagger issue caused Azure Storage Rust SDK failed on Azurite. See https://github.com/Azure/Azurite/issues/2061/
Please fix swagger by adding the header to make it aligned with server behavior and rest API doc.

Hi,
Is there any update for this?


Azure Databricks Pyspark Support

Hi,
Just wondering if this would work with resizing azure databrick clusters!

Hi roshmath, Fiber doesn't support azure databrick clusters (yet). We want to hear from the community how much need there is for each of the platforms and prioritize our resources on the platform that are most requested.
At the same time, we also welcome contributions from the community. Adding a new backend for fiber isn't that hard, here is an example of how Kubernetes are supported in Fiber: https://github.com/uber/fiber/blob/master/fiber/kubernetes_backend.py

Sounds great, thank you!


fix: correctly compare compressed/expanded ipv6 in targets

When syncing a target with a compressed IPv6 (for example [2001:db8:fd73::e]:1326) decK can successfully create it but it fails to detect its existence when pulling the config from the API because the CP expands the address (for example to [2001:0db8:fd73:0000:0000:0000:0000:000e]:1326) causing decK to attempt to recreate the target and failing because the target already exists.
This commit fixes this by using some helpers to expand the ipv6 before trying to sync/dump.
Summary
SUMMARY_GOES_HERE
Full changelog

[Implement ...]
[Fix ...]

Issues resolved
Fix #XXX
Documentation

[ ] Link to the website documentation PR

Testing

[ ] Unit tests
[ ] E2E tests
[ ] Manual testing on Universal
[ ] Manual testing on Kubernetes

Codecov Report
Attention: Patch coverage is 82.60870% with 8 lines in your changes missing coverage. Please review.

Project coverage is 41.35%. Comparing base (1047bbb) to head (efb9dd5).

Files
Patch %
Lines

pkg/file/builder.go
82.60%
7 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   41.17%   41.35%   +0.17%
==========================================
  Files          75       75
  Lines       10807    10853      +46
==========================================
+ Hits         4450     4488      +38
- Misses       5895     5902       +7
- Partials      462      463       +1

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


Change to the correct rocksdb version

This closes #181.
Signed-off-by: Baohua Yang<EMAIL_ADDRESS>

Thanks! LGTM


Memory leak when having multiple webviews after version 13.0.0

Bug description:
Theres a memory leak when webview on multiple tabs.
I suspect the issue got in on PR #2930 ,since using earlier versions works.
I am not sure what the solution is here, maybe mark it as unsafe that you have to pass a prop to get it ?
To Reproduce:
Here's a repo I created: webview-memory-leak
Screenshots/Videos:
https://github.com/react-native-webview/react-native-webview/assets/109456266/42682df5-940f-42a4-92eb-541d8df90ae5
Environment:

OS: iOS
OS version: doesnt matter
react-native version: 0.72
react-native-webview version: 13.0.0

Are you using something to detect an actual memory leak here? The repo you linked just mentions a crash. I've experienced an occasional crash with version 13 but I don't think it's a leak... in my experience it happens "randomly," including sometimes on only the second WebView (or perhaps cleaning up the first? I'm still not certain of the cause)...

Are you using something to detect an actual memory leak here? The repo you linked just mentions a crash. I've experienced an occasional crash with version 13 but I don't think it's a leak... in my experience it happens "randomly," including sometimes on only the second WebView (or perhaps cleaning up the first? I'm still not certain of the cause)...

Good point, I didn't use anything to identify that its a memory leak, the error logs and the stack trace of iOS led me to believe it was, but I am not sure for certain it is.
I'd be happy to help debugging this.
Also changing the title.


How to handle local communication

Hello,
I'm using a previous version of Meltingpot and in the substrate configuration file, there was the following code:
config.individual_observation_names = [
      "RGB",
      "READY_TO_SHOOT",
      # Debug only (do not use the following observations in policies).
      "POSITION",
      "ORIENTATION",
  ]
  config.global_observation_names = [
      "WORLD.RGB",
  ]

I saw that you removed the debug observations to speed up the stepping.
I would like to use an RL algorithm with local communication, and to do so, I need to check the relative position/distance between two agents.
Is there a proper way to obtain this information?

Is what you really want a list of players near the focal player on each timestep? You can directly report that data as an observation  without using the absolute positions. The way to do it is to add a component on the avatar that runs a query for avatar objects in a rectangle (or whatever shape you like) around the focal avatar on each step. Then add an observation that reports this list out of the environment.
If you do want the absolute positions then just add the component that reports them and turn on the observations (basically undoing the commit that removed them).

Ok everything clear, thanks for the answer


Deleting Font-level and Glyph-level guides at the same time produces a traceback

I am getting a traceback when deleting font-level and glyph-level guides together. Tool still allows me to delete them separately, so this is pretty minor.
Reproduce:

Drag in a glyph-level guide from edge
Hold cmd, drag in a font-level guide from edge
Select both and hit delete key

Traceback:
  File "/Users/colinmford/Library/Application Support/RoboFont/plugins/Guide Tool.roboFontExt/lib/guideTool/tool.py", line 555, in keyDown
AttributeError: 'NoneType' object has no attribute 'undo'

The Line 555 in question.
Possibly related to #8 ?

This should be fixed. Thanks!

LGTM!


[FilledInput] Fix type error from undefined color

[x] I have followed (at least) the PR section of the contributing guide.

Closes https://github.com/mui/material-ui/issues/32256
Problem:

Explicitly setting color prop to undefined leads to crash in FilledInput

<FilledInput color={undefined} // Fixed "❌ TypeError: Cannot read properties of undefined (reading 'main')"/>

Before:

Code sandbox

After:

Code sandbox

@siriwatknp I committed your suggestion! Merging this now.


Devportal renders a blank white page when after click view in devportal button in Publisher portal

Description:
when I follow the below steps APIM-4.1.0 devportal gives a blank white page

create a tenant and tenants user
log in using tenant user
create & publish a new API
click view in devportal button

sometimes this happens when login in using carbon super admin user credentials as well.

Affected Product Version:
APIM-4.1.0

The issue is not reproduced in latest pack


Add GitHub Actions to auto-deploy

Add GitHub Actions to auto-deploy master branch to gh-pages
Works fine on my fork. See:

https://github.com/Rapiz1/incubator-apisix-website
https://github.com/Rapiz1/incubator-apisix-website/tree/gh-pages

Resolves https://github.com/apache/incubator-apisix-dashboard/issues/213
@juzhiyuan Kindly review

Thanks!


Feat sdk billing details screen

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.66%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5899

Relevant Lines:
14160

💛  - Coveralls


Adding extra git commands that windows users should run to receive correct line endings

As discussed in #743 and #1299 for Windows users to receive the correct line endings they need to reset their git repository after they set core.eol
I spent a couple hours trying to debug this.
P.S. First EVER pull request, sorry if I'm doing it wrong.

Very well done for a first pull request :D LGTM. Waiting for a second ACK to merge

LGTM2. Thanks @SeanBannister! If we haven't said it already, we really do appreciate all the feedback and help making this better.


Fix issue #56

Dear Thirumal/Anon,
I have written code to resolve issue #56 and it is tested. Can you please review the code and share the feedback.
Thank you

This pull request is now managed with CodeReviewHub. Refresh.


Main menu

Create main menu scene in Godot with labels and maybe even buttons eventually

Is there any specific design that you are looking for? Or can I just add whatever I want?

Go ahead and implement whatever you want! We can figure out details later.
From: AndrewYeh1 @.>
Date: Friday, April 16, 2021 at 12:50 PM
To: osa-computer-society/osa-epic-cool-thing @.>
Cc: Alexander Cai @.>, Author @.>
Subject: Re: [osa-computer-society/osa-epic-cool-thing] Main menu (#10)
Is there any specific design that you are looking for? Or can I just add whatever I want?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/osa-computer-society/osa-epic-cool-thing/issues/10#issuecomment-821466217, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AITXZFKD726OTEZ5CIZQTWDTJCBHHANCNFSM4Y2VF5LQ.


Feature Request: wt.exe to return PID of new child process on Standard Output

Description of the new feature/enhancement

In some scenarios it is important to be able to store the PID of a newly created shell (to kill it or wait for its termination asynchronously, to name two).
When launching a new wt.exe instance in order to run a command in another tab or split-pane, the console is actually instantiated from the already running instance. Hence, the calling instance terminates and the new console is not a child of the invoking console.
While it makes no sense to keep all the WT instances alive, it would be great if the PID of the newly created console could be returned on standard output, upon request.
Proposed technical implementation details (optional)

Add an additional option that is used when launching a new command, console, tab, or pane, e.g. --return-pid that would return the PID of the invoked child process, if wt.exe terminates.

This might be better-served by wt --wait as proposed in #8856; would you agree? After all, wt might spawn more than one process (in the case of wt split-pane x; split-pane y) and then we're getting dangerously close to specifying a protocol rather than simply returning a pid.

Thanks, @DHowett, for looking into this.
I do understand where you're coming from regarding specifying a protocol. However, most likely --wait will not work, because wt.exe (new) might hand over the execution of the child process to an already existing wt.exe (existing), as described by @zadjii-msft in this comment. In which case the x.exe would not be an actual child of wt.exe (new), but wt.exe (existing) in terms of process tree.
This was actually the reason why I was asking to return the PIDs. Because in my case it is about the process tree. (Specifically I am using gsudo and would want to inherit the credential cache to the newly created child.exe.
However, returning the PID(s) might also be a solution for the other use case, as it is quite simple to then wait for those processes to end:
 :waitforpid
 tasklist /fi "pid eq %1" 2>nul | find "%1" >nul
 if %ERRORLEVEL%==0 (
   timeout /t 2 /nobreak >nul
   goto :waitforpid
 )
 goto :eof

(Credits: PA. @ https://stackoverflow.com/a/22559493/3102305)
Of course it could be an option that wt.exe --wait does that job for the user and returns only, when all the "child" PIDs are gone. But both concepts could rely on the same internal implementation of getting the child PIDs from wt.exe (existing).
Regarding the protocol: would it not make sense to return a semi-colon separated list of PIDs, where the number of semicolons match the number of semicolons in the command line arguments to wt.exe (new) and each PID corresponds to the respective subcommand?

We've actually been having some discussions offline about what wt --wait means after #14843 and I think we've got a lot of new ideas that would make this work after all. I think the thing we're considering is that wt --wait would wait for all the children spawned by that commandline to exit, without necessarily waiting on the root terminal process (which will likely outlive those processes).
I'd much rather do that than try and return something on standard out.
But also, might it not just be easier to spawn cmd.exe, and just let the "default terminal" functionality (in Windows 11) parent the client to the Terminal of the user's choice? So, you're not waiting on the Terminal process, but the client app you actually care about/?

Thank you, @zadjii-msft for your reply and your considerations.

We've actually been having some discussions offline about what wt --wait means after #14843 and I think we've got a lot of new ideas that would make this work after all.

Sounds promising!

I think the thing we're considering is that wt --wait would wait for all the children spawned by that commandline to exit, without necessarily waiting on the root terminal process (which will likely outlive those processes).

Makes total sense. But it's a different use case.

I'd much rather do that than try and return something on standard out.

What is the issue about returning something on the standard out? I assume, normally there would never be output on STDOUT, but only on STDERR, e.g. in case a subcomand was not found. Hence, the STDOUT could easily be used without breaking backwards compatibility, as there would only be output if the user explicitly asks for the PIDs to be returned using --return-pid.

But also, might it not just be easier to spawn cmd.exe, and just let the "default terminal" functionality (in Windows 11) parent the client to the Terminal of the user's choice? So, you're not waiting on the Terminal process, but the client app you actually care about/?

Well, the WT has the possibility of having multiple panes in one tab, which is amazing. In one example I'm using that functionality to show in one tab

the main process in a big pane, and
the log files in a smaller pane.

Use-case
Let's assume, I have a script.bat that starts a httpd.exe in foreground-mode, and one tail.exe -f /path/to/error.log. Let's further assume that tail.exe should be terminated, once httpd.exe exits. That script contains the following line to start these children:
wt.exe new-tab httpd.exe; split-pane -H --size .2 tail.exe -f /path/to/error.log

Why --wait wouldn't do
The --wait parameter wouldn't make any sense here, because I would not know which process needs to be killed. Even if I would run the split-pane from within the new-tab-session's commandline, I still would not know the PID of the tail.exe.
Of course, this is a simplified example. But let's assume the path to the log file contains the PID of httpd.exe. Or that the second command is not just a tail.exe, but maybe something that needs to kill the httpd.exe if a certain entry appears in the log, or a certain file is appearing in the file system (e.g. /path/to/request.httpd.stop). How would these processes know the PID of httpd.exe (unless the httpd.exe itself creates a PID-file)?
However, with the --return-pid I could do something like:
Example 1: Kill commandline2 once commandline1 has ended
REM  #### Example 1: Kill commandline2 once commandline1 has ended
REM
setlocal enableextensions enabledelayedexpansion

set /a count = 0

REM run the command and save the returned PIDs
for /f "tokens=; usebackq" %%f in (`wt.exe --return-pid new-tab httpd.exe; split-pane -H --size .2 tail.exe -f /path/to/error.log`) do (
	set /a count += 1
	if !count! == 1 set /a httpd_pid = %%f
	if !count! == 2 set /a tail_pid = %%f
)

REM wait for as long httpd.exe is running
:waitforhttpd
tasklist /fi "pid eq %httpd_pid%" 2>nul | find "%httpd_pid%" >nul
if %ERRORLEVEL%==0 (
	timeout /t 2 /nobreak >nul
	goto :waitforhttpd
)

REM kill tail.exe
taskkill /PID %tail_pid%

endlocal

Example 2: Show log file with PID of commandline1
As such, it would also allow my other example, with the PID of httpd.exe in the path of the log file to be traced:
REM  #### Example 2: Show log file with PID of commandline1
REM
REM start httpd.exe and save PID
set /a httpd_pid = 0
for /f "tokens=* usebackq" %%f in (`wt.exe --return-pid new-tab httpd.exe 2>NUL`) do (
	if not "%%f" == "" set /a httpd_pid = %%f
)

REM now run tail.exe using the PID from httpd.exe in the log file path
if not "%httpd_pid%" == "0" wt.exe split-pane -H --size .2 tail.exe -f /path/to/error.%httpd_pid%.log

No protocol
On order to avoid the protocol in the STDOUT, --return-pid could also be limited to wt.exe calls that do only have one single command. Hence, wt new-tab --return-pid commandline1; new-tab commandline2 would cause a syntax error and fail. Such a command would have to be split into wt new-tab --return-pid commandline1 & wt --return-pid new-tab commandline2. Then there would only ever one integer or an empty string on STDOUT, if --return-pid is set.
My previous example 1 would then have to be slightly changed:
REM  #### Example 1b: Kill commandline2 once commandline1 has ended
REM
...
REM run the command and save the returned PIDs
for /f "tokens=* usebackq" %%f in (`wt.exe --return-pid new-tab httpd.exe 2>NUL`) do (
	if not "%%f" == "" set /a httpd_pid = %%f
)
for /f "tokens=* usebackq" %%f in (`wt.exe --return-pid split-pane -H --size .2 tail.exe -f /path/to/error.log 2>NUL`) do (
	if not "%%f" == "" set /a tail_pid = %%f
)
...

Conclusion

I specifically would like to run the processes in WT, thanks to its possibility to split tabs into panes and to "load" those panes with my commands
--wait would not help in my use-cases
--return-pid, however, would allow for great flexibility
at a later point, maybe that PID could even be used to identify a specific pane, e.g. to give it the focus with wt.exe focus-tab --child-pid %pid%

Sorry for leaving you on read for such a long time here. This is a solid writeup, with good scenarios. I still dunno how I feel about returning the PIDs raw on stdout. It's specifically good for this use case, sure. But it is also totally unprecedented territory here. We've got nothing else that uses wt's stdout at this time, and no grand plans for it yet. That actually makes me more reluctant to do something like this, specifically because it means if we ever do have some grand plan for stdout, we'd have to square it with this feature.
 I'm not sure I totally agree here. Although it might be a bit less ergonomic, you could always do something like
wt -w 0 --wait new-tab httpd.exe
wt -w 0 --wait split-pane -H --size .2 tail.exe -f /path/to/error.log

Basically, break the multiple subcommands into separate wt calls. The -w 0 is a flag to wt to tell it "run this command in the current/most recent window". (You could always use something like wt -w httpd or something, to have a specific "httpd" window).
I think when it comes to Example 2, now we're really pushing the boundaries of raw shell scripting here. This super feels like the kind of place where a more powerful language (python, go, heck even pwsh) might be called for.
As far as this request particularly, I think I'm inclined to just reject it in this form. There's probably better ways of going about doing what you're looking for. But returning the pids on stdout is probably not the right way.

@zadjii-msft Thank you for your reply. Much appreciated.
Also thank you for the reminder regarding the -w 0 flag.
Yet I'm sorry you're making the decision based on some potentially never used feature. Particularly, since there would be always the option to return a syntax error, as mentioned in my No Protocol section. If there would every be such a grand (or even not so grand) feature, that would collide with this, then there could be a documented incompatibility of the two features.
Anyway: Would it be an option to save the PID to an Environment Variable? In this case the "protocol" could be to either

number the variable names (e.g. WT_CHILD_PID_0, WT_CHILD_PID_1, WT_CHILD_PID_2 ..., or
have as many semicolons in the output variable, as the original command line, e.g.

WT_CHILD_PID=1234 for wt new-tab --return-pid commandline1
WT_CHILD_PID=1234;1235 for wt new-tab --return-pid commandline1; new-tab commandline2

If you could consider such a scenario, should I create a new request or shall we adapt this one to be more specific about the technical details?

I'm pretty sure environment variables wouldn't work either. wt could set it for itself, sure, but the environment of the calling shell (cmd, pwsh) would remain unchanged. At that point, reading the env vars of the spawned wt.exe would require something quite a bit lower level than CMD scripting 😝


CUDA Bindings

It may be useful to bind to the CUDA C interface. We will need to investigate what the licensing around CUDA is, and how useful just the C interface is.
From my vague memory, CUDA somehow hooks into the C(++) compiler and that way allows in line calling of shaders, which we can't somehow port.

The CUDA headers are included in the CUDA Toolkit, which we won't be able to bind as it's barred by a proprietary license agreement.

Thing is they explicitly mention Java and Python wrappers on their website. I have not yet figured out whether they have made those or whether they have a deal with library authors, or whether they somehow "left a hole" in the license

For the parallel heterogeneous compute scene, I believe that OpenCL covers the majority of use cases if OpenGL or Vulkan Compute Shaders aren't good enough for you. OpenCL is capable of using the "good SPIRV" i.e. the SPIRV that allows for creation of advanced compute kernels, not just plain old pipelined shader modules.
Some wrappers are open source under OSI-approved licenses, however it's sneaky because the moment you actually use CUDA yourself you become bound by the CUDA EULA. I don't think we can reasonably force users to sign this agreement (sure there's the NuGet "force acceptance" thing, but that isn't respected by all IDEs and doesn't really compare to NVIDIA actually locking the code behind a "please accept this EULA" barrier), and without doing so we'd set the .NET Foundation up for potentially a nasty bit of legal trouble.
The wrappers they refer to are likely also part of the CUDA Toolkit, which is EULA-barred. I don't think this is something the Silk.NET project can safely get involved in.
Moreover, from a moral standpoint we'd be a free and open-source project promoting access to a proprietary API that creates tie-ins to the NVIDIA GPU platform. We have no upside, and we are creating upsides (and potentially a revenue stream) for NVIDIA.
If, despite all this, we did want to take this further we should consult with the Foundation to get us in touch with a IP lawyer.
TL;DR Silk.NET can't safely implement this from a legal standpoint, and has nothing much to gain anyway.

Moreover, from a moral standpoint we'd be a free and open-source project promoting access to a proprietary API that creates tie-ins to the NVIDIA GPU platform.
I very much agree with this. Especially due to me personally not even being able to test whether it works 🙂

If a future reader wants this and has some reasoning why it makes sense to include this in Silk.NET they are free to comment on this issue.


chore(deps): Update all non-major client dependencies

This PR contains the following updates:

Package
Type
Update
Change

@angular-devkit/build-angular
devDependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular-eslint/builder (source)
devDependencies
patch
18.4.0 -> 18.4.1

@angular-eslint/eslint-plugin (source)
devDependencies
patch
18.4.0 -> 18.4.1

@angular-eslint/eslint-plugin-template (source)
devDependencies
patch
18.4.0 -> 18.4.1

@angular-eslint/schematics (source)
devDependencies
patch
18.4.0 -> 18.4.1

@angular-eslint/template-parser (source)
devDependencies
patch
18.4.0 -> 18.4.1

@angular/animations (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/cli
devDependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/common (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/compiler (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/compiler-cli (source)
devDependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/core (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/forms (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/platform-browser (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/platform-browser-dynamic (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@angular/router (source)
dependencies
patch
19.0.0-rc.1 -> 19.0.0-rc.3

@typescript-eslint/eslint-plugin (source)
devDependencies
minor
8.14.0 -> 8.15.0

@typescript-eslint/parser (source)
devDependencies
minor
8.14.0 -> 8.15.0

alpine
final
minor
3.18 -> 3.20

eslint (source)
devDependencies
minor
9.14.0 -> 9.15.0

node (source)

minor
v20.11.1 -> 20.18.0

tailwindcss (source)
devDependencies
patch
3.4.14 -> 3.4.15

Release Notes

angular/angular-cli (@angular-devkit/build-angular)
v19.0.0-rc.3
Compare Source
@schematics/angular

Commit
Type
Description

4b4e000dd
fix
don't show server routing prompt when using browser builder

9e6ab1bf2
fix
use default import for express

@angular-devkit/build-angular

Commit
Type
Description

ea5ae68da
fix
bring back style tags in browser builder

25d928b4f
fix
fix hanging terminal when browser-sync is not installed

@angular/build

Commit
Type
Description

2551df533
fix
fully disable component style HMR in JIT mode

@angular/ssr

Commit
Type
Description

4ecf63a77
fix
export PrerenderFallback

fb05e7f0a
fix
use wildcard server route configuration on the '/' route when the app router is empty

v19.0.0-rc.2
Compare Source
@angular/cli

Commit
Type
Description

52dcd551c
fix
support default options for multiselect list x-prompt

@angular/build

Commit
Type
Description

5a7a2925b
fix
add missing redirect in SSR manifest

53b6cd33c
fix
allow .js file replacements in all configuration cases

3602bbb77
fix
avoid overwriting inline style bundling additional results

172f3c25a
fix
improve URL rebasing for hyphenated Sass namespaced variables

@angular/ssr

Commit
Type
Description

280ebbda4
fix
support for HTTP/2 request/response handling

angular-eslint/angular-eslint (@angular-eslint/builder)
v18.4.1
Compare Source
This was a version bump only for builder to align it with other projects, there were no code changes.

angular-eslint/angular-eslint (@angular-eslint/eslint-plugin)
v18.4.1
Compare Source
This was a version bump only for eslint-plugin to align it with other projects, there were no code changes.

angular-eslint/angular-eslint (@angular-eslint/eslint-plugin-template)
v18.4.1
Compare Source
This was a version bump only for eslint-plugin-template to align it with other projects, there were no code changes.

angular-eslint/angular-eslint (@angular-eslint/schematics)
v18.4.1
Compare Source
🩹 Fixes

update dependency ignore to v6 (#2047)

angular-eslint/angular-eslint (@angular-eslint/template-parser)
v18.4.1
Compare Source
This was a version bump only for template-parser to align it with other projects, there were no code changes.

angular/angular (@angular/animations)
v19.0.0-rc.3
Compare Source
compiler

Commit
Type
Description

2be161d015
fix
fix :host parsing in pseudo-selectors (#58681)

806a61b5a6
fix
fix multiline selectors (#58681)

e5d3abb298
fix
resolve :host:host-context(.foo) (#58681)

80f56954ce
fix
transform chained pseudo-selectors (#58681)

migrations

Commit
Type
Description

e6514b9f3d
fix
do not migrate next calls in template if not an EventEmitter (#58631)

v19.0.0-rc.2
Compare Source
compiler

Commit
Type
Description

560282aa9b
fix
control flow nodes with root at the end projected incorrectly (#58607)

compiler-cli

Commit
Type
Description

791ee323dc
fix
correct extraction of generics from type aliases (#58548)

9bbb01c85e
fix
report individual diagnostics for unused imports (#58589)

core

Commit
Type
Description

71ee81af2c
fix
clean up event contract once hydration is done (#58174)

5fe57d4fbb
fix
fixes issues with control flow and incremental hydration (#58644)

51933ef5a6
fix
prevent errors on contract cleanup (#58614)

migrations

Commit
Type
Description

b84ed2b628
fix
include the output migration in the defaults of the signal migration (#58635)

e26797b38e
fix
replace removed NgModules in tests with their exports (#58627)

typescript-eslint/typescript-eslint (@typescript-eslint/eslint-plugin)
v8.15.0
Compare Source
🚀 Features

eslint-plugin: [prefer-nullish-coalescing] fix detection of ignoreConditionalTests involving boolean ! operator (#10299)
eslint-plugin: new rule no-unsafe-type-assertion (#10051)
eslint-plugin: added related-getter-setter-pairs rule (#10192)

🩹 Fixes

utils: add defaultOptions to meta in rule (#10339)
eslint-plugin: report deprecations used in default export (#10330)
eslint-plugin: [explicit-module-boundary-types] and [explicit-function-return-type] don't report on as const satisfies (#10315)
eslint-plugin: [await-thenable, return-await] don't flag awaiting unconstrained type parameter as unnecessary (#10314)
eslint-plugin: [consistent-indexed-object-style] handle circular mapped types (#10301)

❤  Thank You

Josh Goldberg ✨
Kim Sang Du @developer-bandi
Luis Sebastian Urrutia Fuentes @LuisUrrutia
Phillip Huang
Ronen Amiel
Szydlak @wszydlak

You can read about our versioning strategy and releases on our website.

typescript-eslint/typescript-eslint (@typescript-eslint/parser)
v8.15.0
Compare Source
This was a version bump only for parser to align it with other projects, there were no code changes.
You can read about our versioning strategy and releases on our website.

eslint/eslint (eslint)
v9.15.0
Compare Source
Features

01557ce feat: Implement Language#normalizeLanguageOptions() (#19104) (Nicholas C. Zakas)
2edc0e2 feat: add meta.defaultOptions (#17656) (Josh Goldberg ✨)
bdec50e feat: fix no-useless-computed-key false negative with __proto__ (#19123) (Milos Djermanovic)
3087c9e feat: add meta object to @eslint/js plugin (#19095) (Francesco Trotta)

Bug Fixes

fd33f13 fix: update types for no-restricted-imports rule (#19060) (Nitin Kumar)
bd35098 fix: switch away from Node.js node:assert and AssertionError (#19082) (Josh Goldberg ✨)
9db5b15 fix: unsafe report for no-lonely-if (#19087) (Abraham Guo)
68fa497 fix: ignore files on a different drive on Windows (#19069) (Francesco Trotta)
4ce625a fix: upgrade<EMAIL_ADDRESS>to avoid debug logs (#19102) (Milos Djermanovic)

Documentation

d927920 docs: fix styles in no-js mode (#18916) (Tanuj Kanti)
09bc2a8 docs: Update README (GitHub Actions Bot)
39089c8 docs: add no-useless-computed-key examples with object patterns (#19109) (Milos Djermanovic)
895c60f docs: add missing messageId property and suggestion properties (#19122) (fnx)
298625e docs: Change CLI -c to use flat config (#19103) (Nicholas C. Zakas)
522d8a3 docs: add deprecation on indent, quotes and semi rule types (#19090) (Marco Pasqualetti)

Chores

2967d91 chore: upgrade<EMAIL_ADDRESS>(#19133) (Milos Djermanovic)
b441bee chore: package.json update for @eslint/js release (Jenkins)
7d6bf4a chore: upgrade<EMAIL_ADDRESS>(#19131) (Milos Djermanovic)
902e707 chore: upgrade<EMAIL_ADDRESS>(#19130) (Milos Djermanovic)
5ff6c1d chore: bump cross-spawn (#19125) (Ian Bobinac)
cceccc7 chore: update dependency @eslint/json to ^0.6.0 (#19117) (renovate[bot])
0da3f73 chore: update algolia referrer (#19114) (Strek)
4f08332 ci: unpin trunk-io/trunk-action (#19108) (Francesco Trotta)
6b75683 perf: optimize text-table by replacing regex with trimEnd (#19061) (Nitin Kumar)

nodejs/node (node)
v20.18.0: 2024-10-03, Version 20.18.0 'Iron' (LTS), @targos
Compare Source
Notable Changes
Experimental Network Inspection Support in Node.js
This update introduces the initial support for network inspection in Node.js.
Currently, this is an experimental feature, so you need to enable it using the --experimental-network-inspection flag.
With this feature enabled, you can inspect network activities occurring within a JavaScript application.
To use network inspection, start your Node.js application with the following command:
$ node --inspect-wait --experimental-network-inspection index.js

Please note that the network inspection capabilities are in active development.
We are actively working on enhancing this feature and will continue to expand its functionality in future updates.

Network inspection is limited to the http and https modules only.
The Network tab in Chrome DevTools will not be available until the
feature request on the Chrome DevTools side is addressed.

Contributed by Kohei Ueno in #53593 and #54246
Exposes X509_V_FLAG_PARTIAL_CHAIN to tls.createSecureContext
This releases introduces a new option to the API tls.createSecureContext. From
now on, tls.createSecureContext({ allowPartialTrustChain: true }) can be used
to treat intermediate (non-self-signed) certificates in the trust CA certificate
list as trusted.
Contributed by Anna Henningsen in #54790
New option for vm.createContext() to create a context with a freezable globalThis
Node.js implements a flavor of vm.createContext() and friends that creates a context without contextifying its global
object when vm.constants.DONT_CONTEXTIFY is used. This is suitable when users want to freeze the context
(impossible when the global is contextified i.e. has interceptors installed) or speed up the global access if they
don't need the interceptor behavior.
Contributed by Joyee Cheung in #54394
Deprecations

[64aa31f6e5] - repl: doc-deprecate instantiating node:repl classes without new (Aviv Keller) #54842
[4c52ee3d7f] - zlib: deprecate instantiating classes without new (Yagiz Nizipli) #54708

Other Notable Changes

[b80da2f964] - buffer: optimize createFromString (Robert Nagy) #54324
[02b36cbd2d] - (SEMVER-MINOR) lib: add EventSource Client (Aras Abbasi) #51575
[879546a9bf] - (SEMVER-MINOR) src,lib: add performance.uvMetricsInfo (Rafael Gonzaga) #54413
[f789f4c92d] - (SEMVER-MINOR) test_runner: support module mocking (Colin Ihrig) #52848
[4eb0749b6c] - (SEMVER-MINOR) url: implement parse method for safer URL parsing (Ali Hassan) #52280

Commits

[013c48f0e9] - benchmark: --no-warnings to avoid DEP/ExpWarn log (Rafael Gonzaga) #54928
[194fc113ac] - benchmark: add buffer.isAscii benchmark (RafaelGSS) #54740
[7410d51cb9] - benchmark: add buffer.isUtf8 bench (RafaelGSS) #54740
[2393f21e8a] - benchmark: add access async version to bench (Rafael Gonzaga) #54747
[b8779721f0] - benchmark: enhance dc publish benchmark (Rafael Gonzaga) #54745
[4078aa83ff] - benchmark: add match and doesNotMatch bench (RafaelGSS) #54734
[66acab9976] - benchmark: add rejects and doesNotReject bench (RafaelGSS) #54734
[6db777fb3a] - benchmark: add throws and doesNotThrow bench (RafaelGSS) #54734
[8f101560ce] - benchmark: add strictEqual and notStrictEqual bench (RafaelGSS) #54734
[2c9e4c936e] - benchmark: adds groups to better separate benchmarks (Giovanni Bucci) #54393
[671c3ac633] - benchmark: fix benchmark for file path and URL conversion (Early Riser) #54190
[8c8708cb5b] - benchmark: use assert.ok searchparams (Rafael Gonzaga) #54334
[8b71fa79e2] - benchmark: add stream.compose benchmark (jakecastelli) #54308
[93ee36e3a0] - benchmark: rename count to n (Rafael Gonzaga) #54271
[f2971b6f0b] - benchmark: change assert() to assert.ok() (Rafael Gonzaga) #54254
[f48f2c212c] - benchmark: support --help in CLI (Aviv Keller) #53358
[0309b0520b] - benchmark: remove force option as force defaults to true (Yelim Koo) #54203
[b6e8305b2d] - benchmark: use assert.ok instead of assert (Rafael Gonzaga) #54176
[90c660d26a] - benchmark: add require-esm benchmark (Joyee Cheung) #52166
[1b8584b52e] - benchmark,doc: add CPU scaling governor to perf (Rafael Gonzaga) #54723
[0b9161b330] - benchmark,doc: mention bar.R to the list of scripts (Rafael Gonzaga) #54722
[84bf93b7ea] - buffer: allow invalid encoding in from (Robert Nagy) #54533
[d04246a0d7] - buffer: optimize byteLength for common encodings (Robert Nagy) #54342
[f36831f694] - buffer: optimize createFromString (Robert Nagy) #54324
[f5f40c8088] - buffer: optimize for common encodings (Robert Nagy) #54319
[76c37703be] - buffer: add JSDoc to blob bytes method (Roberto Simonini) #54117
[3012d31404] - buffer: use faster integer argument check (Robert Nagy) #54089
[3505782801] - buffer: make indexOf(byte) faster (Tobias Nießen) #53455
[d285fc1f68] - build: upgrade clang-format to v18 (Aviv Keller) #53957
[d288ec3b0a] - build: fix conflicting V8 object print flags (Daeyeon Jeong) #54785
[e862eecac9] - build: do not build with code cache for core coverage collection (Joyee Cheung) #54633
[f7a606eb96] - build: turn off -Wrestrict (Richard Lau) #54737
[71ca2665e4] - build: reclaim disk space on macOS GHA runner (jakecastelli) #54658
[82d8051c39] - build: don't clean obj.target directory if it doesn't exist (Joyee Cheung) #54337
[6e550b1f26] - build: update ruff to 0.5.2 (Aviv Keller) #53909
[e2ea7b26d7] - build: fix ./configure --help format error (Zhenwei Jin) #53066
[eb2402d569] - build: enable building with shared uvwasi lib (Pooja D P) #43987
[45732314d4] - build: sync V8 warning cflags with BUILD.gn (Michaël Zasso) #52873
[6e0a2bb54c] - build: harmonize Clang checks (Michaël Zasso) #52873
[3f78d4eb28] - cli: add --expose-gc flag available to NODE_OPTIONS (Juan José) #53078
[a110409b2a] - console: use validateOneOf for colorMode validation (HEESEUNG) #54245
[231ab788ea] - crypto: reject dh,x25519,x448 in {Sign,Verify}Final (Huáng Jùnliàng) #53774
[a5984e4570] - crypto: return a clearer error when loading an unsupported pkcs12 (Tim Perry) #54485
[f287cd77bd] - crypto: remove unused kHashTypes internal (Antoine du Hamel) #54627
[1fc904f8c4] - deps: update cjs-module-lexer to 1.4.1 (Node.js GitHub Bot) #54846
[95b55c39b1] - deps: update simdutf to 5.5.0 (Node.js GitHub Bot) #54434
[cf6ded5dd3] - deps: update cjs-module-lexer to 1.4.0 (Node.js GitHub Bot) #54713
[7f8edce3f1] - deps: update c-ares to v1.33.1 (Node.js GitHub Bot) #54549
[9a4a7b7ecc] - deps: update undici to 6.19.8 (Node.js GitHub Bot) #54456
[87ca1d7fee] - deps: update simdutf to 5.3.4 (Node.js GitHub Bot) #54312
[d3a743f182] - deps: update zlib to <IP_ADDRESS>-motley-71660e1 (Node.js GitHub Bot) #53464
[926981aa9f] - deps: update zlib to <IP_ADDRESS>-motley-c2469fd (Node.js GitHub Bot) #53464
[654c8d1fdc] - deps: update zlib to <IP_ADDRESS>-motley-68e57e6 (Node.js GitHub Bot) #53464
[2477e79172] - deps: update zlib to <IP_ADDRESS>-motley-8b7eff8 (Node.js GitHub Bot) #53464
[3d8113faf5] - deps: update zlib to <IP_ADDRESS>-motley-e432200 (Node.js GitHub Bot) #53464
[ac294e3db4] - deps: update zlib to <IP_ADDRESS>-motley-887bb57 (Node.js GitHub Bot) #53464
[239588b968] - deps: update c-ares to v1.33.0 (Node.js GitHub Bot) #54198
[6e7de37ed3] - deps: update undici to 6.19.7 (Node.js GitHub Bot) #54286
[38aa9d6ea9] - deps: update acorn to 8.12.1 (Node.js GitHub Bot) #53465
[d30145f663] - deps: update undici to 6.19.5 (Node.js GitHub Bot) #54076
[c169d9c12b] - deps: update simdutf to 5.3.1 (Node.js GitHub Bot) #54196
[92f3447957] - doc: add missing EventSource docs to globals (Matthew Aitken) #55022
[2879ce9681] - doc: fix broken Android building link (Niklas Wenzel) #54922
[096623b59a] - doc: add support link for aduh95 (Antoine du Hamel) #54866
[1dfd238781] - doc: run license-builder (github-actions[bot]) #54854
[a6c748fffb] - doc: experimental flag for global accessible APIs (Chengzhong Wu) #54330
[d48a22fa14] - doc: add ERR_INVALID_ADDRESS to errors.md (Aviv Keller) #54661
[4a840cecfa] - doc: add support link for mcollina (Matteo Collina) #54786
[ec22d86512] - doc: mark --conditions CLI flag as stable (Guy Bedford) #54209
[77c702ca07] - doc: fix typo in recognizing-contributors (Tobias Nießen) #54822
[62953ef9fb] - doc: clarify --max-old-space-size and --max-semi-space-size units (Alexandre ABRIOUX) #54477
[e2bab0f2b2] - doc: replace --allow-fs-read by --allow-fs-write in related section (M1CK431) #54427
[9cbfd5b33a] - doc: add support link for marco-ippolito (Marco Ippolito) #54789
[53167b29ef] - doc: fix typo (Michael Dawson) #54640
[87f78a35f7] - doc: fix webcrypto.md AES-GCM backticks (Filip Skokan) #54621
[7c83c15221] - doc: add documentation about os.tmpdir() overrides (Joyee Cheung) #54613
[4bfd832d70] - doc: add support me link for anonrig (Yagiz Nizipli) #54611
[22a103e5ec] - doc: add alert on REPL from TCP socket (Rafael Gonzaga) #54594
[b6374c24e1] - doc: fix typo in styleText description (Rafael Gonzaga) #54616
[2f5b98ee1f] - doc: add getHeapStatistics() property descriptions (Benji Marinacci) #54584
[482302b99b] - doc: fix information about including coverage files (Aviv Keller) #54527
[b3708e7df4] - doc: support collaborators - talk amplification (Michael Dawson) #54508
[c86fe23012] - doc: add note about shasum generation failure (Marco Ippolito) #54487
[d53e6cf755] - doc: fix capitalization in module.md (shallow-beach) #54488
[cdc6713f18] - doc: add esm examples to node:https (Alfredo González) #54399
[1ac1fe4e65] - doc: fix error description of the max header size (Egawa Ryo) #54125
[244542b720] - doc: add git node security --cleanup (Rafael Gonzaga) #54381
[69fb71f54c] - doc: add note on weakness of permission model (Tobias Nießen) #54268
[83b2cb908b] - doc: add versions when --watch-preserve-output was added (Théo LUDWIG) #54328
[460fb49483] - doc: replace v19 mention in Current release (Rafael Gonzaga) #54361
[994b46a160] - doc: correct peformance entry types (Jason Zhang) #54263
[f142e668cb] - doc: fix typo in method name in the sea doc (Eliyah Sundström) #54027
[9529a30dba] - doc: mark process.nextTick legacy (Marco Ippolito) #51280
[7e25fabb91] - doc: add esm examples to node:http2 (Alfredo González) #54292
[6a4f05e384] - doc: explicitly mention node:fs module restriction (Rafael Gonzaga) #54269
[53f5c54997] - doc: warn for windows build bug (Jason Zhang) #54217
[07bde054f3] - doc: make some parameters optional in tracingChannel.traceCallback (Deokjin Kim) #54068
[62bf03b5f1] - doc: add esm examples to node:dns (Alfredo González) #54172
[fb2b19184b] - doc: add KevinEady as a triager (Chengzhong Wu) #54179
[24976bfba0] - doc: add esm examples to node:console (Alfredo González) #54108
[4e7edc40f7] - doc: fix sea assets example (Sadzurami) #54192
[322b5d91e1] - doc: add links to security steward companies (Aviv Keller) #52981
[6ab271510e] - doc: move onread option from socket.connect() to new net.socket() (sendoru) #54194
[39c30ea08f] - doc: move release key for Myles Borins (Richard Lau) #54059
[e9fc54804a] - doc: refresh instructions for building node from source (Liran Tal) #53768
[f131dc625a] - doc: add documentation for blob.bytes() method (jaexxin) #54114
[8d41bb900b] - doc: add missing new lines to custom test reporter examples (Eddie Abbondanzio) #54152
[2acaeaba77] - doc: update list of Triagers on the README.md (Antoine du Hamel) #54138
[fff8eb2792] - doc: expand troubleshooting section (Liran Tal) #53808
[402121520f] - doc: clarify useCodeCache setting for cross-platform SEA generation (Yelim Koo) #53994
[272484b8b2] - doc: test for cli options (Aras Abbasi) #51623
[c4d0ca4710] - doc, build: fixup build docs (Aviv Keller) #54899
[2e3e17748b] - doc, child_process: add esm snippets (Aviv Keller) #53616
[c40b4b4f27] - doc, meta: fix broken link in onboarding.md (Aviv Keller) #54886
[beff587b94] - doc, meta: add missing , to BUILDING.md (Aviv Keller) #54409
[c114585430] - doc, meta: replace command with link to keys (Aviv Keller) #53745
[0843077a99] - doc, test: simplify test README table (Aviv Keller) #53971
[2df7bc0e32] - doc,tools: enforce use of node: prefix (Antoine du Hamel) #53950
[0dd4639391] - esm: fix support for URL instances in import.meta.resolve (Antoine du Hamel) #54690
[f0c55e206d] - fs: refactor rimraf to avoid using primordials (Yagiz Nizipli) #54834
[f568384bbd] - fs: refactor handleTimestampsAndMode to remove redundant call (HEESEUNG) #54369
[2fb7cc9715] - fs: fix typings (Yagiz Nizipli) #53626
[596940cfa0] - http: reduce likelihood of race conditions on keep-alive timeout (jazelly) #54863
[6e13a7ba02] - http: remove prototype primordials (Antoine du Hamel) #53698
[99f96eb3f7] - http2: remove prototype primordials (Antoine du Hamel) #53696
[41f5eacc1a] - https: only use default ALPNProtocols when appropriate (Brian White) #54411
[[59a39520e1](https://red

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Angular 19 released https://github.com/angular/angular/releases/tag/19.0.0 🎊
Within this PR maybe we can directly upgrade to version 19 (instead of release candidate) and do some cleaning of the standalone: true from the components since it is by default true. Also, we can set strictStandalone: true under tsconfig. json --> angularCompilerOptions as well


checkUserByEmail in App\Repositories\Backend\Access\User\UserRepository

Dear guys,
I think you should add withTrashed() in the query as it give an error if email already exists and trashed
BR
David

and you should also add this test in the create function

@invaders-xx I am planning to add Laravel dusk test cases, with that I'll check this as well.

@invaders-xx  Your suggestion is implemented. Thanks.


Bug 54108923: [React Native Gallery -> Radio Buttons]: Screen reader is not announcing the associated group name and selection state of the radio button.

Build Details:
OS Version: Dev (OS Build 27695.1000)
App: React Native Gallery
App Version: <IP_ADDRESS>
React Native Windows Version: 0.75.0
AT: Screen Reader
Pre-Requisite:
Turn on Narrator (Ctrl + Win + Enter).
Repro-Steps:
Launch the React Native Gallery app.
Navigate to ‘All samples’ button present in left navigation pane and invoke it.
Navigate to ‘Radio Buttons’ button present under “Basic Input” group and invoke it.
Navigate to radio buttons and listen to the narrator's announcement.
Actual Result:
Screen reader is not announcing the associated group name and selection state of the radio button. When user navigated to the radio buttons, narrator is simply announcing the information about the radio button and it not announcing the label name and state of the button.
Expected Result:
Label “Options” should be associated with radio group and screen reader should announce the selection state of the radio button. When user navigates to the radio button, narrator should announce as "Options, Option 1 radio button, 1 of 3, selected".
User Impact:
Users with visual impairment who rely on screen reader will not be able to know the label while navigating to the radio group if screen reader does not announce that.
Link to ADO bug: https://dev.azure.com/microsoft/OS/_workitems/edit/54108923

Role is set but isn't creating the right announce behavior around the group/parent. May be a platform issue assuming everything is tagged correctly.


[MSYS2] Disable download timeout (test)

test

pacman-5.2.1-7 is up to date -- reinstalling
looking for conflicting packages...

Packages (1) pacman-5.2.1-7

Total Installed Size:  38.38 MiB
Net Upgrade Size:       0.00 MiB

:: Proceed with installation?
:: Processing package changes...
reinstalling pacman...
error: cannot remove /usr/bin/pacman.exe (Permission denied)
warning: warning given when extracting /usr/bin/pacman.exe (Could not unlink)

@BillyONeal
???

That's indeed very strange, I don't know why CI didn't run on this. But given that CI keeps getting broken every time something that touches msys runs I'd prefer one of those other PRs get merged soon...

 That might make Azure Pipelines less confused but I'm ot positive

Looks like that got it unstuck. We'll see if this wedges the builders like everything else that's touched msys as of late


Attempt to build the volto/components Storybook only when something c…

…hanges in its directory

@sneridagh I don't know whether this will actually work until it is merged to main and it has a cached commit ref, but this is the basic idea. We can look at the Netlify logs (for example, https://app.netlify.com/sites/plone-components/deploys/65953f2fe56175000813fb55#L11) for this site to see whether it does what we want, and fiddle around with it. Sometimes we have to use lots of echo commands to troubleshoot.


css build  isn't inserted into jade file

Got this build in my jade file:
doctype html
html
    head
        link(href='//fonts.googleapis.com/css?family=Open+Sans:400,300,700,600' rel='stylesheet' type='text/css')

        //- build:css /public/css/app/app.css
        link(rel='stylesheet', href='/public/app/main.css')
        //- endbuild

        meta(name="google" content="notranslate")
        meta(name="viewport" content="width=device-width, initial-scale=1, user-scalable=no")

And the file this gulp task:
gulp.task('usemin', function() {
  gulp.src('./server/views/app/*.jade')
    .pipe(usemin({
      css: [uglifycss(), rev()],
      js: [uglify(), rev()],
      assetsDir: 'server'
    }))
    .pipe(gulp.dest('server/views/dist'));
});

The uglified css file is generated just fine and it's given a revisioned file name and stored at the correct location. But the  tag disappears from the jade output file. Anybody know what's going on? I've got the latest version. And it's working just fine with javascript build.
output:
doctype html
html
    head
        link(href='//fonts.googleapis.com/css?family=Open+Sans:400,300,700,600' rel='stylesheet' type='text/css')

        meta(name="google" content="notranslate")
        meta(name="viewport" content="width=device-width, initial-scale=1, user-scalable=no")

it's working in 1.0.5

reason it was changed: https://github.com/niftylettuce/gulp-jade-usemin#110


Add languages to WebUI

Description:
Add existing languages to WebUI
French
German

Investigate this issue and if it's possible to implement then can go forward.
We probably need some strings translated.

Good find. "Calibrate" and "Clear" in the offset screen was hardcoded. The name dialog is not hardcoded, "Update" is correct for de according to the localization repo

I think we use aktualisieren in de, changed with pull request.

QA passed


Add get specifications methods

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Adds the get_specifications_value and get_specifications_values methods.
Why should this Pull Request be merged?
Needed to complete #160
What testing has been done?
Created new system test to validate methods.

[x] I have run the automated tests (required if there are code changes)

/azp run

/azp run

Please update STATUS.md to include information for GetSpecificationsValues.

Please update STATUS.md to include information for GetSpecificationsValues.

I also took the opportunity to fix some of the formatting in STATUS.md

/azp run


Twilio Integration!

Related Issue or Background Info

Resolves #829
Blocked by #813

Changes Proposed

Adds Twilio SDK and relevant SMSController & Service
Disables CSRF in SecurityConfiguration
Exposes the Patient Experience endpoints for anonymous users
Adds Terraform entries for Twilio secrets

Additional Information

Please note that #919 was auto-closed in the midst of a shoddy rebase & merge situation. 🤦

@jeskew-gov Resurfacing our conversation on the previous PR, which I accidentally auto-merged (into my other working branch) during a rebase. You asked if CSFR needed to be disabled solely due to SmsController, and I replied:

No, disabling CSRF is also necessary for PatientExperienceController – I hadn't ever scanned the QR code on a mobile device in a deployed environment – so we didn't discover that it didn't work until this integration.
Regardless it's a very small lift to turn this controller into a mutation resolver and add it to the GQL schema – just lmk

Would you prefer that SmsController be represented as a mutation in the GraphQL schema?

@jeskew-gov Resurfacing our conversation on the previous PR, which I accidentally auto-merged (into my other working branch) during a rebase. You asked if CSFR needed to be disabled solely due to SmsController, and I replied:

No, disabling CSRF is also necessary for PatientExperienceController – I hadn't ever scanned the QR code on a mobile device in a deployed environment – so we didn't discover that it didn't work until this integration.
Regardless it's a very small lift to turn this controller into a mutation resolver and add it to the GQL schema – just lmk

Would you prefer that SmsController be represented as a mutation in the GraphQL schema?

That would be my preference even though this operation is only triggering a side effect and not altering data. For me at least, it helps conceptually if we only break out of the GraphQL service when necessary.


added json format

Per @robth-microsoft: This is still not updated. Need to have all the text above that describes both file types.


ACM-15294 - Add Velero properties to the ACM BackupSchedule CRD

Description
Enable Velero settings on ACM schedules
Related Issue
https://issues.redhat.com/browse/ACM-15294
Changes Made
Added the following Velero properties to the ACM BackupSchedule

skipImmediately
useOwnerReferencesInBackup

Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[x] I have tested the changes locally and they are functioning as expected.
[x] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[x] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[x] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/cc @dislbenn

/lgtm


PostgreSQL - Display internally generated constraint triggers

In PostgreSQL, it could be greate to see internally generated triggers.
For exemple, Foreign Key uses internally generated trigger.
Those can be find in the view : pg_trigger  which have a column tgisinternal to check if the trigger is internal.
When you use ALTER TABLE DISABLE TRIGGER ALL, it disable those trigger (but ALTER TABLE DISABLE TRIGGER USER not). Seeing those triggers can alert on what you might do by disabling all triggers.

Hello @bazmap
Could you please provide DDL examples?

Yeah shure :
CREATE CONSTRAINT TRIGGER "RI_ConstraintTrigger_a_34278"
AFTER DELETE ON my_schema."my_table"
FROM my_schema."my_other_table"
NOT DEFERRABLE INITIALLY IMMEDIATE
FOR EACH ROW
EXECUTE PROCEDURE "RI_FKey_cascade_del"()

Those trigger are listed in the pg_trigger view :
SELECT
	pg_class.relnamespace::regnamespace AS "schema",
	pg_class.relname AS "table",
	pg_trigger.tgname,
	pg_get_triggerdef(pg_trigger.oid) AS definition
FROM
	pg_catalog.pg_trigger
LEFT JOIN
	pg_class
	ON pg_class.oid = pg_trigger.tgrelid
WHERE
	pg_trigger.tgisinternal = TRUE
;

For me there is no need to refer to the underlying procedure but having a link to the constraint can be useful

Okey, thanks for the idea.


Make Provider<> for things using in configuration safer

Have Gradle more explicitly support Providers/Properties used at configuration
Expected Behavior
There are places where the Android Gradle Plugin reads properties that it exposes as API during configuration time, that currently we don't it'd be great if there was a way to construct a 'configuration-time' property that does something along the lines of:

Disallows setting to anything that carries a task or similar dependency
Explicitly allows the creating plugin to call get() during configuration, when that hopefully gets restricted to be during execution.

Current Behavior
Calling Provider.get() is allowed at configuration (if disallowUnsafeRead is not used), and will return potentially incorrect/broken results if it's made by mapping {} reading a task output file.
Context
The Android Gradle Plugin makes a lot of decisions in afterEvaluate (so Properties are especially useful for avoiding
brittle ordering - one plugin can set a value and another can use it as a task input).
However some properties, we want to make decisions at configuration time, and it'd be good

Disallows setting to anything that carries a task or similar dependency

I wonder how this part could work, given that you can set the property to some arbitrary Provider, and then later link that Provider to a task output. To detect this we'd need to check the downstream consumers of a Provider every time its value changes.

Disallows setting to anything that carries a task or similar dependency

I wonder how this part could work, given that you can set the property to some arbitrary Provider, and then later link that Provider to a task output. To detect this we'd need to check the downstream consumers of a Provider every time its value changes.

That's a good point - From AGP perspective I think we'd be happy to have this be more restrictive and only allow setting to other properties explicitly marked as 'configuration-time' so we can give the error when 'set' is called. Or we could just allow it to fail on the get call, but that is less useful for the build author to understand what happened as the error will not be directly related to their code.

I think we'd be happy to have this be more restrictive and only allow setting to other properties explicitly marked as 'configuration-time' so we can give the error when 'set' is called.

I think that's a really great point. I could see an API where you have to make all Property/Provider be flagged the same. This way if a 3P plugin uses one of its own Property (which is exposed as part of their API/DSL) to set the value in ours, theirs also needs to be queriable at config time and therefore should refuse any set on them that contains a task dependency.


CI: Install additional requirements from atest/requirements-run.txt

Codecov Report

Merging #3533 into master will decrease coverage by 1.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3533      +/-   ##
==========================================
- Coverage   73.63%   72.43%   -1.20%
==========================================
  Files         199      197       -2
  Lines       16115    16032      -83
  Branches     2642     2636       -6
==========================================
- Hits        11865    11611     -254
- Misses       3817     3955     +138
- Partials      433      466      +33

Impacted Files
Coverage Δ

src/robot/running/libraryscopes.py
100.00% <100.00%> (ø)

src/robot/utils/encoding.py
54.91% <0.00%> (-29.41%)
:arrow_down:

src/robot/utils/unic.py
69.85% <0.00%> (-26.98%)
:arrow_down:

src/robot/running/runkwregister.py
68.75% <0.00%> (-25.00%)
:arrow_down:

src/robot/utils/compat.py
52.78% <0.00%> (-22.22%)
:arrow_down:

src/robot/utils/encodingsniffer.py
63.08% <0.00%> (-21.53%)
:arrow_down:

src/robot/utils/robottypes.py
81.25% <0.00%> (-18.75%)
:arrow_down:

src/robot/utils/robotpath.py
56.61% <0.00%> (-16.98%)
:arrow_down:

src/robot/version.py
55.56% <0.00%> (-11.11%)
:arrow_down:

src/robot/utils/etreewrapper.py
74.33% <0.00%> (-8.10%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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

Thanks @tysonite!


PWA install on iPad gets stuck in auth flow

Steps to Reproduce:

install PWA on iPad
open it and open a github repo to trigger the auth flow
go through the flow
when you finish, you end up on the vscode.dev callback page and it doesn't close automatically. It also doesn't show any chrome to close the page and return back to the editor.

This is because we have been opening the page with noopener for safari:
https://github.com/microsoft/vscode/blob/main/src/vs/base/browser/dom.ts#L1200
for security reasons, this is a good idea... however, we should allow an embedder like vscode.dev to specify trusted urls that are allowed to open with the opener set so that window.close() will work.
If you're curious why noopener is needed for this case, look at the html spec:
https://html.spec.whatwg.org/multipage/window-object.html#dom-window-close

verification: follow steps, you shouldn't get stuck you should end up back in vscode.dev.

worked fine for me.


Stage directory field doesn't work (SOURCE)

Describe your problem here.
So I made a stage with a custom directory called week1, I added the folder with all the images. When I load it in the game for some reason Paths.hx returns null? And it's all black and every object for the stage has been replaced with the HaxeFlixel logo
This is the json
{
	"directory": "week1",
	"defaultZoom": 0.9,
	"isPixelStage": false,

	"boyfriend": [580.6, 521],
	"girlfriend": [400, 130],
	"opponent": [100, 100],
	"hide_girlfriend": false,

	"camera_boyfriend": [-200, 0],
	"camera_opponent": [0, 0],
	"camera_girlfriend": [0, 0],
	"camera_speed": 1
}

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Opening cutscene

please provide the full directory for it

Extra info
so i used trace(path) and this is what I found.
source/Paths.hx:360: assets/images/bg.png
source/Paths.hx:370: oh no its returning null NOOOO
source/Paths.hx:360: assets/images/lights.png
source/Paths.hx:370: oh no its returning null NOOOO
source/Paths.hx:360: assets/images/columns.png
source/Paths.hx:370: oh no its returning null NOOOO
source/Paths.hx:360: assets/images/MirrorWorld.png

instead of searching in the week1 folder it's using the assets one? I have no idea why

that week1/ folder has to be inside assets but without being inside preload/ or shared/
Also, the images go in the folder images/ inside week1

If that doesn't work, try changing the path on the stage sprite itself on the source code
And make sure the folder is defined in project.xml

I got help we had to modify BG Sprite.hx and PlayState by adding a new argument
var mirrors:BGSprite = new BGSprite('bg', 'week1', 960, 417.3, 0.9, 0.9);


Release for androidx

Hi there!
First of all, thank you for the great library.
I wanted to ask if it is done intentionally, that the latest available release is not yet using androidx Activities and Fragments?

Hello,
any update on this topic? Will this library be further maintained?


Add support for Cronos zkEVM Testnet

Chain Name
Cronos zkEVM Testnet
Chain ID
282
RPC URL
https://testnet.zkevm.cronos.org
Block Explorer URL
https://explorer.zkevm.cronos.org/testnet
Deployment Funds
Yes

Since this is a zkEVM, can you confirm (or point to docs) that confirms that address derivation is the same as on Ethereum? I want to verify that deploying with nonce 0 will give the expected address

I am not sure where that exact info is, but I'm hoping it is in these official developer docs: https://docs-zkevm.cronos.org/for-developers/resources-for-developers

Asked Cronos team member, let's hope he answers :)
https://x.com/0xArtCro/status/1824227575889035460

Closing until we know this answer


256/truecolor support for :terminal?

-> Should be off by default
-> Enable it with a togglable option
-> (Set t_XX codes if needed, not needed in gvim?)
It would (probably) slow down performance, hence why I'm suggesting it as an optional eyecandy feature

Can you explain what you mean?  What aspect of 256 or truecolor doesn't already work?

The :terminal command only supports 16 colors. Vim could use the appropriate escape sequences (as controlled by the t_xx options) to display 256/truecolor in a gui / terminal with proper color support. As I've said, this is very much optional eyecandy.

The :terminal command only supports 16 colors

I don't think that's true.

The :terminal command only supports 16 colors

I don't think that's true.

I don't see the helpfiles stating otherwise, and on my Windows machine both Terminal and GUI Vim (from the vim-win32-installer project, latest release) only show 16 colors (in particular, 256/truecolor codes are mapped to the 16 color palette provided b the colorscheme)
Perhaps it is a Windows only issue, but I doubt that somewhat.

Your example appears to work fine for me (both in terminal and gvim), but then I'm on linux.

Oh Bummer! I guess it's a Windows only issue then.

When I use :terminal on macOS, the resulting custom bash prompt exhibits truecolors.

As said. Appears to be a windows only issue. Dunno if this fixable, if not this can he closed, else please rename it appropriately (And I apologise for assuming this was across all systems without testing beforehand, despite having a working Linux system.)

I guess it's a Windows only issue then.

Yes. Current version of Windows doesn't provide a way to capture 256 or full colors.
Next version of Windows (1810) will provide new ConPTY API. If Vim or winpty supports the new API, 256 or full colors might be enabled. Currently @ntak is working on it.

Link: #3474

Link: #3794

This should be fixed in 8.1.0870.

The ConPTY support was disabled by default in 8.1.0909, because ConPTY in Win10 1809 is considered still unstable.
To use ConPTY, set 'termwintype' to "conpty" or remove winpty(32/64).dll from the PATH.

ConPTY in Win10 1809 is considered still unstable
well, not by Microsoft.

Should we re-enable it by default for 1903 (build number 10.0.18362.0)? It's improved quite a bit since 1809.

https://github.com/microsoft/terminal/issues/370 is not fixed yet.

That doesn’t necessarily mean it's unstable, just that it doesn’t work
100% as expected for 100% of inputs. It’s a different set of trade-offs:
winpty has some issues with fidelity and color reproduction and
performance, and conpty has some lingering issues with character sizing.
Does that mean winpty is unstable? Of course not!

The main reason why we disable conpty by default is that issue.
It is a critical problem for CJK users.

This might be off topic but I notice that when I a using a python REPL Gvim only shows colors in Normal mode.
Insert mode:

Normal mode:


[UpdatePlugin]  404d autoutils v0.2

Please update my plugin:
Author: Me
Repo name: autoutils
Release: v0.2
I might have forgotten to check where binja gets its description before sending it in initially.

Previously added just not closed.


[WIP] Optimizing _rand

Samples have a _rand value that is populated at sample-creation time. This value is used to implement randomized operations like take() and shuffle().
The _rand values are currently set by generating random numbers seeded by the filepath of the samples. This works, but is slow and results in a noticeable decrease in add-sample-to-dataset time.
This PR is a WIP exploration to optimize that.

Update on this PR: the tests in tests/isolated/rand_tests.py suggest that there's not too much difference between the various options. So, this change likely would not improve efficiency much


Uncaught YAMLException: duplicated mapping key at line 32, column 24:    show_social_icons: true...

[Enter steps to reproduce below:]

Installed jekyll package in atom
opened a jekyll post file
selected Packages/Jekyll/Toggle Server menu item

Atom Version: 1.9.6
System: Mac OS X 10.11.6
Thrown From: jekyll package, v1.4.2
Stack Trace
Uncaught YAMLException: duplicated mapping key at line 32, column 24:
show_social_icons: true
^
At /Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:168

YAMLException: duplicated mapping key at line 32, column 24:
    show_social_icons: true
                           ^
    at generateError (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:162:10)
    at throwError (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:168:9)
    at storeMappingPair (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:305:7)
    at readBlockMapping (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1065:9)
    at composeNode (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1326:12)
    at readDocument (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1488:3)
    at loadDocuments (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1544:5)
    at load (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1561:19)
    at Object.safeLoad (/Users/sarah/.atom/packages/jekyll/node_modules/js-yaml/lib/js-yaml/loader.js:1579:10)
    at Object.module.exports.handleConfigFileOpen (/Users/sarah/.atom/packages/jekyll/lib/jekyll/utils.coffee:25:40)
    at /Users/sarah/.atom/packages/jekyll/lib/jekyll/utils.coffee:20:21
    at FSReqWrap.oncomplete (fs.js:82:15)

Commands
     -0:14.4.0 markdown-writer:manage-post-tags (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -0:00.0 jekyll:toggle-server (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "ignoredNames": [
      ".hg",
      ".svn",
      ".DS_Store",
      "._*",
      "Thumbs.db"
    ],
    "themes": [
      "one-light-ui",
      "solarized-light-syntax"
    ]
  }
}

Installed Packages
# User
atom-beautify, v0.29.10 (inactive)
jekyll, v1.4.2 (active)
markdown-writer, v2.5.1 (inactive)
pigments, v0.31.2 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.51.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.43.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.3.2 (inactive)
one-light-ui, v1.3.2 (active)
one-dark-syntax, v1.2.0 (inactive)
one-light-syntax, v1.2.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (active)
about, v1.5.2 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.1 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.0 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.0 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.38.3 (active)
fuzzy-finder, v1.3.0 (active)
git-diff, v1.1.0 (active)
find-and-replace, v0.198.0 (inactive)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.0 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.0 (active)
metrics, v0.53.1 (active)
notifications, v0.64.1 (active)
open-on-github, v1.2.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.238.0 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.4.1 (active)
styleguide, v0.46.0 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.98.0 (active)
timecop, v0.33.1 (active)
tree-view, v0.208.0 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.32.2 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.0 (active)
language-csharp, v0.12.1 (active)
language-css, v0.36.2 (active)
language-gfm, v0.86.0 (active)
language-git, v0.13.0 (active)
language-go, v0.42.0 (active)
language-html, v0.44.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.22.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.0 (active)
language-less, v0.29.3 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.0 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.68.5 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.52.0 (active)
language-shellscript, v0.22.3 (active)
language-source, v0.9.0 (active)
language-sql, v0.21.1 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.8 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

Sounds like you have show_social_icons more than once in your YAML.
Can you post your _config.yml?


soundmodem

Project description
Audio based modem for ham radio supporting ax.25
Metadata

homepage URL: http://soundmodem.vk4msl.id.au/
source URL:http://soundmodem.vk4msl.id.au/soundmodem-0.18.tar.gz
license: GPL v2
platforms: unix, linux, darwin, windows

I'm trying to build it right now, but the builder throws following error:
In file included from gendf9icfilt.cc:39:
../matlib/mat.hh: In function 'void mdet(const T*, unsigned int)':
../matlib/mat.hh:101:24: error: return-statement with a value, in function returning 'void' [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-fpermissive-fpermissive8;;]
  101 |                 return 0;
      |                        ^
make[2]: *** [Makefile:349: gendf9icfilt.o] Error 1
make[2]: Leaving directory '/build/soundmodem-0.18/fsk'
make[1]: *** [Makefile:348: all-recursive] Error 1
make[1]: Leaving directory '/build/soundmodem-0.18'
make: *** [Makefile:288: all] Error 2
builder for '/nix/store/pjmpzcm01191h6ic75afa6lrd28wb2n3-soundmodem.drv' failed with exit code 2

As I haven't worked with C++, I don't know if this is a gcc issue or an error in the source code, could you please check?
I guess changing the return value might fix it.

I'm trying to build it right now, but the builder throws following error:
In file included from gendf9icfilt.cc:39:
../matlib/mat.hh: In function 'void mdet(const T*, unsigned int)':
../matlib/mat.hh:101:24: error: return-statement with a value, in function returning 'void' [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-fpermissive-fpermissive8;;]
  101 |                 return 0;
      |                        ^
make[2]: *** [Makefile:349: gendf9icfilt.o] Error 1
make[2]: Leaving directory '/build/soundmodem-0.18/fsk'
make[1]: *** [Makefile:348: all-recursive] Error 1
make[1]: Leaving directory '/build/soundmodem-0.18'
make: *** [Makefile:288: all] Error 2
builder for '/nix/store/pjmpzcm01191h6ic75afa6lrd28wb2n3-soundmodem.drv' failed with exit code 2

As I haven't worked with C++, I don't know if this is a gcc issue or an error in the source code, could you please check?
I guess changing the return value might fix it.

Ok, looks like I've managed to fix that but there's more errors popping up. I'm sorry, but I don't have time and the knowledge to fix a package that also has issues on the ArchLinux AUR according to the last comment there from 2015.
If you have the time and knowledge of C++, please fix it and I'll be happy to continue with the package. I won't close the issue for now, but I won't continue working on it until these issues are fixed.

Ok, looks like I've managed to fix that but there's more errors popping up. I'm sorry, but I don't have time and the knowledge to fix a package that also has issues on the ArchLinux AUR according to the last comment there from 2015.
If you have the time and knowledge of C++, please fix it and I'll be happy to continue with the package. I won't close the issue for now, but I won't continue working on it until these issues are fixed.

That's right though, it is configured by running soundmodemconfig first.

That's right though, it is configured by running soundmodemconfig first.

Ok, I guess this is working now, feel free to open a new issue or reopen if there are any other issues popping up!

Ok, I guess this is working now, feel free to open a new issue or reopen if there are any other issues popping up!


Implement Presentation Exchange

There are various presentation exchange elements that need to be implemented in Swift. Not entirely sure what exactly needs to be implemented right now, but Kotlin only seems to be usingPresentationExchange.satisfiesPresentationDefinition. Let's start there!

All of these are shown in our developer guides:

PresentationExchange.selectCredentials
PresentationExchange.validateDefinition
PresentationExchange.satisfiesPresentationDefinition
PresentationExchange.evaluatePresentation
PresentationExchange.createPresentationFromCredentials
PresentationExchange.validateSubmission


Lighthouse CI action

https://github.com/marketplace/actions/lighthouse-ci-action

Should run on updates to storefront (SSR Function & Hosting) only
Use current domain from content/settings
Deal with CDN cache, specially SWR (how ? 🤔 )

👉🏾 https://github.com/ecomplus/cloud-commerce/blob/main/action.yml


Vulkan: Naming broken for command buffers

Description
RenderDoc shows a generic tag like "Command Buffer 215" instead of the actual object name. The name is displayed correctly in other debuggers like NVIDIA Nsight Graphics.
Steps to reproduce

Create a command buffer.
Name it using vkSetDebugUtilsObjectNameEXT() with VK_OBJECT_TYPE_COMMAND_BUFFER.
Capture a frame.

Environment

RenderDoc version: v1.34
Operating System: Windows 11
Graphics API: Vulkan

Yes since these names are not semantic and it's up to each tool to interpret them, this is a known/deliberate case where names are not applied to command buffers due to their complex lifecycle and because in the vast majority of cases an individual command buffer object doesn't have a meaningful name.
For command buffers, annotation is much better provided by vkCmdBeginDebugUtilsLabelEXT/vkCmdEndDebugUtilsLabelEXT and vkCmdInsertDebugUtilsLabelEXT within the commands themselves.

Thanks for the quick reply (and this excellent tool)! I agree the utility is small for most apps. In case you ever revisit the decision, it shouldn't be too difficult: lifecycle states are irrelevant for naming, just make sure to track when the command buffer is destroyed by vkFreeCommandBuffers() or vkDestroyCommandPool().


Decouple Label Position from textAlign

Decouple Label Position from textAlign
The ol/style/Text offers the textAlign option,
however, textAlign also affects the label position. You can see it in this image:

Proposal for Point Placement
Separate textAlign into two option.

textAlign: only text alignment. The values stay the same: {'left', 'center', 'right'} (case: point placement)
position: introduce new option with extended values:

Position Values

center
left
left-top
top
right-top
right
right-bottom
bottom
left-bottom

Few Position Examples
Few position examples, whereat, text align is set to default value 'center'.

Contribution
If the proposed API is good enough, I would like to implement the solution and create a pull request. Improvements and/or concerns are welcome too.

Thanks for the suggestion! i agree the current API lacks positioning options, but this would be a breaking change. I think we could also handle this with a non-breaking change for now. At some point, we have more work to do to consolidate the style API, and when that time has come we should make all breaking changes altogether.
I think a viable solution would be to add a justify option. When not set, everything is as it is now, the meaning would be "justify text towards the textAlign anchor". When set to left, center or right, text justification within the box aligned by textAlign will change. This would also avoid confusion with the (poorly chosen) textBaseline option. So the examples you posted in the above image would then be:

center: textAlign: "center", textBaseline: "middle"
right: textAlign: "left", textBaseline: "middle", justify: "center"
left-top: textAlign: "right", textBaseline: "bottom", justify: "center"
bottom: textAlign: "center", textBaseline: "top"

Would this be something you could live with?

For now, I agree on introducing the new option justify, because position makes less sense in case of placement: line and so it requires more discussion.
I will start implementing justify, if it is fine for you.
If there are some important things to consider while doing that, please comment.
Some thoughts for later ol/style refactorings: since the options differ depending on the placement, it might be useful to have two classes for placement, e.g. ol/style/PointPlacement and ol/style/LinePlacement. They might look like this:
new PointPlacement({
  position: 'top' || 'top-right' || 'right' || 'bottom-right' || 'bottom' || 'bottom-left' || 'left' || 'top-left',
  textAlign: 'left' || 'center' || 'right',
});

new LinePlacement({
  position: 'start' || 'center' || 'end',
  textAlign,
  textBaseline,
})

Thanks @rycgar. Please go ahead with the implementation and feel free to ask for assistance.
Regarding future: we're going to move away from class-based styles to an object literal based approach. And the goal is to bring it close to the Mapbox Style specification.


feat: add note regarding using act for github actions testing

Description
Add note about using https://github.com/nektos/act for github action testing. This will create awareness about the tool, and also help developers debug this without triggering multiple jobs. (running jobs locally is faster also since we don't have dedicated gh-actions machines)

Ah, the spellchecker caught a false-positive, it thinks that the word "github" should be capitalized even in the URL.

@stonecharioteer I think it was me who remove Github from the dictionary. There is either github or GitHub. The first one can be used in URLs and the second one is the original brand capitalization.

Aha. I'll change it then. Thanks for pointing that out.
On Fri, 24 Mar 2023, 22:20 Peter Kalambet, @.***> wrote:

@stonecharioteer https://github.com/stonecharioteer I think it was me
who remove Github from the dictionary. There is either github or GitHub.
The first one can be used in URLs and the second one is the original brand
capitalization.
—
Reply to this email directly, view it on GitHub
https://github.com/ChainSafe/engineering-handbook/pull/96#issuecomment-1483111439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXSLC4NFJG6NXJ7BDNGO6DW5XGD5ANCNFSM6AAAAAAWFKEZHU
.
You are receiving this because you were mentioned.Message ID:
@.***>