[BUG]

Describe the bug
This platform could not train a MLE model.
When I load the MLE model for GDPL, PPO, PG, it could train with no problem, but it never gets to the optimal score(I run evluate.py to see the model). Actually it goes down after few eopchs. And here is a graph I made in GDPL, and PPO is pretty similar to this one.
To Reproduce
Steps to reproduce the behavior:
Simply run train.py in PG/GDPL/PPO, and it will give this issue. I write a script which could evluate all of the models in one dir, and here is the graph I made about GDPL.
Expected behavior
The evluate score should go higher when loaded MLE model.
Score for PPO:
[0.52, 0.53, 0.54, 0.49, 0.44, 0.49, 0.46, 0.47, 0.44, 0.43, 0.42, 0.44, 0.47, 0.48, 0.49, 0.46, 0.45, 0.46, 0.45, 0.48, 0.46, 0.48, 0.49, 0.49, 0.49, 0.48, 0.45, 0.47, 0.43, 0.43, 0.43, 0.42, 0.42, 0.41, 0.42, 0.43, 0.44, 0.47, 0.45, 0.43]
So, the max of PPO is just goes to 0.53, not like 0.74.

Thanks! We will check it in a few days.

Please check if you re-install ConvLab-2 since the code has changed. (pip install -e . in ConvLab-2)

Yeah, I have already change the code, sure about that.

I have tried these:
in mle/multiwoz/: python train.py
in ppo/: python train.py --load_path ../mle/multiwoz/save/best_mle
in policy/: python evaluate.py --model_name PPO --load_path ppo/save/9_ppo (I choose 9_ppo randomly)
and get:
All 100 0.73
reward: 11.143058441558441

Oh, that is cool, for me the PPO is also good, but for GDPL, there are still some problem exists.

I have tried these:
in mle/multiwoz/: python train.py
in ppo/: python train.py --load_path ../mle/multiwoz/save/best_mle
in policy/: python evaluate.py --model_name PPO --load_path ppo/save/9_ppo (I choose 9_ppo randomly)
and get:
All 100 0.73
reward: 11.143058441558441

Hi @zqwerty , Thanks for the tips. But when I run these commands for the latest commit (c6372b1), I found the problem is still unsolved.
The performance of PPO will still rise to 65% at the beginning and then start dropping at later training stage (to around 35%).
Could you help look into it? Thanks!

move to #54

I have tried these:
in mle/multiwoz/: python train.py
in ppo/: python train.py --load_path ../mle/multiwoz/save/best_mle
in policy/: python evaluate.py --model_name PPO --load_path ppo/save/9_ppo (I choose 9_ppo randomly)
and get:
All 100 0.73
reward: 11.143058441558441

Hey guys, I tested PPO on the latest version of convab-2 today and I got a success rate of 84% that is way bigger than the reported 73%, I wonder if there are any mistakes? If not, I think the performance record should be updated.


BUILD_LIBRARY_FOR_DISTRIBUTION not set to Yes on build settings

Hi,
Got an issue when trying to use the binary uploaded on your releases.
Both releases so far haven't had the BUILD_LIBRARY_FOR_DISTRIBUTION flag set. So when we try to use the binary we have to have the exact same swift tooling or it won't build. Even if we have a more recent swift tool version.
I tried cloning the repo and noticed the property is set to No for debug and release

Can you set that to Yes, and create a new binary for release? The previous two aren't actually usable currently 😅
From here: https://www.swift.org/blog/library-evolution/

When using Xcode to develop for Apple platforms, set the BUILD_LIBRARY_FOR_DISTRIBUTION build setting in the framework’s target. This setting turns on both library evolution and module stability. Be sure to use the setting in both Debug and Release builds.
The BUILD_LIBRARY_FOR_DISTRIBUTION Xcode build setting and associated .xcframework support were presented at WWDC 2019 in the talk titled Binary frameworks in Swift.

Hi @MarkWatson16!  Thanks for bringing this to our attention.  We will work on updating this in our next release.

Awesome, thanks @mliao95!

Hi, we have released new version to include above.
https://github.com/amazon-connect/amazon-connect-chat-ios/releases/tag/v1.0.2
Please feel free to open if issue persists!


Added correct icons to order steps.

Changes

added correct icon for failed order step
Add Order initiated override that will show success icon if it did not fail and approval was already approved
hide step info after first failed step.

Failed order
Before

After

Order initiated override
Before

After

Codecov Report

Merging #172 into master will decrease coverage by 0.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   82.36%   82.34%   -0.02%
==========================================
  Files          86       86
  Lines         822      827       +5
  Branches       68       71       +3
==========================================
+ Hits          677      681       +4
- Misses        132      133       +1
  Partials       13       13

Impacted Files
Coverage Δ

src/smart-components/order/create-order-row.js
92.85% <85.71%> (-7.15%)
:arrow_down:

Continue to review full report at Codecov.

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


[INLONG-6258][Dashboard] Source data delimiter changed from ASCII to symbol

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes https://github.com/apache/inlong/issues/6258

Keep the usage of ASCII for data separators.


Add LRO Grow-Up

Addresses https://github.com/azure/autorest.csharp/issues/2050 and https://github.com/azure/azure-sdk-for-net/issues/28125
Adds Convert helper method for LRO + handwritten methods in DPGClient + tests

RawLRO not covered here?

RawLRO not covered here?

@pshao25, what is RawLRO?

RawLRO not covered here?

@pshao25, what is RawLRO?

I have added at https://github.com/Azure/autorest.csharp/pull/2211


chore: go1.15 --> go1.17

Test output:
$ make test integration-test 2>&1 | grep "\-\-\-"
--- PASS: TestAxonConfigListenerMode (0.00s)
    --- PASS: TestAxonConfigListenerMode/default_(notify) (0.00s)
    --- PASS: TestAxonConfigListenerMode/specify_notify (0.00s)
    --- PASS: TestAxonConfigListenerMode/specify_replication (0.00s)
    --- PASS: TestAxonConfigListenerMode/bad_mode (0.00s)
--- PASS: TestParseSemver (0.00s)
    --- PASS: TestParseSemver/9.5 (0.00s)
    --- PASS: TestParseSemver/9.5___(And_some_other_stuff... (0.00s)
    --- PASS: TestParseSemver/9.5.1__(patch_version_ignored) (0.00s)
    --- PASS: TestParseSemver/12.34__(multi-digit_check) (0.00s)
--- PASS: TestPipeline (0.30s)
--- PASS: TestNewConfigFromEnv (0.00s)
    --- PASS: TestNewConfigFromEnv/test_with_namespace (0.00s)
    --- PASS: TestNewConfigFromEnv/test_with_no_namespace (0.00s)
    --- PASS: TestNewConfigFromEnv/test_parse_database_config (0.00s)
--- PASS: TestParseLogLevel (0.00s)
--- PASS: TestVersionMigration (38.47s)
    --- PASS: TestVersionMigration/9.5To9.6 (19.00s)
    --- PASS: TestVersionMigration/custom11To11 (19.46s)

@bernielomax added dockerfile edits in the force-push


cloudlog: 2.4.3 -> 2.4.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for cloudlog is: Web based amateur radio logging application built using PHP & MySQL
meta.homepage for cloudlog is: https://www.magicbug.co.uk/cloudlog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 2.4.4 with grep in /nix/store/8smwh95pwlm5g0n4yfhsy7ql5zs1dg71-cloudlog-2.4.4
found 2.4.4 in filename of file in /nix/store/8smwh95pwlm5g0n4yfhsy7ql5zs1dg71-cloudlog-2.4.4

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cloudlog

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8smwh95pwlm5g0n4yfhsy7ql5zs1dg71-cloudlog-2.4.4 \
  --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 cloudlog https://github.com/r-ryantm/nixpkgs/archive/181e4986d1aa8da64f0ae3810f176ddd9cbdcd2e.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/8smwh95pwlm5g0n4yfhsy7ql5zs1dg71-cloudlog-2.4.4
ls -la /nix/store/8smwh95pwlm5g0n4yfhsy7ql5zs1dg71-cloudlog-2.4.4/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:

cloudlog

Maintainer pings
cc @mattmelling for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for cloudlog is: Web based amateur radio logging application built using PHP & MySQL
meta.homepage for cloudlog is: https://www.magicbug.co.uk/cloudlog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 2.4.5 with grep in /nix/store/4k868ymrqpzds4f1mm5pld1zl8ziq4g6-cloudlog-2.4.5
found 2.4.5 in filename of file in /nix/store/4k868ymrqpzds4f1mm5pld1zl8ziq4g6-cloudlog-2.4.5

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cloudlog

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4k868ymrqpzds4f1mm5pld1zl8ziq4g6-cloudlog-2.4.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 cloudlog https://github.com/r-ryantm/nixpkgs/archive/bb8999eaf4b6e5fa738448ca98c92da0005a0f9b.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/4k868ymrqpzds4f1mm5pld1zl8ziq4g6-cloudlog-2.4.5
ls -la /nix/store/4k868ymrqpzds4f1mm5pld1zl8ziq4g6-cloudlog-2.4.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.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cloudlog

Maintainer pings
cc @mattmelling for testing.

still lgtm :)

Builds are failing for linux on ofborg.

Builds are failing for linux on ofborg.

Yes, module test was failing, pr at #244463. Package is otherwise fine.


Merge release 0.17.2

This merges the 0.17.2 release

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: kantai:white_check_mark: BrunoBernardino:white_check_mark: brandonparee:white_check_mark: bitpshr:x: larrysalibraYou have signed the CLA already but the status is still pending? Let us recheck it.


pants version not detected corrected in non-conventional pants.toml location

Hi, I am trying out the SCIE pants and specify pants 2.16.0rc0 in pants.toml. This file is put under a subdirectory of my repo so before I have an env var PANTS_TOML to indicate that. however I found this is not going to work. If I move pants.toml to the top level it would work.
The problem I am seeing is
The `pants.toml` at /Users/xxxx/xxxx/xxxxx/pants.toml has no `pants_version` configured in the `GLOBAL` section.
Would you like set `pants_version` to the latest stable release? (Y/n):

