[SPARK-32106][SQL] Implement script transform in sql/core

What changes were proposed in this pull request?

Implement SparkScriptTransformationExec based on BaseScriptTransformationExec
Implement SparkScriptTransformationWriterThread based on BaseScriptTransformationWriterThread of writing data
Add rule SparkScripts to support convert script LogicalPlan to SparkPlan in Spark SQL (without hive mode)
Add SparkScriptTransformationSuite test spark spec case
add test in SQLQueryTestSuite

And we will close #29085 .
Why are the changes needed?
Support user use Script Transform without Hive
Does this PR introduce any user-facing change?
User can use Script Transformation without hive in no serde mode.
Such as :
**default no serde **
SELECT TRANSFORM(a, b, c)
USING 'cat' AS (a int, b string, c long)
FROM testData

no serde with spec ROW FORMAT DELIMITED
SELECT TRANSFORM(a, b, c)
ROW FORMAT DELIMITED
FIELDS TERMINATED BY '\t'
COLLECTION ITEMS TERMINATED BY '\u0002'
MAP KEYS TERMINATED BY '\u0003'
LINES TERMINATED BY '\n'
NULL DEFINED AS 'null'
USING 'cat' AS (a, b, c)
ROW FORMAT DELIMITED
FIELDS TERMINATED BY '\t'
COLLECTION ITEMS TERMINATED BY '\u0004'
MAP KEYS TERMINATED BY '\u0005'
LINES TERMINATED BY '\n'
NULL DEFINED AS 'NULL'
FROM testData

How was this patch tested?
Added UT

Test build #127370 has started for PR 29414 at commit b4c4b40.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31990/
Test PASSed.

FYI @maropu  @cloud-fan

Test build #127373 has started for PR 29414 at commit c58729b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31993/
Test PASSed.

Test build #127370 has finished for PR 29414 at commit b4c4b40.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class SparkScriptTransformationExec(
case class SparkScriptTransformationWriterThread(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127370/
Test PASSed.

Test build #127373 has finished for PR 29414 at commit c58729b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127373/
Test PASSed.

Test build #127402 has started for PR 29414 at commit fc5bbce.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32023/
Test PASSed.

Test build #127406 has started for PR 29414 at commit fdfc987.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32027/
Test PASSed.

Test build #127402 has finished for PR 29414 at commit fc5bbce.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127402/
Test PASSed.

Test build #127406 has finished for PR 29414 at commit fdfc987.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127406/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32454/
Test PASSed.

Test build #127829 has started for PR 29414 at commit dabae9b.

Test build #127829 has finished for PR 29414 at commit dabae9b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127829/
Test FAILed.

retest this please

Test build #127834 has started for PR 29414 at commit dabae9b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32459/
Test PASSed.

Test build #127834 has finished for PR 29414 at commit dabae9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127834/
Test PASSed.

retest this please

Test build #127841 has started for PR 29414 at commit dabae9b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32467/
Test PASSed.

Test build #127841 has finished for PR 29414 at commit dabae9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127841/
Test PASSed.

Any update?

ping @maropu  @HyukjinKwon  @cloud-fan

retest this please

Test build #128204 has started for PR 29414 at commit dabae9b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32829/
Test PASSed.

Test build #128204 has finished for PR 29414 at commit dabae9b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128204/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #128205 has started for PR 29414 at commit dabae9b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32830/
Test PASSed.

Test build #128205 has finished for PR 29414 at commit dabae9b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128205/
Test FAILed.

retest this please

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32840/
Test PASSed.

Test build #128214 has started for PR 29414 at commit dabae9b.

Test build #128214 has finished for PR 29414 at commit dabae9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128214/
Test PASSed.

ping  @maropu @cloud-fan  Is ok to merge this pr and  https://github.com/apache/spark/pull/29421
We can start the rest  work。

retest this please

Test build #128239 has started for PR 29414 at commit dabae9b.

Merged build finished. Test PASSed.

Test build #128239 has finished for PR 29414 at commit dabae9b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128239/
Test FAILed.

retest this please

Test build #128252 has started for PR 29414 at commit dabae9b.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32880/
Test PASSed.

Test build #128252 has finished for PR 29414 at commit dabae9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128252/
Test PASSed.

cc @wangyum too

ping @maropu @cloud-fan Is ok for this pr to merge?  @alfozan will raise pr about spark native serde after this pr merged.

ping @cloud-fan

ping @cloud-fan @wangyum

I haven't checked it very closely but looks good to go. WDYT @wangyum?

retest this please

Test build #129349 has started for PR 29414 at commit dabae9b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/33960/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/33960/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33960/
Test PASSed.

Test build #129349 has finished for PR 29414 at commit dabae9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129349/
Test PASSed.

ping @cloud-fan

ping @cloud-fan @dongjoon-hyun

retest this please

Test build #130255 has started for PR 29414 at commit dabae9b.

Test build #130255 has finished for PR 29414 at commit dabae9b.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130255/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34855/
Test FAILed.

Test build #130256 has started for PR 29414 at commit d68066e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34856/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34856/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34856/
Test PASSed.

Test build #130256 has finished for PR 29414 at commit d68066e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130256/
Test FAILed.

retest this please

Test build #130269 has started for PR 29414 at commit d68066e.

Test build #130269 has finished for PR 29414 at commit d68066e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130269/
Test FAILed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34869/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34869/

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34869/
Test FAILed.

retest this please

also ping @HyukjinKwon

Test build #130291 has started for PR 29414 at commit d68066e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34892/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34892/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34892/
Test PASSed.

Test build #130291 has finished for PR 29414 at commit d68066e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130291/
Test PASSed.

@maropu @HyukjinKwon @cloud-fan Have resolve the conflicts, can we start these work again?

Test build #131038 has started for PR 29414 at commit 3ad44e3.

Looks fine to me. Btw, if we merge this, we can close #29085? If so, please add "Closes #29085" in the PR description.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35644/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35644/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35644/
Test PASSed.

Test build #131038 has finished for PR 29414 at commit 3ad44e3.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131038/
Test FAILed.

retest this please

Looks fine to me. Btw, if we merge this, we can close #29085? If so, please add "Closes #29085" in the PR description.

Yes, we can close #29085  after merge this, and for some other small point under https://issues.apache.org/jira/browse/SPARK-31936 . I will raise separated PR based on this.

Test build #131052 has started for PR 29414 at commit 3ad44e3.

Test build #131052 has finished for PR 29414 at commit 3ad44e3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131052/
Test FAILed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35657/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35657/

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35657/
Test FAILed.

FYI @maropu This build failed  because  https://github.com/apache/spark/pull/29421 was merged and  have updated transform.sql.output

Test build #131086 has started for PR 29414 at commit 5b41fbc.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35689/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35689/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35689/
Test PASSed.

Test build #131086 has finished for PR 29414 at commit 5b41fbc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131086/
Test PASSed.

Any update?

Can we restart this part?

Test build #132490 has started for PR 29414 at commit d3c65d4.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37092/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37092/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37092/

Test build #132490 has finished for PR 29414 at commit d3c65d4.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132490/

Test build #132769 has started for PR 29414 at commit d3c65d4.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37371/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37371/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37371/

Test build #132769 has finished for PR 29414 at commit d3c65d4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132769/

I am good with this change. I don't mind if somebody merges.

Test build #132794 has started for PR 29414 at commit a10c5a6.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37396/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37396/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37396/

Test build #132794 has finished for PR 29414 at commit a10c5a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132794/

Hope merge this and start next work

gentle ping @maropu @cloud-fan

gentle ping @tejasapatil @sameeragarwal

okay, I will merge this into master in a few days if nobody has more comments.

retest this please

Test build #132972 has started for PR 29414 at commit a10c5a6.

Test build #132973 has started for PR 29414 at commit a10c5a6.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37574/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37574/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37574/

Test build #132973 has finished for PR 29414 at commit a10c5a6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132973/

Test build #132980 has started for PR 29414 at commit a10c5a6.

Test build #132972 has finished for PR 29414 at commit a10c5a6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132972/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37581/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37581/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37581/

Test build #132980 has finished for PR 29414 at commit a10c5a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132980/

okay, I believe we have much time to revisit this until the Spark v3.2.0 released, so I'll merge this into master for now so as to encourage @AngersZhuuuu 's following work. FYI:  @HyukjinKwon @cloud-fan @dongjoon-hyun @gatorsmile

Merged to master.

Nice, I support your decision @maropu.

Thank you for informing, @maropu !


Use 168 byte exit id [Contracts v2]

is it this? https://github.com/omgnetwork/ex_plasma/pull/80/commits/d36e34b5c3226f7a7d54c762215864a26c763697

@InoMurko yes


Show nicer view of swap outputs

🚨 ~ 331 line changes are from tests
Per #424, we wanted to show a more user-friendly view of swaps and swap claims. This has taken a lot of work in past PRs on the core and WASM side of things, so that we have the necessary data to render everything.
This PR ties it all together by rendering a swap's outputs (and unfilled amount) on a swap page, and rendering the claimed amounts on a swap claim page.
Swap page
The swap page is a little complex, because it has to handle the following cases:
One swap input exists but no swap outputs exist.
This happens we're viewing the swap inside the transaction approval dialog, so the transaction hasn't yet been built and thus has no outputs yet. In this case, we show e.g., "100UM -> TestUSD" — that is, we just show the symbol for test_usd, but not the amount, since we don't know the amount yet.

One swap input exists and one swap output exists.
This is the typical case — there's enough liquidity to cover the entire swap, so the user does a one-way swap from asset 1 to asset 2.

One swap input exists and two swap outputs exist.
This happens when there isn't enough liquidity to cover the entire swap (or, even, any of it). In this case, a user does a one-way swap from asset 1 to asset 2 and gets two outputs: the amount of asset 2 that the liquidity was able to cover, and the amount of asset 1 that the liquidity was not able to cover (sort of like change). This is shown as the "Unfilled" amount.

Two swap inputs exist and two swap outputs exist.
This is a two-way swap and it... basically will never happen. This means the user is swapping from asset 1 to asset 2 AND from asset 2 back to asset 1 in the same Swap action. There would never be a good reason to do this — especially since you're just burning fees. That said, I did account for this possibility in the view code; but since no Penumbra code makes this type of transaction possible, I didn't test it. However, this is what the two-way swap component looks like (ignore the fact that amount of each currency doesn't make sense — I was just rendering it with test data):

Swap claim page
This is a bit simpler. We just have to account for when:
One output exists.
This is the typical one-way swap case where there was enough liquidity to cover the entire swap.

Two outputs exist.
This happens when either A) there was an unfilled amount, so the user can claim some of asset 1 AND some of asset 2; or B) the user did a two-way swap, so there is some of both assets to claim (again, this should never happen).

Closes #424

@jessepinho tagging #830 as potential relevant scope

@TalDerei that'd be separate scope from this ticket — this is just about how swap transactions are displayed in A) the txn approval dialog and B) the transaction view. I agree with you on the need for better UX when submitting a swap + swap claim, though!


__avn_debug command not found

I get a "command not found" error when trying to run __avn_debug or _avn in the shell.
I installed via "npm i -g avn avn-nvm" and successfully ran "avn setup".

What shell and what version are you using? What does your profile (~/.{bash|zsh}{_profile|rc}) look like for that shell? Did you launch a new shell after running avn seup or just continue to use the same shell? Can you try launching a new shell?
Also, anything here that you can provide would help.

Shell: bin/bash

avn 0.2.3

node 4.0.0 / 0.12.1, depending on my nvm setup

nvm 0.31.0

avn is loaded in my ~/.{bash|zsh}{_profile|rc} file with:
[[ -s "$HOME/.avn/bin/avn.sh" ]] && source "$HOME/.avn/bin/avn.sh" # load avn

there is nothing avn-related in .bashrc

I basically installend avn with npm install -g avn avn-nvm with both of my node versions (v4.0.0 and v0.12.1) and ran avn setup for both. Opening up a new shell doesn't solve the problem.
Thank you

What happens when you source it directly with source "$HOME/.avn/bin/avn.sh"?

Now it works - I added source "$HOME/.avn/bin/avn.sh" to my .bashrc. Somehow, .bash_profile is not recognized in my system (and redundant, anyway, because I have .bashrc). Maybe avn should check for the existence of .bashrc upon setup. Thanks!

Yup, it's hard to get right. See #31.

@grssnbchr What operating system were you running at the time (March 25, 2016), including version number? I'm guessing probably not one based on Ubuntu 16.04, unless you were running a beta or release candidate OS.

@edsfocci Ubuntu 12.04


Set installconfig.networking.machineNetwork in install-config.yaml

$openshift-install explain installconfig.networking.machineNetwork
KIND:     InstallConfig
VERSION:  v1

RESOURCE: <[]object>
  MachineNetwork is the list of IP address pools for machines. This field replaces MachineCIDR, and if set MachineCIDR must be empty or match the first entry in the list. Default is <IP_ADDRESS>/16 for all platforms other than libvirt. For libvirt, the default is <IP_ADDRESS>/24.

FIELDS:
    cidr <Any> -required-
      CIDR is the IP block address pool for machines within the cluster.

Fixed in devel branch, will be merged into master via PR #159


Icon of console application doesn't get used

Windows Terminal version
1.15.2875.0
Windows build number
10.0.22621.819
Other Software
FAR Manager or any console app with an embedded icon
Steps to reproduce
Launch a console application that has an embedded icon in its executable, such as FAR Manager. The Windows update that causes every console application to launch in Windows Terminal (instead of the old console app) must be installed.
Expected Behavior
The embedded icon in the executable should be used as the tab icon inside Windows Terminal.
The embedded icon in the executable should be used as the taskbar icon.
Actual Behavior
The tab icon remains the default generic icon (which looks like C:\ on a white background).
The taskbar icon remains the default generic icon (which looks like >_ on a black background).
This makes it difficult to distinguish between different console applications.

Good catch on us not having an issue for this... probably just a bullet point in some other place we never formally tracked. Thanks for filing it, @gyorokpeter!

Oh wait, this might actually be tracked in #7552


Identity factors

There are several places where sparse factors are created for packing/unpacking tuples or variants, which need to be made more efficient. I think there are just two kinds:
{
  "function": "directsum",
  "type": [ "a+b", "a" ],
  "sizes": [ 123, 456 ]
}

{
  "function": "tensorproduct",
  "type": [ "(a,b)", "a", "b" ],
  "sizes": [ 123, 456 ]
}

What about equality, is it eliminated by some optimization already, or a special case of both of these kinds?

I was thinking it’s a special case of both kinds.
On Mon, Aug 1, 2022 at 22:18 Chung-chieh Shan @.***>
wrote:

What about equality, is it eliminated by some optimization already, or a
special case of both of these kinds?
—
Reply to this email directly, view it on GitHub
https://github.com/diprism/perpl/issues/97#issuecomment-1201938629, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKVY2PB3FRFGO2UZ5MOXLDVXCAPPANCNFSM55JM2GDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

For proper directsum and tensorproduct factors (as opposed to equality aka identity aka diagonal factors), I hope torch-semiring-einsum doesn't mind handling input tensors that are views on other tensors (i.e., with non-default stride and/or non-zero offset)?


Update release 1.76 distro to include proposed API

Closes https://github.com/microsoft/vscode/issues/175601

We have merged https://github.com/microsoft/vscode/pull/175635 and I believe this isn't needed anymore

Thanks!


Use separated test configs for redis & memcached

use separated configs for testing

Testing configurations can go underneath tests/ ... see tests/_support/Config, or alternately inside tests/system/Config/fixtures. There is no need to add these to application/Config


522 geographic coverage note

https://github.com/uwlib-cams/MARC2RDA/blob/main/Working Documents/5XX.csv

https://docs.google.com/spreadsheets/d/1Hd5Kwp7HOrqJfzUQYzwnrb8Q7cgzfCCJssM-KTq0ZAM/edit#gid=1293943126
Hello everyone,
Please see the first pass of 522 - Geographic Coverage Note (R). Would someone be able to take a look?
Best,
Alice

Reviewed, moved to ready for transform


There is not an easy way of chaining requests

If I want to test a flow that has two requests and the data of the second request depends on the data coming back from the first request there is no documentation on how to access to the response object more than the response code.

https://github.com/icemobilelab/minosse/wiki/Steps#save-the-response-body
Some pseudo code:
Scenario: Account activation
        And I set property request body to testdata account.good
        When I send a POST request to /account
        Then the response status code is 201
        And I set property activationKey to property activationKey of response body
        # Activate
        When I send a GET request to /v1/account/activate/{activationKey}
        Then the response status code is 200

Oh nice, cheers!


nitro.exe has stopped working when load model by post request,within cpu mode on win server2012 r2.

