Support Spring 6 HTTP interface

The Spring Framework 6 supports define declarative HTTP services using Java interfaces like Feign.
https://www.baeldung.com/spring-6-http-interface
We can also support this by providing HttpClientAdapter
It already work if WebFlux integration module is used.
interface BooksService {

    @GetExchange("/books")
    List<Book> getBooks();

    @GetExchange("/books/{id}")
    Book getBook(@PathVariable long id);

    @PostExchange("/books")
    Book saveBook(@RequestBody Book book);

    @DeleteExchange("/books/{id}")
    ResponseEntity<Void> deleteBook(@PathVariable long id);
}

WebClient webClient = ... // WebFlux webClient which is created via auto configuration
HttpServiceProxyFactory httpServiceProxyFactory = HttpServiceProxyFactory
  .builder(WebClientAdaptor.forClient(webClient))
  .build();
booksService = httpServiceProxyFactory.createClient(BooksService.class);

I realized that this already works if Armeria WebFlux integration module is used.
Please let us know if you want to use this feature without WebFlux.

We've changed our integration for Spring 6 HTTP interface to provide HttpClientAdapter. #4912

I‘m interested in this 🤔

Usually, we don't tell contributors a schedule, but this issue needs a schedule. We are planning to implement it in 3Q because there is a desire for this feature at LINE internally.
The non-draft PR should come out by mid-August because review time is needed to match the release schedule to 3Q. It seems possible because we have enough time.
If you can meet this schedule, I'd like to ask you to implement this function.
Once a non-draft is out, I'll review it and make some commits if necessary.
Anyway, thanks for your interest. 🙏

