httpx: 1.3.7 -> 1.3.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/projectdiscovery/httpx/releases.
meta.description for httpx is: Fast and multi-purpose HTTP toolkit
meta.homepage for httpx is: https://github.com/projectdiscovery/httpx
meta.changelog for httpx is: https://github.com/projectdiscovery/httpx/releases/tag/v1.3.8
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.8 with grep in /nix/store/5r7pqm9c185fs4v4lrpyq7q7i2qna57f-httpx-1.3.8
found 1.3.8 in filename of file in /nix/store/5r7pqm9c185fs4v4lrpyq7q7i2qna57f-httpx-1.3.8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
httpx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5r7pqm9c185fs4v4lrpyq7q7i2qna57f-httpx-1.3.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A httpx https://github.com/r-ryantm/nixpkgs/archive/de1b7af74c06b902ffc5c3d48c5ed87c054aa6c0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/de1b7af74c06b902ffc5c3d48c5ed87c054aa6c0#httpx

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5r7pqm9c185fs4v4lrpyq7q7i2qna57f-httpx-1.3.8
ls -la /nix/store/5r7pqm9c185fs4v4lrpyq7q7i2qna57f-httpx-1.3.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

httpx

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 281291 run on aarch64-darwin 1

1 package built:

httpx


Move Automations Library into docs

Create a new automations library inside the docs.

Migrate contents of /automations into the docs site.
Update Quickstart example page to optimize it for the new library
Add automations/automation-library to site nav

Sorry about the wonky commits, I'm not sure what the deal was.


mapproxy: 1.13.0 -> 1.13.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mapproxy is: "Open source proxy for geospatial data"
meta.homepage for mapproxy is: "https://mapproxy.org/"
meta.changelog for mapproxy is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2/bin/.mapproxy-seed-wrapped passed the binary check.

/nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2/bin/.mapproxy-util-wrapped passed the binary check.

/nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2/bin/mapproxy-seed passed the binary check.

/nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2/bin/mapproxy-util passed the binary check.

2 of 4 passed binary check by having a zero exit code.

2 of 4 passed binary check by having the new version present in output.

found 1.13.2 with grep in /nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2

found 1.13.2 in filename of file in /nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2

directory tree listing: https://gist.github.com/6063949f09e485ea313f5ba8b306679f

du listing: https://gist.github.com/c0ca860c7c7bfdc3cf9e989dbbf1e948

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mapproxy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mapproxy https://github.com/r-ryantm/nixpkgs/archive/a8d8fab7b73a381f8830773cfbe7ede5d2bdfd8b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2
ls -la /nix/store/zdik4hijm3mkvs69vmgr7yycac3rq6d0-MapProxy-1.13.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mapproxy

Maintainer pings
cc @rakesh4g for testing.

Result of nixpkgs-review pr 134084 at a8d8fab7 run on aarch64-linux 1

1 package failed to build:

mapproxy

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Add make update-deps

Add a simple make target to update go deps in one command

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
1971

Relevant Lines:
5101

💛  - Coveralls


Revert change to sf::Drawable::draw signature which made sf::RenderStates a const reference, and make it a copy again

Prerequisite Checklist

[X] I searched for existing issues to prevent duplicates
[X] I searched for existing discussions on the forum to prevent duplicates
[X] I understand SFML's scope and believe the feature fits it

Describe your feature request here
I've noticed that the change around the signature of sf::Drawable::draw in SFML master for v3 does not improve the users experience when implementing custom drawable types, especially if they inherit from sf::Transformable.
For example, I was producing a custom type to represent a particle emitter. I want to be able to position the emitter and scale it, so I made it inherit from sf::Transformable, and to render it I made it inherit from sf::Drawable. In 2.x I would have been able to adjust the sf::RenderStates parameter to set the transform (see API example below) without making a copy, but when using SFML master the signature has changed to a const reference parameter. This makes the more shorter more appealing form of the method I'd like to write impossible, and forces me to produce a copy.
I don't think the magnitude of the original change was severe, but there was a useful pattern that emerged from the previous version that I think lots of poweruser with existing projects (using 2.x) would like to retain.
Use Cases
The usecases for this are that people who inherit from sf::Drawable will often want to adjust the value of the sf::RenderStates parameter (to apply shaders, textures, transform etc).
With the current signature a user is forced to make a copy themselves, where previously this was already done by the method signature taking a parameter.
API Example
// Revert from:
void MyCustomType::draw(sf::RenderTarget& target, const sf::RenderStates& states) const
{
    sf::RenderStates statesCopy = states;
    statesCopy.transform *= getTransform();
    target.draw(m_vertexBuffer, statesCopy);
}

// Back to:
void MyCustomType::draw(sf::RenderTarget& target, sf::RenderStates states) const
{
    states.transform *= getTransform();
    target.draw(m_vertexBuffer, states);
}

While I'm all for some theoretically performance gains, we're not sure if those performance gains are meaningful compared to the very real ergonomic issues this change has introduced. I'm open to reverting this.

sf::RenderStates is quite big as it contains a sf::Transform, plus other 5 members of integer size. We'd be copying sizeof(float) * 16 + sizeof(int) * 5 every time we invoke draw. This could be significant if users are drawing many drawables in a loop.
Is the explicit copy really that annoying? If anything, it shows that quite a bit of data is being copied explicitly.
This API is kinda weird though, maybe this is a symptom of a larger problem? Can RenderStates be taken by non-const reference?

sf::RenderStates is quite big as it contains a sf::Transform, plus other 5 members of integer size. We'd be copying sizeof(float) * 16 + sizeof(int) * 5 every time we invoke draw. This could be significant if users are drawing many drawables in a loop.
Is the explicit copy really that annoying? If anything, it shows that quite a bit of data is being copied explicitly.
This API is kinda weird though, maybe this is a symptom of a larger problem? Can RenderStates be taken by non-const reference?

I would suppose there are implications in it being non-const in effecting draw calls subsequent to your custom drawable. I think the API is rather strange but has previously made for nice ergonomics. Anyone making custom drawables is likely to be manipulating the RenderStates object and thus they will have to eat the cost of the copy anyway. And honestly the draw itself is more costly (also more problematic in immediate mode rather than the model favoured by later GL versions and other rendering APIs. I also think it's likely that powerusers who make use of sf::Drawable likely have some scheme of batching which means the cost of the copy is not so bad but the removal of the explicit copy is favourable for readability. And in general the issue with the copy on lots of drawables would be better sorted with batching (considering that the GL draw call itself is the largest overhead).
I may be speaking largely for myself in this issue, which is why I raised it, maybe I should merely avoid the sf::Drawable base but I've utilised it in many projects for the smooth ergonomics it offers when performing a draw on my own custom types (as it was intended for). Would be nice to see some other heavy SFML users chime in with their viewpoint.

For whatever it's worth, I agree with Bambo. The cost of a draw call is enormous, I don't think that the gains are going to meaningfully offset the still enormous cost of a draw call. You'd have to have A LOT of draw calls I imagine, at which point you've already lost.

For whatever it's worth, I agree with Bambo. The cost of a draw call is enormous, I don't think that the gains are going to meaningfully offset the still enormous cost of a draw call. You'd have to have A LOT of draw calls I imagine, at which point you've already lost.

That is fair, and you are right.
Still, I'd take this as an opportunity to figure out why the API is so awkward in the first place before jumping on a revert. Maybe we can do something even better, but not opposed to a revert.

For whatever it's worth, I agree with Bambo. The cost of a draw call is enormous, I don't think that the gains are going to meaningfully offset the still enormous cost of a draw call. You'd have to have A LOT of draw calls I imagine, at which point you've already lost.

That is fair, and you are right.
Still, I'd take this as an opportunity to figure out why the API is so awkward in the first place before jumping on a revert. Maybe we can do something even better, but not opposed to a revert.

While I feel there is merit in deeper interrogation of the drawing API, I think it's largely out of the scope of the issue raised here. I think realigning the API to work as it did in 2.0 via a revert so that the ergonomics are kept the same presently is worth it. And revisiting the API discussion when the move away from legacy GL is an option is likely the best time. We're better off bringing this back to how it was before prior to a release of 3.0.

For whatever it's worth, I agree with Bambo. The cost of a draw call is enormous, I don't think that the gains are going to meaningfully offset the still enormous cost of a draw call. You'd have to have A LOT of draw calls I imagine, at which point you've already lost.

That is fair, and you are right.
Still, I'd take this as an opportunity to figure out why the API is so awkward in the first place before jumping on a revert. Maybe we can do something even better, but not opposed to a revert.

While I feel there is merit in deeper interrogation of the drawing API, I think it's largely out of the scope of the issue raised here. I think realigning the API to work as it did in 2.0 via a revert so that the ergonomics are kept the same presently is worth it. And revisiting the API discussion when the move away from legacy GL is an option is likely the best time. We're better off bringing this back to how it was before prior to a release of 3.0.

I'm not talking about a major change to the API. I'm saying that if -- for instance -- it makes more sense to pass sf::RenderStates as a non-const reference, this is the right time to make the change (prior to 3.0 release).

What hasn't been mentioned is that the fix was original done because it popped up in static code analysis tools, see the original PR #2008
Unfortunately, due to Coverity having the worst UI ever, I don't know if we can still see what it complained about exactly. Maybe @binary1248 still remembers or can dig up the original report.
Performance is in my view a non-issue. We've been living with this in SFML 2 for the past ten years or so and I don't remember having people run into a bottleneck with this part.
For me it's generally fine to revert the change.
Yes, maybe there's a more ergonomic API and since it probably would be breaking, it would be the "right time". On the other hand, it might also not be the right time, as we may come up with quite a different rendering API for SFML 4 and having SFML 3 be a quality of life release for C++17 support, I believe we don't need a major API revamp for this.

See https://github.com/SFML/SFML/pull/2949

It was SonarCloud and yes... they truly have one of the most unusable user interfaces in recent memory.


Split IYAXSerializer<T> to IYAXSerializer<T>,  IYAXDeserializer<T> and IYAXMultiStateDeserializer<T>

Added also IYAXMultiStateDeserializer<T> otherswise IYAXDeserializer<T> can't be covariant

Really an excellent way to distinguish

Unavoidable, but only 2 conflicts easy to resolve ;-)