nitro.exe has stopped working when load model by post request ":3928/inferences/llamacpp/loadmodel"  ,  with cpu mode in win server2012 r2.
(nitro version is  https://github.com/janhq/nitro/releases/download/v0.3.6/nitro-0.3.6-win-amd64.tar.gz  )
post data json is:
{
"llama_model_path": "./model/tinyllama-1.1b-chat-v1.0.Q4_K_M.gguf",
"ctx_len": 512,
}

but,when i run in my laptop within cpu mode, everything is fine.
What's this problem? How to solve this ? anybody helps ?

I would advise to have more than 8GB of ram to attempt loading model into memory, 2GB is quite limiting


[backport][7.59.x] gitlab - deploy_containers: allow final otel beta jobs on beta branch builds

What does this PR do?
Backports #30890 to 7.59.x
Motivation
Slightly safer OTel beta release pipelines (we should only target beta for now`).
Describe how to test/QA your changes
Gitlab change, release pipeline validation.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


add CLI

implement a CLI that allows manipulation of AR4SI tokens.  Specifically:

[ ] create
[ ] pretty print
[ ] verify
[ ] sign

Initial proposal
Verify
The verify sub-command is used to cryptographically verify and pretty-print the contents of a EAR, including the trustworthiness vector.
arc verify \
    --input <file> \
    --pkey <file> \
    [--short] \
    [--color]

Parameters

parameter
meaning

--input
a JWT wrapping an EAR claims-set

--pkey
verification key in JWK format

--short
trustworthiness vector short report (default is verbose)

--color
trustworthiness vector report colourises the tiers (default is B&W)

Output

Validation status of the cryptographic signature.

If successful:

The decoded trust vector is printed to stdout (the exact format depends on short and color).
The EAR claims-set is also printed to stdout.

Create
The create sub-command is used to synthesise an EAR given the full claims-set.
arc create \
    --input <file> \
    --skey <signing key> \
    --output <file>

Parameters

parameter
meaning

--input
EAR claims-set in JSON

--skey
signing key in JWK format

--output
the signed EAR claims-set as in JWT format

Output
A one-liner saying success status and path of the JWT file that was created.

Basically:

create creates and signs
verify verifies and, if successful, pretty-prints

Done in #14


Bug - Assets not showing, (both Ubuntu 16 & Win10)

Using this app, it is not showing any assets listed in both Ubuntu & Windows.
The software connects just fine, however, even after waiting multiple hours, and multiple "rebuild index" attempts it is unable to show what assets the wallet contains.
neo> show state
Height: 93788/94059/1816414, Nodes: 5
neo> show node
::ffff:<IP_ADDRESS> port:10333 listen:10333 [1/5]
::ffff:<IP_ADDRESS> port:10333 listen:10333 [2/5]
::ffff:<IP_ADDRESS> port:10333 listen:10333 [3/5]
::ffff:<IP_ADDRESS> port:10333 listen:10333 [4/5]
::ffff:<IP_ADDRESS> port:10333 listen:10333 [5/5]
neo> list address
AMuiZnPhmH..........
neo> list asset
neo> show gas
unavailable: 0
available: 0
This issue had been brought up already, but closed with no resolution.
https://github.com/neo-project/neo-cli/issues/50

I'm facing the same issue. I assumed that list asset will show the assets except Neo and Gas, in case transacted. I'm doing the testing on CoZTestnet, and tried getting help from there.
I've received some gas and neo from CoZ faucet. If I do get transaction for that hash, it is giving me the transaction. I'm able to verify that vout has my wallet's standard address (RPC Call). But if I'm doing getbalance for the asset, it is giving me 0.
getaccountstate for my address returns nothing -
{
    "id": "1",
    "jsonrpc": "2.0",
    "result": {
        "balances": [],
        "frozen": false,
        "script_hash": "0xf145739a3b8168ec2c1194b1623a0f56595d7b54",
        "version": 0,
        "votes": []
    }
}

I'm on version <IP_ADDRESS>, if that helps. Next thing I'm gonna try is try an older version.

Have tried 2.3.4, 2.7.0
Same issue. Do I need to specify something like I'll be storing x-y assets when creating a wallet?

I used the Neon wallet, and imported the wallet created in neo-cli.  It shows everything just fine, and I can send and receive using it.
Looks to be just a bug with the cli version.

Ok, I raised a false alarm. I just figured out somehow my balance is moving to a different address
https://coz.neoscan-testnet.io/address/APUaC8wxx8jynfMD3a5KBxi8ZzW94tY9Yv
As soon as I received the money, it got transferred to some other account. (Still figuring out how this happened)

I am facing same issue and even my funds are getting transferred into same address as above AGQGKFJ7MTMvePZcz1KEmd9UVWzZbRsotC
I also raised another issue https://github.com/neo-project/neo/issues/178 , but nobody understands.
It's a bug in neo-cli I think, because whenever I create wallet using NEON then my funds stay in my wallet but when I create using neo-cli it got transferred in AGQGKFJ7MTMvePZcz1KEmd9UVWzZbRsotC everytime
Please look into this, blocked from long time


Add Astro Forum docs

Please describe the feature you'd like to see
Request from Steven to add https://forum.astronomer.io/ as a data source.
Describe the solution you'd like
Add forum docs for ingest.  Some older posts are inaccurate so a "recent" (ie. last 2 years) cutoff date will be needed.   Also we would want to check tags for "software" and decide whether we want to ingest those.
Are there any alternatives to this feature?
No
Additional context
Forum is powered by Discourse. https://meta.discourse.org/t/does-discourse-support-export-conversations-as-an-organized-bulk-of-data/180537
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

PR being reviewed


Terminal screen is left aligned

I have a custom modpack and have used Toms several times before but this time the terminal screen is stuck being aligned on the far left of the screen. Im on 1.19.2 and dont know what might be causing this

Found the issue. It was also crashing my game when trying to interact with the terminal. It was the mod Thanks I have REI which removes the recipe book that was causing the issue.


Addition of four new inequality filters

when the materialization version is bumped you should add the check that the server is running the required version of materialization when using these keyword arguments, else the user will get a 500 error that wont be that obvious what is wrong

materialization engine >=4.34.0 has it

Are you saying I should add that to the requirements.txt for CAVEclient?

@kebwi https://github.com/CAVEconnectome/CAVEclient/blob/1d08a9b49a86fc0fa6ac52123ea709c535682b73/caveclient/chunkedgraph.py#L812 here's an example usage of a decorator I wrote for doing this kind of check on the version running on the server before executing the query

@fcollman @bdpedigo I believe I've incorporated the server version check correctly. I tested it with "4.34.1" and it refused to run the query and then set it to "4.34.0" and committed it. So I think it works.


Begin documenting initialization checking

Edit: Rendered

cc @pnkfelix

cc @nikomatsakis

@nikomatsakis What do you want to do with this RFC?

I'm going to try to follow @nikomatsakis 's request and turn this PR into a PR against  https://github.com/nikomatsakis/nll-rfc/ ; after I have created that second PR, I will close this one.


How to add the trace-id's into the http request/response headers?

Hi,
How can I append/add the trace-id's and span-id's etc in the http request and response headers?
if that's not possible, how can I intercept the request and add it so I can push a PR for a new plugin?

[ V ] This only affects the JavaScript OpenTelemetry library
[ V ] This may affect other libraries, but I would like to get opinions here first

This is doable using propagators.
There are several built-in propagators in the core package.
You need to use the same propagation mechanism across all your services to have distributed tracing properly used everywhere.


Noises.html : Improving on Riki's idea. (Superseding #5)

Thank you for your hard work @bempfoobq052021 @steelmaker86
PR #5 's commit histories is tangled and jumbled together, and there's a merge conflict with current master.
I find it very hard to untangle and merge it as it is.
Thus I had to rebase everything to keep the attributions correctly as well.
I hope you are okay with that.
Rebasing is also the recommended way of updating from master or latest trunk.
With that being said, please check and make sure this still works are you intend them to.
Changes:

Rebase all commits from current master
Adjust code styling
Moved sounds.json into subfolder assets/etc
Updated the footer of noises.html with current master

Merging this first. For additional changes and improvements please do it in a new PR based on latest master.
Thank you again!


Generated nonce for secure headers middleware uses NodeJS Buffer API

What is the feature you are proposing?
I would like to use the secure headers NONCE value using only web platform APIs. It currently uses Buffer.from() to convert the data to base64.

https://github.com/honojs/hono/pull/2577
https://github.com/honojs/hono/blob/v4.3.8/src/middleware/secure-headers/index.ts#L115

With Cloudflare Workers, that requires enabling nodejs_compat.
oslo.js contains an alternate implementation that does not require NodeJS APIs. It works for me, but I can understand you might not want to add a new dependency.

https://oslo.js.org/reference/encoding/base64
https://github.com/pilcrowOnPaper/oslo/blob/main/src/encoding/base64.ts

import { base64 } from "oslo/encoding"

const generateNonce = () => {
  const buffer = new Uint8Array(16)
  crypto.getRandomValues(buffer)
  return base64.encode(buffer)
}

It may be possible to use a different Buffer or base64 polyfill.

Hi @rmarscher
Thanks. This will be fixed in the next minor release. Pelase wait a bit.


Error "Failed to notifiy project evaluation listener" in Gradle 5

Hello.
I tried building simple hello world project in Gradle 5 but it has this error:
* What went wrong:
A problem occurred configuring root project 'myjava'.
> Failed to notify project evaluation listener.
   > org.gradle.api.tasks.SourceSetOutput.getClassesDir()Ljava/io/File;

If I use gradle 4.x, it compiles fine. Any idea to fix this? Thank you.

Same error here, affecting Protelis/Protelis#195

Released version 3.7.1 which should be compatible with gradle 5+


when i use Real-Captured Data to test，it will cause wrong result

when i use your Real-Captured Data to test，it will cause wrong result.that's why? thank you

when i use your Real-Captured Data to test，it will cause wrong result.that's why? thank you

Could you provide more details on the result? Has the code been modified?

when i use your Real-Captured Data to test，it will cause wrong result.that's why? thank you

Could you provide more details on the result? Has the code been modified?

gt image is this, example.

but test image is

when i run amass dataset,it will success, but in real-capture-data, it fail. it need more action?

when i use your Real-Captured Data to test，it will cause wrong result.that's why? thank you

Could you provide more details on the result? Has the code been modified?

is that Problems with coordinate systems？

when i use your Real-Captured Data to test，it will cause wrong result.that's why? thank you

Could you provide more details on the result? Has the code been modified?

is that Problems with coordinate systems？

It seems there may be an issue with the coordinate system, but with the limited information provided, I'm unable to assist in diagnosing the problem. Could you clarify if this issue occurs with the original open-source code or with modified code?


please add a good example for caching

There is a example for caching but not explained where to put it.
I'm using the fork of this repo, putting the cache part in anywhere in job section in the build.yml fails to cache since the main buildozer job is auto cleaning the files

Hi. You can see example in https://github.com/kivymd/KitchenSink/blob/86b08cb248686e8a50cfd31dbdab521597b8e8cc/.github/workflows/main.yml#L24-L33
There are caching for global and local .buildozer directories.

Thanks


ip_hash load balancing policy does not add revived upstreams back to the pool

1. What version of Caddy are you running (caddy -version)?
v0.9.2
2. What are you trying to do?
Use the ip_hash load balancing policy with the proxy directive.
3. What is your entire Caddyfile?
Caddyfile_proxy
http:// {
    errors CaddyErrors_proxy.log
    log CaddyLog_proxy.log
    proxy / localhost:580 localhost:581 {
        transparent
        policy ip_hash
        health_check /health.html
        health_check_interval 10s
        health_check_timeout 5s
    }
}

Caddyfile_upstream1
http://:580 {
    errors CaddyErrors_upstream1.log
    log CaddyLog_upstream1.log
    root WebRoot1
}

Caddyfile_upstream2
http://:581 {
    errors CaddyErrors_upstream2.log
    log CaddyLog_upstream2.log
    root WebRoot2
}

WebRoot1 and WebRoot2 directories contain files:

index.html with contents PROXY1 and PROXY2 respectively
health.html with contents OK

4.  What did you expect to see?
When upstream1 fails, requests from previous and new clients are redirected to upstream2 (given that they were previously assigned to upstream1) as upstream1 is taken out of the pool. Once upstream1 comes back online, health check detects this and adds the upstream back to the pool for new matching requests.
6. What did you see instead (give full error messages and/or log)?
Once upstream1 came back online, health check did not resume and the upstream was still considered down. No health check requests arrived to the upstream afterwards. According to the request log of upstream2, once upstream1 failed, health checks started being performed twice against upstream2 (see attached logs). Afterwards when upstream2 failed, the proxy started returning 502 even when upstream1 was already up for a while. It seems like the upstream is removed from the proxy configuration permanently. The proxy had to be restarted to add upstream1 back to the pool.
CaddyLog_proxy.log
::1 - [26/Sep/2016:10:35:23 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:24 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:24 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:25 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:25 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:34 +0200] "GET / HTTP/1.1" 200 8 <<< upstream1 was terminated before this request
::1 - [26/Sep/2016:10:35:35 +0200] "GET / HTTP/1.1" 200 8

CaddyLog_upstream1.log
::1 - [26/Sep/2016:10:35:21 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:35:23 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:24 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:24 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:25 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:25 +0200] "GET / HTTP/1.1" 200 8

CaddyLog_upstream2.log
::1 - [26/Sep/2016:10:35:21 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:35:33 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:35:34 +0200] "GET / HTTP/1.1" 200 8 <<< first failover request from upstream1
::1 - [26/Sep/2016:10:35:35 +0200] "GET / HTTP/1.1" 200 8
::1 - [26/Sep/2016:10:35:41 +0200] "GET /health.html HTTP/1.1" 200 4 <<< doubled health check requests start here
::1 - [26/Sep/2016:10:35:41 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:35:51 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:35:51 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:36:01 +0200] "GET /health.html HTTP/1.1" 200 4
::1 - [26/Sep/2016:10:36:01 +0200] "GET /health.html HTTP/1.1" 200 4
...

7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Start upstream1, upstream2 and proxy instances of caddy. Curl localhost:80 and watch upstream access logs for health check requests. If curl request are assigned to upstream1, terminate the upstream and curl again. Then bring the terminated upstream up again, wait 10s and terminate another upstream. The proxy will return 502 instead of forwarding requests to restarted upstream1. See upstream1 access logs - there will be no new health check requests. See upstream2 access logs - health check requests will be doubled.
This issue does not occur with round_robin policy - health checks to failed upstreams resume correctly once they are up again and are not fired twice against another upstream.

Thanks for the detailed report!
Hmm, @krishamoud -- I didn't notice this before, but the IP hash policy is the only one that changes the HostPool. I actually don't think it's necessary, and would explain why they don't get added back once they go down. Are you in a position to look into this and submit a fix?

Yeah I'll look into this today.


All MARC and DPUL created fields are now hyperlinks in Badakhshan Genealogical Document Collection Exhibit

For some reason, All the MARC metadata, along with the medatadata fields created in DPUL, are now hyperlinks.  This appears to only be affecting the Badakhshan Genealogical Document Collection exhibit. I will look at a few others to ensure this is the case.
Example:  https://dpul.princeton.edu/badakhshan_collection/catalog/3n204458c

the "view in catalog" field, which is supposed to take you to the item in orangelight (it works fine!):https://catalog.princeton.edu/catalog/11404938

ALL MARC field links take you to a search listing (://dpul.princeton.edu/badakhshan_collection/catalog?f[readonly_publisher_ssim][]=Jumādá+al-ākhir+1344+A.+H.+[December+1925+or+January+1926]).

The DPUL created field (BMP#) link takes you nowhere: https://dpul.princeton.edu/badakhshan_collection/catalog?f[exhibit_badakhshan_collection_bmp-number_tesim][]=BMP046
The DPUL exhibit tags link works fine too: https://dpul.princeton.edu/badakhshan_collection/catalog?f[exhibit_tags][]=Shāh+Khāmūsh

Thank you, @tpendragon !  “link to search” boxes were all clicked for the metadata, which was making everything linky (technical term). Closing ticket.


Load filter program once and move v4, v6 policy to different indices.

Description
This PR has the following changes.

Have separate policy indices, one for v4 and one for v6 in the attachPoint.
Load filter programs only once per hook, per attachPoint.

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/merge-when-ready


require input password everytime

i think it will be better if input password once, and can use many times,  just like in a unix shell.

welcome you PR %)
https://github.com/automation-stack/electron-sudo/blob/master/src/lib/sudoer.js#L134
if (!option.cachePassword) {
    await self.reset();
}

please wait.

Due to remove sudo usage from module all subprocess call will force require input password everytime

so, this can't be implemented at all?


leader discovery

Looking at the README, one is supposed to run one exporter with the -master flag pointed to the current master. What happens in the event of that master crashing and some other node becoming master?
The old deprecated exporter could discover the master, is that something that will be merged in this exporter - or can I just run the master exporter on all masters in my cluster?

The expected usage is that you run one exporter next to each master. This  means each master will be scraped individually.
If you are interested only in the leading master, you can have a look at the mesos_master_elected metric:
mesos_master_elected{cluster="us-tirefire1",instance="master1.example.org:9091",job="mesos-master"} 0
mesos_master_elected{cluster="us-tirefire1",instance="master2.example.org:9091",job="mesos-master"} 0
mesos_master_elected{cluster="us-tirefire1",instance="master3.example.org:9091",job="mesos-master"} 1

Scraping each master individually provides some great flexibility. For example, you could use this to do alerting on the overall state of your cluster:
ALERT MesosMasterNoLeader
  IF sum(mesos_master_elected{job="mesos-master"}) == 0
  FOR 5m
  LABELS {severity="critical"}
  ANNOTATIONS {description="There is no leading mesos-master", runbook="Good luck!", summary="No mesos-master leading!"}

ALERT MesosMasterTooManyMasters
  IF sum(mesos_master_elected{job="mesos-master"}) > 1
  FOR 5m
  LABELS {severity="critical"}
  ANNOTATIONS {description="There should be exactly one leader!", runbook="Good luck!", summary="Multiple mesos-masters leading!"}

If you look at an individual metric like mesos_master_mem you will notice that the leading master is reporting the true value and the other masters are reporting 0. You can select the metric from the master either via a join in PromQL or via an aggregation. In most cases I therefore use a form of the following:
# Query
max(mesos_master_mem) without (instance)
# Result
{cluster="devcluster",job="mesos-master",type="used"}	1779200
{cluster="devcluster",job="mesos-master",type="free"}	4662717

Hope this gets you started!

Yeah definitely! The README states I should be running just one exporter, so perhaps that can be cleared in the docs. Thanks 👍

If you're running this with Mesos-DNS you can just point it to leader.mesos and it'll always scrape the leading master.


text_on_image Creator - Web design #100

https://images01.nicepagecdn.com/page/55/71/html-template-5571820.jpg Design a text_on_image like the above image
Create a file "text_on_image_NUM_by_(YourName).html" in the modules/text_on_image folder.
Use simple HTML and CSS and JAVASCRIPT to create the text_on_image animation. You can use the interface development https://kwickerhub.com/ (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "text_on_image_NUM_YOUR_NAME_.html" in the "modules/text_on_image" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/text_on_image/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
<div>
    <!---YOUR CODE---->
</div>

<style>
    /* YOUR STYLE */
</style>

<script>
    // YOUR SCRIPT if any
</script>

The goal of this issue is to create modules that other people/dev/designers can use or easily import into their projects. Avoid the desire to style all elements in the page by using codes like:
<style>

*{

}
/* OR */

div, a, h1, pre{

}

</style>

use classes or id to style the elements to avoid a collision of styles.
Remember to make your module usable across different screens(Make your design responsive).
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Good-luck and thank you.

I need to work on open source projects as a part of my university course. I am interested to contribute to this problem.
Can you please assign this to me?

Hello, I have made commits for this issue by creating a text image for web design 100. When will this issue be reviewed and merged? I am working on issues for a class and need the PRs merged by Friday to receive credit. Is this possible?


Erroneous stateDiff in API calls for failed transactions

Description
The stateDiff field of the result of a trace_replayBlockTransactions call is null
Steps to Reproduce (Bug)
On an archive Besu node on the

Have RPC access to an archive Besu node following Holesky
Replay this block and look at the result for the 16th tx with the following curl/jq command:

curl -X POST -H "Content-Type: application/json" --data '{
    "jsonrpc": "2.0",
    "id": 1,
    "method": "trace_replayBlockTransactions",
    "params": ["0x1D1921", ["stateDiff"]]
    }' http://localhost:8545 | jq ".result[15]"

Expected behavior:
stateDiff should contain the balance changes for the tx sender and the block validator (here is the correct output as provided by QuickNode:
{
  # This is the sender
  "0x5324ff21932b3ae69a6d46aaf29eca5bbfb86b73": {
    "balance": {
      "*": {
        "from": "0x5bb96a5487a77495",
        "to": "0x5bb9573b1619f045"
      }
    },
    "code": "=",
    "nonce": {
      "*": {
        "from": "0xe3",
        "to": "0xe4"
      }
    },
    "storage": {}
  },
  # This is the miner
  "0x9baa3244565d51d9c7897c0eb6679ed4890e536e": {
    "balance": {
      "*": {
        "from": "0xa39659f932cf1b4668b",
        "to": "0xa39659fa646633eb68b"
      }
    },
    "code": "=",
    "nonce": "=",
    "storage": {}
  }
}

Actual behavior:
stateDiff is null.
Frequency:
100%
Versions (Add all that apply)

Software version: freshly compiled from main branch, as well as 24.5.2
Java version: 17 & 21
OS Name & Version: Debian bookworm
Consensus Client & Version if using Proof of Stake: Teku 24.4.0

Additional Information (Add any of the following or anything else that may be relevant)
It seems that this issue is triggered by simple transfer TXs running out of gas. The bug may be coming from this line, where the generation of the state diff is entirely skipped if there are no frames in the tx execution, hence ignoring the balance changes to the sender & miner.

@daniellehrner @shemnon

@lu-pinto

Thank you!


D2IQ-71117: adds ColorCodedBadge component

Closes D2IQ-71117

Testing
Just look at Storybook
Trade-offs

Dependencies

Screenshots

Checklist for PR creator

[x] If any new components were added, there are exported from packages/index.ts
[x] If this PR is associated with a JIRA, it is mentioned in commit message footer ("Closes …")
[ ] If this PR contains breaking changes, is stated in commit message body ("BREAKING CHANGE: …")
[x] Info for applicable sections above is provided

Argh sorry I was a bit premature on the merge - only noticed after that Pierre said his was not a thorough approval. Let me know if I should undo this!


Track the timezone set in Kingdom of Loathing

Although the default timezone is Africa/Abidjan from my personal experience, I've left the default of this preference as empty to indicate that mafia does not know what timezone has been set.
The use of this change is to help the user identity what timezone is being used when the game presents a time/date using the timezone set in the player's settings.
This allows future changes to mafia where the time can be converted back into a unix time for display purposes and for internal calculation. Such as store logs shown using local time, instead of the provided time by kol.
My own use is to filter store purchases to retrieve the purchases made between two date points, such as the time during which a script was run, or displaying purchases separated by kol day.
This PR could be extended further with the addition of either adding a record for TimeZone, or adding functions to retrieving the offset of a timezone, and daylight savings data.
However given the complicated nature of daylight savings, I've left that alone in interests of scripts using a library instead to handle the timezone, such as moment.

That suggests to me that the only benefit to capturing the setting in KoLmafia is for a user who can't remember how they set it and doesn't want to use a browser or a user who changes the account setting frequently.

Or for a script (probably a relay script) that want to run successfully on every user's mall log?

Or for a script (probably a relay script) that want to run successfully on every user's mall log?

To compare data from different users that have different account time zones you either need to capture what the time zone was when the data was dumped or the dumping process needs to normalize to a known timezone.
If I'm logged in and dumping data I can capture the timezone if I need it without making KoLmafia manage a preference.
I'm thinking this is a case where if the only user of the preference is a script, it is simple enough for the script to find out what it is.  If KoLmafia needed such a preference it would be in connection with a new Feature that injected the value to any output stream that had dates or used the preference to convert all the dates in the stream to a known time zone.
To the extent that mafia is doing the writing the simplest thing to do might just be to extend timestamps so that they always included the timezone.
I'm not going to have a hissy fit if it is committed but it does seem like something that is very niche.

Yeah, I'm going to close this as I think it'd be nicer for the script to manually check the first time it's run each day for example.
So that when the user does change their timezone without mafia knowing, the script still catches it.

I think that since this is never needed in a limited mode (like the fight
or choice lock) there's no reason for the script not to pull the data
itself. I agree. We can close the PR and reject this feature
On Mon, 5 Dec 2022, 17:05 Jamie Adams, @.***> wrote:

Or for a script (probably a relay script) that want to run successfully on
every user's mall log?
To compare data from different users that have different account time
zones you either need to capture what the time zone was when the data was
dumped or the dumping process needs to normalize to a known timezone.
If I'm logged in and dumping data I can capture the timezone if I need it
without making KoLmafia manage a preference.
I'm thinking this is a case where if the only user of the preference is a
script, it is simple enough for the script to find out what it is. If
KoLmafia needed such a preference it would be in connection with a new
Feature that injected the value to any output stream that had dates or used
the preference to convert all the dates in the stream to a known time zone.
To the extent that mafia is doing the writing the simplest thing to do
might just be to extend timestamps so that they always included the
timezone.
I'm not going to have a hissy fit if it is committed but it does seem like
something that is very niche.
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/1330#issuecomment-1337740558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LWDBOXO3BLA3GANZJDWLYOENANCNFSM6AAAAAASTZ354Y
.
You are receiving this because your review was requested.Message ID:
@.***>


Auto add sites to my autoflags prefs

I had all (173 at the time) sites selected for autoflagging, since then 3 new sites came into public beta, so I had to go there and manually hit select all.
Could there be a system that if you  have all sites selected, you can opt in that it automatically adds in the ones to follow?

Needs to be controllable by an opt-in user preference. We shouldn't modify users' conditions or preferences without asking them first; add a thing to the user's account settings to let them turn this on.

I don't know - it seems reasonable to say "if you have this on across all sites, we'll auto add sites as they're created". As long as it's clear, I don't see a problem with not having a separate setting.


installs newrelic

(did you know elife-dashboard is still on python 2 ??)

(did you know elife-dashboard is still on python 2 ??)

it's funny you post that cat, I have two cats that hang around my place and one looks just like this one. Him and a black+white cat were having a street fight just minutes ago outside my office window. I had to break them up with a broom. Fur everywhere.

Yet another predator animal destabilizing Australia's ecosystem :cat:

yeah, you're absolutely right. I have a tree full of native birds right outside my office window and the cat that resembles the one in the phone will climb the tree and lie on the trunk (well camouflaged) and stalk them.


treewide: handle *Phases variables __structuredAttrs-agnostically (round 2)

This PR clears the remaining non-__structuredAttrs-agnostic appending to the *Phase list not caught by #339117.
For the "insert the new phase before certain phase" operation, this commit adds a new Bash function replaceElement to setup.sh to provide a more graceful solution to the string-replacement hack in #339117 and previous implementations. (Previous discussion: https://github.com/NixOS/nixpkgs/pull/339117#issuecomment-2334961783).
Cc: @philiptaron @wolfgangwalther
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

I opened #360450 to address those special cases. If we do this, then we don't need introduce any new helpers here.

@wolfgangwalther Thank you for fixing them. I dropped the commits already adjusted by #360450.

There are still a lot of rebuilds. I'll re-target this feature branch to staging.

@ofborg build tests.devShellTools.nixos
@ofborg build tests.dotnet
@ofborg build tests.testers
@ofborg build tests.trivial-builders.references
@ofborg build tests.vim

The build logs don't show why OfBorg failed to build the package tests, and I cannot reproduce such failures on my laptop (all built successfully).


Tabs with Router - link is not active on the whole tab only on anchor text

Reproduction link
https://stackblitz.com/edit/angular-xmgl6x?file=src/app/app.component.ts
Steps to reproduce

Have a NzTabSet with nzRouterLink directive
Click on the edge of a tab, whithin the tab.

What is expected?
The router navigation should occur.
What is actually happening?
The tab is activated but there is no router navigation.

Environment
Info

ng-zorro-antd
10.0.0

Browser
Chrome 85 and Firefox 82

If you click on the anchor text, the navigation is activated.
The pointer changes over the tab but the link is not active unless you hover the anchor text.
There is a "padding: 8px 16px" surrounding the anchor element within the tab element. It is a direct cause of the issue.
It was working in 9.3.x version.
It can also be reproduced within the Tab component documentation (reproduction is a direct copy/paste of the documentation code sample).

I have also same issue!

About 3 weeks passed but no information yet! It`s pity that other tiny bugs fixed in 2 minor release and this issue ignored.


runtime: Cargo feature to make block production faster

Add a compilation-time feature to shorten the block interval to 3 seconds. Intended mainly for testing.
This is the approach to adjusting block interval that I found easiest to implement. The PR is mainly intended as a discussion starter. Do we want a command-line switch (would require some hacks)? Is this sufficient for testing?
Use cargo build --release --features fast to enable 3s block times.

OMG... I'm working on this RIGHT NOW... hehe.
Good on you. I just called the feature short-block-time instead. I don't recommend "fast" because it can mean many things.

OMG... I'm working on this RIGHT NOW... hehe.
Good on you. I just called the feature short-block-time instead. I don't recommend "fast" because it can mean many things.

Changed the feature name to yours. Are our code changes now byte-identical? :smile:

OMG... I'm working on this RIGHT NOW... hehe.
Good on you. I just called the feature short-block-time instead. I don't recommend "fast" because it can mean many things.

Changed the feature name to yours. Are our code changes now byte-identical? 😄

Yep, except that I chose 5 seconds instead of 3. I think the shorter the better though.
I just tested this in the functional tests and this seems to work and the tests are 3x fast. I'll add a commit with CI changes and then we merge. Then we get everyone to rebase on this if it passes here.


How to convert Kinv(or Kmat) into a (FoVx,FoVy,H,W) format?[Bug/issue]

Hi, gengshan. Thanks for your excellent code.
Could you share how to convert Kinv(or Kmat) into a (FoVx,FoVy,H,W) format?
I notice that if
            FoVx = 2 * torch.arctan(Kmat[0, 2] / Kmat[0, 0])             FoVy = 2 * torch.arctan(Kmat[1, 2] / Kmat[1, 1])
It is not correct.
When evaluating, the true resolution(e.g. 64x64) doesn't match [Kmat[0, 2], Kmat[1, 2]] (e.g. [60.01, 68.03])

Hi,
Kmat is in the following format
[[focal_x, 0, px],
 [0, focal_y, py],
 [0, 0,       1]]

where focal_x/y and px/y are in pixel unit
To convert focal to fov, you need
focal_unit = 2 * focal / img_size # normalize
fov = 2 * torch.atan(1 / focal_unit)

hope it helps.


Adding map tile assets

Just a place to put image assets, as there isn't a good spot in teams.

GAME ASSETS:

Make a player asset (different color shirts for different players, M/F variations?)
Make complete home asset
Make complete garden asset

With those assets we could build a map randomly which might be cool.


Type Inference on Lambdas may not be working

Version: 0.2.1-942-gb7e1747 (osx; from github)
Repro case:
actor Main
new create(env: Env) =>
let l1 = lambda(x: U32, y: U32) => x + y end
env.out.print(l1(5,6).string())

let l2 = lambda(x: U32, y: U32) : U32 => x + y end
env.out.print(l2(5,6).string())

Expected output:
11
11
Actual output:
None
11
Suspicion: ponyc's surmising a wrong type on L1 somehow, because the return type is not being explicitly declared.

I thought this was the correct behavior. The examples I see in the docs where a value is returned all have an explicitly defined return type.
Being that a lambda is sugar for an object with a method, and the "methods" section of the tutorial states that the return type can be omitted if you're not going to return anything, this would seem correct.

That makes a certain sense coming from the implementation details, but then
I'd expect a lambda of no return value to error on assignment.
On Mon, Jun 13, 2016 at 7:24 PM, Perelandric<EMAIL_ADDRESS>wrote:

I thought this was the correct behavior. The examples I see in the docs
where a value is returned all have an explicitly defined return type.
Being that a lambda is sugar for an object with a method, and the
"methods" section of the tutorial states that the return type can be
omitted if you're not going to return anything, this would seem correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ponylang/ponyc/issues/935#issuecomment-225763063, or mute
the thread
https://github.com/notifications/unsubscribe/AAeOrztWu-323k43PlQqN39FkFBzlysHks5qLhDSgaJpZM4I04JK
.

Pony doesn't try to guess or infer a return type based on the last expression for any function or any lambda - if you have no explicit return type, None is used as the default.
Note that this does not mean the lambda has "no return value" - instead, it returns the primitive named None, which happens to have a string method that returns the string "None".

I see, the error was in my intuition.  I expected 'fun derp(x: U32) => x + 5' to be type-inferred in the same way that a let of a typed fun is, and indeed it isn't.  Thanks for the pointers!

Yeah, I can see why that would seem counterintuitive.  I think the idea is that functions are used for their side-effects (when impure) just as often as they are used to produce a return value, so having the return type default to None is reasonable and common for those that are used for side effects only.  Assignments, by contrast, are conceptually always for producing a specific value to be assigned, so it wouldn't make much sense to assign None instead by default.

I feel a little weird about core language functionality returning None --
feels like null by another name.  Gonna have to mull that one over for a
while though.
On Tue, Jun 14, 2016 at 10:11 AM, Joe Eli McIlvain<EMAIL_ADDRESS>
wrote:

Yeah, I can see why that would seem counterintuitive. I think the idea is
that functions are used for their side-effects (when impure) just as often
as they are used to produce a return value, so having the return type
default to None is reasonable and common for those that are used for side
effects only. Assignments, by contrast, are conceptually always for
producing a specific value to be assigned, so it wouldn't make much sense
to assign None instead by default.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ponylang/ponyc/issues/935#issuecomment-225949945, or mute
the thread
https://github.com/notifications/unsubscribe/AAeOr4ur6H1lrKJNF14MzYNasI0uyvq4ks5qLuC2gaJpZM4I04JK
.

The difference is that None is really a primitive just like any other.  You could easily define primitive MyNone within the language, and there would be no substantial difference between it and the builtin None, other than the fact that they represent distinct types when doing subtype checks and distinct values when doing is comparisons.  So, in Pony None is not the absence of a value or type, it's just a specific type and value that we all agree to call by the name None.


Use HTTPS to resolve dependencies in Maven Build

Resolve dependencies of the AKSW repository over https.

Makes sense. Any concerns? @TortugaAttack @tohardik


Google Cloud Logging/Stackdriver doesn't include request ID field

I found it very interesting that the log from Google Cloud Logging/Stackdriver monitoring activities in my Bucket does not have the request ID. In Google Storage access log it has the request ID of any activities happening in the Bucket. It would be nice if the log from the Stackdriver has request ID field that can be correlated to the Google Storage access log.

Closed due to the lack of actionable information


Link to definition of "Application" as you did for "tenant"

It would be nice to have a link from the first occurrence of the word "application" to the definition (much like you did for "tenant", which was a link).
If the term "app" is a synonym for "application", it would be nice to clarify that. If it's not a synonym, then you should define it. The following is slightly confusing:
"When you register your application with Azure AD, you are creating an identity configuration for your application that allows it to integrate with Azure AD. When you register an app in the Azure portal, you..."
The context implies that they are the same, so this doesn't seem very confusing, but I've heard people talk about "apps" in an Azure authentication context, and what they were describing didn't sound much like an application. It's possible that the person teaching me failed to explain it well, but it's also possible that you're using the term "app" to mean something different from (or, more likely, broader than) the traditional meaning of the word "application [program]".
There is a definition of the term "application" at:
https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/what-is-application-management
in the section titled "What is an Identity and Access Management (IAM) system?"

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

ID: 6c12e139-8cb6-dd6d-2e08-89d4f22c7eef
Version Independent ID: 675b5ee9-bfe6-344f-7b39-ca0e2bc18b9b
Content: Apps & service principals in Azure AD - Microsoft identity platform
Content Source: articles/active-directory/develop/app-objects-and-service-principals.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@sfc-gh-mgilkey
Thanks for your feedback! We will investigate and update as appropriate.

I made a pull request to update the article. In the future feel free to make your own pull requests for changes like these. https://github.com/MicrosoftDocs/azure-docs-pr/pull/127288
CC @rwike77


Fix internal link

Before we can merge community PRs, the contributor has to sign the InfluxData Contributors License Agreement. Once you do that, I'll go ahead and merge this in. Thanks!

Before we can merge community PRs, the contributor has to sign the InfluxData Contributors License Agreement.

Done.

Thanks! Merging.


添加获取用户手机号接口

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Codecov Report

Merging #689 (45af1fd) into master (724c230) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##           master     #689   +/-   ##
=======================================
  Coverage   75.49%   75.50%
=======================================
  Files         103      103
  Lines        5007     5009    +2
=======================================
+ Hits         3780     3782    +2
  Misses       1227     1227

Flag
Coverage Δ

unittests
75.50% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

wechatpy/client/api/wxa.py
50.90% <100.00%> (+0.90%)
:arrow_up:

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 724c230...c85012c. Read the comment docs.

lgtm


Documentation and presentation

The project lacks for installation, describing the purpose of the project, etc. Furthermore, a presentation is also required.

Done d70bf29 and 205081b


WIP: [ci] [R-package] re-enable clang-18 testing (fixes #6369)

Fixes #6116
Fixes #6369
CRAN tests every submission with R-devel and clang-18. Until #6357, LightGBM's CI did as well, to reduce the risk of issues during submissions of the R package to CRAN.
As documented in #6369, those clang-18 CI jobs started failing a few weeks ago.
The root cause? It seems that the rhub/debian-clang-devel images are no longer getting updated, and the new clang-18 packages can't be installed in the last released version (https://github.com/microsoft/LightGBM/issues/6369#issuecomment-2016698615).
The maintainers of that image and the others used by R Hub appear to have abandoned https://github.com/r-hub/rhub-linux-builders in favor of https://github.com/r-hub/containers. This PR proposes re-enabling clang-18 tests by using an image from https://github.com/r-hub/containers.
Notes for Reviewers
This new test uses Ubuntu, not Debian
According to https://r-hub.github.io/containers/, the image that project uses to mimic r-devel-linux-x86_64-debian-clang is an Ubuntu image.

So it's not Debian... but testing against R-devel + clang-18 on every commit would be better than not testing that combination. And I'm proposing keeping the clang-16 and clang-17 jobs running on rhub/debian-clang-devel for as long as we can keep them working.

closing this... it'll be replaced by #6523


how to set training epoch when using miprov2

how to set training epoch when using miprov2

Hi @hjc3613 , do you mean num_batches?

Training epochs is not a concept in MIPRO. We're working on new optimizer docs, but in the meantime see the new MIPRO paper and the visual thread on MIPRO from Michael Ryan on twitter.

Hi @hjc3613 , do you mean num_batches?

the training time of miprov2 is a little long than my expection, so i want to know how to complete it early

Michael Ryan on twitter.

can you give me the link about the new MIPRO blog? thanks!!


as_OpenSpecy() for v1.0

[x] finalized functions
[x] added unit tests
[ ] complete documentation

Exciting! Hoping I can find some time soon to whip up this documentation 🙂
On Mon, Dec 12, 2022, 4:25 PM Zacharias Steinmetz @.***>
wrote:

@wincowgerDEV https://github.com/wincowgerDEV Sorry that it took so
long. Here is the first set of functions for finalization 🥳
—
Reply to this email directly, view it on GitHub
https://github.com/wincowgerDEV/OpenSpecy-package/pull/122#issuecomment-1346697400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMUJU25ZM3DU7HDV7LEPTDWM47X7ANCNFSM6AAAAAAS4CHCDY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think I got this pretty good now for the documentation. Were you hoping to make some additional tests for it too? I can go into testthat and check those out next.


Feature/1327/encode url

Leftover from ga4gh/dockstore#1327 and more

Encode the version name for the GA4GH endpoints (when branch is "feature/thing")
Handle get dockerfile errors to suppress console.logs
Fixes the highlight.js going crazy by giving it the thing to highlight when it thinks it needs to highlight (by always showing content when there is content even though it may not be a valid)

Codecov Report

Merging #270 into develop will decrease coverage by 0.03%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #270      +/-   ##
===========================================
- Coverage    59.08%   59.05%   -0.04%
===========================================
  Files          118      118
  Lines         3593     3595       +2
  Branches       466      466
===========================================
  Hits          2123     2123
- Misses        1239     1241       +2
  Partials       231      231

Impacted Files
Coverage Δ

src/app/shared/file.service.ts
26.66% <0%> (ø)
:arrow_up:

...c/app/container/dockerfile/dockerfile.component.ts
42.85% <0%> (-2.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 8481345...c2e478e. Read the comment docs.


Replace versioned Hazelcast docs with latest

Updated existing Hazelcast 5.x documentation links to use latest instead

@yuce are you able to take a look at this PR, please?

@yuce thanks, are you able to merge, please?


Add a new view for displaying awarded points in a modal dialog.

Overview
This branch adds a NewPointsView that can be used to display information about points earned in a modal dialog.

Most of the content was specific to this new view, but I did split out the AnimatedRing as a separate component that can be reused.
Security
REMINDER: All file contents are public.

[x] I have ensured no secure credentials or sensitive information remain in code, metadata, comments, etc.

[x] Please verify that you double checked that .storybook/preview.js does not contain your participant access key details.
[x] There are no temporary testing changes committed such as API base URLs, access tokens, print/log statements, etc.

[x] These changes do not introduce any security risks, or any such risks have been properly mitigated.

Describe briefly what security risks you considered, why they don't apply, or how they've been mitigated.
Testing

[x] MyDataHelps iOS app
[x] MyDataHelps Android app
[x] MyDataHelps website

Documentation

I have added some entries in the Storybook to show how the new components can be configured.

I'd like to merge this PR fairly soon if someone has the time to give it a review.  Thank you.

Bumping in case anyone has a chance to review.  Not urgent, but it would be nice to close this out this week.


Improve IF logging protocols

Types of changes

[x] Enhancement (project structure, spelling, grammar, formatting)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

A description of the changes proposed in the Pull Request

add --debug command functionality
add debug log messages into the codebase accordingly

this log appear whenever if is executed without adding --debug flag.

hi @manushak I found a problem wit this PR - it doesn't output to --stdout unless --debug is raised.

this log appear whenever if is executed without adding --debug flag.

Can you provide the command that you ran? I don't get that message

@manushak  yea - I fixed the manifest issue first and then found the bug. Try with any working manifest.

@manushak we are getting ton of checking aggregation method logs. Each field should appear once.
mock-obs-time-sync.yml was used to reproduce.

@manushak when running if with debug flag, exhaust's logs are gone under INFO: .

@manushak we are getting ton of checking aggregation method logs. Each field should appear once. mock-obs-time-sync.yml was used to reproduce.

@MariamKhalatova we get these huge logs because it traverses every input field.
@jmcook1186 Wdyt is this the right behavior?

@manushak It's aggregation log and it's called on each input (imagine what would happen with complex pipelines of 1000 nodes). There was memoize log function in code base, you can use that to log each one once.

@manushak @MariamKhalatova I think the aggregation check only needs to be displayed one time for each parameter (i.e. for each element in inputs[0]), as the operation is entirely idempotent - nothing can change between inputs[i] and inputs[i+1]...inputs[N] (or at least the probability is so low that it is not worth having the additional logs).

@MariamKhalatova, I've fixed. Please test

what's the status of this @manushak @MariamKhalatova ? have @MariamKhalatova 's requested changes been addressed?

what's the status of this @manushak @MariamKhalatova ? have @MariamKhalatova 's requested changes been addressed?

the requested changes have been fixed, and are waiting for testing


Breakpoint not hydrated after SSR render

Description
Breakpoint not hydrated
scroll-2024-01-23_15.25.38.webm
Steps to reproduce
Clone repo
pnpm i
pnpm dev
got to localhost:3000
in first load, breakpoint not apply, rezise with screen, breakpoint will hydrate when detect changed
Snack or a link to a repository (optional)
https://github.com/lobor/breakpoint-unistyle-repro
Unistyles version
2.1.0
React Native version
0.73.0
Platforms
SSR
Engine
Hermes
Architecture
Paper (old)

Hello 👋
Unistyles 2.0 has no custom CSS compiler, making it impossible to statically generate all styles. This includes the breakpoint feature, as concepts like window width do not exist on the server side. However, there is a plan to include this support in Unistyles 3.0.
There are some workarounds available, please check:

This thread: https://github.com/jpudysz/react-native-unistyles/discussions/86#discussioncomment-7875892
Documentation: https://reactnativeunistyles.vercel.app/reference/server-side-rendering/
Example repository with SSR: https://github.com/jpudysz/react-native-unistyles-ssr-example

You might also try experimentalCSSMediaQueries, which could assist with your case: https://reactnativeunistyles.vercel.app/reference/web-support/


Add Note About .icns File Format for OS X

:wave:
You're sitting right next to me and saw me type these things so I hope you like it :pizza: :cat2:

Do we have to do a special build with  the .icns format just for OSX or can we still do a build for all platforms with that format? If a special build is needed it would make sense to add a conversion tool, like png to icns to support uniform implementation when the 'all' member is set to true, or non OSX builds are mixed with OSX builds.

@RIAEvangelist it's a little bit more complicated than that. In my app, I generate .icns and .ico files from a .svg file, and found that the PNG :arrow_right: ICNS tool depends on what the build OS is.

@RIAEvangelist I would guess that a module that did icns, ico, png and svg conversion would be pretty heavy. I think the simplest thing we could do is have e.g. --mac-icon, --linux-icon, --win-icon flags (or maybe using the platform name so --darwin-icon etc)

Currently the icon option automatically strips the extension of the filename and looks for icns for Mac and ico for Windows.

@kfranqueiro ...I did not know that. That should be documented :smile:

Wow @kfranqueiro ! Sweet knowledge share there! That is indeed very important to be documented.

It was implemented as part of #88, since otherwise --all with --icon didn't make a whole lot of sense.  Looking at how it's doc'd (or not), yeah, I should've thought to expand on it.  I was probably not sure of how much I should touch docs at the time.

I updated the PR to have a list stating the OS X and Windows filetypes.
I have no idea how Linux is supposed to be done? Do users need a .desktop file?

Linux just needs a .png, and it doesn't need to be set in --icon, per my comments in https://github.com/maxogden/electron-packager/issues/90#issuecomment-117808176.

@malept Hmm. I read that thread but it wasn't clear to me and I had tried with a PNG. But I will try again not passing in --icon at all and just relying on the png in BrowserWindow .

@jlord I think I need some clarification on the behavior you're expecting. AFAIK, in Linux file managers, you can't embed an icon in the executable and have it show up when it's in the executable list. The BrowserWindow setting is required to show the icon in the window list (unlike, IIRC, OS X).

@malept the thing I'm testing/trying for is an app icon in the file manager and dock on Ubuntu.

Heres a snippit from my video player :

Here's a snippit from my video player its in main.js :
app.on(
    'ready',
    function() {
        player = new BrowserWindow(
            {
                title:'Electron Video Player',
                'accept-first-mouse':true,
                width: 640,
                height: 480,
                'min-width': 640,
                'min-height': 480,
                frame:false,
                icon:__dirname+'/img/logo-256.png',
                'text-areas-are-resizable':false
            }
        );

        player.loadUrl('file://' + __dirname + '/player.html');

        //player.openDevTools();

        player.on(
            'closed',
            function() {
                player = null;
            }
        );
    }
);

you can see where I set the linux icon.

@RIAEvangelist Hmm, really not sure what I'm doing differently but that doesn't work for me. I'll open another issue for this specifically so we can clear it up and add it to the docs.
Otherwise @maxogden this PR is good to go for at least specifying the filetypes for OS X and Windows.


Is there a way to export two different tables (which are linked by ForeignKey in TabularInline) in one excel file?

Like, models.py
class Person(models.Model):
         name = models.CharField(max_length=50)
         age = models.IntegerField()
         income = models.CharField(choices=income_choices, max_length=50)

class DetailsOfEducationQualification(models.Model):
         person = models.ForeignKey(Person, on_delete=models.CASCADE)
         course_class = models.CharField(max_length=50, blank=True)
         satisfaction_of_study = models.BooleanField(default=False, blank=True)
         scholarship_received = models.BooleanField(default=False, blank=True)
         "Specify the scholarship Left"
         drop_out = models.BooleanField(default=False, blank=True)
         "Reason Of DropOut Left"
         drop_out_reason = models.CharField(max_length=1000, blank=True)

class ScholarshipNames(models.Model):
         details = models.ForeignKey(Person, on_delete=models.CASCADE)
         name = models.CharField(max_length=200, blank=True)

In admin.py,
from .models import Person, DetailsOfEducationQualification, ScholarshipNames

class DetailsOfEducationQualificationInline(admin.TabularInline):
model = DetailsOfEducationQualification
extra = 1
class ScholarshipNamesInline(admin.TabularInline):
model = ScholarshipNames
extra = 1
class PersonAdmin(admin.ModelAdmin):
fieldsets = [
(
'Personal Information', {
'fields':[
'name', 'age', 'education_level', 'occupation_status', 'income',
]
}
),
]
inlines = [
DetailsOfEducationQualificationInline, ScholarshipNamesInline,
]

The question I asked is, How do I put these three models in one CSV file while exporting since it is related to each other...

Like, models.py
class Person(models.Model):
         name = models.CharField(max_length=50)
         age = models.IntegerField()
         income = models.CharField(choices=income_choices, max_length=50)

class DetailsOfEducationQualification(models.Model):
         person = models.ForeignKey(Person, on_delete=models.CASCADE)
         course_class = models.CharField(max_length=50, blank=True)
         satisfaction_of_study = models.BooleanField(default=False, blank=True)
         scholarship_received = models.BooleanField(default=False, blank=True)
         "Specify the scholarship Left"
         drop_out = models.BooleanField(default=False, blank=True)
         "Reason Of DropOut Left"
         drop_out_reason = models.CharField(max_length=1000, blank=True)

class ScholarshipNames(models.Model):
         details = models.ForeignKey(Person, on_delete=models.CASCADE)
         name = models.CharField(max_length=200, blank=True)

In admin.py,
`
from .models import Person, DetailsOfEducationQualification, ScholarshipNames
class DetailsOfEducationQualificationInline(admin.TabularInline):
model = DetailsOfEducationQualification
extra = 1
class ScholarshipNamesInline(admin.TabularInline):
model = ScholarshipNames
extra = 1
class PersonAdmin(admin.ModelAdmin):
fieldsets = [
(
'Personal Information', {
'fields':[
'name',  'age',  'education_level', 'occupation_status', 'income',
]
}
),
]
inlines = [
DetailsOfEducationQualificationInline, ScholarshipNamesInline,
]`
The question I asked is, How do I put these three models in one CSV file while exporting since it is related to each other...

Greetings!  That's a bit complicated, so is the type of thing you'd have to implement yourself on export, perhaps using the dehydrate functionality.

Hey, I'm running into a similar issue, when exporting 2 or more related models, I want the values to be in a single CSV file.
But I find myself needing to dehydrate almost every field in x number of related models that I want to export to make it work.
Is there a cleaner workaround for this?

Greetings! That's a bit complicated, so is the type of thing you'd have to implement yourself on export, perhaps using the dehydrate functionality.

Hi. How about importing? Is there a way to import inlines of a model?

We added functionality to filter exports, which could help solve this kind of issue.  Closing due to inactivity.


[ENHANCEMENT] keep being so awesome

I love this, thank you so much

Thanks so much for the feedback, but this issue type is for requesting features rather than giving feedback.😅 If you want to share feedback, you can do son on the discord server.


Consul session support

The stuff I added so far is enough for me for the time being, but will probably need to add more eventually in the future when I need more functionality.

Merged. Great job!


Duplicate structure placement

I've been parsing some release-20180923 replays and noticed that sometimes two nearly identical building placements happen in the span of a few seconds. For example:
        {
          "structure": "proc",
          "game_time": {
            "formatted": "00:03:30",
            "msec": 210000
          },
          "placement": {
            "x": 66,
            "y": 66
          }
        },
        {
          "structure": "proc",
          "game_time": {
            "formatted": "00:03:32",
            "msec": 212400
          },
          "placement": {
            "x": 65,
            "y": 66
          }
        },

Here's a replay where that occurs twice (with PROC and WEAP): http://openra.mine.nu/mIRROR/ora_replays/RAGL_Season_Six/R02/Master/RAGL-S06-MASTER-R02-NNG-ZXG-G1.orarep

Can't currently see anything in the order data that I could use to indicate whether the PlaceBuilding command failed or succeeded, will have to consult IRC / @pchote on this.

@dragunoff I think this was down to timing calculations being incorrect, I got help from an openra dev on Discord and they should be working as expected after https://github.com/AMHOL/openra-ruby/pull/10 (2.2.0)