[Snyk] Upgrade @aws-sdk/client-sqs from 3.621.0 to 3.622.0

Snyk has created this PR to upgrade @aws-sdk/client-sqs from 3.621.0 to 3.622.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 22 days ago.

Release notes

Package name: @aws-sdk/client-sqs

3.622.0 - 2024-08-013.622.0(2024-08-01)
Chores

codegen:

sync for command ref in context (#6347) (245450ea)
move eventstream services to id-auth sra (#6340) (6bf32b17)

clients:  enable identity-and-auth SRA by default (#6299) (56ce5e5e)
turbo:  update Turborepo devDependency to 2.0.9 (#6334) (042b7dda)

Documentation Changes

client-memorydb:  Doc only update for changes to deletion API. (bd90490b)
client-support:  Doc only updates to CaseDetails (c28e9390)
client-controltower:  Updated Control Tower service documentation for controlcatalog control ARN support with existing Control Tower public APIs (2f7a9651)
supplemental-docs:  add documentation for using queue names with SQS Client (#6337) (9f4b6df4)

New Features

client-iam:  Add v2 smoke tests and smithy smokeTests trait for SDK testing. (b754d411)
client-bedrock:  API and Documentation for Bedrock Model Copy feature. This feature lets you share and copy a custom model from one region to another or one account to another. (b85315c5)
client-sagemaker:  This release adds support for Amazon EMR Serverless applications in SageMaker Studio for running data processing jobs. (b43bb894)
client-rds:  This release adds support for specifying optional MinACU parameter in CreateDBShardGroup and ModifyDBShardGroup API. DBShardGroup response will contain MinACU if specified. (d2d3328a)
client-ssm-quicksetup:  This release adds API support for the QuickSetup feature of AWS Systems Manager (71f373db)
client-controlcatalog:  AWS Control Tower provides two new public APIs controlcatalog:ListControls and controlcatalog:GetControl under controlcatalog service namespace, which enable customers to programmatically retrieve control metadata of available controls. (c897818b)

For list of updated packages, view updated-packages.md in assets-3.622.0.zip

3.621.0 - 2024-07-303.621.0(2024-07-30)
Chores

codegen:

remove fast-xml-parser from AwsDependency (#6328) (197bc7d8)
bump smithy package versions (#6329) (8dd7fb63)

deps:  bump fast-xml-parser from 4.2.5 to 4.4.1 (#6330) (c547e234)

Documentation Changes

client-elasticache:  Doc only update for changes to deletion API. (3cf46f19)
client-workspaces:  Removing multi-session as it isn't supported for pools (01552b00)

New Features

clients:  update client endpoints as of 2024-07-30 (ecc97a25)
client-lex-models-v2:  This release adds new capabilities to the AMAZON.QnAIntent: Custom prompting, Guardrails integration and ExactResponse support for Bedrock Knowledge Base. (4d9c7a12)
client-eventbridge:  Add v2 smoke tests and smithy smokeTests trait for SDK testing. (65290c81)
client-rolesanywhere:  IAM RolesAnywhere now supports custom role session name on the CreateSession. This release adds the acceptRoleSessionName option to a profile to control whether a role session name will be accepted in a session request with a given profile. (5bc30087)
client-cloudwatch-logs:  Add v2 smoke tests and smithy smokeTests trait for SDK testing. (88b6a41c)
client-codepipeline:  AWS CodePipeline V2 type pipelines now support stage level conditions to enable development teams to safely release changes that meet quality and compliance requirements. (5e8b92d3)
client-elastic-load-balancing:  Add v2 smoke tests and smithy smokeTests trait for SDK testing. (6cbe7bf6)
client-tnb:  This release adds Network Service Update, through which customers will be able to update their instantiated networks to a new network package. See the documentation for limitations. The release also enhances the Get network operation API to return parameter overrides used during the operation. (cee6948a)
client-auto-scaling:  Increase the length limit for VPCZoneIdentifier from 2047 to 5000 (23471630)
client-appstream:  Added support for Red Hat Enterprise Linux 8 on Amazon AppStream 2.0 (bbb4450b)

Bug Fixes

package.json homepage links to use the main branch (#6336) (e3322302)
polly-request-presigner:  override middleware name to allow concurrent presigning (#6335) (54990fa2)

For list of updated packages, view updated-packages.md in assets-3.621.0.zip

    from @aws-sdk/client-sqs GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

Project coverage is 100.00%. Comparing base (f99d854) to head (533fca0).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #25   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           87        87
=========================================
  Hits            87        87

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


Navigation properties in DbQuery

In model-first DbSets the database structure is generated to fit the navigation properties defined on the entity models.
On the other hand, according to documentation, a DbView model has to match a view that already exists and is not generated by EF.
What I couldn't find in the documentation is how to create navigation properties in a DbView model, along with an example on how to support that navigation when creating the view in the database.
Thanks

Duplicate of #809


Sonobuoy running multiple plugins failing with nil pointer exception

What steps did you take and what happened:
sonobuoy run --plugin  --plugin  is failing with the below error:
PS D:\azure-arc-kubernetes-conformance\src\plugins> sonobuoy run --plugin .\core\conformance.yaml --plugin
.\arc_agent_cleanup\arc_agent_cleanup.yaml
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x38 pc=0x179b1a2]
goroutine 1 [running]:
github.com/vmware-tanzu/sonobuoy/pkg/client.autoAttachResultsDir(0xc00028df10, 0x2, 0x2, 0x1bd7275, 0x15)
/home/runner/work/sonobuoy/sonobuoy/pkg/client/gen.go:295 +0x62
github.com/vmware-tanzu/sonobuoy/pkg/client.(*SonobuoyClient).GenerateManifestAndPlugins(0xc000399830, 0xc00030a180, 0x3, 0x28a1550, 0xc000778160, 0xc00080fd30, 0x5fe8ff, 0x38, 0x2839c70, 0x1bc8a01)
/home/runner/work/sonobuoy/sonobuoy/pkg/client/gen.go:204 +0xcb7
github.com/vmware-tanzu/sonobuoy/pkg/client.(*SonobuoyClient).GenerateManifest(...)
/home/runner/work/sonobuoy/sonobuoy/pkg/client/gen.go:89
github.com/vmware-tanzu/sonobuoy/pkg/client.(*SonobuoyClient).Run(0xc000399830, 0xc00030a180, 0x28a1550, 0x0)
/home/runner/work/sonobuoy/sonobuoy/pkg/client/run.go:212 +0x11e
github.com/vmware-tanzu/sonobuoy/cmd/sonobuoy/app.submitSonobuoyRun.func1(0xc00012f400, 0xc000093480, 0x0, 0x4)
/home/runner/work/sonobuoy/sonobuoy/cmd/sonobuoy/app/run.go:100 +0x24a
github.com/spf13/cobra.(*Command).execute(0xc00012f400, 0xc000093400, 0x4, 0x4, 0xc00012f400, 0xc000093400)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:830 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0xc0000dac80, 0x0, 0x1906640, 0xc00007e058)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914 +0x30b
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:864
main.main()
/home/runner/work/sonobuoy/sonobuoy/main.go:27 +0x2e
What did you expect to happen:
Successfully able to run both plugins
Anything else you would like to add:
Environment:

Sonobuoy version: v0.55.0
Kubernetes version: (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:52:00Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.9", GitCommit:"ed93e7ea8f833824d05578765c2ad3efbebbfebe", GitTreeState:"clean", BuildDate:"2021-10-21T03:31:08Z", GoVersion:"go1.15.14", Compiler:"gc", Platform:"linux/amd64"}
Kubernetes installer & version: AKS
Cloud provider or hardware configuration: Azure
OS (e.g. from /etc/os-release):
Sonobuoy tarball (which contains * below)

Could you provide the yaml files for those plugins? My first repro was just to confirm that the default plugins worked liked this --plugin e2e --plugin systemd-logs and those were fine, then I added a few other plugins I had around and those worked fine as well.
Clearly there is something to fix here, but seeing your plugin that triggered this will help me follow the logic and potentially forsee other issues that may be related.

conformance.yaml:
podSpec:
  containers:
    - name: clusterconnecttest
      image: arck8sconformance.azurecr.io/arck8sconformance/clusterconnect:0.1.5
      imagePullPolicy: Always
  restartPolicy: Never
  serviceAccountName: sonobuoy-serviceaccount
sonobuoy-config:
  driver: Job
  plugin-name: azure-arc-platform
  result-format: junit
spec:
  image: rapatchi/arc-core:v0.1.4
  imagePullPolicy: Always
  name: plugin
  resources: {}
  volumes:
  - name: results
    emptyDir: {}

cleanup.yaml
podSpec:
  containers:
    - name: clusterconnecttest
      image: arck8sconformance.azurecr.io/arck8sconformance/clusterconnect:0.1.5
      imagePullPolicy: Always
  restartPolicy: Never
  serviceAccountName: sonobuoy-serviceaccount
sonobuoy-config:
  driver: Job
  plugin-name: azure-arc-platform
  result-format: junit
spec:
  image: rapatchi/arc-core:v0.1.4
  imagePullPolicy: Always
  name: plugin
  resources: {}
  volumes:
  - name: results
    emptyDir: {}

That one works fine and I can use it 2x (via the @ override) without an issue. So whatever is causing the panic seems to be related to the definition of that other plugin.

@johnSchnake corrected the second plugin

👍 And I repro'd on my side. Perfect, thanks. Should get the patch up today I'm sure.

Makes perfect sense why it bugged out; my fault on the code. Reused an index and didnt realize it. So it was using an index meant for one thing, for another. Since one of those plugins had the sidecar and the other didnt, it ended up leading to the panic. Fix already incoming.

Fix is on main branch now; I think I'll release a point release today or tomorrow with just a few things that have gone in since the v0.55 release, so you'll have an official release you can use with this fix by then.

Thanks @johnSchnake , If possible please trigger the release today. Will try to make progress with this fix tomorrow if you have a choice.


Ensure terminal links are not draggable

Coverage increased (+0.008%) to 67.904% when pulling adde8ab2c3e09a4f317afaae99952e72a197e3ab on Tyriar:non_draggable_links into 6d25706300ae6724bd61491b4de418f6242f28bf on sourcelair:master.

Coverage increased (+0.008%) to 67.904% when pulling da14f64d1b1fcba7b3e706c217694fe1f7c76cb3 on Tyriar:non_draggable_links into ef1b1c8ffc169e5ac70ddf7c3cedb098a26b612f on sourcelair:master.


autoupdate: pre-commit

Automated changes by create-pull-request GitHub action

Codecov Report

Merging #54 (21772b6) into master (d09dda6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #54   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          459       459
=========================================
  Hits           459       459

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Upgrade to Angular 15

Great package! It deserves way more stars.  I am having trouble upgrading to Angular 15 while.  It would be great if it could be upgraded.
Thanks!

It's done !!!!!. Thnks for the feedback


Popups don't respond to content changes

Hello, been using this binding for a few days now and have to say, it's fantastic!!!! I have ~50,000 points/polylines displayed and performance is excellent, especially compared to the alternative. 👍
I have a few basic click-handlers inside popup content which toggle classes (tabs).  I've found changes within popup content do try to trigger a re-render (componentWillReceiveProps is called) but the content is not updated.
Here's the quick fix I made:
componentWillReceiveProps(nextProps) {
  const { div, popup } = this;
  const {
    children,
    coordinates,
    dangerouslySetInnerHTML,
    text
  } = nextProps;

  if (children) {
    render(children, div);
  } else if (
    this.props.dangerouslySetInnerHTML &&
    dangerouslySetInnerHTML !== this.props.dangerouslySetInnerHTML
  ) {
    popup.setHTML(dangerouslySetInnerHTML);
  } else if (text !== this.props.text) {
    popup.setText(text);
  }

  if (this.props.coordinates !== coordinates) {
    popup.setLngLat(coordinates);
  }
}

Hello, thank you, can you open a PR ?

sure, although not sure when as I'm snowed under at the moment. cheers, dave

+1

Fixed and published with a new version 0.9.0.
https://github.com/alex3165/react-mapbox-gl/releases/tag/v0.9.0


 I don't fully understand how it works, so you might be able to see a more elegant way to make this work.
Doing this allows me to put fetchMock.restore in the afterEach function instead of in the then and catch blocks of each test.

Actually this is much less valuable than I thought. The problem comes in when restore/reset is called when it's not mocking, so Arjun pointed me more in this direction and now it's fine:
afterEach(function() {
    if(fetchMock.isMocking){
        fetchMock.restore();
    }
});

Just seen this, and have just done a big refactor so difficult to merge. cheers for the PR though


fix(parser): should be treated comments where after ( as leading comments of next token

same as #6355
The behaviour is also same as esbuild and babel does

#6588  👈
main

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

Join @Dunqing and the rest of your teammates on  Graphite


Error in lift engine

To reproduce, create a project with prisma2 as the dev dep and basic project setup, save and run a migration.
Copy this project, remove node_modules and run yarn, running yarn prisma2 save --name init breaks with ENOENT error now.
[migration-engine] error: Error: spawn /Users/divyendusingh/Documents/prisma/p2-315/node_modules/migration-engine ENOENT
Error: Error: Error in lift engine:
    at ChildProcess.child.on.err (/Users/divyendusingh/Documents/prisma/p2-315/node_modules/.bin/prisma2:283105:24)
    at ChildProcess.emit (events.js:182:13)
    at ChildProcess.EventEmitter.emit (domain.js:442:20)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:235:12)
    at onErrorNT (internal/child_process.js:406:16)
    at process._tickCallback (internal/process/next_tick.js:63:19)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Not reproducible.


UI Design

When I pass nil for image and nil for title, the gap between the top of the alertbox and first line of text is quite large.

My mistake. Didn't pass nil but null string.

Cool, no worries


Wrap ToC items. Fixes #858.

Applies inline-block style in packages/nextra-theme-docs/src/components/toc.tsx to fix uneven item wrapping.

Oh, it looks like these test errors predate my changes. I'm going to back out my changes to the .snap files since I'm probably just exacerbating the problem.


Updates readme

This PR:

Adds functionality to generate CLI reference docs similar to https://github.com/cert-manager/cert-manager/tree/master/tools/cobra
Adds generated reference docs
Updates Readme, to add information on how to use jsctl to generate a 'Venafi TPP profile' Jetstack Secure components installation, removes some bits that I though users could discover from reference docs instead and moves the JSS user creation to the bottom

I've opened https://github.com/jetstack/jsctl/pull/16 from a branch pushed to this repo instead


docs($compile): fix typo

I'm not native speaker, but "like like" construction is weird for me...

I signed it!

yes, this looks like an improvement. will land


feat: Support label lists

We have some situations where we want to run different Charts based on environment, but instead of using .Environment.Name and doing conditional Go templating to that end, we want the ability to leverage labels-as-lists and conditionally run them.
For instance, we have certain charts we want applied to non-prod, prod or both. Using labels like:
releases:
  - name: some-chart
    labels:
      env:
        - npd
  - name: other-chart
    labels:
      env:
        - npd
        - prod

By running helmfile -l env=npd sync we get both charts to sync, running helmfile -l env=prod syncs just other-chart, etc.
Is this something that is inline with the vision of labels or can we accomplish this some other way?

@MLBMatt Hey! Would the env/npd: y style work?
releases:
  - name: some-chart
    labels:
      env/npd: y
  - name: other-chart
    labels:
      env/npd: y
      env/prod: y

helmfile -l env/npd sync  would sync both charts and helmfile -l env/prod sync would sync the other chart only.
Probably this would work too:
releases:
  - name: some-chart
    labels:
      env/npd:
  - name: other-chart
    labels:
      env/npd:
      env/prod:

With e.g. helmflie -l env/npd= sync.

This would work but isn't necessarily as clean as I was thinking

@MLBMatt I hear you! We might need something other than that.
Extending labels to accept multiple values didn't seem very natural where we k8s users usually take it as just a list of kv pairs, where the proposal makes it key-values pairs.
Is the environment name only filter you want? Then how about this
releases:
- name: some-chart
  environmentsInstalled:
  - npd
- name: other-chart
  environmentsInstalled:
    - npd
    - prod

Or a more generic alternative could be:
releases:
- name: some-chart
  installed: false
  environments:
    npd:
      installed: true
- name: other-chart
  installed: false
  environments:
    npd:
      installed: true
    prod:
      installed: true

With YAML anchors we can make it a bit more DRY:
templates:
  base: &base
    installed: false
  npd: &npd
    npd:
      installed: true
  prod: &prod
    prod:
      installed: true

releases:
- <<: *base
  name: some-chart
  environments:
    <<: *npd
- <<: *base
  name: other-chart
  environments:
    <<: *npd
    <<: *prod

The latter alternative would open up possibilities to customize other release settings per environment, like values and kubeContext...
releases:
- name: other-chart
  installed: false
  values:
  - default.values.yaml
  environments:
    npd:
      installed: true
      kubeContext: npdcluster
      values:
      - values.npd.yaml
    prod:
      installed: true
      kubeContext: prodcluster
      values:
      - values.prod.yaml

@MLBMatt I'd appreciate your feedback if you have some time

Hey @mumoshu, sorry getting back to you so late. So we ended up going the more traditional approach using conditionals in the templates and flags in the environments to enable/disable. Your method of using tags per environment did work but we decided to move in the other direction. Thanks for your help and thanks for making such a great utility!


Hello, this is a wonderful work, could the code be released, thank you.

Hi, thank you for your interest in our work. We are busy with other work this month. The code will be released very soon, expected in the first week of April.

@junjiehe96 Can u release the paper please? Or some teaser pngs?

It seems better than sparseinst!
Is this model easy to convert to onnx or other form?
Can't wait to have a try!!

Hi, thank you for your interest in our work. We are busy with other work this month. The code will be released very soon, expected in the first week of April.

Thank you for your work and sharing, I am looking forward to the release code

Hello, everyone. We have released the code. welcome to try it!


High CPU usage?

lack of sleep?

I've fixed the example in https://github.com/Flying-Toast/simple-websockets/commit/e543bcd6d8c00fd08189411e767133e9b8251a98, could you take a look at the change and see if your code is doing the same thing? Please re-open the issue if you're still having issues.

Looks good now, cheers~


Users - Identify the source of users

Hi there,
A (very) small request. What do you think of adding an icon or a text to identify the source of users ?
Something like that

In my case, it may not be very useful because there are few users. But I'm thinking of people who have many sources and many users.

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I have three sources for my users.
1 - Local users
2 - Active Directory users
3 - OpenLDAP users
For 2 and 3, I configured a server in "System: Access: Servers"
On the users screen (System: Access: Users), I think it would be great to add a text or an icon specifying which source the user is from (see my image)
Describe alternatives you've considered
No
Additional context
No


saving and restoring state of terminal

export_view and import_view are very useful for restoring the visual state of the terminal, but what about history and any nested interpreters?  I am using jquery.terminal in a durandal ( knockout.js) single page web app and even though I save a reference to the plugin in my view model, every time I compose the view for the tab, the view is rebound and a new terminal object is created.  I have used:
                term.import_view(self.term.export_view());

In the OnInit function to transfer the visual state from the saved terminal reference to the newly bound one and it looks great.  However, the other internal state obviously is not transferred, so history doesn't work and I am no longer nested inside of interpreters.  Is there any way to save/restore state for the jquery.terminal?  I have also tried term = self.term because I thought term was being passed by reference but history and interpreter stack still doesn't seem to transfer.

In devel branch it now save and restore interpreter stack.

Nice!  I cant wait to try it out.  Incredible plugin 😊
Sent from my Windows Phone

From: Jakub<EMAIL_ADDRESS>Sent: 8/20/2015 9:57 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: Re: [jquery.terminal] saving and restoring state of terminal (#208)
In devel branch it now save and restore interpreter stack.

Reply to this email directly or view it on GitHub:
https://github.com/jcubic/jquery.terminal/issues/208#issuecomment-133075493


Unable to update themes/packages

Description
Unable to look for packages/theme updates

Steps to Reproduce

Open Atom
Go to Settings
Click On Updates

Expected behavior: List Of Updates available
Actual behavior: Unable to get updates
Reproduces how often: Everytime...
Versions
1.17.0
Additional Information
Running on MACOS Sierra - 10.12.4

as a temporary solution you can use "apm upgrade" in terminal.

Do you have seti-syntax or seti-ui installed @prateekkathal?

@rlex I tried, and even that gives me an output of undefined

@rsese Yup, I did... And I removed those themes.... Nothing is helping....
Following are my current themes and packages....
Themes

Packages

@rlex @rsese
So I found the main issue here...

I tried removing it and then downloading the latest version again.
Even after that it wasn't able to fetch the updates
I went to ~/.atom/packages and saw that a folder with the name seti-syntax was already there.... I removed it and now it works 👍

Sadly, I will have to re-install my old packages 😞 , I wish we had a cloud thingy with which can automatically sync our plugins/themes.


Unable to monitor Azure Cloud in paladin cloud

I have enabled to monitor Azure Cloud in settings/local.py and also added respective service principle attributes (clientid, secretid, tenantid) with Reader permissions to the Azure resource group.
I am unable to view the Azure cloud data and either the data is collected from the Azure cloud.
Can someone please help?

And I am using the latest tag v1.2.0 only. Any update on this?

We just released 1.2.0 a while ago. Can you please confirm if you have installed it from the downloaded zip of 1.1.0 or from master?

I did git checkout v1.2.0 and then run the installer.
Do you want me to use v1.1.0?

@santhosh-challa
Tried installing with v1.1.0, no luck. Below are the errors

ok, Thanks for confirming. We have just released 1.2.0 so just want to confirm.
Also, can you please let us know why you checkout to v1.2.0 instead of installing from the latest stable release? This is to understand if there is any gap in the current documentation so that we can improve. Also can you please confirm your latest commit hash so that we can see confirm if that is the latest or not?
To debug the issue, Can you please navigate to AWS Batch in the account where Paladin is installed and follow the below steps:

Navigate the Jobs on the left nav bar
Enable Search And Filtering on the top banner
Search of pacbot-azure-discovery-job
Click on pacbot-azure-discovery-job job
Provide us the cloud watch log file.

I was about to update the ticket. You do not need to install from 1.1.0. I was trying to understand if you have installed from latest tested commit from 1.2.0 or not.
Can you check out to tag 1.2.0 and do
sudo python3 manager.py redeploy
And let us know if it is successful or not.

I have downloaded the v1.2.0 tar.gz and tried installing it. Still the same issue, unable to fetch data from Azure Cloud.
Please see the screenshot below for reference.

@santhosh-challa : Can we have a quick call to discuss? If yes, could you please send me the invite link.

@kkhamruddin-pk I have a call coming up. Can we have it in the next 30 mins? (3.30PM IST)

Sure. That works for me @santhosh-challa. Once you are ready please share me the link to connect

@kkhamruddin-pk can you join this:
To join the video meeting, click this link: https://meet.google.com/zef-itjy-cru
Otherwise, to join by phone, dial +1<PHONE_NUMBER> and enter this PIN: 645 827 652#
To view more phone numbers, click this link: https://tel.meet/zef-itjy-cru?hs=5

Summary of the call:
AWS Batch jobs are stuck at a Runnable state. As non of the jobs were completed, UI is not showing any data.
We tried to dig into the issue for about an hour and could not figure out the actual root cause. We never faced it internally. Compute can spin the EC2 machines but the jobs are not able to proceed to the Running state.
@kkhamruddin-pk This might be helpful to dig further if you have the reproducible state:
https://docs.aws.amazon.com/batch/latest/userguide/troubleshooting.html
Please let us know your findings.

Hi @kkhamruddin-pk ,
Did you try to install Paladin again? Please let us know if you have that issue. We can try debugging further.
Thanks,
Santhosh

Hi @kkhamruddin-pk ,
Just checking if you have tried installing the Paladin again. Please update the issue if you need any help in installing it.
Thanks,
Santhosh

Hi @kkhamruddin-pk , please let us know if you still have issues or you are able to update successfully.

Hi @kkhamruddin-pk , great call today.. as we have successfully deployed the instance with connections to Azure.. please close out this ticket.

Please join our Community Slack channel for our further engagement on this project.

Thank you so much Nithin, Subbu and Steve for taking time and resolving the issues. I really appreciate in helping to resolve all the issues.


Cannot use defineMessages on an object reference

I think this might be similar to #81, but I'm getting compiler errors instead of undefined keys.
Passing a predefined object reference to defineMessages fails, I'm guessing the code isn't following the reference in the Babel AST and instead assuming an object is directly defined:
import { defineMessages } from "react-intl.macro"
const messages = {
  key: { id: "key", defaultMessage: "Default Message" }
}
defineMessages(messages)

this fails with:
TypeError: react-intl.macro: properties.map is not a function Learn more: https://www.npmjs.com/package/react-intl.macro
    at Array.map (<anonymous>)

My use case is I have a shared library where the messages are defined, but the build process isn't working correctly for the sub module (different issues with Rollup). So I'm trying to import the messages in my app and forward them to defineMessages so they get picked up when I build the available translations.

This is unfortunately not possible.
Macros are limited, they can hardly follow reference in same file.
With import this is really mission impossible.


MediaPlayer Vorbis COMException (0xC00DB3B0)

Describe the bug
I am experiencing an issue with the vorbis decoder in the media player. The audio plays for a short time, but after roughly 30 seconds the audio stops and the MediaFailed event (regardless of position, if the audio is played at 00:01:00 for example, the error is raised at 00:01:28~30) is raised with the following error:
args.ExtendedErrorCode = "Connection to app service providing a media extension failed. (0xC00DB3B0)"
args.Error = "Decoding Error"

This issue only occurs with vorbis files and does not happen with MP3 files. Moreover I have tested this in UWP and the error does not occur. Only in WinAppSdk. The same file that gives an error in WinAppSdk also plays fine in the New Media Player, which I assume is made with UWP.
Steps to reproduce the bug

Create a media player with a vorbis file as the source.
In my case:

//audioStreamStream is an arbitrary Stream class.
var randomAccessStream = audioStreamStream.AsRandomAccessStream();
var source = MediaSource.CreateFromStream(randomAccessStream , "audio/ogg");

Play the audio
Observe the audio stopping after roughly 30 seconds and the MediaFailed event being raised

Expected behavior
The audio should play continuously without stopping and the MediaFailed event should not be raised, but the audio stops after roughly 30 seconds and the MediaFailed event is raised.
Screenshots
N/A
NuGet package version
Windows App SDK 1.2.2: 1.2.221209.1
Packaging type
Packaged (MSIX)
Windows version
Insider Build (xxxxx)
IDE
Visual Studio 2022
Additional context
No response

https://github.com/christosk92/VorbisPlayerWinUI3
I have created a sample project with a perfectly fine OGG file from Wikipedia (https://commons.wikimedia.org/wiki/File:Median_test.ogg)
Hit the Play button and you can see the Debugger breaking in the MediaFailed function (Debugger.Break())

Logged tracking bug for media pipeline team: internal bug

Thank you. I am looking forward to the next release as this is breaking for me right now. I switched to Libvlc but am missing out on native platform features now.

FYI: I'm still seeing this error in 1.4.230518007-experimental1

Any progress? It has been SIX months!

@codendone
Apologies for the ping but it’s been quite some time now.
Has there been any updates on this? Will there be a bugfix that’s planned for an upcoming release?
Thanks !

@christosk92 The media pipeline team says this is feature work to support these media extensions outsides of UWP. They agree this would be valuable to implement but don't currently have a timeline to share.

@codendone I have tested this in an Avalonia project (By targeting to net8.0-windows10.0.19041.0), and the MediaPlayer seems to play fine without failing the vorbis file.
So, it does seem like it's an issue that's localized to WinAppSdk, not necessarily the fact that its outside of UWP.
I hope this helps.

Any updates on this? It's almost been a year now.

@christosk92 we are actively working on this, but still no firm timeline to share.

Thanks

Is there some sort of timeline available yet as to when we can expect this to be fixed?
Just tested this in the latest version and I am still experiencing this.

@christosk92 yes, not resolved yet, but we are still actively working on it.

Any updates yet ? It's almost been two years now..


Migration Fail in X64 build

Describe what is not working as expected.
I have a .NET Project targeting .NET 4.7 and Entity Framework 6.2.0. The project works fine as expected in x86 mode but fails with the following exception when we try to execute add migration command in x64 mode.
Exception message:
Stack trace:

System.BadImageFormatException: Could not load file or assembly '*******' or one of its dependencies. An attempt was made to load a program with an incorrect format.
File name: '*******'
   at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.RuntimeAssembly.InternalLoad(String assemblyString, Evidence assemblySecurity, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean forIntrospection)
   at System.Reflection.RuntimeAssembly.InternalLoad(String assemblyString, Evidence assemblySecurity, StackCrawlMark& stackMark, Boolean forIntrospection)
   at System.Reflection.Assembly.Load(String assemblyString)
   at System.Data.Entity.Migrations.Design.ToolingFacade.BaseRunner.LoadAssembly(String name)
   at System.Data.Entity.Migrations.Design.ToolingFacade.BaseRunner.FindConfiguration()
   at System.Data.Entity.Migrations.Design.ToolingFacade.ScaffoldRunner.Run()
   at System.AppDomain.DoCallBack(CrossAppDomainDelegate callBackDelegate)
   at System.AppDomain.DoCallBack(CrossAppDomainDelegate callBackDelegate)
   at System.Data.Entity.Migrations.Design.ToolingFacade.Run(BaseRunner runner)
   at System.Data.Entity.Migrations.Design.ToolingFacade.Scaffold(String migrationName, String language, String rootNamespace, Boolean ignoreChanges)
   at System.Data.Entity.Migrations.AddMigrationCommand.Execute(String name, Boolean force, Boolean ignoreChanges)
   at System.Data.Entity.Migrations.AddMigrationCommand.<>c__DisplayClass2.<.ctor>b__0()
   at System.Data.Entity.Migrations.MigrationsDomainCommand.Execute(Action command)

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

Steps to reproduce
MigTest.zip

Build the attached project in x64 configuration
Open package manager console
Run the command add-migration test
It gives the error “Could not load file or assembly '****' or one of its dependencies. An attempt was made to load a program with an incorrect format.”

Further technical details
EF version: 6.1.2
Operating system: Windows 10
Microsoft Visual Studio Enterprise 2015
Version 14.0.25431.01 Update 3
Microsoft .NET Framework
Version 4.7.03056

Duplicate of https://github.com/aspnet/EntityFramework6/issues/458


Update quickstartquide

Signed-off-by: ranjithwingrider<EMAIL_ADDRESS>

Already changes are reflected


Add digital certificate

Can I add a digital certificate to .PDF?
.pfx .crt .pem

If you are asking whether qpdf can sign a PDF using PKI, the answer, unfortunately, is not at this time. See also #201. This is something I would like to support, and since the original issue, some of the prerequisites have been satisfied (e.g. being able to link with gnutls or openssl), but right now, we are not particularly close to this functionality.
If that's not what you're asking, please clarify. Once I'm sure I'm answering the right question, if that's what you're asking, I'll probably close this in favor of #201 and invite you to subscribe to that issue if you're interested in following the progress. Thanks!

Correct, I need to add digital certificate to PDF file!
I will follow up as a post!


net/frr: Issue #1509. Fixing issue with net/frr BGP diagnostics summary tab

… by frr adding a new line of output to the vtysh command. I reworked the code for processing the 'header' to decrease the likelihood of this happening with future additions in the header. The new code looks for header lines it is interested in, and skips all others until it gets a blank line, at which time it assumes the tabular data is to follow

Merged, thank you! ❤


memory leak in commentsController

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

VS Code Version: 1.84.2
OS Version: Ubuntu 23.04

Steps to Reproduce:

Open an editor
Close the editor
Notice that a CodeEditorWidget was created but not removed

{
  "instanceCountsWithSourceMap": {
    "before": [
      {
        "name": "CodeEditorWidget",
        "count": 1
      }
    ],
    "after": [
      {
        "name": "CodeEditorWidget",
        "count": 2
      }
    ]
  },
  "isLeak": true
}

In Chrome devtools, the retainer chain points to commentsController.ts:

I discussed with @bpasero, and when an editor is closed it is not disposed unless it is the last editor in the group and the group isn't the default group, so in the scenario described in this issue, the CodeEditorWidget is still expected to exist. Here's how to check if the CodeEditorWidget is being leaked:

Open an editor. Check how many CodeEditorWidgets you have.
Open a second editor in a new editor group.
Close the editor in the second group. Check how many CodeEditorWidgets you have. If there is no leak, there should be the same number as in step 1.

I have tested, and I don't see a memory leak from CommentsController.

@alexr00 Thanks a lot for the info, it's really helpful!
 I was so focused looking for something that would explain the two CodeEditorWidgets and didn't really think about anything else.
Your steps make much more sense and with that I don't see a memory leak from CommentsController.
No wonder it took me so long to find the apparent source, when there is none... xD
Thanks again!


Text property moved to wrong location after :substitute followed by :undo

To Reproduce

Run vim --clean -S reproduce.vim

" reproduce.vim
call setline(1, 'The number 123 is smaller than 4567.')
call prop_type_add('number', {'highlight': 'Constant'})
call prop_add(1, 12, {'length': 3, 'type': 'number'})
execute "normal! i\<c-g>u\<esc>"
s/number/foo
undo

Expected behavior
123 should remain highlighted.
Environment
Vim version 8.1.1333
OS: Windows 10
Terminal: GUI

Fixed by https://github.com/vim/vim/commit/338dfdad3844ebb1ce1d56c421d1f698c086eb0c, thanks.


Make Rhai rugged

With user-land code, it can never be guaranteed.  If a user registers a function with Engine and that function panics, then there is nothing Rhai can do about it.
However, right now it is still possible to panic the system with pure text-only Rhai scripts:

division by zero
arithmetic overflows

Not considered:

line number / character position overflow during parsing (with a very large script) - is this necessary?  Both line number and position use usize which is the largest script that a machine's CPU architecture can handle.

Any others?

Most arithmetic overflows/underflows are already avoided by https://github.com/jonathandturner/rhai/pull/105

Should protect against accidental infinite recursion, stack overflow, or infinite loops?

It is also possible to overflow the stack by defining a function with a godzillion number of parameters.
So limiting the number of parameters to a function is necessary.  Say... 64 or 42 (who writes such functions anyway?)

PR https://github.com/jonathandturner/rhai/pull/154 now has the ability to impose limits on script resources during runtime.
However, it still does not protect against panics occurring during parsing...  for example, deliberating nesting too deeply may exhaust the stack, or as the above, things like too many parameters, or too many items in an array literal etc.

Version 0.15.0 now blocks nesting expressions too deeply and/or loading too many modules.

Right now, Rhai should be quite safe to use (and quite difficult to panic).
It traps infinite loops, overflow errors, division by zero, deeply-nested function calls and expressions, pathological scripts, etc.
Can people think of any way to panic the scripting engine?

Add data size protection - https://github.com/jonathandturner/rhai/pull/163
Protect against creating data values that are too large, e.g. very long strings, very large arrays.


SWARM-1526: topology.js should have a mechanism to wait for initial topology

Motivation
When topology() is called, it subscribes to the SSE stream,
which immediately sends back the current topology. There
should be a mechanism to wait for that, otherwise users are
susceptible to race conditions.
Modifications
Added topology.awaitInitialTopology() function which returns
a promise that is resolved when the first event comes from the
stream.
Ideally, the entire topology() function should return a promise
that behaves like this, but that would be a breaking change.
Result
Users are able to avoid race conditions by awaiting initial
topology.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

I was thinking along the lines of
var topo = topology( function(initial) { ... } );

or return a future from topology() constructor
var topo = topology().then( function(initial){...} )

But I'm also not a js expert.  Maybe we should ask @lance

Yea, topology().then(...) is exactly what I was thinking about too, but that's a breaking change. If that's fine, I can change this.

Ah, in that case, the first one, with an optional callback arg to the topology() factory maybe.

Except topology() can already accept an argument, which is an object with options. Obviously, it can accept a 2nd optional argument... and I guess JS world is just used to this... oh well. I'll take a stab at this.

In principle, I'm against adding surface area to an API no matter how small. So my initial reaction is -1 on this. But I do see the issue you are facing could be a real problem. The goal for me would be to minimize any noticeable changes to the external API.
So I understand correctly, the race condition you are trying to avoid is when calling the factory method to obtain the initial topology, the object returned has a topology property which may or may not yet be initialized. Since neither the factory function
Ref:
https://github.com/Ladicek/wildfly-swarm/blob/6c832c58475387c871333e6aa0b58201b6f7a3ad/fractions/topology-webapp/src/main/resources/topology.js#L11
Nor the returned object's topology property wait around for initialization:
Ref:
https://github.com/Ladicek/wildfly-swarm/blob/6c832c58475387c871333e6aa0b58201b6f7a3ad/fractions/topology-webapp/src/main/resources/topology.js#L220-L233
Personally, I think my preferred change would be to just change the factory function to return a Promise that resolves when everything has been fully initialized. But as you said, @Ladicek that would be a breaking change. How are those handled in general with swarm?
Another option, although still a breaking change, would be to just change the topology property to be a function that returns a promise which resolves once the initial topology has been obtained, and after that returns Promise.resolve(_localCachedTopology). Honestly, I kind of like this option a little better than changing the factory function to return a promise because it allows for deprecation.
The existing .topology property accessor can stay the same, but issue a deprecation warning with a link to some description of the race condition, and a new async function, e.g. currentTopology() could be added to takes its place. Eventually .topology property is removed, and API surface area reduced to initial size.
Adding awaitInitialTopology feels a little clunky to me, personally since it requires changes to existing code to avoid the race condition (as would all of the solutions discussed), but it also never goes away, and is, in my opinion less elegant than changing either the factory or the property to behave correctly.

So changing the topology property itself isn't enough, because there are also the getJSON, postJSON and ajax functions that also assume that topology has already been obtained.
Instead of changing this PR, I added a second version here: https://github.com/Ladicek/wildfly-swarm/commits/await-initial-topology-2 After we decide on the approach to use, I will of course amend this PR.
When it comes to breaking changes: I think Swarm generally breaks quite often, but I just don't share that mindset. Maybe that's a mistake on my side :-) If I hear from @bobmcwhirter or @kenfinnigan that breaking change here is fine, I'd gladly change the topology() function to return a promise. It's the cleanest solution I can think of.

I like your second implementation, fwiw.

Thanks @lance. I amended this PR.

IRC vote was "it's okay to break backwards compatibility" if it makes it more correct.  (right @kenfinnigan?)

indeedy

Ah I didn't see that, I closed my IRC client a while ago. I'll take a stab at that tomorrow.

Pushed a third solution, where the topology() function returns a promise of the Topology object, which is resolved when initial topology is obtained.

I'm not sure why yet but looks like https://ci.wildfly-swarm.io/job/wildfly-swarm-docker-test-linux/94/console had a transient failure:

curl -v POST http://neo4j:neo4j@localhost:7474/user/neo4j/password -dpassword=changethis
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* Could not resolve host: POST; Name or service not known

Closing connection 0
curl: (6) Could not resolve host: POST; Name or service not known
About to connect() to localhost port 7474 (#1)
Trying ::1...
Connected to localhost (::1) port 7474 (#1)
Server auth using Basic with user 'neo4j'

POST /user/neo4j/password HTTP/1.1
Authorization: Basic bmVvNGo6bmVvNGo=
User-Agent: curl/7.29.0
Host: localhost:7474
Accept: /
Content-Length: 19
Content-Type: application/x-www-form-urlencoded

} [data not shown]

upload completely sent off: 19 out of 19 bytes
Recv failure: Connection reset by peer

100    19    0     0  100    19      0   2324 --:--:-- --:--:-- --:--:--  2324

Closing connection 1
curl: (56) Recv failure: Connection reset by peer

retest this please

https://ci.wildfly-swarm.io/job/wildfly-swarm-docker-test-linux/95/ passed this time.

I had to push a small change -- the function didn't return the promise, but the "promise controller". Fixed now.

@lance, would appreciate if you can take another quick look at this and give your ok. Thanks

Are we ready to merge this?

Thanks, can we also merge https://github.com/wildfly-swarm/wildfly-swarm-users-guide/pull/33?


Multiple additions and bugfixes

Hi Max,
Hope you're doing well in your new job :)
As I'm writing the Nextflow pipeline that runs fasta_windows, I found several things to change, and I thought I'd edit the software directly rather than having separate scripts.
New metrics computed:

AT skew
CpG proportion
mono-nucleotide + N counts as integers in their own file, to mimic the di-, tri-, and tetra- nucleotide counts and allow any other downstream analyses to run on exact numbers.

Bugfixes:

removed an extra \t in the tsv output

Changes:

Added _freq to the name of the main output file so that all files are named ${output_name}_*_windows.tsv, which is more convenient for pattern matching

Cheers,
Matthieu

Hi Matthieu,
Thank you for this, it looks really good! I'll merge straight away. I guess I can bump the version and upload to conda?
I'm not doing too badly at all thank you, though I'm missing hacking around on these scripts more often. Hope all is well with you, and really cool to see these Nextflow pipelines developing.

Hi Max.
Super, thank ! Yes it'd be great if you can tag the release and update conda as well, and I can use it straight away.
Nice to hear from you. Are you still evangelising Rust ;) ?

I'll do this now, and yes I am indeed evangelising Rust. Keeping myself code-happy :)

Done! Hope there are no bugs, but if there are, we shall investigate.

Super, thank you so much !


Upload page for Download Page Example

Download Example
Is there a way where i can make an upload file page. Im looking to have an upload/download page on my website

Hey @ImTotallyNotMessi.  This is probably not the correct place to ask this kind of question.  You can try posting on the express gitter, or maybe stack overflow.  Good luck!


Better Point type factoring, again

Intended to replace #635 and #690 .
It's factored into 2 commits that could be reviewed separately (the third one is a tiny unrelated fix to byron-proxy).
The first commit introduces the Ouroboros.Network.Point module and uses it throughout ouroboros-network, making it build and tests pass.
The second makes ouroboros-consensus build and tests pass.
This commit retains the mistake pointHash :: Point block -> SlotNo rather than fixing it to be pointHash :: Point block -> WithOrigin SlotNo. The origin point is given SlotNo 0. It's kept this way so that the pull request will remain short and people will be willing to review it.

@avieth shall we wait for @dcoutts to give his blessing?

@avieth @mrBliss I am fine merging without @dcoutts blessing, :sunglasses: ;). I see this PR as an improvement on top of Duncan's #690.

bors r+


:bug: Fix CirclCI badge

Didn't realize that this would also get messed up when I changed the repo name

Codecov Report

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

@@          Coverage Diff          @@
##           master    #15   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines          45     45
=====================================
  Hits           45     45

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 f80caa1...41f353f. Read the comment docs.


Reworked Avro encoding and decoding API

Reworked Avro encoding and decoding API in order to have type safety for required encoding / decoding settings.
The following code would perfectly compile:
val input = openInputStream<T>(serializer) {
    format = AvroFormat.BinaryFormat
}

but at runtime an exception will be thrown because the readerSchema needs to be set for BinaryFormat.
To prevent this wrong API usage, the API has been redesigned so that the compiler can detect these errors. The call would look like this with the new API:
val input = openInputStream<T>(serializer) {
   decodeFormat = AvroDecodeFormat.Binary(mySchema)
}

@sksamuel I changed the API a little bit and removed the deprecated bits. As you designed the API, I would be very happy if you can take a look at the changes.

Hey @sksamuel, do you think you'll have the time to look at this PR? Otherwise, I will just go forward and merge this PR soon.

If you're happy to merge it, then please do.
I'm back from vacation now, and just catching up. I will setup the creds
tomorrow or Tuesday and add github actions to release.
On Sun, 8 Nov 2020 at 22:31, Thorsten Hake<EMAIL_ADDRESS>wrote:

Hey @sksamuel https://github.com/sksamuel, do you think you'll have the
time to look at this PR? Otherwise, I will just go forward and merge this
PR soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avro-kotlin/avro4k/pull/75#issuecomment-723747246,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGUWVNZ3L7HUSK2AMR3SO5WBBANCNFSM4SWQLJDQ
.


ref: migrate announcement_bar to ts

Summary
migrate announcement_bar to typescript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/16742
Release Note
NONE

Hello @majidsajadi,
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.

/update-branch

Updating the branch to fix the tests. There were some issues with CI that were fixed upstream.


Page ordering/sorting fix, fix to use unique temp dir if userinput not provided also with better cleanup, added async shutil (aioshutil)

resolves #33
resolves #25
resolves #26 (Not required now with this PR)

Changes

update missing minor documentation changes for closed Issue #31. Replaces zerox ouput for python sdk with updated output containing token counts.
Fixed documentation for python sdk - import "from zerox from zerox" instead of "from py_zerox import zerox"
Fixing page order issue by adding alphanumeric human like sorting utility (doesn't require padding). Fixes Issue #25
Better way of handling tmp_dir - user input or unique named temperory directory if None. Incorporates #26 (So this PR is no longer required)

Note: PR #26 is no longer required as this PR incorporates its ideas plus other minor fixes also all together.

@xdotli, @annapo23 Could you review this PR.

@tylermaran, all review threads incorporated. Tested imports locally also. Good to merge now!

@tylermaran, I can't see the newer package on pypi, do you manually need to publish or something is broken?


appId option is required errors showing in the log

After upgrading from 2.10.0 to 2.13.0 we see a lot of error messages "appId option is required"  in the log.
{
    "name": "cla.domain.name",
    "hostname": "ip-10-0-34-139.ec2.internal",
    "pid": 10138,
    "level": 50,
    "err": {
        "message": "[@octokit/auth-app] appId option is required",
        "name": "Error",
        "stack": "Error: [@octokit/auth-app] appId option is required\n    at createAppAuth (/home/ec2-user/cla-assistant/node_modules/@octokit/auth-app/dist-node/index.js:492:11)\n    at new Octokit (/home/ec2-user/cla-assistant/node_modules/@octokit/core/dist-node/index.js:113:20)\n    at new _a (/home/ec2-user/cla-assistant/node_modules/@octokit/core/dist-node/index.js:167:30)\n    at new OctokitWithDefaults (/home/ec2-user/cla-assistant/node_modules/@octokit/core/dist-node/index.js:147:9)\n    at new _a (/home/ec2-user/cla-assistant/node_modules/@octokit/core/dist-node/index.js:167:30)\n    at new OctokitWithDefaults (/home/ec2-user/cla-assistant/node_modules/@octokit/core/dist-node/index.js:147:9)\n    at getInstallationAccessToken (/home/ec2-user/cla-assistant/src/server/src/services/github.js:87:24)\n    at Object.callWithGitHubApp (/home/ec2-user/cla-assistant/src/server/src/services/github.js:137:33)\n    at createStatus (/home/ec2-user/cla-assistant/src/server/src/services/status.js:68:23)\n    at updateStatus (/home/ec2-user/cla-assistant/src/server/src/services/status.js:142:16)"
    },
    "msg": "[@octokit/auth-app] appId option is required",
    "time": "2023-03-08T17:39:05.177Z",
    "src": {
        "file": "/home/ec2-user/cla-assistant/src/server/src/services/github.js",
        "line": 141,
        "func": "callWithGitHubApp"
    },
    "v": 0,
    "req_id": "1e058fe0-bdd8-11ed-934e-7fc39d6f16b8"
}

I see the appId is only referenced in the config.js and takes env GITHUB_APP_ID as a value.
Still, the environment variable is neither mentioned in the README.md nor listed in .env.example.
$ git grep appId
src/server/src/config.js:                appId: process.env.GITHUB_APP_ID,

$ git grep GITHUB_APP_ID
src/server/src/config.js:                appId: process.env.GITHUB_APP_ID,

Is the GITHUB_APP_ID mandatory now? If not, could you skip the problematic code where it's used?

Not only are they showing in the logs, since Slack integration is implemented as a logger, they're also getting posted to the configured Slack channel.
Looks to be caused by commit
0529ac3d42bd001917e701b479578418eb1c5caf
Add GitHub App as a hidden feature (#781)
This commit seems to add support for regular GutHub apps but breaks OAuth apps in the process.


HIP12: Remote Location Assert

Submitted by: @rawrmaan
Rendered HIP markdown: https://github.com/helium/HIP/blob/master/0012-remote-location-assert.md
Original PR: https://github.com/helium/HIP/pull/30
Description from original PR:
Since the GPS location assertion check will soon be enforced, more than 1500 hotspots will need to be adjusted in some way in order to have a good GPS fix and continue participating in Proof of Coverage (see current hotspot gps_fix_quality states below). For many hotspots, this will require updating the asserted location to accurately reflect the hotspot's latest location, which can currently only be done with physical access to the hotspot.
gps_fix_quality | count
----------------+-------
good_fix        |  2275
no_fix          |  1090
bad_assert      |   514

This HIP proposes a new transaction, assert_location_v2, which will allow a hotspot's location to be asserted remotely without physical access to the hotspot.

Does this issue as it's currently written still exist or is it still an issue?
I think the overall process associated with geolocation for the network needs work still.

@anthonyra yes, being able to re-assert a Hotspot's location remotely is useful whether or not GPS fix is required

I would 100% agree that this step in the process needs a little buff and also allowing 'remote' asserting could be very useful.
I haven't typed up a formal HIP for this but considering it lands within the realms of this one maybe it'll be worthy to merge.
From my understanding, the current system is a hinder to a patron or just individuals trying to acquire hosts. These owners have to set up the assert_location with their account so that the hotspot is linked to their wallet. Please correct me if I'm wrong I have the one hotspot that I had set up over a year ago. I have, however, moved that hotspot from VA to NY.
Concerns:

If you can remote assert it this would allow farms to be moved rather easily.
The current version of assert_location already allows for individuals to pin a hotspot in a location that isn't the current location. (This would allow someone to set up hotspots in a building but make the network think those hotspots are placed perfectly around the city)

My proposal:
Split ownership and asset_location into two steps. Allowing the owner of the hotspot to claim ownership over the hotspot. They then can continue the process if they like or select the option that the hotspot will have a host. Once the owner finds a host they ship the hotspot to them. The host creates a helium account/wallet and completes the setup for the hotspot with their device.
Detailed Proposal:

Owner receives hotspot. They would then use the assign_ownership transaction to link the hotspot to their account.
They would ship the hotspot to the host.
The host would then use the helium app to create a wallet and finish the setup process. They would submit an assert_location. This assert_location would have a confidence rating based on the phones geolocation and the hotspot pinned location.
If the hotspot has a GPS then that confidence could be boosted when the hotspot GPS validates the initial phone assert_location. This would be optional but a step that would increase the confidence of that geolocation.

5)This would be the final validation and would require a decent amount of setup on Helium's side. Helium would create a location validation tool that would simply be a modified Tab. Then a new owner, or an owner looking to boost its hotspots confidence, would be shipped with one of these validators. This device would be online during the initial shipment and during return shipment. This device would be optional but would significantly increase the confidence of the hotspot's location. This step alone has a lot to it and will require me to write a really in-depth explanation of how this would be huge for the network.

The host would then use the helium app to create a wallet and finish the setup process. They would submit an assert_location. This assert_location would have a confidence rating based on the phones geolocation and the hotspot pinned location.

^This assumes all distributors of hotspots are permitting or instructing their hosts to interact directly with the helium app, which may not necessarily be true.
I do see value in remote location assert of a gateway. The ability to assert location via inputting address instead of dropping a pin would also be of value.

The host would then use the helium app to create a wallet and finish the setup process. They would submit an assert_location. This assert_location would have a confidence rating based on the phones geolocation and the hotspot pinned location.

^This assumes all distributors of hotspots are permitting or instructing their hosts to interact directly with the helium app, which may not necessarily be true.
I do see value in remote location assert of a gateway. The ability to assert location via inputting address instead of dropping a pin would also be of value.

I do have that listed as a current limitation. However, this could also be done with a website (web app). Since Helium is already using React for the website and App it would be fairly easy to do also.
Do you think if the host receives the hotspot with a qr code that they wouldn't be able to use a website to finish the setup?

Some gateway distribution groups may not want hosts interacting with helium's app, website, etc., for a number of reasons.
If your goal is WIFI connectivity, this will be possible with future DIY gateways via wifi backhauling, where the host-user can interact with the gateway directly to connect to their wifi network. WIFI connectivity will not be exclusive to using helium's app, as it currently is with helium manufactured hotspots.

I believe managing multiple hotspots will be made much easier with:

remote assert
ability to assert via address (and fine tune by dragging the pin)

Recent updates have been made to diminish gaming of the system by hotspots using a false location, and as such, I see no reason why hosts should be required to add an additional layer of location verification. The network should sort this out, and make this easy. Not harder.
Adding any additional layers of friction will not help this network grow.

I generally support the need for remote assertion. I wonder if this could be achieved without modifying the current assert_location_v1 transaction and instead enabling the mobile app / gateway-config bluetooth tools to allow it.
As background, an assert location transactions works as follows:

The miner (which, in the case of official hotspots, is onboard the hotspot) generates an assert_location transaction signed with the miner's key. The mobile app connects to the hotspot via bluetooth to initiate this.
The transaction is signed by the account key (wallet) and submitted to the blockchain.

Today, this is done as part of a single workflow in the mobile app. I suspect it could be split into a two step process. I believe the mobile app today requires that the owner be signed into to the app (via their wallet seed words) in order to connect to the hotspot as a matter of policy / security not a technical reason. If this is changed, it is feasible that the following process is used instead:
A. Non-owner uses a mobile app (maybe the Helium app or a dedicate/slimmed down app) to connect to hotspot over bluetooth and initiate a assert_location transactions.
B. non-owner send the transaction to the owner
C. Owner uses their wallet app to sign and submit the transaction.
The benefit of such an approach maintains the existing assert_location_v1 transaction, limiting the impact to the blockchain. Additionally, to the extent it matters, it keeps both the host and the owner involved in the transaction. The proposed assert_location_v2 transaction removes the hotspot host from the workflow.
The drawbacks of this is the need to allow non-owners to access the hotspot via bluetooth. This may potentially diminish security. However, it could be made to allow only the assert_location transaction for non-owners to limit the impact.

Rough consensus among the community was discussed and memorialized at the September 2020 community call: https://docs.google.com/document/d/1bMm2alBigBj3detA775Dn0Gz9UM5XczAeK9vnjBB3l0/edit#bookmark=id.sdsw04n7xjcg


adjusted nav padding

👷 Deploy Preview for cg-eleventy-blog processing.

Name
Link

🔨 Latest commit
d681654b30bb0627e032dd43ef8ea0af5b4a0957

🔍 Latest deploy log
https://app.netlify.com/sites/cg-eleventy-blog/deploys/65c83680150d100008401ba9


Document.reorder: invalidate OcrdMets cache before save

As discussed in OCR-D/core#957, the reorder method of Document changes the XML structure without OcrdMets being aware of it. We introduced optional caching of METS data in v2.42.0 and when enabled this led to inconsistency because OcrdMets.physical_pages did not reflect the changes because caching hid them.
@mehmedGIT therefore introduced a method OcrdMets.refresh_caches to refresh the cache from the XML.
This PR adds the refresh_caches call just before self.save_mets() which fixes the problem.
Also additonally run all tests with caching enabled to make sure we notice caching issues in the future.

@kba I'm really sorry that took so long, I had dayjob stuff to do and a broken ocrd/python/os-environment.

@kba I'm really sorry that took so long, I had dayjob stuff to do and a broken ocrd/python/os-environment.

Not a problem, thanks for continuing to work on browse-ocrd, it has become my preferred tool to inspect/present OCR-D results and we're very happy to have it :-)


missing a recently opened tab

count of the open tabs is properly shown in the icon, but in the listing only n-1 tabs are listed.

That is by design since you will never be switching to your current tab.  Also by excluding the current tab you can easily switch to the previous tab by launching the extension (using your keyboard shortcut) and hitting enter.


Cannot start docker, ngrok throws error

Running make start throws this error:
 => ERROR [ngrok 4/6] RUN ngrok --version                                                                                0.6s
------
 > [ngrok 4/6] RUN ngrok --version:
0.342 /bin/ngrok: line 1: syntax error: unexpected ";"
------
failed to solve: process "/bin/sh -c ngrok --version" did not complete successfully: exit code: 2
make: *** [Makefile:34: start] Error 17

Steps to replicate:

Clone repo
Add .env values
Add ngrok token
make start

@reilg thanks for the report! I haven't been able to reproduce this and am pretty stumped. It almost looks like there is something wrong/corrupted with ngrok on your machine, since it seems like ngrok itself is unable to run. A couple of thoughts:

Can you make sure you have recent versions of ngrok and Docker installed?
Does ngrok --version work when you run it directly from the terminal?
If you go into the "containers" tab of Docker desktop and click on the container and then click on the logs tab, do you see anything more useful / detailed in the logs?

Closing this due to inability to repro, feel free to comment / re-open if the suggestions above do not solve the issue.


Unable to parse reddit atom feed

Try to parse
http://www.reddit.com/.rss
rss.Read returns empty struct with nil error.
Can this library parse atom feeds?
How to reproduce:
v , err:= rss.Read("http://www.reddit.com/.rss")

Environment:
go version go1.12.8 linux/amd64

https://www.reddit.com/r/redditdev/comments/5w60r1/error_429_too_many_requests_i_havent_made_many/de7ngx6/
Adding --user-agent "curl:myappname:v0.0 (by /u/ocelost)" to the curl request ensures that the reddit returns content. I will implement it.

@SuddenGunter Could you test?

@030 tried using cod based on this https://github.com/ungerik/go-rss/blob/2f6706b577373dacdb1d6819334e9c8d89932ac4/example/main.go -  works good, thank you.
Although one thing I've noticed: not sure reddit is the correct name for variable at Read function - well, it's very specific imho. Mb should call it something like sendUserAgent


Update service domain for local_file from 'camera' to 'local_file'

Breaking Change:
This change breaks existing service call references to the camera.local_file_update_file_path by changing the service call to local_file.update_file_path. My understanding is that because this is not a service provided by the base camera component, it should live in the domain of the local_file component instead. If that's the case, then it also makes sense to update the service name.
Description:
Update the domain and service name for camera.local_file_update_file_path. I started down this path because of #27289 and based on the introduction here and comment here, I felt this change made sense. If I am misunderstanding the intent please let me know so that I don't keep moving through these changes, as I have noticed this pattern emerging across a bunch of integrations as I commented here. Unfortunately I don't have a device to test most of these integrations.
Related issue (if applicable): Related to #27289
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#11234
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Check the services section in the UI and see if the drop down of entities filters by cameras. Changing the domain may get rid of that user friendly option

@dshokouhi did not think about that. The end result is that there is no filter applied to the list:

While I agree that the camera entity filtered list is better than the unfiltered list I get when I change the domain of the service, it can still lead to bad service calls since it is not guaranteed that all camera entities are using the local_file platform. I feel like directionally moving this service to the local_file domain would lead to a better approach because the entity discovery feature could be updated to filter entities where entity platform = service domain. Where I see this being a potential challenge is when an integration supports multiple device types. In order for the entity discovery feature to work properly, it implies that an optional device_type property should exist for service definitions in services.yaml, which could then be used to find the right device_type specific entity for entity discovery.
Thoughts? I realize this is well out of scope of the PR, and I am open to closing this PR and opening a new discussion in the architecture repo or somewhere else more appropriate.

This service should be moved. Flaws in the UI should not guide our backend decisions 👍


HBASE-27821 Split TestFuzzyRowFilterEndToEnd

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.


Index p_index out of size and connect: Condition ' s->slot_map.has(target) ' is true

Fix for issue https://github.com/godotengine/godot/issues/12189 on the 2.1 branch.

uh... I have no idea what this huge list of stuff is.

Well: https://github.com/rcurtis/godot/tree/2.1_issue12189

This branch is 1263 commits ahead, 5018 commits behind godotengine:master.

You should make your pull request against the 2.1 branch, not master.


Tests for Conditionals and Unquoted Strings

Changes

Adds two tests for conditional blocks covering Cicero issues: https://github.com/accordproject/cicero/issues/554 and https://github.com/accordproject/cicero/issues/555
Strings and Relationship variables are now unquoted in Slate: https://github.com/accordproject/web-components/issues/136

Flags

Needs to be tested live in web-components
@Michael-Grover

Coverage increased (+0.03%) to 89.939% when pulling a4968e9fa408a19600f107bc85e3985f4017d7d1 on parserv2 into 6266b41acb34bbccb4c8b1860dd85d64890a1556 on master.


TypeError: string indices must be integers when iterating over models from list_models

Describe the bug
Hi, It seems it is an internal error. When I iterate over the whole list (now retrieved as Python Iterators), it fails at some point and throws a TypeError exception.
Also, it outputs Invalid model-index. Not loading eval results into CardData. for some models, which it isn't an exception I guess. I've seen this issue from nateraw/modelcards archived repo. What is the meaning of this warning? It appeared 978 times before crashing (978 from 358114 iterated models, value from i in the reproducible code).
Reproduction
from huggingface_hub import HfApi
api = HfApi()
models = api.list_models(full=True, cardData=True, fetch_config=True, sort="lastModified", direction=-1)
i = 0
for d in models:
    i +=1
print(i)

This code is just to iterate over all models.
Logs
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/test_api/testAPI.ipynb Cell 16 line 2
      1 i = 0
----> 2 for d in models:
      3     i +=1
      4 print(i)

File ~/anaconda3/envs/hf/lib/python3.9/site-packages/huggingface_hub/hf_api.py:1328, in HfApi.list_models(self, filter, author, search, emissions_thresholds, sort, direction, limit, full, cardData, fetch_config, token)
   1326 if "siblings" not in item:
   1327     item["siblings"] = None
-> 1328 model_info = ModelInfo(**item)
   1329 if emissions_thresholds is None or _is_emission_within_treshold(model_info, *emissions_thresholds):
   1330     yield model_info

File ~/anaconda3/envs/hf/lib/python3.9/site-packages/huggingface_hub/hf_api.py:527, in ModelInfo.__init__(self, **kwargs)
    524 self.mask_token = kwargs.pop("mask_token", None)
    525 card_data = kwargs.pop("cardData", None) or kwargs.pop("card_data", None)
    526 self.card_data = (
--> 527     ModelCardData(**card_data, ignore_metadata_errors=True) if isinstance(card_data, dict) else card_data
    528 )
    530 self.widget_data = kwargs.pop("widget_data", None)
    531 self.model_index = kwargs.pop("model-index", None) or kwargs.pop("model_index", None)

File ~/anaconda3/envs/hf/lib/python3.9/site-packages/huggingface_hub/repocard_data.py:305, in ModelCardData.__init__(self, language, license, library_name, tags, datasets, metrics, eval_results, model_name, ignore_metadata_errors, **kwargs)
...
--> 552     name = elem["name"]
    553     results = elem["results"]
    554     for result in results:

TypeError: string indices must be integers

System info
- huggingface_hub version: 0.19.0
- Platform: Linux-6.2.16-4-pve-x86_64-with-glibc2.28
- Python version: 3.9.12
- Running in iPython ?: Yes
- iPython shell: ZMQInteractiveShell
- Running in notebook ?: Yes
- Running in Google Colab ?: No
- Token path ?: /root/.cache/huggingface/token
- Has saved token ?: False
- Configured git credential helpers:
- FastAI: N/A
- Tensorflow: N/A
- Torch: N/A
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: N/A
- hf_transfer: N/A
- gradio: N/A
- tensorboard: N/A
- numpy: N/A
- pydantic: N/A
- aiohttp: N/A
- ENDPOINT: https://huggingface.co
- HF_HUB_CACHE: /root/.cache/huggingface/hub
- HF_ASSETS_CACHE: /root/.cache/huggingface/assets
- HF_TOKEN_PATH: /root/.cache/huggingface/token
- HF_HUB_OFFLINE: False
- HF_HUB_DISABLE_TELEMETRY: False
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_EXPERIMENTAL_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False
- HF_HUB_ETAG_TIMEOUT: 10
- HF_HUB_DOWNLOAD_TIMEOUT: 10

I checked some things. It doesn't crash on a specific repo. When sorting (sort="lastModified", direction=-1) it crashes with pidanr/pegasus-bbcnews repo, and when not sorting with Waynehillsdev/Wayne_NLP_mT5.
I guess it has to do with the evaluation results values, but I'm not seeing the failure point.

PR is merged. I shipped a hot-fix release (v0.19.1) with the fix: https://github.com/huggingface/huggingface_hub/releases/tag/v0.19.1
Thanks again for the report!


The home page has a button to customize the game before creating a new room.

Completed with commit 0b51e4b8ae93c40a9d60e88536f04210f531cb1f


CMake build error

So I downloaded the code and did cmake -Bbuild on the root directory
but then I got the following error:
CMake Error at CMakeLists.txt:2 (project):
  Running

   'nmake' '-?'

  failed with:

   no such file or directory

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!

What might be the reason that this happens?

Of course! I am on Windows 11.

Okay, I'm not very familiar with Windows CMake, because it's less universal, as opposed to systems like Meson. Do you think you could try doing it the Meson way and then tell me if it works?
Thank you and sorry about this. I wrote the program on Linux.

@turrnut, do you think you could try it with Meson?

@aarikpokras Meson worked. Thanks!

Of course! I'm glad it worked.

@turrnut I believe this is because the nmake command is not installed on your computer (I guess for some reason it's not installed natively on any windows machine 🤷). If you have Visual Studio Code, you can install the CMake tools extension as nmake comes with this. Otherwise, I believe you might be able to use WinGet.

@aarikpokras I installed CMake tools extension in VSCode, but I do not have nmake.
also, it seems like I don't have winget on my system for some reason. Is there another way to install nmake?

@turrnut, you could also try to look for it in the native app installer. I would recommend looking for nmake itself or CMake.

@turrnut, sorry about this. It seems that in VSC, you can install C++ Build tools and nmake should be included with this.

@aarikpokras I have the C++ Build Tools installed but it doesn't seem to work.

@turrnut, here's an alternative using a different makefile generator. Please tell me if this works.
cmake -G "MinGW Makefiles"

@aarikpokras Neither command worked and it seems like there are some missing packages.

@turrnut, sorry. I guess CMake is just not very good. If you're okay with it, I'd recommend just using Meson.

@aarikpokras Yea, Meson works fine with me.


[Feature request] send from image browser to outpaint

Hi, awesome extension :), is there any chance to add button in the image browser – that we can send directly to outpaint

Image browser... You say the extension?

Image browser... You mean the extension?

yes :)

@zero01101 do you know if there is any inter-extension support on the webui? If there is nothing we will probably have to JavaScript our way into the extension...

:/ can't say that i do unfortunately; from what i've poked around with in webUI's codebase i didn't see anything that'd accommodate that (certainly doesn't mean it's not there however) nor did it seem like any of the extensions i have installed do anything like that... wonder how that'd work via JS though, like what happens if openOutpaint is initialized before the image browser?

We can deal with that in JavaScript, no issue. But I think it would be hard to get generation info in that case. Sending the image would be quite okay I imagine.

For now, we have a send image from image browser to openoutpaint button. Still, generation configuration is not an option for now.
@Rogal80 can you give this a look? You need to enter the extension folder (/extensions/openOutpaint-webUI-extension) and run git checkout testing and git pull.
Then you go to the webui extensions tab and click on 'Apply and restart UI'.

@Rogal80 This is now in main, so just need to update the extension and it should work.

thx :)


1and1.com hosted CEW. Notify their fraud dept. M-F 9am-5p 1-877-206-4253

https://github.com/ethereumproject/etherwallet/issues/62


DDP set device id

Hi, I am using LAVIS to train albef model recently, while I find a potential issue.
At here, we are applying torch DDP to the original model that requires device id, i.e., the rank of current process.
However, I think it is not the responsibility of config file to set rank for each process and it results in failure. I suggest using get_rank instead.

Well , I find that the problem is init_distributed_mode is not called in current trainer.py. Then setting gpu in config is good.


Integration Test - Release - 2022/01/07 08:40:47

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign NonAllowedUser

#assign PRMergerTest1

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

#please-close

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser