Tortoise fails to connect to database

@mjhea0 Tried to fix this by refering to multiple resources. Can you please look into it?

Cloned the repo,
Build the images and spin up the containers, using:
$ docker-compose up -d --build
Applying migrations it failed to connect with database with below traceback:
$ docker-compose exec backend aerich upgrade

Traceback (most recent call last):
  File "/usr/local/bin/aerich", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 321, in main
    cli()
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1656, in invoke
    super().invoke(ctx)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 45, in wrapper
    loop.run_until_complete(f(*args, **kwargs))
  File "/usr/local/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 95, in cli
    await Migrate.init(tortoise_config, app, location)
  File "/usr/local/lib/python3.9/site-packages/aerich/migrate.py", line 66, in init
    await Tortoise.init(config=config)
  File "/usr/local/lib/python3.9/site-packages/tortoise/__init__.py", line 600, in init
    await cls._init_connections(connections_config, _create_db)
  File "/usr/local/lib/python3.9/site-packages/tortoise/__init__.py", line 392, in _init_connections
    await connection.create_connection(with_db=True)
  File "/usr/local/lib/python3.9/site-packages/tortoise/backends/asyncpg/client.py", line 106, in create_connection
    raise DBConnectionError(f"Can't establish connection to database {self.database}")
tortoise.exceptions.DBConnectionError: Can't establish connection to database hello_fastapi_dev