I'm sorry, busy weeks. Will try to work on the pull requests this weekend

Will reopen when I have more time


recent_topic: Add timestamp updating interval.

Fixes: #23005.

Just a note that this will need a rebase after #25037.

@shameondev left a comment, LGTM otherwise.

Hey, @timabbott. Please tell me what changes you think we should make in this PR so we can move forward. Thanks!

@shameondev Could you please rebase this PR?

Resolved merge conflicts!

Addressed the feedback about module conversion; ready for review.

Looks like I forgot to mention that we'd merged the TypeScript conversion part as 84c1a7dc2e6a183a3d97d90d1e0099cb1308edbb.

Closing in favour of #27296


Fix spelling

Hi,
This is a brief pull request to fix the spelling of "queries" in a filename, and one import of that file.
Kind regards,
Israel

Is anyone out there? :)

@hhurtado


[MBL-806] Update (Remove) WebViewViewModel

📲 What
Remove the viewmodel for this activity, it wasnt doing anything
🤔 Why
RX1 is old and busted, making room for the new hotness
🛠 How
✂
📋 QA
Test pages that are currently web view controlled (creator bio, updates, help, etc)
Story 📖
MBL-806

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

Comparison is base (10a75aa) 74.27% compared to head (8c247c8) 74.25%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1925      +/-   ##
============================================
- Coverage     74.27%   74.25%   -0.03%
  Complexity     1963     1963
============================================
  Files           337      336       -1
  Lines         19672    19653      -19
  Branches       2718     2718
============================================
- Hits          14612    14593      -19
  Misses         3490     3490
  Partials       1570     1570

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


[FFM-6854][FFM-6842] Update SDK release notes

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-652

Preview environment: https://ci.pr.harness.io/pr-652


imp: make event emission functions unexported

Description
Most of the event emission functions can be unexpected.
closes: #XXXX
Commit Message / Changelog Entry
imp: make event emission functions unexported

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[x] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

I think I could also make EmitAcknowledgementEvent unexported if we implement OnRecvPacket in the controller keeper and move this code from the middleware to the keeper.

Codecov Report

Merging #3205 (306fbfe) into main (761277a) will increase coverage by 0.00%.
The diff coverage is 80.45%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3205   +/-   ##
=======================================
  Coverage   78.60%   78.60%
=======================================
  Files         177      177
  Lines       12410    12420   +10
=======================================
+ Hits         9755     9763    +8
- Misses       2230     2231    +1
- Partials      425      426    +1

Impacted Files
Coverage Δ

modules/core/02-client/keeper/events.go
88.46% <33.33%> (ø)

modules/apps/transfer/ibc_module.go
64.81% <40.00%> (-0.16%)
:arrow_down:

modules/apps/transfer/keeper/relay.go
90.63% <60.00%> (ø)

...les/apps/27-interchain-accounts/host/ibc_module.go
91.11% <100.00%> (+1.11%)
:arrow_up:

...s/apps/27-interchain-accounts/host/keeper/relay.go
71.08% <100.00%> (ø)

modules/apps/29-fee/keeper/escrow.go
92.81% <100.00%> (ø)

modules/apps/29-fee/keeper/events.go
100.00% <100.00%> (ø)

modules/apps/29-fee/keeper/msg_server.go
97.14% <100.00%> (ø)

modules/core/02-client/keeper/client.go
94.28% <100.00%> (ø)

modules/core/02-client/keeper/proposal.go
80.00% <100.00%> (ø)

... and 7 more


[client] Do not store empty plist file to the server

Closes #2659

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- web/client/codechecker_client/cmd/store.py  1

See the complete overview on Codacy

Hi,
this patch removed the error for me. In the webinterface the run is visible with the same number of defects than presented with CodeChecker parse.
Thank you very much !

@gyorb You are right. I fixed it and I have also added a new test case which will check the storage of such plist files.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- web/client/codechecker_client/cmd/store.py  1

Clones removed
==============
+ web/tests/functional/skip/test_skip.py  -5

See the complete overview on Codacy

@gyorb As you wrote the problem is that at here the reports are not parsed yet so it is not known which file should be stored or not.
Also I do not think we should handle this use case. This bug was introduced in the master branch and there is no such CodeChecker version which can produce such a plist file. With this patch this problem will be solved so in the new CodeChecker release it will work properly.
With this patch the content_hashes.json will not contain any extra files. If the diagnostics section is empty in a plist file then no source files will be added to the content_hashes.json file.


get in BitMatrix goes out of bounds for Maxicode in certain image.

The image below makes the library crash because the algorithm gives a point outside of the image. I have already made a fix, and the PR is on its way.

Thanks for the PR, I'm reviewing it now.

I am going to merge #29, and adjust the internal working of the BitMatrix struct to return false if a requested point is out of bounds. I will document in the comments. I don't know if I think it's exactly the correct way to handle things, but the idea of having the entire library have to handle an Option or Result for every get or get_point seems sorta bad.
In the original java they just let the ArrayOutOfBoundException fly and caught it as part of their regular error handling, sometimes using it to detect if a transform had resulted in an impossible final set of points.
I'm open to revisiting this if it is the wrong choice.

I also added this image as test case github_issues::issue_28


Add more preservation policies to the website

Add information to the website about what we're doing to preserve our content (e.g., web archiving through Internet Archive, Zenodo). This could be added to the existing preservation policy area.

@swipperman if you already have some content for this, feel free to edit the text on the site here:
https://github.com/oaworks/oa-works/blob/main/src/policies/preservation.md
Let me know if you need a quick intro to Markdown and editing on Github :)

@sssoz I don't have any specific content at the moment but am happy to help write some up! I could absolutely use a quick intro to Markdown/editing on GitHub if you are willing/have the time. It's been several years since I've done it.
I just went in and created a few minor edits, but then I realized I'd be forking the project because I don't have edit privileges, got scared, and cancelled the changes.

@swipperman Let’s have a quick re-intro next week then :)

Ooops, I’ll give you editing rights on the dev site.

Let's do it! I'll suggest a time on our calendars. Thanks, Sophy!
On Wed, Feb 23, 2022 at 6:10 PM Sophy @.***> wrote:

Ooops, I’ll give you editing rights on the dev site.
—
Reply to this email directly, view it on GitHub
https://github.com/oaworks/oa-works/issues/133#issuecomment-1049305593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJRYQGVD3LJMGBAQKAEQC3U4VSNLANCNFSM5IRQTNLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


java/kotlin/android intellij set up documentation

Write documentation on how we can set up an intellij IDE for development

Currently using some of these commits: https://github.com/lyft/envoy-mobile/tree/57fb4d405d11c89f028b10e6e00c7b5aa3d8ddd2 in this branch: https://github.com/lyft/envoy-mobile/tree/hack-for-intellij


orders / option "immediate_or_cancel" not working

orders/ "immediate_or_cancel" flag not working.
As far as I can tell, I am setting this parameter correct to true, but in the transaction it turns up as false and the order is placed in the ledger.

I'm unable to reproduce. Can you provide explicit instructions for reproducing the issue?