and if I select no, it would crash with
ERROR: Failed to establish atomic directory /Users/xxxx/Library/Caches/nce/142382d6e3ac0960e7d4f81b9df9de9559b7ac698db13d20cbd0e4aa57f68c1d/locks/configure-a04b16475909adc53343e9b2a72885b57d783b5743b4fbcdad57d8d31e81ace7. Population of work directory failed: Boot binding command failed: exit status: 1
if I select yes, it would update my pants.toml to 2.15.0

Thanks for pointing this out @clsx524. I now see I missed this feature in the ./pants script when porting to scie-pants:
https://github.com/pantsbuild/setup/blob/15988fe15cc87031235ecd0e94ebc38df692a910/pants#L36-L39

I'll not @clsx524 you will need to mark your build root still. Now that the ./pants script is gone, that cannot be the marker any longer and so you'll need to create an empty BUILD_ROOT file at the root of your repo and check that in.

@clsx524 the fix is released in scie-pants 0.6.0. You can run SCIE_BOOT=update pants to grab the update.


C# Error with Unique Node - returns always null

Godot version
4.0 8-Beta
System information
Ubuntu 22
Issue description
I encountered a bug in C#. When I want to get a unique node, the return value is null.
I think this issue has something to do with singleton scenes added to the scene tree.
Steps to reproduce

Create C# Project
Add a "Unique Node"
Add a Singelton scene to the project
Getting the Unique Node doesn't work anymore and returns null

Link for the minmal project https://github.com/OatsProduction/Godot-unique-node-bug-demo
Minimal reproduction project
Godot-unique-node-bug-demo-main.zip

When running the MRP I'm getting this error:
ERROR: System.InvalidCastException: Unable to cast object of type 'Godot.Control' to type 'Hud'.

Looking at the scene Main.tscn, it seems the Hud node does not have the Hud.cs script attached. When the script is attached everything seems to be working correctly, I get:
ERROR: Node not found: "%Hud" (relative to "/root/GlobalScript").

This is expected because you are trying to access a unique node from a separate scene. Unique nodes are only accessible from the scene they are in because they are only unique in their scene, you can have multiple unique nodes with the same name as long as they are in different scenes.
Also note that this is not exclusive to C# but also happens with GDScript. I suppose this could be better documented in https://docs.godotengine.org/en/latest/tutorials/scripting/scene_unique_nodes.html so I'll mark this as a documentation issue.

I just ran into this issue as well, but I am having it happen when I am trying to access a CharacterBody3D from a Control Node. Is this also intended?
This is all in the effort of finding if a player is stationary. I can either deal with non-unique path figuring out or have a unique name that allows me to look up the players state. The work around at the moment is that the CharacterBody3D calls the general group that these nodes are in and informs them that it is moving (not ideal in any sense).

I just ran into this issue as well, but I am having it happen when I am trying to access a CharacterBody3D from a Control Node. Is this also intended?

Whether it's possible to access a node by their unique name or not does not depend on the node type. A scene unique node can only be retrieved by a node inside the same scene.
There's a PR that adds more clarification about how scene unique nodes work and should close this issue: https://github.com/godotengine/godot-docs/pull/6670.
Please, take a look and let us know if that resolves your questions about how it works. Here's a semi-rendered view of the doc page from the PR:

https://github.com/31/godot-docs/blob/dev/31/sceneunique/tutorials/scripting/scene_unique_nodes.rst

@raulsntos Sorry for the very late answer. This solves the issue for me.
Thank you :)


Chrisda to Master

Docs Build status updates of commit 35aedb7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Add-ADPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Add-MailboxPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Add-PublicFolderAdministrativePermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-Group.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MessageTraceDetail.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-OrganizationRelationship.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-ADPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-MailboxPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-PublicFolderAdministrativePermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-OrganizationRelationship.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Grant access for repository yjaf-gatway-proxy to Steve Hobden

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository are you requesting access to (e.g. money-to-prisoners)?
yjaf-gateway-proxy
What is the GitHub username of the person who needs access?
stephenhobden
What is their full name?
Steve Hobden
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "MadeTech", "Department for Education")?
NECSWS
What level of access should they have?

[x] Read
[ ] Triage
[x] Write
[ ] Maintain
[ ] Admin

Why is this access being requested?
For work on the gateway proxy project
Which MoJ team/person is responsible for this request? This must be someone with a "*.justice.gov.uk" email address.
This should be in the form Awesome Team<EMAIL_ADDRESS>Team details are preferred, rather than individuals, to make it easier to follow-up at a later date.
Thomas Tipler<EMAIL_ADDRESS>Review date
Date after which this collaborator's access should be reviewed. This should be a date in the format: 2021-01-15, no more than one year into the future.
09/02/2022

Hi @ttipler,
Grateful if you could confirm Steve's GitHub ID. Info provided is not a GitHub ID.
Look forward to hearing from you.

Hi Antony,
I have updated the request with steve's username it should be: stephenhobden
Thanks
Tom
On Wed, 9 Feb 2022 at 11:36, Antony Bishop @.***> wrote:

Hi @ttipler https://github.com/ttipler,
Grateful if you could confirm Steve's GitHub ID. Info provided is not a
GitHub ID.
Look forward to hearing from you.
—
Reply to this email directly, view it on GitHub
https://github.com/ministryofjustice/github-collaborators/issues/464#issuecomment-1033666430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWD3XHG5GCEPQN7U7KOSIQTU2JGUXANCNFSM5N46EQLQ
.
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:
@.***>

--
This email is sent on behalf of NEC Software Solutions UK Limited and its
associated companies (together "NEC Software Solutions") and is strictly
confidential and intended solely for the addressee(s). Full details of
those companies can be found at
https://www.necsws.com/group-company-details/
https://www.necsws.com/group-company-details/
If you are not the intended
recipient of this email you must: (i) not disclose, copy or distribute its
contents to any other person nor use its contents in any way or you may be
acting unlawfully; (ii) contact NEC Software Solutions immediately on
+44(0)1442 768445 quoting the name of the sender and the addressee then
delete it from your system.
NEC Software Solutions has taken reasonable
precautions to ensure that no viruses are contained in this email, but does
not accept any responsibility once this email has been transmitted.  You
should scan attachments (if any) for viruses.

Many thanks


removed link to wallet

@thisisjoshford I removed the link to the wallet page in the getting started section since the AccountID walk-through already covers how to use the wallet.
However, I left the page that simply links to the NEAR wallet so that if people look specifically for the wallet in the search bar, they can still find it.

Hey @AnaisUrlichs!  Just reviewed this PR and totally agree with your thoughts on redundancy and that we should consolidate our documentation. This solution you provided, however, does leave an orphan page that doesn't flow with the rest of our documentation. (see below)

There are a few orphan pages that we need to clean up and would rather not add another one to them. I think what could be a more elegant solution would be to either update both the create-account.md and near-wallet.md to complement each other better or update create-account.md to have a cleaner wallet section that will show up for searches.

@thisisjoshford create Account ID covers everything; the problem is that if we name it wallet, crypto-newbies (which the get started section is targeting), will be confused. However, if we name it Account IDs or similar, people looking for the wallet will be confused.
I would rather, either

remove the wallet page and try to update all associated links
only provide the link to the wallet and link to the Account ID creation tutorial => in case the page is linked anywhere externally people would at least find valuable resources instead of an empty page link

What do you think?

This is getting completely redone shortly, so I closed this PR :)


Can I use the content of as course material for my education website for kids?

Is your feature request related to a problem? Please describe.
I have a education website for kids https://codekidz.ai, which focus in teaching kids programming and AI, taught by virtual AI teacher. I am currently looking for high-quality course content and partners. The "generative-ai-for-beginners" has a great course structure and content. I hope to create a simplified version of the course on my website (for example, removing steps like installing Jupyter). Who should I discuss this matter with?

Hi @sailxjx , sound very interesting. Please reach out to me at koreyst [at] microsoft [dot] com

Hey @koreyspace, as we've discussed previously, I'll try to incorporate the content into our education website for kids, the first version will surely have many issues, let's optimize it together. If you have more suggestions, such as how to add executable sections on the web (preferably offline), feel free to discuss with me, my email is jingxin at codekidz.ai.
I will close this issue, but will keep following up.


Publish alpha/beta/rc releases

I noticed that the 4.0 branch now has 4.0.0-alpha.1 version in the package.json, but that it hasn't been published. Are there any plans to publish alpha/beta/rc releases along the way to 4.0.0?

:+1:

This isn't likely.  The reason the 4.0 branch's version was changed was only to work with the new CLI.  npm makes it easy to install from a branch, so there is no real need to publish prereleases to the registry.

@phated many more people are more confident in published npm releases than installing from a Git branch. Also, branches can change, npm releases really don't..does it hurt to publish it if you are going to bump the version?

