Add client and server support for apns-id headers

As requested in #559 and initially implemented by @aleksandr-lapushkin in #563, this adds support for the apns-id header in notifications sent to the server and in responses received from the server.
Having implemented this fully and benchmarked it, I recommend against moving forward because, in my assessment, the costs outweigh the benefits. As I understand the argument put forth in #559, the main benefit of exposing the APNs ID is that, in cases where the APNs server accepts a notification but the destination device does not report receiving the notification, callers could cite the APNs ID when contacting Apple to report the problem. I believe this is a flimsy argument because APNs itself is a best-effort protocol that does not guarantee delivery even if the APNs server accepts a notification. As such, I believe the probability of Apple investigating and providing technical support for individual delivery attempts is identically zero.
As for the costs, I believe there are two major areas of concern:

Parsing UUIDs introduces a new Thing That Can Go Wrong™; it's possible that the UUID sent by the client could be malformed (or the server's expectation about format might change in the future), although I consider both of those cases very unlikely. The server could also send crazy data back (also unlikely) for its apns-id header, and we need to be prepared to handle UUID parsing failures in the client. In theory, the client could just report the apns-id from the server as a String to avoid parsing time and potential errors, and that may be worthy of consideration.
The much larger cost is, simply, performance (as @aleksandr-lapushkin predicted in #563). Adding proper handling significantly reduces throughput in benchmarks. Benchmark results are shown below ("scores" are the time taken to send 10,000 notifications, and so lower scores are better):

Ignoring apns-id (i.e. the current behavior in master):

Concurrent connections
Score
Error
Units

1
0.317
± 0.014
s/op

4
0.215
± 0.010
s/op

8
0.208
± 0.008
s/op

With apns-id handling:

Concurrent connections
Score
Error
Units

1
0.422
± 0.021
s/op

4
0.249
± 0.017
s/op

8
0.226
± 0.007
s/op

UUID parsing in Java is definitely slower than it needs to be, and so it's possible that we could use a more optimized implementation in Pushy. As mentioned before, we could also skip parsing for UUIDs included in responses from the server. Still, because I'm not persuaded by the existing arguments for including this feature at all, I'm not really inclined to go too crazy trying to make this happen.
I want to be very clear that I'm open to arguments in favor of this feature even though I haven't found the arguments so far persuasive. I'd like to leave this open for discussion, but would like to bring the discussion to a close within a couple weeks. Given a complete picture of the implementation and its implications, what do you all think?
This closes #559.

I started to suspect that the thing that was actually getting measured by the benchmarks was mock server performance and not client performance. I created an entirely separate benchmark server in #585, and that does indeed reveal that the performance hit for the apns-id header is smaller than I initially believed. From there, I switched from the JVM-default UUID#fromString(String) and UUID#toString() to using fast-uuid, and now the performance difference is basically undetectable.
I'm pretty comfortable moving forward with this if everybody agrees the implementation is reasonable.

Having skimmed the changes, everything seems reasonable. At the very least on the API level.

I am very happy with everything that allows to fetch the apns-id out of the response from Apple actually, so I'm all for it! thanks a ton!

Is APNs using apns-id to identify the push notification ?

Is APNs server using apns-id to identify the push notification ?

Yes.

if client do not set the apns-id, how does APNs server check whether it is the same request and not generate a new apns-id for the push notification ?

The APNs server does not perform any checks for duplicates.

In order to avoid sending push notification duplicately, Is it necessary to set the apns-id in the client side ?

No. Again, this is not functionality provided by the APNs server.


Cleanup error after upgrading from 2.0.0-alpha.3 to 2.0.0-alpha.4

ember build is giving below errors and gets stuck indefinitely.
version: 0.1.12
Cleanup error.
Object #<Babel> has no method 'cleanup'
TypeError: Object #<Babel> has no method 'cleanup'
    at cleanupTree (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/broccoli/lib/builder.js:128:19)
    at /Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/broccoli/node_modules/promise-map-series/index.js:11:14
    at $$$internal$$tryCatch (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:490:16)
    at $$$internal$$invokeCallback (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:502:17)
    at $$$internal$$publish (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:473:11)
    at Object.$$rsvp$asap$$flush [as _onImmediate] (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1581:9)
    at processImmediate [as _immediateCallback] (timers.js:354:15)
Build failed.
Invalid tree found. You must supply a path or an object with a `read` function.
Error: Invalid tree found. You must supply a path or an object with a `read` function.
    at readAndReturnNodeFor (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/broccoli/lib/builder.js:71:13)
    at RSVP.resolve.then.then.finally.readTreeRunning (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/broccoli/lib/builder.js:92:24)
    at $$$internal$$tryCatch (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:490:16)
    at $$$internal$$invokeCallback (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:502:17)
    at /Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1096:13
    at Object.$$rsvp$asap$$flush [as _onImmediate] (/Users/dilip/Documents/Code/AdminConsole-Org/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1581:9)
    at processImmediate [as _immediateCallback] (timers.js:354:15)

Output of ember -v
version: 0.1.12
node: 0.10.37
npm: 2.1.8

That is strange, as there aren't any build related changes from what I can see between alpha 3 to alpha 4: https://github.com/dockyard/ember-validations/compare/v2.0.0-alpha.3...master

@kushdilip Hey, had this exact same issue yesterday. was solved by updating Ember CLI. I believe one of its dependencies (RSVP maybe) got upgraded to only cooperate with a recent CLI.
I was on 0.1.8, and was solved by upgrading to 0.2.2 at least.
Hope this helps


Error inside an error

#foobar open (when foobar is undefined) results in the following:
Sorry: definition for foobar not found in the enclosing block(s).
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ #foobar open hydrate | ]
  │
  ├ due to 'Unhandled exception:  (Novika::Died)
  from src/novika/forms/form.cr:23:7 in 'die'
  from src/novika/forms/block.cr:259:20 in 'top'
  from src/novika/errors.cr:74:22 in 'report'
  from src/novika/errors.cr:16:7 in 'report'
  from src/novika.cr:196:5 in 'frontend'
  from src/novika.cr:151:3 in 'frontend'
  from src/novika.cr:200:1 in '__crystal_main'
  ...

#foobar open in the REPL results in the following:
Sorry: definition for foobar not found in the enclosing block(s).
  │
  ├ due to 'startSession', which was opened here:
  │
  │  [ this child $: […] @: '…' echo startSession | ]
  │
  ├ due to 'loop:', which was opened here:
  │
  │  [ false $: […] $: […] @: loop: | ]
  │
  ├ due to 'loop', which was opened here:
  │
  │  [ ahead eject loop | ]
  │
  ├ due to 'startLoop', which was opened here:
  │
  │  [ @: iterBody #break breakLoop opens iterBody #next nextLoop opens startLoop | ]
  │
  ├ due to 'hydrate!', which was opened here:
  │
  │  [ this =: orphan loopBody hydrate! | ]
  │
  ├ due to 'gulp', which was opened here:
  │
  │  [ dup 0 |to newContinuation conts gulp | ]
  │
  ├ due to 'shove', which was opened here:
  │
  │  [ swap shove | ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ orphan iterBody hydrate | repeat ]
  │
  ├ due to 'br:', which was opened here:
  │
  │  [ '…' readLine br: | ]
  │
  ├ due to 'br', which was opened here:
  │
  │  [ ahead eject ahead eject br | ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ sel open hydrate | ]
  │
  ├ due to 'runLine', which was opened here:
  │
  │  [ runLine | ]
  │
  ├ due to 'do', which was opened here:
  │
  │  [ 0 $: pgRoot new $: pgStack shallowCopy $: […] do | error br: […] […] pgStack echo ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ […] swap hydrate | ]
  │
  ├ due to 'measure', which was opened here:
  │
  │  [ […] @: […] measure | =: timeTakenMs ]
  │
  ├ due to 'do', which was opened here:
  │
  │  [ monotonic $: do | monotonic $: t2 t2 t1 - ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ […] swap hydrate | ]
  │
  ├ due to 'hydrate!', which was opened here:
  │
  │  [ pgStackCopy pgRootInstance line slurp hydrate! | ]
  │
  ├ due to 'gulp', which was opened here:
  │
  │  [ dup 0 |to newContinuation conts gulp | ]
  │
  ├ due to 'shove', which was opened here:
  │
  │  [ swap shove | ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ #foobar open hydrate | ]
  │
  ├ due to 'Sorry: no top for block.
  │
  ├ due to 'startSession', which was opened here:
  │
  │  [ this child $: […] @: '…' echo startSession | ]
  │
  ├ due to 'loop:', which was opened here:
  │
  │  [ false $: […] $: […] @: loop: | ]
  │
  ├ due to 'loop', which was opened here:
  │
  │  [ ahead eject loop | ]
  │
  ├ due to 'startLoop', which was opened here:
  │
  │  [ @: iterBody #break breakLoop opens iterBody #next nextLoop opens startLoop | ]
  │
  ├ due to 'hydrate!', which was opened here:
  │
  │  [ this =: orphan loopBody hydrate! | ]
  │
  ├ due to 'gulp', which was opened here:
  │
  │  [ dup 0 |to newContinuation conts gulp | ]
  │
  ├ due to 'shove', which was opened here:
  │
  │  [ swap shove | ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ orphan iterBody hydrate | repeat ]
  │
  ├ due to 'br:', which was opened here:
  │
  │  [ '…' readLine br: | ]
  │
  ├ due to 'br', which was opened here:
  │
  │  [ ahead eject ahead eject br | ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ sel open hydrate | ]
  │
  ├ due to 'runLine', which was opened here:
  │
  │  [ runLine | ]
  │
  ├ due to 'do', which was opened here:
  │
  │  [ 0 $: pgRoot new $: pgStack shallowCopy $: […] do | error br: […] […] pgStack echo ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ […] swap hydrate | ]
  │
  ├ due to 'measure', which was opened here:
  │
  │  [ […] @: […] measure | =: timeTakenMs ]
  │
  ├ due to 'do', which was opened here:
  │
  │  [ monotonic $: do | monotonic $: t2 t2 t1 - ]
  │
  ├ due to 'hydrate', which was opened here:
  │
  │  [ […] swap hydrate | ]
  │
  ├ due to 'reportError', which was opened here:
  │
  ╰  [ reportError | true =: error self resume ]

Reason is unknown for now.

Related to #20.


(BOLT-750) Remove apply module and apply::resource task

This module was an experimental inclusion to provide basic functionality
to reuse existing Puppet resource types. Since we've now introduced the
apply() keyword as a general, supported solution to this problem, the
apply module is no longer necessary. We're choosing to immediately
remove this rather than add a deprecation based on the fact that we have
no known users of the feature.

Updated the test to just check for package and service, which are much less likely to be removed.

CLA signed by all contributors.


when do you plan to upgrade AlamofireObjectMapper to match the new Alamofire

Alamofire has updated to v5.2, when will AlamofireObjectMapper match more later  Alamofire?

@MrShiY do you face the warning issue of "conversion to swift 5 is available"?. Because I'm facing the same.

Is this ever going to happen? Or does anyone have an alternative?

@reni99 this library appears to be abandoned. This question has come up several times without so much as a peep from @tristanhimmelman .

pod 'AlamofireObjectMapper', :git => 'https://github.com/RomanPodymov/AlamofireObjectMapper.git'


Respect #sourceLocation directives in SourceLocationConverter

Add a presumedFile and presumedLine property to SourceLocation that contains the file and line of the location while taking #sourceLocation directives into account.
The terms “presumed file” and “presumed line” have been taken from LLVM and the Swift compiler.
rdar://99187174

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows

@swift-ci Please test

@swift-ci Please test Windows


Compatibility with PTL 1.6

Todos:

[x] Check if we need to_state_stream / load_state_stream P(0)
[x] Check multi node (P0)
[x] Check multi GPU/multi node (P0)
[x] Fix / change tests (P0)
[x] Check that recent PRs are included, e.g. https://github.com/ray-project/ray_lightning/pull/156 P(0.5-1)
[x] Check Ray client (P1)
[x] Check fractional GPUs (P2)
[x] DDP sharded (P2)

Could we also merge in this PR to rename everything to strategy following the 1.6 convention.

Error:

[ ] enable gpu training

- [ ] multi-node gpu case, it hangs up forever.

to_state_stream / load_state_stream seems to be useful because
the weights will not be changed

from this test:
https://github.com/ray-project/ray_lightning/blob/6aed848f757a03c03166c1a9bddfeea5153e7b90/ray_lightning/tests/utils.py#L241-L245

for the Q1:

Check if we need to_state_stream / load_state_stream P(0)

Yes, we need this. On the remote task, the weights is not going to dump / load from the ckpt from the hard-disk. And to_state_stream / load_state_stream provides a elegant way to fetch and pass the weights to driver.

the test files used are https://github.com/JiahaoYao/ray_lightning/blob/3df599a8bb1ac917bf6352b8baef63ad64f21595/ray_lightning/tests/test_ddp_gpu.py

test_ddp.py passed

can close this pr @krfricke @amogkam


⚠ Bofrost.lu has degraded performance

In 896c369, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1009 ms

Resolved: Bofrost.lu performance has improved in c8fd0fc.


datarepo.py refactor

In preparation for adding validation

@dcolligan can we close based on switch to ORM?

I still think the refactor here would be a good idea, but we can close since we're not planning to merge anytime soon (and to implement similar ideas now it might be better to start from scratch rather than do the rebase...)


Remove and ignore .DS_Store file

Very tiny one 😄 This PR removes the committed .DS_Store and ignores it (.gitignore), so it won't be added by mistake in the future.
I will find some time later this month for a bigger contribution on my next PR.

Thank you for your contribution.

No worries @BurtDS! My bad, just realized I forgot to commit the .gitignore file. Just raised a PR with it :) https://github.com/BurtDS/laravel-vatnumber-checker/pull/3


Clears Old Storage for Session pallet

Fixes https://github.com/paritytech/polkadot/issues/4691
The old migration didn't clear the data properly. This PR adds a V2 to clear everything under Session::HistoricalSessions and Session::StoredRange
polkadot companion: https://github.com/paritytech/polkadot/pull/7132

As we just missed a migration while migrating to the pallet macro version of the pallet, we don't need the migration here in Substrate. This is a chain specific migration, a migration that needs to be run once and can be put into the Polkadot/Kusama runtimes directly.


Update to Rust 2018

Merging now, as this is all boring boilerplate that should be uncontroversial and is highly likely to to create annoying merge conflicts.


Provide rust_bpmn_analyzer using wasm-bindings for JS.

Use a WASM binding to avoid calls to a server and needing a server that runs the rust code.
We could even serve the page without a backend just using GitHub pages.

It already works fine, see wasm branches in this repo and the bpmn-analyzer-js repo.
The performance of the WASM binding also does not seem to be too slow compared to native rust. However, this has to be investigated in detail.
There is still some duplicated code between the WASM project and the web project due to DTO's.

Served using GH-pages:
https://timkraeuter.com/bpmn-analyzer-js/


Add xz compression to CentOS build tools

Signed-off-by: Violeta Sebe<EMAIL_ADDRESS>

What's this required for?

It is required to unpack git source: git-2.15.0.tar.xz (in Extract git source task.


Added Layout, Navbar, UI, URL changes

Closes #5  and #7
Signed-off by : Aman Jagotra<EMAIL_ADDRESS>


Cypress Tests for Player Event Registration

Description

Write E2E Cypress tests for #241 Player Event Registration.
Possible Implementation

View Event Sessions
Register to an Event Session
Unregister from an Event Session

Please add your planning poker estimate with ZenHub @pascwhale

Hey team! Please add your planning poker estimate with ZenHub @bcobo341 @IanjhPhillips @JasonGadoury @luigibesani @willyyhuang


fix: HttpRequestMessage instance per request

Fixes the exception System.InvalidOperationException: The request message was already sent. Cannot send the same request message multiple times. when the HttpRequestMessage instance is being reused.
It's a breaking change

@PatrykPlewaOfficial thanks a lot for your contribution!
@innomaxx please review it


nbd-tracker: e2e for all access mode and volume mode

Describe the bug
The idea is to cover e2e tests as much as possible for access modes like:
ReadWriteOnce
ReadWriteMany
ReadOnlyMany
with FS mode and Block mode (as needed).

@Madhu-1 we should qualify these for alpha support at 3.4.0 ?

As we have a volume healer coming in 2nd sprint it's good to cover this one to make a support statement.


composefs: init at unstable-2023-08-23

Description of changes
Added composefs as a package. It is a file system implemented using erofs, overlayfs, loop devices and a content-addressible storage directory for file contents on linux 6.5 and higher.
Support for fs-verity verification is currently depends on features from overlay-next.
Integration with ostree is currently "experimental": https://ostreedev.github.io/ostree/composefs/
It would be great if someone could check if the licenses are correct, if I'm reading the repository correctly the binaries are GPL2Only, while the library is LGPL21Only or GPL2Plus.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Just noticed they have an active PR for an official stable release, so we should probably wait until that is merged before merging this PR.

It would be great if someone could check if the licenses are correct, if I'm reading the repository correctly the binaries are GPL2Only, while the library is LGPL21Only or GPL2Plus.

Checked the actual license in the source files, the library component is lgpl21Plus, while the binary tools are gpl3Plus

Remaking PR since 1.0.0 was just released.


[Request] Back to October link/button

Hi,
Great plugin, very useful! Just one suggestion, when on the login screen, I think it would be useful to either incorporate the October CMS nav at the top, or have some sort of back link to take you back to the CMS backend.

Hello Daniel.
I'm planning this, but not sure how to do it.
Don't want to touch adminer source code.
One alternative can be an iframe.

I've added a "iframe mode" on v1.1.0.

Thanks for adding this, works perfectly - I would just make the iFrame option as the default, as I don't really see the point in having it full screen as yoru stuck in the same situation as my initial comment i.e you can't get back to the October backend with ease - just a thought - but good job :)


Settings Type Mismatch

Hi, currently using latest 4.5.5 and have noticed a few type differences in some settings.
Haven't scoured over all settings, but ones I am intending to use are:
Settings.RETINA_PREFIX
Settings.PRECISION setting with Pixi.Precisions
So Retina Prefix should be a RegExp: we can cast this at the moment.
Settings.RETINA_PREFIX = cast new RegExp("scale-(.+)");
Trying to set Settings.PRECISION but using a magic var as consts seems mistyped and maybe misnamed.
Pixi.Precisions.XXX should be a string, currently is an enum, maybe we can cast this extern to string?
But also Pixi.Precisions is undefined for me, maybe it's named wrong, docs have same name but no "S" on the end.
Looking at some older libraries this used to be a typedef where each member was a string.

Also have just noticed that PIXI.settings.PRECISION is now deprecated.
Replaced by
PIXI.settings.PRECISION_FRAGMENT
PIXI.settings.PRECISION_VERTEX
Could these be added to a new version, was only needing to modify the Fragment setting anyway.
Thanks,
Dan.

should be good now.. delete and reinstall pixijs haxelib to check

@adireddy Just a note on the RegEx, EReg actually didn't work for me, had to use the JS specific RegExp, EReg is missing exec function.

And the precision vars maybe should type:Pixi.Precisions
But this const is broken for me, think the extern should be:
@:native("PIXI.PRECISION") instead of @:native("PIXI.PRECISIONS")

try again now
changed EReg to js.RegEx and fixed PRECISION

@adireddy RegEx good, Precision Extern now good.
If PRECISION_VERTEX & PRECISION_FRAGMENT could be changed from string to Precision as well would all be perfect?
Thanks.

done 👍

@adireddy Since last change, need to add import at top of Settings.hx file
import pixi.core.Pixi.Precision;

:) this time I didn't compile

should be fixed now

Awesome, yep everything is perfect, thanks :)


sendDwnRequest returning web Readable

DWNManager has sendRequest and processRequest both return the DwnResponse type.
However sendRequest returns a web Readable as the data property while processRequest returns an isomorphic Readable.
This was causing confusion for me during testing as dealing with the data is different between the two.
I added some tests that around large data payloads in order to test data streams.

Talked to @LiranCohen about this PR yesterday and he's going to review and determine whether it should be closed or taken out of Draft state for review.

@LiranCohen said he'll re-open later once ready for review


contactState_ needed for Legged Control ???

Hello, is those variables really necessary for the controller?
https://github.com/qiayuanliao/legged_control/blob/eff523ddec6b392414073d7ff5f1534cd411c533/legged_examples/legged_unitree/legged_unitree_hw/src/UnitreeHW.cpp#LL57C3-L59C4
My robot lib doesn t have it..
Thanks
Vincent

Yes, it is needed for this project. However, you can hack and use the MPC output as the contact state. I may write some code for this feature in the future.

Ohhh bad for me 😭
Thanks friend for the quick answer.
Now my robot code is nearly ready.
I tested a standup, and motors jiiters.
I am missing those contactstates.
Can I use an existing variable?
Will you have time, very soon, to code the mpc hack you said?
Have a nice day.

hi, @qiayuanliao , did you have time to add MPC output as the contact state ?
Thanks.

small up ?!

Hi Jim. Any hope for any mpc contact states?
See you.
Vincent

Sorry for the dealy. I am working on another project currently. I will add this feature within a month.

Ohh happy for you, my friend. Wish you the best.
And don't forget me/us, please 😉
I'll wait necessary time. 🙏

There is a quick way to hack:
Replace the  line  with contactFlag[i] = leggedInterface_->getSwitchedModelReferenceManagerPtr()->getContactFlags(currentObservation_.time)[i]
So the state estimator directly acquires gait schedule instead of contact sensors.

Ohhh thanks a lot...
I'll test it tonight... I'm impatient. 👍

hello @LifelongYuan
well, tested, and I can't standup. too much motors trembling (bad kp, kd ?)
here is the terminal error :
...
[CppAdInterface] Loading Shared Library: /tmp/legged_control/a1/RH_FOOT_position/cppad_generated/RH_FOOT_position_lib.so
[CppAdInterface] Loading Shared Library: /tmp/legged_control/a1/RH_FOOT_velocity/cppad_generated/RH_FOOT_velocity_lib.so
[CppAdInterface] Loading Shared Library: /tmp/legged_control/a1/RH_FOOT_orientation/cppad_generated/RH_FOOT_orientation_lib.so
WARNING: Failed to set threads priority (one possible reason could be that the user and the group permissions are not set properly.)
WARNING: Failed to set threads priority (one possible reason could be that the user and the group permissions are not set properly.)
WARNING: Failed to set threads priority (one possible reason could be that the user and the group permissions are not set properly.)
[ WARN] [1680904568.708592601]: The root link base has an inertia specified in the URDF, but KDL does not support a root link with an inertia.  As a workaround, you can add an extra dummy link to your URDF.
[ WARN] [1680904583.005091446]: Controller Manager: To switch controllers you need to specify a strictness level of controller_manager_msgs::SwitchController::STRICT (2) or ::BEST_EFFORT (1). Defaulting to ::BEST_EFFORT.
[ INFO] [1680904583.006608700]: Waiting for the initial policy ...
[ INFO] [1680904583.034767466]: Initial policy has been received.
[ WARN] [1680904583.036258342]: Cycle time exceeded error threshold by: 0.0259329s, cycle time: 0.034266262s, threshold: 0.005s
ERROR:  Maximum number of working set recalculations performed
ERROR:  Maximum number of working set recalculations performed
ERROR:  Maximum number of working set recalculations performed
ERROR:  Maximum number of working set recalculations performed
...

I did :
roslaunch legged_unitree_hw legged_unitree_hw.launch (my modified controller)
roslaunch legged_controllers load_controller.launch cheater:=false

here is ok !
and when :
rosservice call /controller_manager/switch_controller "start_controllers: ['controllers/legged_controller']
stop_controllers: ['']
strictness: 0
start_asap: false
timeout: 0.0"

terminal writes => ok: True
Then, all the engines are shaking al lot, and i must then cut their power.
Any idea ?  a kp kd problem ? Where could i reduce them ?
Impatient to read you,
Vicent.

Seems working.. thank you so much


Layout/CaseIndentation auto-correct not indent "end"

A image can describe clearly.

 ╰─ $ rubocop -V
0.69.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.2 x86_64-linux)