When the order is placed (it is placed...which should not if I understand correct), then the value "immediate_or_cancel" is false.
I'm using the following code (in bash shell script on linux):
Trade account (fill in your account)
TRADE_ACCOUNT="xxx"
USD.SnapSwap
TAKER_GETS_CURRENCY=USD
TAKER_GETS_COUNTERPARTY="rMwjYedjc7qqtKYVLiAccJSmCwih4LnE2q"
USD.Bitstamp
TAKER_PAYS_CURRENCY=USD
TAKER_PAYS_COUNTERPARTY="rvYAfWj5gh67oV6fW32ZzP3Aw4Eubs59B"
USD.SnapSwap
TAKER_GETS_VALUE=$(awk 'BEGIN { printf "%.6f", ('$AMOUNT') }')
USD.Bitstamp
TAKER_PAYS_VALUE=$(awk 'BEGIN { printf "%.6f", ('$PRICE'*'$AMOUNT') }')
Secret (fill in secret)
SECRET="xxx"
ORDER_PARAMS='
{
"secret" : "'$SECRET'",
"immediate_or_cancel" : true,
"order" : {
"type" : "sell",
"taker_pays" : {
"currency" : "'$TAKER_PAYS_CURRENCY'",
"counterparty" : "'$TAKER_PAYS_COUNTERPARTY'",
"value" : "'$TAKER_PAYS_VALUE'"
},
"taker_gets" : {
"currency" : "'$TAKER_GETS_CURRENCY'",
"counterparty" : "'$TAKER_GETS_COUNTERPARTY'",
"value" : "'$TAKER_GETS_VALUE'"
}
}
}'
RESULT=$(curl   -s
-k
-H Content-Type:application/json
-H application/json
-X POST
-d $ORDER_PARAMS
https://<IP_ADDRESS>:5990/v1/accounts/$TRADE_ACCOUNT/orders?validated=true)

In your script, immediate_or_cancel is outside the order object. Try moving it inside the order object.

ok thanks! I have tried and it works :+1:
sorry if it's not a bug ;-) sometimes hard to tell where to place an issue. I guess in this case it would better be phrased as a question on the forum..
Thanks again for your help
rgds.
Jarl


Layer event handlers are not being triggered as expected when using type raster

Hello everybody,
I'm trying to use the OnClick,OnMouseEnter,OnMouseLeave event handlers within the Layer component and they are not doing anything, i'm rendering images on the map using a source of type image and a layer of type raster and want to be able to click the image but when I click nothing happens! Here is how i'm implementing it:
const renderedImagesLayers = renderedImagesData.map((data) =>
            <Layer
            id={data.id}
            type={data.type}
            sourceId={data.sourceId}
            before={data.before}
            onMouseEnter={e => this.onToggleHover(e,'pointer')}
            onMouseLeave={e => this.onToggleHover(e,'')}
            onClick={e=>console.log(e)}
            ></Layer>
        )

I'm using the onClick method in the Map component as well, can this be causing the above issue?
I'm using the following versions:
react-mapbox-gl: "4.6.0"
react: "16.9.0",

Hello, I have the same issue, no event handlers trigger when using Layer component of type "raster"
      <Source
        id="source_id"
        tileJsonSource={RASTER_SOURCE_OPTIONS}
      />
      <Layer
        type="raster"
        id="layer_id"
        sourceId="source_id"
        onClick={() => console.log('layer click')}
      />

Same here, Mouse enter/leave and click are not triggered properly:
<Source id="source_id" tileJsonSource={Image_SOURCE_OPTIONS} />
                <Layer type="raster" id="layer_id" sourceId="source_id"
                    onMouseEnter={(map) => console.log('mouse enter', map)}
                    onMouseLeave ={(map) => console.log('mouse leave', map)}
                    onClick ={(map) => console.log('onclick', map)}
                />

I got the same problem with Layer and Feature.

Here is a workaround I used to avoid this bug by using a transparent GeoJSONLayer over my raster with the same size:
const polygonGeoJSON = {
                "type": "FeatureCollection",
                "features": [{
                    "id": "ID",
                    "type": "Feature",
                    "properties": {
                        "sourceId": SOURCE_ID
                   },
                    "geometry": {
                        "type": "Polygon",
                        "coordinates": YOUR_COORDINATES
                    }
                }]
}
const fillLayout = { visibility: 'visible' };
const fillPaint = {
    'fill-color': '#FFFFFF',
    'fill-opacity': 0
};

<GeoJSONLayer
     id={'geojson-' + IMAGE_KEY.id}
     data={polygonGeoJSON}
     fillLayout={fillLayout}
     fillPaint={fillPaint}
     fillOnClick={YOU_FUNCTION}
     fillOnMouseEnter={e => this.onToggleHover(e, 'pointer')}
     fillOnMouseLeave={e => this.onToggleHover(e, '')}
/>


Avinya Systems Architecture

Systems architecture for the Avinya stack.

The following pictures are from systems architecture discussions on 9/16:

API design

Organizational structure and logical role-based segmentation

Domain and tenant model

[ ] Create initial draft of systems architecture and discussion items from 9/16


Feature request: Pre Action before opening Software Update window

The ability to add an action that runs after user presses Update Device and before the software update opens. It's similar to actionButtonPath but after resolving the action it continues with the build in dynamic capability for updateDevice.
The though behind this is the feature to add a logic that elevates users too administrators if needed. We use Privileges in our environment, this feature would ensure us the user is admin when entering software update.

I'm not sure how to accommodate this as I would have no way for Nudge to understand when another tool outside of its control is complete. I don't have any interesting coding specific things to Privileges, so unless you can outline what generic ruleset Nudge could use to determine this, I will likely close this feature request.

If you're asking users to update their device (like, from 12.3 to 12.3.1), that does not require admin privileges - only volume owner credentials if the device has an Apple Silicon chip. Only OS upgrades (example: upgrading from Big Sur to Monterey) require admin privileges.

I am unfortunately not familiar with Swift. I was thinking that the Pre Action as a shell command that does what we want then finishes. After that Update Device would trigger. Nudge wouldn't need to know what the Pre Action actually did. It just waits for the Pre Action to finish and continues as usual after that.

Do you need Adminrights for the shell command, to give the user admin rights with SAP Privileges APP? Then it is not possible because nudge only runs in the User Context.
You could bundle the update device button with a Jamf policy or Munki policy to achieve this.

The command doesn't need admin rights. I want to run open /Applications/Privileges.app which prompts the user to elevate their account to admin.

@erikng : this is no longer needed for non-admin users from macOS 12.3 onward.


commented out code in cookie cutter utils.rs references verstion 37.0

I just noticed that the plugin cookiecutter has commented out code for a function that accepts two columns, and it references an older version than the rest of the polars dependencies in the Cargo.toml (0.41.2).  Figured it was worth mentioning.
// and make sure you have
//
//     polars-arrow = { version = "0.37.0", default-features = false }
//     polars-core = { version = "0.37.0", default-features = false }
//
// in your Cargo.toml file.

yup, thanks @ATL2001 !


instruction to manually run UI and docker

@Fluder-Paradyne is this ready to be reviewed and merged?


Build dependencies are listed as runtime dependencies

This results in a very large manifest file and a 67MB zip file for a py_binary running on windows that uses a dotnet_binary as a data dependency.

todo: investigate this with current implementation

With the recent trimming of sdk files, I believe this is no longer an issue.


docs: Fixed broken link

Description
Just replaced a broken link with one that works.
Related Issues
https://github.com/firebase/flutterfire/issues/10590
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x ] No, this is not a breaking change.

We've already merged another PR that fixes this issue. Thanks 👍


[Feature Request] - Support SCAN of Redis or at least "bad Keys*"

Hi there :)
Thanks for this awesome package.
May I ask to improve this package by implementing the SCAN functionality of Redis ?
Because I want to find certain keys within my redis store by Pattern, and as I read here: KEYS * is bad: https://stackoverflow.com/questions/40854654/redis-node-delete-with-prefix-wildcard
Thank you in advance :)
Meywether

I don't think I want to add any more Redis-specific commands so we can keep the library generic.  However, I think you can do something like this as a workaround:
caching.store.getClient().scan();

That's assuming you're using this as your redis store: https://github.com/dabroek/node-cache-manager-redis-store

@BryanDonovan
Thanks for your quick response, yes I use NestJS 8 and the mentioned lib but I get following error:

Can you tell me sth about this error ?
Thank you in advance !!! :)

Looks like you just need to pass some argument(s) to the scan method.

This is available in redis ( with get it is "catchable ):

This is my query '/api/*'

Thank you :)

aha, i am one step further 👍

Please leave this open, if i have questions, I will close it afterwards or will tell you if any questions arises :)

Looks like the scan method has an odd signature.  See https://github.com/redis/node-redis/blob/v3.1/examples/scan.js#L9

ah nice, thanks I will dive into and provide a sample of working code for your docs :)

Greets @BryanDonovan
figured it out :)
Really strange syntax, as you mentioned ^^
If you want, you can close ist
  async getKeysByCID(cid: string): Promise<void> {
    await this.cache.store
       // eslint must be disabled because getClient() is not a defined function in generic lib but in redis available
      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
      // @ts-ignore
      .getClient()
      .scan(0, 'MATCH', `/api/v1/companies/${cid}/*`, (err, res) => {
        const cursor = res[0];
        const keys = res[1];
        if (keys.length > 0) {
          console.log('Array of matching keys', keys);
        }
        if (cursor === '0') {
          return console.log('Iteration complete');
        }
        return this.getKeysByCID(cid);
      });
  }

Good to hear!


beaconRequest state never set

hi,
the beaconRequest state is never set. only in stopMonitoring and then it is pointless.

Please refer to #60 . Thanks for your report.


crashed due to null pointer dereference

any way i can fix? am i doing  somthing wrong?

Having same issue on unleashed fmw, ui boots but when I go to start measuring it crashes


Warning "Can't call method "get_context" on an undefined value" when using "start_active_span" "child_of"

I get this warning:
(in cleanup) Can't call method "get_context" on an undefined value at /usr/local/share/perl/5.36.0/OpenTracing/Implementation/DataDog/Client.pm line 439 during global destruction.
When doing this:
sub receive {
        # Mock receieve headers....
	my $received_headers = HTTP::Headers->new();
	$received_headers->header('X-Datadog-Parent-Id' => '6498514476147820228');
	$received_headers->header('X-Datadog-Trace-Id' => '7906975651567419548');
	my $received_opentracing_context = $tracer->extract_context( $received_headers );

	say "---- received span context -----";
	print Dumper($received_opentracing_context);

	my $scope = $tracer->start_active_span('receive', 'child_of' => $received_opentracing_context);
	say "---- receive span context";
	print Dumper($scope->get_span()->get_context());
	start(); # which starts / closes it's own child span.

	$scope->close();
}

The 'received' spanId is correctly propagated, but I get a warning at the end. I don't know how to force a stack trace at that place.
start() doesn't throw an exception. So $scope->close() is being called.
If I intentionally don't close the outer $scope I get the same warning plus: (in cleanup) Scope not programmatically closed before being demolished at (eval 473) line 23..

Hi @marrem,
I'm sorry for the late response, and I'm afraid you and I need to discuss how you have tried to setup your OpenTracing / DataDog environment.
I think I have found part of the issue, which might be related to our internal HTTP client that tries to send a HTTP request to the dd-tracer-agent (is that how it is called ?). When that request fails, the internal client can become unstable. Which, of course, needs fixing.
However, it also shows me that your environment is not working correctly.
I'll reach out to you on LinkedIn to further discuss part of the problem, as I will investigate what can be done to make this module more robust and keep folks informed what is really going on.


Missing problems

A range of problem does not exist. It leaps from 242 - 257

ticking "Show Locked" in the plugin settings will get these to show. I have noticed though that some problems are missing entirely, such as 2877.

Delete the cache files in /home/${USER}/.lc/leetcode/cache and restart vscode will fix the it when facing missing problems. Same for the one in leetcode-cn.

Delete the cache files in /home/${USER}/.lc/leetcode/cache and restart vscode. Then it will be fixed when facing missing problems. Same for the one in leetcode-cn.

Sorry --- can I ask where this cache file is for MacOS ?


we need part 3 man. dont leave us hanging here.