Agree with @ilanbiala  . Though you can also fix the version by appending a commit hash to the url (e.g. https://github.com/gruntjs/grunt/archive/77155e3f61d213c529a16ea595fc486fdeca28c9.tar.gz) .
Anyway people are usually mentally more comfortable with using an npm release.
Another serious problem I had in the past with including git dependencies, is that npm downloads them using git and then git's own proxy settings apply, instead of the ones from npm.
With npm dependencies I could simply set an "always" proxy, because npm deps are always outside of my corporate network, but with git I have to set mixed rules and excludes (because of our internal repos) and in addition the git protocol is blocked by our corporate proxy. I spent some time to setup it up correctly and now my git simply rewrites git:// urls to https://, but that requires at least git 1.8.4.
git config --global url."https://github.com/".insteadOf<EMAIL_ADDRESS>To cut a long story short - npm releases simply work, git dependencies are (sometimes) complicated.

though that doesn't help when someone rewrites the history - deletes the commit. .

Not really true in practise. GitHub preserves it for a long time, as long as you have the SHA.

But I do agree npm releases are better for this.

Yes, please publish an rc, this way it'll be more likely to be accepted and used.

@contra @phated @sindresorhus a prime example, this SHA 3eef36ab126e5b858884c44dfbee3a0aecd73fdb used to exist, and now it doesn't seem to resolve properly on GitHub...so please someone publish Gulp#4.0.0-alpha1 and more alphas and betas and rcs....

I've started slowly migrating to Gulp 4.0 and agree, it would be really nice to have some kind of RC/beta releases :+1:

:+1:  +1 , but sha's should be good enough for npm-shrinkwrap correct?

this should be published to npm as an alpha :+1:

:+1:

@phated Any notes as to why we can't publish this to npm? I seem to remember you having a reason on IRC

@phated @contra :+1:

:+1:

The 4.0 publish path is going to mirror npm3.  Locking this

Closing in favor of #1486


Different line color depending on the source: RE80 (and probably other lines too)

The RE80 has different line colors in official documents:
AVV network plan: #006da7 (+ rounded)
Bavaria network plan: #008fbf
GABY network plan: #096da9
GABY timetable: #006da7 (current color in line-colors.csv)
VGN network plan: #096da9
Which color should be used?

This is like a bit complicated because this line operates in more than one transport network.
Therefore I would suggest using the colors communicated by Go-Ahead. I'll add this up to the specs, too.

I clarified the case "Single line operates in multiple local transport networks"

Therefore I would suggest using the colors communicated by Go-Ahead.

But even Go-Ahead communicates two different colors. Which one of them takes precedence? The color from the network plan or the color from the timetables?

I'll add this up to the specs, too.

Maybe a priority ranking would be helpful, because sometimes the operator does not communicate a line color and shape at all, e.g. RE16 of DB Regio Bayern AG:
AVV network plan: #d1050c (+ rounded)
BEG network plan: #e50000
DB Regio AG Bayern: not found
VGN network plan: #d91920
For example:

operator network plan
other resources from operator
transport network plan (but which takes more precedence?)
local transport authority resources


Container reuse across r2dbc:tc and jdbc:tc URLs

Could you please enable container reusage across r2dbc and jdbc urls?
Use case (in Spring Boot):
spring:
  r2dbc:
    url: r2dbc:tc:postgresql:///test?TC_IMAGE_TAG=12-alpine
  flyway:
    url: jdbc:tc:postgresql:12-alpine:///test

https://github.com/testcontainers/testcontainers-java/issues/4282
https://github.com/spring-projects/spring-boot/issues/28076
https://github.com/flyway/flyway/issues/3282

HI @yerzhant,
thanks for raising this issue and brining it to our attention, this is indeed a very interesting feature. However, implementation might be a bit more involved, so I can't promise when we will have time to work on this.
Contributions by the community exploring possible implementations are welcome.

Does anyone have any current workarounds for this? This seems to block any kind of integration tests that leverage Flyway (since they don't have r2dbc support, see here)

@unredundant
You can always use the regular Java object-based Testcontainers integration and use @DynamicPropertySource to construct your specific URLs from the same container object.

@unredundant you can find an example of the workaround here: https://github.com/ThomasVitale/cloud-native-spring-in-action/blob/main/Chapter08/08-end/order-service/src/test/java/com/polarbookshop/orderservice/OrderServiceApplicationTests.java#L38

I think for JDBC this works by appending ?&TC_REUSABLE=true.
I'm using the following:
spring:   datasource:     flyway:       url: jdbc:tc:sqlserver:2019-latest://testcontainers/testdb?TC_REUSABLE=true

I think for JDBC this works by appending ?&TC_REUSABLE=true. I'm using the following:
spring:
  datasource:
    flyway:
      url: jdbc:tc:sqlserver:2019-latest://testcontainers/testdb?TC_REUSABLE=true

Would be good to adjust the documentation accordingly if this is indeed intentional 😸

I think this was done with https://github.com/testcontainers/testcontainers-java/pull/6216: https://java.testcontainers.org/features/reuse/


Update de.json

Scrolled through de.json and noticed, that some things were not translated.

Thank you @Staubgeborener

Well. That was fast.


Filter to only starting tasks when reconciling

This should fix https://github.com/HubSpot/Singularity/issues/2122 by filtering properly to only launching tasks. It was previously targeted all tasks that were not starting which I am pretty certain the opposite of the intent. Please review #2122 for all the details.

Thanks for the PR, my internet is out and I haven’t been able to work on much via LTE :cry:

@pschoenfelder can you help make sure this makes it through the branches and gets merged?

👍 Will start rolling out on Monday.

Looks like the merge resolution screwed up the formatting and the formatter is angry (why CI failed). I can fix Monday morning.

Coming back to this, it seems to have had the opposite effect on our end. Reading the logic again if we want all tasks in a launching state and we are inspecting the the presence of a 'starting' history event in zk, the previous logic was actually correct. If the 'starting' event is not present in zk, explicitly reconcile the task because we want to see if we missed the first event back from the executor for it
This seems to point to #2122 being caused by a separate issue and tasks not actually making it from launching -> starting for some reason in your cluster
I'm going to open a PR to revert this for the moment as it is dramatically slowing down some of our internals. Feel free to stay on the older commit for the moment and we can keep looking into the cause of your issue as well @relistan


Multi-file Join

Hey there, been a while. :)
I just tired to use tsv-join on a bunch of files and was very surprised when it applied the filter file to each of them in turn.  Is there an approach that lets me nicely pull them all together that I've missed?
Why?: One of the really nice things about tsv-join is, since you're working with a key, ordering doesn't matter and holes in the data can be mitigated.  But it's clumsy to have to repeatedly join a whole pile of intermediate things to get an end result.
To clarify: with the following data files...
left
label   v1
foo     1
bar     11
baz     111

centre
label   v2
foo     2
bar     22
baz     222

right
label   v3
foo     3
bar     33
baz     333

...I'd like to see something like this:
$ tsv-join -f left -k 1 left centre right
label   v1      v2      v3
foo     1       2       3
bar     11      22      33
baz     111     222     333

Instead, that just prints each file in turn.  (N.B. I reused the filter file as a data file because I want to preserve the column ordering.)
Trying this instead...
$ tsv-join -f right -k 1 -a 2 left centre
label   v1      v3
foo     1       3
bar     11      33
baz     111     333
label   v2      v3
foo     2       3
bar     22      33
baz     222     333

...closer.  Sort of.  (I guess?)
This is the best I've got right now and I think it should be reliable but I haven't tested it on something substantial:
$ tsv-select -f1 left > labels
$ for I in left centre right; do tsv-join -f left -k1 $I | tsv-select -f2 > intermediate-$I; done
$ paste labels intermediate-{left,centre,right}
label   v1      v2      v3
foo     1       2       3
bar     11      22      33
baz     111     222     333

Hey, good to see you back!  So, there is a way to do this:
$ tsv-join -f right -k 1 -a 2 centre | tsv-join -f - -k 1 left -a 2-3
label	v1	v2	v3
foo	1	2	3
bar	11	22	33
baz	111	222	333

I agree though, the current tsv-join APIs are clunky for this case. And it's a common enough case. Generally I'm pretty happy with the APIs of the TSV Utilities tools, but this one could be better. It's on my list to create a alternate API or perhaps a another tool that would make this simpler. Perhaps tsv-join-files tool, or something like this. I'm not sure when I'll get to it though.

Here's another way in the current tool. This will probably feel more natural than the first example I gave:
$ tsv-join left.tsv -k 1 -f centre.tsv -a 2 | tsv-join -f right.tsv -k 1 -a 2
label	v1	v2	v3
foo	1	2	3
bar	11	22	33
baz	111	222	333

Both forms are doing similar things: join two files in the first tsv-join call, passing the output to another tsv-join call. Every subsequent tsv-join adds one more file. The second version should be better because the files are added in the order you want them listed. And, the same column numbers are appended each time. In the first form the column numbers have to be adjusted each call.
The results of the two styles are the same if there are no duplicate keys. This is normally the case when doing this type of multi-file join operation.
The time duplicate keys matters is when you are using one file as a filter on another the other. In that case, the "filter file" (-f|--filter-file) is being used as a filter on all the other files. The other files may have duplicate keys, the join doesn't care. (It's streaming join, so you can have an infinite length input.)

Ah, thanks for the tip.  So generalising it looks something like this:
function rabid-join {
        for I in $*; do
                if [[ -z $SEED ]]; then
                        SEED=$I;
                        continue;
                elif [[ -z $FORE ]]; then
                        FORE="tsv-join $SEED -k 1 -f $I -a 2";
                        printf "$FORE"
                        continue
                else
                        printf " | tsv-join -f $I -k 1 -a 2"
                fi
        done
}

eval $(rabid-join left centre right more-right further-right absurdly-right)

Only problem I have is I can't seem to convince this to deal with the situation where a later file has keys the first one does not.


407 Proxy Authentication Required

Describe the bug
The bug cause when using smart proxy to bard because my location waasn't matched
Version
OS: Window 11
Python: 3.10.13
Bard API: Latest
Using proxy: Smart Proxy
Legion:
Code
# ... Import thing ...
proxy_url = "http://xxxxxxxxx@smartproxy.crawlbase.com:8012"
proxies = {"http": proxy_url, "https": proxy_url}
newSessions = requests.Session()
# ... Some other code ...
newSessions.headers = SESSION_HEADERS
newSessions.cookies.set("__Secure-3PSID", token)
bard = Bard(token=token, session=newSessions, proxies=proxies)
answer = bard.get_answer(prompt)['content']

