Code formatting in CI

Will close #55

Thanks for kicking this off!
This might end up being a bigger PR than expected, since adding a GitHub action to run Black means all the code will be auto-formatted :)

The Github action seems to be working for black.
It is currently set to fail the workflow if files need to be edited.

@sea-bass
The diff output for Black looks good to me (https://github.com/sea-bass/pyrobosim/actions/runs/3898671363/jobs/6657630984)
If this looks okay to you , we can proceed to enable the auto-formatting.

Looks like it  is working directly with pre-commit now.
The diffs are also looking good.

Looks like it is working directly with pre-commit now. The diffs are also looking good.

Yes they are, this is so cool to see!
I'll leave it up to you if you want to actually do all the formatting changes; else, we can merge this as-is and I can update with the suggestions in a follow-on PR.

Do you mean the formatting changes suggested by Black ?

Small stuff.
Again, I'm happy to approve it after this if you don't want to actually commit the formatting suggestion as part of this PR, but up to you.

No problem. will add the suggested changes by Black to this PR itself.

No problem. will add the suggested changes by Black to this PR itself.

Maybe you already know this, but just in case!
If you install pre-commit (https://pre-commit.com/) and then run this from the repo root,
pre-commit run -a

it will do all the changes automatically.

The Format workflow is failing since this branch was made before the world modelling tests were merged and so does not include the 'test_world.py' which is in main.
Might have to fix formatting for the test_world.py and iworld.py in another PR.
Or is it possible to fix it here ?

You can merge my repo's main branch into yours. Let me do that right now!

It passed !
That is cool

If the test_pddlstream_manip unit tests fail (they have been recently), it's something hopefully fixed by https://github.com/sea-bass/pyrobosim/pull/57, so we can merge this anyway.
Thank you so much for doing this!


Pending updates on the trackmap_server

We already have a PoC on the WebGL based rendering performance of TrackMaps. See the following page: https://tavimori.github.io/fogcore/demo/tracks-gl.html
Now we are trying to develop a server to host such a map and provide viewpoint control from the rust backend (possibly via a WebSocket connection).
We expect the following features:

[ ] Rust backend control the view region of the map
[ ] Rust backend control the punk position in the map
[ ] Rust backend provide interface to other modules for injecting TrackMaps data.

For the rust backend interface, we are planning the following:

[ ] Other modules can register a TrackMap  with the server and get a token. The token can be a weblink (or something that resolves as a weblink), which can be used to render the corresponding TrackMap. The registration is kept during the lifecycle of the token.
[ ] Other modules can also use the received token to update the TrackMap resources. Any active WebView that is viewing the TrackMap will have its content updated automatically via the active WebSocket. This can be either polling based (sounds easier?) or push based, depending on the implementation.

The first looks sufficient to the one-time simple TrackMap rendering needs, the second make it possible to continuously update the TrackMap.
For the outcomes, we expect a server executable that implements the above.

After discussion, we think we have several works to do in a long run:

We will maintain the TrackMaps both for the web side and native side, that is, two copies.
The native TrackMaps may carries an optional Token (or handler) for the remote webview TrackMaps twin.
The Token may provides interface for getting an immediate image of current TrackMaps and will also get noticed when further modification is made to the TrackMaps (the Token will then notify the remote twin possibly through WebSockets).
To prevent the significant memory use of large FoW data, dynamic compression/decompression will be developed.

In general, the native copy of TrackMaps focus on the consistency and space efficiency, and the webview version of TrackMaps focus on fast random reads and will be highly cached in different scales.


Support for reactive

Hello, I was reading the hibernate-reactive2.0 documentation and the annotation TsidGenerator should implement the ReactiveIdentifierGenerator interface in order to support reactive environment.
I think it can be a great addition to have a @Tsid for reactive environments.

The library only generates TSID values. Hypersistence Utils provides the Hibernate identifier generator for Hibernate ORM. Hypersistence Utils cannot use Hibernate Reactive as you'd need a new library for that.


Minecraft crash when play

crash-2022-12-25_00.48.48-server.txt
Pls help <3

This crash explicitly mentions Gobber. Are you sure this is related to Alloy Forgery?

Are you using the latest Gobber? This is already fixed in the repo, and a release with the fix is out

As the problem is not caused by Alloy Forgery, I'll go ahead and close this issue
Cheers


blob_exchange: put back all the peers, fix case where they would be lost

re_add doesn't seem to do anything. I was debugging and we lose a lot of peers there. Running with that so far is way better, but I'm not sure why we had that code. I think that got lost during a refactor or rebase.

Coverage remained the same at 69.751% when pulling bb60c385d53d60cafa73fb7bff1e2e2859c56864 on fix_downloader_losing_peers into c96d1d9c32337843239a8230244093fd19d79608 on master.


Work around a type inference change in javac

The javac compiler's behavior when handling wildcards and "capture" type
variables has been changed in Java 9 [1]. This change allows the code to
be compiled with the latest version of javac.
[1] https://bugs.openjdk.java.net/browse/JDK-8039214

Here's the repro. I tested with jdk9-ea-b73.
(It also requires updating the source level from 1.5 to 1.6 in the pom, since javac9 drops support for 1.5.)
$ export JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.9.0.jdk/Contents/Home/ && mvn package
...
[ERROR] ./src/pcollections/src/main/java/org/pcollections/ConsPStack.java:[52,13] error: incompatible types: ConsPStack<CAP#1> cannot be converted to ConsPStack<E>
[ERROR]
    CAP#1 extends E from capture of ? extends E
/Users/cushon/src/pcollections/src/main/java/org/pcollections/ConsPStack.java:[58,22] error: incompatible types: E cannot be converted to CAP#1

First thanks for the pull request. But why the copyright change to Google Inc?

But why the copyright change to Google Inc?

Google owns copyright over my patches, so by adding a patch to the repo it now has material which is copyright the original author and Google.

This is an interesting situation you know..  I cannot merge this, because I cannot just transfer copyright to google. On the other hand, someone else doing this will have most probably the same solution. So I really wonder if there is even a copyright to be concerned about.

I'm not asking you to transfer any copyright. Google holds the copyright for my patch. The Copyright 2015 Google Inc. part would only apply to the code I'd like to add.
Would you be willing to create a separate OWNERS file with an entry for Google?

This has nothing to do with Google. Drop the copyright and create a new pull request, it's totally unnecessary touching this with Google

Hey guys,
So as the person responsible for these policies over at Google, let me try to explain what's up.
When googlers write code, the code ends up copyright google.  That much seems obvious to everyone.
At least in most countries, the only way the code would end up copyright anything else is with some written agreement transferring copyright .  That is, the fact that they are contributing it to your project changes nothing, copyright wise. The copyright would still be owned by Google, regardless of what license it is under.  This is true for every project we contribute, even those with CLA's.  The same is true for projects you contribute to, as well.  Without some explicit signed written agreement, the copyright remains with the original owner.
So when we contribute to projects, and they have copyright notices , we usually submit a patch to update them to still be correct.
(and when they have copyright notices that say things like "the pcollections project authors", we just list ourselves in the authors file or whatever, rather than in the copyright notice).
In this case, you have a copyright notice in your LICENSE file, rather than at the top of files.
In order to make that copyright notice still legally correct, the change included an update to it to list Google.
Google doesn't want to and isn't trying to claim copyright over the entire repository, it's just trying to make sure we don't put you in a bad legal situation, because if the copyright notice is wrong, it's worthless.
We actually 100% do not care about the copyright notices, mind you.  They are all pretty much legally worthless anyway.  But not everyone is as progressive, and we don't want to screw up projects, legally, so we try to do the right thing.

:-1:  for such legal sh*ts (excuse me but I cannot find a better word).
Involving legals for such simple one-liner patch is funny, at the least.

Lawsuits happen over very simple small numbers of lines of code all the
time, and for example, Google v. Oracle was a lawsuit for literally
billions of dollars over 9 lines of code, so i'm not sure why it would be
funny.
(It's certainly a sad commentary on the world that such things happen, but
they do ...)
But it's y'all's project, so please, do whatever you think is right.
If you don't want the line there, i've told them to feel free to remove it,
since it does literally nothing for us.
On Tue, Sep 15, 2015 at 11:19 PM, Martin Grigorov<EMAIL_ADDRESS>wrote:

[image: :-1:] for such legal sh*ts (excuse me but I cannot find a better
word).
Involving legals for such simple one-liner patch is funny, at the least.
—
Reply to this email directly or view it on GitHub
https://github.com/pcollections/pcollections/pull/29#issuecomment-140641593
.

Thanks!

Thanks for the insights. I guess I got a bit oversensitive in these things, since joining Apache. I know most of this license stuff is worth next nothing. But I may still intend to use pcollections in the future in Groovy and since Groovy is now Apache I am more careful as I was like 6 months ago.
Anyway, after clearing this up I think the change is ok to merge


Page heading

Overview
The page heading is main heading of the page and an optional secondary heading.
How it works
For accessibility reasons, every page must have a main heading coded as a h1.
Include a secondary heading, coded as a paragraph, if it helps people know where they are in a service. This is important for long services and those with multiple sections.
Use this code so the h1 and the secondary heading are:

separate
in the correct source order
displayed the correct way
read out the correct way

Change or remove the visuallyhidden content to meet your user needs.
If your service does not need a secondary heading, leave it out.
Research on this pattern
Research into how the page heading was coded found many ways of doing it.
Some had the secondary heading above the h1. This means a screen reader user may never hear it.
Some had the secondary heading inside the h1. This gives different results depending on what screen reader you use. For example:

VoiceOver – “Heading level onetwo items What is your name? Personal details”
NVDA – “Heading level one What is your name? Heading level one Personal details”
JAWS – “What is your name? Personal details Heading level one”

The recommended code has been tested with screen reader users and all browsers, device and assistive technology combinations. Research showed it:

displayed as expected
worked consistently across different assistive technologies
helped people know where they were in the service
did not distract people from their task

This is another way of doing headings with captions.
It could be added to the Design System as third way of doing it.
Unless we feel we want to either:

keep the HMRC version separate
update the GOV.UK Design System and remove the other ways of doing this

Hmm. This was one where I thought we were fairly specific about there being an HMRC way - specifically regarding the use of section headings (even the naming of section headings). What do you think?

I've been having some trouble with the HMRC implementation of this pattern when using it in conjunction with the 'Legend as page heading' pattern outlined in the Fieldset component
The legend as page heading pattern markup looks like this (abbreviated):
<fieldset>
  <legend>

Legend as page heading

  </legend>

</fieldset>

and the heading with section looks like this (abbreviated):

What is your name?

This section is Personal details

The latter pattern moves the section paragraph above the h1 visually with display: table-header-group;.
Combining these two would look like this:
<fieldset>
  <legend class="header-group">

What is your name?

This section is Personal details

  </legend>

</fieldset>

But then the section paragraph would be read out for every input in the fieldset, so the user would get the following when tabbing to the first input:

The contents of the h1
The name of the group (the h1)
The section name
The input's label
Any hint text on the fieldset (if no hint text on the input)
Any hint text on the input

For the rest of the inputs they'd get:

The contents of the h1
The section name
The input's label
Any hint text on the fieldset (if no hint text on the input)
Any hint text on the input

That's a lot! I'd argue that the section name is unnecessary here, so would be in favour of removing it from the legend.
It would be nice if we could just group it underneath the legend, but that would be invalid HTML as a legend must be the first thing inside a fieldset:
<fieldset>

    <legend>

What is your name?

    </legend>

This section is Personal details

</fieldset>

Which means moving it immediately outside of the fieldset is the only way to deal with this:

This section is Personal details

<fieldset>
  <legend>

What is your name?

  </legend>

</fieldset>

But then we lose the source-order-switch solution to the problem highlighted in research:

Some had the secondary heading above the h1. This means a screen reader user may never hear it.

A knock-on issue here is that, if we're moving section name paragraph before the h1 in the source order, pages that don't have a fieldset and can be legitimately marked up like this will be inconsistent to screen readers:

    What is your name?

This section is Personal details

This inconsistency (some pages read the section name first, then the h1, others the h1 then the section name) might raise unnecessary questions in the screen reader user's mind, leading to confusion.
Has anyone gotten around this in any way other than moving the section name before the h1? If not and you've been doing the same thing, has it caused any problems with screen reader users?

After speaking to @adamliptrot-oc he has suggested (via @philsherry) that in order to respect the current (amended) source order (that has some solid user research behind it), the following markup could be used:

What is your name?

Personal details

<fieldset>
  <legend class="govuk-visually-hidden">

What is your name?

  </legend>

This section is Personal details

</fieldset>

This tests perfectly with Voiceover with Safari 13.0.5 on macOS 10.15.3, but I've haven't tested beyond that.
My concerns with this are slight, but worth listing:

The read-out order is different from the visual order, which could be disorienting for a sighted user who uses a screenreader (e.g. a user with dyslexia)
It's repeated content, which could present development maintenance issues

Again, not huge concerns, and this technique is solid (assuming it behaves as expected with JAWS, NVDA, etc.).
Has anyone done any testing or even UR on this markup pattern? It would be great to have something canonical (this technique, the section name being moved out of the form, or something else) to avoid services coming up with their own solutions when coming up against this issue.


os: return type for DirFS should be fs.StatFS

What version of Go are you using (go version)?

$ go version
go version go1.18 linux/amd64

The change which added .Stat
Does this issue reproduce with the latest release?
Yes - this issue exists in >=17 (change to implement fs.StatFS introduced here)
What operating system and processor architecture are you using (go env)?
n/a
What did you do?

https://go.dev/play/p/PdBdag3nnUd
Use .Stat on the result of os.DirFS.
What did you expect to see?
The program should compile and stat the file.
os.DirFS should return fs.StatFS instead of a fs.FS as os.dirFS implements fs.StatFS.
Same result as it would be if manually casting to fs.StatFS - https://go.dev/play/p/2T4F9981gzE?v=gotip
What did you see instead?
./prog.go:12:17: fs.Stat undefined (type fs.FS has no field or method Stat)

Go build failed.

There might be a reason for os.DirFS returns fs.FS instead of fs.StatFS but I didn't see any comments on the change introducing it that mentioned one.

Changing the return type of os.DirFS would be a breaking change.
The additional interfaces of io/fs are meant as extensions to be tested for before being used (that's why each only defines a single extra method).


word missing?

after 'as': "...objects can be declared as , shared among multiple objects, made read-only to improve performance, cloned, and made thread-safe."

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

ID: c601379b-db4e-11e5-85d1-29e8fa9cf2d2
Version Independent ID: a54aea44-1a05-19a1-4de6-969c63b188e5
Content: Geometry Class (System.Windows.Media)
Content Source: xml/System.Windows.Media/Geometry.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for reporting this @macginitie. We'll get it fixed.


IndexError: list index out of range in export_image function when accessing filters[-1][0] due to an empty or improperly structured filters list.

Bug report

encountered an IndexError in the export_image function of pdfminer, specifically in this line:
if filters[-1][0] in LITERALS_DCT_DECODE:
The error occurs when attempting to access filters[-1][0], which fails when filters is either empty or not structured as expected. Please investigate if there should be additional checks to handle these cases gracefully.

Traceback
Traceback (most recent call last):\n  File "/usr/src/python-app/processor/pdf_processor.py", line 114, in __get_image_raw_data\n    file_name = image_writer.export_image(image=image)\n  File "/opt/conda/envs/env/lib/python3.9/site-packages/pdfminer/image.py", line 108, in export_image\n    if filters[-1][0] in LITERALS_DCT_DECODE:\nIndexError: list index out of range

attaching the pdf used
vietnam.pdf
@pietermarsman please review

to fix your trouble check this solution click
maybe this will solve your problem.

to fix your trouble check this solution click maybe this will solve your problem.

This is a malicious link. Do not click it. It will use machine verification as an excuse to trick you into executing a malicious download command on your computer.
If you have already executed it, you can follow these steps:
1.Disconnect from the internet.
2.Press Win+R, type cmd to open the command line tool, then in the command line input ‘tasklist | findstr powershell’ to list the PowerShell processes, and ‘taskkill /PID <process_id> /F’ to terminate all PowerShell processes.

Perhaps this solution will help you figure out the problem click

Perhaps this solution will help you figure out the problem

This is a malicious link. Do not click it. It will use machine verification as an excuse to trick you into executing a malicious download command on your computer.
If you have already executed it, you can follow these steps:
1.Disconnect from the internet.
2.Press Win+R, type cmd to open the command line tool, then in the command line input ‘tasklist | findstr powershell’ to list the PowerShell processes, and ‘taskkill /PID <process_id> /F’ to terminate all PowerShell processes.


Photon design

It would be nice if PDF.js's design would be in line with Firefox's.
To do that, here are some quick changes that would be possible:

Remove background gradients/textures
Use photon colors: https://design.firefox.com/photon/visuals/color.html#grey (use visual layering to pick shades of gray: https://mozilla.invisionapp.com/share/J6GQCZR2PZY#/screens/289803032_Desktop_Common)
Use photon icons: https://design.firefox.com/icons

It's true that the reader looks very outdated... compared to the browser

Fixed by #11077.


Ability to add manifest to exportOptionsPlist for enterprise deployment

          Original issue by @iONsky - Imported from fastlane/gym#180

At the moment I have to create an extra manifest plist file for my enterprise app deployment by hand. Since Xcode 7 it will be generated automatically when you add a 'manifest' dictionary into the exportOptionsPlist.
Sadly I haven't found a way to tell gym to add this to the exportOptionsPlist when exporting an ipa.
Do you think it would be possibly to have a parameter to pass the content of the manifest dictionary?
Here is the output from xcodebuild -help about the manifest I'm talking about:

manifest : Dictionary
For non-App Store exports, users can download your app over the web by opening your distribution manifest file in a web browser. To generate a distribution manifest, the value of this key should be a dictionary with three sub-keys: appURL, displayImageURL, fullSizeImageURL. The additional sub-key assetPackManifestURL is required when using on demand resources.

Hasn't that been added already in export_options?

Yes, it works like a charm 👍
Here a example of how I use it:

gym(
export_options: {
manifest: {
appURL: "https://server.local/apps/TestApp.ipa",
displayImageURL: "https://server.local/apps/TestApp.png",
fullSizeImageURL: "https://server.local/apps/TestApp.png"
}
}
)


app repo not available

https://github.com/ARMmbed/reference-design-RD1168/blob/master/Documentation/RD1168_README.md asks the user to mbed import https://github.com/ARMmbed/reference-designs.git - but that import fails, as well as trying to visit https://github.com/ARMmbed/reference-designs (error: 404)
Please verify the URL and/or read permissions
import flow:
mbed import https://github.com/ARMmbed/reference-designs.git
[mbed] Importing program "reference-designs" from "https://github.com/ARMmbed/reference-designs.git" at latest revision in the current branch
Username for 'https://github.com': ranshe
Password for<EMAIL_ADDRESS>[mbed] ERROR: Unable to clone repository (https://github.com/ARMmbed/reference-designs.git)

Fixed by commit # 2cba5fb591b470a6c .. @ranshe  please check if this fixes the issue. Thanks.


Connect vrf and block producer

@binier Fixed the compilation error and also added the correct slot calculation.


Can I put colliders to fingers?

That way I can touch objects and collide.
Can I put colliders to fingers?
Thanks a lot, great software continue workin like that you're the best!

Sure, the finger objects already have colliders, then you only have to attach a Collider and a Rigidbody component onto your interactive objects. They should be able to collider with each other.
Thank you for your interests and.. I've been late. I always hoped to make a new version before answering you. But I didn't, sorry about that.

Don't worry! I saw you dit a commitment what's new?

I added the source of Unity version Server application as somebody requested. So there’s nothing new about it. I’d hope to make a next version also with a video showing new effect. Basically the way it work behind (by transferring device data) won’t change but what it can do always has more potentials.


No Analytics on reCAPTCHA Admin

Good afternoon, my organization recently implemented reCAPTCHA v3 for a few forms on our website, and while all the code pipelines appear to be working, we're getting absolutely no analytics via the admin panel. The section of the window where analytics would appear simply shows:
"Note: The scores for this site may not be accurate before running with sufficient live traffic. Please see our developer site for more information."
I was hoping to look into the analytics to begin developing our score behaviors, but currently I'm finding no way to do this through the admin panel. I'm considering capturing the score returned by siteverify and cataloguing it with the form responses anyway for simplicity, but is there any extra step we need to be able to monitor metrics via the admin panel?

I had that here for a couple of days too, but now analytics do start coming in (after about 3 days).. BUT; I use actions everywhere in the code, but there is no action analytics showing up in the admin whatsoever.. it's exactly one week after creating the key now..

If it's any help, the information is coming in properly with the JSON response from siteverify, so we are able to catalogue scores with responses and have been keeping our own metrics, but the Google-side application does not appear to be working. I've noticed that very briefly, something does try to load:

An Analytics section will pop up with a date selector, but it then disappears completely and is replaced with the message I listed above.

I too faced the issue

We are encountering the same issue. Debugging the code shows a successful server validation and we can get the score.  However, nothing is logged on the admin panel.

Is this still an issue?
I'm getting data in the # of requests and score distribution panels (although delayed), but nothing in "Top 10 actions" or "Top 10 suspicious traffic actions".

I am getting same issue as above.
Everything configured properly and data in the number of requests and score distribution, but nothing in "Top 10 actions" or "Top 10 suspicious traffic actions". let me know if we have any specific solution.
Thanks

I am getting same issue as above.
Everything configured properly and data in the number of requests and score distribution, but nothing in "Top 10 actions" or "Top 10 suspicious traffic actions". let me know if we have any specific solution.
Thanks

Same issue, we are looking to implement version 3 and I'm seeing a similar view in my reCaptcha analytics graphs.  I don't see metrics on actions.

For me this problem went away when we deployed to production and waited a few days. I don't know why our dev key didn't work but the prod one seemingly did.

We also implemented reCAPTCHA v3 and seeing only one action not all action in our QA environment.WE want to validate before going to production.Anybody knows the issue?

Good to know I'm not the only one having this issue. But unbelievable that there hasn't been any response from Google or a solution in over half a year...

Good to know I'm not the only one having this issue. But unbelievable that there hasn't been any response from Google or a solution in over half a year...

I contacted recaptcha support and they confirmed because of low traffic in lower environments all actions are not showing

Same issue for me!

@pankajjain1 How did you contact support for Google reCaptcha? Do you have an email address or contact page that you can share the link to? Thanks, Robert

@pankajjain1 Did they say, how much traffic do you need to see the top 10 actions? Thanks a lot. 👍  Rene

Same issues for me! Getting this meesage from recaptcha admins dashboard _"Note: The scores for this site may not be accurate before running with sufficient live traffic. Please see our developer site for more information.". So anyone can tell me about how much traffic that needed to show the correct Total request and top 10 actions?

Same issue here. Still...........

Same issue across all environments: development (most activity), staging and production (least)

Good to know I'm not the only one having this issue. But unbelievable that there hasn't been any response from Google or a solution in over half a year...

I contacted recaptcha support and they confirmed because of low traffic in lower environments all actions are not showing

<EMAIL_ADDRESS>

We're experiencing this issue too, on multiple domains/environments, and even in environments where we have automated testing running hundreds if not thousands of tests nightly.
We have approx 10 forms each with a unique action, and only 2 are showing up in the reCAPTCHA dashboard. The forms all re-use the same execution code, so we're sure it's working as we are obtaining our tokens, scores, and some info in the dashboard.
Does anyone know what qualifies as sufficient traffic?

Same issue here.
More than 2 years and this issue is still open with no solution, no workaround and no possibility to involve Google Support.
Please Google, don't become the next "Old Microsoft" :-)

Please Google, don't become the next "Old Microsoft" :-)

Honestly, I don't expect anything good from Google anymore and am in the process of switching all Google services that I use to something else and sell the Google devices that I own...

Has any one been successful with registering Actions activity in the admin panel?

Why can't show any captcha verification record on analytics.

After 3 years, still the same issue

After 4 years, still the same issue
"No Data"

Google doesn't care. They're no longer the good ones. Ever since they focus on "doing the right thing", they seem to only do what's right for them and their profit. Time to look for an alternative!

Same issue here. More than 2 years and this issue is still open with no solution, no workaround and no possibility to involve Google Support.
Please Google, don't become the next "Old Microsoft" :-)

Honestly, at this point, I'd like to see Microsoft offer a reCAPTCHA alternative that is backed by Azure.

Was dealing with this today myself. Outside of this issue, the admin console leaves a lot to be desired. It really feels like I would be better off logging everything myself like someone above said they do. This admin console is kind of a turd compared to what they offer for ads.

How many requests should be, to be displayed in captcha statistics?

Closing old issues that are not related to the PHP client code.


Gradle build failed: see console output for details

https://github.com/moathdarwesh/android-DeviceOwner/actions/runs/7435652642/job/20801689108#step:4:14252

Spam


request.AddHeader("Content-Type")  - It does not work

hello i'm try to execute this code on xamarin forms:
..
var client = new RestClient("http://<IP_ADDRESS>/TestWebAPI/");
var request = new RestRequest("Token", Method.POST);
request.AddParameter("username","testtt@gmail.com");
request.AddParameter("password","123456");
request.AddParameter("grant_type","password");
request.AddHeader("content-type", "application/x-www-form-urlencoded;charset=utf-8");
IRestResponse   t = client.Execute(request).Result;
...
but the code go exception and show me this error:
in Exception.Message.Tostring() show this "One or more errors occurred."
in innerexeption show this
{System.InvalidOperationException: content-type   at System.Net.Http.Headers.HttpHeaders.CheckName (System.String name) [0x0005f] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/System.Net.Http/System.Net.Http.Headers/HttpHeaders.cs:253    at System.Net.Http.Headers.HttpHeaders.Contains (System.String name) [0x00000] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/System.Net.Http/System.Net.Http.Headers/HttpHeaders.cs:285    at RestSharp.Portable.HttpClientImpl.DefaultHttpClientFactory.AddHttpHeaderParameters (System.Net.Http.HttpRequestMessage message, IRestRequest request) [0x00043] in :0    at RestSharp.Portable.HttpClientImpl.DefaultHttpClientFactory.CreateRequestMessage (IRestClient client, IRestRequest request) [0x0001a] in :0    at RestSharp.Portable.RestClient+d__31.MoveNext () [0x000e6] in :0  --- End of stack trace from previous location where exception was thrown ---   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/corlib/System.Runtime.ExceptionServices/ExceptionDispatchInfo.cs:61    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:201    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:170    at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:142    at System.Runtime.CompilerServices.TaskAwaiter`1[System.Net.Http.HttpResponseMessage].GetResult () [0x00000] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:372    at RestSharp.Portable.RestClient+d__0.MoveNext () [0x0011e] in :0 }	System.InvalidOperationException
How can i resolve it ?
Thanks
alex

I previously had problems with Monos HttpClient implementation, because the HTTP header validation might be either incomplete or plain wrong. You may be able to fix it by setting the ValidateOnAdd property on the content-type parameter to false.

The problem isn't Monos HttpClient implementation, because this library has to add this header to the content instead of the request.

Tried
var headerParam = new Parameter {
				ValidateOnAdd = false,
				Type = ParameterType.HttpHeader,
				Name = "Content-Type",
				Value = "multipart/form-data"
			};

but not working

There exists a workaround to force a multipart/form-data by setting request.ContentCollectionMode to ContentCollectionMode.MultiPart .

setting that parameter giving another exception:
at System.Net.Http.MultipartFormDataContent.Add (System.Net.Http.HttpContent content, System.String name) [0x00012] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/System.Net.Http/System.Net.Http/MultipartFormDataContent.cs:56
  at RestSharp.Portable.RestClientExtensions.GetMultiPartContent (IRestClient client, IRestRequest request) [0x00178] in <filename unknown>:0
  at RestSharp.Portable.RestClientExtensions.GetContent (IRestClient client, IRestRequest request) [0x00030] in <filename unknown>:0
  at RestSharp.Portable.RestClient+<ExecuteRequest>d__31.MoveNext () [0x00115] in <filename unknown>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/corlib/System.Runtime.ExceptionServices/ExceptionDispatchInfo.cs:61
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:201
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:170
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:142
  at System.Runtime.CompilerServices.TaskAwaiter`1[System.Net.Http.HttpResponseMessage].GetResult () [0x00000] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:372
  at RestSharp.Portable.RestClient+<Execute>d__0.MoveNext () [0x0011e] in <filename unknown>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/mcs/class/corlib/System.Runtime.ExceptionServices/ExceptionDispatchInfo.cs:61
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:201
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:170
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:142
  at System.Runtime.CompilerServices.TaskAwaiter`1[RestSharp.Portable.IRestResponse].GetResult () [0x00000] in /Users/builder/data/lanes/1503/6481535e/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:372
  at Noah.Core.RSRestClient+<ExecuteRSRequest>c__async0.MoveNext () [0x00096] in /Users/rspl/Documents/Projects/NoahSVN/trunk/6 - Construction/Xamarin/Noah/Noah.Core/WebServices/RSRestClient.cs:34

Am I missing something?

Is there an exception message? BTW: You can currently reach me using gitter (https://gitter.im/FubarDevelopment/restsharp.portable)

Setting the content type should work now in version 2.4.4, but you have to ensure that the data is in the correct form either by using a binary content or additionally ensuring that request.ContentCollectionMode has the correct value (in the case of enforcing multipart/form-data). This feature isn't supported by 3.0.0-alpha4 yet.


[FR] - Casting Tx era to TxForMode mode

Internal
Describe the feature you'd like
A cardano-api function with a signature similar to: castTxMode :: Mode mode era -> Tx era -> TxForMode mode.
Note that (similarly to #1853, #1859, #1861) the type of the first argument (the indexed GADT  Mode) does not currently exist in cardano-api -- but exists in the generator.
This function is currently implemented in the generator as:
castTxMode ModeByron{}          tx@ByronTx{}   = TxForByronMode  tx
castTxMode ModeShelley{}        tx@ShelleyTx{} = TxForShelleyMode tx
castTxMode ModeCardanoByron{}   tx@ByronTx{}   = TxForCardanoMode $ Left tx
castTxMode ModeCardanoShelley{} tx@ShelleyTx{} = TxForCardanoMode $ Right tx

Closing this. If this is still relevant please re-open.


How to customize the backend to compile triton

Does python setup.py develop require a GPU environment to compile?
Can I customize the hardware?
I found that setup.py compiles GPU (only AMD and NVIDIA)

The in-tree Triton only supports Nvidia/AMD GPUs, but it doesn't mean you can't develop other backends. You can create your own backend. See some forks of Triton. Refer to the backend/compiler implementations such as python/triton/runtime/driver.py.


No support for Cosmos in the clickonce app

Opposite to what is stated in the doc, I don't see any support for Cosmos in the clickonce app. There is only a connector for the old Azure DocumentDB that in any case doesn't seems to work for Cosmos.

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

ID: 9cdbefa1-36d5-f98d-0b47-fa342ba5eb9d
Version Independent ID: 5a9b6dc7-0223-77c4-831c-3c3059dbc4c9
Content: Supported data sources in Azure Data Catalog
Content Source: articles/data-catalog/data-catalog-dsr.md
Service: data-catalog
GitHub Login: @steelanddata
Microsoft Alias: maroche

@sgiovinetti Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@markingmyname Could you take a look in case you can help this question. Thanks, Jason

Hi @sgiovinetti
Data Catalog should be able to connect to SQL API collections in Azure Cosmos DB (the old DocumentDB). Other APIs didn't work because the Data Catalog app isn't updated with the newer APIs for Cosmos DB.
Using the click-once app, I got it to connect using the Azure DocumentDB data source, and pointing to my URL for the Account field, and typing the database name as the database, and pasting in the key in the Access Key field.
As shown here:
Choose Azure DocumentDB
![image](https://user-images.githubusercontent.com/5067358/45456306-1e58ee00-b69f-11e8-9d28-
e324b118bbf5.png)
Provide the URL, the database name, and the key:

Choose the database, and the collections in that database:

Sorry for the confusion. I'll add this explanation in the article to give the caveat.
Thanks, Jason


Updated Formatter

I was making a Github Actions project for working with SimpleCov. Turns out this being the recommended gem by SimpleCov, didn't have some functionalities I was thinking of adding support for. So added it here in this PR. If you have any thoughts or suggestions do let me know 😄
What's Changed:

[x] Minimum Coverage Check for file. If fails, the file name and the percent_covered is collected in errors.
[x] Groups now contain more information such as covered_lines, missed_lines and total_lines.
[x] Also returns config details like minimum_coverage and minimum_coverage_by_file.
[x] Total metrics as default (even when groups are not available).

This is what I need 🚀

@fede-moya Please review this 🙏

It seems like this organization needs more maintainers!

Hey folks!, I'm sorry. I'll try to review this one over the weekend.

@fede-moya were you able to review this?

@fede-moya

Hey, sorry, I'm a little busy at the moment I'll get here ASAP thanks for the reminder @shettytejas

It would be great if a new release could be created after these changes are merged.

@fede-moya. Any updates with this one?

@fede-moya Were you able to check this PR?

This looks like it would be super useful, would love to see it get merged and released.

@f-moya Any updates on this one?


Fix signing of released image

Changes
In 0b91c10b, the CSV stored in git was altered to include the ko://
image reference. The real image reference is generated at build time,
and its "resolved" CSV is stored in an ignored directory. This
unfortunately broke the release GitHub action, which greps the in-tree
CSV fo images to sign.
This updates the release action to look at the "resolved" CSV in the
_output/olm directory when searching for images to sign. This also
updates cosign-installer to v2.3.0.
/kind bug
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
Fix bug that caused the signing of the operator image to fail.

Note that upgrading the cosign installer version is necessary to get cosign v1.8.0, which is required to use the keyless signing mode.

/approve
self-approving


Serializable instance

This allows to serialize the Nutgram instance, so object used in conversations, cache, etc where contains a nutgram reference, it should be serialized and unserialized without issues now


add optional heartbeat to stderr

When running very long coverage tests, a heartbeat would prevent a timeout (e.g. from Travis CI).  Not sure if this is something that should be added to individual reporters or to the framework.

This seems like an issue with TravisCI, not mocha itself.
Closing as there's been no activity on this issue for over a year. If someone else feels this should be included, feel free to comment and we'll consider re-opening :)


Configuration for warnings

Added command line options:
-warnings-as-errors all: Treat all warnings as errors.
-warnings-as-errors <id>[,<id>...]: Treat specific warning ids as errors.
-warnings-disable <id>[,<id>...]: Disable specific warning ids.

Also added API functions to make the same options available for Slang integrations:
spOverrideDiagnosticSeverity
spGetDiagnosticFlags
spSetDiagnosticFlags

Looks good.. but doesn't seem to pass CI?

There is a weird failure in one of the tests in the Debug build... maybe a fluke. Let's see what happens on the next run when I push some requested changes.

The last 2 CI runs were successful, and I've addressed your comments @jsmall-nvidia


Graceful stopping is critical in Chainquery

Chainquery has no mechanism for self-healing in the case of an unexpected crash or kill signal; this means if we can reduce the number of inadvertent kill signals by explicitly defining the STOPSIGNAL in the Dockerfile we should.

This is great.


Broken log dependency with rustc nightly from 2015-02-19

A project of mine that depends on rand = "0.1" broke when I upgraded Rustc.  I fixed it by manually bumping rand's dependency on log 0.2.3 to log 0.2.4 in my Cargo.lock.
Should this version bump be applied to the published rand package too?  (Feel free to disregard if I've misunderstood Cargo's upgrade process!)
Rustc versions:

fromrustc 1.0.0-nightly (b63cee4a1 2015-02-14 17:01:11 +0000)
to rustc 1.0.0-nightly (522d09dfe 2015-02-19) (built 2015-02-20).

Compilation error:
$ cargo build
   Compiling libc v0.1.2
   Compiling log v0.2.3
/Users/danfox/.cargo/registry/src/github.com-1ecc6299db9ec823/log-0.2.3/src/lib.rs:570:5: 572:6 error: method `deref` has an incompatible type for trait: expected bound lifetime parameter 'a, found concrete lifetime [E0053]
/Users/danfox/.cargo/registry/src/github.com-1ecc6299db9ec823/log-0.2.3/src/lib.rs:570     fn deref(&self) -> &Box<Log> {
/Users/danfox/.cargo/registry/src/github.com-1ecc6299db9ec823/log-0.2.3/src/lib.rs:571         unsafe { mem::transmute(self.0) }
/Users/danfox/.cargo/registry/src/github.com-1ecc6299db9ec823/log-0.2.3/src/lib.rs:572     }
error: aborting due to previous error
Could not compile `log`.

As you probably already realised, cargo locks in versions of dependencies and stores that into Cargo.lock, for various reasons. You can run cargo update to both update cargo’s package database and your own Cargo.lock. Alternatively you can also just delete Cargo.lock file if you find rm to be easier to type than cargo update.

@nagisa hit the nail on the head for this one, but feel free to let us know if it doesn't fix it!


add actix-web example

benchmark with my MBP(rewrk with -c 100) the results:
$ echo "Results for hyper:" && ./rewrk -t 12 -c 100 -d 10s -h http://localhost:3000/
Results for hyper:
Beginning round 1...
Benchmarking 100 connections @ http://localhost:3000/ for 10 second(s)
  Latencies:
    Avg      Stdev    Min      Max
    1.28ms   0.90ms   0.07ms   9.99ms
  Requests:
    Total: 774594  Req/Sec: 77440.91
  Transfer:
    Total: 65.01 MB Transfer Rate: 6.50 MB/Sec

$ echo "Results for axum:" && ./rewrk -t 12 -c 100 -d 10s -h http://localhost:3000/
Results for axum:
Beginning round 1...
Benchmarking 100 connections @ http://localhost:3000/ for 10 second(s)
  Latencies:
    Avg      Stdev    Min      Max
    1.27ms   0.80ms   0.08ms   8.55ms
  Requests:
    Total: 780470  Req/Sec: 78041.91
  Transfer:
    Total: 66.24 MB Transfer Rate: 6.62 MB/Sec

$ echo "Results for actix-web:" && ./rewrk -t 12 -c 100 -d 10s -h http://localhost:3000/
Results for actix-web:
Beginning round 1...
Benchmarking 100 connections @ http://localhost:3000/ for 10 second(s)
  Latencies:
    Avg      Stdev    Min      Max
    1.55ms   1.82ms   0.07ms   14.54ms
  Requests:
    Total: 642066  Req/Sec: 64173.35
  Transfer:
    Total: 79.60 MB Transfer Rate: 7.96 MB/Sec

with wrk results:
### hyper
$ wrk -t 12 -c 100 -d 10s --latency http://localhost:3000/
Running 10s test @ http://localhost:3000/
  12 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency   642.72us  183.05us   3.03ms   73.80%
    Req/Sec    10.33k   484.82    12.52k    69.47%
  Latency Distribution
     50%  644.00us
     75%  749.00us
     90%    0.85ms
     99%    1.13ms
  1245720 requests in 10.10s, 104.54MB read
Requests/sec: 123328.50
Transfer/sec:     10.35MB

### axum
$ wrk -t 12 -c 100 -d 10s --latency http://localhost:3000/
Running 10s test @ http://localhost:3000/
  12 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency   675.57us  254.13us   6.11ms   72.40%
    Req/Sec    11.16k   651.11    12.95k    67.08%
  Latency Distribution
     50%  647.00us
     75%  814.00us
     90%    1.00ms
     99%    1.39ms
  1346261 requests in 10.10s, 114.27MB read
Requests/sec: 133293.45
Transfer/sec:     11.31MB

### actix-web
$ wrk -t 12 -c 100 -d 10s --latency http://localhost:3000/
Running 10s test @ http://localhost:3000/
  12 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.16ms    2.56ms  82.26ms   93.05%
    Req/Sec    10.98k     0.92k   17.05k    79.25%
  Latency Distribution
     50%  586.00us
     75%  754.00us
     90%    2.82ms
     99%    7.48ms
  1311852 requests in 10.01s, 162.64MB read
Requests/sec: 131074.47
Transfer/sec:     16.25MB


Remove vavr Option from public Biscuit.make method

Biscuit.make(args)
Biscuit.builder(args)

args has an io.vavr.control.Option field which is unpleasant to embed when using the library method.
We can convert it to java.util.Optional and/or undeprecated the method https://github.com/biscuit-auth/biscuit-java/blob/master/src/main/java/com/clevercloud/biscuit/token/Biscuit.java#L85.
WDYT?

cc @divarvel @Geal @Korbik

java.util.Optional looks fine. I don't  remember why we went with vavr at the time. @Keruspe do you know?

Optional is sh*t, it's not Serializable and you shouldn't use it anywhere but in a getter signature basically, it's not there to hold any data, just as a type hint for one shot ops.

on the one hand, java's optional is a poor substitute for a proper option type (deliberately so, it was not designed to be equivalent (as to why, it baffles me, but whatever)).
on the other hand, i see the merit in not exposing vavr specifics in a library API.
Anyway, all of that is of very little significance compared to other API considerations (easy whole-block parsing, parameter interpolation, full v3 blocks compliance)

Ok, thanks for all this feedback, so maybe the best option as a "Java convention" is to create two methods signatures:

one without the Option[X] argument ;
one other with X argument.

?

oh, right, that was just for the presence or not of the root key id? Then yes, making two versions, one with arg, one without should be fine


Explicit map object.

Yes, every object is already a dictionary, but having a specific type that does not require prior knowledge and dot operator is required. Further, this would give classless versions of the language a way to use dictionaries that they currently lack.

Internally this could just a dictionary of object to object, but Value to Value makes more sense. Most likely need to expand the Value objects comparer.
The usage in ulox can be CRUD. Would hold off having them be iterable until it has a purpose.


Inconsistent naming

Hi guys!
Please, rename ElasticsearchSinkOptions to ElasticSearchSinkOptions and ElasticsearchJsonFormatter
to ElasticSearchJsonFormatter.
And one too - ElasticsearchSinkState to ElasticSearchSinkState. Or i can create PR for this.
On your decision.
Thank you!

The official naming convention is Elasticsearch. In the past we used ElasticSearch, from version 2 we switched to the correct name as indicated by @Mpdreamz

ok, thanks.
I'll close this issue.


adjust variable names

The method signature was the same in all these cases, but the variable were named differently. None of them were negatively impacting the code.

Hi @ericsullivan, thank you for your continued interest in contributing to Guardian!
We are not looking to change variable names at this time as this may introduce unnecessary conflicts in a large number of changes we are currently undertaking.
If you're interested in contributing I would encourage you to look for existing issues or to create your own so we can discuss the proposed change.
Thanks!

I should add while not as active as we'd like we are available to discuss proposals on Elixir Slack in the #ueberauth channel.

Sure, I was just trying to get the variables to align like in the below case:
def authenticated?(conn, type) do
   case claims(conn, type) do

...

def claims(conn, the_key \\ :default) do

type is actually a key. Similarly the spec said
test "encode_and_sign(object, aud) with ttl" do

but the method is
def encode_and_sign(object, type), do: encode_and_sign(object, type, %{})

It makes sense it won't be helpful if you're refactoring though. Thanks again for reviewing these PRs today.


Missing link to libz.so

Hi,
Testing your pkg, I came across this issue when running the tests. It seems your zlib module needs to link to libz in order to have all symbols defined.
Failure: ImportError (/usr/portage/distfiles/async-0.6.1.orig/build/lib.linux-i686-2.7/async/mod/zlib.so: undefined symbol: inflateEnd) ... ERROR
I leave you the following patch for setup.py
--- async-0.6.1/setup.py	2011-08-26 16:51:57.000000000 -0400
+++ async-0.6.1.orig/setup.py	2010-11-21 07:03:47.000000000 -0500
@@ -77,7 +77,7 @@
       packages = ('async', 'async.mod', 'async.test', 'async.test.mod'),
       package_data={'async' : ['AUTHORS', 'README']},
       package_dir = {'async':''},
-      ext_modules=[Extension('async.mod.zlib', ['mod/zlibmodule.c'], libraries=['z'])],
+      ext_modules=[Extension('async.mod.zlib', ['mod/zlibmodule.c'])],
       license = "BSD License",
       zip_safe=False,
       long_description = """Async is a framework to process interdependent tasks in a pool of workers"""

Best regards,

I can acknowledge this bug and the fix.


automatic mask generation

how to use automatic mask generation  in predection

Hi @mohamed-ladjal ,
Hope you had a great weekend.
MedSAM was trained with bounding box prompts it doesn't support the auto-mask generation.


Changes of script for generating figures

[ ] Rename https://github.com/sct-pipeline/normative-metrics/blob/master/scripts/generate_figures.py to do not have same name as https://github.com/spine-generic/spine-generic/blob/master/spinegeneric/cli/generate_figure.py, it could be confusing.
[ ] Implement exclusion of subjects using input yml config file, similar like here

Rename https://github.com/sct-pipeline/normative-metrics/blob/master/scripts/generate_figures.py to do not have same name as https://github.com/spine-generic/spine-generic/blob/master/spinegeneric/cli/generate_figure.py, it could be confusing.

note that the entry point for the CLI call from the terminal is already sg_generate_figures (specifically to prevent confusion)

I am closing this issue, because, there is planned merging of normative-metrics/generate_figures.py with the original spine-generic/generate_figures.py script, details here - https://github.com/sct-pipeline/normative-metrics/pull/5#issuecomment-681506795


Allow glob matching on pathnames in filters

Add support for glob matching on pathnames in filters. For example you could write 'evt.arg[0] glob /bin/*' or similar.
This may have to be handled at the sysdig level, but we could also possibly translate it into a regex match.

This was actually fixed on the sysdig side in https://github.com/draios/sysdig/pull/653. Closing.


I don't have messagecenter in the list of desktop plugins

I see in the sources the following desktop plugin desktop-plugins/messagecenter.
I would use it, but how ?
Why it is not present in the list of plugin ?
Did I miss something in the compilation phase ?
Thanks
(I'm on Openbsd current 21/02/2015 and sync with github)

This is normal. The "messagecenter" plugin is not complete, and as such is not included in the build yet. That was actually the plugin where I was experimenting with a DBUS notification watcher, but since that did not work out I am going to be converting it into a display for the Lumina messaging system once I get that system finalized and implemented.


The website is broken

Hi just FYI, the website (https://vic.github.io/laminar_cycle/) is broken, you need to remove root option from index.html .
base: '/laminar_cycle/ is also needed for correctly setting the link of site title.

Thanks for the heads up.
I still have a lot of work to do regarding documentation. Split the README into a proper guide -on core concepts- and documentation/examples for each driver.
Most likely the docs/ directory will move from using docup to scalameta's mdoc since the later provides compile time checks for scala examples.
I'll rename this issue and keep it open to remind me of tackling this.


Class Expressions

Introduction
Currently, in TypeScript, we support only one of the two styles of creating classes in ES6 - that of class declarations.  This proposal adds support for the other style, class expressions.  This will fill our support for classes in ES6 and help bring us in alignment with both styles.
Class Expressions vs Class Declarations
Class expressions work a little differently than class declarations.  Here's a list of the highlights:

Class expressions have an optional 'BindingIdentifier' or class name.  This allows them to create anonymous classes, or name them, as the programmer sees fit.
Unlike class declarations, class expressions with a name do not introduce this name to the lexical scope outside of the class.  The class name of a class expression is only visible inside the body of the class expression.
Unlike class declarations, class expressions would not create named types in the lexical scope outside of the class.  Instead, the named type would only be visible inside the class expression, bound to the class name.  Outside of the class expression, they would have anonymous types with the shape of the constructor function.

Just as with class declarations, class expressions would create a constructor function that can be used to construct instances.  Like class declarations, you may also declare property members and index members, as well as use the constructor parameters.
Examples
var Rect = class {
  area: number;

  constructor(public length: number, public width: number) {
    this.area = this.length * this.width;
  }
}
var rect = new Rect(5, 10);

var MyNode = class Node {
  next: Node;

  set nextNode(node: Node) {
    this.next = node;
  }
  constructor() {  }
}
var node = new MyNode();
var nextNode = new MyNode();
node.nextNode = nextNode;

Emit
Class expressions, just like class declarations, create a constructor function.  We would emit them similarly to how we do now, with the exception that they are not immediately bound to a variable.  Instead, they create the function expression (IIFE) that builds the constructor function, and allows the developer to assign the result of this expression where appropriate.
Emit of .d.ts
The type emitted for class expressions is the object literal representation of the constructor functions, just as with other anonymous types.
Class expressions can also introduce recursion in the type literal.  Anonymous types that have possibly recursive components (like the Node example above) will need special treatment, which should be part of a related spec (Edit: #517).

Something that just came to mind: will constructors get contextually typed?


Phosh runs on top of Steam

Describe the bug
I'm trying different desktop sessions to use in conjunction with Steam.  (I'm using the Jovian flake).
Other sessions like plasma-mobile or gnome can be passed into jovian.steam.desktopSession.  Steam+gamescope will autostart, but you can exit from Steam to your desktop session.
Phosh takes over the whole screen as soon as your flake rebuilds.  It doesn't play nicely with the surrounding system.
Steps To Reproduce
Steps to reproduce the behavior:

Add Phosh to your flake:

    services.xserver.desktopManager.phosh = {
      enable = true;
      user = config.mainUser.userName;
      group = "users";
    };

Expected behavior
Phosh doesn't autostart unless an option like services.xserver.displayManager.gdm.enable is set, and plays nicely with other sessions like gamescope.
Actual behavior
Phosh takes over the entire screen as soon as the flake rebuilds.  I can hear Steam running in the background, but I can't access it.
Additional context
I think the problem is that it's being run by systemd instead of exposing config.passthru.providedSessions like the other compositors do.
Notify maintainers
@zhaofengli @tomfitzhenry @masipcat

Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.2, NixOS, 24.05 (Uakari), 24.05.20240102.2e5d2c5`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

I just removed Phosh from the flake.  Now I'm stuck in a sessionless purgatory because Phosh didn't clean up its Wayland session upon exit (similar to https://github.com/hyprwm/Hyprland/issues/4226).
systemctl --user show-environment

shows that WAYLAND_DISPLAY is set to wayland-0 (when it should be unset during session exit).

This sounds more like a jovian issue than a generall nixos one.

I disagree.  This is a Phosh packaging issue.
As explained in the original report, Phosh is not exposing a providedSession (as the other desktop environments do), and it takes over the entire screen as soon as the flake rebuilds.  Neither of those have anything to do with Jovian.  They are mistakes in the packaging that make Phosh not play well with the rest of the ecosystem.


MSL: Implement support for EXT_mutable_descriptor_type and general aliasing with argument buffers

Here be dragons, illegal pointer casts, and all sorts of fiendish constructions.
Verified that MSL codegen should work. texture types are represented as pointer to internal texture type in AIR.
Supersedes #2292.

Here be dragons

Nice one. Gave me a chuckle. 😉


[UDP Proxy] Fix a bug in UDP session filter

Commit Message:
Additional Description: onSessionComplete() is called twice when a session filter is a read write filter, because "ReadFilter" and "WriteFilter" is declared without virtual inheritance.
Risk Level: low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

/retest

@danzh2010 @yanavlasov Can you, please, take a look on this PR as you were the reviewers of my previous PR:
[UDP Proxy] Add onSessionComplete for UDP session filters (https://github.com/envoyproxy/envoy/pull/31094)

@yanavlasov ping

Assigning Dan as codeowner
/assign @danzh2010

Assigning Yan as a senior-maintainer.
/assign @yanavlasov


Implement toggle buttons for ReqRes Component

[x] Implement toggle buttons for ReqRes Component

Complete. First Pass 🤞


⚠ Bofrost.fr has degraded performance

In 0df03e1, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1386 ms

Resolved: Bofrost.fr performance has improved in b017baf after 11 minutes.


chore: merge develop into master for v0.4.4 release

Purpose
We want to increment the version and publish the package for two reasons:

a new dependency has been introduced (fs)
new attributes has been created in the Grid class that are used in PreREISE

Release notes
How is the purpose executed?
Closed issues

Enable getting profiles from scenario in Execute state (#522)

Merged pull requests (features)

access onshore and offshore wind profiles (#505) @kasparm
support getting profiles from execute state (#523) @jon-hagg
support passing extract_data for container and local simulations (#525) @jon-hagg
make build_supply_curve work with any combinations of generation types (#533) @BainanXia

Merged pull requests (fixes, etc.)

rename function (#518) @rouille
fix N818 warning from pep8-naming (#519) @jon-hagg
create class variables for supported models and engines (#517) @goccert25
update Scenario documentation to reflect simpler syntax (#521) @danielolsen
fix typos in docstrings (#524) @danielolsen
add clp to list of allowed sovlers (#526) @danielolsen
add get_base_grid() to states (#528) @goccert25
use setup.cfg (#530) @jon-hagg
add workflow to build and upload package (#531) @jon-hagg
simplify data access using pyfilesystem (#520) @jon-hagg
bump version number to to v0.4.4 (#535) @rouille

Time estimate
5min

Codecov Report

Merging #534 (09d18a4) into master (e480a62) will increase coverage by 0.11%.
The diff coverage is 46.95%.

@@            Coverage Diff             @@
##           master     #534      +/-   ##
==========================================
+ Coverage   74.08%   74.20%   +0.11%
==========================================
  Files          98       99       +1
  Lines        7602     7424     -178
  Branches      983      969      -14
==========================================
- Hits         5632     5509     -123
+ Misses       1790     1736      -54
+ Partials      180      179       -1

Impacted Files
Coverage Δ

powersimdata/data_access/tests/test_execute_csv.py
86.84% <0.00%> (+2.03%)
:arrow_up:

...owersimdata/data_access/tests/test_scenario_csv.py
85.07% <0.00%> (+2.21%)
:arrow_up:

...imdata/design/generation/clean_capacity_scaling.py
28.72% <0.00%> (-0.16%)
:arrow_down:

powersimdata/design/investment/investment_costs.py
70.47% <0.00%> (+0.70%)
:arrow_up:

powersimdata/design/transmission/mwmiles.py
59.57% <0.00%> (-0.85%)
:arrow_down:

powersimdata/design/transmission/upgrade.py
85.92% <0.00%> (ø)

powersimdata/input/change_table.py
70.55% <ø> (+0.17%)
:arrow_up:

powersimdata/input/input_data.py
71.01% <ø> (-0.42%)
:arrow_down:

powersimdata/output/output_data.py
24.07% <ø> (+0.43%)
:arrow_up:

powersimdata/scenario/delete.py
0.00% <0.00%> (ø)

... 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 e480a62...09d18a4. Read the comment docs.


support multiple comma-separated text prompts for grounded sam demo

Made a small change to support multiple comma-separated text prompts in the grounded sam demo.
Here is an example running the script with --text-prompt "bear, gravel"
grounded_sam_output.jpg

mask_bear.jpg

mask_gravel.jpg

Here is another example running the script with --text-prompt "rock, boat"

Here is an example running the script with --text-prompt "rock, boat"

Nice work~ we will review this today~ thanks for your contribution!

Thanks for your PR. We suggest separating names with . for multiple objects, resulting in better performance. The comma may make the sentence ambiguous.

Thanks for the feedback. I made the requested change.

@SlongLiu  My apologies, I didn't realize that functionality was already present in Grounding DINO. Then yes, we can close this PR, though I might suggest mentioning the multiple text prompt ability in the readme for this repository as well. Thanks for your time!


feat(examples/*): enhance spoke vms - enable ssm-agent

Description
The ssm-agent needs to be presented in every spoke VM to access from session manager. The proper role was attached.
Motivation and Context
#47
How Has This Been Tested?
The loop waits for Internet connection for yum update - The ssm-agent needs public endpoint as well.
Screenshots (if appropriate)

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[ ] All new and existing tests passed.

/idempotence paths="examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale"


docker-compose crashes on initialization

Environment
Self-Hosted (Docker)
System
ubuntu server 20.04
Version
1.9.9 (arm64v8 docker image pulled today)
Describe the problem
fails to start.
says something is wrong with the config file.
Checking config file against schema...

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
  Warning: 29 issues found in config file
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
1. additionalProperties must NOT have additional properties in /appConfig
2. additionalProperties must NOT have additional properties in /appConfig
3. additionalProperties must NOT have additional properties in /appConfig
4. additionalProperties must NOT have additional properties in /appConfig
5. additionalProperties must NOT have additional properties in /appConfig
6. additionalProperties must NOT have additional properties in /sections/0/items/0
7. additionalProperties must NOT have additional properties in /sections/0/items/1
8. additionalProperties must NOT have additional properties in /sections/0/items/2
9. additionalProperties must NOT have additional properties in /sections/0/items/6
10. additionalProperties must NOT have additional properties in /sections/0/items/7

my docker-compose.yml is the following:
---
version: "3.8"
services:
  dashy:
    image: lissy93/dashy:arm64v8
    container_name: Dashy
    deploy:
      resources:
        limits:
         cpus: '0.5'
         memory: 350M
    volumes:
      - '/home/ubuntu/dashy/conf/dashy_conf.yml:/app/public/conf.yml'
    ports:
      - 4000:80
    # Set any environmental variables
    environment:
      - NODE_ENV=production
    # Specify your user ID and group ID. You can find this by running `id -u` and `id -g`
      - UID=1000
      - GID=1000
    # Specify restart policy
    restart: unless-stopped
    # Configure healthchecks
    healthcheck:
      test: ['CMD', 'node', '/app/services/healthcheck']
      interval: 1m30s
      timeout: 10s
      retries: 3
      start_period: 40s

the same happens if I remove the resources limits. previously, the same config was working fine (same docker-compose.yml file). similarly, dashy_conf.yml has not changed.
Additional info
No response
Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

I close this issue because I replaced dashy with another tool

@mybuntu-git2 what tool are you using now ?


Figure out Major Restricted Parsing

GET EXCITED BOYS! THIS WILL BE IMPLEMENTED SOON!!! 😱

HOLY POGGERS! PROBLEM SOLVED!


Why cant webtransport trigger http3 data event?

I changed the headers in accordance to https://github.com/cloudflare/quiche/issues/1132#issuecomment-1018293465 and the connection got established in https://googlechrome.github.io/samples/webtransport/client.html. But when I send a bidirectional stream, the event quiche::h3::Event::Data is not triggered. I think the bidirectional frame has this format: https://www.ietf.org/id/draft-ietf-webtrans-http3-02.html#name-bidirectional-streams. Any help would be appreciated.

WebTransport changes the semantics of an HTTP/3 connection. That requires changes into quiche's h3 library to support branching between operating as a conventional HTTP/3 endpoint OR a WebTransport over HTTP/3 endpoint.
For instance, sending DATA frames as the first frame on new bidirectional streams is not permitted by the  HTTP/3 specification; see https://datatracker.ietf.org/doc/html/draft-ietf-quic-http-34#section-4.1. quiche enforces this rule; you won't see a quiche::h3::Event::Data in that circumstance.
But WebTransport over HTTP/3 doesn't use DATA frames. https://datatracker.ietf.org/doc/html/draft-ietf-webtrans-http3-02#section-4.2 sayes

WebTransport endpoints can initiate bidirectional streams by opening
an HTTP/3 bidirectional stream and sending an HTTP/3 frame with type
WEBTRANSPORT_STREAM (type=0x41).  The format of the frame SHALL be
the frame type, followed by the session ID, encoded as a variable-
length integer, followed by the user-specified stream data
(Figure 2).  The frame SHALL last until the end of the stream.

As per HTTP/3, quiche ignores unknown frame types. To support Webtransport, quiche needs additional code for frame parsing and other stream-state handlers. There's no quick win here. quiche needs to be modified to support a semantic extension and it needs to apply the rules of each specification appropriately in all circumstances and variants concurrently.

Thanks for the reply :) I was able to edit the frametype, and get a bidirectional stream connected and got the data. I am trying to build a simple echo server (get data on the bidirectional stream, and send the same data back on the same stream) to send data back on the bidirectional stream, should I use the underlying quiche's stream_send() method, or the h3.send_body. I am able to send some data with stream_send() but sometimes it throws Error::done, but h3.send_body throws stream: FrameUnexpected, is this because of the same issue as recieving? Does this mean, I need to alter h3.send_body also, rather than using the underlying stream_send()?

If you're tying to send on bidirectional streams, quiche's HTTP/3 layer will also enforce the rules I mentioned above. It sounds like you're hitting that with the FrameUnexpected error on send_body()
It is possible to call stream_send() directly. However, as a long term solution this poses some problems because of the possibility of things becoming desynchronized. As a workaround for experimentation it should work. You should expect Error::Done to be returned when the stream doesn't have enough capacity, this is normal due to flow control or congestion control https://docs.quic.tech/quiche/struct.Connection.html#method.stream_send. The app will need to pause stream writes, flush packets, and enter the read loop to process packets that might grant more credits.

Got it, thanks. Is there a document on how to prevent/handle the Error::Done issue?

Ok, so I was able to make send_body() work, the problem was I hadnt initialized the local part of the stream, so this condition was being hit:
(https://github.com/cloudflare/quiche/blob/master/quiche/src/h3/mod.rs#L1000)
if !s.local_initialized() {
    return Err(Error::FrameUnexpected);
}

However I am receiving some extra bytes in the webtransport chrome example. If I send [u8] containing [119, 119, 119] I receive in chrome [0, 3, 119, 119, 119]. I suspect it is some extra data that quiche is sending that is required by the http3 protocol, but webtransport is perhaps parsing it as payload data.
Why would this be happening? Any insights into whats going on?

WebTransport streams are not HTTP/3 requests or response. WebTransport rewrites the rules for bidirection streams, see https://datatracker.ietf.org/doc/html/draft-ietf-webtrans-http3-02#section-4.2, and quiche's HTTP/3 layer does not support other things redefining the rules. So fighting against quiche is not going to be a path to fulfillment.

I suspect it is some extra data that quiche is sending that is required by the http3 protocol, but webtransport is perhaps parsing it as payload data.

Yes, this is the DATA frame, which is how request and response body is sent - https://datatracker.ietf.org/doc/html/draft-ietf-quic-http-34#section-7.2.1
If you just want to experiment, the is would sufice to service the Extended CONNECT request for session establishment and then start using quiche's stream_send()/stream_recv() directly. This is, however, not a very sustainable long-term solution. Instead, quiche's h3 layer should be modified as I already mentioned.

makes a lot of sense. I have modified the h3 layer only and the data being sent is what is being received back, so thank you for your help :)
The new problem now is that if I send data from the webtransport client at a rate higher than 2 megabits per second, some of the data never makes it to my quiche app. I suspect that congesion control is playing a role there, and will be probing the new reno congestion algorithm in quiche.
But other than that, so far it seems to be working. Thanks 🥧.

The congestion control and flow control aspect should be common no matter what happens at the application. You'll have some capacity to send with, and stream_send() will consume it. The only way to get more is to send and receive packets to/from the peer. If you get the sequence right, then it's possible to get use most of the available network capacity. Once in the 10 gigabit/s other factors start to come into play.

quiche doesn't formally support WebTransport. The interaction between HTTP/3 and WebTransport is a known constraint that applies to any form of HTTP/3 semantic extension. So I don't see this as any specific issue with quiche. It seems like you've been able to work around these constraints, which should allow experimentation. Therefore I'm going to close this issue.
If there are quiche-specific issues that you identify beyond these known constraints, feel free to open a new, separate issue.

Hi @kalradivyanshu, nice to see you experimenting with WebTransport here! Do you keep a public fork with your modifications for WebTransport? I'm interested in checking this out, too! Cheers

Hey, unfortunately the code is part of a NDA I have signed. However if you have questions, I can answer them. Do note, I havent implemented webtransport 100% yet, I am still struggling with throughput. Cheers.

Hey @LPardue I am now able to handle flow control much better. Thanks for the advice there. However I am still struggling to get the bitrate to go above 10 mbps (for one connection), (I am not hitting Done error or congestion, and the server bandwidth is about 4 - 8 gbps), my current hypothesis is that it is because of I am using the default rust std::net::UdpSocket I came across this blog: https://blog.cloudflare.com/accelerating-udp-packet-transmission-for-quic/ but couldn't find any library that implemented UDP GSO for rust, except for one, which has been abandoned. Is my hypothesis correct? Or am I looking in the wrong direction? I am using tokio for my event loop, and running the server on a single core, and the client on a separate  core.
Any help is appreciated, thanks :)

You should definitely be able to get over 10 mbps even without GSO. You'd need to look into the debug and/or qlogs to figure out what's really happening here.

Oh okay I will do that. At what point will I start hitting the limit with std::net::UdpSocket? Can I push it to 1 gbps?

There's a load of potential factors at play, the best bet is to benchmark UDP perf with iPerf (beware caveats  so read up on those) and use that as your baseline for QUIC perf.
As per the blog, you can modify socket behaviour with cmsg. So using something like nix crate will give access to more options configuring socket. For example see https://github.com/cloudflare/quiche/pull/1138


Menu/Breadcrumb: Capability to navigate to sub-views

I'm submitting a ...
[ ] bug report
[ X ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Feature

What is the current behavior?
You select an item in the menu and in the view that shows up, you navigate to another view and from here the breadcrumb is still pointing to the previous view.

Moreover, if you refresh the browser, you loose the menu selection and the breadcrump is empty after the Home.

What is the expected behavior?
The breadcrumb handles sub-views like any other breadcrumbs
The Menu keeps the main selection
Both works after a browser refresh

What is the motivation / use case for changing the behavior?
Navigate to sub-views

you can activate in your component the correct active link:
 private state: GlobalState
this.state.notifyDataChanged('menu.activeLink', { title: 'component.title' });

Thanks Newan but that doesn't work as there is no link in the menu corresponding to that view.
Let me describe you a use case:
You have a list of users (link in the menu available) and you want to update one of them.
Then, you navigate to a new view (sub-view) to edit it.
Here I would need the breadcrumb to be updated with the user I'm changing and when I refresh this sub-view, I want the User link in the menu to be reselected and the breadcrumb updated with the right value.

I changed the match of the menu with the URL to be less restrictive and it works. Main menu is selected now for sub-views.


Remove onDidChange file watcher

@connor4312 I don't think this is required anymore with the new direction to subscribe to the vscode events for document changes, it confused me for a bit.
https://github.com/microsoft/vscode-extension-samples/blob/8fdf822985243c42aa7ed922a9a6c1fc5540c478/test-provider-sample/src/extension.ts#L114

I guess related, what if a test file changes via activity not currently in a vscode window? Seems to me the current updater won't detect it, so maybe this needs to be preserved and rewired back up to discovery? And if a vscode window is open, if this is wired up too how do you prevent double re-discovery?

what if a test file changes via activity not currently in a vscode window? Seems

That is actually what the file watcher is for. You're correct that document changes will generally pick up changes made inside a VS Code editor, and the watcher will pick up any changes that happen outside of there.
Double/re-discovery is not really a big problem. It's not optimally efficient, but the sample is fast enough and an extension could optimize it further if they wanted (e.g. by keeping a quick hash of the file contents and seeing if it's changed or not since the last discovery).

@connor4312 Thats my point though, at least currently the onDidChange filehander contentChange event isn't wired to anything anymore, it just fires into the ether, unless I'm missing where it's wired up to.

Oh, right! Misread the issue and/or the code 😄


Improved cross-platform support

The changes allow the program to load the user's home directory programmatically and use cross-platform compatible methods to move files.

LGTM! Thanks for your contribution.


Plugins aren't working with multiple Zustand instances

Describe the Bug
I have a setup with multiple Zustand instances. When I disable the plugins like this:
<ReactFlow
	nodes={nodes}
	edges={edges}
	onNodesChange={onNodesChange}
	onEdgesChange={onEdgesChange}
	onConnect={onConnect}
	onConnectStart={onConnectStart}
	onConnectEnd={onConnectEnd}
	nodeTypes={nodeTypes}
	edgeTypes={edgeTypes}
	proOptions={proOptions}
	// Interactive
	elementsSelectable={!running}
	nodesDraggable={!running}
	nodesConnectable={!running}
	>
	{/* <MiniMap />
	<Controls showInteractive={false} />
	<Background /> */}
</ReactFlow>

I get no error. (Note that my <Reactflow /> component is wrapped around a <ReactFlowProvider /> as I am using useReactFlow() in the same component.
But when I activate the plugins, this is the error I am getting:

Also note that the errors are actually happening inside the plugin components themselves:

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
N/A
Expected behavior
I expect the plugins not to throw this error.
Screenshots or Videos
No response
Platform

OS: [e.g. macOS]
Browser: [e.g. Safari]
Version: [e.g. 16.1]

Additional context
No response

can you provide a codesandbox so that we can reproduce this?

Here is a copy of the project I'm working on:
Eula-Reactflow.zip
Just type yarn dev to start the web server.
You'll find the <ReactFlow /> instance in packages/editor/src/Editor.tsx

The problem is that you installed different versions of React Flow. Update the RF version of the editor to "^11.3.2" and it should work.


Can't load Kernel binary: Invalid SDK hash. No active package get_cli.

console error when input get command in terminal
Can't load Kernel binary: Invalid SDK hash.
No active package get_cli.

flutter version is 3.0.1 on stable channel
but get was worked on flutter 3.0.0

its work
flutter pub global activate get_cli


When resize a png with square block, it's changed

I have a image that with qrcode, when I resize and zoom in, I find that it changes and lower the recognition capability.
Origin:

Resized:

Scaling down small images always has the risk of deformations, so it's always best to generate these kinds of images in the right size instead of resizing it afterwards.
Having said that, ChunkyPNG has two resize algorithms: nearest neighbor and bilinear. Nearest neighbor is the default;  maybe bilinear has better results for you? Try image.resize_bilinear(width, height)


[Linux] Does not launch on Linux with error FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable

Issue
Running NetPad on some Linux environments results in the app crashing with an error:
[300044:0103/022825.743082:FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable. Goodbye.

This appears to be an issue exhibited by a number of Electron-based apps:

Atom: (https://github.com/atom/atom/issues/23608)
SimpleNote: (https://github.com/Automattic/simplenote-electron/issues/3044)
Useful info: Link

Workaround
Until this is fixed, a workaround is to run with the --no-sandbox option. ie:
netpad --no-sandbox

or you can modify /usr/share/applications/netpad.desktop and add --no-sandbox to Exec like this:
[Desktop Entry]
Name=NetPad
Exec=/opt/NetPad/netpad --no-sandbox %U

Other useful information can be found in the linked issues above.
Note
If you ran NetPad and it crashed because of this, the app might still be running in the background. Kill the app process(es) using your task manager before attempting to run it again.

Fixed in new NetPad v0.2.0 release. Fix comes from upstream Electron.


MSBuild fails with "no compiler found"

TypeScript Version:  3.2.0

Search Terms: TypeScriptToolsVersion different version msbuild cordova
Expected behavior: MSBuild should succeed without errors for a project using TypeScript.
Actual behavior: Build fails with error:
Error		Your project file uses a different version of the TypeScript compiler and tools than is currently installed on this machine.  No compiler was found at .  You may be able to fix this problem by changing the  element in your project file.

Applies to Dev14/Dev15 with Typescript SDK install only (Nuget packages aren't affected). In this scenario anything above TypeScript 2.3 already gives compilation errors due to the Cordova typings being out of date.


no more empty tests

fix #1

Thanks! I changed the commit message a little bit and merged.

@mattst88 Thanks for merging; I'm happy to help!


Methodology for determining last successful pipeline run doesn't work if there are multiple kinds of pipelines

Orb version: 1.1.2
What happened:
We have a setup for our CD system where we run a particular pipeline when new code is merged to main. That pipeline uses the nx affected commands for building and testing.
When that pipeline is finished, our code is deployed. The deployment event triggers a second pipeline to run which executes e2e tests. That pipeline does not include any Nx commands.
The orb currently interprets a successful completion of either one of these pipelines as the "last successful" run on the main branch. However, only the first "build & test" pipeline is the one that should be considered as a "run". This causes commands that use nx affected to receive the wrong sha.
Expected behavior:
Ideally, the orb would handle for this case by filtering the set of pipelines its getting from the CircleCI API using the name of the pipeline that it's currently running in (if that's possible). Or provide a parameter that can specify the name of the pipeline to watch for.

Thank you @ajwootto for creating the issue.
Your proposal is correct, having an optional second parameter to specify with workflow to use would solve your issue.
This is a similar PR for GitHub action we just merged in today - https://github.com/nrwl/nx-set-shas/pull/7.
If you want you can propose a PR for this.

Fixed by #9


Simplify dealing with multiple weapons for (N)PCs

Instead of requiring an individual config script per weapon, it would be much easier -- in particular for the busy GM with his hordes of NPCs -- if there was a generic weapon selection script that could be called for any token/character sheet. It would

look up the character's available weapons (ideally, the token's available weapons, since Barbarian 2 may have already lost his battleaxe even though it would still be listed in the Generic Barbarian character sheet, as indicated by an appropriate status marker);
generate some means for the player to select which weapon to use for attacking (or, for that matter, for defending if a warrior wants to switch between a shield and a parry weapon, for example) -- the most feasible solution seems to be generating a Roll20 chat menu;
somehow transmit the necessary information about the chosen weapon (its ID in the character sheet) to the attack script that is being called. This may be possible without new MMM functionality by simply storing that ID in a new attribute with the respective character -- something worth trying, I guess.

So we can easily generate and display a list of a character's available weapons:
!mmm script
!mmm   for attrName in findattr(sender, "Angriff", "Waffe")
!mmm     do whisperback(getattr(sender, attrName))
!mmm   end for
!mmm end script

attrName is the internal ID for the weapon entry in the attack table, so we can get all the weapon's other attributes by looking it up like this:
!mmm set weaponDamage = getattr(sender, findattr(sender, "Angriff", "Waffe", getattr(sender, attrName), "WaffeSchaden"))

Now we have a problem, though. How do we transmit the weapon ID if a chat menu can only call static commands/scripts, and MMM cannot generate scripts (if it could write into abilities rather than just attributes, we could easily generate a customize script, for example).
Building a chat menu generator that would take a particular character, pull out all the weapons and build the chat menu is easy.
Expanding the attack/defense scripts such that they take a stored weapon ID to determine the weapon details is equally easy.
But how to get the weapon ID from the chat menu into some place where it can be retrieved...?

One somewhat ugly but functional solution, at least temporarily, would be to manually create a centralized batch of config scripts per weapon that can be used for any NPC. Like this:
!mmm customize
!mmm   set cWeaponLabel = "Magierstecken"
!mmm   set cWeaponType = "Stabwaffe"
!mmm   set customizable cHealthAttr = "bar3"
!mmm   set customizable cEnduranceAttr = "bar2"
!mmm   set customizable cOwnID = "@{selected|token_id}"
!mmm end customize
%{MacroSheet|meleeAttack}```

We would need probably two dozen of those to cover everything including natural weapons (claws etc.) that our GM's NPCs use to kill us, and occasionally the GM would have to copy & paste this for a new weapon.

Once we had those, we could build a chat menu for weapon selection like this:

```!mmm script
!mmm  set token = "@{selected|token_id}"
!mmm  combine chat
!mmm    chat: /w GM ${"&"}{template:default} {{name=${token.token_name}: Angriff mit...}}
!mmm    for weaponLabel in findattr(token, "Angriff", "Waffe")
!mmm      chat: {{ [${token.(weaponLabel)}](~MacroSheet|npcAttackWeapon_${token.(weaponLabel)}) }}
!mmm    end for
!mmm  end combine
!mmm end script

This works already for the weapon Magierstecken in our TEST game (all the druid tokens have access to it). Need to find a solution to deal with spaces in weapon labels, though...

I don't see a solution for weapon labels with spaces, without https://github.com/michael-buschbeck/mychs-macro-magic/issues/31 to escape/replace them somehow.

We implemented an ugly but workable workaround for now in which our GM created all these scripts and renamed his NPC's weapons without using spaces.

Check if new features allow for auto-selecting only weapons if just one is available for a particular type of attack (melee, ranged) or defense. Might depend on syntax for manipulating struct data types, but maybe not.

fafcb1638228d729f03d7cf45dd6f42af178420a (defense 1.10.0) automates defense weapon selection for standard cases:

If there is only one defense "weapon"
If there are more but one is called Abwehr ohne Schild

Generating selection menus requires that for each menu item, a separate customize script is available and assigned. For players, given the problem with spaces identified above, I'm leaving this to everyone's own initiative and liking.

So there is another idea. MMM scripts can write attributes, and attributes can contain executable user controls of the ?{...} variety. Every player's autorun script could generate a set of weapons selector dropdowns this way and store them in an attribute used by the respective attack script to ask the player to select their weapon of choice. Would relieve us of the need for manually maintained config scripts per weapon if a single weapon identifier would be sufficient to grab all the weapon info -- and since some weapons have quite a few complicated pieces of data attached to them, this might work

only for standard weapons (could help for all the standard stuff, NPCs and unarmed combat defaults)
for any weapon only once MMM allows storing structured data (such as JSON) in attributes.

/dev/mgd/weapons-selectors has the beginnings of an implementation:

For all selected tokens, [buildWeaponSelector](https://github.com/phylll/mychs-macro-magic/blob/dev/mgd/weapons-selectors/examples/midgard/tools/buildWeaponSelector.mmm) puts together three separate Roll20 weapons selector dropdowns for defense, melee and ranged attacks, and stores them in the linked character's attributes m3mgd_defense_weapon_selector, m3mgd_melee_weapon_selector and m3mgd_ranged_weapon_selector. For generic NPCs, this means that you only need to select a random token based on that generic NPC sheet, and the sheet will get its weapon selector attribute.
cc08bacc197598bc33124bad873ba89060d96eff has functions to provide the data for which we have so far needed customization scripts for every weapon. For defense and melee attack weapons, this is only the weaponType according to the lists of valid weapon types. For ranged attack weapons, the code returns a list including the type, low/mid/far range and a standard name for an ammunition attribute.

TODO:

Write a script that checks new (generic or not) (N)PCs for being ready for our combat scripts, adds the weapons selectors, perhaps the ammo attributes, as well, and tells the player a list of manual todos for non-standard weapons.
Modify combat scripts to pull missing info automatically (but still prioritizing customizations when present, to allow for special or magic weapons).
Clean up character sheets and remove newly redundant customization scripts.

Per bfb57bc3971ae0b0c0d8cf1c3120ec15cf951607, there is a better system of weapons selection implemented that does not need selector attributes in every character sheet. All the above-described functionality is available, except for the automatic check for (N)PC sheets if ranged attack ammunition attributes are present. charCombatValidator could easily expand in that direction, though.
Ready to close with next PR.

Closed via https://github.com/michael-buschbeck/mychs-macro-magic/pull/188


GH2628: Add Jérémie to copyright & nuspec

fixes #2628

@devlead your changes have been merged, thanks for your contribution 👍


tetragon: detect execve of unlinked inodes

Detect execution of anonymous binaries, these are binaries that are not linked on a filesystem, where nlink is zero.

An anonymous shared memory file
https://man7.org/linux/man-pages/man7/shm_overview.7.html.
An anonymous file obtained with memfd API
https://man7.org/linux/man-pages/man2/memfd_create.2.html.
Or it was deleted from the file system.

"process_exec": {
   "process": {
      "binary_properties": {
        "file": {
           "inode": {
              "number": "182698",
              "links": 0
            }
         }
      }
   }
}

Other than comment about pid for map key. LGTM.

Other than comment about pid for map key. LGTM.

Ok updated fixed all, but also removed that empty "info": {} , now entries that don't have:  "inode.initialized == 1 && inode.nlinks == 0" will not have any displayed info field.
{
  "process_exec": {
    "process": {
      "exec_id": "OjYwMjQ1NTcxNTg2MDU6MTA5MzY2",
      "pid": 109366,
      "uid": 1000,
      "cwd": "/home/tixxdz/work/station/code/src/github.com/tixxdz/tetragon",
      "binary": "/proc/self/fd/3",
      "flags": "execve",
      "start_time": "2022-10-22T22:22:13.554073460Z",
      "auid": 1000,
      "parent_exec_id": "OjYwMjQ1NTYwNjA2ODI6MTA5MzY2",
      "info": {
        "inode": {
          "deleted": true
        }
      }
    },
    "parent": {
      "exec_id": "OjYwMjQ1NTYwNjA2ODI6MTA5MzY2",
      "pid": 109366,
      "uid": 1000,
      "cwd": "/home/tixxdz/work/station/code/src/github.com/tixxdz/tetragon",
      "binary": "/usr/bin/memfdloader",
      "arguments": "/bin/true",
      "flags": "execve clone",
      "start_time": "2022-10-22T22:22:13.552976581Z",
      "auid": 1000,
      "parent_exec_id": "OjExMjc0MDAwMDAwMDA6MTA0MTU=",
      "refcnt": 2
    }
  },
  "time": "2022-10-22T22:22:13.554073906Z"
}

Don't merge yet, I will convert it to follow base execve v6.0 kernel versions

Reviving this

Thanks! I have added some comments. Nothing very difficult to change, but good to have these as well before merging.

@tpapagian much appreciated for the review ;-) ! all comments handled , let's see if green


A way to configure the window size

I really like the plugin!
Only thing i need now is to make this window a bit smaller, that would be perfect is this possible?

Just saw the settings :)


Issue with the example

I get the following error:
Neither the property "numberOfWheels" nor one of the methods "getNumberOfWheels()", "isNumberOfWheels()", "hasNumberOfWheels()", "__get()" or "__call()" exist and have public access in class "Lucky\RNDBundle\Form\CreateVehicleStep1Form"

@pavan168 I want to repeat a step within the flow form. Have you been able to achieve this? It is in your 2. question.


fix: check str type before using string method in pay.utils.dict_to_xml

check str type before using string method in pay.utils.dict_to_xml

Thanks!