Really liked your last walkthrough parts. great work, and really well designed. looking forward to the next part.

Part 3 is on it's way. Almost completed.

Done!


Check mode don't work

Hi when i use my playbook to install your role thje check mode failed on this task , it could be cool to make it work in check mode to be sure to what changes will be made
TASK [robertdebock.fail2ban : test if fail2ban_jail_configuration is set correctly] *******************************************************************************************************************************************************************************************************************************************
fatal: [192.168.1.X]: FAILED! => {}
MSG:
The conditional check 'item.value | length > 0' failed. The error was: Unexpected templating type error occurred on ({% if item.value | length > 0 %} True {% else %} False {% endif %}): object of type 'bool' has no len()

Hi, that's unexpected, you are seeing tasks/assert.yml fail because something in the fail2ban_jail_configuration is set to yes or no. I don't see how that relates to check_mode, it should also fail in regular mode.
I can reproduce this issue using this fail2ban_jail_configuration:
fail2ban_jail_configuration:
  - option: ignoreself
    value: true
    section: DEFAULT

In the example above the value true should be quoted to prevent Ansible from making a boolean out of it. This would work:
fail2ban_jail_configuration:
  - option: ignoreself
    value: "true"
    section: DEFAULT

If you continue to struggle, can you show me the variables you're feeding this role so I can analyse further?
Regards,
Robert de Bock.

I think this is related to a Jinja filter boolean, which is somewhat new. Please update Jinja.


Manual source has bug , version 12

Hello friend
Thank you for new release.
Today i try to test it and i got many error
when i install new version manually i got this error
Cannot find type 'GADInterstitialAd' in scope
Source > Internal > SwiftyAdsInterstitial.swift
I did compared pod files and manual source
Your cods are different
check your source file

I'm Sorry! New version working perfect.
My project had some issue and i received it.
Thank you Dominik .

Hey,
No problem at all. I will push some more small updates this week as well if I notice any problems. Please feel free to open another issue if needed.


Version Control of engine

Would it possible to let users decide what version of the engine they want to use? It would be really useful in case an update rolls in, and breaks some circuits. It would be nice to have a drop-down menu on the corner that lets you choose a version of the engine. This would help preserve old circuits in working state, while still letting users update their circuits to work with newer versions if they feel like it.
Feature requested by @HypheX

@tachyons I want to work on this issue. If some ref. is there it would be really great then.


Plans to release project files?

Hi
Are there plans to release the project files for the PCB?
I am trying to create a pick and place file, but I need the project files for that (i.e KiCAD, Horizon EDA etc.)
Thanks

I believe they already exist here: redox-keyboard/redox/pcb/rev1.0/kicad_files/

Thanks. I was looking in redox-w/rev2.0WHS)/pcb/.


Traccar 3.13 API /positions issue

Hey! I've issue - Installed fresh Traccar Server, everything worked fine with /api/positions to get all devices' current locations, but after the server restart, it asks for the date. So, after restart, API doesn't work without any params. Is it a bug or what? How to fix it? I've made a 3rd party application which uses /api/positions to get current location of vehicles, so it's important to working this API. :?

The problem was with 'Accept' header. Fixed it, thanks!


Removed sealed keyword

Several key classes (StorageManager, TestStorage, TestKitSilo) are marked as sealed.
This means their behaviour cannot be customized.
I'm currently using a storage provider and grains that are event-sourced, and need custom behaviour around state handling (state is not just a single object, it's a stream of events). I also want to allow for richer tests when reading / writing state.
The alternative is to fork the whole project, wchich I would like to avoid.
Would you please consider dropping sealed keywords? I'm happy to make a PR.
It's a bit urgent for me, so would hugely appreciate your response.

@jkonecki-techfabric Sounds fine to me. But is sealed enough? Are there methods that would also need to be marked virtual? If you can, take a pass over the classes and submit a PR.
I believe the last release process ran fully automated. Publishing a new version on NuGet should be as simple as tagging the desired commit after the PR is merged.

Yes, TestStorage methods would be virtual. I'll make a PR shortly.
Thank you very much for a prompt response!