Error
HTTPSConnectionPool(host='bard.google.com', port=443): Max retries exceeded with url: / (Caused by ProxyError('Unable to
connect to proxy', OSError('Tunnel connection failed: 407 Proxy Authentication Required')))

Hamza-Ikhlaq might be able to help you.
This is not an issue or error that can be resolved at the package level.

This issue is a difficult problem to address at the package level, so it might be better to seek help from crawlbase.
Therefore, I am closing this issue. Thank you.


Added Charts

Added the following charts:
Line Chart - Stepped
Line Chart - Interpolation
Line Styles
Point Styles
Stacked Group
Line Chart with Different Point Sizes
Area Line Boundaries Chart
Stacked (Line)
Dataset (Line)
Radar Chart
Doughnut
Polar Area Chart
Scatter Chart
Scatter (Multi Axis)

Are the changes of this PR included in https://github.com/thombergs/wicked-charts/pull/101?


Missing Window Functions from Polars

Polars has the following "rolling" (explorer calls it window) functions:

polars.Series.rolling_apply
polars.Series.rolling_max
polars.Series.rolling_mean
polars.Series.rolling_median
polars.Series.rolling_min
polars.Series.rolling_quantile
polars.Series.rolling_skew
polars.Series.rolling_std
polars.Series.rolling_sum
polars.Series.rolling_var

Although some of the rolling computations are already available in Explorer, the absence of a rolling_apply equivalent makes it less convenient to calculate certain statistics and models.
As a result, users are forced to resort to workarounds that are far from ideal. For example, one could calculate the 21-day rolling standard deviation by using additional columns and a combination of existing functions. However, for users familiar with Pandas, this approach can feel unusual.
Is there any plan to support rolling_apply in Explorer, or am I overlooking something?

I don't think we can support rolling_apply because it is not possible to call Erlang from C/Rust without using message passing. As far as I see, the python version linked is fully implemented in C.
Can you provide a more concrete example that you are trying to address and how you are addressing it? Perhaps we can provide higher level conveniences without having it named rolling_apply itself?

Would you like to send a PR for Series.window_map btw?

Created this PR to explore a bit the codebase and test the waters. Waiting for review on it to make sure everything is ok. After that I plan on adding a bunch of functions that I need as well. Hope it helps.

Hi, I also have a use case for this. Here's equivalent code in Python:
df['atl'] = df['tss'].rolling(window=7).apply(lambda x: calculate_atl_recursive(x))

Here I'm applying a calculation using a rolling window (or nil values when I'm out of historical data to cull). I can't solve this with the current package API, unless I'm missing something.
Thank you!


Consider new transaction type: Distribute to Token Holders

Is your feature request related to a problem? Please describe.
The new tx type carries only the entire value to be distributed to all token holders.
Describe the solution you'd like
It should show the real value that I got.

Available since 1.2.1


Will the model weights be released?

We can directly obtain the experimental results reported in your paper in this way.

Sorry bro, the weights of the model were lost due to the break of my portable storage device. However, I believe you can train the model yourself, and the method itself is not sensitive to the seed, which can be reproducible anyway.

Hey bro, the instruction and weights you used are completely correct, and the paper's result is mPQ 53.6 on the CoNSeP (Det+Seg+Cls) task, which is very close to the 52.2 result you achieved. Considering this, I think there are many factors that could cause this slight gap, such as the version of torch (1.10), platform (Linux or Windows), etc. Therefore, I suggest you try running it again with a different seed. Good luck


Release Google.Cloud.AIPlatform.V1 version 3.3.0

Changes in this release:
New features

Add UpdateDeploymentResourcePool method to DeploymentResourcePoolService v1 (commit 66af553)
Add private_service_connect_config and service_attachment fields to DedicatedServingEndpoint v1 (commit 66af553)
Add satisfies_pzs and satisfies_pzi fields to Model v1 (commit 66af553)
Add deploy_metadata to PublisherModel.Deploy v1 (commit 66af553)
Add satisfies_pzs and satisfies_pzi fields to Tensorboard v1 (commit 66af553)

Documentation improvements

Update comments of AutoscalingSpec v1 (commit 66af553)

Created release for Google.Cloud.AIPlatform.V1-3.3.0


Release v0.35.0 on Dec 14, 2022 release

Checklist

[x] Create the new release branch with minor version, i.e. release-0.21 for v0.21.0.

You can use the existing branch for patch releases.

[x] Tag release by running Tag action in Github UI against release branch, i.e. release-X branch.

Use v-prefixed version, like v0.21.0, for both description and tag.

[x] Run CI action against release branch.
[x] Tag the next pre-release by running Tag action in Github UI against master branch, if it's not a patch release.

The tag should be v0.22.0-rc.0 if you're releasing v0.21.x.

[x] Validate that you can install the published version, basic sanity check.
[ ] Run Promote action to promote it in alpha channel if it's pre-1.0.
[ ] Use Github UI to generate release notes.

Make sure to scan all PRs marked with breaking-change and add instructions for users to handle them.

[x] Create the next release issue with a title that has its date, 4 weeks after the current release day.
[ ] Announce in Slack, Twitter etc.

tested upgrade v0.34.0 -> v0.35.0
NAME                  INSTALLED   HEALTHY   PACKAGE                                                  AGE
provider-aws          True        True      docker.io/crossplanecontrib/provider-aws:v0.35.0         22d

tested basic resources
NAME                                     READY   SYNCED   AGE
bucket.s3.aws.crossplane.io/test-wwh34   True    True     22d
bucket.s3.aws.crossplane.io/test-wwh35   True    True     2m14s


Desktop 1.6 on Win 10 thinks it's an Xbox game?!

I don't know if I've just got a weird install, but every time I start Ghost Desktop, it gives me some popups about recording my game, or sometimes it says "Press Windows + G to start", which then offers me to open GameBar.  🤔

😆 Oh Windows... it's possible that you once, by accident, hit the secret Windows Game shortcut (Win + (Alt) + G). I'm not entirely sure how you turn that off once it happened...
https://support.xbox.com/en-US/xbox-on-windows/social/record-game-clips-game-dvr-windows-10. I did find this though: https://support.steampowered.com/kb_article.php?ref=6239-DZCB-8600
Either way, I'm afraid that this isn't something we can control from within Ghost, so I'll have to close this. Sorry!


Add fuzzing blogpost

To be published on 30th June.
@yaron2 @artursouza @ItalyPaleAle

@AdamKorcz there's a technical issue where the PR must not come from a fork. Please re-open the PR from a branch in upstream

https://github.com/dapr/blog/pull/112#issuecomment-1593193698

Opened here: https://github.com/dapr/blog/pull/115


I often get the error Exception: net::ERR_SOCKS_CONNECTION_FAILED

Describe the bug
The watches often result in the error Exception: net::ERR_SOCKS_CONNECTION_FAILED. However when I relaunch the check, it usualy works.
I use the cloud product changedetection.io, I wonder if it's a hosting issue (to many watches ran at the same time maybe?)
Version
Exact version in the top right area: 0.46.02
Expected behavior
To not have the error
Screenshots

Addictional context
I have 170 watches that are run every 6 hours. I have once do a "recheck all", so I guess they are all scheduled roughly at the same time every 6 hours

thanks! looking into it, i can reproduce it

Thank you very much :)

should be solved now, let me know if theres more problems :)

Thank you !! :)


Is there a way to provide grpc call options like MaxCallRecvMsgSize

I would like to increase the default size that can be received by the API that I am testing. Is there a way to provide this as a CLI option?

Hello, thanks for submitting this issue. This is not possible right now. Will look into adding this.

Should be supported in 0.99.0.
New CLI options --max-recv-message-size and --max-send-message-size can be specified using human readable value such as 42 MB.
There is a new API WithDefaultCallOptions() which can be used to specify the default call options (including max sizes).


browser-external:source-map-js:9 Module "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapConsumer" in client code. See https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details

getting this error when i am using it in clientcode (browser)

We use this module only the server side, and in most cases that is best practice. Front end use cases exist, support for that comes from the community so I've flagged this as "seeking contributions." It is not a bug as the code functions as designed.

The annoying warning originates from the postcss plugin. Here is a suggestion to ship postcss as ESM which would potentially fix it: https://github.com/postcss/postcss/issues/1858
Alternatively exchange postcss plugin.


Packages for OS X Python 3.4 and 3.5 missing

Appears there were failures to upload the OS X packages for Python 3.4 and 3.5 from Travis CI. Have restarted both of those builds. Hopefully that will fix the issue. Please feel to restart again if needed. Please close this once those packages are uploaded.
cc @pmlandwehr

Appears to be fixed.


Unable to stop crawler. Facing error: "'PlaywrightCrawler' object has no attribute 'stop'"

Bug Report: stop() method missing in Python PlaywrightCrawler
Description
While using the Python SDK of Crawlee with PlaywrightCrawler, I encountered an issue where the stop() method mentioned in the documentation is not available. This is particularly problematic when trying to implement custom crawling logic that requires programmatic stopping of the crawler.
When attempting to call pw_crawler.stop() from within the request handler, I receive the error:
AttributeError: 'PlaywrightCrawler' object has no attribute 'stop'
Expected Behavior
According to the documentation, we should be able to call crawler.stop() to gracefully terminate the crawling process. This is especially important when implementing custom logic like:

Stopping after processing specific content types
Handling custom request limits
Managing crawler state based on business logic

Current Behavior
The Python implementation of PlaywrightCrawler doesn't expose the stop() method, making it impossible to programmatically stop the crawler from within async request handlers or navigation hooks.
Impact
Without a proper stop mechanism, developers are forced to:

Raise exceptions to terminate crawling (which feels hacky)
Implement complex state management
Cannot gracefully stop the crawler while preserving the crawled data
Risk potential memory leaks or resource management issues

Environment

Crawlee Python: 3.11
Python Version: 3.11
Platform: Linux/Docker

Additional Context
This is critical for production environments where we need precise control over the crawling process, especially when:

Implementing custom request limits
Handling mixed content types (web pages and files)
Managing crawler resources in containerized environments
Implementing graceful shutdown mechanisms

Would appreciate clarification on the recommended approach for programmatically stopping the crawler in the Python SDK.

Hi @abhichek and thank you for your interest in Crawlee! This feature is not yet released - see the changelog. If you want to test this before 0.5 is released, you can install a beta - https://pypi.org/project/crawlee/0.5.0b21/

Hey @janbuchar , thanks for the response! The beta version looks promising, but for production environments we need the stable release. Could you share an estimated timeline for when version 0.5 will be released?
Also, is there a recommended approach for gracefully stopping the crawler in the current stable version? We're seeing some edge cases where Crawlee's internal request counter and our custom counter get out of sync, and a proper stop mechanism would help us manage this better.
Thanks.

Hi @abhichek, we will probably release 0.5 in the first days of 2025 - most of us will not be available during Christmas to provide support and hotfix any possible bugs.
That aside, crawlee-python still hasn't reached 1.0, so calling it stable is a bit of an overstatement. We're happy that you are using it nevertheless :slightly_smiling_face:


Add ImageStreamTag change trigger for Kubernetes resources

This will fix #1291.
Change does not affect standard deployments.

@nicolaferraro Would you like to rebase and push again.

sure, done

I should have done all necessary changes, can you please re-check @hrishin ?

[merge]


It is not initialized. If the code setXXX is not used, it cannot be used

It is not initialized. If the code setXXX is not used, it cannot be used

I'm not sure I understand what you mean. If you setup your CircleImageView using XML, setImageDrawable will be called internally in the same way as you would manually (have a look at the constructor code of ImageView).


Add enumeration for NetworkDevice types