Detailed traceback:

  Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/tortoise/backends/asyncpg/client.py", line 103, in create_connection
    self._pool = await asyncpg.create_pool(None, password=self.password, **self._template)
  File "/usr/local/lib/python3.9/site-packages/asyncpg/pool.py", line 407, in _async__init__
    await self._initialize()
  File "/usr/local/lib/python3.9/site-packages/asyncpg/pool.py", line 435, in _initialize
    await first_ch.connect()
  File "/usr/local/lib/python3.9/site-packages/asyncpg/pool.py", line 127, in connect
    self._con = await self._pool._get_new_connection()
  File "/usr/local/lib/python3.9/site-packages/asyncpg/pool.py", line 477, in _get_new_connection
    con = await connection.connect(
  File "/usr/local/lib/python3.9/site-packages/asyncpg/connection.py", line 1981, in connect
    return await connect_utils._connect(
  File "/usr/local/lib/python3.9/site-packages/asyncpg/connect_utils.py", line 732, in _connect
    con = await _connect_addr(
  File "/usr/local/lib/python3.9/site-packages/asyncpg/connect_utils.py", line 632, in _connect_addr
    return await __connect_addr(params, timeout, True, *args)
  File "/usr/local/lib/python3.9/site-packages/asyncpg/connect_utils.py", line 682, in __connect_addr
    await compat.wait_for(connected, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/asyncpg/compat.py", line 103, in wait_for
    return await asyncio.wait_for(fut, timeout)
  File "/usr/local/lib/python3.9/asyncio/tasks.py", line 479, in wait_for
    return fut.result()
asyncpg.exceptions.InvalidCatalogNameError: database "hello_fastapi_dev" does not exist
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/bin/aerich", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 321, in main
cli()
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1128, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1053, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1656, in invoke
super().invoke(ctx)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 754, in invoke
return __callback(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/click/decorators.py", line 26, in new_func
return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 45, in wrapper
loop.run_until_complete(f(*args, **kwargs))
File "/usr/local/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
return future.result()
File "/usr/local/lib/python3.9/site-packages/aerich/cli.py", line 95, in cli
await Migrate.init(tortoise_config, app, location)
File "/usr/local/lib/python3.9/site-packages/aerich/migrate.py", line 66, in init
await Tortoise.init(config=config)
File "/usr/local/lib/python3.9/site-packages/tortoise/init.py", line 600, in init
await cls._init_connections(connections_config, _create_db)
File "/usr/local/lib/python3.9/site-packages/tortoise/init.py", line 392, in _init_connections
await connection.create_connection(with_db=True)
File "/usr/local/lib/python3.9/site-packages/tortoise/backends/asyncpg/client.py", line 106, in create_connection
raise DBConnectionError(f"Can't establish connection to database {self.database}")
tortoise.exceptions.DBConnectionError: Can't establish connection to database hello_fastapi_dev

Nevermind! Turns out docker image I had pulled/present in cache was broken.
Deleted old postgres images from the local environment and pulled fresh which worked fine!


Refactor/fix broken links

:scroll: Description
This PR brings the following change(s):

:bug: Bug Fix - a non-breaking change that fixes an issue.
:book: Documentation Change - changes or updates the documentation of a particular project, function, etc.

Fixes # (issue number)
:heavy_check_mark: Checks

[ ] Tests are added and passed for all relevant functions (if any).
[x] My code follows the style guidelines as mentioned.
[x] In case of new functionality, proper comments have been added to the code.
[x] Changes do not generate any warnings for existing functions.
[x] Add a new entry in documentation, if fixing a bug or added a new feature.

#17 dry run for sphinx build is not triggered on pr


Options subfolder

This fix the following use case where both .options and .proto are located in a folder hierarchy that need to be preserved for the generated files. Some of the proto files could be imported using their path related to the proto folder
proto
├── A
│       ├── A.proto
│       ├── A.options
├── B
│      ├── B.proto
│      ├── B.options
│      ├── A.proto
│      ├── A.options
app
├── CMakeLists.txt
├── build
│        ├── A
│        │       ├── A.pb.h
│        │       ├── A.pb.c
│        ├── B
│        │      ├── B.pb.h
│        │      ├── B.pb.c
│        │      ├── A.pb.h
│        │      ├── A.pb,c
The issues where:

It was quite hard to make sure that protoc takes the proto folder has project root and not one of the subdirectory
When the correct folder was taken as project root the plugin could not find options files (A/A.options B/B.options) because proto/A/ and proto/B/ where added to the option file but not proto/.

Is it possible that this could break builds for some of the existing users of the CMake rules?

The change about the discovery of the option files won't break anything.
The modification of the order of import path might be more problematic but I think it should be OK.
The following import path have been pushed back:

GENERATOR_PATH: This one does not contains any .proto, so not sure why it's needed by probably shouldn't be the project root
GENERATOR_CORE_DIR: This one contains nanopb.proto so it should not be considered as the project root
CMAKE_CURRENT_BINARY_DIR: This does not contains any .proto, o not sure why it's needed by probably shouldn't be the project root
So no issue there.

When RELPATH is not set no logic is changed. And when set I don't see a correct use case where a user would like to have the generated import dir before it.

I marked it ready for review, if you remember some recent contributor of the FindNanopb.cmake file it might be good to have their input on that

Thanks!
I checked a few of the recent CMake contributors earlier, but they aren't particularly active on GitHub currently.
This seems straightforward enough that I just merged it directly.
The protoc include path order can be a complex things so I'll add a migration note just-in-case, even though I don't expect this to cause problems for anyone.


Install pytorch in conda (Windows)

As the time goes by, current sample command in readme will not install a proper pytorch with cuda support.
According to pytorch's official site, the latest windows version does not support of CUDA 10.2 anymore.
Insted I tried install 11.3 as recommend (And also required for cards like 3060)
conda install pytorch torchvision cudatoolkit=11.3 -c pytorch
Then it works. I don't have git client in my recent environment so is unable to push, but I think this information can be shared in readme as well?

Sorry for the long silence. I've updated the README file.


BackHandler listener not removed

When calling navigation.popToTop() in a StackNavigator, willUnmount is called before willBlur, and the BackHandler listener is not removed.

hi @londonoliver, thanks for reporting. For the record, the problem is that willUnmount is called first and removes the willBlur subscription and the BackHandler.removeEventListener in willBlur is therefore not called.


ARC-1519 - Improve Jira Routing for connect/app creation

What's in this PR?

The combination of the following PRs:

https://github.com/atlassian/github-for-jira/pull/1415
https://github.com/atlassian/github-for-jira/pull/1417
https://github.com/atlassian/github-for-jira/pull/1418
https://github.com/atlassian/github-for-jira/pull/1419
https://github.com/atlassian/github-for-jira/pull/1420
https://github.com/atlassian/github-for-jira/pull/1421
https://github.com/atlassian/github-for-jira/pull/1422
https://github.com/atlassian/github-for-jira/pull/1423
https://github.com/atlassian/github-for-jira/pull/1424
https://github.com/atlassian/github-for-jira/pull/1425

Added/Updated routes fixes for

Connect new server in jira-select-server.hbs
Connect new application in jira-select-github-cloud.hbs
Query string corrected in jira-server-url.hbs
Route added for Manual app in jira-select-app-creation.hbs

Why

To test all the new routes combined in one branch.

How has this been tested?

In local environment
In staging environment
Test cases

I am sure all sub PRs are already approved. Let me know if you need me to approve this one, but since I wasn't highly involved in the whole journey, I reckon please wait for approval from @mboudreau or @rachellerathbone.

Yup, its all approved already. I've merged it all and created this PR so that it can all be tested and merged in one go.


Pup Fails to Run on an M1 Mac in Big Sur

I cannot get pup to run on my M1 Mac mini. I downloaded the latest build from the Downloads page, but I'm always getting Segmentation fault: 11 when run in Bash, or fish: 'pup' terminated by signal SIGSEGV (Address boundary error) when run in my default shell, Fish. I've allowed it in Security preferences, but there seems to be a crash going on. I tried installing with brew, but that gets stuck with the install of Go on the ARM architecture.

Have you installed Rosetta 2? https://osxdaily.com/2020/12/04/how-install-rosetta-2-apple-silicon-mac/
I don't have an M1, but one thing I keep hearing is that you have to quit and restart Terminal after installing Rosetta, for it to work.
pup M1 bottles on Homebrew will have to wait for Go 1.16, since that's the first officialy supported release on M1.

Yes, I have Rosetta 2. But I've found a replacement tool for my use for pup, so all is well.

Users on Intel report crashes too. See #85, #123.


Hookup TRS to descriptor type versions

Is your feature request related to a problem? Please describe.
After this information is available, hook it up to TRS
Additional context
Blocked by https://ucsc-cgl.atlassian.net/browse/SEAB-4857
#4320
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.14
┆Issue Number: DOCK-2282
┆Sprint: 101 - Cheeks, Sandy
┆Issue Type: Story

On QA, the text on the swagger page still says 2.0.0, although the link is correctly 2.0.1:

Workflow language versions work as expected in QA.
On the format zip, it now looks like the Accept header is the only thing that matters, and the query parameter is irrelevant. For example, curl -X 'GET' 'https://qa.dockstore.org/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2Fcoverbeck%2Fdockstore-whalesay-wdl-only/versions/master/WDL/files?format=zip' leads to a 400 response, whereas, no query parameter, curl -X 'GET' 'https://qa.dockstore.org/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2Fcoverbeck%2Fdockstore-whalesay-wdl-only/versions/master/WDL/files?' -H "Accept: application/zip returns a zip.
Whether the response is a ZIP is purely determined by Accept: application/zip. I think this is the correct behavior, although it's a little confusing given that the query parameter is baked into the spec.
I think it'll be OK, but I want to make sure you're aware of the behavior.

Hmmm, I think that is a bug.
It should use the query parameter when there is no accept header or the accept header is a wildcard.


Bubble Chart from 3d.js

Hello!
Have you any plans to bringing Bubble Chart from d3.js to this project?
Thank you.

Hi @Groude thanks for the issue. Can you specify what chart you mean and maybe add a picture? That would help a lot because vx can already do a lot of stuff.
From googling "d3 bubble chart"  I found this link https://beta.observablehq.com/@mbostock/d3-bubble-chart. Is that what you mean?
If so you can already do that with vx. It's part of @vx/hierarchy and called pack. Unfortunately it's not in the docs yet but you have an example in the gallery here https://vx-demo.now.sh/pack.

Going to close this. Feel free to follow up here or open a new issue if you run into a blocker or question with your implementation.


make the statistic of configuration distribution in sidecar more clea…

…r for istioctl
There are some misunderstanding for us to see the output for checking the statistic of configuration distribution in sidecar when I execute the following command:
Command:
istioctl experimental wait --for=distribution --threshold=.99 --timeout=20s virtualservice bookinfo.default
Output:
Resource VirtualService/default/bookinfo present on 24 out of 24 sidecars
In fact, there are only 8 sidecars and each sidecar with 3 types of distribution configuration, so I changed these code to make the description more clear.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Hi @zhlsunshine. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@zhlsunshine: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
e5244bf6839503c441bcab6b0ceeaa471a7e7e11
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
e5244bf6839503c441bcab6b0ceeaa471a7e7e11
link
/test lint_istio

release-notes_istio
e5244bf6839503c441bcab6b0ceeaa471a7e7e11
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
e5244bf6839503c441bcab6b0ceeaa471a7e7e11
link
/test lint_istio

release-notes_istio
28e8dd780f3b486e339dbf1ca8849f2b5103e327
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes_istio


[WIP] Julia docs

Description
Since the dmlc.ml domain is gone we lost our Julia docs (and other things).
This PR adds it to our regular website deployment pipeline. 💥
It pretty much works, but well... let's see what happens in CI. 🤷♂
Preview
It ends up in /site in the Julia API area.
http://<IP_ADDRESS>/versions/julia/api/julia/site/index.html

Hmm maven failed in CI - that should be unrelated to this PR.
@zachgk @lanking520  - any ideas? I'll restart it anyways....

If we have this PR merged, do we still need the Julia doc build pipeline in Jenkins?
(this one: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwebsite/detail/PR-15454/2/pipeline/86)

@mxnet-label-bot  add [Julia, Doc]

If we have this PR merged, do we still need the Julia doc build pipeline in Jenkins?
(this one: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fwebsite/detail/PR-15454/2/pipeline/86)

I think having a separate test for the docs package like Julia is a good idea and wish we had that for each kind, like C++ docs, Scala, etc. With this PR, it is duplicating the test, but this was the easiest way and keeps all of the docs generated in the same pipeline as the website output.
Let's leave it in there for now. Make sure everything is working how we want it, (docs are being built!) then discuss further how things should work, and try to streamline.

It seemed to be something to do with the the libmxnet.so problem. I just restart the test to see if it is reproducible

It seemed to be something to do with the the libmxnet.so problem. I just restart the test to see if it is reproducible

It did it twice now already. This new test is the third attempt.

@iblis17 There's a line in the ubuntu setup guide that says "The MXNet package for Julia is hosted in a separate repository, MXNet.jl, which is available on GitHub." But this isn't true anymore is it?

@aaronmarkham It isn't true anymore.

@aaronmarkham It isn't true anymore.

Thanks, I'll change that and rebase so maybe this CI/maven problem will be fixed.
How far should the Julia docs go back? I could trigger the build in 1.4.x or earlier in settings.ini, but I'm pretty sure that those versions must be compatible with Julia 0.7 and 1.0 since invoking Julia with MXNet and running docs seemed different for earlier versions. Also, I don't really think it is needed to engineer a solution for older versions. If so, I'd put that in a different PR.
For this PR though, I'd trigger whatever is compatible plus maybe note what's going on there if people browse the older versions.

@iblis17 I tried to add some more details on getting Ubuntu configured natively (since all I did here was get it working via Docker and to run the docs), but I ran into this:
julia> using Pkg; Pkg.build("MXNet")
ERROR: The following package names could not be resolved:
 * MXNet (not found in project or manifest)
Please specify by known `name=uuid`.
Stacktrace:
 [1] pkgerror(::String) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/Types.jl:120
 [2] #ensure_resolved#43(::Bool, ::Function, ::Pkg.Types.EnvCache, ::Array{Pkg.Types.PackageSpec,1}) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/Types.jl:900
 [3] ensure_resolved at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/Types.jl:870 [inlined]
 [4] #build#53(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:455
 [5] build at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:436 [inlined]
 [6] build(::String) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:431
 [7] top-level scope at none:0

I saw a post that you might need to add this to the registry like this, but it didn't work:
   Cloning git-repo `https://github.com/apache/incubator-mxnet/tree/master/julia`
ERROR: failed to clone from https://github.com/apache/incubator-mxnet/tree/master/julia, error: GitError(Code:ERROR, Class:Net, unexpected HTTP status code: 404)

Could you paste the preview link?

It's in the main description:
http://<IP_ADDRESS>/versions/julia/api/julia/site/index.html
But since the website CI is done you can see that one too:
http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-15454/8/index.html

BTW, Julia v1.0.4 is out. Let's test it: https://github.com/apache/incubator-mxnet/pull/15502.

LGTM! Verified Julia installation and doc building process on an Ubuntu EC2 instance.

Nice work! @aaronmarkham


Fail to provision and edge device under Azure .cn (China) subscription

Describe the bug
I am trying to provision a device into IoTHub via DPS. I am using a SymetricKey provision type and a scope Id from a azure.cn subscription.
The provisioning command returns success but NO IoTEdge device entity is created by DPS.
Provisioning command used: Provision-EflowVm -provisioningType DpsSymmetricKey -symmKey "" -scopeId  -registrationId
Accessing to the "/etc/iotedge/config.ymal" file I can see the property provisioning > global_endpoint is set to "https://global.azure-devices-provisioning.net".
Since we are using a .cn scope Id I think the global_endpoint should be set to "https://global.azure-devices-provisioning.cn".
Changing the global_endpoint in the config file manually and restarting the EFLOW VM. The Edge device entity is correctly created under the IoTHub portal (cn subscription).
Why is the service assuming the wrong global_endpoint if I am passing a .cn scope Id as a parameter?
Please let me know I can provide additional troubleshooting data.
Expected behavior
I expect that the IoTEntity war was created on IoTHub .cn portal without manually changing the global_endpoint property on the VM file.
Windows Host OS (please complete the following information):

Edition: [Pro]
EFLOW:  v1.1.2111.17111
Version: [19044.1348]
Physical Device located in Mainland China

Hi @ngmartins,
I'm transferring this issue to the IoT Edge team, as they will probably have better knowledge to help you with this issue.
Thanks,
Francisco


Mouse XY sensitivity setting

Description of the new feature / enhancement
Mouse XY sensitivity setting
Scenario when this would be used?
When moving the mouse starting from the wrist, it is difficult to move the mouse in the Y-axis direction.
Supporting information
No response

Can you explain how just the normal mouse speed wouldn't handle this?  /needinfo  Would love to better understand the exact ask.


Combining multiple utterances from continuousSpeech with white space

Is your feature request related to a problem? Please describe.
I would like to combine the multiple RecognitionResults from a continuous speech recognition into a single transcription with language correct whitespace.
Describe the solution you'd like
I would like to combine RecognitionResult texts together into a single transcription with correct white space.

Right now I am adding a single white space character between utterance texts in my own code.
I'm concerned that that might not always be appropriate to do, like for Japanese, where there is less use of white space.

Questions:

Is there any utils available for combining multiple utterances with correct white space?
Is an utterance always guaranteed to end with some punctuation, after which a white space is appropriate in all languages?

Thanks for the question. I'm checking with our team that owns punctuation and will get back once I have a definitive answer.

Sorry this took a while to get an answer back.
We don't have a utility to combined multiple phrases, and each phrase, while likely, isn't guaranteed to end in punctuation.
We've had a thought of exposing another method on the API surface to get the entire recognition in one call, but haven't had the right opportunity to take it further.

Thanks @rhurey.  For my current use case getting the entire recognition would be great (assuming its still low latency).


Question - data from ocean gif tutorial

Hello - please could you elaborate on medusarun the data used in this tutorial? Where did you source it? https://nbviewer.org/github/OceanParcels/parcels/blob/master/parcels/examples/documentation_homepage_animation.ipynb
I'm trying to model the trajectories of particles released from the mouth of the Missisipi into the gulf of Mexico. I understand I need an ocean current netcdf of some form.
Thanks,
Grace

Dear @graceebc9, the data from the medusa run comes from http://opendap4gws.jasmin.ac.uk/thredds/nemo/root/catalog.html, but that link is often down.
You could also have a look at the data at https://www.mercator-ocean.eu/en/solutions-expertise/accessing-digital-data/product-details/?offer=4217979b-2662-329a-907c-602fdc69c3a3&system=d35404e4-40d3-59d6-3608-581c9495d86a (with documentation here), which is the 'workhorse model' we now use in our group. The advantage is that it's on an original C-grid, so no interpolation


DEV.QUEUE.4 and above are missing

According to https://www.ibm.com/docs/en/order-management?topic=toolkit-configuring-running-agents-in-containerized-developer there should be DEV.QUEUE.1 through 10 available out of the box...
But when I run the image (even the latest), only DEV.QUEUE.1 - 3 are present...
Why??

This MQ sample has only ever had three DEV queues available.  Either that documentation is wrong, or it's referring to changes made in the IBM Sterling Order Management developer toolkit, which is not managed in this repository.  I'm afraid I don't know how you could raise issues on that toolkit.

Ah ok... I was under the impression that those were referring to the same... :-(
I am using this image along with TestContainers to test some MQ functionality - and it is a great pain that I cannot (easily) create my own test queues - let alone only have 3 independent ones to play around with.
Would it be possible to allow creation of my own queues using environment variables during start of the container?

I'm not sure it would make sense to create them using an environment variable, as there are lots of options to set when you create a queue.  The pre-configured queues are just for getting started.  You can create queues using a file, which you can drop into /etc/mqm, with a .mqsc extension.  For example:
DEFINE QLOCAL('DEV.QUEUE.4') REPLACE
DEFINE QLOCAL('DEV.QUEUE.5') REPLACE
DEFINE QLOCAL('DEV.QUEUE.6') REPLACE
DEFINE QLOCAL('DEV.QUEUE.7') REPLACE
DEFINE QLOCAL('DEV.QUEUE.8') REPLACE
DEFINE QLOCAL('DEV.QUEUE.9') REPLACE
DEFINE QLOCAL('DEV.QUEUE.10') REPLACE


Users with only read rights can see editing tools

Steps to Reproduce:

Assign a user to the sitename-web-base group, which has read-only permissions.
As the read-only user:
Possibility to edit dictionaries. It looks like it saved the edit, but it does not happen.
Possibility to create a language copy at sitename content and experience fragment. It looks like it create a language copy, but it does not happen.
Possibility to add to the configuration. When you try to save it, you will receive a notification 'no write access'.

@michielspiritus the description is a bit vague, can make make it more clear? :)


[WIP] Resolve imports and exports

fixes: #2373

@nicojs It's mostly done, but to progress I need your help with resolving these problems since I have no idea what's an issue :c

BTW, don't look into the green windows tests. This is something we need to fix still, see https://github.com/nicojs/node-link-parent-bin/issues/20

Since there are a lot of changes nowadays, I think I'll postpone this PR for later on (after a release most likely)


fix(viewtoggle): update aria attribute

resolves #1680

https://github.com/pluralsight/design-system/blob/master/packages/viewtoggle/src/react/__specs__/index.spec.tsx#L28
^ please update the jest test to run the axe tests not that it should be passing


[MRG] Update README

Fixes #27
Updating the command-line examples in the README

Codecov Report

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

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   84.77%   84.77%
=======================================
  Files          12       12
  Lines         381      381
=======================================
  Hits          323      323
  Misses         58       58

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 fe98fa1...5a7290e. Read the comment docs.


Feature: Add more clarity to examples in basic shell commands

Is your feature request related to a problem? Please describe.

While reading through the examples, we take our user through the command line, starting with the ls command. Terminal commands can get confusing if you don't specify the exact starting point.
For example, whether the user is currently in the root directory of the project, or if they (mistakenly) are pointing to somewhere else on their computer such as a directory up one level - this would give them a different output.
Describe the solution you'd like

It would be helpful to add more clarity to examples in basic shell commands, to specify they are starting in the root directory of the project. This will avoid confusion for someone trying to understand where they are in the terminal when following along.
Additional context

Update the output to list for the ls command to show additional files that are now in the repo.

Fixed via https://github.com/bobbyiliev/introduction-to-git-and-github-ebook/pull/75


Request to upgrade jsdom from v11 to latest v15

🚀 Feature Proposal
To upgrade jsdom dep from v11 to latest v15.
Motivation
In order to support latest browser APIs, such as shadow dom.
Example
Pitch
This has been brought up few times before. #7763, #7926.
The decision was to hold off and continue to support nodejs v6, which jsdom v12+ does not.
Since nodejs v6 is EOL half a year ago, please update jsdom to latest. Thanks very much.
This will be a breaking change.

It's already upgraded, just not released yet. You can try it under jest@next iirc

Thx!


Redirecting back to app after billing / how to get app name

I'm using appSubscriptionCreate to create a subscription which needs a returnUrl and according to shopify support this returnUrl needs to be back in my embedded app. Would be nice if this could be automatic (the user just returns back to the page they were at before billing).
I don't know how to get a return URL in the backend of the embedded app. I saw a function called getEmbeddedAppUrl somewhere, but this only seems to work well during the auth flow (where it's used) where the host and shop name are passed as query parameters.
I also thought about manually constructing the URL www.store.myshopify.com/admin/${APP_NAME}, but then I need an app name

@mkevinosullivan, the guides appear to be out of date. I am wondering if there is no easier way to get the store and host, it seems they are used internally to check auth but not exposed anywhere in the shopify-api-js api.
Is there a reason why getting this apparently basic information is not easily accessible?


settings: Rename "default" views to "home" views in settings and docs.

This PR renames "default" views to "home" views in the setting labels, keyboard shortcuts list, help documentation and its urls.
This PR does not do changes in variable and class names, setting field in database, API docs and changelog.
Fixes part of #27251.

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I have not renamed the variables and class/ID of relevant elements, since the settings field returned by server in user_settings object is still default_view and might result in confusion.
I have also not updated the text in API docs because the API field is still default_view, but I am not sure about this.
I don't think we would want to update the API and the release changelog as well, though I have updated them to use the new links.

Added "Fixes part of .." line in commit message to make sure we do not close the issue, if we want further changes mentioned above.

I have not renamed the variables and class/ID of relevant elements, since the settings field returned by server in user_settings object is still default_view and might result in confusion.

Yeah, I think we can have a separate follow-up PR for changing that API (which is easy, since mobile doesn't use it). Maybe worth an "api design" thread first, as we probably might want to rename it to something like web_home_view or something.

Merged, thanks @sahil839!


Support for Open Distro for Elasticsearch as a Service

Open Distro for Elasticsearch https://opendistro.github.io/for-elasticsearch/ stems from AWS. They use it for their managed AWS service https://aws.amazon.com/elasticsearch-service/.
Although there is large feature overlap with Elastic's offering. There are differences when it comes to plugins and some implementation details.
Version mapping: https://opendistro.github.io/for-elasticsearch-docs/version-history/
Docker documentation: https://opendistro.github.io/for-elasticsearch-docs/docs/install/docker/
Potential quirk if you're used to using https://github.com/elastic/elasticsearch-php to connect
https://github.com/opendistro/for-elasticsearch-docs/issues/22.

This may require a new service so we could avoid image overlap. Please add this as a new container request here: https://github.com/tighten/takeout/discussions/224


LNK fail on macOS

Steps to reproduce
lnk add https://brew.sh

Expected behavior
It does not add a link to my static file
Actual behavior
/Users/bhdicaire/.vanityURLs.conf: line 6: ~/.config/bin/: No such file or directory
grep: invalid option -- P
usage: grep [-abcdDEFGHhIiJLlMmnOopqRSsUVvwXxZz] [-A num] [-B num] [-C[num]]
	[-e pattern] [-f file] [--binary-files=value] [--color=when]
	[--context[=num]] [--directories=action] [--label] [--line-buffered]
	[--null] [pattern] [file ...]
Error: Your input LONG_URL ('add') is invalid.
Hints:
- Does it start with 'http'?
- Does it contain a '.'?

System configuration
which lnk
/Users/bhdicaire/.config/bin/lnk

cat ~/.vanityURLs.conf
# This file is your config for vanityURls to personalize and share branded URLs.
# For more info, visit https://github.com/bhdicaire/vanityURLs

# The path to your local scripts folder included in your path
# For example: ~/.config/bin
SCRIPT_DIR= "~/.config/bin/"

# The path to your local copy of vanityURLs.
# For example: ~/vanityURLs
REPO_DIR="/Volumes/Tarmac/codePublic/vanityURLs/v8s.link"

# Your tiny internet domain served by Cloudflare.
# For example: dicai.re or felleg.xyz
MY_DOMAIN="v8s.link"

# Your cloudflare's specific page URL.
# For example: vanityURLs.pages.dev
MY_PAGE="v8s-link.pages.dev"

# This variable determines how long your random code will be.
# For example, with SHORTCODE_LENGTH=3, your resulting URL
# could be something like "https://dicai.re/XXX" (3 X)
SHORTCODE_LENGTH=3

# Uncomment this value if you want to run the `scripts/lnk` script
# in dry run mode, i.e. files will not be modified.
#DRY_RUN=true

ls -1 /Volumes/Tarmac/codePublic/vanityURLs/v8s.link
build
dynamic.lnk
LICENSE
Makefile
README.md
scripts
static.lnk
vanityURLs.conf

lnk https://brew.sh
/Users/bhdicaire/.vanityURLs.conf: line 6: ~/.config/bin/: No such file or directory
grep: invalid option -- P
usage: grep [-abcdDEFGHhIiJLlMmnOopqRSsUVvwXxZz] [-A num] [-B num] [-C[num]]
[-e pattern] [-f file] [--binary-files=value] [--color=when]
[--context[=num]] [--directories=action] [--label] [--line-buffered]
[--null] [pattern] [file ...]
From github.com:vanityURLs/v8s.link

branch            main       -> FETCH_HEAD
Already up to date.
[main 5c0def5] add redirect /ALM --> https://brew.sh
1 file changed, 2 insertions(+), 5 deletions(-)
Enumerating objects: 5, done.
Counting objects: 100% (5/5), done.
Delta compression using up to 14 threads
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 561 bytes | 561.00 KiB/s, done.
Total 3 (delta 2), reused 0 (delta 0), pack-reused 0 (from 0)
remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
To github.com:vanityURLs/v8s.link.git
1cf514e..5c0def5  main -> main
Your shortened URL is https://v8s.link/ALM
Please give ~15 seconds for your link to become valid

which grep
grep: aliased to grep --color=auto
~ ➜ grep --version
grep (BSD grep, GNU compatible) 2.6.0-FreeBSD

I just added homebrew's grep and the grep error disappeared.
which grep
/opt/homebrew/opt/grep/libexec/gnubin/grep

grep --version
grep (GNU grep) 3.11
Packaged by Homebrew
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Mike Haertel and others; see
<https://git.savannah.gnu.org/cgit/grep.git/tree/AUTHORS>.

grep -P uses PCRE2 10.44 2024-06-07

lnk https://github.com/vanityURL
/Users/bhdicaire/.vanityURLs.conf: line 6: ~/.config/bin/: No such file or directory
From github.com:vanityURLs/v8s.link
 * branch            main       -> FETCH_HEAD
Already up to date.
[main e399032] add redirect /VVa --> https://github.com/vanityURL
 1 file changed, 1 insertion(+)
Enumerating objects: 5, done.
Counting objects: 100% (5/5), done.
Delta compression using up to 14 threads
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 557 bytes | 557.00 KiB/s, done.
Total 3 (delta 2), reused 0 (delta 0), pack-reused 0 (from 0)
remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
To github.com:vanityURLs/v8s.link.git
   5c0def5..e399032  main -> main
Your shortened URL is https://v8s.link/VVa
Please give ~15 seconds for your link to become valid

cat /Volumes/Tarmac/codePublic/vanityURLs/v8s.link/static.lnk
/ https://vanityURLs.link/
/blog https://vanityURLs.link/en/blog
/github https://github.com/vanityURLs/

/gitlab https://gitlab.com/bhdicaire/
/linkedin https://linkedin.com/in/bhdicaire/
/x https://twitter.com/BHDicaire/
/ALM https://brew.sh
/VVa https://github.com/vanityURL

I notice this error in your output:
/Users/bhdicaire/.vanityURLs.conf: line 6: ~/.config/bin/: No such file or directory
This seems related to this variable in your ~/.vanityURLs.conf:
SCRIPT_DIR= "~/.config/bin/"
Other than that, it looks like the problem got solved. We learned that the script requires the grep cli from brew. I wonder where would be the best place to document this dependency. scripts/README.md?

We could add a check if grep -P fails, and if brew is installed, perform a brew install grep. This should solve the issue on most Macs.


[BUG]

echo "db_address = /run/redis-openvas/redis.sock" | sudo tee -a /etc/openvas/openvas.conf
This line seems to be wrong, redis is actually at run/redis-openvas/redis-server.sock after following the guide.

Thank you @subhappy will make sure to update this with the new revision.


Fix setting-exists check. Fixes #315

The check was always returning True 'cause vim.eval() returns a string if the expression evals to a number and bool('0') is True

Doh, not sure how I didn't notice when testing the v2 branch.
LGTM, will merge since it's a small hotfix.


Adds Onestar mech ghost-role enemy

About the PR
My biggest PR yet. It was a fun (and frustrating) experience, learned a couple of things.
This PR adds a new ghost role, the "Onestar mech" from Eris.
What's the spawning condition? When admins feel like it perhaps, I was thinking about it being a random spawn in a very rare event like the slaughter demon, but for now, it's more of an event mob that you can place in, have someone possess it and cause some chaos.
I have tried to make it as balanced as possible. I created two weapons, including the casing, projectiles, and magazines (this hurt).  One of them is the multi-missile launcher, which rapidly tosses missiles that deal 1 light explosion damage and the minigun was ported from TG, and I created a new type of weaponry and caliber, HMGs or Rrifle (heavy machine gun), it's just as a hugboxxy as the missiles launcher, it has like 1000 bullets but only deals 5 piercing damage, I could rise it of course. It also includes an un-edited xray laser and an energy blade.
It has 1000 hp and it's very slow, the collision radius is 1 so going through a hole of two tiles is kind of a tight fit.
Screenshots
https://user-images.githubusercontent.com/43253663/156431518-f8f86f69-7eaf-4de8-bf22-2f86b0ebb9e2.mp4

Changelog

:cl:

add: There's intel of an unknown enemy corp creating experimental mechas and testing them on stations.

Ready for review.

huge motherfucking spritesheet

Big boi.

There's missing pixels but i fixed it.
https://user-images.githubusercontent.com/43253663/157347117-54d496f9-8c18-480b-9006-1d0388b53c11.mp4


afterDisconnect on sails.lower does not wait

Sails version: 1.1.0-3
Node version: 10.11.0
NPM version: 6.4.1
DB adapter name: N/A
DB adapter version: N/A
Operating system: Kubuntu 18.04

In afterDisconnect, I need to do some asynchronous work. That works for "normal" cases, e.g. the user closes their browser window, socket closes, afterDisconnect called.
So far, so good.
However, also I also need that asynchronous logic to be done for each socket when Sails is being shut down. But that doesn't work.
When I close Sails (via Ctr+C) and thus trigger sails.lower(), the asynchronous part of the code might not be executed.
Consider this code:
afterDisconnect: function(session, socket, callback) {
    console.log("afterDisconnect called");
    setTimeout(function () {
      console.log("Waited long enough!");
            return callback();
    },6000);
  },

When triggered by sails.lower() I will never see "Waited long enough!" on the console. Same goes if I wrap that in an async/await structure:
waitForSomeTime: function () {
    console.log("Start to wait!");
    return new Promise(function (resolve,reject) {
      setTimeout(function () {
        console.log("Waited enough!");
        resolve();
      },6000);
    });
  },

  afterDisconnect: async function(session, socket, callback) {
    console.log("afterDisconnect called");
    await module.exports.sockets.waitForSomeTime();
    return callback();
  },

I don't know whether afterDisconnect was never meant to be fully executed upon sails.lower() or if this was un-intentionally done when fixing https://github.com/balderdashy/sails/issues/3397. Anyway, it definitely causes inconvenience for me. Is there at least a work around?

@cschwaderer Perhaps giving Sails more time before shutting down might help?
sails.on('lower', function() {
   setTimeout(
      function() { process.exit(1); },
      6000 // 6 seconds or change to the amount of time you need.
   );
});

add to the config/bootstrap.js

Thanks! However, I've found a different, probably better solution
In the callback of sails.lift I've added:
process.removeAllListeners('SIGINT');

process.on('SIGINT', async function () {

    // Do my stuff for every socket with await

    sails.lower(function (err) {

     });
});

That way, I can do whatever I want on SIGINT and only call sails.lower when I'm done. I don't have to rely on a timeout.
However, I think the fact that sails.lower does trigger some events (like afterDisconnect), but does not wait for their handlers to complete is worth mentioning in the docs - because it's not what I expect.

@cschwaderer Great work! Glad to help!


Replace dismiss button in Flash with IconButton

What are you trying to accomplish?
Fixing https://github.com/github/primer/issues/2550 by replacing the custom dismiss button with a PVC IconButton
Screenshots
Visual Diffs
Integration
No
List the issues that this change affects.
Closes https://github.com/github/primer/issues/2550
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

What approach did you choose and why?
There was some padding adjustment I did for the position of the button.
Anything you want to highlight for special attention from reviewers?

Accessibility

Fixes axe scan violation - This change fixes an existing axe scan violation.
No new axe scan violation - This change does not introduce any new axe scan violations.
New axe violation - This change introduces a new axe scan violation. Please describe why the violation cannot be resolved below.

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Lookbook)
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

I want to get @maximedegreve's review before merging this

Hmm, can you fix the deploy so we can see it in Lookbook?

Looks good. Thank you for fixing this @jonrohan


Fixed geometry Tree with actual type enforcement

closes #351.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 97.504%

Files with Coverage Reduction
New Missed Lines
%

input_parser/syntax_node.py
12
98.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5703

Relevant Lines:
5849

💛  - Coveralls


Add action buttons where appropriate on the /submissions/ table

Now that there are a variety of Submission statuses that may require the user to act upon a Submission (review submission, edit submission, attention required, etc.), when looking at the /submissions/ list it can be ambiguous which submissions you need to do something with and which you are waiting for someone else to act on.  The Action column in the /submissions/ list should be customized to display buttons for any submission that can be acted on by the user.

closed by #763


Validator crashes fastify application upon invalid / undefined field value

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Fastify version
4.3.0
Plugin version
No response
Node.js version
16.15.1
Operating system
Windows
Operating system version (i.e. 20.04, 11.3, 10)
10
Description
When the fastify validator calls the validator function (https://github.com/fastify/fastify/blob/66fc397212531c2565a9cb86a2c0f4a2f06a8c77/lib/validation.js#L79), it checks if a return is type or has an error on it.
The TypeBox provider actually throws an excetion (https://github.com/fastify/fastify-type-provider-typebox/blob/5901092bab5f41cb7da2da89b3be19257036c7b5/index.ts#L28), which causes fastify to crash the entire application since there is no try-catch on this request phase.

Steps to Reproduce

Create a simple application using the TypeBox provider
Register a route with a given schema (I have tested with a body param)
Try to make a request to the route with a missing field

fastify.route({
    method: "POST",
    url: "/user/sign-up",
    schema: {
        body: Type.Object({
            email: Type.String()
        })
    },
    handler() { }
})

Expected Behavior
No response

My personal solution was to make the provider do what is is actually commented to do: return validation errors:
export const TypeBoxValidatorCompiler: FastifySchemaCompiler<TSchema> = ({ schema }) => {
    const typeCheck = TypeCompiler.Compile(schema);

    return (value): any => {
        if (typeCheck.Check(value)) {
            return;
        }

        const errors = [...typeCheck.Errors(value)];

        return {
            error: errors.map((firstError) => ({
                message: firstError.message,
                schemaPath: firstError.path
            } as FastifySchemaValidationError))
        };
    }
}

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Fix submitted on PR https://github.com/fastify/fastify-type-provider-typebox/pull/28

I have the similar issue. Is funny that this week I decided to try fastify although I did the PoC in express just because I had a bit of time in my hand and I saw you guys accept PR of company SVG for your website. I hope you guys are as reliable as you are in benchmarks.
PS : When my service goes live, I'd love to represent my company as I said before. I work at Goldman Sachs,  NodeJS services are written in express and internal libraries like for auth, etc are made to work on top of express. Please do accept my PR :)


Use COMgr to read Kernel Args Metadata

Replace the read_kernargs_metadata implementation with a COMGR implementation. This is in preparation to handling Code Object V3 as well.

@wkwchau

Jenkins job is failing because it cannot find location of COMGR lib and header. Is COMGR installed on the Jenkins machines? I set the default locations to check /opt/rocm/include and /opt/rocm/lib.
[rocm_head] CMake Warning at CMakeLists.txt:214 (MESSAGE):
[rocm_head]   Code Object Manager header not found.  Use
[rocm_head]   -DCODE_OBJ_HEADER_DIR=<path_to_amd_comgr.h>.
[rocm_head]
[rocm_head]
[rocm_head] CMake Error at CMakeLists.txt:224 (MESSAGE):
[rocm_head]   Code Object Manager not found.  Use
[rocm_head]   -DCODE_OBJ_MGR_DIR=<path_to_libamd_comgr.so>.
[rocm_head]
[rocm_head]
[rocm_head] -- Configuring incomplete, errors occurred!

@aaronenyeshi I think you'll have to update docker script for the HIP CI to install the code object manager package

@aaronenyeshi I think you'll have to update docker script for the HIP CI to install the code object manager package

Do you know which repo the Dockerfile is located?

@aaronenyeshi I think you'll have to update docker script for the HIP CI to install the code object manager package

Do you know which repo the Dockerfile is located?

I have fixed the rocm-head Docker image to install code object manager. However this PR breaks build on NVCC path as well since we are explictly requiring code object manager there as well. You need to guard the cmake changes against HIP_PLATFORM=nvcc.


Add the container image to the GitHub Actions release workflow

While working on this, we can also evaluate using GitHub Packages.

Good reference: https://github.com/LukeMathWalker/cargo-chef/blob/main/.github/workflows/docker.yml

https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry


Add config to support publishing heir_play to PyPI

Currently the project owner is a google-specific account (google-fhe), but I can add more owners as collaborators if desired.
I have already published it on PyPI, so you can try it out https://pypi.org/project/heir-play/

We can probably downgrade it. I don't know what the minimum working version is.

Downgraded the min ipython version


Unclear error message - Unhandled QR Payload

Hello Team - I want to highlight an opportunity to IMHO improve the UI around SeedQR flow, specifically around one button and one error message.
I am testing the stateless, airgapped, SeedQR flows in firmware 0.1.45.
After scanning in a SeedQR and resting on the main wallet fingerprint screen, the "QR Scan" menu choice was not intuitive to me. Is this where I scan a PSBT? Can I add multiple seeds as is the case in Seedsigner? Does this button only allow me to switch to a new seed and PSBT flow is somewhere else? What other reasons might I scan a QR code right now?
Not being sure, I thought I'd test scanning in a different SeedQR at this point. I received an error code that is surely technically accurate, but I feel there's an opportunity to use the screen real estate here to guide the user into the workflows the Jade is expecting:

I don't know what all the possible or supported workflows are here, but I'm generally most familiar with PSBT signing. If that's the intended flow here, perhaps an error message of "Error - No PSBT identified in QR code" would be clearer to the user. Of course if that's the only possible flow, I might argue you'd also help the user by re-naming the menu option from "QR Scan" to "Scan PSBT".
Logic to identify that a new seed has been scanned might also be useful here. Potential feedback message "New seedphrase detected - would you like to switch to new wallet?"
Overall great work to see this SeedQR functionality running on the Jade! Thank you!!!

Thanks for the feedback! Here's a list of all supported QRs for the "QR Scan" function: https://help.blockstream.com/hc/en-us/articles/14817768566297-Which-types-of-QR-codes-are-supported-by-QR-Scan-

PSBTs
Addresses for validation
Multisig configurations
Message signing with specter desktop

Jade doesn't currently allow for 2 wallets to be available at the same time, however the "New seedphrase detected" idea is interesting if technically feasible - where Jade uses the 'logout' function behind 'session' and then opens your new wallet, or at the very least lets you know you need to logout first and rescan your SeedQR

Yes, we could look to add recognising the supported 'seed' formats here and doing something as suggested above.

FYI intend to handle this case in next fw release.  ie. if you scan a wallet/seed qr, it will pop up a dialog asking you to confirm that you want to switch wallets.

Great work by you all on the QR codes!!

Jade fw 1.0.21 should accept a wallet/seed QR from the generic 'Scan QR' screen, and ask if you want to switch wallets.
Closing as addressed - please reopen if this is not what was intended / is not sufficient.
Thanks.


Typo

Looks like it but not.
Conform>>Confirm
The following are the various requirements that you must conform to when deploying AD FS:

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

ID: 19f465e2-bc11-db54-2a2c-432833e6551d
Version Independent ID: 06c21eee-9ed9-8ce5-a6c8-e7c869d889af
Content: Federation Server Farm Using SQL Server
Content Source: WindowsServerDocs/identity/ad-fs/design/AD-FS-Requirements.md
Product: windows-server-threshold
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close

#assign:IngridAtMicrosoft


podvm: revert agent-config path to /etc

The ./podvm kata-agent unit has been set to use a config file in /etc. Having the kata-agent config file in /run will break the CAA libvirt tests, since we have dependencies that rely on the config being in /etc. ./podvm-mkosi will override this path to a configuration in /run.
Tested w/ PodVM images built w/ packer + mkosi

I guess there is still the question of whether the guest-components that read directly from /etc/agent-config.toml will be broken in the mkosi build if that is still using the new /run/peerpods/ path?

I guess there is still the question of whether the guest-components that read directly from /etc/agent-config.toml will be broken in the mkosi build if that is still using the new /run/peerpods/ path?

yes, independently from this PR, cc_kbc doesn't work w/ mkosi at the moment, as the relevant paths are hardcoded in guest-components, so we need to add a fix for that still.


chore(tests): update thiserror to v2

This mainly follows https://github.com/dtolnay/thiserror/releases/tag/2.0.0 and should be fairly straightforeward

WTF??
How did this automerge with CI not aggreeing on how I merged this..


Added my folder and created a readme.md file

Description

I have added a folder(Named it as my GitHub user id)
I have added a Readme.md file inside the folder
I understand the GitHub Forking Concept fully

Description

[ ] I have added a folder(Named it as my GitHub user id)
[ ] I have added a Readme.md file inside the folder
[ ] I understand the GitHub Forking Concept fully


Empty GUIDs in subject when creating new users

Hi.
I use a combination of IdSrv + IdSrv.AspNetIdentity + IdManager + IdManager.AspNetIdentity and I encountered an issue with creating new users from IdentityManager UI.
I use AspNet.Identity users with GUIDs as Ids. When I create a new user, his subject is always set to empty Guid, so I can't create any more users this way.

Can you debug into it a bit to help me determine what might be the issue?

In the IdentityUser base class, it sets the ID to a new guid. The subject in IdentityManager for ASP.NET Identity is just that ID. Not sure why yours is getting an empty guid.

Any updates?

I'm sorry for the delay. I know that this issue has been already closed but I would like to share my solution/work around of the problem. Actually this problem is not strictly related to Identity Manager itself, but rather to Microsoft AspNet Identity.
While creating a new user in IdentityManager's UI the collected form values (username and password) are passed to the AspNetIdentityManagerService (delivered with IdentityManager.AspNetIdentity) and then inside of the CreateUserAsync method a new user class is created with its parameterless constructor.
I use my own user class which derives from IdentityUser<TKey, TLogin, TRole, TClaim> from Microsoft.AspNet.Identity.EntityFramework nuget package. Everything that I had to do was to generate a new GUID for user's id in the parameterless constructor of my user class.


Url validation doesn't seem to work?

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When using
  <TextFieldElement
    autoComplete="off"
    fullWidth
    label="Url"
    name="url"
    type="url"
    required
    variant="standard"
  />

Url validation doesn't seem to occur
Expected behavior 🤔
I would expect to see a validation error with helper text saying "Url is not valid"
Steps to reproduce 🕹
Steps:

Add provided TextFieldElement
Enter an invalid url string
Submit

@hypergig react-hook-form-mui doesn't add a url validation when you specify the type="url" for input field. Try adding a validation using the validation prop.
  validation={{
    pattern: {
      value: /^(https?):\/\/[^\s/$.?#].[^\s]*$/,
      message: 'Please enter a valid url',
    },
  }}

this is changed in v7 beta and rules property is providing this functionality


Create Aiopg pool lazily

I was playing with the new pools (cf #173) in my project, and I had a few issues with the new design.
In order to create an App, I need a Connector. And in order to create a Connector, I either need to create my own pool, or to use PosgresConnector.create_with_pool_async which will create one for me. This has two implications:

I can't create a pool without an event loop, so I can't instantiate an App before starting my event loop. Instantiating an App usually happen very early (before defining tasks), so it should not have the side effect of creating an event loop, nor require one.
I can't instantiate an App without connecting to my DB. I like to be able to start my services without a DB connection for resiliency. This makes it close to impossible. Also it's very simple to end up in a situation where you have side-effects at import time, which is usually a bad idea.

So I've tried to mess around a bit with the PostgresConnector in order to create the loop lazily, like the connection is in Procrastinate 0.6. My solution is probably not perfect, and it will require some doc changes (but the current doc is not up-to-date with master anyway).
Please let me now what you think :) .
Successful PR Checklist:

[ ] Tests
[ ] Documentation (optionally: run spell checking)
[ ] Had a good time contributing? (if not, feel free to give some feedback)

We can discuss this but, why not consider creating the app lazily then ?

We can discuss this but, why not consider creating the app lazily then ?

I think I agree with @k4nar that application developers would expect to be able to create the Application instance very early in the startup process, and possibly before the event loop is created.
If we don't want to go the "create pool lazily" way we could expose set_pool and create_pool methods.
app = procrastinate.Application(connector=procrastinate.PostgresConnector())

And then, after the event loop has been created:
pool = await app.connector.create_pool(**pool_args)
app.connector.set_pool(pool)

PostgresConnector.create_pool() would be provided as a convenience. The application developer could also create the pool by herself:
pool = await aiopg.create_pool(**pool_args)
app.connector.set_pool(pool)

I do agree that we may have to end up doing this.
I feel a bit sad about the fact you can't instantiate a pool synchronously (even if you tell it not to connect right away, it insists on being created with a coroutine, and even if you tried calling the constructor yourself, it expects a running loop). So Aiopg gets away while saying "I want to be started after the loop", but it looks like we can't and I'm not sure what's the big difference that makes it acceptable for them and not us. It's because they refuse to instantiate a pool synchronously before starting the loop that we end up with the same problem.
That being said, I can accept that we're simply trying to do better than aiopg.

FWIW I'd rather expose create_pool and set_pool functions than creating the pool lazily in execute_query. Because the user can control when the pool is created. And because that gives an API for passing ones own pool to Procrastinate.

What I have in mind: https://github.com/peopledoc/procrastinate/compare/ele_pool.

That's a good alternative indeed. In my project I try to connect to the DB as late as possible, in order to be able to start before the DB without issues. I'll try to see if I can still do it with set_pool.


Cannot turn on Delonghi Primadonna anymore

Hello, thanks for this great integration, I love it !
Since the last release my Primadonna stopped working, which means I cannot turn it on by Home Assistant any more.
The status, like descaling, is it running or not seems to work, these entities update if I turn the Primadonna on manually.
In the protocols I have after each try to turn it on 2 logs, a warning and and error:
this is the warning log:
Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht
Logger: custom_components.delonghi_primadonna.device
Source: custom_components/delonghi_primadonna/device.py:355
Integration: DeLonghi BLE (documentation, issues)
First occurred: 7. November 2023 um 22:10:00 (9 occurrences)
Last logged: 07:34:47
BleakDBusError: [org.bluez.Error.Failed] Operation failed with ATT error: 0x0e (Unlikely Error)
BleakDBusError: [org.bluez.Error.Failed] Operation already in progress
and this is the error-log:
Logger: homeassistant
Source: components/bluetooth/wrappers.py:292
First occurred: 07:33:17 (9 occurrences)
Last logged: 07:34:47
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/config/custom_components/delonghi_primadonna/device.py", line 294, in power_on
await self.send_command(BYTES_POWER)
File "/config/custom_components/delonghi_primadonna/device.py", line 377, in send_command
await self._connect()
File "/config/custom_components/delonghi_primadonna/device.py", line 222, in _connect
raise error
File "/config/custom_components/delonghi_primadonna/device.py", line 216, in _connect
await self._client.connect()
File "/usr/src/homeassistant/homeassistant/components/bluetooth/wrappers.py", line 292, in connect
connected = await super().connect(**kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/bleak/init.py", line 605, in connect
return await self._backend.connect(**kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/bleak/backends/bluezdbus/client.py", line 249, in connect
assert_reply(reply)
File "/usr/local/lib/python3.11/site-packages/bleak/backends/bluezdbus/utils.py", line 22, in assert_reply
raise BleakDBusError(reply.error_name, reply.body)
bleak.exc.BleakDBusError: [org.bluez.Error.Failed] br-connection-canceled
I hope this helps ;)

What Bluetooth adapter are you using? How many other Bluetooth devices are you accessing in HA? Are you using Bluetooth Proxies? Is the coffee machine far away from your HA device?

I have the same issue as @mannipulator with the same machine (Delonghi Primadonna). This integration is the only one using bluetooth on my home assistant system. The machine is aprox. 6m away from my bluetooth adapter which worked fine with this integration for the last months.

In my case, the problem has probably solved itself and everything is working as usual again. Nevertheless, here is some more information about my setup:
HassOS on RPi4
Home Assistant Version 23.11.2
Delonghi BLE v1.5.3
Devices connected with Bluetooth: 2

Ok it is back again
After each major update of HASS, the integration stops working.
I recognized that if I turn the Primadonna manually on, the Bluetooth symbol on the display appears around every 10 seconds for less then a second and then disappears. Maybe that’s a sign of a failing connection.
here is the log:
Logger: homeassistant
Source: custom_components/delonghi_primadonna/device.py:222
Integration: DeLonghi BLE (documentation, issues)
First occurred: 07:14:40 (2 occurrences)
Last logged: 07:16:35
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/config/custom_components/delonghi_primadonna/device.py", line 302, in power_on
await self.send_command(BYTES_POWER)
File "/config/custom_components/delonghi_primadonna/device.py", line 403, in send_command
await self._connect()
File "/config/custom_components/delonghi_primadonna/device.py", line 234, in _connect
raise error
File "/config/custom_components/delonghi_primadonna/device.py", line 222, in _connect
raise BleakError(
bleak.exc.BleakError: A device with address 00:A0:50:14:19:B0                             could not be found.

Is the app working for you? Not being able to find the MAC address at all suggests that the coffee machine itself is not broadcasting or not within range of your Bluetooth adapter.

No, it stopped to work now completely, I cannot get any connection to the device.


Add resnet50 benchmark (#443)

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #443

Modifications of the original resnet50 in torchvision:
replace out += identity with out = out + identity
ReLU(replace=False)
set num_batches_tracked = None for BatchNorm (a workaround since add_ is utilized in nn.BatchNorm3d when num_batches_tracked is not None).
needs: #365

@kiya00 could you try resnet50 w/o modifications? I expect it to work following https://github.com/Lightning-AI/lightning-thunder/issues/633

@kiya00 could you try resnet50 w/o modifications?   Supported types are <class 'thunder.core.dtypes.dtype'>

thunder/core/baseutils.py:103: ValueError

handle shape list with mixed IntegerProxy and int, is this a known problem?

Two parts to this question: 1. nvfuser should work with dynamic reshape (AFAIK). So it could be just a nvfuser_impl thing in thunder.
2. thunder doesn't really support dynamic shape properly yet. I'm scared about having a shape list with IntegerProxy in it at this point. I think we should go dig out where it's coming from.

thunder doesn't really support dynamic shape properly yet. I'm scared about having a shape list with IntegerProxy in it at this point. I think we should go dig out where it's coming from.

t4702 = prims.reshape(t4687, (i1001, 1024, 64, 13, 13)), the i1001 comes from the IntegerProxy passed from forward trace:
the augment_fwd_trace return int numbers, but backward_fn takes it as IntergerProxy, is that expected? @jjsjann123
# fwd return
return {'output': t1234, 'flat_args': [x, t_bn1_bias, t_bn1_num_batches_tracked, ...], 'flat_output': (t1234,)}, ((t0, t1011, ...), (False, 3, 0, 1, 0, 1, 1, 1, 1, 1, 1, 0, 1, 1, 0, 0, 1, 0, 0, 1, 1, 1, 0, 0, 1, 1, 0, 0, 1, 0, 1, 1, 0, 0, 0, 1, 1, 0, 0, 1, 0, 1, 1, 0, 0, 1, 1, 0, 0, 1, 0, 2, 2, 1, 1, 1, 1, 0, 0, 1, 0, 1, 1, 0, 0, 1, 1, 0, 0, 1, 0, 2, 2, 0, 0, 1, 1, 0, 0, 1, 0, 1, 1, 0, 0, 1, 1, 0, 0, 1, 0, 3, 2, 2, 1, 1, 2, 1, 1, 0, 0, 1, 1, 0, 1, 1, 1, 0, 0, 1, 1, 0, 0, 1, 2, 0, 1, 2, 2, 0, 1, 0, 1, 1, 0, 0, 1, 0, 0, 0, 1, 1, 2, 1, 0, 1, 0, 0, 1, 1, 0, 0, 1, 0, 0, 1, 2, 2, 1, 1, 1, 1, 0, 0, 1, 0, 3, 0, 1, 1, 0, 0, 1, 1, 0, 0, 1, 0, 2, 2, 0, 0, 1, 1, 0))

# bwd inputs
C0, C1, = saved_for_backward
  clear_mutable_collection(saved_for_backward)
  del saved_for_backward
  t11, = cotangents
  clear_mutable_collection(cotangents)
  del cotangents
  t0, t1011, t1012, t1013, t1021, t1024, t1084, t1085, t1086, t1094, t1096, \
  ... = C0
  clear_mutable_collection(C0)
  del C0
  b66, i10, i1000, i1001, i1011, i11, i12, i136, i137, i138, i139, i14, i140, \
  i141, i143, i144, i145, i15, i155, i16, i198, i199, i200, i201, i202, i203, \
  i205, i206, i207, i217, i257, i258, i259, i26, i260, i261, i262, i264, i265, \
  i266, i276, i319, i320, i321, i322, i323, i324, i326, i327, i328, i338, i381, \
  i382, i383, i384, i385, i386, i388, i389, i390, i400, i443, i444, i445, i446, \
  i447, i448, i450, i451, i452, i462, i502, i503, i504, i505, i506, i507, i509, \
  i510, i511, i521, i564, i565, i566, i567, i568, i569, i571, i572, i573, i583, \
  i62, i626, i627, i628, i629, i63, i630, i631, i633, i634, i635, i64, i645, i65, \
  i688, i689, i690, i691, i692, i693, i695, i696, i697, i7, i707, i74, i747, \
  i748, i749, i75, i750, i751, i752, i754, i755, i756, i76, i766, i77, i78, i79, \
  i8, i809, i81, i810, i811, i812, i813, i814, i816, i817, i818, i82, i828, i83, \
  i871, i872, i873, i874, i875, i876, i878, i879, i880, i890, i9, i93, i933, \
  i934, i935, i936, i937, i938, i940, i941, i942, i952, i992, i993, i994, i995, \
  i996, i997, i999, = C1
  clear_mutable_collection(C1)
  del C1

the augment_fwd_trace return int numbers, but backward_fn takes it as IntergerProxy, is that expected?

That's not.
i.e. we should keep it consistent if the forward generates a vanilla integer. And likewise when it is a numberproxy. We just need to further patch #244
I'll give it a try later.

#706 patches nvfuser's reshape with dynamic input.
Meanwhile. I'm still trying to get a repro where grad transform generates a numberproxy where it shouldn't have. Looking at the trace, I think it's coming from convolution....

Status update: Thanks to @jjsjann123 's patch #706, the failure in #451 (comment) is gone, but there is an nvfuser failure about " Unsupported loop structure. Two loops are mapped together.bS323{1} and bS319{1}", I'm asking nvfuser team's help to identify if it's an nvfuser issue

This might be https://github.com/NVIDIA/Fuser/issues/2685, but the RN50 on this PR is likely a more sane reproducer.

Apology for the slow turnaround.
With nvfuser's issue patched, I'm no longer seeing any failures coming from this. cc'ing @kiya00
root@aca1f438e9da:/opt/pytorch/lightning-thunder# pytest thunder/benchmarks/targets.py -k test_resnet50[backward-thunder]
======================================================= test session starts =======================================================
platform linux -- Python 3.10.12, pytest-8.1.1, pluggy-1.5.0
Test order randomisation NOT enabled. Enable with --random-order or --random-order-bucket=<bucket_type>
benchmark: 4.0.0 (defaults: timer=torch.utils.benchmark.utils.timer.timer disable_gc=False min_rounds=5 min_time=0.000005 max_time=1.0 calibration_precision=10 warmup=True warmup_iterations=100000)
rootdir: /opt/pytorch/lightning-thunder
configfile: pyproject.toml
plugins: timestamper-0.0.10, hypothesis-6.104.2, random-order-1.1.1, timeout-2.3.1, cov-5.0.0, benchmark-4.0.0, xdist-3.6.1, shard-0.1.2, typeguard-4.3.0
timeout: 900.0s
timeout method: signal
timeout func_only: False
collected 728 items / 727 deselected / 1 selected
Running 1 items in this shard

thunder/benchmarks/targets.py

.                                                                                [100%]

------------------------------------------------------ benchmark: 1 tests ------------------------------------------------------
Name (time in ms)                        Min       Max      Mean  StdDev    Median     IQR  Outliers     OPS  Rounds  Iterations
--------------------------------------------------------------------------------------------------------------------------------
test_resnet50[backward-thunder]     110.6647  111.1892  110.7535  0.1542  110.7138  0.0271       1;1  9.0291      10           1
--------------------------------------------------------------------------------------------------------------------------------

Legend:
  Outliers: 1 Standard Deviation from Mean; 1.5 IQR (InterQuartile Range) from 1st Quartile and 3rd Quartile.
  OPS: Operations Per Second, computed as 1 / Mean
==================================== 1 passed, 727 deselected, 7 warnings in 134.28s (0:02:14) ====================================
root@aca1f438e9da:/opt/pytorch/lightning-thunder#


[HW - Tiktok Clone Workshop]

Name: Saptarshi Banerjee
Email Id<EMAIL_ADDRESS>App URL: https://tiktok-workshop-rishi.netlify.app

Created the tiktok workshop Cassandra db in astra

Cloned the app into my github repo

Linked the app to netlify and created the site workspace

Build the app in gitpod and started it dev server

Configured the app with Astra db and build it production

Deployed the app into prod using netlify

Tiktok clone successfully deployed in netlify

Great Job, thank you for your work.
Here is your badge:
<EMAIL_ADDRESS>I am pretty sure I also look at homework for the Netflix workshop coming for you. Thank you for being with us for multiple weeks, appreciate the quality work.

Thanks Cedrick, I am looking forward for more sessions like this. This actually helping me to learn new concepts.. Great efforts from datastax team.


GarageGateOpener init issues

I recently added a GarageGateOpener and debugging the code with my Homebridge-Mi-Platform ,and notice that sometimes method setTargetDoorState will be called when start running homebridge and the hexstring is open hex data.
that means if you restart your homebridge for some reason , your garage gate will be opened because of that ..

exactly same issue with window covering accessories: when Homebridge starts it closes some blinds and then the home app shows open but they are closed. Weird.

I have the same issue too

Can you please provide logs and your config. I've been unable to reproduce this

Closing, please re-open if you would still like help


Subscribing to inactive mapping

Currently the mapper subscribes to topics even the mapping is not activated.

This is due the current implementation and concept that multiple mappings (different mappingTopic) could be linked to the same subscription (same subscriptionTopic).
activating/deactivating a mapping means that the messages are still received but not processed.
An other case where two mappings listen to the same topic is the following:

mapping-event subscribes to topic panel
mapping-alarm subscribes to topic panel

They both consume the same messages but process them differently.
If you deactivate  mapping-event and unsubscribe from panel, the other mapping mapping-alarm is deactivated as well.
Which might not be what you want.

We could come up with a logic that checks if there is any mapping active that subscribes on a topic and if so only start subscribing on that topic and vice versa if there is no other mapping active that subscribes on that topic we just unsubscribe.
But I would see that as a nice-to-have feature, so low priority currently.

Changed the implementation to only subscribe to a topic if any mapping is active.
Changes are included in: a4a6c4993730075893d0b73efe0a70e4c4a02f9a
Could you pls. test if this fixes the issue?


Failed to build wasm for v0.6.1: multiple rules generate lib/libjxl.a

Describe the bug
due to certain limits, I have to build wasm for version 0.6.1 and ran into this issue. not sure how to resolve it. any helps is appreaciated.
Run: BUILD_TARGET=wasm32 ENABLE_WASM_SIMD=1 emconfigure ./ci.sh release
Ouput: error below

Build files have been written to: /Users/username/Documents/work/libjxl/build-wasm32

cmake_build_and_test
ASAN_OPTIONS=detect_leaks=0
cmake --build /Users/username/Documents/work/libjxl/build-wasm32 -- all doc
ninja: error: build.ninja:3802: multiple rules generate lib/libjxl.a.

Will check soon

Problem comes from modern versions on doxygen / ninja.
Likely will build on some older OS (e.g. should try Ubuntu 18.04 in docker).
On my PC mostly builds with the following  changes:
diff --git a/ci.sh b/ci.sh
index e4809e77..37165e1d 100755
--- a/ci.sh
+++ b/ci.sh
@@ -463,7 +463,7 @@ cmake_configure() {
 cmake_build_and_test() {
   # gtest_discover_tests() runs the test binaries to discover the list of tests
   # at build time, which fails under qemu.
-  ASAN_OPTIONS=detect_leaks=0 cmake --build "${BUILD_DIR}" -- all doc
+  ASAN_OPTIONS=detect_leaks=0 cmake --build "${BUILD_DIR}" -- all
   # Pack test binaries if requested.
   if [[ "${PACK_TEST:-}" == "1" ]]; then
     (cd "${BUILD_DIR}"
diff --git a/lib/jxl.cmake b/lib/jxl.cmake
index c0290ab0..484896ef 100644
--- a/lib/jxl.cmake
+++ b/lib/jxl.cmake
@@ -484,8 +484,8 @@ endif()  # JPEGXL_ENABLE_TCMALLOC
 # Install the static library too, but as jxl.a file without the -static except
 # in Windows.
 if (NOT WIN32)
-  set_target_properties(jxl-static PROPERTIES OUTPUT_NAME "jxl")
-  set_target_properties(jxl_dec-static PROPERTIES OUTPUT_NAME "jxl_dec")
+  #set_target_properties(jxl-static PROPERTIES OUTPUT_NAME "jxl")
+  #set_target_properties(jxl_dec-static PROPERTIES OUTPUT_NAME "jxl_dec")
 endif()
 install(TARGETS jxl-static DESTINATION ${CMAKE_INSTALL_LIBDIR})
 install(TARGETS jxl_dec-static DESTINATION ${CMAKE_INSTALL_LIBDIR})
@@ -551,8 +551,8 @@ endforeach()
 install(TARGETS jxl
   DESTINATION ${CMAKE_INSTALL_LIBDIR})
 else()
-add_library(jxl ALIAS jxl-static)
-add_library(jxl_dec ALIAS jxl_dec-static)
+#add_library(jxl ALIAS jxl-static)
+#add_library(jxl_dec ALIAS jxl_dec-static)
 endif()  # TARGET_SUPPORTS_SHARED_LIBS AND NOT JPEGXL_STATIC

 # Add a pkg-config file for libjxl.
diff --git a/lib/jxl_threads.cmake b/lib/jxl_threads.cmake
index 85ceda49..07deaa2d 100644
--- a/lib/jxl_threads.cmake
+++ b/lib/jxl_threads.cmake
@@ -41,7 +41,7 @@ set_target_properties(${_target} PROPERTIES
 # Always install the library as jxl_threads.{a,so} file without the "-static"
 # suffix, except in Windows.
 if (NOT WIN32)
-  set_target_properties(${_target} PROPERTIES OUTPUT_NAME "jxl_threads")
+  #set_target_properties(${_target} PROPERTIES OUTPUT_NAME "jxl_threads")
 endif()
 install(TARGETS ${_target} DESTINATION ${CMAKE_INSTALL_LIBDIR})


server: replace the sys-info with sysinfo

Signed-off-by: Lonng<EMAIL_ADDRESS>
What have you changed?
Split the https://github.com/tikv/tikv/pull/6135 into small PRs.
This PR replaces the sys-info with sysinfo.
What is the type of the changes?
Pick one of the following and delete the others:

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?
Please select the tests that you ran to verify your changes:

Existing unit test

/merge

/test


Fix compilation on linux

Fix Linux compilation.
Libraries are fun

Not needed as I've already fixed this in 23b36d1 ^^

Well thanks I missed that


syscall: windows doesn't allow read-only directories

On Windows, right now we map the lack of the writable bit in Unix file permissions to FILE_ATTRIBUTES_READONLY.  This sort of works fine, but MSDN says that it's not honored for directories. This has implications for the Go module cache.
I'm not suggesting that we change this mapping to something else right now, but in case things pop up down the line related to this, here's a bug to track it.
cc @bcmills @alexbrainman

cc @jayconrod like the other bug.

Same question here: is there anything to actively do for this issue? (Should we leave it open, or close it and just use it as a point of reference?)

Hmm... At the very least, the documentation for os.Chmod should be updated.
It currently says:

On Windows, only the 0200 bit (owner writable) of mode is used; it controls whether the file's read-only attribute is set or cleared. The other bits are currently unused. For compatibility with Go 1.12 and earlier, use a non-zero mode. Use mode 0400 for a read-only file and 0600 for a readable+writable file.

It notably does not mention any behavior for directories.

Windows files and directory permissions cannot be mapped onto Unix user/group/everyone + read/write/execute. Most Windows users don't care about file permissions - as long as their files inherit permissions of containing directories, most things just work. For anything unusual Windows Go users would have to use Windows APIs directly.
More documentation is not always better. If documentation is long, people won't read it.
But we should correct wrong documentation.
Alex

I'm in way over my head, but from what I can tell there may be a reasonable mapping of the unix “write” permission bit to the windows ACTRL_DIR_CREATE_OBJECT, ACTRL_DIR_CREATE_CHILD, and ACTRL_DIR_DELETE_CHILD permissions.
Similarly, the execute bit might reasonably map to ACTRL_DIR_TRAVERSE.
(But none of that is to say that we should actually implement that mapping.)

I'm in way over my head,

Same here. I will let Jason reply to you, if he wants to. I think we should leave things as they are.
Alex

I'm in way over my head, but from what I can tell there may be a reasonable mapping of the unix “write” permission bit to the windows ACTRL_DIR_CREATE_OBJECT, ACTRL_DIR_CREATE_CHILD or ACTRL_DS_CREATE_CHILD, and ACTRL_DIR_DELETE_CHILD or ACTRL_DS_DELETE_CHILD permissions.
Similarly, the execute bit might reasonably map to ACTRL_DIR_TRAVERSE or ACTRL_DS_LIST.
(But none of that is to say that we should actually implement that mapping.)

Mapping Unix permission bits to Windows ACLs probably makes more sense than what we do now, which is mapping a single Unix permission bit to a "file attribute", which is not an ACL (and apparently doesn't work for directories either).
I think there probably would be a quasi coherent scheme we could come up with, for all of user, group, and world, and maybe even for the sticky bit. It would map better than what we currently have and would wind up supporting things like directories.
But it does certainly require more than a moment's thought. As with most things Windows, the devil is in the details.
The actual implementation would wind up looking something like:
func unixPermToSA(perm FileMode) *SECURITY_ATTRIBUTES
func saToUnixPerm(sa *SECURITY_ATTRIBUTES) FileMode
And then all of the various Windows file creation and retrieval functions will take or return a security attribute struct, which we're currently passing in as nil.
But, as I said, the devil is really in the details, and there are tons of unanswered questions. Then, let's say we get the mapping down. Do we want to emulate umask too? Probably, since people tend to pass in 0666 rather than 0644. So now we're doing umask emulation...
It all seems possible to do, but probably hard, and risky if it's not done right. So maybe this kind of project is best deferred until somebody actually complains compellingly about the status quo.

@jstarks has Microsoft published a guide re mapping Unix filemode to/from Windows security_attributes? Or is there .NET code to handle this?

There is no such document that I am aware of.
I don't think you want to try to do anything automatic with ACLs. You're unlikely to come up with a policy that will make everyone happy.


setup_docs: Add command to terminate WSL2 environment.

This commit adds an alternative way to shutdown the zulip WSL2 environment.

Fixes:

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@amanagr could you please take a look?


Tracing back installed openwrt

To whom it may regard,
is "AO<EMAIL_ADDRESS>the tag for this custom build?
I am not 100% sure I got my build from here, but here's the output from the terminal on my router.
root@OpenWrt:~# ubus call system board {         "kernel": "5.15.150",         "hostname": "OpenWrt",         "system": "ARMv8 Processor rev 4",         "model": "FriendlyElec NanoPi R4S",         "board_name": "friendlyarm,nanopi-r4s",         "rootfs_type": "ext4",         "release": {                 "distribution": "OpenWrt",                 "version": "23.05.3",                 "revision": "r23809-234f1a2efa",                 "target": "rockchip/armv8",                 "description": "AO<EMAIL_ADDRESS>       } }

Yes 2024.03.27 is the date of build and combined with version (23.05.3) you can identify the build. AO = anaelorlinski ;)
You can identify rootfs_type to ext4. I am not sure how you can idenfity the mini vs full however, maybe check if docker is available in the image, only avail in the full version.


Update _index.md

zapier-integration link updated

Hello @josephjosedev,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

yes

Thank you @justinegeffen  I would  prefer to open a new PR


[ unexpected in sh on linux

Running folderfix.sh from page 60 yields this error on ubuntu bash.
no% 2016-10-09 12:30:23 ☆  sayth-Latitude-E6410 in ~/php/homestead_improved-master
○ → sh bin/folderfix.sh
bin/folderfix.sh: 3: [: Linux: unexpected operator
bin/folderfix.sh: 7: [: Linux: unexpected operator

realised from laravel page it is bash bin/folderfix.sh I should've run.


Remove player from notification when destroying track player on iOs

There were an issue destroying player on iOs ( tracked here #886 ), the destroy method didn't seemed to be linked to any code (except the print).
So I just followed instructions from a SwiftAudio issue (https://github.com/jorgenhenrichsen/SwiftAudio/issues/83#issuecomment-575966626) and seems to be working now for iOs.

@MAVERlCK @dcvz I think the destroy function also needs to set hasInitialized = false. Without it you won't be able to call setupPlayer again.


Http server implementation , get post content

Using your already implemented http server,is it possible to access "post" data from received requests? If so, how?
I made a workaround by placing the required data in queryparameters,since i have access to the full link requested by a client, but it would be great to be able to get more data from the received http requests.
It's not a blocking issue since it can be worked around with query parameters
Thank you in advance, haven't found any other issues with your project so far. It's great :D

Ok, so in the latest version of the HTTP server all header fields are part of the request object passed to the request handler function.
POST data can be read in the request handler by calling connection.socket_read(content_length).
Here is an example:
https://github.com/eerimoq/pumbaa/blob/master/examples/http_server/main.py#L45-L60

Part of release 2.6.1.

Using your sample code, connection.socket_read( any length ) gets stuck until i cancel the post request from postman
Everything else works fine, i'm able correctly receive GET responses to esp32's https server

What does the POST request look like in Wireshark?

i'm currently debugging using osx, i see that wireshark only supports windows/ unix
here is the post request from postman ( chrome extension ) that i'm using
POST /test HTTP/1.1
Host: <IP_ADDRESS>
Content-Type: multipart/form-data; boundary=----WebKitFormBoundary7MA4YWxkTrZu0gW
Cache-Control: no-cache
Postman-Token: 3c7b8335-1389-ff08-b830-947533bf6c5e
------WebKitFormBoundary7MA4YWxkTrZu0gW
Content-Disposition: form-data; name="var"
val
------WebKitFormBoundary7MA4YWxkTrZu0gW--
esp32 responds correctly to get request ; it gets the correct content_length from the request. connection.socket_read(1) just gets stuck until i close the socket from the osx

Ok,what i haven't mentioned is that i'm trying to get POST content from the httpS server*.
My bad on this part xD
https://github.com/eerimoq/pumbaa/blob/master/examples/https_server/main.py

Does it work with non-encrypted HTTP?

Try changing https://github.com/eerimoq/pumbaa/blob/master/src/module_inet/class_http_server.c#L133 to
res = socket_read(&self_p->connection_p->chan_p, vstr.buf, size);

changing it to
res = socket_read( (struct socket_t*) (&self_p->connection_p->chan_p), vstr.buf, size);
solved the blocking issue, but the actual post data is ignored, all bytes returned are 0
I haven't tested it on non-encrypted HTTP, i'll test it asap, but i don't think there's an issue with the SSL

It should be chan_read(), not socket_read().

I pushed a fix for this issue. See 81f93387141b1fa519566ea4863c63baf2d12c42.

Works like a charm ! Thank you.

=D


[BugBash] url incorrect in readme

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
follow https://github.com/Azure/azure-webpubsub/blob/webpubsub-apollo-experimental/experimental/sdk/webpubsub-graphql-subscribe/how-to-host-websockets.md#open-graphql-explorer-and-update-the-subscription-url
wss://demo1.webpubsub.azure.com/client/hubs/graphql_subscription should be customer's endpoint, to correct this:
`wss://.webpubsub.azure.com/client/hubs/graphql_subscription``

Exceptions (if any)

Further technical details

SDK and its version you use

The latest doc updated the URL