From: Kyle Dodson<EMAIL_ADDRESS>Sent: Friday, July 3, 2020 6:08:50 PM
To: OrleansContrib/OrleansTestKit<EMAIL_ADDRESS>Cc: Jakub Konecki<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [OrleansContrib/OrleansTestKit] Removed sealed keyword (#92)
@jkonecki-techfabrichttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_jkonecki-2Dtechfabric&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=J5xwzqk2JV6JmMQ8LmNyE6cRZCpM-ICHZBNHSFKl9k8&m=EMcUmuXWBJmyZdcLlLQNd_Yme0kDMb3AZCEvylC-0-c&s=xpXlwhsRo09ZfuU3JbQnZhuVmtANlhHvx1v69pfbJdg&e= Sounds fine to me. But is sealed enough? Are there methods that would also need to be marked virtual? If you can, take a pass over the classes and submit a PR.
I believe the last release process ran fully automated. Publishing a new version on NuGet should be as simple as tagging the desired commit after the PR is merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_OrleansContrib_OrleansTestKit_issues_92-23issuecomment-2D653625859&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=J5xwzqk2JV6JmMQ8LmNyE6cRZCpM-ICHZBNHSFKl9k8&m=EMcUmuXWBJmyZdcLlLQNd_Yme0kDMb3AZCEvylC-0-c&s=NvBzGlFYxPPQ7YQgQ9fCn2kMcqiSEUuasb0BHvbXEvU&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ANYCISNCPIRI3KECWOAVHV3RZYGCFANCNFSM4OP6SALA&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=J5xwzqk2JV6JmMQ8LmNyE6cRZCpM-ICHZBNHSFKl9k8&m=EMcUmuXWBJmyZdcLlLQNd_Yme0kDMb3AZCEvylC-0-c&s=E9zht-8rvI4soL2MtgizhzQftzkAwibLXSCGXDbENuc&e=.


add XMP support in AVIF

Test file is convert to AVIF with Sharp, and copied the metadata with ExifTool.

Is there anything that this PR needs before it can be merged? I think this is affecting our ability to use rotated heic correctly in our application.


vTableSizes performance problem

[Enter feedback here]
The vTableSizes view never returns anything i.e. never completes, on our dedicated sql pool even when the server is not under significant load.

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

ID: 473726a0-19e3-a144-56ca-a66058f9ff67
Version Independent ID: 820466f5-2ec1-a3f8-51ae-8ab02fed9395
Content: Designing tables - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-tables-overview.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@ecrutchfield
Thanks for your feedback! We will investigate and update as appropriate.

#assign:WilliamDAssafMSFT

Thanks @ecrutchfield. Just to confirm, you SELECT from the vTableSizes view, perhaps using the follow-up queries, and the query runs continuously?
In the past with issues similar to this, we've recommended Index maintenance to resolve issues with these DMVs. Can you confirm the problem persists after performing a series of ALTER INDEX... REBUILD queries on all tables?
If you have any deeper technical questions, this may not be the best forum, as this space is for documentation updates. I recommend Microsoft Q & A (Azure Synapse Analytics), Azure Synapse Analytics feedback, Microsoft Support, or your support partners.


Remove snippet from art-attention

A message preview with better formatting (e.g. unfurled links) is already being shown thanks to the Link field

/lgtm
/approve


Project logo

A logo for the Service Binding is required.  This logo can be used on the GitHub organization page here:
https://github.com/servicebinding The logo can also be used on the website.
I am not a designer, though I made an attempt here: https://www.overleaf.com/read/gccxgtrkdtwr

The logo has been committed here: https://github.com/servicebinding/artwork
Thanks to Jason Groves for the great work!


itest: cleanup after each test and fix refactor harness test

This PR fixes multiple issues as follows,

structure how a test should be initialized via ht.SubTest()
force a node state validation check after each test is finished via cleanup
reuse standby nodes whenever possible
reduce blocks mined
refactor code and enhance logging wherever possible

Depends on #6192. Only the last 11 commits are new.
This is also the second part(3-of-4) of a larger fix(#5940) which eventually revives our itest. This PR is created so it's easier to be reviewed and the release notes are intentionally skipped. It should NOT be merged unless the final PR is accepted. Please check the last PR to see the itest build results.
We want to reuse nodes whenever possible as it takes lots of time to spin up a new node, thus we can start to treat blocks as a scarce source in our tests. A native benchmark is made here, and the results,
[height:448] setup node :1.485320301s
[height:448] restart node :832.921011ms
===========================================
mine 10 blocks :305.289338ms
setup a new node with 458 blocks: 1.722693468s
restart node :831.285503ms
===========================================
mine 100 blocks :5.899697955s
setup a new node with 558 blocks: 2.345724038s
restart node :831.945432ms
===========================================
mine 1000 blocks :59.836386824s
setup a new node with 1558 blocks: 20.319189488s
restart node :834.446827ms
===========================================
[height:1558] restart old node :835.327677ms

Replaced by #6825.


stats.gamma.fit returns nan for alpha, also no input parameter named "a"?

I've got a numpy.ndarray of shape (24569,) (also tried with (24569,1)) with samples of random variables (not the distribution)
fit_alpha, fit_loc, fit_beta=stats.gamma.fit(data)
results in fit_alpha and fit_loc being nans, while fit_beta = 1
I've tried to suggest starting parameters by using stats.gamma.fit(data,a=k,loc=0,scale=theta)
where k = 4/skew^2, theta=variance/mean are the shape and scale parameters respectively.
But I got the following error:

/usr/local/lib/python2.7/dist-packages/scipy/stats/_continuous_distns.pyc in fit(self, data, *args, **kwds)
   1916         if floc is None:
   1917             # loc is not fixed.  Use the default fit method.
-> 1918             return super(gamma_gen, self).fit(data, *args, **kwds)
   1919
   1920         # Special case: loc is fixed.

/usr/local/lib/python2.7/dist-packages/scipy/stats/_distn_infrastructure.pyc in fit(self, data, *args, **kwds)
   2126         # by now kwds must be empty, since everybody took what they needed
   2127         if kwds:
-> 2128             raise TypeError("Unknown arguments: %s." % kwds)
   2129
   2130         vals = optimizer(func, x0, args=(ravel(data),), disp=0)

TypeError: Unknown arguments: {'a': 0.14115690248522836}.

after using floc=0 as a parameter (which is not documented anywhere as far as I can tell) I got an exception pointing out that negative values are not allowed in data (I got one negative "error" value in the data) - after fixing this, I got results which made sense...
The documentation on that fit function is lacking - it also does not mention that the parameters returned are shape, loc, scale, as well as that fa, floc and fscale are not suggestions of initial fit, but are constraints on the fit to keep these values fixed. There appears to be no way to suggest initial parameters.

Thanks for the report!  This seems to be a duplicate of https://github.com/scipy/scipy/issues/5362
The issue is that a keyword argument, a=k, is not recognized as a starting value (https://github.com/scipy/scipy/blob/master/scipy/stats/_distn_infrastructure.py#L2177).
A workaround is to use the positional argument for the starting value of the shape parameter, gamma.fit(data, k, loc=0, scale=theta). Can you check that it works for you?

It would be nice to fix this actially. Note that https://github.com/scipy/scipy/issues/5362 contains a start in this direction, but it needs some more work to detect impossible combinations of arguments, e.g. gamma.fit(data, 1, a=2) (is the starting value of a 1 or 2? should likely be a TypeError), gamma.fit(data, a=2, fix_a=3) (also a TypeError or a ValueError) etc.
Pull requests welcome, I'd say.

This is the first part of the issue:

I've got a numpy.ndarray of shape (24569,) (also tried with (24569,1)) with samples of random variables (not the distribution)
fit_alpha, fit_loc, fit_beta=stats.gamma.fit(data)
results in fit_alpha and fit_loc being nans, while fit_beta = 1

Without the actual data, it is hard to know what is happening.  The only way I've been able to reproduce this is to put a nan in the input.  E.g.:
In [1]: import numpy as np

In [2]: from scipy import stats

In [3]: data = np.random.randn(100)

In [4]: data[0] = np.nan

In [5]: stats.gamma.fit(data)
Out[5]: (nan, nan, 1.0)

Can you check if your data contains nan?  E.g. check if np.any(np.isnan(data)) is True.

@gbrand-salesforce it sounds like there was difficulty reproducing the first part of this issue (fit returning nan for alpha), and the rest of the issue is covered in gh-5362. If you can provide data so that we can reproduce fit returning nan, please re-open! Otherwise, I think it's safe to close this.

@gbrand-salesforce it sounds like there was difficulty reproducing the first part of this issue (fit returning nan for alpha), and the rest of the issue is covered in gh-5362. If you can provide data so that we can reproduce fit returning nan, please re-open! Otherwise, I think it's safe to close this.


Add ! to allowed cookie value chars

Motivation:
! is missing from allowed cookie value chars, as per https://tools.ietf.org/html/rfc6265#section-4.1.1.
Issue was originally reported on Play!, see https://github.com/playframework/playframework/issues/4460#issuecomment-198177302.
Modifications:
Stick to RFC6265 ranges.
Result:
RFC6265 compliance, ! is supported

Please let me clean this up a bit.

clean up done

@normanmaurer comments addressed

@slandelle sorry one last thing... could you add a unit test ?

Test added.
I discovered another issue. I'll open an issue before submitting a PR, as I want your agreement.

Cherry-picked into 4.0 (0310d79edf3e323c38a58aae6d588674c4b3ece5) and 4.1 (d747438366cffbbfaec80ac114a5b7fad2e9cf2c)


It works only for the simple questions like greetings.

Now it works with simple other questions, like "what is the time?" and "whether of Colombo"


deployer: make use-cluster-credentials less intrusive for gcp clusters

Using deployer use-cluster-credentials on a GCP based cluster has previosly modified the system user state. This caused separate terminal sessions to conflict with each other.
To avoid this, we're now instead configuring an environment variable that makes us use a temporary file with the decrypted service account key.

@yuvipanda I'm working on trying to get GCP various clsuter off standard disks and would benefit from having this PR merged.
Do you have time to have a look?

Thank you @yuvipanda !!!!


Upgraded to JUnit 4.12, removed unused local variables

@smarthi Myriad's getting actively developed here: https://github.com/mesos/myriad. It would be great if you would like to contribute on that fork.

Yeah realized that after I made the PR. Yup pushed my PR to the new home.


Better breaking for generic object creation expressions with no arguments or initializer

closes #578

One potential downside to this
// before this change
            _projectFixersMap = new ConditionalWeakTable<
                IReadOnlyList<AnalyzerReference>,
                ImmutableDictionary<string, List<CodeFixProvider>>
            >();
// after this change
            _projectFixersMap =
                new ConditionalWeakTable<
                    IReadOnlyList<AnalyzerReference>,
                    ImmutableDictionary<string, List<CodeFixProvider>>
                >();

We may want to break after the = if the object creation can fit onto a single line, but I don't know how to accomplish that, and you could agree the "after" is more readable.
https://github.com/belav/csharpier-repos/pull/27/files shows what this does to real world code.


left90 and right90 seem to work in a manner opposite of what one would think?

left90() seems to rotate clockwise, while I'd expect it to rotate counterclockwise.
Similarly, right90() seems to rotate counterclockwise, while I'd expect it to rotate clockwise.
Is this intentional? Or maybe I've missed something / am confused? 😖 💯 😄

It's intentional. I made this library with computer graphics in mind, and the convention there is for the origin to be the top-left corner, with x and y increasing to the right and down respectively. In this coordinate system the rotations are correct. Because the direction of the y axis is flipped, rotations are backwards compared to the Cartesian coordinate system you'd usually adopt when doing linear algebra.
Out of interest, do you have a use case in mind where y increases up the page? As far as I remember the only part of this library with opinions about the coordinate system is rotations, so if there's a small change I could make to accommodate for both coordinate systems I will consider it.


feat: events page

This PR adds the Events page without integration with Google calendar
Preview

Great, can you also add the event branding for each card? I would keep it basic so the logo or image of the event.
For the drop down at the top, can you limit it to just overview and then each high level event rather than listing each year? "Overview, eBPF Summit, Cloud Native eBPF Day, eBPF Track (LPC), bpfconf, and IETF"

Also can you make the newest events the featured ones at the top?

closing in favor of #403


nchat: 5.2.11 -> 5.3.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/d99kris/nchat/releases.
meta.description for nchat is: Terminal-based chat client with support for Telegram and WhatsApp
meta.homepage for nchat is: https://github.com/d99kris/nchat
meta.changelog for nchat is: https://github.com/d99kris/nchat/releases/tag/v5.3.5
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.3.5 in filename of file in /nix/store/5mll4s6js89xpwxxj88k0mpk47bbsvwh-nchat-5.3.5

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

nchat

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5mll4s6js89xpwxxj88k0mpk47bbsvwh-nchat-5.3.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nchat https://github.com/r-ryantm/nixpkgs/archive/67f3c2a29f5ba933882d485b97c73c96fed1ee45.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/67f3c2a29f5ba933882d485b97c73c96fed1ee45#nchat

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5mll4s6js89xpwxxj88k0mpk47bbsvwh-nchat-5.3.5
ls -la /nix/store/5mll4s6js89xpwxxj88k0mpk47bbsvwh-nchat-5.3.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

nchat

Maintainer pings
cc @luftmensch-luftmensch @sikmir for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 360432

x86_64-darwin

:white_check_mark: 1 package built:

nchat


The calling code given by this page are wrong.

In the "Use Vision enhancement with video" section, when I created a video retrieval index, and trying to use it by sample code as follow
{
    "enhancements": {
            "video": {
              "enabled": true
            }
    },
    "dataSources": [
    {
        "type": "AzureComputerVisionVideoIndex",
        "parameters": {
            "computerVisionBaseUrl": "<your_computer_vision_endpoint>",
            "computerVisionApiKey": "<your_computer_vision_key>",
            "indexName": "<name_of_your_index>",
            "videoUrls": ["<your_video_SAS_URL>"]
        }
    }],
    "messages": [
        {
            "role": "system",
            "content": "You are a helpful assistant."
        },
        {
            "role": "user",
            "content": [
                    {
                        "type": "text",
                        "text": "Describe this video:"
                    }
                ]
        },
        {
            "role": "user",
            "content": [
                    {
                        "type": "acv_document_id",
                        "acv_document_id": "<your_video_ID>"
                    }
                ]
        }
    ],
    "max_tokens": 100,
}

I get the error message "video sampling failed because it was not found".
The correct calling method should be

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

ID: f3231140-6a72-c587-61e7-434ec2a52ac9
Version Independent ID: 68609d85-dc97-bb55-f962-d3e8f113bb96
Content: How to use the GPT-4 Turbo with Vision model - Azure OpenAI Service
Content Source: articles/ai-services/openai/how-to/gpt-with-vision.md
Service: azure-ai-openai
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@Matt-Leee
Thanks for your feedback! We will investigate and update as appropriate.

@Matt-Leee
Thank you for bringing this to our attention.
I've delegated this to content author @PatrickFarley, who will review it and offer their insightful opinions.

Hi @Matt-Leee , I suspect the computer vision url needs a / at the end, but I can't verify at the moment.
Can you confirm that the document ID is vedio, not video?

Hi @PatrickFarley, thanks for your response. It's work in my test without the '/' at the end.
For the second question, I'm sorry I don't really understand what you mean here.

@Matt-Leee The value you entered for acv_document_id is vedio. I'm just checking that it's not a typo

@Matt-Leee Did the API call work with the latter JSON input you posted?

yes, I'm sure it's work.

Thank you, fixing the code now https://github.com/MicrosoftDocs/azure-docs-pr/pull/268147
#please-close


Implement interpolate_bilinear and interpolate_bicubic

🚀 The feature, motivation and pitch
Implement

[ ] interpolate_bilinear
[ ] interpolate_bicubic

cases which are skipped:
"test_dtypes_nn_functional_interpolate_bilinear_xpu",
"test_dtypes_nn_functional_interpolate_bicubic_xpu",
root cause:

fallback to cpu‘s implementation but use the dtypes  claim by xpu
Alternatives
No response
Additional context
No response

all passed


Skip import test for _odict_py2

See https://github.com/spacetelescope/pyasdf/issues/187 for a discussion
of why _odict_py2 does not end up being included in python 3 source
packages in asdf.

Thanks @mwcraig!


Better Centering of Hermes' DocumentWidget within Heracles' display

Tested at UI scales 2, 3, and 4.
When there are no tasks or rewards (and hence, no left margin), centering is dead on. When there is a left margin...
...and UI scale is 3: centering is spot on.
...and UI scale is 2 or 4: centering is off by one font "pixel"; a forgivable amount for now.
Undocumented magic numbers are bad and unsatisfying though.
How Hermes' is centering text,  tags, and horizontal rules is a factor. (the later were measured down to the pixel in screen shots as a stand-in for Hermes' position and width)

Agreement when there is no left margin:

Agreement and within one 'font pixel width' when a left margin is present:


add spawn-os-process-detached

This change adds spawn-os-process-detached for cases when we want to fire-and-forget and external process. This is useful when starting processes in user-space that do not require the overhead of being a service.
Testing is light because the external process is detached and thus hard to monitor. There are two cases of particular interest that can be done manually. 1) Fire up and external process, then exit Swish. The external process should remain. 2) Fire up an external process, then have it exit. Swish should remain and not crash (which would happen if we included invalid callback pointers in the osi process struct).