We want to create an enumeration for NetworkDevice types and this will be used as input for the related methods:
https://github.com/metasys-server/basic-services-dotnet/blob/e3d5c21d51ab502f6d25b8285ddad01ece5b084b/MetasysServices/MetasysClient.cs#L762-L783
We could look for an enumset on the Metasys dictionary that contains the list of all the device types.

I committed the changes to solve this issue to the branch 'release-4.1'


Update scala-library to 2.13.6

Updates org.scala-lang:scala-library from 2.13.4 to 2.13.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.13.4).
You might want to review and update them manually.
.travis.yml
README.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

labels: library-update, semver-patch, old-version-remains

Will do this bump manually


Sharenite Synchronization Failed.

Trying to sync the add-on to the site gives me this issue;
Sharenite synchronization failed.
Problem reading cookies from disk: System.IO.FileNotFoundException
From what I can figure out System.IO.FileNotFoundException typically means there is a missing file, but there is no indication of which file. Any help would be highly appreciated, thanks.

@SearchingForGryphons are you authenticated into Sharenite like on the screenshot?

If yes, and that doesn't help, then go to the 'Data folder' and delete all add-on files to start fresh.

Not actually linking seems to have been the problem :( oops,  feel kind of dumb haha, tysm for the help :)


Fix offboard actuator setpoints

Describe problem solved by this pull request
This is a backport of https://github.com/PX4/PX4-Autopilot/pull/18581, which fixed the offboard interface for setting actuator setpoints
Test data / coverage

Tested in SITL with https://github.com/ethz-asl/active-sysid

Additional context

Needed for @pascalau 's SA https://github.com/ethz-asl/active-sysid/pull/2

@tstastny Thanks!


Splash screen now adopts theme color scheme and background

Thought you might like it. Cheers, mate! :)

Implemented into the main theme. Thanks :)


Can't overlay a calculated chart and mark_rule

What happened?
Hi all,
I have been trying to overly a histogram based on standard deviations using the transform_joinaggregate and transform_calculate functions within Altair, and then create a ruler. But when overlaying the graphs it throws this error:
ValueError: DataFusion error: Schema error: No field named "Rating_std". Valid fields are _vf_order, "Title", "US_Gross", "Worldwide_Gross", "US_DVD_Sales", "Production_Budget", "Release_Date", "MPAA_Rating", "Running_Time_min", "Distributor", "Source", "Major_Genre", "Creative_Type", "Director", "Rotten_Tomatoes_Rating", "IMDB_Rating", "IMDB_Votes".

Is there a workaround it? Naturally I can change the DataFrame and just call the variable, although if I could just do it in one shot in the library, it would be awesome.
Below the using the movies dataset.
Thanks!
Best
from vega_datasets import data

movies = data.movies()
movies.head()

rating = (
    alt.Chart(movies, title="Movies Rating Histogram - SD")
    .transform_joinaggregate(mean_val="mean(IMDB_Rating)", std_val="stdev(IMDB_Rating)")
    .transform_calculate(
        Rating_std="(datum.IMDB_Rating - datum.mean_val) / datum.std_val"
    )
    .mark_bar(color="green")
    .encode(
        alt.X("Rating_std", type="quantitative", title="Rating", bin=True).bin(
            maxbins=40
        ),
        alt.Y("count()"),
    )
    .properties(width=400)
)

rule = (
    alt.Chart(df)
    .mark_rule(color='red')
    .encode(
        x=alt.datum(0),
    )
)

alt.layer(rating, rule)

What would you like to happen instead?
A nice overlay of both graphs!
Which version of Altair are you using?
5.3.0

Hi @ale-dg, thanks for the report.
It looks like this error is coming from VegaFusion. Could you try again without VegaFusion enabled? You can run this to turn if off for testing.
import altair as alt
alt.data_transformers.enable("default")

This will help us narrow down where the issue is cropping up.  Also, could you double check your example to make sure all of the imports and variables are defined? For example, I don't see a definition for df here.
Thanks!

Hi, @jonmmease
Thank you for the quick answer. I have made the changes and indeed removing the VegaFusion function removes the error. I have also enabled the option above and now it only shows the ruler (see screenshot below). See the code below..
Thanks
Best
from vega_datasets import data
import pandas as pd
import numpy as np
import altair as alt
alt.data_transformers.enable("default")

movies = data.movies()
movies.head()

rating = (
    alt.Chart(movies, title="Movies Rating Histogram - SD")
    .transform_joinaggregate(mean_val="mean(IMDB_Rating)", std_val="stdev(IMDB_Rating)")
    .transform_calculate(
        Rating_std="(datum.IMDB_Rating - datum.mean_val) / datum.std_val"
    )
    .mark_bar(color="green")
    .encode(
        alt.X("Rating_std", type="quantitative", title="Rating", bin=True).bin(
            maxbins=40
        ),
        alt.Y("count()"),
    )
    .properties(width=400)
)

rule = (
    alt.Chart(movies)
    .mark_rule(color="red")
    .encode(
        x=alt.datum(0),
        size=alt.value(3)
    )
)

rating + rule

Thanks @ale-dg, that's helpful.  There seems to be a Vega-Lite bug when referencing columns created by transform_calculate from an encoding channel with binning enabled, but only when the chart is present in a layer (which is why rating displays fine on its own).
I'll file a Vega-Lite issue soon, but here is a workaround that replaces the use of the encoding-level bin transform, with use of the explicit transform_bin transform.
from vega_datasets import data
import pandas as pd
import numpy as np
import altair as alt
alt.data_transformers.enable("default")

movies = data.movies()
movies.head()

rating = (
    alt.Chart(movies, title="Movies Rating Histogram - SD")
    .transform_joinaggregate(mean_val="mean(IMDB_Rating)", std_val="stdev(IMDB_Rating)")
    .transform_calculate(
        Rating_std="(datum.IMDB_Rating - datum.mean_val) / datum.std_val"
    )
    .transform_bin(field="Rating_std", as_=["Rating_std_bin_start", "Rating_std_bin_end"], bin=alt.Bin(maxbins=40))
    .mark_bar(color="green", x2Offset=-1)
    .encode(
        alt.X("Rating_std_bin_start", type="quantitative", title="Rating"),
        alt.X2("Rating_std_bin_end", title="Rating"),
        alt.Y("count()"),
    )
    .properties(width=400)
)

rule = (
    alt.Chart(movies)
    .mark_rule(color="red")
    .encode(
        x=alt.datum(0),
        size=alt.value(3)
    )
)

rating + rule

Notice that binning is not longer specified as part of the alt.X encoding. By doing this, we lose Vega-Altair's automatic bin-width calculation logic, which is why I needed to add the alt.X2 encoding as well.
Hope that helps in the meantime!

Thanks @jonmmease. I'll use this workaround in the meantime. And thank you for the help with logging the issue.
Best

Reported in https://github.com/vega/vega-lite/issues/9354. Thanks again for taking the time to report this @ale-dg, that's a big help!


Settings editor text input background lacks contrast in Quiet Light theme

In Insiders (though the issue is probably in Stable, too):

Use the Quiet Light theme
Navigate to the settings editor.
Search "emmet". emmet.extensionsPath and emmet.includeLanguages settings should show up in the results.
Add at least one non-empty entry to each setting.
Edit both entries/place both entries in edit mode.
🐛 On the inputbox that is not focused, the inputbox background colour is the same as the settings editor background colour.

@rzhao271 I'm not seeing either of those settings

but verified using files.exclude


Update the function of find command

Change the find command to also search for the newly added field Company.
Update the tests for find command.
Fixes #39

LGTM


Deploy USDCFlow.cdc to the emulator by default

Issue To Be Solved
USDCFlow.cdc for USDC is a very commonly used contract in the Flow ecosystem and developers often will want to test with it on the emulator before deploying to testnet, but it is difficult to deploy and manage.
Suggest A Solution

Update flow-go bootstrapping to deploy FiatToken.cdc to the emulator service account and store the minter in the service account storage.
Update documentation in various places to mention that it is deployed by default and devs can use the service account to mint USDC

as no more FiatToken.cdc maybe we can add USDCFlow ( but I am not sure if it requires also some kind of bridging contract deployment )

Good suggestion! I updated the title and description of the issue to reflect that. We can just deploy it to the service account and store the minter there like we do with FlowToken. I don't think we need to connect it to the bridge on emulator for now

We will need a lite version of the contract ( as it imports bridge ) and go bindings probably.

@joshuahannan any chance you can you can have a look at this one ?

yeah, I can add it to bootstrapping this week


localCache in BaseExecutor cached uncertain results.

MyBatis version
3.5.2
Database vendor and version
Mysql 5.6.16
Test case or example project
<select id="getRand" resultType="double">
		select rand();
	</select>

Steps to reproduce
session.selectOne("getRand");
session.selectOne("getRand");

Expected result
Random results from database;
Actual result
Each call returns the same value.

This behavior is specification of MyBatis because the local cache scope is SESSION by default.
You can clear local cache using SqlSession#clearCache. Or you can to prevent local cache by setting localCacheScope=STATEMENT on global configuration.

I see. Thank you.


chore: Analytics util refactor

Description
Break the AnalyticsUtil monolith into smaller parts for easier management and readability

Create separate classes of each service provider where their core logic

Segment
Mixpanel
Smartlook
Sentry

Create a separate TrackedUser class to ensure separation of concerns
Making AnalyticsUtil leaner by making it more of a controller class for the analytics providers
Removes the unused zipy library

Mixpanel Session recordings
As we move from smartlook to mixpanel gradually, made some changes to have it work on CE instances as well

Mask all fields by default
Stop session recordings while the user in on admin settings

Also check out companion EE PR to check how the classes are extended
https://github.com/appsmithorg/appsmith-ee/pull/5727
Fixes #38122
Fixes #38142
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!IMPORTANT]
🟣 🟣 🟣 Your tests are running.
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/12310761588
Commit: e3f876845616cbfca89ea4d3b07be164930550a5
Workflow: PR Automation test suite
Tags: @tag.Sanity
Spec: ``
Fri, 13 Dec 2024 06:28:34 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Summary by CodeRabbit
Release Notes

New Features

Introduced analytics tracking with Mixpanel and Segment integration.
Added a TrackedUser class for managing user tracking.

Improvements