Oh yeah. :smiley:
I was implementing some of the features, but I think they could be better implemented if we reviewed them internally. (If he/she post a PR, I'll review it together).

https://github.com/heowc/armeria/commit/fd1e928527b580223db45cf9fd40a3bdcc2a293e

If you already have implemented the feature. Please continue to implement and finish.

We are going to add the Spring 6 HTTP interface to armeria-spring6 module since it is not related to Spring Boot. Hence, we don't need to consider Spring Boot 3.2 schedule.

I agree 😎

I have a question.
For spring related clients, there are mostly internal converter services like armeria's annotation service that look at the content-type or body type and make the appropriate conversion, but for armeria clients, the user selects and converts.
If we implement that feature, will we do it like for spring related clients? Or like armeria client?

I have a question.
For spring related clients, there are mostly internal converter services like armeria's annotation service that look at the content-type or body type and make the appropriate conversion, but for armeria clients, the user selects and converts.
If we implement that feature, will we do it like for spring related clients? Or like armeria client?

This question may be related to this issue.

https://github.com/line/armeria/issues/4910

For spring related clients, there are mostly internal converter services like armeria's annotation service that look at the content-type or body type and make the appropriate conversion, but for armeria clients, the user selects and converts.

What content-types are supported by Spring 6? As JSON is a dominant serialization format for REST API, why don't we start with JSON first? We may add an extension API to add a custom converter for requests and responses like Retrofit later.
https://github.com/line/armeria/blob/41d097efd92446d60f034cce3bee9a26f0ba47f9/retrofit2/src/main/java/com/linecorp/armeria/client/retrofit2/ArmeriaRetrofitBuilder.java#L153-L156

Linking to https://github.com/line/armeria/pull/5370

Closed by #5370


Dou consider adding a request timeout setting and judgment?

Hi, in your code
iso2_code4=$(curl -4 -sS https://chat.openai.com/cdn-cgi/trace | grep "loc=" | awk -F= '{print $2}')

iso2_code6=$(curl -6 -sS https://chat.openai.com/cdn-cgi/trace | grep "loc=" | awk -F= '{print $2}')

does not add timeout setting, it may block the run for a long time, do you consider adding a request timeout setting and judgment?
Otherwise the following will occur:

This is a very good suggestion. I'll optimize it later when I have time. @spiritLHLS

Any progress?

I'm sorry about that. You have to understand that this is an open source project, not my work.

I understand and thank you for the project.


Errors on TRWPeak

This is just a note on something I saw today. I was fitting peaks with a TRWPeak and I repeated the same command by mistake (pushed up and hit enter) and it gave me a different values for the error on the fits. The central values were the same and the fits were effectively identical, this can be seen in the screenshots. I didn't change any parameters in the fit between repeating the command, but the errors bounced back and forth between two values of +/-1173 and +/-700 seemingly randomly. I looked at the covariance matrix of the fit and tried to reproduce the error but I could not reproduce it. I tried a few other things as well, making sure I defined the fit the same way in each circumstance but still could not reproduce the error. This is primarily a note to let people know that I saw this effect, although if anyone else is able to reproduce it in the future it would be good to understand why this happened. This might be more of a root error minimization issue than a grsisort issue but I think people should still be aware that it was seen.
Additional information:
grsisort version - <IP_ADDRESS>
root version - 6.08.06
Screenshots:

Was it in fact oscillating? Or was this just a case of fitting a peak twice and having it find different minimums starting with initial guesses? If you fit it a third time what did you get? If you start at a minimum, the errors estimation could give you erraneous results (this is normal of this gradient error estimation procedure)

I should add that the bg_offset is limited to the fit range I believe. (Maybe that's an incorrect thing to do?) Either way, you are crashed up against that limit, and that could cause errors in the uncertainty estimation of that variable as the gradient can't be calculated below 190.

The fit was repeated multiple times and the uncertainties changed every single time (but only those, the actual values of the parameters stayed the same). You can actually see that the uncertainties reported by root for A and B almost double in the second fit, while bg_offset didn't really change.
The more puzzling thing was that we couldn't reproduce this behaviour at all. After re-compiling with an added print of the covariance matrix, all fits gave the exact same uncertainties.

My issue here is that the bg_offset is at it's limit, and the background has a A*bg_offset term. If it's struggling to figure out the uncertainty in the offset, this will mess up the covariance in all of the backround terms. I know it's not much, but the bg_offset is changing slightly and at the limit, it could have a significant effect on the uncertainty that it has trouble calculating.

Yes, but the subsequent fits with the covariance matrix being output (and the ones where the output was turned off again) all have the bg_offset at its limit and always report the same uncertainties.

Does it have anything to do with the fact that the pf was used on multiple histograms over the course of that session? I have done fits months apart and get the same results.

There's no check in pf to see if you changed the histogram. It currently only initializes once. If you fit a histogram that has vastly different intial conditions (height, etc), this could be an issue if you fit a second, very different histogram with it? I think by calling AddPeak again with a new peak, it resets the initial conditions. It might be worth adding this check.

Try fitting many spectra in a row to see if it reproduces the issue. I could see it being an issue with fitting peaks in angular correlation spectra. You might also want to pass the option "V" to the fit to get an idea of everything it's doing, and if any of the other parameters are outside of their limits, etc.
Either way, to make this more safe, we should declare a private TH1* last_hist_fit. At line 67 of TPeakFitter.cxx we should change to
if(!fInitFlag && !fit_hist && (fit_hist != last_hist_fit){
    //Stuff....
    last_hist_fit = fit_hist;
}

It would probably even make the init flag thing redundant at that point.

We'll leave this open until Andrew or someone PR's the above fix.
The issue was reproduced by fitting many histos in a row that were very different.
The fix seemed to work.


feat: Add provider for Decentralized Identifiers (DID)

This PR adds support for using DIDs as identities in OrbitDB. It uses the dids package under the hood, which relies on EIP2844.
In addition I changed to a smaller dependency for the EthIdentityProvider, and made sure to export the Ethereum and DID providers so they are accessible by developers.

This is really cool, thank you!
Looks like you have some linting to take care of but otherwise, pending some local testing on my part, this looks pretty good and nonintrusive enough to merge in.

This is really cool, thank you!
Looks like you have some linting to take care of but otherwise, pending some local testing on my part, this looks pretty good and nonintrusive enough to merge in.

Also thank you for fixing the ether dep :pray:

Also thank you for fixing the ether dep :pray:

Thanks @aphelionz fixed the linting :)

Thanks @aphelionz fixed the linting :)


[T5A1][W11-B3]Fan Wegiuang

Ready for review
The first two commits should be ignored, because I forgot to checkour master branch before creating the branch for next activity, so I reverted the commit for this activity.

Some comments added. Please close the PR after reading comments.

Comments reviewed. Thank youvery much!


administrator user

Hello
I would like to use this api with a mongodb 3 with auth enabled.
How can I configure the api to use an username and password with admin role ?

Hi don't worry I have discover the right MONGODB_URI format with admin user
tsuru env-set MONGODB_URI="mongodb://admin:password@mongodbserver/admin"


Package request: govulncheck-db

Project description
The database used by govulncheck. https://go.dev/security/vuln/database
Reason
govulncheck by default downloads the db during runtime. This behaviour makes it impossible to run govulncheck as part of a flake check (since flake checks are network sandboxed). It would be great if the db would be packed in nixpkgs so one could use the -db flag of govulncheck to pass the path to a local db (that could be used inside a flake check).
Since this is more or less static data (but should be kept as up-to-date as possible), it would be best if this wasn't manually updated from time to time but automated by some bot or github action or whatever.
@jk @SuperSandro2000 since you are the maintainers of govulncheck
Metadata

homepage URL: https://vuln.go.dev/
source URL: https://vuln.go.dev/index/*.json
license: ?
platforms: all

Since this is more or less static data

No, it isn't. If the db is not updated you will not receive new advisories and constantly bumping the db in nixpkgs creates an unreasonable amount of noise. Also backporting the bumps to stable creates more work.
If you want to run the check in flake check I would recommend to download the db yourself or you run the checks in a none sandboxed environment.


docs: updated login example

This updates the login example in the readme as it was slightly incorrect (I think).
Thanks for the module ❤

Thanks for the catch, please resolve the context and I can merge :)

@itpropro I'm going to close in favour of #19. Please reconsider the removal of TypeScript from the <script setup> examples. Thanks ❤


[Feature Request] Support for GGUF models (llama.cpp compatible)

These run on both GPU and CPU. A lot of OSS community uses them I guess, and the models are quite light on VRAM.

Thank you for submitting.
If you're using txtai 6.2+ you can do the following.
# Embeddings index
writable: false
cloud:
  provider: huggingface-hub
  container: neuml/txtai-wikipedia

# llama.cpp pipeline
llama_cpp.Llama:
    model_path: path to GGUF file

# Extractor pipeline
extractor:
  path: llama_cpp.Llama
  output: reference

txtchat.pipeline.wikisearch.Wikisearch:
  # Add application reference
  application:

workflow:
  wikisearch:
    tasks:
    - action: txtchat.pipeline.wikisearch.Wikisearch

You just need to make sure you also have https://github.com/abetlen/llama-cpp-python installed.

Did you run the exact configuration provided above?

Just added a fix with #13 that should fix the KeyError message you're receiving above.
If you install txtai from source, there is now direct support for llama.cpp models. See this article for more.
https://neuml.hashnode.dev/integrate-llm-frameworks


feat: Improve cef parser with full parsing

:tada: This PR is included in version 2.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


NPM Module

Should be available via NPM
The bundled js should also be part of the module.

https://github.com/rackerlabs/canon-react/issues/18
React should be required in each file that needs it. We should modify the build system to allow react being supplied as a global variable or not.

place the built js in a /dist directory and store in version control

BTW, if you are only publishing to NPM, you can leave built files out of source control and build as part of the pre-publish command.

oh cool, thanks!

I put together a quick prepublish script and was about to start a PR around this but I don't have access to this repo it seems. Do I need to be explicitly added to RackerLabs?

@SethNute Leaving my IRC comment here as well. It would be nice to have access to the raw source files. It looks like the file indicated by the main field of package.json includes React and is minified. I want to control my own version of React, and the minified file makes debugging hard. My build system already supports CJS, so we don't really want prebuilt files.


Adding multiple domains/certificates to the SSL load balancer

Hi!
Great work as usual, just one question.
Describe the solution you'd like
Ability to add more than 1 domain/ssl certificate to the load balancer (HTTPS)
Additional context
Just wondering why is https://github.com/gruntwork-io/terraform-google-static-assets/blob/master/modules/http-load-balancer-website/main.tf#L30 and https://github.com/gruntwork-io/terraform-google-static-assets/blob/master/modules/http-load-balancer-website/main.tf#L35 taking single value and forwarding as a list when underlying module supports lists already. Why not simply taking a list on input and forwarding as-is?

Apologies for the delay in responding to this issue. Please see below:
Sunset notice
We believe there is an opportunity to create a truly outstanding developer experience for deploying to the cloud, however developing this vision requires that we temporarily limit our focus to just one cloud. Gruntwork has hundreds of customers currently using AWS, so we have temporarily suspended our maintenance efforts on this repo. Once we have implemented and validated our vision for the developer experience on the cloud, we look forward to picking this up. In the meantime, you are welcome to use this code in accordance with the open source license, however we will not be responding to GitHub Issues or Pull Requests.
If you wish to be the maintainer for this project, we are open to considering that. Please contact us at<EMAIL_ADDRESS>

As per the message above, we will be closing this issue.


test fixes for TileMapServiceImageryProvider

test fixes for #3150 in reference to #2814
(added epsilons to floating point comparisons, pollToPromise for credit)
@pjcozzi @kring

Thanks @tiffanylu!


Google Colab Pro | No share link in Start Stable-Diffusion Cell

Was working fine last night, but ever since this morning, I'm getting this message when I run the notebook:
Loading weights [5dd1c2014e] from /content/gdrive/MyDrive/sd/stable-diffusion-webui/models/Stable-diffusion/cyberrealistic_v21.safetensors
Creating model from config: /content/gdrive/MyDrive/sd/stable-diffusion-webui/configs/v1-inference.yaml

Could not create share link. Missing file: /usr/local/lib/python3.10/dist-packages/gradio/frpc_linux_amd64_v0.2.

Please check your internet connection. This can happen if your antivirus software blocks the download of this file. You can install manually by following these steps:

1. Download this file: https://cdn-media.huggingface.co/frpc-gradio-0.2/frpc_linux_amd64
2. Rename the downloaded file to: frpc_linux_amd64_v0.2
3.

Tried downloading the file, but my anti-virus considers it a threat, then I realized I wouldn't know where to put it anyways since I searched the sd directory and no folder titled "usr" came up.

I have this issue as well. using google colab. cant find the folder on Drive.

I have this problem too.

Same here
but the flder is not linked to the Drive, but to the virtual server we are launching

but the flder is not linked to the Drive, but to the virtual server we are launching

So, presumably, the virtual server's antivirus is flagging it and that's why the link's not spawning?

This pb occured last year here, and it was a problem from Gradio

That sounds like a problem on Gradio's end and not something A1111 can fix?

I have the same problem, I use colab and I don't know where to place it.

Gradio API is down : https://status.gradio.app/793595965

It may be that gradio is down.

Gradio is back up, as is the shared link.


Turn the whole project into a PR for FSharp.Data

I really would like some feedback here, especially because reading the recommended guidelines I'm not sure if this project qualifies for having the FSharp namespace.
I opted for a separate project to avoid interfering with a similar attempt; but also because I like the idea of immediately shipping something useful, albeit maybe not tested enough for such a mature and popular library which is FSharp.Data.

I think a Type Provider qualifies perfectly for the FSharp.* namespace.
The separation from FSharp.Data is a good choice for the moment until the implementation is more mature. Later a merge into FSharp.Data could be considered.


Area chart bug in stack mode

It appears that the "getStackedData" in series utils assumes that that there will always be the same number of children in every series and that there will not be any missing values.  It is not clear that this will always be the case.
Aside: there are no tests for getStackedData

👋  I'm experiencing the main issue you've pointed out in this ticket, primarily, I have a fixed range e.g. 1 - 12 and multiple data points, however for each datapoint the number varies. As a result, it appears that an error is thrown TypeError: Cannot read property 'y' of undefined. Would be good to know if this is something that will be addressed?

Hey @nazimjamil can you share a sample of the  the code & data you are using?

Hey @mcnuttandrew thanks for the quick response, I actually ended up trying to recreate the issue in isolation and was unable to. So I'm thinking the issue is elsewhere. I'm yet to add a few other things, e.g. crosshair so I may be back here with more info!
Currently testing here codepen


Update uk.coffee

Looks like there are some merge conflicts. Let me try to resolve them.

Looks like that did it!


429 Error - Too many requests - when uploading larger files.

While smaller files below 50MB are loading successfully, I'm getting this 429 server error on larger files after about 400 chunks have uploaded.  The error aborts the upload process.
I'm using vue.js
I understand this might be resolved through a header instruction though I'm not sure how to handle this,  or possibly increasing the chunk size?
Would appreciate any advice here.

@xongooli This seems server-related -- as in you need to look into why your server is pushing 429s to you.


Upgrade to AFJ 0.3.2

This is in a holding pattern waiting for #830 to be merged and the package to be released. Without this the app will become crashey.


[SPARK-12251] Document and improve off-heap memory configurations

This patch adds documentation for Spark configurations that affect off-heap memory and makes some naming and validation improvements for those configs.

Change spark.memory.offHeapSize to spark.memory.offHeap.size. This is fine because this configuration has not shipped in any Spark release yet (it's new in Spark 1.6).
Deprecated spark.unsafe.offHeap in favor of a new spark.memory.offHeap.enabled configuration. The motivation behind this change is to gather all memory-related configurations under the same prefix.
Add a check which prevents users from setting spark.memory.offHeap.enabled=true when spark.memory.offHeap.size == 0. After SPARK-11389 (#9344), which was committed in Spark 1.6, Spark enforces a hard limit on the amount of off-heap memory that it will allocate to tasks. As a result, enabling off-heap execution memory without setting spark.memory.offHeap.size will lead to immediate OOMs. The new configuration validation makes this scenario easier to diagnose, helping to avoid user confusion.
Document these configurations on the configuration page.

Test build #47469 has started for PR 10237 at commit 2dd2d9e.

Test build #47482 has started for PR 10237 at commit 216fc46.

Any preferences RE: offHeap vs offheap?

Test build #47469 has finished for PR 10237 at commit 2dd2d9e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47469/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Test build #47486 has started for PR 10237 at commit 216fc46.

Test build #47482 has finished for PR 10237 at commit 216fc46.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47482/
Test FAILed.

Test build #47486 has finished for PR 10237 at commit 216fc46.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47486/
Test FAILed.

LGTM will merge once it passes tests

Test build #2200 has started for PR 10237 at commit 216fc46.

Test build #2199 has started for PR 10237 at commit 216fc46.

Test build #2201 has started for PR 10237 at commit 216fc46.

Test build #47541 has started for PR 10237 at commit 12ffce3.

Test build #2200 has finished for PR 10237 at commit 216fc46.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):\n  * public class JavaBinarizerExample \n  * public class JavaBucketizerExample \n  * public class JavaDCTExample \n  * public class JavaElementwiseProductExample \n  * public class JavaMinMaxScalerExample \n  * public class JavaNGramExample \n  * public class JavaNormalizerExample \n  * public class JavaOneHotEncoderExample \n  * public class JavaPCAExample \n  * public class JavaPolynomialExpansionExample \n  * public class JavaRFormulaExample \n  * public class JavaStandardScalerExample \n  * public class JavaStopWordsRemoverExample \n  * public class JavaStringIndexerExample \n  * public class JavaTokenizerExample \n  * public class JavaVectorAssemblerExample \n  * public class JavaVectorIndexerExample \n  * public class JavaVectorSlicerExample \n

Haha, because of the Spark PRs update issues / downtime, your NewSparkPullRequestBuilder runs triggered runs of an older commit rather than the latest commit containing the fix for that test failure.

Test build #2199 has finished for PR 10237 at commit 216fc46.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):\n  * public class JavaBinarizerExample \n  * public class JavaBucketizerExample \n  * public class JavaDCTExample \n  * public class JavaElementwiseProductExample \n  * public class JavaMinMaxScalerExample \n  * public class JavaNGramExample \n  * public class JavaNormalizerExample \n  * public class JavaOneHotEncoderExample \n  * public class JavaPCAExample \n  * public class JavaPolynomialExpansionExample \n  * public class JavaRFormulaExample \n  * public class JavaStandardScalerExample \n  * public class JavaStopWordsRemoverExample \n  * public class JavaStringIndexerExample \n  * public class JavaTokenizerExample \n  * public class JavaVectorAssemblerExample \n  * public class JavaVectorIndexerExample \n  * public class JavaVectorSlicerExample \n  *         logInfo(s\"HBase class not found $e\")\n  *         logDebug(\"HBase class not found\", e)\n

Test build #2201 has finished for PR 10237 at commit 216fc46.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Test build #47541 has finished for PR 10237 at commit 12ffce3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47541/
Test PASSed.

Merged build finished. Test PASSed.

Merged into master 1.6


update helm github actions and versions release-1.24

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

/hold
lets wait discussion in https://github.com/kubernetes/cloud-provider-openstack/issues/2175

/hold cancel

/override pull-cloud-provider-openstack-check

/lgtm
/approve


add support for rawgit and gitcdn links

like jsdelivr https://www.jsdelivr.com/rawgit
example

https://gitcdn.xyz/repo/JJ/Test-Text/master/text/en/text.md

@neoascetic you can add support yet for https://www.staticaly.com/

and for https://gitcdn.link/ mirror


Implement the conda environment builder

To build a conda environment, we need to:

download and cache micromamba
use it to build the environment based on a list of conda packages, which should always include the appose project

As a starting point for (1), there is some code here written by @carlosuc3m for bootstrapping a conda setup for use with jep. I would cherry-pick any useful code from there, adapting it to use micromamba instead.
This builder is necessary to fulfill Appose's primary initial use case: invoking DL models from Java, by executing Python in a separate process.

The album project also has conda/mamba/minimamba-based environment building support. See the source code here.

On 2023-11-06, @carlosuc3m wrote on Zulip:

I have been working on a fork
https://github.com/carlosuc3m/appose-java/tree/env-builder
I got the env builder to work with micromamba
Micromamba downloads a .tar.bz2 zip so iI had to use apache commons compress to decompress the executable:
https://github.com/carlosuc3m/appose-java/blob/env-builder/src/main/java/org/apposed/appose/MambaInstallerUtils.java
Also it is important to note that the micromamba link does quite a lotof redirecting, so I tried to deal with it with this method:
https://github.com/carlosuc3m/appose-java/blob/93a23355fb7bdfcef6b1945e6315399683a19ff4/src/main/java/org/apposed/appose/MambaInstallerUtils.java#L160
Also in apple and linux the file needs to be set as an executable and sometimes it might encounter security problems with apple, there seems to be a way to circumvent it using a command:
xattr -d com.apple.quarantine /Users/Cgarcia/git/deep-icy/appose/micromamba/bin/micromamba

In theory it bypasses the gatekeeper of Apple.
With these changes I have been able to install micromamba, install envs and run python from micromamba pretty smoothly.
However, I have not been able to use certain options that conda allows such as setting environment variables for a certain env as conda does.

@carlosuc3m For macOS (and maybe all platforms), I wonder if shipping micromamba with Fiji would alleviate the pain, assuming we finally start shipping a DMG installer for Fiji for macOS (fiji/fiji-builds#1). I believe that copying a .app out of a DMG to a new location causes macOS to clear the quarantine flag on all involved files under the hood. We will need to test this, of course. It may work in our favor with micromamba here.


paneMoveable is not disabling zooming

Hi everyone!
It seems like I either found a bug in the code or in the docs. On https://reactflow.dev/docs/api/component-props/ it says for the prop: paneMoveable: default: true - If set to false, panning and zooming is disabled.
However when I am setting it to false zooming still works.
Here is a small reproduction:
https://codesandbox.io/s/optimistic-archimedes-25l7i?file=/src/App.js

Thanks for the report! The docs are wrong ;) I will update it. If you want to disable zoom you can set zoomOnScroll={false}.


SCB-491 Fix the broken links for China Region

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
 A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #349 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #349   +/-   ##
=======================================
  Coverage   69.95%   69.95%
=======================================
  Files          17       17
  Lines        3418     3418
=======================================
  Hits         2391     2391
  Misses        863      863
  Partials      164      164

Impacted Files
Coverage Δ

server/service/rule.go
76.82% <0%> (-0.43%)
:arrow_down:

server/service/tag.go
76.02% <0%> (+0.58%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage remained the same at 73.504% when pulling 4716e5567a877fad32bbf95f56eec881bed8b97b on asifdxtreme:master into 772adf7f611d5deb9eebb3a7265f3a6ef350bd9b on apache:master.


Propose some functions and macros for unitful integration.

In particular this PR provides a q_str macro which accesses u_str to define quantities via the corresponding data in Unitful.
I don't know how to define the macros in the extension, so at this stage it would make Unitful a full dependency. IMHO would be ok, but of course this is open for discussion.
See my proposal in #10 - this PR could be an implementation.

Thanks for the PR! I am in principle on board with this, just need to consider things a bit more. I am wondering whether it makes sense to make Unitful.jl a hard dependency or not. According to the benchmark above, it bumps the load time to 0.6s. After #7, the load time would go down to 0.07s, so this would be a big increase if we do choose that route.
@oscardssmith thoughts?
I definitely like the idea of exporting a q_str though. I think you could export it from an extension by defining a macro that calls some function _q_str. Then overload that function inside the extension.
If we break the target audience into two groups, I see the following use-cases:

Internal use by package maintainers.

They might want to eliminate reliance on Unitful.jl completely, and just manipulate dimensions. Here, perhaps we just want to reduce load time of the package, and simplify things as much as possible.
I think this will be the main user base?

End users (people doing scientific calculations/analysis).

They probably want to use units.
They may want to try DynamicQuantities.jl for faster calculations.
Having them install both DynamicQuantities.jl and Unitful.jl (as well as importing both) would make things slightly less user-friendly.

In talking through this, I wonder if it makes the most sense to just have people import both packages, and convert between them. In other words, maybe it makes sense to keep Unitful.jl as an extension, without the q_str macro. Then they can still rely on the Unitful.jl documentation and other unit libraries, rather than treating DynamicQuantities.jl as a front-end API and expecting the relevant docs to be here. What do you think?

@odow I'd be interested to hear your thoughts as well

Interesting considerations!
My opinion here is that Unitful.jl is too rigid to be used by end users (who ase a rule are inexperienced programers and don't see programming as their foremost purpose). km and cm being different types feels IMHO is not very intuitive. I am working  with others on a PDE simulation stack for applicationsm and I really would like to provide the possibility to do calculations with units to end users and modelers. With Unitful one needs to strip units before going into the solver (and not forget upreferred - I once tried to get some convenience calls which should have made it easier for end users into Unitful).
For me this ended up in LessUnitful.jl which is just a band aid.
For DyanamicQuantities I indeed see the potential that this package can work well for less experienced end users. I see the potential that quanties can be passed through e.g. a PDE solver if it is well designed. This opens up great possibilities for simulation work, teaching etc.
I see the need for being able to expose end users to quantities with units  in an easy to handle consistent way, Julia and its ecosystem certainly have the potential, and DynamicQuanties could make it possible.  So indeed I would put them first...

Ok the wind blew the other direction, so I think we can close this.


how can we setup this using  Postgres instead of cleardb as  Postgres is free to use and doesn't  ask for credit card details .

How to configure it?


Pass kerning setting through into harfbuzz shaping (SDL2)

Fixes #339
Harfbuzz docs on enabling/disabling features like this: https://harfbuzz.github.io/shaping-opentype-features.html

I haven't tested but yes this sounds ok !


fix: set dev source for psi

Set thex-source header when calling the EaaS PSI API according to environment (prod or dev).
As per https://cq-dev.slack.com/archives/C065U16LPSA/p1706191131440079

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

v1.8.2
GitHub release

Your semantic-release bot :package::rocket:


Added one new resource

Added the Scaler Blogs URL - Front End Developer URL to the section - Reference, providing useful insights to aspiring tech professionals in the front-end development domain.
Please let me know if further changes are required.
Thank you

This is great. Thank you for your contribution!


Hope the "onnxruntime" issue is resolved soon！

I have been tortured by onnxruntime for a long time recently. The author said it is because CUDA must be 11.8 before it can be used. The current comfyui official version is 12.1, and my system is 12.3. Downgrading to 11.8 is a huge problem. Later, I discovered that there are many plug-ins that use onnxruntime. All the .onnx models, such as was, such as comfyui-reactor-node; I only need to install plug-ins such as comfyui-reactor-node and was in advance, and then install cn, and the problem will be solved. Perfectly solved. If you install controlnet_aux/ first, controlnet_aux/ will mess up everything. This is not my problem alone, many friends are encountering similar problems now!
Since plug-ins such as comfyui-reactor-node can solve this problem, I believe controlnet_aux/ can also do it.

There is currently no need to downgrade to cuda11.8, other plug-ins can use onnxruntime normally.

@yinghanzi Are you using Windows?

I have the same problem and use Windows

Getting this on a new comfyui portable install, on windows.

Maybe you can disable you GPU acceleration for onnx as reactor-node suggestted https://github.com/Gourieff/comfyui-reactor-node/discussions/73

@Layer-norm Opencv already has built-in onnx loader which use CPU by default

@Fannovel16 Yes, gpu is much more faster, especially for video. For a 720p video around 300 frames, onnxruntime-gpu only takes about 1 minute to finsh the task, while cpu takes around 1hours. And if you talk about onnxruntime in cpu, I can't tell the difference.

reactor-node suggestted Gourieff/comfyui-reactor-node#73 -> I can't get it to work since the Comfy update. Additionally I “destroy” the DW-Pose processor.
At the moment I get the following message:
C:\Spezial\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\node_wrappers\dwpose.py:24: UserWarning: DWPose: Onnxruntime not found or doesn't come with acceleration providers, switch to OpenCV with CPU device. DWPose might run very slowly
warnings.warn("DWPose: Onnxruntime not found or doesn't come with acceleration providers, switch to OpenCV with CPU device. DWPose might run very slowly")
... , but it works. to create images and is not a problem.

I am sorry that onnxruntime-gpu may not support cuda12.3 now. If you wan't to use onnxruntime-gpu in cuda12.x and above in windows, you can try to build it yourself instead of using prebuilt wheel, This link may help you https://onnxruntime.ai/docs/build/eps. And you can find scripts source from here. However, it only support cuda version below 12.2 now.

To use CUDA 12.* instead of 11.8, you can try install nightly binary like the following:
pip install ort-nightly-gpu --index-url=https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ort-cuda-12-nightly/pypi/simple/

@f-rank @tianleiwu @murphylanga
onnxrutime-gpu support cuda12 now.
YOU SHOULD REMOVE THE OLD VERSION COMPLETELY
pip uninstall onnxruntime onnxruntime-gpu and remove onnxruntime folder in (your python folder)/Lib/site-packages if it exists after uninstall.
Then pip install -U onnx onnxruntime onnxruntime-gpu
after that you can use onnx model with your gpu in cuda12 now.
请先完全卸载onnxruntime onnxruntime-gpu
并在卸载后确认python embeded/Lib/site-packages文件夹里是否有onnxruntime文件夹残留
如果有请将整个onnxruntime文件夹扔进回收站（命令可能卸载不干净）
然后重新安装onnx onnxruntine onnxruntime-gpu
python -m pip install -U onnx onnxruntime onnxruntime-gpu

👍 Thank you so much for your assistance in solving the problem.
Unfortunately, the Ultimate Upscale Node crashes now when it comes after the Control Net Openpose in a workflow.

@f-rank @tianleiwu @murphylanga onnxrutime-gpu support cuda12 now. YOU SHOULD REMOVE THE OLD VERSION COMPLETELY pip uninstall onnxruntime onnxruntime-gpu and remove onnxruntime folder in (your python folder)/Lib/site-packages if it exists after uninstall. Then pip install -U onnx onnxruntime onnxruntime-gpu after that you can use onnx model with your gpu in cuda12 now. 请先完全卸载onnxruntime onnxruntime-gpu 并在卸载后确认python embeded/Lib/site-packages文件夹里是否有onnxruntime文件夹残留 如果有请将整个onnxruntime文件夹扔进回收站（命令可能卸载不干净） 然后重新安装onnx onnxruntine onnxruntime-gpu python -m pip install -U onnx onnxruntime onnxruntime-gpu

非常感谢，困扰我3个小时的问题，各种找解决方案，这个最有效。
我的controlnet插件没出问题，是rembg移除背景的插件报错。

python -m pip install -U onnx onnxruntime onnxruntime-gpu is the right way. It installs ORT for CUDA 11, not for CUDA 12. Also, there is no need to install onnxruntime package (it is CPU only), which might overwrite the gpu package.
Please make sure you only install one ORT package (like onnxruntime, onnxruntime-gpu or nightly variants). It is because all those packages will be installed to same onnxruntime subdirectory under site-packages (run pip show onnxruntime-gpu to see the site-package location).
Sometime, the issue is caused by some dependency (like VC Runtime, CUDA and cuDNN) not installed. Please follow https://onnxruntime.ai/docs/install/ to install onnxruntime-gpu for CUDA 12 for detail. The command to install official package is like the following:
pip install onnxruntime-gpu --extra-index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/onnxruntime-cuda-12/pypi/simple/

for late comer:
here's the way to enable gpu accelerate on cuda12.x
track the issue here for version changes: https://github.com/microsoft/onnxruntime/issues/13932
runtime nightly: https://dev.azure.com/onnxruntime/onnxruntime/_artifacts/feed/onnxruntime-cuda-12
ort nightly:https://aiinfra.visualstudio.com/PublicPackages/_artifacts/feed/ort-cuda-12-nightly
#with cu12.*
pip install coloredlogs flatbuffers numpy packaging protobuf sympy
pip install ort-nightly-gpu --index-url=https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ort-cuda-12-nightly/pypi/simple/
pip install onnxruntime-gpu==1.17.0 --index-url=https://pkgs.dev.azure.com/onnxruntime/onnxruntime/_packaging/onnxruntime-cuda-12/pypi/simple/

need help i got this error :
DWPose: Using yolox_l.onnx for bbox detection and dw-ll_ucoco_384_bs5.torchscript.pt for pose estimation
DWPose: Caching ONNXRuntime session yolox_l.onnx...
2024-11-29 08:40:04.1171188 [E:onnxruntime:Default, provider_bridge_ort.cc:1745 onnxruntime::TryGetProviderInfo_CUDA] D:\a_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1426 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_providers_cuda.dll"
*************** EP Error ***************
EP Error D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
when using ['CUDAExecutionProvider', 'CPUExecutionProvider']
Falling back to ['CUDAExecutionProvider', 'CPUExecutionProvider'] and retrying.

2024-11-29 08:40:04.2046481 [E:onnxruntime:Default, provider_bridge_ort.cc:1745 onnxruntime::TryGetProviderInfo_CUDA] D:\a_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1426 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_providers_cuda.dll"
!!! Exception during processing !!! 'NoneType' object has no attribute 'get_providers'
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 419, in init
self._create_inference_session(providers, provider_options, disabled_optimizers)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 483, in _create_inference_session
sess.initialize_session(providers, provider_options, disabled_optimizers)
RuntimeError: D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose\wholebody.py", line 39, in init
self.det = ort.InferenceSession(det_model_path, providers=ort_providers)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 432, in init
raise fallback_error from e
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 427, in init
self._create_inference_session(self._fallback_providers, None)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 483, in _create_inference_session
sess.initialize_session(providers, provider_options, disabled_optimizers)
RuntimeError: D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 323, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 198, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 169, in map_node_over_list
process_inputs(input_dict, i)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 158, in process_inputs
results.append(getattr(obj, func)(**inputs))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\node_wrappers\dwpose.py", line 72, in estimate_pose
model = DwposeDetector.from_pretrained(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose_init.py", line 238, in from_pretrained
t = Wholebody(det_model_path, None, torchscript_device=torchscript_device)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose\wholebody.py", line 41, in init
print(f"Failed to load onnxruntime with {self.det.get_providers()}.\nPlease change EP_list in the config.yaml and restart ComfyUI")
^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'get_providers'

need help, i got this error
DWPose: Using yolox_l.onnx for bbox detection and dw-ll_ucoco_384_bs5.torchscript.pt for pose estimation
DWPose: Caching ONNXRuntime session yolox_l.onnx...
2024-11-29 08:40:04.1171188 [E:onnxruntime:Default, provider_bridge_ort.cc:1745 onnxruntime::TryGetProviderInfo_CUDA] D:\a_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1426 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_providers_cuda.dll"
*************** EP Error ***************
EP Error D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
when using ['CUDAExecutionProvider', 'CPUExecutionProvider']
Falling back to ['CUDAExecutionProvider', 'CPUExecutionProvider'] and retrying.

2024-11-29 08:40:04.2046481 [E:onnxruntime:Default, provider_bridge_ort.cc:1745 onnxruntime::TryGetProviderInfo_CUDA] D:\a_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1426 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_providers_cuda.dll"
!!! Exception during processing !!! 'NoneType' object has no attribute 'get_providers'
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 419, in init
self._create_inference_session(providers, provider_options, disabled_optimizers)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 483, in _create_inference_session
sess.initialize_session(providers, provider_options, disabled_optimizers)
RuntimeError: D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose\wholebody.py", line 39, in init
self.det = ort.InferenceSession(det_model_path, providers=ort_providers)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 432, in init
raise fallback_error from e
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 427, in init
self._create_inference_session(self._fallback_providers, None)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\python_embeded\Lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 483, in _create_inference_session
sess.initialize_session(providers, provider_options, disabled_optimizers)
RuntimeError: D:\a_work\1\s\onnxruntime\python\onnxruntime_pybind_state.cc:891 onnxruntime::python::CreateExecutionProviderInstance CUDA_PATH is set but CUDA wasnt able to be loaded. Please install the correct version of CUDA andcuDNN as mentioned in the GPU requirements page  (https://onnxruntime.ai/docs/execution-providers/CUDA-ExecutionProvider.html#requirements),  make sure they're in the PATH, and that your GPU is supported.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 323, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 198, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 169, in map_node_over_list
process_inputs(input_dict, i)
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\execution.py", line 158, in process_inputs
results.append(getattr(obj, func)(**inputs))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\node_wrappers\dwpose.py", line 72, in estimate_pose
model = DwposeDetector.from_pretrained(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose_init.py", line 238, in from_pretrained
t = Wholebody(det_model_path, None, torchscript_device=torchscript_device)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\ComfyUI MHD 3.0\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\dwpose\wholebody.py", line 41, in init
print(f"Failed to load onnxruntime with {self.det.get_providers()}.\nPlease change EP_list in the config.yaml and restart ComfyUI")
^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'get_providers'


"Invoked program with no predicate named main" error occurs in unrelated syntax error cases

The "Invoked program with no predicate named main" error gets thrown in cases where main is defined but some other syntax error is present. There are a lot of cases where this happens, but from what I can tell all of the cases boil down to any unexpected text appearing outside of a pair of curly braces.

Thanks for all of your work on this! Sorry about the scope creep, I'll try not to do that again 🙇♂


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit de43514:
:clock10: Pending: waiting for processors (11 builds ahead of you)
:warning: Docs Build is busy, currently there are 11 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit de43514:
:clock10: Full build: the average full build time is 5 min(s) 16 sec(s), this is based on the last 30 build(s) for this repository.

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

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

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


Syntax error in getting started documentation

I do believe in this section the last } should be a ]
Creating Data to Vuex Store
// Assuming this data structure is the response from the API backend.
const posts = [
  {
    id: 1,
    title: 'Hello, world!',
    body: 'Some awesome body...',
    author: {
      id: 1,
      name: 'John Doe',
      email<EMAIL_ADDRESS>    }
  }
}

store.dispatch('entities/posts/create', { data: posts })

should be
// Assuming this data structure is the response from the API backend.
const posts = [
  {
    id: 1,
    title: 'Hello, world!',
    body: 'Some awesome body...',
    author: {
      id: 1,
      name: 'John Doe',
      email<EMAIL_ADDRESS>    }
  }
]

store.dispatch('entities/posts/create', { data: posts })

Oh yes! Thank you for pointing this out. I'll fix it!

Done fixing! Thank you so much for the report 👍

@kiaking I'll try to find time to finish my PR about linting JS in docs. it spotted that one!

@paparent Yeah that would be awesome! Thanks!


filter out all intermediate calls from presence call

Coverage remained the same at 43.554% when pulling a6d83e829f0431b7d384bf86bd956dafde8aa202 on u9520107:bug/intermediate-calls into 49a8a553a7433b2a2a9cdaa7d6f9fde71f977bd1 on ringcentral:master.


Performance difference

I reconduct experiments with 5 random seed based on your open-source code, But fail to hit the appearance as your paper reported.
Concrectly, in the paper:

While in my expremiment result:

I wonder if I miss something or it is due to the version difference of a specific dependency package.
Thank you for your reply and time!

Sorry for the late reply and thanks for these reproduced results.  As the results show, performances over multiple runs often fluctuate for segmentation tasks especially when you changed the seed. I think the averaged results are bearable.


[playground] No parsing of TypeScript type information?

I'm experimenting with the TSDoc Playground and I can't seem to make it parse TypeScript type information. Here's a sample:
/**
 * Returns the average of two numbers.
 *
 * @remarks
 * This method is part of the {@link core-library#Statistics | Statistics subsystem}.
 *
 * @param x - The first input number
 * @param y - The second input number
 * @returns The arithmetic mean of `x` and `y`
 *
 * @beta
 */
function getAverage(x: number, y: number) {
    return (x + y) / 2.0;
}

As you can see, the ParamBlocks in the AST don't include any type annotations for x and y.
  - ParamBlock
    - BlockTag
      * BlockTag="@param"
    * Spacing=" "
    * ParamBlock_ParameterName="x"
    * Spacing=" "
    * ParamBlock_Hyphen="-"
    * Spacing=" "
    - Section
      - Paragraph
        - PlainText
          * PlainText="The first input number"
        - SoftBreak
          * SoftBreak="\n"
  - ParamBlock
    - BlockTag
      * BlockTag="@param"
    * Spacing=" "
    * ParamBlock_ParameterName="y"
    * Spacing=" "
    * ParamBlock_Hyphen="-"
    * Spacing=" "
    - Section
      - Paragraph
        - PlainText
          * PlainText="The second input number"
        - SoftBreak
          * SoftBreak="\n"

Thoughts?

The TSDoc playground doesn't include a TypeScript compiler at all.  (One of the "features" of the @microsoft/tsdoc library is that it's completely self-contained and doesn't bring in any other libraries.  So you can easily use it in other compilers besides TypeScript.)
The playground admittedly doesn't make this very obvious however, since:

We include TypeScript source code in our samples, and
The Monaco texteditor applies syntax highlighting to the TypeScript code

We should make this more clear, but I wasn't sure what's the best way to do that.  Some ideas:

remove the TypeScript code from the samples
disable syntax highlighting for the TypeScript language
add a text disclaimer explaining that the TypeScript code is only being shown for illustrative purposes

@iclanton  FYI

TSDoc is only concerned with parsing comments. TypeScript provides the type information. Any documentation generator will need to interact with the TypeScript compiler API to iterate through documentable nodes and then parse their comments to TSDoc. Take a look at the advanced demo source to see one way this could look.

Here's the other example that @Wenzil gave in #116:
/**
 * Some sample class
 */
export class MyClass {
    /**
     * Returns the average of two numbers.
     */
    getAverage(x, y) {
        return (x + y) / 2.0;
    }
}

We need to improve the playground UI to make it clear that this is not an expected input.

Alternatively, we could actually support fill TypeScript source files.  E.g. the "HTML" tab would dynamically change to render whatever comment that your cursor was inside.  That's probably a much more involved change, though. ;-)

Thanks for the replies. I understand the scope of the library better now. I like the suggestions to state the expected input more prominently, both in the source README.md and the playground

If the playground code sample looked like this:
/**
 * Returns the average of two numbers.
 *
 * @remarks
 * This method is part of the {@link core-library#Statistics | Statistics subsystem}.
 *
 * @param x - The first input number
 * @param y - The second input number
 * @returns The arithmetic mean of `x` and `y`
 *
 * @beta
 */

instead of this:
/**
 * Returns the average of two numbers.
 *
 * @remarks
 * This method is part of the {@link core-library#Statistics | Statistics subsystem}.
 *
 * @param x - The first input number
 * @param y - The second input number
 * @returns The arithmetic mean of `x` and `y`
 *
 * @beta
 */
function getAverage(x, y) {
    return (x + y) / 2.0;
}

Would that have been more clear?  If we do that, we could also report an error if there is any text after the comment.

I understand that perhaps parsing the actual TypeScript code is outside of the scope of what TSDoc is trying to accomplish, but it was definitely something I hoped would be included since I first noticed this project gaining some traction.
In the README it says:

The majority of the standard JSDoc tags are preoccupied with providing type annotations for plain JavaScript, which is an irrelevant concern for a strongly-typed language such as TypeScript.

However, the fact that my documentation will have zero indication of the type of a parameter means that this could never serve as a replacement for JSDoc for me personally. If I write a library in TypeScript and it is consumed by a JavaScript application, I would like for there to be typings in the documentation, and not require that people are relying on IDE tooling or digging into the package to find types.
The first thing that drew me to this was the idea of it parsing types from the code to save the redundant declarations in the documentation.
Is there a reason why this is excluded from this project? It seems like a gap to not include typing in documentation personally.

Hmmm... I had not thought of this requirement, where you want to work in TypeScript but accomodate developers who code in plain JavaScript.
But if I were doing that, I definitely wouldn't want to ask our devs to write all their type information twice (once in the code, and again in the TypeScript doc comments). Wouldn't it make more sense for the compiler to emit type annotations into the .js files, based on its own type analysis? That would save a lot of human effort.
In this case technically the .d.ts files could get TSDoc emitted into them and the .js files could get JSDoc. But obviously we would want to align the two as much as possible...
We should open a separate GitHub issue since this question is unrelated to the playground.

@fastfedo encountered the same confusion in  https://github.com/Microsoft/tsdoc/issues/152#issuecomment-474026798 .  He gave a couple suggestions:

Two things might have helped me here:

Documentation examples in/linked from the README that show usage with different TypeScript constructs (interface, type, class, enum, etc).

A note on the TSDoc playground that it's not actually parsing TypeScript, or that only the first
comment is being parsed, as you stated above. So the playground is not a place to explore how
more complex types would be documented.


Write a tool that can reliably remove a set of macro calls from a collection of docs

Write a tool that can reliably remove a set of macro calls from a collection of docs, and write the result back into the Wiki DB.
"remove" means that the macro call no longer appears in the source doc.
Acceptance criteria
We can easily selectively remove KumaScript macros from subsets of the Wiki.

Command line tool that allows:

define a list of docs (order of 100s)
define list of macros
outputs rendered files
user can diff rendered files if need be
allow a push or rendered files to Kuma MySQL


Behat 3.0.4 parameter definition under Contexts

Hi,
Since updating from a beta version of Behat v3 to v3.0.4, I've been having issues with specifying parameters under my Contexts within the behat.yml file.
The context.feature example shows the following format:
    Given a file named "behat.yml" with:
      """
      default:
        suites:
          default:
            contexts:
              - FeatureContext:
                  parameter1: val_one
      """

My behat.yml is as follows:
default:
  autoload:
    '': %paths.base%/library
  suites:
    default:
      contexts:
        - FeatureContext:
            url:
              google:    'https://www.google.co.uk'
              bbc:         'https://www.bbc.co.uk'
            key2:         'val2'

This fails as my FeatureContext is passed NULL for the parameters. If I wrap all my parameters within FeatureContext under a 'parameters' key, then the above is loaded in successfully.
Please could you let me know the correct syntax to use for passing parameters through via the config file?

But how do I nest parameters when I have FeatureContext class path defined? I am using Behat 3.0.15
default:
    suites:
        default:
            contexts: [Blah\Tests\FeatureContext]

I can't find this documented anywhere.

Well, see the comment above giving the answer, where FeatureContext is the fully qualified class name of the context class. It works the same when the class is namespaced (except quoting the class name with single quotes may be needed to have a valid Yaml syntax because of the backslash, but I'm not even sure it is needed)

Ah, I see, I somehow thought FeatureContext was reserved key name, not a class name. Thanks.

This was the best place I found on the whole internet that explained how to pass params to context contructor. If its documented on docs.behat.org, I could not find it. Thanks.

default:
suites:
default:
contexts:
- FeatureContext:
- base_url:
'http://localhost:3000'

Try with '-' it should help
default:   suites:     default:       contexts:         - FeatureContext:             - base_url:                 'http://localhost:3000'


Chat Tab (Frontend)

[x] nav bar to get in and out of chat room
[x] accept text input from user and send over wifi??
[x] accept emoji input from user and send over wifi??
[x] fetch and output messages displaying message + sender
[x]  circles with housemates initials formatted for displaying sender
[x] move text input field to middle of screen when keyboard is pulled up
[x] styling with provided hexcode color scheme

Peer Review

[x] nav bar to get in and out of chat room
[x] accept text input from user and send over wifi??
[x] accept emoji input from user and send over wifi??
[x] fetch and output messages displaying message + sender
[x]  circles with housemates initials formatted for displaying sender
[x] move text input field to middle of screen when keyboard is pulled up
[x] styling with provided hexcode color scheme

@LeoSpallino
Looks good.  There are a few matters to look over before we close this issue:

[ ] The text input field separates from the keyboard, it's position should have zero spacing and relative to the keyboard.
We should now look into user account detection to know who is sending the messages, this will be a partly backend issue.


Translate Chinese text in English README

Pull request checklist

[x] The PR has a proper title. Use Semantic Commit Messages. (No more branch-name title please)

[x] Make sure this is ready to be merged into the relevant branch. Please don't create a PR and let it hang for a few days.

 These submissions will be prioritized for review:
Introduce improvements in program execution speed;
Introduce improvements in synthesis quality;
Fix existing bugs reported by user feedback (or you met);
Introduce more convenient user operations.

PR type

Bug fix

Description

There was some Chinese text in the English Readme file, I have translated it
It affects a sentence in the README.en.md file

Screenshot

Before

After

@blaisewf Thanks for the approval!


move controller's main into cmd

Note:
Move gateway-api main.go into go idiomatic cmd directory.
Tests:

make run
build image with docker
docker build .
docker run <new_image_id>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3472

Relevant Lines:
10454

💛  - Coveralls


Add css containment for embeds

See: https://developer.mozilla.org/en-US/docs/Web/CSS/contain

No good.


Allow application developer to specify TLS client certificate

What would you like to be added:
Application developer should be able to define a client certificate that the proxy uses when it connects to developer's backend service with TLS.
Why is this needed:
Background:
This feature has been previously discussed and it was included as BackendPolicy.spec.tls.clientCertificateRef. Later, Gateway level configuration was preferred #395 and feature was removed.  As per discussion in the comments of #400, I would like to propose this capability to be re-introduced as an optional feature,  as an alternative to global (per gateway) configuration.
Use case details:
Assume we have application developer working within Kubernetes namespace. Application developer wants to secure the traffic between proxy and their backend service.
1. Application developer generates their own certificates and configures server certificate to their backend service
2. To get the proxy to validate the server certificate, the application developer configures reference to the CA certificate secret to BackendPolicy.spec.tls.certificateAuthorityRef
Next the developer would like to protect the backend service even further, by allowing incoming TLS connections only from the proxy. Symmetrically, application developer could enable client authentication by themselves, like they did for the server authentication in step 2 above but this would require client certificate to be configurable per backend service, not per Gateway which would be outside of application developers domain.
Implementing this enhancement has following advantages:

Allows "self-service" workflow for the application developer, since they do not need to depend on cluster admin to configure client certificate on gateway level
Cluster admin does not need to manage / coordinate / distribute single CA certificate for the development teams working in the cluster. Each team can configure their own client certificates for the proxy and CA certificates to their backend for validating the client cert

This seems reasonable to me, with the proviso that the BackendPolicy client certificate details are documented as specifying the details for a TLS interaction between the Gateway and the serving Pods.

/remove-lifecycle stale

This API has changed substantially with v1alpha2, we will need to revisit how we can achieve this.

It seems like it would be possible to achieve this with some form of policy attachment initially, not sure if there's a better approach or built in policy resource we can offer here.

/remove-lifecycle stale

/remove-lifecycle stale
/lifecycle frozen

While grooming we saw that this one was open for a long period of time without anyone with a strong use case to champion it. We're going to close this as we don't expect anyone's ready to drive it forward, but if you still want this feature and have a strong use case we will be happy to reconsider this and re-open.

The use case is presented in the description and in my mind it is strong: application developer should be able to configure TLS certificates to protect backend communication without bothering the cluster administrator.
I have not participated actively in the gateway-api project though, so it is of course fair to close this.

Noting for posterity and shared context that this appears to be a duplicate of https://github.com/kubernetes-sigs/gateway-api/pull/1430.

Hi @mikemorris

Noting for posterity and shared context that this appears to be a duplicate of #763

While being closely related,

this current issue #622 was about allowing application developer to manage the client certificate for proxy when it initiates TLS connection towards their upstream/backend service.
The discussion in #763 is about allowing application developer to manage server certificate for proxy when it terminates TLS connection from an external client. That is the same use case as Ingress.spec.tls.secretName targets, where application developer can set certificate for a virtual host(s) that they manage.

After very quick look at #1430 it seems to me like "GEP-1282: Describing Backend Properties" would rather allow addressing this (closed) issue than #763?  Cc @candita.

Thanks for the explanation @tsaarni, that makes sense and I think #1282 could possibly address both use cases.

@youngnick reviewing your previous comment and reviewing #1282, it appears that #1282 now includes the scope of this issue so it seems we should be able to close this as a duplicate. If I'm mistaken or I've missed something, please re-open!

Hmm, we actually haven't solved this one, because #1282 is basically declined. However, this and #1067 are basically duplicates.
@candita, there's some more background for #1067 here.


Refactor logging

Logging currently has a threading issue which causes file contents to be lost when files are rotated.

https://docs.python.org/3/howto/logging-cookbook.html#sending-and-receiving-logging-events-across-a-network


Geolocation taking too much time to get position

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ x ] 3.x
I'm submitting a ...  (check one with "x")
[ x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When I try to get my current position, the device just takes a really long time (2-3 minutes) to get it, while my browser does it perfectly. Also, native geolocation works good with other apps at my phone, so it's not a device issue, but software's.
Expected behavior:
Should get the current position really quick, as it does with other apps.
Steps to reproduce:
It stops for so long time, at this particular line:
this.geolocation.getCurrentPosition().then((position) => {
-->
Related code:
insert any relevant code here

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.4.0
    Cordova CLI      : 7.0.0
    Ionic CLI        : 3.4.0

local packages:

    @ionic/app-scripts              : 1.3.1
    @ionic/cli-plugin-cordova       : 1.4.0
    @ionic/cli-plugin-ionic-angular : 1.3.1
    Cordova Platforms               : android 6.2.3
    Ionic Framework                 : ionic-angular 3.0.1

System:

    Node       : v7.9.0
    OS         : Windows 10
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 4.2.0

This issue is not in ionic. it might be ionic-angular, or the plugin itself.

Some more info: Xiaomi Mi5s is being so special-ish. This code just doesn't work in it, but it does in Samsung S5. Navigator GPS geolocation doesn't work well in Xiaomi either, but it does work perfect in Google Maps.
So... must be this a problem with MIUI? In that case, this problem is critical, since it affects all Ionic developers with this platform.

Again, this is not an issue with ionic and as such here is not the place for it.
You can open it in https://github.com/driftyco/ionic-native, but there is a low chance it is an issue with that either. Probably the plugin itself - https://github.com/apache/cordova-plugin-geolocation

@AmitMY Already posted it in the ionic-native place, let's see if somebody finds out what's going on.

Hello all. I am going to close this issue as a duplicate of https://github.com/ionic-team/ionic-native/issues/1703

Please check this issue: https://github.com/ionic-team/ionic-native/issues/2571


WipperSnapper firmware fails at network connection when loaded from MacOS but succeeds on Windows

Describe the bug
WipperSnapper firmware fails at network connection when loaded from MacOS but succeeds on Windows.
Arduino board
ItsyBitsy ESP32
To Reproduce
Steps to reproduce the behavior:

Using the espwebtool, I erased the flash
Then I uploaded the Factory Reset bin
After the reset bin was running successfully, I went thru the WipperSnapper GUI in Chrome to load the firmware.
On the Connect! step, the yellow light flashes, blue flashes multiple times and then is solid blue

Expected behavior
When I follow these steps on Windows 10, the firmware loads properly and the device is added to my account. I'm able to add components as well.
Which components are connected to your device
N/A
Screenshots
N/A
Desktop (please complete the following information):

OS: MacOS Ventura 13.3 (M1)
Browser: Chrome
Version: Chrome 123.0/WipperSnapper v1.0.0-beta.78

Additional context
I tried taking the board with the firmware loaded via MacOS to my Windows 10 computer but it still hangs at the Connect! step (solid blue light)
Happy to test with other boards too if that's helpful.

@tyeth  Do you have a laptop similar to Liz' that you can replicate this with? If you don't, I'll self assign and try to replicate.

@brentru I've an Intel Mac (not M1) running a recent beta of Sonoma (14.2 with an update to 14.5 available). Can attempt to initially recreate, but if you have an M1/2/3 or still on v13 then maybe that would be better

@tyeth  I have a M1 on Ventura, self-assigned this to myself.

the only ones working are beta 72 and beta 66

I've been testing on windows 11

@videopixil @BlitzCityDIY I followed your replication steps. However, I can't replicate this with a Macbook Pro running Ventura 13.6.3 and a QT PY ESP32 Pico.
Follow-up Questions:

Is this only with the ESP32 Pico, or other boards have the same issue?

When you connect your ESP32 to the mac - could you open Arduino IDE's serial monitor? Then, press RESET to let the board attempt to connect. You should see serial output - could you copy that and paste it here?

and here is the output from the same device after successfully loading the firmware with windows 10:
ets Jul 29 2019 12:21:46

rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 271414342, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0030,len:1256
load:0x40078000,len:13832
load:0x40080400,len:4
load:0x40080404,len:3048
entry 0x40080590
Adafruit.io WipperSnapper
-------Device Information-------
Firmware Version: 1.0.0-beta.78
Board ID: itsybitsy-esp32
Adafruit.io User: BlitzCityDIY
WiFi Network: wubbalubbadubdub
MAC Address: 0C:8B:95:96:BC:70
-------------------------------
ESP32 CPU0 RESET REASON: NO_MEAN
ESP32 CPU1 RESET REASON: POWERON_RESET
E (298) task_wdt: delete_entry(232): task not found
Generating device's MQTT topics...
Running Network FSM...
Establishing network connection...
Performing a WiFi scan for SSID...SSID found!
Connecting to WiFi (attempt #0)
Connected to WiFi!
Connecting to AIO MQTT (attempt #0)
WiFi Status: 20
Registering hardware with WipperSnapper...
Registering hardware with IO...
Encoding registration request...Encoding registration msg...Published!
Polling for registration message response...2
GOT Registration Response Message:
Hardware Response Msg:
	GPIO Pins: 22
	Analog Pins: 6
	Reference voltage: 3.30v
Completed registration process, configuration next!
Polling for message containing hardware configuration...
Polling for message containing hardware configuration...
Polling for message containing hardware configuration...
cbSignalTopic: New Msg on Signal Topic
2 bytes.
decodeSignalMsg
cbSignalMsg
Sub-messages found: 1
Signal Msg Tag: Pin Configuration
Initial Pin Configuration Complete!
Publishing to pin config complete...
Hardware configured successfully!
Registration and configuration complete!
Running application...
PING!
PING!

could it be at the IO account level? i could try having Noe login on my windows machine and vice versa

hi hi @brentru  i tried
feather 32 s2 and s3
qtpy32 s2
pi pico w
itsybitsy 32

@videopixil  It seems to be an issue with connecting to Adafruit IO, not the WiFi network. Can you verify that the adafruit io key for the pixil3d account matches what appears on io.adafruit.com? Make sure the same key is used on the devices.

@brentru  yes verified io keys match - the format is:
{
"io_username": "pixil3d",
"io_key": "same-IO-key",
"network_type_wifi": {
"network_ssid": "fake-network1",
"network_password": "fakepass"
},
"status_pixel_brightness": 0.2
}

it does seem to be something with the account. i signed into the Mac with my Adafruit account and was able to load the firmware with the same ItsyBitsy.

@brentru I regenerated the io key and it worked! - new devices are able to connect 🎉

Weird, but glad it was account-centric. Closing for now, feel free to reopen if you have this issue occur again.


MS Store version crashes with latest version from master

Cloned and built latest version from the master branch
When trying to loads mods into the MS store version of Deep Rock Galactic, I get to the initial startup screen after which the game force closes with error 0xe06d7363
Tested/confirmed with an identical modlist on an older version of MINT which had no issues

Does it still crash if you only install a simple mod?
Please also share your mods list that triggers this crash.

Crashes even with just Mod Hub installed - I had dm'd assembly about it, he said it's sort of a known issue (something with the menus being slightly different on the MS store version) but that he hadn't gotten around to testing it. Advised me to make a post on here anyway


collect_training_data Error

My Step:

Run on PC: collect_training_data.py
Run on RPi: stream_client.py
Then i get this error.
Error Screen Shot:
http://prntscr.com/o0oc2e
I not have RC Controller. Can i use Keyboard on my PC for Control Car ?
How i can fix this error ?

@annak1n change your serial port to the one you have connected the arduino


Add dataset fld

Hi LogiTorch Developers,
Thanks for such a great framework. We are currently using LogiTorch, which significantly helps us accelerate research experiments.
We have been working on our dataset FLD, which was proposed in our ICML 2023 paper "Learning Deductive Reasoning from Synthetic Corpus based on Formal Logic" (https://proceedings.mlr.press/v202/morishita23a.html).
Could it be possible to add FLD to LogiTorch?
We have implemented all the elements, i.e., the dataset and the collator, the model, the pipeline, and the training and evaluation scripts.
We have tested that the training succeeds by training_examples.py, and the model performs well by eval_examples.py.
Best,
Terufumi Morishita

Thanks :)
Can you add comments to your code?


refactor GarbageCollectionLogger so one instance per context

Move GarbageCollectionLogger to ServerContext and start scheduled task in AbstractServer so that the is only one instance running for a server.
Bumping the the spotbugs version in #2824 highlighted a synchronization issue.  This change minimizes possibly having multiple instance of GarbageCollectionLogger running as well as making sure one runs per server instance.

While I appreciate the attempt to improve the use of this class, I really think the logic of this class should exist in the TabletServerResourceManager. I am not sure how exactly it would be incorporated with the other memory checks but we have a memoryGuardThread already running in the tserver. This thread is more geared towards Tablet memory usage but I feel like the Java GC checking thread here should be running with the other resource management of the tserver. The GarbageCollectionLogger class feels like it was just created quickly on the fly to try for debugging a complex production issue and just thrown into the tserver.

I really think the logic of this class should exist in the TabletServerResourceManager

The JVM GC Logger is used by all server processes, not just the TabletServer.

I believe that I have addressed the issues as much as can be done without a total refactor.  I think it represents an incremental improvement of the current code and larger changes should be handled as a separate issue.  Or, just abandon this can stay with what we currently have?

I believe that I have addressed the issues as much as can be done without a total refactor. I think it represents an incremental improvement of the current code and larger changes should be handled as a separate issue. Or, just abandon this can stay with what we currently have?

I am not sure this change is worth the benefit, especially if there is a good chance this class is going to be removed in the future. I would be more in favor of a change that makes it easier to remove the class. Now that I think about it, these changes might make it more difficult to drop the class.

How does this make future changes harder?

It moves the instantiation of instances to one place.
Instead of multiple instances running in scheduled executors there is one.
The instance is created in the AbstractServer code so that any service gets an instance without needed to add code that is not obvious that we use.

How does this make future changes harder?

I didn't say that. I said it makes it more difficult to drop the class. The class is being renamed, moved and its usage refactored. It is being introduced into new classes where it wasn't referenced previously. This PR increases the complexity of the class and its usage. I am advocating for doing nothing and just removing the class. For example... GarbageCollectionLogger is now only referenced 11 times, while the new class is 29.

The rename was a suggestion by @milleruntime
...I think the name of this class is confusing. Some reference to Java or the JVM in the name would greatly help to differentiate it from the Accumulo GC classes. Something like JVMGcLogger or JavaMemoryLogger or JavaGcLogger would be a lot better.

The rename was a suggestion by @milleruntime
...I think the name of this class is confusing. Some reference to Java or the JVM in the name would greatly help to differentiate it from the Accumulo GC classes. Something like JVMGcLogger or JavaMemoryLogger or JavaGcLogger would be a lot better.

Yes, the name is confusing.

I'm also in favor of removing this class, and share @milleruntime 's concern about introducing it in new classes where it wasn't used before. However, I do find the new name less confusing, if we end up keeping it. I also like making it a singleton, if we're keeping it around. We don't need more than one of these per process.
I think we should push this off to 3.0, at least, and decide then what we want to do with it.

This PR will be superseded by the changes with https://github.com/apache/accumulo/pull/3161

This PR will be superseded by the changes with #3161

In that case, I'm closing this, so we can focus on reviewing that one.


PHP 8 shows a warning on routes with dynamic variables

On routes that have dynamic, non-regex variables (e.g. /foo/:id) there's a warning on PHP 8.1 about passing null to substr:

Deprecated: substr(): Passing null to parameter #1 ($string) of type string is deprecated in vendor/shardj/zf1-future/library/Zend/Controller/Router/Route.php on line 288

The line is this:
https://github.com/Shardj/zf1-future/blob/9a10c633b8ef0c82f482f5eeefd7854bd46924d1/library/Zend/Controller/Router/Route.php#L288
It should be:
if ($part !== null && substr($part, 0, 2) === '@@') {

$part can be null at that point as mentioned on line 96 ("stores [its] regex requirement or null") and there's already another null check later on line 298, so it's not a case of an invalid route or other configuration issue.

Or
$part = $this->_parts[$pos] ?? '';

instead of
https://github.com/Shardj/zf1-future/blob/9a10c633b8ef0c82f482f5eeefd7854bd46924d1/library/Zend/Controller/Router/Route.php#L273

$part = $this->_parts[$pos] ?? ''; does not work, because there is a check for $part !== null afterwards.
Used the solution of juhana which is already used in the same way a few lines below.


Customize the main toolbar

Main problem
Although I went through the docs, issues and roadmap, i'm still not sure if this functionality already exists or not or if it is WIP, but i'd like to be able to customize the main toolbar.
In my case I want to have a vertically aligned toolbar on the left side of the screen, change the icons, help messages, yet keep the essential working the same. Somewhat similar to the idea mentioned in #6140 where i'll be able to group the settings separately so that the settings toolbar (that pops up on the left when a user clicks on pen, shape etc) isn't permanently visible and takes up too much space.
Of course i'm not saying that we should replace the current UX in any way as that works for a lot of people but i'd like to have a children component API for the main and settings toolbar.
If this already exists in any way or is being worked on please let me know.
I have to implement this regardless at my job so it's either directly in the library as a part of the children component bit (open to contribute) or separately in my own project from scratch to just solve my own use-case.
I'm open to other ideas too.
Toolbar in question

Hi @hakotaco Did you manage to customize the toolbar? Please advise

Could you provide an example or snippet, if possible?

@grnd-alt @hakotaco Possible to share how did you achieve this ?

Also - I need a feature to make the top toolbar/side menu toolbar text bigger. Control + should make the toolbar text size bigger as well.
I can barely see what the toolbar is saying as it's so small on my external monitor. Competitor Draw.io has this feature.


Missing readme config for publishing

Some checks failed for missing long descriptoin

Codecov Report

Merging #21 (70fc375) into main (c9d387d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   92.41%   92.41%
=======================================
  Files           5        5
  Lines         211      211
=======================================
  Hits          195      195
  Misses         16       16

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 c9d387d...70fc375. Read the comment docs.


Fastest way to access pixel data.

I'm looking forward to write an utility (in Node) that extracts things like prominent colors from images.
Considering the most recent version of sharp, what would be the fastest way to access and read the raw pixel data from an image?
I'm not interested in writing or transforming the image later.

Hello, did you see raw?

Ok, that would work, thanks.

Hello, did you see raw?

This link points to a hijacked site?

@exxy Thanks, I've updated the original comment.
(Please see #2132 for an existing enhancement to clean/update/remove issues from 2017 and earlier, such as this.)


Sync eng/common directory with azure-sdk-tools repository

Sync eng/common directory with azure-sdk-tools repository

Can one of the admins verify this patch?

/azp run python - appconfiguration - ci


Why the indefinite updating?

After following all of the steps and restarting the webui-user.bat file, the automatic update goes on for ages, consuming internet with no end.
What is downloading? Where is it downloading to? And what for?
There really needs to be some more transparency as Windows already makes such things difficult - and the odd "server-108-158-61-54.bom78.r.cloudfront.net" source of the download stream further adds suspicion to it.
Edit: I don't intend to make much of a drama about it, but would still like to make a point that it would be really appreciated to have more (transparency) than just silence after "[auto-sd-paint-ext] Pull upstream." As I also don't have a very fast internet connection, so an hour of waiting (approx +5GB downloaded) becomes rather concerning, for an extension.
2nd Edit: After two hours and about 7GB downloaded, I've decided to move install.py out of the folder. Stuff works alright and I will leave it that way until this makes more sense to me.

wait what

Okay a quick google reviews cloudfront seems to be Amazon's CDN. So I guess TIL Github uses AWS for content delivery.
Also, you can just read install.py, I literally just tell git to pull the repository updates. It sounds to me your git install might somehow be broken.

Hey sorry for earlier. It turns out to have been the Dreambooth extension that caused the weird download.
And I'm not sure if it was just sheer coincidence, but to someone using the command line with both extensions installed, it would appear as if auto-sd-paint-ext were the one starting the download as the CLI would freeze/stop for the download, right after the lines from install.py were printed out.
The dreambooth extension doesn't print anything to the CLI while downloading, which had me make the wrong presumptions.
Once again, sorry! Everything actually works great now.


Failed to load previous state.

Hi,
"load(store)" fail to load state from memory.
Does anyone have a solution, please ? Thanks
Here is my code:
import { applyMiddleware, compose, createStore } from 'redux';
import { fromJS } from 'immutable';
import createSagaMiddleware from 'redux-saga';
import createReducer from './state';
import * as storage from 'redux-storage'
import devTools from 'remote-redux-devtools';

import thunk from 'redux-thunk';
import createEngine from 'redux-storage-engine-reactnativeasyncstorage';

const engine = createEngine('xyz');

const storageMiddleware = storage.createMiddleware(engine);

const sagaMiddleware = createSagaMiddleware();

function configureStore(initialState = fromJS({})) {

  const middlewares = [
    sagaMiddleware,
    thunk,
    storageMiddleware
  ];

  const enhancers = [
    applyMiddleware(...middlewares),
  ];

  if (__DEV__) {
    enhancers.push(devTools());
  }

  const store = createStore(
    createReducer(),
    initialState,
    compose(...enhancers)
  );

  const load = storage.createLoader(engine);

  // Notice that our load function will return a promise that can also be used
  // to respond to the restore event.
  load(store)
      .then((newState) => console.log('Loaded state:', newState))
      .catch(() => console.log('Failed to load previous state'));

  // Extensions
  store.runSaga = sagaMiddleware.run;

  return store;
}

module.exports = configureStore;

Sorry , the cause of issue was in "createReducer" function.

Re-opened as @guns2410 want's to work on this project: Give him a warm welcome 👏

Thanks @michaelcontento
Happy to see @petervavro's issue is resolved.