To summarize, libuv always watches the processes it spawns. On macOS, even if we uv_close the process handle, the watched process remains even after it exits as seen by ps -x. As a result, the safest option for all platforms is to hook the exit_cb to call close to free the memory.

I tried 2 more alternatives and they both failed. I squashed everything together and will let Travis have another chance to build it.

Thanks Bob!


SqlQuery: Cannot create sql query: Internal Server Error

What happened?
(Databricks) Now encountering problems even creating a SQL query:
Diagnostics:
  pulumi:pulumi:Stack (databricks-core-dev):
    Shared Directory ID: [REDACTED]
    SQL Endpoint ID: [REDACTED]

    error: update failed

  databricks:index:SqlQuery (q1):
    error: 1 error occurred:
        * cannot create sql query: Internal Server Error

ref: https://www.pulumi.com/registry/packages/databricks/api-docs/sqlquery/
Example
The query -- I've tried numerous variants, including from docs:
export const q1 = new databricks.SqlQuery("q1", {
  dataSourceId: sqlEndpoint.id,
  query: "SELECT 1",
  runAsRole: "owner",
  description: "Query 1",
  name: "q1",
  tags: ["t1", "t2"],
});

Output of pulumi about
❯ pulumi about
CLI
Version      3.81.0
Go Version   go1.21.0
Go Compiler  gc

Plugins
NAME        VERSION
aws-native  0.8.0
databricks  1.22.1
nodejs      unknown

Host
OS       arch
Version  "23.0.2"
Arch     x86_64

This project is written in nodejs: executable='/usr/bin/node' version='v16.20.2'

Current Stack: WP-LKL/[REDACTED]

TYPE                                      URN
pulumi:pulumi:Stack                       urn:pulumi:dev::databricks-core::pulumi:pulumi:Stack::databricks-core-dev
pulumi:providers:databricks               urn:pulumi:dev::databricks-core::pulumi:providers:databricks::default_1_22_1
databricks:index/directory:Directory      urn:pulumi:dev::databricks-core::databricks:index/directory:Directory::sharedDir
databricks:index/sqlEndpoint:SqlEndpoint  urn:pulumi:dev::databricks-core::databricks:index/sqlEndpoint:SqlEndpoint::pulumi-4DBU/h
databricks:index/groupMember:GroupMember  urn:pulumi:dev::databricks-core::databricks:index/groupMember:GroupMember::[REDACTED]-devMember
databricks:index/groupMember:GroupMember  urn:pulumi:dev::databricks-core::databricks:index/groupMember:GroupMember::[REDACTED]-devMember
databricks:index/groupMember:GroupMember  urn:pulumi:dev::databricks-core::databricks:index/groupMember:GroupMember::[REDACTED]-devMember
databricks:index/groupMember:GroupMember  urn:pulumi:dev::databricks-core::databricks:index/groupMember:GroupMember::[REDACTED]-devMember
pulumi:providers:databricks               urn:pulumi:dev::databricks-core::pulumi:providers:databricks::default_1_22_0
databricks:index/user:User                urn:pulumi:dev::databricks-core::databricks:index/user:User::[REDACTED]
databricks:index/group:Group              urn:pulumi:dev::databricks-core::databricks:index/group:Group::ws-dev
databricks:index/group:Group              urn:pulumi:dev::databricks-core::databricks:index/group:Group::ws-admin
databricks:index/group:Group              urn:pulumi:dev::databricks-core::databricks:index/group:Group::ws-powerDev

Found no pending operations associated with dev

Backend
Name           pulumi.com
URL            https://app.pulumi.com/WP-LKL
User           WP-LKL
Organizations  WP-LKL, [REDACTED]

Dependencies:
NAME                VERSION
@pulumi/aws-native  0.8.0
@pulumi/databricks  1.22.1
@pulumi/pulumi      3.79.0
@types/node         16.18.46

Pulumi locates its logs in /tmp by default

Additional context

The endpoint (SQL warehouse) is verified as active.
The folder/directory does exist succesfully.
How can we debug further?

Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @WP-LKL, this issue looks similar to this upstream one: https://github.com/databricks/terraform-provider-databricks/issues/1469
If sounds like you need to point your dataSourceId to sqlEndpoint.dataSourceId not sqlEndpoint.id.
Property doc: https://www.pulumi.com/registry/packages/databricks/api-docs/sqlendpoint/#datasourceid_nodejs
Comment: https://github.com/databricks/terraform-provider-databricks/issues/1469#issuecomment-1234946205
Let me know if this helps.

Thank you @mikhailshilkov!


usm: tls: tests without a tracer

What does this PR do?
The PR allows running TLS monitoring tests using USM monitor, without the old requirement of having a full Tracer.
The PR simplifies tests and moves them back to USM packages.

Motivation
Simplify tests
The PR https://github.com/DataDog/datadog-agent/pull/21259 was a preliminary step, and now USM owns all kernel related code that once was in NPM

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

General comment: I do not like the fact that after the change TestRawTraffic is different from all other tests in its way we run it. Do you want to change it as part of this PR? or I would draft one?

/merge

/remove


[feature] ability to toggle trix editor attachments

changes
[feature] ability to toggle trix editor attachments
Addresses: https://github.com/restarone/violet_rails/issues/661
Demo Video
https://user-images.githubusercontent.com/35935196/176429682-a19cc7b7-f692-4f18-8bc9-7f95947fc9b2.mp4
challenges
Unchecked checkbox not passing false to params:
It took me some time to figure this out. In the end, I followed these reference for the workaround;
https://github.com/rails/rails/issues/5937
https://stackoverflow.com/questions/20379311/how-to-get-blank-checkboxes-to-pass-as-false-to-params/20380051#20380051

@alis-khadka the required rule does not work for rich text. This will need to be addressed. Happy to keep the release candidate under monitoring if you are able to submit a quick patch against this branch

@alis-khadka the required rule does not work for rich text. This will need to be addressed. Happy to keep the release candidate under monitoring if you are able to submit a quick patch against this branch

The required option was not working when I tested on old commit (before my change).
It turns out that required attribute works for native elements only.
reference: https://github.com/basecamp/trix/issues/328

@alis-khadka yes, I believe so. But we are trying to replace HTML text area usage with this.
Can we implement this on the client (just like we have for HTML text inputs/Text areas) by using trix for validation?


Emit type declaration file and use it for "types" field in package.json

As @timocov pointed out on https://github.com/timocov/dts-bundle-generator/issues/53, the package.json file of this project references "Signal.ts", which contains both TypeScript declarations and definitions. The file should however only contain the declarations. This pull request makes TS create a Signal.d.ts file and it fixes compilation with TypeScript 2.9.2.

Coverage remained the same at 100.0% when pulling 2bd8016e451db6c92affae2edd40832e91439d80 on rhuitl:type-declaration into 232a11f4333fd5b5af15f47edef7e0acd76b519d on Lusito:develop.

Thanks!
Will add some things and then release a new version.

New version has just been released.

Perfect, thanks! Tested it and works 👍


Start Page - sample notebook needs some cleaning up.

Open the Python start page
Click the Explore our sample notebook to learn about notebook features

There are several things we should consider fixing:

[x] The name of the sample notebook is "Untitled-x.ipynb". Shouldn't it be something more meaningful?
[ ] Might want to consider indicating that a python file must be loaded to have the "Python: Open Start Page" command to be available (i.e. the extension has to be loaded).
[x] "This view is what we call our Notebook Editor, an interactive environment that lets you write and execute code." Consider changing to something like: "You are currently viewing what we call our Notebook Editor. It is an interactive document based on Jupyter Notebooks that supports the intermixing of code, outputs and documentation.
[x] The bullet in this cell is in the wrong place. .
[ ] On the cell referenced above, do we really want to use html in a markdown cell in our sample notebook? Maybe, but we should be sure this is a conscious decision. I doubt that html in notebook markdown is common. I realize that we probably did this to show the images in the text.
[x] I think that if we're going to take the time to describe how to execute a code cell (a very basic notebook operation), we should also describe how to edit a markdown cell. It's not intuitive to press enter or double click the cell.
[x] The animated gif that describes using the dataframe viewer shows the icon to open a dataframe viewer is on the right of a dataframe line. It's going to be on the left by the time we ship and is  what the markdown describes.
[x] Markdown cells that represent top level headings could be more nicely split by adding a "---" to draw a horizontal line, better separating cells.
[x] It is confusing when you change the notebook kernel, the default python kernel in the status bar doesn't change with it. Even if we decide not to do something to make this more sane, the animated gif could probably be just as effective without showing the status bar and the notebook kernel selector at the same time.
[x] The animated gif showing "ENTER YOUR URL HERE" is ... mmmm.... just goofy. Why not just show the user entering an https://... address???