Enhanced error handling for analytics initialization and user invitation processes.
Simplified logic by removing unnecessary dependencies and constants.

Bug Fixes

Improved robustness in obtaining user source and handling analytics events.

Chores

Streamlined code by removing outdated functions related to analytics initialization.

@hetunandu Can you also check the implementation of the fn waitForSegmentInit? We have been using this in the AppViewerEngine and AppEditorEngine.

@dvj1988 That should still work as expected as this change is still maintained
https://github.com/appsmithorg/appsmith/pull/38089/files#diff-dcc8378c79f5521536f94b2234c150ae8bea00636159191e01361439929db765R137


Provide a way to configure own SerializerAdapter instance while building XxxBlob clients

At the moment, XxxBlob client builders (BlobClientBuilder, BlobContainerClientBuilder, BlobServiceClientBuilder, ...) do not provide a way to supply own SerializerAdapter instance and rely on JacksonAdapter::createDefaultSerializerAdapter. It works in most cases but sometimes, fe. when application runs under SecurityManager and needs to use reflection, it is not sufficient and using custom SerializerAdapter would be very helpful. The underlying AzureBlobStorageImpl already supports that however the ability to pass the SerializerAdapter instance down the chain of builders is not supported.
The PR provides a way for the XxxBlob client builders to allow using user-provided SerializerAdapter instance. The change is low risk (arguably) and does not break an existing APIs.
Fixes https://github.com/Azure/azure-sdk-for-java/issues/24373.
Thank you!

Thanks for submitting the issues and filing this PR @reta!
Just have a few questions around specifics you were seeing which resulted in SecurityManager throwing errors in an application.

Did the reflective access failure occur when JacksonAdapter attempted to reflectively access a model type in a com.azure package?

The way the SDKs are designed is that com.azure models needs to be accessible by Jackson and com.azure.core, so it may be the case where one of these isn't holding true.

What is your expected use case when being able to configure the SerializerAdapter? Is it going to be a specific Thread with additional permissions?

What APIs were being used when the exception was seen? Was BinaryData being used?

@alzimmermsft went down the Jackson Databind hole, additional context https://github.com/FasterXML/jackson-databind/issues/997 and https://github.com/FasterXML/jackson-databind/issues/992. Probably, another argument to allow SerializerAdapter configuration/customization :-) (fe, enable/disable MappingFeatures, ...). Thank you.

@alzimmermsft @alzimmermsft apologies, any concerns left unaddressed? Thank you!

@reta, this is still being looked into, and unfortunately this has opened a bigger "can of worms" than initially expected. As mentioned previously, preventing Jackson from setAccessible(true) on constructors, fields, or methods results in deserialization failing to access inaccessible constructors, fields, or methods during runtime, instead of the SecurityManager exceptions of attempting to setAccessible. As you've likely seen, this is due to how our model types are annotated and handled in serialization scenarios by relying on private field access for mutating the deserialization object or for accessing the field value during serialization.
Looking forward, we're looking in ways to begin remedying this through a few different avenues:

Researching usage of MethodHandles.Lookup passed from SDKs into serialization to offer an "on behalf of" style functionality which would get around the need to setAccessible(true) as the MethodHandles.Lookup would have private lookup capabilities for a given SDK as it created the lookup.
Changing the pattern of how POJOs are annotated and handled in serialization to limit the number of scenarios where non-public constructors, fields, or methods need to be used.

Unfortunately, both of these resolutions are much, much more comprehensive and will likely take a few months to implement and verify that there are no regressions. Given that, I would love to explore other potential avenues we could take in the meantime to unblock the issue you're running into. In the replication for this issue you gave you had a code path which leveraged AccessController.doPrivileged, would this be something that could unblock your application for now?
cc: @JonathanGiles @lmolkova @srnagar

Thanks a lot for the detailed investigation @alzimmermsft , I certainly understand the complexity involved.

In the replication for this issue you gave you had a code path which leveraged AccessController.doPrivileged, would this be something that could unblock your application for now?

That is right, the cleanest way to do that is to use the AccessController.doPriviledged during the serialization / deserialization, preventing the need to weaken broader security constraints.  It could be done either using inheritance or delegation, just a quick example:
public class PriviledgedJacksonAdapter extends JacksonAdapter {
   @Override
   public String serialize(Object object, SerializerEncoding encoding) throws IOException {
       return AccessController.doPriviledged(() -> super.serialize(object, encoding));
   }
}

But the showstopper is this case is the inability to provide own SerializerAdapter, hence the feature request and this pull request, so to have a path to decorate serialization / deserialization with AccessController.doPriviledged blocks. The last resort is to run a whole HTTP worker thread (Netty or OkHttp) within AccessController.doPriviledged block :-(
Thank you @alzimmermsft !

Hi @reta, in the last release of azure-core (1.22.0) a new environment configuration (AZURE_JACKSON_ADAPTER_USE_ACCESS_HELPER) was added which escalates running JacksonAdapter serialization with AccessController: https://github.com/Azure/azure-sdk-for-java/pull/25004

Thank you so much @alzimmermsft ! I am closing this one :)


[BUG] Missing fromBackup Parameter in API Request When Restoring Multiple Files from Backup List

Describe the bug
When attempting to restore multiple files from the backup list, the create volume API request payload is missing the fromBackup parameter. Due to this issue, the restored volume size defaults to 2MB instead of the expected size based on the backup files.
Failed

Passed

To Reproduce

Navigate to the backup list.
Select multiple files to restore.
Initiate the restore process.
Check the created volume size.

Expected behavior
The latest backup should be successfully restored when selected from the backup page.
Support bundle for troubleshooting
Environment

Longhorn version: master-head, v1.8.x-head, v1.7.2
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of control plane nodes in the cluster:
Number of worker nodes in the cluster:

Node config

OS type and version:
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context

Workaround and Mitigation

cc @a110605, @houhoucoop

@roger-ryao Does v1.6.3 have the same issue?

@roger-ryao Does v1.6.3 have the same issue?

Yes, v1.6.3 has the same issue.

Thanks @roger-ryao.
Interesting. We haven't received any reports from users.

All backport merged.

Verified on master-head 20241226

longhorn master-head https://github.com/longhorn/longhorn/commit/7cd6412137171db36e06eeae82e97364558a877d
longhorn-ui master-head https://github.com/longhorn/longhorn-ui/commit/f580a92bc884a80b58d25dd0155b9fa85985faa7

Verified on v1.8.x-head 20241226

longhorn v1.8.x-head https://github.com/longhorn/longhorn/commit/ede8f46a75704352e33fbc45b01f14fa2770a827
longhorn-ui v1.8.x-head https://github.com/longhorn/longhorn-ui/commit/628773e247d865d7bdc1770d22701271f3db37af

The test steps
https://github.com/longhorn/longhorn/issues/10050#issue-2757125332
Result Passed


In use artwork: Style differently and link to ebook

Check this out, @jobcurtis:

For artwork that is in_use, we style it differently, and it has a link to the ebook the artwork is used in.
If you'd like to modify your Artworks table in place and not drop and recreate it, you can run:
MariaDB [se]> ALTER TABLE Artworks ADD COLUMN EbookWwwFilesystemPath varchar(255) NULL AFTER ArtworkPage;

Progress on #234

Small change to this PR: I made a common ArtworkStatus template because previously I had a complicated line copy–pasted in ArtworkDetail and ArtworkList.


aws_eks_addon resolve_conflicts attribute is depricated

See terraform deprecated attribute when enabling addons
│ Warning: Argument is deprecated
│
│   with module.eks.aws_eks_addon.this["kube-proxy"],
│   on .terraform/modules/eks/main.tf line 392, in resource "aws_eks_addon" "this":
│  392:   resolve_conflicts        = try(each.value.resolve_conflicts, "OVERWRITE")
│
│ The "resolve_conflicts" attribute can't be set to "PRESERVE" on initial resource creation. Use "resolve_conflicts_on_create" and/or "resolve_conflicts_on_update" instead
│
│ (and 3 more similar warnings elsewhere)

Duplicate of #2635


Feedback implementation

Overview
Last round of feedback implementations.

Gain selectors removed. No extent year or % available
Some Dynamic Sentences updated
Plantations loss maths fixed!

Made the changes! 👍


[change] v2.0.0.