It turns out that autocorrection for cop Layout/EndAlignment has been disabled by default since 2015: https://github.com/rubocop-hq/rubocop/pull/1789
You could add the following to your .rubocop.yml to enable it:
Layout/EndAlignment:
  AutoCorrect: true

Layout/DefEndAlignment:
  AutoCorrect: true

Thanks @buehmann! This problem comes up every now and then. If and when we write an FAQ, the above is sure to be featured.


reverse include matcher?

When a function's output is nondeterministic, I sometimes see developers write a spec like:
expect(result).to eq :a || :b

which of course doesn't do what they expect, and eventually they usually settle on include:
expect([:a, :b]).to include result

which works, but subverts the standard pattern of expect(actual).to match(expected).
Would a PR for a be_included_in matcher be welcome?
expect(result).to be_included_in([:a, :b])

(Apologies if this has been discussed and rejected; I looked around a bit but didn't find anything but a few lonely gists.)

This use case is why we added support for compound matchers in RSpec 3:
expect(result).to eq(:a).or eq(:b)

or:
expect(result).to eq(:a) | eq(:b)

With or in place, I'm not convinced be_included_in is necessary.  Thoughts?
BTW, if you want to read more about the new compound matchers in RSpec 3, it was announced here:
http://rspec.info/blog/2014/01/new-in-rspec-3-composable-matchers/

Thanks, that is great – I'd read about compound matchers but hadn't made the connection with this use case.
I think I've had other tests where and and or would be unwieldy – where there's a longer list of potential values, say, or a range. But I don't have those at hand, and I can't say whether they are otherwise justifiable.
I'd say close this, and at least the concept will still be here if I or someone else wants to argue for it in the future.

With a longer list, you could do something like:
def be_included_in(first_value, *rest)
  rest.inject(eq(first_value)) do |matcher, value|
    matcher.or eq(value)
  end
end

I can't imagine how compound matchers could work in many non-trivial cases, like this:
let(:urls) { %w[http://example.com/example.jpg http://example.com/example2.jpg] }

it 'sets proper urls on download files' do
  create_downloads
  expect(download_files).to all(have_attributes(url: be_included_in(urls)))
end

it 'sets proper urls on download files' do
  create_downloads
  expect(download_files).to all(have_attributes(url:
    eq('http://example.com/example.jpg').or eq('http://example.com/example2.jpg')
  ))
end

Or:
it 'sets proper urls on download files' do
  create_downloads
  expect(download_files).to all(have_attributes(url: match(~r|\Ahttp://example\.com/example2?\.jpg\z|)))
end

That was the point. I want to reference an arbitrary collection instead of listing all values in the matcher.

We have an existing custom matcher:
matcher :item_processed_info do |state, stage, count, items|
  match do |info|
    info.state == state       &&
    info.stage == stage       &&
    info.stage_count == count &&
    info.failure.nil?         &&
    items.include?(info.item)
  end
end

I half wondered whether we could rewrite it like...
def item_processed_info(state, stage, count, items)
  have_attributes(state: state,
                  stage: stage,
                  stage_count: count,
                  failure: nil,
                  item: whatever_matcher_matches_one_item_from(items))
end

But I wasn't able to think of a good matcher to fill in the blank. It seemed like a new matcher which was the exact inverse of include...
Or we build a custom matcher from satisfy, and then include that?

I have the problem @eostrom mentioned above, when the expected value is an dynamic array with 1-n values:
expect(400).to be_included_in(dynamic_array)

@myronmarston your example may be fine ... but i would expect that in the rspec standard, because it's quite basic ... so i assert that most developers switch expectation and actual value and forget about it.

@entwanderer if you'd like to create a PR I'd be happy to take a look at it


Add a feature to open/convert a password protected XLSX file by adding the password as a parameter

🚀 Feature Proposal
Add a new feature to be able to open a password protected XLSX file
Motivation
We have a use case to open numerous password protected XLSX file when vendors send data to us and would like to programatically convert the XLSX to CSV
Example
Please provide an example for how this feature would be used.

@bpayfox You can try this library that contains decryption with password protected excel function. @zurmokeeper/exceljs
// read from a file
const workbook = new Excel.Workbook();
await workbook.xlsx.readFile(filename);

// read from a file, decrypt excel files encrypted with password
const workbook = new Excel.Workbook();
await workbook.xlsx.readFile(filename, {password:'123456'});
// ... use workbook

// read from a stream
const workbook = new Excel.Workbook();
await workbook.xlsx.read(stream);

// read from a stream, decrypt excel files encrypted with password
const workbook = new Excel.Workbook();
await workbook.xlsx.read(stream, {password:'123456'});
// ... use workbook

// load from buffer
const workbook = new Excel.Workbook();
await workbook.xlsx.load(data);

// load from buffer, decrypt excel files encrypted with password
const workbook = new Excel.Workbook();
await workbook.xlsx.load(data, {password:'123456'});
// ... use workbook


Fix for WFCORE-5116, Bootable JAR, JMX not properly initialized at boot.

Issue: https://issues.redhat.com/browse/WFCORE-5116

@jfdenise thanks


Documentation: Fixes ApplicationRegion comments and links for region settings in CosmosClientOptions.

Description
Updates the docs on ApplicationRegion and ApplicationPreferredRegions, clarifying fallbacks and linking to more in-depth docs.
Type of change

[x] Bug fix (docs only)

Closing issues
Closes #2733.

/azp run

Thanks @spygi for your contribution!


Temporarily fixed in v1.0.2 by adding an if (mode != 'delete') clause in the initialize.js for adding first monosaccharide to the parent div.

Fixed at least in part in v1.1.0


chore(deps-dev): Update ESLint and its Typescript plugins

Goal
Dependabot is suggesting we upgrade to ESLint 8.0: https://github.com/Pocket/terraform-modules/pull/365.
The upgrade to ESLint 8.0 needs to be accompanied by major version upgrades to @typescript-eslint/eslint-plugin and @typescript-eslint/parser to continue working as intended.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Solving environment: failed

Hi @thanhlecongg,
Running the command conda env create -f environment.yml results in the following error:
Collecting package metadata (repodata.json): done
Solving environment: failed

ResolvePackageNotFound:
  - sqlite==3.39.2=h5082296_0
  - openssl==1.1.1q=h7f8727e_0
  - zlib==1.2.12=h5eee18b_3
  - xz==5.2.5=h7f8727e_1
  - ld_impl_linux-64==2.38=h1181459_1
  - pip==21.2.2=py36h06a4308_0
  - tk==8.6.12=h1ccaba5_0
  - ca-certificates==2022.07.19=h06a4308_0
  - libgomp==11.2.0=h1234567_1
  - libgcc-ng==11.2.0=h1234567_1
  - python==3.6.13=h12debd9_1
  - _openmp_mutex==5.1=1_gnu
  - ncurses==6.3=h5eee18b_3
  - setuptools==58.0.4=py36h06a4308_0
  - certifi==2021.5.30=py36h06a4308_0
  - libstdcxx-ng==11.2.0=h1234567_1
  - readline==8.1.2=h7f8727e_1
  - libffi==3.3=he6710b0_2

My system:
$ sw_vers
ProductName:		macOS
ProductVersion:		13.2.1
BuildVersion:		22D68

$ conda --version
conda 23.7.4

Any idea?

Hi @jose,
The following library are enough for running semantic classifier of invalidator:
pip3 install tqdm
pip install z3-solver
pip3 install importlib_resources

Just quick check with the same MacOS (Lol, we look to use the same kind of PC) and it run ok.
ProductName:		macOS
ProductVersion:		13.2.1
BuildVersion:		22D68

You can quick check by unzipping my data and run
python3 invalidator.py

I have included a test case in default values

It works.  Thanks.


Remove nspecresmin

This variable serves no purpose. It is supposed to replicate the process for initialising NPRTRW and NPRTRV in the IFS (SUMPINI). There one can specify in the IFS namelist a maximum allowed value for NPRTRW, though I'm not sure the reason you would need this. In ecTrans we don't need this feature, so it's best to just delete this.
This PR was prompted by Lucian Anton at ECMWF trying to run ecTrans on 8000 tasks without specifying --nprtrw and --nprtrv. The automatic logic does not allow running with so many tasks due to the check on nspecresmin.

Looks fine to me


Newbie: need some hand holding.

First attempt at using cmake. Errors. My project root CmakeLists.txt:

cmake_minimum_required(VERSION 3.2)
project(blink C CXX ASM)
get_board_id(board_id nano atmega328)
add_arduino_executable(blink ${board_id} src/main.cpp)
upload_arduino_target(blink "${board_id}" /dev/ttyACM0)

Q1: where do I find _documentation for get_board_id. I have googled and googled. What should I be passing for an Elegoo Uno R3?
Now to the make process

rgr@development-nuc  ~/development/arduino/Blink-PlatformIO/build   master ● ?  cmake -DCMAKE_TOOLCHAIN_FILE=../cmake/Arduino-Toolchain.cmake ..
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- The ASM compiler identification is GNU
-- Found assembler: /home/rgr/development/arduino/arduinoSDK/hardware/tools/avr/bin/avr-gcc
-- Arduino SDK version 1.8.8: /home/rgr/development/arduino/arduinoSDK
-- Determined Platform Header: /home/rgr/development/arduino/arduinoSDK/hardware/arduino/avr/cores/arduino/Arduino.h
-- Arduino SDK version 1.8.8: /home/rgr/development/arduino/arduinoSDK
-- Determined Platform Header: /home/rgr/development/arduino/arduinoSDK/hardware/arduino/avr/cores/arduino/Arduino.h
-- Check for working C compiler: /home/rgr/development/arduino/arduinoSDK/hardware/tools/avr/bin/avr-gcc
-- Check for working C compiler: /home/rgr/development/arduino/arduinoSDK/hardware/tools/avr/bin/avr-gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Arduino SDK version 1.8.8: /home/rgr/development/arduino/arduinoSDK
-- Determined Platform Header: /home/rgr/development/arduino/arduinoSDK/hardware/arduino/avr/cores/arduino/Arduino.h
-- Check for working CXX compiler: /home/rgr/development/arduino/arduinoSDK/hardware/tools/avr/bin/avr-g++
-- Check for working CXX compiler: /home/rgr/development/arduino/arduinoSDK/hardware/tools/avr/bin/avr-g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at cmake/Platform/Other/TargetFlagsManager.cmake:67 (set_target_compile_flags):
set_target_compile_flags Function invoked with incorrect arguments for
function named: set_target_compile_flags
Call Stack (most recent call first):
cmake/Platform/Targets/ExecutableTarget.cmake:17 (set_executable_target_flags)
CMakeLists.txt:16 (add_arduino_executable)
CMake Error at CMakeLists.txt:19 (upload_arduino_target):
Unknown CMake command "upload_arduino_target".

Thats it. Where do I go from here. I can find nothing at all in google and this CMakesList was lifted from the first page in the Wiki.
Many thanks for helping me out of the quagmire I find myself in! ;)

@rileyrg Hi and welcome to the Arduino-CMake project!
I'm sorry to hear that you're having such a hard time using this framework.
Reviewing your logs, I suspect you've cloned the master branch of this repo, which is not the stable branch as it is in some other git repos! Instead, here it acts as the main dev branch, while official releases are bundled with GitHub's Releases. Use them (Desirably the latest one).
As for the board_id dimness - The function is nowhere to be found on Google since it's defined by the framework and not documented anywhere in the Wiki (Not as a standalone function at least).
Generally what it does is constructing a string combining both the ID of the board to be used and possibly its' CPU, that later should be passed as an argument to many other API functions.
Just to make it clear, take for example the Uno board you've mentioned - It's not only the name of the board's model, but also it's ID (According to Arduino, not this framework). This is usually like this.
However, looking at the example we see the nano board ID and a possibly strange string, atmega328. This is the board's CPU, and it's required because some boards are compatible with multiple CPUs.
You shouldn't worry about that, just get some basic info about your board in advance.
Back to the get_board_id function - This entire concept you see in the example is about to be deprecated (In fact it's already deprecated in master, however not documented yet), omitting the use the board_id argument by users like you entirely.
With that being said - Please follow the Wiki docs (And the above explanation) until the new version is officially released.
Now one last thing regarding boards - You've mentioned the Elegoo Uno R3 board, however, I'm not sure it's entirely supported by the framework yet. It seems to be a 3rd party platform, i.e. not plain, native Arduino, which is currently not supported.
You should first try to use the project after reading what I've written above, but if that doesn't work - Do not be so surprised 😞
I hope I could en-light your path to happy Arduino-CMake development! 😄

Many thanks. I did get it working in the end but not using the documented
samples. I shall get back with my cmake set up.   I had considered using
platformio but I can't see a way to generate compile_commands.json which I
need for the excellent LSP/ccls Emacs integrations. So, thank you. And
you'll hear from me soon.
On Fri, 15 Mar 2019, 11:31 Timor Gruber<EMAIL_ADDRESS>wrote:

Closed #77 https://github.com/arduino-cmake/Arduino-CMake-NG/issues/77.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/arduino-cmake/Arduino-CMake-NG/issues/77#event-2206051692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWJi2yQ0n-UJcfdGa98nf30sZenM_9Qks5vW3aOgaJpZM4beKmC
.


merge from arlac77/template-github-action,arlac77/template-kronos-component

package.json

chore(deps): add ^3.13.0 remove ^3.12.1 (devDependencies.ava)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #698   +/-   ##
=======================================
  Coverage   99.08%   99.08%
=======================================
  Files           1        1
  Lines         109      109
  Branches        9        9
=======================================
  Hits          108      108
  Misses          1        1

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 0bc2328...171a611. Read the comment docs.


requester ip

I need to get the remote ip for the requester and return it back in the response. I could do that on regular controllers using
request.remote_ip

however, in grape calls, the 'request' is not a defined variable.
how can I get access to the remote ip on grape?

undefined method remote_ip' for #Grape::Request:0x0000011504a090`
remote_ip is not a part of the Grape::Request.  Not sure if there is another way to access it there.
env["REMOTE_ADDR"] appears to work.

hey @dblock is there any official documentations where @teamon took the env["REMOTE_ADDR"] and env["REMOTE_HOST"] from? I dont seem to find it anywhere in the code

I am going to guess it's somewhere from Rack.

I'm always getting <IP_ADDRESS> from env["REMOTE_ADDR"]. Maybe the way my load balancer is setup?

@tibbon most probably you're right, https://easyengine.io/tutorials/nginx/forwarding-visitors-real-ip/ might help


Demo doesn't really seem to have any knowledge about git repos after sync

No matter what I try it tells me to be more specific or that it has no knowledge.

Same for me. The bot tells me he has access to my repo but when I ask him something specific about a class etc. he is "hallucinating"

@ilikenwf @michsindlinger
The demo only scan the 1st level of the repo (not recursively) because of the lack of resources, so it might not get the full picture of a given repo, however, the self-hosted version will do a full recursive scan for the given repo.
Also, can you try to give it ~2 minutes to full ingest all data, duo to the high load it might be a bit slow to ingest everything, and make sure you are in AI programmer mode.

Let me know if this fix the problem

@gharbat Thanks for the fast response.
I used the self hosted version to test it, but it seems he couldn't fetch the data from my repo.
It's a private github repo. I added a GITHUB_ACCESS_TOKEN, so it should work actually.

@michsindlinger
How you added the GITHUB_ACCESS_TOKEN? you should add it inside the llm-server container.

@gharbat I added it to the docker-compose.yml (see the atached screenshot).
Gitguh also shows me that the new generated key has been used.

It's simply because you need to chose "Pair Programmer" while talking to your chatbot, otherwise he won't have any context and will tell you he has no knowledge.

Having the same issue, locally hosting it with everything setup properly, still the bot has no idea about repo


refactor(rest): separate protobuf utils from grpc utils

Part of the work for #10037
While using REST transport requires proto support, it does not need grpc support. This PR separates the proto specific pieces that the REST logging decorators will require. rest_log_wrapper.h will be added in a later PR.

This change is

While using REST transport requires proto support, it does not need grpc support. This PR separates the proto specific pieces that the REST logging decorators will require. rest_log_wrapper.h will be added in a later PR.

That might be true, but separating this library will not stop customers from needing to link gRPC:

The generated *.pb.cc and *.grpc.pb.cc files go into the same CMake library
The client libraries will always link the gRPC dependencies (for both CMake and Bazel)

If you are trying to help customers not link gRPC, then this is not enough, and I am not sure we can achieve that.


Spin the Mayfly Wing into its own repo

@aufdenkampe I think it should stand on its own, since it's only proximal to the library.

I like this idea. Is there a fancy Git way to do this that preserves commit history and issues?

Yes, but I'd have to look it up.  You can do things like clone and prune to keep only certain folders.

@SRGDamia1, with @neilh10's recent work, it's definitely time to spin the hardware into it's own repo. See https://github.com/EnviroDIY/SensorModbusMaster/issues/14#issuecomment-773421857.
Do you have any suggestions on how to do this in a way to preserve both commits and issues? I'm OK doing the work, including pruning, if I have some guidance. I'll start digging for articles online.

@SRGDamia1, I'm going to move forward with this, now that Neil has made additional contributions. I also found these instructions, which seem quite reasonable to follow. https://docs.github.com/en/github/getting-started-with-github/splitting-a-subfolder-out-into-a-new-repository

Shall we call the new repo "Mayfly-Modbus-Wing"?

Hmm name - valuable for "Presentation"  and searching - ~  how to suggest version 2.
Currently is hardware/Modbus-Mayfly_WingShield  -    and effectively version 1
The board has printed on it
KNH002r6
Mayfly RS485
would Mayfly-Wing-Rs485ver2/  for top level
Mayfly-Wing-Rs485ver2/knh002r6 for specific hardware release.
I personally think the hardware is better represented with RS485, and Modbus is the protocol that sits on top of RS485.
I can see that just being consistent with the Modbus might simplify an early user - so don't have too much opinion on what it is called
so could be  Mayfly-Modbus-Wing2/  for top level
Mayfly-Modbus-Wing2/knh002r6  for specific hardware package. The numbering "knh002" is somewhat arbitary and just needs to be unique - so I have Kicad NH (for me) and 002 for the board number in my releases, and of course "r6" for the board spin.
However equally could be Mayfly-Modbus-Wing and then the text could I identify it as next generation that supersedes "Modbus-Mayfly_WingShield "

So another way of thinking about splitting out SensorModbusMaster/hardware
https://docs.github.com/en/github/getting-started-with-github/splitting-a-subfolder-out-into-a-new-repository
You'll  create a new repo   and crate in it current files - readme.MD
https://github.com/EnviroDIY/SensorModbusMaster/tree/master/hardware
Then create a  and put all files <Modbus-Mayfly_WingShield>  changing the text name description to
"Mayfly-Modbus-Wing version1" - but everything else is same
I could then do  and that would have my specific descriptions including "Mayfly-Modbus-Wing version2"
Then create directory  put all my kicad files into it?
However I can make any of it work .....

@neilh10, yeah, that's basically what I was thinking for the process of spinning into a new repo. The new repo would just contain the commits for files presently in the hardware folder. I don't think the issues will follow, but that's OK. We'll then modify the main ReadMe.md, etc.
Regarding naming and releases, my thinking is that the repo name would not include the version number, but we would  handle that via official GitHub releases. We can release historical versions from the commit history (I recently did this for another repo).
For the name, I like either "Mayfly-Modbus-Wing" or Mayfly-RS485-Wing". I recognize that RS485 is the physical communication protocol enabled by the hardware, and that Modbus is a logical communication protocol that is enabled by the software outside of the repo. Nevertheless, for the EnviroDIY community, I might lean slightly in favor of "Mayfly-Modbus-Wing", given it's likely wider recognition and that this hardware is specifically designed to work with the Mayfly to enable Modbus sensors.
What do you all think?

Mayfly-Modbus-Wing sounds good, happy with either one.
Well on versions if you want it to be a single stream that can be made to work  - it keeps the view "simple" - the repo keeps the history.  I personally find that I  step into previous releases or want to do a comparison between revisions.  I'll be maintaining my  KNH002r6 sources -  that is the magic number that identifies this  physical board - then  release to  stream as needed.

@neilh10, I just made the move! Check it out at https://github.com/EnviroDIY/Mayfly-Modbus-Wing
I just invited you to be a maintainer, so you should have full write+ access. Please share your latest rev7.
Next steps for me are adding a license and a few other cleanup items.
If things look good to you, I'll create a "historic" release of the original version and start modifying the main readme to describe your new version.

All hardware related issues should now be posted at https://github.com/EnviroDIY/Mayfly-Modbus-Wing


Add error message if step is called twice

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Describe changes

Add error message if step is called twice
Improve error message if step class is passed twice

@htahir1 Basically just refactored a hard-coded fixture that returns three steps to make it configurable (now you're able to specify the amount of steps you need)


Test failures with python 3.10

Vendored pkg_resources thinks python 3.10 < 3.5:
**********************************************************************
File "/home/msarahan/code/guildai/guild/tests/deprecated-options.md", line 30, in deprecated-options.md
Failed example:
    test("guild ls --unlabeled")
Expected:
    WARNING: option --unlabeled is deprecated and will be removed in version 0.8 - use '--label -' instead
    guild: no matching runs
    <exit 1>
Got:
    WARNING: option --unlabeled is deprecated and will be removed in version 0.8 - use '--label -' instead
    Traceback (most recent call last):
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 584, in _build_master
        ws.require(__requires__)
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 901, in require
        needed = self.resolve(parse_requirements(requirements))
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 792, in resolve
        raise VersionConflict(dist, req).with_context(dependent_req)
    pkg_resources.ContextualVersionConflict: (psutil 5.9.0 (/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages), Requirement.parse('psutil<5.8,>=5.6.3'), {'guildai'})
    <BLANKLINE>
    During handling of the above exception, another exception occurred:
    <BLANKLINE>
    Traceback (most recent call last):
      File "/home/msarahan/mambaforge/envs/guildai/bin/guild", line 33, in <module>
        sys.exit(load_entry_point('guildai', 'console_scripts', 'guild')())
      File "/home/msarahan/code/guildai/guild/main_bootstrap.py", line 40, in main
        _main()
      File "/home/msarahan/code/guildai/guild/main_bootstrap.py", line 66, in _main
        guild.main.main()
      File "/home/msarahan/code/guildai/guild/main.py", line 33, in main
        main_cmd.main(standalone_mode=False)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/core.py", line 829, in __call__
        return self.main(*args, **kwargs)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/core.py", line 782, in main
        rv = self.invoke(ctx)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/core.py", line 1259, in invoke
        return _process_result(sub_ctx.command.invoke(sub_ctx))
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/core.py", line 1066, in invoke
        return ctx.invoke(self.callback, **ctx.params)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/core.py", line 610, in invoke
        return callback(*args, **kwargs)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/click/decorators.py", line 21, in new_func
        return f(get_current_context(), *args, **kwargs)
      File "/home/msarahan/code/guildai/guild/click_util.py", line 213, in fn
        return fn0(*(args + (Args(**kw),)))
      File "/home/msarahan/code/guildai/guild/commands/ls.py", line 96, in ls
        from . import ls_impl
      File "/home/msarahan/code/guildai/guild/commands/ls_impl.py", line 26, in <module>
        from . import remote_impl_support
      File "/home/msarahan/code/guildai/guild/commands/remote_impl_support.py", line 21, in <module>
        from guild import remote as remotelib
      File "/home/msarahan/code/guildai/guild/remote.py", line 21, in <module>
        from guild import entry_point_util
      File "/home/msarahan/code/guildai/guild/entry_point_util.py", line 17, in <module>
        import pkg_resources
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 3254, in <module>
        def _initialize_master_working_set():
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 3237, in _call_aside
        f(*args, **kwargs)
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 3266, in _initialize_master_working_set
        working_set = WorkingSet._build_master()
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 586, in _build_master
        return cls._build_from_requirements(__requires__)
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 599, in _build_from_requirements
        dists = ws.resolve(reqs, Environment())
      File "/home/msarahan/code/guildai/guild/external/pkg_resources/__init__.py", line 787, in resolve
        raise DistributionNotFound(req, requirers)
    pkg_resources.DistributionNotFound: The 'scikit-learn<0.21,>=0.19.1; python_version < "3.5"' distribution was not found and is required by guildai
    <exit 1>
**********************************************************************

We probably just need to port whatever fix pkg_resources currently has.

Moving along with trying to get python 3.10 working. When I replace the vendored pkg_resources and setuptools (which are merged now), things mostly work but a couple more errors pop up:
Setuptools appears to override distutils, but is not able to do so in our test suite:
**********************************************************************
File "/home/msarahan/code/guildai/guild/tests/imports.md", line 25, in imports.md
Failed example:
    for name in sorted([m.__name__ for m in iter_mods()]):
      print(name) # doctest: +REPORT_UDIFF
Exception raised:
    Traceback (most recent call last):
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/doctest.py", line 1346, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest imports.md[6]>", line 1, in <module>
        for name in sorted([m.__name__ for m in iter_mods()]):
      File "<doctest imports.md[6]>", line 1, in <listcomp>
        for name in sorted([m.__name__ for m in iter_mods()]):
      File "<doctest imports.md[5]>", line 14, in iter_mods
        yield importlib.import_module(mod_name)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/importlib/__init__.py", line 126, in import_module
        return _bootstrap._gcd_import(name[level:], package, level)
      File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
      File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
      File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
      File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
      File "<frozen importlib._bootstrap_external>", line 883, in exec_module
      File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
      File "/home/msarahan/code/guildai/guild/package_main.py", line 27, in <module>
        import setuptools
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/setuptools/__init__.py", line 8, in <module>
        import _distutils_hack.override  # noqa: F401
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/_distutils_hack/override.py", line 1, in <module>
        __import__('_distutils_hack').do_override()
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/_distutils_hack/__init__.py", line 72, in do_override
        ensure_local_distutils()
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/_distutils_hack/__init__.py", line 59, in ensure_local_distutils
        assert '_distutils' in core.__file__, core.__file__
    AssertionError: /home/msarahan/mambaforge/envs/guildai/lib/python3.10/distutils/core.py
**********************************************************************

Error text has changed (just need to update tests to account for this:
**********************************************************************
File "/home/msarahan/code/guildai/guild/tests/notebooks.md", line 259, in notebooks.md
Failed example:
    logs.print_all()
Expected:
    WARNING: error parsing Python source "'unterminated string": EOL
    while scanning string literal (<unknown>, line 1)
Got:
    WARNING: error parsing Python source "'unterminated string": unterminated string literal (detected at line 1) (<unknown>, line 1)
**********************************************************************

Method replacement in python_utils doesn't appear to be working:
**********************************************************************
File "/home/msarahan/code/guildai/guild/tests/python-utils.md", line 335, in python-utils.md
Failed example:
    calc.incr(1, 2)
Expected:
    Traceback (most recent call last):
    TypeError: incr() takes ...
Got:
    Traceback (most recent call last):
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/doctest.py", line 1346, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest python-utils.md[58]>", line 1, in <module>
        calc.incr(1, 2)
    TypeError: Calc.incr() takes 2 positional arguments but 3 were given
**********************************************************************

I believe this one corresponds to this example:
Here's a function that supports a second argument, which specifies the
amount to increment by. Note we return a result by raising
`python_util.Result` (as in the previous example). Note we don't use
the original method (represented by the `_incr` argument) because
we're replacing it altogether.

    >>> def incr2(_incr, x, incr_by=1):
    ...   raise python_util.Result(x + incr_by)

We wrap the original:

    >>> python_util.listen_method(Calc, "incr", incr2)
    <guild.python_util.MethodWrapper object at ...>

And here's our new behavior:

    >>> calc.incr(1)
    2
    >>> calc.incr(1, 2)
    3

Finally, VCS stuff got messed up somehow. This might have been a timeout issue with the tests waiting on me to type in my password, as I had started the tests and walked away for lunch.
**********************************************************************
File "/home/msarahan/code/guildai/guild/tests/vcs-utils.md", line 56, in vcs-utils.md
Failed example:
    commit, status = vcs_util.commit_for_dir(repo)
Exception raised:
    Traceback (most recent call last):
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/doctest.py", line 1346, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest vcs-utils.md[13]>", line 1, in <module>
        commit, status = vcs_util.commit_for_dir(repo)
      File "/home/msarahan/code/guildai/guild/vcs_util.py", line 89, in commit_for_dir
        raise NoCommit(dir)
    guild.vcs_util.NoCommit: /tmp/guild-test-_0uzp3wr
**********************************************************************

Talked with Garrett, and we'll drop python <=3.5 so we can remove the requirements.txt lines that are causing issues. The other issues require more investigation.

I was really keen to try out Guild AI but my project is using 3.10. It's failing with
  File "/home/datapastry/.cache/pypoetry/virtualenvs/rankeval-YsU-ilYr-py3.10/lib/python3.10/site-packages/guild/external/pip/_vendor/html5lib/_trie/_base.py", line 3, in <module>
    from collections import Mapping
ImportError: cannot import name 'Mapping' from 'collections' (/usr/lib/python3.10/collections/__init__.py)

I'm guessing that's covered by this issue. Do we have a workaround or a working branch we can use the in meantime?

You can try the master branch with https://github.com/guildai/guildai/pull/324
the tests still fail with that pr but the normal code paths might work

fixed in #341; closing


JS brakes when Combine JS is enabled.

Hey,
The js script brakes on my review page when it is combined. When I disable the JS combine feature in Magento backend everything works, but I would like to keep it combined, for optimization.
Here is the preview of the code that is first to cause problems - event-simulate.js:

I tried multiple times to reproduce this issue but failed. For me it worked nicely with jquery and other plugins. Most probably you have one script wrongly formatted or that is missing a semicolon


initial changes

made the changes... you are permitted to make any modification

Just sent the nav section branch...

Thank you for the permission


Border's not properly applied for HorizontalBar chart

I've been trying out HorizontalBar chart using the sample data on Chart.js documentation page, but here's what I got.

The border's applied in a same way the original (vertical) bar does - border on left, top and right but not bottom. Pretty sure this is not what HorizontalBar content is intended to be, according to the sample demonstrates(https://www.chartjs.org/samples/latest/charts/bar/horizontal.html - borders are applied on top, right, and bottom)
I don't think it's a matter of options or styles.
Thanks!

You can add the bottom border yourself.
Here is a similar issue which gets resoved by playing with y- and x-axis configuration in chart's options object.
Hope this helps

Closing. Reason: older than 6 months and not enough or outdated information supplied. Please re-open this issue if the problem persists, but provide more information. (including a sandbox link if possible)


Dynamic objects does not work as expected

System Information

Steam Audio version: 4.4.1
Unity version: 2022.3.12f1
Operating System and version: W11 10.0.22631
CPU architecture: x64

Issue Description
I have 2 rooms, 1 with a steam audio source (more audio sources playing the same audio so far away, playing the audio at the same time), and between them a dynamic object door.
All the walls and the door have occlusion 100%. So no sound should tresspass.
Then my tests are opening and closing the door from the room without audio.
It works great as the player keeps moving. But if I stop moving the player just before the door closes, I keep listening the audio as a reflection.
Example moving the dynamic object transform: https://www.youtube.com/watch?v=CZUDK2PGfXQ
Example moving the dynamic object rigidbody: https://www.youtube.com/watch?v=IPP6YmnLHtc
As you can see in the video, the audio keeps playing until the listener transform moves.
EDIT
* If I duplicate the audio source and disable the original it works, but only for the first time. So dynamic objects are only
working with 1 audio source.
--
Also tried to reproduce it with 3 rooms and 2 audio sources (middle room is empty). The only thing I get so far is that the dynamic objects only works if the audio source is a new one.
I mean: If I duplicate the RoomA audio source (and disable the original copy) the dynamic object will oclude the sound and the RoomB will not.
Then I duplicate the RoomB audio source (and disable the original copy) and the dynamic object will oclude the sound and the roomA will not.
--
Also tested with primitive objects as walls and as dynamic objects.
All is being tested with realtime reflections.
Images attached:
Dynamics objects config (doors):

Audio sources:

Steam Audio settings:

@jordihuertas - I cannot repro the issue on a setup similar to yours. Are you in a position to share a standalone issue? Might be faster to resolve the issue that way. Here is some additional insight:
See SteamAudioDynamicObject.cs (Lines 61-67). We rely on transform.hasChanged to make sure Steam Audio internal geometry structure is updated and that simulation is indeed updated. Is there a special way you might be handling transform for the door?
           // Only update the dynamic object if it has actually move this frame             if (transform.hasChanged)             {                 mInstancedMesh.UpdateTransform(SteamAudioManager.CurrentScene, transform);                 SteamAudioManager.ScheduleCommitScene();                 transform.hasChanged = false;             }

@achandak I do not know what is a standalone issue, could you please explain to me?
There is no any special way, just a transform position. The second video I attached is just pushing a rigidbody with a hinge.

@achandak I think I am a little closer to the problem. Maybe you can reproduce it this way.
2 audio sources playing the same clip from a script.
The dynamic object only excludes the last created one.

Sorry, I meant a standalone project or a small reproable unity project. We tried all obvious things we thought could go wrong.
I will try spawning additional sources.

Yes, I will try to reproduce in a new project. I will feedback you tomorrow.

@achandak here it is. https://drive.google.com/drive/folders/1BOPLlH_I8TjPL_-n-BS3vCa83EH6sjbb?usp=drive_link
Readme (also inside the assets folder)
STEPS:

Import Steam Audio.
Created demo scenario (cubes).
Configure SteamAudio settings like my project.
Created AudioSource_A and configured it like my project.
Duplicated AudioSource_A and renamed to AudioSource_B.
Created Steam Audio Material like my project.
Added Steam Audio Geometry and Steam Audio Static Mesh to Scenario GameObject.
Added Steam Audio Geometry and Steam Audio Dynamic Object to both DynamicObject_X GameObjects.
Export Dynamic Objects and Export Active Scene.

Now for me, B door is occluding and A is not.
There is a script attached to each dynamic object (orange ones) with a button to start/restart the moving process (open close doors).
On finish closing doorA, you can move the camera a little to stop playing the sound.

Remember to configure the project settings audio to use Steam Audio.
With the imported project, Dynamic Objects will not occlude the audio when "close".
You can clone an audio source and disable the original copy to check that the new audio source will be occluded.
If you do the same with the second one, the first one will stop working as expected (the door will not occlude the audio).

Here is a demo video https://www.youtube.com/watch?v=TZgbQ0Pz5so&ab_channel=PepeGrillo

@jordihuertas - Thanks for providing a repro. We indeed found a bug where only one source was being updated when dynamic geometry was present. The fix is inside phonon.dll so we should have this fixed in the next release.

I'm glad to hear that you finally found it!

This issue should now be fixed in v4.5.1.

Sorry for the late response, I had some problems when updating because some unity crashes related to git.
Retested. It works!

Thanks for retesting. Closing.


Allow StartupConfig to be an http or https url

Of your configs are e.g. stored in a git repo with http(s) UI or pastebin or something along those lines, you can now define a raw url as the startup-config.
This will result in clab creating a clab-download in your $TEMP directory (e.g. /tmp/clab-download) where the referenced files will be downloaded before the nodes are being started.
If you want to redeploy with new configs e.g. the git branch evolved, simply add the --reconfigure flag to the clab deploy command.

So I realized that the download paths etc should come from topopaths as well, to source all our paths from there.
hence the calculation of also the filename is done there now and follows the pattern you've suggested. If for whatever reason an error occured when parsing the provided url or the resulting string is one of zero length, the filename / postfix component of the filename will fall back to CalcFilename_Undefined = "undefined".

Hey @steiler
Thanks, this is a great addition. I did a bit of a refactoring, ptal if that makes sense to you.
The testing harness has been a bit simplified to allow easy local testing. If you have rf installed, then to run tests locally you do

make build to build a binary from the current tip of a branch
CLAB_BIN=bin/containerlab bash tests/rf-run.sh docker tests/02-basic-srl/01-two-srls.robot

And that will fire off tests locally

I've always simply set an alias to the local build, that works equally fine, anyway.
LGTM


Change all item textures to animated versions

Created animated versions of the campfire item textures

Oh wow haha
Do you still want them to be added, since you closed PR?


Update README.md

added one paper -- SplaTraj: Camera Trajectory Generation with Semantic Gaussian Splatting
🔍 Check out our paper at

https://arxiv.org/pdf/2410.06014 📝
https://youtu.be/PUXNBfpeZkg 🎥
code would be released soon

Awesome. Thank you!


Parquet format support for data import

Is your feature request related to a problem? Please describe.
Currently parquet data format is not supported for import.
Describe the solution you'd like
There are is already a parquet package but I'm not sure how much effort it would take to integrate or if it would be possible:  https://github.com/ironSource/parquetjs

Is this a thing? I had assumed Parquets would be supported by default just like CSVs. Is there a reason Parquet isn't a supported file type?


Azure: Fix Azure ingest walker

Close #5136

@N-o-Z can you verify this is a fix - the current test cover the case the base dir can be an empty path.

@N-o-Z can you verify this is a fix - the current test cover the case the base dir can be an empty path.

@nopcoder look at the example of gcs.
What happens is that you are importing the entire container without taking into account the requested prefix (folder).

@nopcoder  Added an additional test validation the demonstrates the issue


Example command "python train_qm9.py -g 0" get "main/accuracy" over 100score...☹

Enviroment
Ubuntu 16.04.3
Python 3.5.4
chainer (3.1.0)
chainer-chemistry (0.1.0)
cupy (<IP_ADDRESS>)
"main/accuracy" and "validation/main/accuracy" get over 100 score...☹

And, I tried the command "python train_qm9.py --method=nfp  --gpu=0 ", but the score get over 100 score...☹

Please, teach me.

We are sorry for few documentation.
qm9 example is regression task. "loss" is taken as mean squared error of standardized data, and "accuracy" represents mean absolute error of non-standardized data.
Therefore, even it is written as "accuracy", it is actually not the classification accuracy.
We should consider update the documentation or example itself.

http://chainer-chemistry.readthedocs.io/en/latest/tutorial.html
Please also refer this tutorial for detail explanation.

Oh, I see 😮
I skipped reading...😓
Thank you !!


Bump opensearch ref to 1.1 in CI

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
Core has 1.1 branch cut and bumped 1.x minor version to 1.2, use opensearch ref to 1.1 in CI
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #205 (dae3291) into main (728d415) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head dae3291 differs from pull request most recent head d388230. Consider uploading reports for the commit d388230 to get more accurate results

@@            Coverage Diff            @@
##               main     #205   +/-   ##
=========================================
  Coverage     99.89%   99.89%
  Complexity     2512     2512
=========================================
  Files           245      245
  Lines          5853     5853
  Branches        342      342
=========================================
  Hits           5847     5847
  Misses            5        5
  Partials          1        1

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 728d415...d388230. Read the comment docs.

The SQL build and test workflow is "skipped" for some reason, could you take a look of it?

The SQL build and test workflow is "skipped" for some reason, could you take a look of it?

Fixed


508: Update Button Colors and Contrast

Buttons on the application homepage have the wrong background for Accent Cool, Hover, and Active Properties.
Reference:
$eds-accent-cool: #00bde3;
$eds-accent-cool-hover: #28a0cb;
$eds-accent-cool-active: #07648d;

Update the font color in the button to #000000 for proper contrast

Locations to be updated:
Home Screen:

View Monitoring Plans
View QA & Certifications
View Emissions

I'm still getting low contrast of 3.0:1 for the hover state, it needs to be at least 4.5:1

Text changed to black and now has a contrast of 7:1, closing ticket


pkg/types: add annotation keys

Move the annotation keys to pkg/types to allow for referencing them
without pulling in a number of redundant dependencies.  Update all
internal users to use these new consts.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>Release Note
Annotation keys of the Cosign spec have been to pkg/types.

@dlorenc @imjasonh PTAL

I support this change, just a few small comments/questions:

Is there any value in moving any of these to sigstore/sigstore?
To avoid breaking users (I'm not sure how many there even are outside of cosign...), is it worth keeping the existing consts, pointing to the new consts, with a Deprecated comment?
Is there any indication of how much this improves the situation for consumers of these packages? Binary size before/after, or go.mod or go.sum changes that this enables?

1. Is there any value in moving any of these to `sigstore/sigstore`?

I am cool with moving them to sigstore/sigstore but do not feel strongly.

2. To avoid breaking users (I'm not sure how many there even are outside of `cosign`...), is it worth keeping the existing consts, pointing to the new consts, with a `Deprecated` comment?

Sure thing.  I am unsure about the stability guarantee of the changed code, so I opted for the lazy version :^)

3. Is there any indication of how much this improves the situation for consumers of these packages? Binary size before/after, or `go.mod` or `go.sum` changes that this enables?

If users only care about these consts we go from importing go.mod to importing zero. pkg/types just has a couple of consts.

Is there any value in moving any of these to sigstore/sigstore?

FWIW I think, ultimately, external users should not have to individually enumerate these annotations. They should provide/receive one of:

Something like an OCI descriptor + contents, as a single unit (similar to the registry representation)
A map[string]string annotation values
Some kind of (serializable?) object that represents everything about a signature as a single struct

(and the API should probably allow at least two of these).
If Cosign decides to add one more annotation (or remove one, as has recently happened with the “timestamp” annotations), I think most users should not need to care unless it specifically affects the secret credentials, or the verification policy, they need to provide.
If the sign/verify implementation moves to sigstore/sigstore, the annotation key constants should fairly likely move there as well — but it’s not obvious to me that they, long-term, need to be a public API at all.
Unfortunately we can’t practically get to such a design in a single step. So making step-wise changes to the API in the Cosign repo is much more practical (we don’t need to keep two independent repos and PRs in sync); and after we have something we are happy with, we can move that to sigstore/sigstore as a larger unit.

That sgtm @mtrmac

SGTM
Let's do the consolidation once the work in sigstore/sigstore is done.


MVC Template throws exception on deploy

Describe the bug
Get this error message when trying to deploy MVC Template

Unhandled exception.  This is a bug.  Please copy the stack trace below and file a bug at https://github.com/aws/aws-dotnet-deploy.
Source sequence doesn't contain any elements.
at System.Linq.AsyncEnumerable.g__Core|283_0[TSource](IAsyncEnumerable1 source, CancellationToken cancellationToken) in d:\a\1\s\Ix.NET\Source\System.Linq.Async\System\Linq\Operators\First.cs:line 24    at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.<>c__DisplayClass26_0.<<GetDefaultVpc>b__0>d.MoveNext() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479 --- End of stack trace from previous location ---    at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.HandleException[T](Func1 action) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 887
at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.GetDefaultVpc() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479
at AWS.Deploy.Orchestration.Orchestrator.ApplyAllReplacementTokens(Recommendation recommendation, String cloudApplicationName) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Orchestrator.cs:line 222
at AWS.Deploy.CLI.Commands.DeployCommand.InitializeDeployment(String cloudApplicationName, DeploymentSettings deploymentSettings, String deploymentProjectPath) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 250
at AWS.Deploy.CLI.Commands.DeployCommand.ExecuteAsync(String applicationName, String deploymentProjectPath, DeploymentSettings deploymentSettings) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 118
at AWS.Deploy.CLI.Commands.CommandFactory.b__43_0(DeployCommandHandlerInput input) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/CommandFactory.cs:line 255

Expected Behavior
Deploy MVC template to AWS
Current Behavior
Get error message

Unhandled exception.  This is a bug.  Please copy the stack trace below and file a bug at https://github.com/aws/aws-dotnet-deploy.
Source sequence doesn't contain any elements.
at System.Linq.AsyncEnumerable.g__Core|283_0[TSource](IAsyncEnumerable1 source, CancellationToken cancellationToken) in d:\a\1\s\Ix.NET\Source\System.Linq.Async\System\Linq\Operators\First.cs:line 24    at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.<>c__DisplayClass26_0.<<GetDefaultVpc>b__0>d.MoveNext() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479 --- End of stack trace from previous location ---    at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.HandleException[T](Func1 action) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 887
at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.GetDefaultVpc() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479
at AWS.Deploy.Orchestration.Orchestrator.ApplyAllReplacementTokens(Recommendation recommendation, String cloudApplicationName) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Orchestrator.cs:line 222
at AWS.Deploy.CLI.Commands.DeployCommand.InitializeDeployment(String cloudApplicationName, DeploymentSettings deploymentSettings, String deploymentProjectPath) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 250
at AWS.Deploy.CLI.Commands.DeployCommand.ExecuteAsync(String applicationName, String deploymentProjectPath, DeploymentSettings deploymentSettings) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 118
at AWS.Deploy.CLI.Commands.CommandFactory.b__43_0(DeployCommandHandlerInput input) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/CommandFactory.cs:line 255

Reproduction Steps
dotnet new web -o AWSDeploy
cd AWSDeploy
dotnet aws deploy
AWS .NET deployment tool for deploying .NET Core applications to AWS.
Project Home: https://github.com/aws/aws-dotnet-deploy

Configuring AWS Credentials using AWS SDK credential search.
Configuring AWS region using AWS SDK region search to us-west-2.
Select an existing AWS deployment target to deploy your application to.
-----------------------------------------------------------------------
1: MVCTemplate (CloudFormation Stack) (default)
2: *** Deploy to a new Cloud Application ***
Choose deployment target (default 1):
1
Loading MVCTemplate (CloudFormation Stack) settings:
----------------------------------------------------
Elastic Beanstalk Application: MVCTemplate
Environment Name:
        Environment Name: MVCTemplate-dev
Environment Type: SingleInstance
Application IAM Role: *** Create new ***
Service IAM Role: *** Create new ***
Beanstalk Platform: arn:aws:elasticbeanstalk:us-west-2::platform/.NET Core running on 64bit Amazon Linux 2/2.3.3
Managed Platform Updates:
        Enable Managed Platform Updates: True
        Preferred Start Time: Sun:00:00
        Update Level: minor
Enable AWS X-Ray Tracing Support: False
Reverse Proxy: nginx
Environment Variables:
Virtual Private Cloud (VPC):
        Use a VPC : False
Dotnet Build Configuration: Release
Self Contained Build: False
Unhandled exception.  This is a bug.  Please copy the stack trace below and file a bug at https://github.com/aws/aws-dotnet-deploy.
Source sequence doesn't contain any elements.
   at System.Linq.AsyncEnumerable.<FirstAsync>g__Core|283_0[TSource](IAsyncEnumerable`1 source, CancellationToken cancellationToken) in d:\a\1\s\Ix.NET\Source\System.Linq.Async\System\Linq\Operators\First.cs:line 24
   at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.<>c__DisplayClass26_0.<<GetDefaultVpc>b__0>d.MoveNext() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479
--- End of stack trace from previous location ---
   at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.HandleException[T](Func`1 action) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 887
   at AWS.Deploy.Orchestration.Data.AWSResourceQueryer.GetDefaultVpc() in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Data/AWSResourceQueryer.cs:line 479
   at AWS.Deploy.Orchestration.Orchestrator.ApplyAllReplacementTokens(Recommendation recommendation, String cloudApplicationName) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.Orchestration/Orchestrator.cs:line 222
   at AWS.Deploy.CLI.Commands.DeployCommand.InitializeDeployment(String cloudApplicationName, DeploymentSettings deploymentSettings, String deploymentProjectPath) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 250
   at AWS.Deploy.CLI.Commands.DeployCommand.ExecuteAsync(String applicationName, String deploymentProjectPath, DeploymentSettings deploymentSettings) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/DeployCommand.cs:line 118
   at AWS.Deploy.CLI.Commands.CommandFactory.<BuildDeployCommand>b__43_0(DeployCommandHandlerInput input) in /codebuild/output/src830051283/src/codestar-connections.us-west-2.amazonaws.com/git-http/393431854674/us-west-2/eb780f07-2f46-42e0-9c15-df6f00fe3c1c/aws/aws-dotnet-deploy/src/AWS.Deploy.CLI/Commands/CommandFactory.cs:line 255

Possible Solution
No response
Additional Information/Context
No response
Version used
1.1.15+5c57ebd489
Operating System and version
Windows 11

Can you try updating the deploy tool to the latest version and try again?

It doesn't error but it hangs on the CDK step

If I close and try again I get this

dotnet aws deploy
AWS .NET deployment tool for deploying .NET Core applications to AWS.
Project Home: https://github.com/aws/aws-dotnet-deploy

Configuring AWS Credentials using AWS SDK credential search.
Configuring AWS region using AWS SDK region search to us-west-2.
Select an existing AWS deployment target to deploy your application to.
1: MVCTemplate (CloudFormation Stack) (default)
2: *** Deploy to a new Cloud Application ***
Choose deployment target (default 1):
1
Loading MVCTemplate (CloudFormation Stack) settings:
Elastic Beanstalk Application: MVCTemplate
Environment Name:
Environment Name: MVCTemplate-dev
Environment Type: SingleInstance
Application IAM Role: *** Create new ***
Service IAM Role: *** Create new ***
Beanstalk Platform: arn:aws:elasticbeanstalk:us-west-2::platform/.NET Core running on 64bit Amazon Linux 2/2.3.3
Managed Platform Updates:
Enable Managed Platform Updates: True
Preferred Start Time: Sun:00:00
Update Level: minor
Enable AWS X-Ray Tracing Support: False
Reverse Proxy: nginx
Environment Variables:
Virtual Private Cloud (VPC):
Use a VPC : False
Dotnet Build Configuration: Release
Self Contained Build: False
Current settings (select number to change its value)

Environment Variables:

Enter 'more' to display Advanced settings.
Or press 'Enter' to deploy:

Creating deployment zip bundle
Using the dotnet CLI build the project and zip the publish artifacts.

Creating Dotnet Publish Zip file...
MSBuild version 17.3.0+92e077650 for .NET
Determining projects to restore...
All projects are up-to-date for restore.
AWSDeploy -> D:\dev\AWSDeploy\bin\Release\net6.0\AWSDeploy.dll
AWSDeploy -> C:\Users\isaac\AppData\Local\Temp\b56715fa-93e9-4aa3-8f55-a204ff61be98\

Configuring AWS Cloud Development Kit (CDK)
Ensure a compatible CDK version is installed and the CDK bootstrap CloudFormation stack has been created. A CDK project to perform the deployment is generated unless an existing deployment project was selected.

Generating AWS Cloud Development Kit (AWS CDK) deployment project
Confirmed CDK Bootstrap CloudFormation stack already exists.

Deploying AWS CDK project
Use the CDK project to create or update the AWS CloudFormation stack and deploy the project to the AWS resources in the stack.

npm WARN config global --global, --local are deprecated. Use --location=global instead.
✨  Synthesis time: 19.49s
MVCTemplate: deploying...
[0%] start: Publishing 3e38191482508526b7142519b6af723206a51e0955f51af5b2ab43db4ba80928:563011967245-us-west-2
[0%] start: Publishing 4021cdec9e544860a835dc518dedcbd433b48d5c35e6b1ba35716b4c3250d463:563011967245-us-west-2
[50%] success: Published 4021cdec9e544860a835dc518dedcbd433b48d5c35e6b1ba35716b4c3250d463:563011967245-us-west-2
[100%] success: Published 3e38191482508526b7142519b6af723206a51e0955f51af5b2ab43db4ba80928:563011967245-us-west-2
MVCTemplate: creating CloudFormation changeset...
❌  MVCTemplate failed: Error [ValidationError]: Stack:arn:aws:cloudformation:us-west-2:563011967245:stack/MVCTemplate/8cf71b80-1513-11ed-8d1a-0689948d0e75 is in UPDATE_IN_PROGRESS state and can not be updated.
at Request.extractError (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\protocol\query.js:50:29)
at Request.callListeners (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\sequential_executor.js:106:20)
at Request.emit (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\sequential_executor.js:78:10)
at Request.emit (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\request.js:686:14)
at Request.transition (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\request.js:22:10)
at AcceptorStateMachine.runTo (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\state_machine.js:14:12)
at C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\state_machine.js:26:10
at Request. (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\request.js:38:9)
at Request. (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\request.js:688:12)    at Request.callListeners (C:\Users\isaac\AppData\Roaming\npm\node_modules\aws-sdk\lib\sequential_executor.js:116:18) {
code: 'ValidationError',
time: 2022-08-26T00:07:32.821Z,
requestId: 'c658fd98-ed42-4b7e-b567-d1495adfd810',
statusCode: 400,
retryable: false,
retryDelay:<PHONE_NUMBER>7350196
}
Stack:arn:aws:cloudformation:us-west-2:563011967245:stack/MVCTemplate/8cf71b80-1513-11ed-8d1a-0689948d0e75 is in UPDATE_IN_PROGRESS state and can not be updated.
We had an issue deploying your application to AWS. Check the deployment output for more details. Deployment took 25.24s.
For more information, please visit our troubleshooting guide https://aws.github.io/aws-dotnet-deploy/troubleshooting-guide/.
If you are still unable to solve this issue and believe this is an issue with the tooling, please cut a ticket https://github.com/aws/aws-dotnet-deploy/issues/new/choose.

The reason why you get the error you got when you closed it and tried again is because CDK already pushed the changeset to the AWS account and the process is completing there. Even if you close the tool, the CloudFormation stack is still being updated. The tool is only showing you the update status. I propose you go to CloudFormation in your AWS account and look for the stack you are deploying. Can you report back the status of the stack as well as any errors shown in the events tab?

The stack failed on deployment and I had to completely delete it. Is it possible to provide a more helpful message to the user? The whole process seems to be a little opaque with what is actually going on. If we can provide a clean message with resolution steps, that would be helpful.

The original issue based on the first stack trace should be fixed in version 1.2.4, which addressed the case of a deployment failure when deploying to an account+region without any existing VPCs.
I agree we can improve detection/messaging when attempting to deploy to a stack that is still being updated, we've changed this to a feature request.

Closing due to lack of traction.


Added coverage geometries for Canadian places

Can’t find a new URL for the ca/bc/prince_george source; otherwise looks good.


Support more content types for Triple protocol

What is the purpose of the change
This PR can provide the capacity to decode more HTTP Content-Types for providers.
Brief changelog
Add more HttpMessageCodec to support followding content type:

multipart/formdata
text/plain
application/x-www-form-urlencoded
application/xml

https://github.com/apache/dubbo/issues/13386
The HTTP response is now encoded based on the HTTP headers (accept & grpc-accept-encoding), rather than being limited to JSON or PB.

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

Comparison is base (a53228d) 40.40% compared to head (383cc56) 40.32%.

Additional details and impacted files
@@            Coverage Diff             @@
##              3.3   #13387      +/-   ##
==========================================
- Coverage   40.40%   40.32%   -0.09%
==========================================
  Files        1807     1815       +8
  Lines       75867    76040     +173
  Branches    10642    10662      +20
==========================================
+ Hits        30657    30665       +8
- Misses      40911    41058     +147
- Partials     4299     4317      +18

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

form-data is bak;multipart/formdata、text/plain、application/x-www-form-urlencoded is ok

What version? you can create a separate bug ticket.
@crazyStar00

What version? you can create a separate bug ticket. @crazyStar00

dubbo version: 3.3.0-beta.2