SampleNotebook (3).zip
Updated the notebook with the changes.
Things changed in the notebook:

Use new notebook

Added section on run-by-line
added section on markdown
Changed all the GIFs/images to represent the new notebook UI
added line separators
Reworded the "Getting started" markdown cell

Things left to do

Rename notebook to "Notebooks intro.ipynb"
Fix bullet in cell

Make the titles clickable in the start page

Validated. Clickable titles look a bit funny to me as they don't have a link color or underline, but they do get a highlight on mouse over.


ValueError: Cannot use chat template functions because tokenizer.chat_template is not set and no template argument was passed!

ValueError: Cannot use chat template functions because tokenizer.chat_template is not set and no template argument was passed! For information about writing templates and setting the tokenizer.chat_template attribute, please see the documentation at https://huggingface.co/docs/transformers/main/en/chat_templating

vila-infer
--model-path /data/workspace/zhaoyong/model/weight_files/VILA1.5-3b
--conv-mode auto
--text "Please describe the image"
--media /data/workspace/zhaoyong/data/图片数据/安全帽/安全帽21_0.jpg

how to deal it?

(vila) (base) user@ubuntu(125):/data/workspace/zhaoyong/model/VILA$ sh 1.sh
[2024-12-17 10:13:22,850] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
2024-12-17 10:13:24.216 | INFO     | llava.conversation:auto_set_conversation_mode:189 - Setting conversation mode to vicuna_v1 based on model name/path /data/workspace/zhaoyong/model/weight_files/VILA1.5-3B.
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████| 2/2 [00:02<00:00,  1.08s/it]
We've detected an older driver with an RTX 4000 series GPU. These drivers have issues with P2P. This can affect the multi-gpu inference when using accelerate device_map.Please make sure to update your driver to the latest version which resolves this.
You are attempting to use Flash Attention 2.0 with a model not initialized on GPU. Make sure to move the model to GPU after initializing it on CPU with model.to('cuda').
The new embeddings will be initialized from a multivariate normal distribution that has old embeddings' mean and covariance. As described in this article: https://nlp.stanford.edu/~johnhew/vocab-expansion.html. To disable this, use mean_resizing=False
The new lm_head weights will be initialized from a multivariate normal distribution that has old embeddings' mean and covariance. As described in this article: https://nlp.stanford.edu/~johnhew/vocab-expansion.html. To disable this, use mean_resizing=False
Traceback (most recent call last):
File "/home/user/miniconda3/envs/vila/bin/vila-infer", line 8, in
sys.exit(main())
File "/data/workspace/zhaoyong/model/VILA/llava/cli/infer.py", line 40, in main
response = model.generate_content(prompt)
File "/home/user/miniconda3/envs/vila/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/data/workspace/zhaoyong/model/VILA/llava/model/llava_arch.py", line 825, in generate_content
input_ids = tokenize_conversation(conversation, self.tokenizer, add_generation_prompt=True).cuda().unsqueeze(0)
File "/data/workspace/zhaoyong/model/VILA/llava/utils/tokenizer.py", line 110, in tokenize_conversation
text = tokenizer.apply_chat_template(
File "/home/user/miniconda3/envs/vila/lib/python3.10/site-packages/transformers/tokenization_utils_base.py", line 1617, in apply_chat_template
chat_template = self.get_chat_template(chat_template, tools)
File "/home/user/miniconda3/envs/vila/lib/python3.10/site-packages/transformers/tokenization_utils_base.py", line 1785, in get_chat_template
raise ValueError(
ValueError: Cannot use chat template functions because tokenizer.chat_template is not set and no template argument was passed! For information about writing templates and setting the tokenizer.chat_template attribute, please see the documentation at https://huggingface.co/docs/transformers/main/en/chat_templating

May I ask how I can solve this？ thanks！！！

Having the same issue

Having the same issue
How I can solve this ?


Required path parameters should be added as constructor arguments

Required path parameters, as called out in the OpenAPI spec, should be moved from "Args" classes and into their corresponding constructors.
For example, in the spec, there is a single model for NetworkSecurityGroup. This model is correctly represented in Pulumi by the Pulumi.AzureNextGen.Network.Latest.Inputs.NetworkSecurityGroupArgs class. However, Pulumi provides another, identically-named class, Pulumi.AzureNextGen.Network.Latest.NetworkSecurityGroupArgs, that has no equivalent in the spec.
The difference is subtle, but Pulumi.AzureNextGen.Network.Latest.NetworkSecurityGroupArgs adds two properties to the model that are used in the creation/modification of the resource itself. These properties come from path parameters in Azure's OpenAPI spec, that are separate from the body parameter.
Removing these "extra" parameters from Pulumi.AzureNextGen.Network.Latest.NetworkSecurityGroupArgs, and adding them as separate arguments would give us a model more consistent with the spec. It also means we would only need one NetworkSecurityGroupArgs class.
Instead, we would add these parameters as constructor arguments, which would also be more consistent with Azure's own SDK.
There has been some similar discussion about this, but this request is focused on required, path parameters. Values that, per REST, should always exist and should never change. And even if they were to change, Azure's SDK would become equally broken (and, thus, not likely to happen).
For deterministic argument order, name would always be first, followed by resourceGroupName , followed by all required, path values and then the "Args" class, eg:
using Pulumi.AzureNextGen.Network.Latest;

public NetworkSecrityGroup(string networkSecurityGroupName, string resourceGroupName, NetworkSecrityGroupArgs args, CustomResourceOptions? options = null);

I moved the issue to the azure-nextgen repository as it's very specific to the Azure resource model. Let me know if you don't agree.

One interesting thing about this discussion is that there's a separate issue https://github.com/pulumi/pulumi/issues/5146 where the desire is to build up the args before passing it to the constructor. Ideally, we'd have some way to reconcile these two requirements.


Added search extension for Zhihu

Zhihu extension:
Zhihu is a Chinese question-and-answer website where questions are created, answered, edited and organized by the community of its users. Its website, zhihu.com, was launched on January 26, 2011. The number of registered users of Zhihu had exceeded 10 millions by the end of 2013,[3] and reached 17 millions as of May 2015[4] with 250 million monthly page views.

just  a copy of quora

Thansk for this -- closing as I already merged a similar extension.
I'm sorry for the very long delay getting to this pull request, and for this copy-and-pasted message:
Thanks for your contribution! To get through my backlog, I'm now merging all reasonable pull requests.
Once done, I'm going to move all user-contributed extensions into a new contrib directory in the repo, so you will eventually find it there. After that, in due course, I will move some contributions back over to the main directory to be published.
Please bear with while I am working though a lot of pull requests at the moment.


Add the ability for a controller to trigger a problem report in an in-flight protocol

We need a way for the controller to instruct an agent to bail out of a protocol and send a problem report. Perhaps this can be an endpoint that takes the protocol name, internal id, and message

This has been added in https://github.com/hyperledger/aries-cloudagent-python/pull/98


.length() not a function on a FilteredArrayWrapper

Not sure why you can't do this but I would have thought this should work:
Lazy([{ name: 'Gary'}, { name: 'Helen'}, { name: 'Bobby'}])
  .where({ name: 'Helen' })
  .length();

// Result: Uncaught TypeError: Lazy(...).where(...).length is not a function
    at window.onload (VM189:49)

However if you convert it to an array it works:
Lazy([{ name: 'Gary'}, { name: 'Helen'}, { name: 'Bobby'}])
  .where({ name: 'Helen' })
  .toArray()
  .length();

As .length() is an "ultimate" getter and "does" something, wouldn't it make sense to allow calling it rather than having to do .toArray() beforehand?
https://jsfiddle.net/5v4d5xtj/

It's a fair question, and seems to be pretty much exactly the same question raised in #209.
My current stance on it is that length() is not available in scenarios where it would require scanning the entire sequence (such as FilteredArrayWrapper) because I'm concerned availability will be mistaken for endorsement and devs will write code like this without realizing how inefficient it is:
for (var i = 0; i < seq.length(); i++) {
  doSomething(seq.get(i));
}

I'm not 100% attached to this stance; it's just where I stand at the moment. I recognize there might be some legitimate cases where you would want to call length() knowing that it requires full iteration; but my gut says those cases will be rare and if you're a dev in that situation you can always write your own helper for that purpose.

I responded in the closed issue: https://github.com/dtao/lazy.js/issues/207#issuecomment-363403526
I did not know that length on Sequence would require a full scan. In that case, I agree that it should not be available so it will not be mistaken for endorsement.
Perhaps it would be an interesting documentation read to find out how to manually add a .length() method to Sequence. For devs who need prefer to have it regardless.

I could see how it might be worthwhile to document this in more depth.
For now, consider an extreme case. Let's say I have a "database" consisting of a million people in an array and I want to filter down to just those people with the last name "Smith". How many results will the result have? Without some kind of index backing this lookup (e.g. what you'd expect from a real database), there is no way to know without actually scanning the data.
Now here's something interesting that I came across just today that I completely forgot about: there is a method available in this library already to eagerly get the count: size(). So for devs who really want that, it's there. Note from the docs:

In some cases, this may require eagerly evaluating the sequence.

This still might change. In hindsight I'm not sure it's better to have two different methods; it might be better to just always make length() available if size() is there anyway. But for now, this is where we are.

Thanks for the clarification and reminding us about size() @dtao


Only github supported for git source downloads

I wrote a module for managing AWS keys and put it on

Bitbucket => https://bitbucket.org/alexs77/terraform-aws-key_pair
Gitlab => https://gitlab.com/alexs-77/terraform-aws-key_pair
Github => https://github.com/alexs77/terraform-aws-key_pair

Trying to use it from my terragrunt sources with this terraform.tfvars file:
terragrunt = {
  terraform {
    source = "git::git@github.com:alexs77/terraform-aws-key_pair.git"
    #source =<EMAIL_ADDRESS>    #source = "git::git@bitbucket.org:alexs77/terraform-aws-key_pair.git"
  }

  include = {
    path = "${find_in_parent_folders()}"
  }

  dependencies {
    paths = ["../vpc"]
  }
}

key_pair_name = "example_key"

key_pair_public_key = "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQD3F6tyPEFEzV0LX3X8BsXdMsQz1x2cEikKDEY0aIj41qgxMCP/iteneqXSIFZBp5vizPvaoIR3Um9xK7PGoW8giupGn+EPuxIA4cDM4vzOqOkiMPhz5XK0whEjkVzTo4+S0puvDZuwIsdiW9mxhJc7tgBNL0cYlWSYVkz4G/fslNfRPW5mYAM49f4fhtxPb5ok4Q2Lg9dPKVHO/Bgeu5woMc7RY0p1ej6D4CKFE6lymSDJpW0YHX/wqE9+cfEauh7xZcG0q9t2ta6F6fmX0agvpFyZo8aFbXeUBr7osSCJNgvavWbM/06niWrOvYX2xwWdhXmXSrbX8ZbabVohBK41<EMAIL_ADDRESS>
It only works, when I use Github as a source; with Gitlab or Bitbucket, it fails. The repositories are always public; ie. a git clone … works fine.
Error with Bitbucket source
alex@system:~/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair$ terragrunt apply
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:51:26 Running command: terraform --version
[terragrunt] 2019/03/14 10:51:26 Reading Terragrunt config file at /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/terraform.tfvars
[terragrunt] 2019/03/14 10:51:27 WARNING: no double-slash (//) found in source URL /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git. Relative paths in downloaded Terraform code may not work.
[terragrunt] 2019/03/14 10:51:27 Downloading Terraform configurations from git::file:///Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git into /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/spEL9gxib7qPpG_hNJ5iaFzsZp0 using terraform init
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:51:27 Initializing remote state for the s3 backend
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:51:29 Detected 1 Hooks
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:51:29 Running command: terraform init -backend-config=region=ap-south-1 -backend-config=skip_requesting_account_id=true -backend-config=skip_metadata_api_check=true -backend-config=bucket=terraform-states-aps1-668273420038 -backend-config=dynamodb_table=terraform-locks-aps1-668273420038 -backend-config=skip_region_validation=true -backend-config=skip_credentials_validation=true -backend-config=encrypt=true -backend-config=key=ap-south-1/keypair/terraform.tfstate -backend-config=skip_get_ec2_platforms=true -input=false -get=false -get-plugins=false -backend=false -from-module=git::file:///Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git -no-color /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/spEL9gxib7qPpG_hNJ5iaFzsZp0
Copying configuration from "git::file:///Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git"...
Error copying source module: error downloading 'file:///Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git': /usr/bin/git exited with 128: Cloning into '/var/folders/nk/mxbh7f2j403crxhw1y17pn480000gn/T/tf149426870/module'...
fatal: '/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:51:29 Detected 1 Hooks
[terragrunt] 2019/03/14 10:51:29 Hit multiple errors:
exit status 1

Error with Gitlab source
alex@system:~/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair$ terragrunt apply
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:52:05 Running command: terraform --version
[terragrunt] 2019/03/14 10:52:05 Reading Terragrunt config file at /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/terraform.tfvars
[terragrunt] 2019/03/14 10:52:06 WARNING: no double-slash (//) found in source URL<EMAIL_ADDRESS>Relative paths in downloaded Terraform code may not work.
[terragrunt] 2019/03/14 10:52:06 Downloading Terraform configurations from<EMAIL_ADDRESS>into /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/LVqU0vlVOSIuEPjGqTObHw_c5RE using terraform init
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:52:06 Initializing remote state for the s3 backend
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:52:08 Detected 1 Hooks
[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:52:08 Running command: terraform init -backend-config=bucket=terraform-states-aps1-668273420038 -backend-config=skip_metadata_api_check=true -backend-config=skip_credentials_validation=true -backend-config=skip_get_ec2_platforms=true -backend-config=encrypt=true -backend-config=region=ap-south-1 -backend-config=key=ap-south-1/keypair/terraform.tfstate -backend-config=skip_requesting_account_id=true -backend-config=skip_region_validation=true -backend-config=dynamodb_table=terraform-locks-aps1-668273420038 -input=false -get=false -get-plugins=false -backend=false<EMAIL_ADDRESS>-no-color /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/LVqU0vlVOSIuEPjGqTObHw_c5RE
Copying configuration from<EMAIL_ADDRESS>Error copying source module: error downloading<EMAIL_ADDRESS>/usr/bin/git exited with 128: Cloning into '/var/folders/nk/mxbh7f2j403crxhw1y17pn480000gn/T/tf227224904/module'...
fatal<EMAIL_ADDRESS>does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

[terragrunt] [/Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair] 2019/03/14 10:52:08 Detected 1 Hooks
[terragrunt] 2019/03/14 10:52:08 Hit multiple errors:
exit status 1

Common
Common is line 5 of these errors, eg.:
[terragrunt] 2019/03/14 10:51:27 Downloading Terraform configurations from git::file:///Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/git@bitbucket.org:alexs77/terraform-aws-key_pair.git into /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/spEL9gxib7qPpG_hNJ5iaFzsZp0 using terraform init
With Github as a source, line 5 reads:
[terragrunt] 2019/03/14 10:59:56 Downloading Terraform configurations from<EMAIL_ADDRESS>into /Users/alex/SysOps/AWS/QA/cloudcraft/ap-south-1/ap-south-1/keypair/.terragrunt-cache/7FJACLBEMlnpJeba-LhQbRwG95E/TrprSvuoV_MVKNhRWamm3LGfqCA using terraform init
Ie. git::file:///Users/… vs<EMAIL_ADDRESS>Seems as if there's somewhere a special case for github, which transforms the URL to ssh://….
Versions
$ terragrunt -version
terragrunt version v0.18.0

$ terraform -version
Terraform v0.11.13

1-gitlab-error.txt
2-bitbucket-error.txt
3-github-okay.txt

Hi @alexs77, great issue report! Terragrunt does not do any special handling of the source parameter. It is processed directly by Terraform (which in turn is passing the argument to the go-getter package). If you check out the Terraform docs on the source parameter for modules, you'll see the syntax you are using is only supported for GitHub:

https://www.terraform.io/docs/modules/sources.html#github

If you use the syntax for a generic git repository, it will work with all remotes:

https://www.terraform.io/docs/modules/sources.html#generic-git-repository

@lorengordon , thanks a lot for the explanation. With git::ssh:// it works all the time, just like you've pointed out :-)
Problem solved.


[Draft] Support half-open duplex streams

#27, reactphp/socket#126

@clue thanks a lot for the thorough review! I will need some time to address the issues.


Cannot call a class function

react v16.0.0
react-tracing v0.1.5
haven't done anything cool yet. So basically my app crashes when:
const tracer = new Tracing({
    serviceName: "Frontend",
    endpoint: "http://localhost:9411",
    kind: "client", // default
});

error says: Cannot call a class function

UP

Sorry, I honestly didn't know anyone was using this already, I should have included a warning in the notes 🙈  I will look into this issue and update the docs to not state my dreams, but what is currently working 👍 Please bare with me for a moment.

I added a note to the readme and I will try to sort out what is supported and what is WIP, hopefully resulting in a stable master and an unstable develop branch. In the meantime, I think the networking stack is working, everything user related not, as a rough sketch of the current state 🙈


stdlib: use the reserved attribute spellings

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

  Compilers are to not error when they encounter a reserved spelling
for an attribute which they do not support.

@swift-ci Please test and merge

Hmm, the failure doesn't seem related?

Just to make it easier to find the failure:
https://ci.swift.org/job/swift-PR-osx/1187/consoleFull#-1905045746ee1a197b-acac-4b17-83cf-a53b95139a76
Seems that Swift :: Reflection/functions.swift failed as swift-reflection-test failed to execute properly.

CI failures are unrelated.


Adding parent, containing latest KC 1.1.0.Final

You need https://github.com/aerogear/aerogear-parent/pull/40 for testing this
Tested and deployed on WF 8.2 -> works fine
NOTE: before merging this, I'd think we should get the parent 0.2.12 released to maven central

@abstractj Let me start with the 0.2.12 release


What are the limitations for scheduled messages?

We are wondering about the limitations for scheduled messages. How far in the future can ScheduledEnqueueTimeUtc be set? How many messages can be enqueued simultaneously? Are there any metrics that we can monitor in order to prevent issues?

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

ID: 034f7b39-4f6b-87c4-5dfa-a09ea0235b6e
Version Independent ID: 6987bbec-c4e9-573f-89c5-aa316af045f9
Content: Azure Service Bus message sequencing and timestamps - Azure Service Bus
Content Source: articles/service-bus-messaging/message-sequencing.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

@skyfrk Just following up here. Hope my previous comment clears things up.

Hi @MayankBargali-MSFT, sorry for reacting late. Your response answered all of our questions, thank you! Maybe your answer can become part of the documentation so others won't stumble upon the same question?

@skyfrk Thanks for your response. I will work with content team to see how best we can update this information in our document. For now, I will go ahead and close this thread.
Feel free to tag me if for any queries or concerns.