This is PR is a result of all work done by @claydiffrient and other contributors.
Most commits available on master will be ported to v1 to make things easier and safer.
Small commits will be added with git am to preserve the original patch.
Some patches were reworked to clean up.
Thank you all for the great work.
Commits checklist:
6442387 [changed] use object className and overlayClassName prop to override the default content and overlay classes; use bodyOpenClassName to override body class name when modal opened [already on v1]
c184ddf Add gzip size badge [already on v1]
38dfc8d Use callback ref in readme
ea3e536 Add changelog entries from the v1 branch
37a6ceb Update README for issue close modal when user click on overlay (#352)
a8b7aa0 [chore] use afterEach to cleanup modals automatically. [already on v1]
ba2fe90 [fixed] respect closeTimeoutMS during unmount [already on v1]
ebec638 [change] improve reliability on focus management. [already on v1]
6ba1c8d (restrry/master) [fixed] Enable click to close in iOS (#301) (#304) (#313) [already on v1]
75a1aa3 [fixed] Infinite loop in tab scoping [fixed when updated javascript syntax]
ad9e115 Updates License (#303)
8108173 Modify the sample code to es2015 syntax in README.md (#295)
a776118 Get rid of CommonJS module (#299) [fixed when updated javascript syntax]
d50918c [fixed] Fix this scope (#296)  [fixed when updated javascript syntax]
e10181d Add badges to readme
3d6142c Setup coveralls and fix some code climate stuff [squashed with previous commit]
721cc1e Add language for CC duplication engine [squashed with previous commit]
a09ff8d Add code climate and code coverage
596f423 Add Linting (#293)
Commits to be added later:
baa1751 [changed] Replace appElement with getAppElement
9cb8441 Update bundling scheme (#347) [needs review]
0dc486a Add sauce labs reporter for better SL statuses
a1287bc Add cross browser testing via SauceLabs (#298)
b261459 Add sauce labs support
Acceptance Checklist:

[x] All commits have been squashed to one.
[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

Changes Unknown when pulling 8bd85b5d78430819770a6835242738524bbcb37a on diasbruno:feature/master-to-v1 into ** on reactjs:v1**.

Changes Unknown when pulling 1428a54bc118ca08fedb6f4aa224f1513ddbff54 on diasbruno:feature/master-to-v1 into ** on reactjs:v1**.

Changes Unknown when pulling 638c53ee04a552ee657500c6de4ce308197554d4 on diasbruno:feature/master-to-v1 into ** on reactjs:v1**.

Changes Unknown when pulling 60ad47d08440231a308b75851c4bab9384ad4355 on diasbruno:feature/master-to-v1 into ** on reactjs:v1**.


fix: update webpack so type-analyzer is not parsed

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
webpack.config.js has been updated to tell Babel not to parse files in the type-analyzer package because it contains CommonJS files. If Babel parses such files, it will mix require and export default in a module, which is not allowed. I have also added a text file that describes the changes. The purpose of the file is to keep track of changes we push to lyft-master but not to apache/incubator-superset so we have an idea of how close or far lyft-master is from apache/incubator-superset.
TEST PLAN
Created Kepler chart successfully
REVIEWERS
@DiggidyDave @betodealmeida @hongbosong

👍


add repace time when run history job and batch complement job ,not current time

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds checkstyle plugin.)
Brief change log
(for example:)

Add maven-checkstyle-plugin to root pom.xml

Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added dolphinscheduler-dao tests for end-to-end.
Added CronUtilsTest to verify the change.
Manually verified the change by testing locally.

please submit an issue  about this.

as the issue : #2208 said,  maybe it not a problem,  if you have any question, please come on
By the way , Thanks for your first contribtuion, Looking forward to your next , haha

for now, I knew what you said
good job

for now , We hope the unit test coverage of new code to 30%, but because it's your first contribution. come on


How to configure rotation / transformation

I'm trying out wlmaker on the MNT Pocket Reform.
It installs and starts just fine - but - the display on the Pocket Reform needs to be rotated by 90 or 270 degrees. Is it possible to change the display orientation (and ideally, scale) in the config file?

For clarity, this works fine when I run in a window under another compositor with rotation already applied, but from going through the code I cannot see that there is an existing configuration option that would allow wlmaker to be the root compositor and have a transformation applied on the display. I think there may be support for a scale already but don't see it documented.

Hej Andy! Happy to hear you've been able to install and run it successfully! (although not rotated correctly).
Unfortunately, wlmaker does currently not (yet) support output transformations.
A quick read of wlroots suggests that 90-degree rotations on the (default) output should only be one function call (wlr_output_set_transform); plus some wiring-up of a corresponding configuration setting. (hence declaring it a "good firs issue" for any contributor).
I was aiming to work on multi-output support for v0.5; but may try to add this next.

Started looking into this at https://github.com/phkaeser/wlmaker/tree/output-transform.
Finding so far:

The actual output transformation is rather easy to do indeed, already included in above branch.
Neat finding: output also permits scaling, which would be another way to help with #99.
But, mapping the pointer coordinates properly doesn't come as easily: There is a mismatch.

Will need more thinking.

Hhhmm... actually, noticing that the pointer coordinate mapping works correctly when running it on a virtual terminal; it "just" failes when running in window mode.

@andypiper -- I'd be curious if you could give it a try. It'll need compiling from head (that is: at #104), and to set Transformation to a suitable value, eg. to Rotate90 in https://github.com/phkaeser/wlmaker/blob/main/etc/wlmaker.plist and use --config_file=... accordingly.

So I have it compiled. I uninstalled wlmaker from Debian unstable, and followed the BUILD and RUN docs.
I am pretty sure that my config file is being read - I changed the terminal that gets started by default from foot to alacritty so I believe this is picking the config file I give it. If I modify Scale, this works when I run wlmaker in a window. However, Rotate90 or Rotate270 are not doing anything, either in a window, or if I run wlmaker in place of sway as my default compositor. I do not see anything in the DEBUG level of logging to show that this is either being picked up from the config file, or that there is an attempt to apply it.
On other strange (?) thing, if I copy the .plist file to $HOME/.wlmaker.plist then wlmaker fails to start with a file not found error.

Oops, embarrassing: Added with #104, but accidentally merged the actual "set_transform" away in #105. Fixing that (and adding INFO log for what it applies) in #108 . My apologies!
Can you pull head and try once more?
The config file issue is unexpected => #109 for that.

I hoped to provide a fast follow-up here... but I've been stymied, potentially by Debian unstable changes. I'm getting a linker error related to an AMD API (!) from libgallium, and mesa-libgallium was updated late yesterday... attempting to pull it out now results in a stack of libgl* dependency issues. I dug fairly far here and even went so far as blasting the local dependencies for wlmaker and starting over; and, trying a second clean install of the OS on the Pocket Reform and going from there, but that just wanted to install the same package.
tl;dr I haven't been able to rebuild and re-test yet, but I'm still interested, and hopefully I've contributed a couple of useful bug reports in the meantime 🙂 will try again when I can.

Testing wlmaker and got it running.
From attempting to setup multiple vertical monitors, it looks like the Output option currently assumes a single monitor.
Could this be configurable by output name?

@ideasman42 -- the code is currently not built to handle multiple outputs.
Question to you: Are you looking for...

A way to specify one Output for wlmaker to use (but OK to display on a single monitor only), or
multi-output support, ie. the workspace spanning across multiple screens?

The former may be doable with limited effort; the latter will be a major update, though.

@ideasman42 -- the code is currently not built to handle multiple outputs.
Question to you: Are you looking for...
1. A way to specify one Output for wlmaker to use (but OK to display on a single monitor only), or

2. multi-output support, ie. the workspace spanning across multiple screens?

The former may be doable with limited effort; the latter will be a major update, though.

The latter, or as an alternative, support for a 3rd party application such as wlr-randr to configure the outputs.

Thanks for clarifying, @ideasman42 . That will be a larger effort; forking into #122 for tracking.

Iḿ back, with good news - we are in action on the MNT Pocket Reform now. I have ~/.wlmaker.plist configured with
Output = {
  Transformation = Rotate90;
  Scale = 1.5;
};

Thanks for supporting this feature!

Glad to hear it now works for you! Thanks for verifying & reporting back!


Changes in destVI code

This submission fixes a bug (link to related issue: https://github.com/openproblems-bio/openproblems/issues/821).
Increased number of epochs for condSCVI and destVI. Reduced mini-batch size as spatial data is quite small. Increased kl term which increases deconvolution for small datasets.

@cane11 please enable github actions on your fork; we are unable to evaluate the unit tests until you do.

Close as duplicate.


Improved xor modifier.

Implement an improved xor modifier which allows you to do:
$a = "This program cannot" xor(1-255)
This will generate every single byte xor for the string from 1 to 255 (inclusive).
This commit also switches the default behavior of xor by itself back to the old behavior of xor'ing with 0 to 255 so it will search for the plaintext too.

I don't know if there is a good answer to this but this string:
$a = "foobar" xor(1-255) wide
will generate generate the xor values from 1 to 255 and the wide string, which is probably not what people will mean when they write this. When I think about those modifiers I think "generate me a wide string and then xor it with 1 to 255", not "generate a wide string and the xor version of that wide string from 1 to 255". A poor way to deal with this is to do:
$a = "f\x00o\x00o\x00b\x00a\x00r\x00" xor(1-255)
I'm not sure how to deal with this. Thoughts?

I don't know if there is a good answer to this but this string:
$a = "foobar" xor(1-255) wide
will generate generate the xor values from 1 to 255 and the wide string, which is probably not what people will mean when they write this. When I think about those modifiers I think "generate me a wide string and then xor it with 1 to 255", not "generate a wide string and the xor version of that wide string from 1 to 255". A poor way to deal with this is to do:
$a = "f\x00o\x00o\x00b\x00a\x00r\x00" xor(1-255)
I'm not sure how to deal with this. Thoughts?

Right now "foobar" xor(1-255) wide extracts the atoms for the wide string (in this case it will be f\x00o\x00) and then apply the xor to those atoms, which results in the interleaved zeroes also xored. That's equivalent to "f\x00o\x00o\x00b\x00a\x00r\x00" xor(1-255), and that's probably what I would expect, right? Would you expect something different?

What I would expect is:
"foobar" xor(1-255) wide   only matches the xored wide string
"foobar" xor(1-255) ascii   only matches the xored ascii string
"foobar" xor(1-255) ascii wide   only matches the xored ascii string and the xored wide string
So, once you use the xor modifier you are not matching the original non-xored string anymore, unless you write something like:
"foobar" xor(0-255) wide  it matches the original wide string because you are XORing with 0. Does it makes sense?

Yes, this makes sense. I'll get it fixed and update this PR.

Updated to address all concerns (I think).
One thing I've realized we can't express is a plaintext string and an xor range that doesn't start at zero. In the current iteration of this PR can you think of a way to do that?
I think the best way to express this would be xor(0,10-20) - that is, the grammar would accept a list of ranges. In this example the 0 is just syntactic sugar for 0-0. Thoughts? Is it worth doing? I'm happy to write the code.

Updated to address all concerns (I think).
One thing I've realized we can't express is a plaintext string and an xor range that doesn't start at zero. In the current iteration of this PR can you think of a way to do that?
I think the best way to express this would be xor(0,10-20) - that is, the grammar would accept a list of ranges. In this example the 0 is just syntactic sugar for 0-0. Thoughts? Is it worth doing? I'm happy to write the code.

Your solution is exactly what I would do, but I would wait and see if this have enough demand and it's worth the effort. The good thing is that this would be backward compatible change. What I would certainly implement now is xor(X) as a syntactic sugar for xor(X-X) because this is easy to implement.


Automatically add snapshot serializer for Enzyme objects

Once again! Trying to figure this out in a way that doesn't suck.
Jest added an ability to serialize snapshots. This automatically adds the serializer to jest.
My main question, is this a bad idea again?
To quote the docs:

expect.addSnapshotSerializer(serializer)
You can call expect.addSnapshotSerializer to add a module that formats application-specific data structures.
For an individual test file, an added module precedes any modules from snapshotSerializers configuration, which precede the default snapshot serializers for built-in JavaScript types and for React elements. The last module added is the first module tested.
import serializer from 'my-serializer-module';
expect.addSnapshotSerializer(serializer);

// affects expect(value).toMatchSnapshot() assertions in the test file

If you add a snapshot serializer in individual test files instead of to adding it to snapshotSerializers configuration:

You make the dependency explicit instead of implicit.
You avoid limits to configuration that might cause you to eject from create-react-app.
See configuring package.json for more information.

From that description, it seems like this is a safe idea. I just want to ensure this isn't going against any best practices.
cc @cpojer @dmitriiabramov @rogeliog

yeah. that seems like the right way to do it!


Atk speed and movement speed

I've seen people using the said atk speed and movement speed hack on Night crows can those things be done while tinkering with packets or need memory access/writing to do so?

Yes, it is possible in this game. Anything that can be done with memory edits can be done with packets and vice-versa

Yes, it is possible in this game. Anything that can be done with memory edits can be done with packets and vice-versa. You can achieve the same effect by sending the packet multiple times when attacking.

thank you


:bug: Fix(api): Docs fixes

Description
Bug fixes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #845   👈

This stack of pull requests is managed by Graphite.


Font unique ID illegal characters

https://github.com/typeworld/typeworld/tree/main/Lib/typeworld/api#uniqueid-2
Probably the comma should also be illegal in the font's unique IDs, as it is used to concatenate several font unique IDs in the installFonts command data.

I will address this on the next update. Thank you.


nil instead of {}

Hi,
When I do this:
XmlSimple.xml_in('<fiche><generation/></fiche>', 'ForceArray' => false)

I get this in return:
{"generation"=>{}}

I want to get:
{"generation"=>nil}

How can I do it?
Thanks for your help.

You have to use the SuppressEmpty option:
XmlSimple.xml_in('<fiche><generation/></fiche>', 'ForceArray' => false, 'SuppressEmpty' => nil)

Wow, that was fast.
Thank you.


Thread doesn't works correctly

I'am trying to load somes Sounds at the begining of my application with  "cpp.vm.thread" and it doesn't works.
That's happen from  Openfl 3.3.2

You can use the loadX functions of Assets to have them loaded asynchronously.


[openvdb] build failure

Operating system
Windows 11 22H2 22621.1635
Compiler
MSVC
Steps to reproduce the behavior
vcpkg install openvdb --triplet=x64-windows

Failure logs
install-x64-windows-static-dbg-out.log
Additional context
installed latest vcpkg and english version vs2022 community
the default x86-windows was compiled successed by vcpkg, but the x64-windows-static is faild

Duplicate of https://github.com/microsoft/vcpkg/issues/26993.


One out of the 8 PNG files failed during the run

pipeline run in dev:
https://prefect1.hedwig-workflow-api.niaiddev.net/default/flow-run/77e9bdd7-9023-49ab-86ac-d291c50aff90
Child 2 failed.
Hedwig UI:
https://goat.niaid.nih.gov:7333/sessions/U2Vzc2lvbjozNTcx/records

The failure was caused by temporary working directory running out of space.


fix(mock/msw): override response if it's false as well

Status
READY
Description
If you pass the value false to a mock handler to override response with that value, it would fallback to the default response. This PR should fix this, and it will now accept any value that isn't undefined.
Todos

[ ] Tests
[ ] Documentation (N/A)
[ ] Changelog Entry (unreleased)

@almyy
Originally, it was assumed that an object would be passed as an argument. The object to be mocked is a JSON API, and there was no expectation that a single false would be returned. Was there a specific use case?

@soartec-lab
We have an endpoint in our API that returns a boolean value, which in itself might be bad API design (but completely valid json), but the point is that the mock-handler that gets generated from this openAPI definition:
"/api/someEndpoint": {
    "get": {
        "tags": ["some-tag"],
        "operationId": "someEndpoint",
        "responses": {
            "200": {
                "description": "OK",
                "content": {"application/json;charset=UTF-8": {"schema": {"type": "boolean"}}}
            }
        }
    }
},

looks like this:
export const getSomeEndpointMockHandler = (overrideResponse?: boolean) => {
    return http.get("*/api/someEndpoint", async () => {
        await delay(1000);
        return new HttpResponse(
            JSON.stringify(overrideResponse ? overrideResponse : getSomeEndpointResponseMock()),
            {
                status: 200,
                headers: {
                    "Content-Type": "application/json",
                },
            }
        );
    });
};

As you can see, if I use this handler: getSomeEndpointMockHandler(false), it would not actually use the overridden response, because it's false.
If you think this change isn't practical enough, I can always create my own handler, which I in fact already am doing because of this "bug" :smile: That being said, I think checking specifically for a non-existent value (undefined) is cleaner, as there might be use-cases for passing other falsy values here as well (empty string?, 0?).

I agree with @almyy as falsiness prevents a valid value of "false".

@almyy @melloware
thank you for let me know. I understand that.
An API that returns only false is correct as a specification, and it actually exists in the API you are developing.
I had never come across an example of this so I had doubts but, if it complies with the specifications for OpenAPI, I think it should be respected. so I think it's reasonable for it to be fixed.
Thanks again for the correction 🙌


blockchain_network.py line 249 SyntaxError

src/agent/k8s/blockchain_network.py line 249 has a syntax error. missing colon.

Can I take up the issue


Issues running on Regression dataset

When running on a Regression dataset with default inputs (i.e. as per your Classification example, but using AutoNetRegression instead), I see an error as follows:
ValueError: Config option loss_modules contains following invalid values {'cross_entropy_weighted'}, chose a subset of ['l1_loss']

However, it would be ideal if the default set of loss_modules for the regression is only the permissible losses :)

You are right, this should not happen. Thank you for the input!

Additionally, I see some other issues, such as my outputs (Y) being treated as inputs (X) during the auto-regression.

/site-packages/autoPyTorch/pipeline/nodes/imputation.py", line 48, in predict
    X = X[:, ~all_nan_columns]
IndexError: boolean index did not match indexed array along dimension 1; dimension is 1 but corresponding boolean dimension is 17

Could you provide a minimal example or the complete traceback?


What's the best way to clear down the navigation stack?

In our app, we need to clear down the navigation stack when certain screens are displayed. What's the best way to do this - use RemoveFromNavigation or switch to a new Navigation stack?
With the former, is there a way of calling RemoveFromNavigation for all pages in the stack?
With the latter, if I do switch to a new Navigation stack, what do I need to do (if anything) to ensure that my PageModels and Pages on the old stack are disposed?

how about poppagemodel? I think to remove all in once poptoroot can be called. I am not quite sure how RemoveFromNavigation work, i never used it. have you found a solution for your purpose?

I don't think PopPageModel or PopToRoot are appropriate here, unfortunately, as they actually carry out navigation.
RemoveFromNavigation works - I'm using it successfully to remove PageModels I know have been (or might have been) put on the stack. However, I have to be careful not to call it for the current PageModel, because this causes an error.
I'm tempted to make a Pull request and add a method myself!

hi @james-lavery  have you found any solution for this? now I have also requirement to clear the stack and come back to the mainpage. poptoroot is also not working on uwp at all. there is already bug open for xamarin.

I created a solution by using the CoreMethods.RemoveFromNavigation() method.
The solution is to call RemoveFromNavigation() for all the ViewModels you know are on the stack.
However, I could not find an in-built method for determining all the ViewModels on the stack (I may be missing something there), so I manage my own Dictionary, keyed by PageModelType, to give a unique list of PageModels we've got on the stack.
Then all you need to do (that phrase which hides some complications) is call RemoveFromNavigation for each of the PageModels you've got on the stack. The challenge is that RemoveFromNavigation takes a Type at compile time and therefore out of the box you can't call it with a different Type at runtime. However, there is a trick to calling it dynamically, by getting its MethodInfo and then calling MakeGenericMethod on it and calling that method.
Here's the code I'm using. When actually clearing down the stack. The PageModelTypesInStack is managed outside this code of course - when a page is pushed and popped.
// RemovePageModelMethodInfo is a static property - it's expensive to determine, and we only need to evaluate it once for
            // the lifetime of the application.
            if (RemovePageModelMethodInfo == null) {
                var MethodList = CoreMethods.GetType ().GetTypeInfo ().GetDeclaredMethods ("RemoveFromNavigation");
                foreach (var item in MethodList) {
                    if (item.IsGenericMethod) {
                        RemovePageModelMethodInfo = item;
                    }
                }
            }

            if (CannotNavigateBackFrom) {

                // Take a copy of the PageModelTypesInStack dictionary - we need to do this because RemovePageModel can trigger the PageWasPopped event
                // which updates the static dictionary, which would result in a 'Collection was altered' error if we iterate over the static dictionary.
                Dictionary<Type, int> Cloned = new Dictionary<Type, int> ();
                foreach (var item in PageModelTypesInStack) {
                    Cloned.Add (item.Key, item.Value);
                }

                Device.BeginInvokeOnMainThread (() => {
                    foreach (var item in Cloned) {
                        if (item.Key != this.GetType ()) {
                            RemovePageModelMethodInfo.MakeGenericMethod (item.Key).Invoke (CoreMethods, new object [] { true });
                        }
                    }
                }
                                               );
            }

I feel like a method can be added to IPageModelCoreMethods to return all the given pageModels on the stack like so:
List<Page> GetNavigationStack() {
    return this._currentPage.Navigation.NavigationStack.Reverse().ToList();
}

I can try to create that PR soon. I'm just not sure if there was a design reason for not already having this. One reason I can think of off the bat is because it was intended that if a situation like this was needed than a custom NavigationStack would be preferred.

Beside that I Upgraded to the latest Version and test it poptoroot. this function is no longer working on Android and uwp. It just doing nothing

@WiteCastle  what do you intend with this function? can you please give more details?
I think that it would be nice to have navigation like navigate and remove page before or navigate and remove all previous pages in stack.

I am just coming back to this issue. My question is what is the problem if call CoreMethods.RemoveFromNavigation for pagemodel that is not on the stack. It doesnt throw an exception. So why do we need to determine if it is on the stack? cant we just call it for all but no the active pagemodel?

I'm coming back to this too. We still need this functionality. I'm currently handling it in our app outside FreshMvvm, but it would be a lot easier if we had the ability to do this in FreshMvvm itself.
The core requirement is that we have pages from which the user cannot navigate back in the app. We can stop back navigation quite easily by overriding the Back button behaviour, but we need to completely clear down the stack when the page is displayed - otherwise old pages and PageModels stay in memory.