[SPARK-25314][SQL] Fix Python UDF accessing attributes from both side of join in join conditions

What changes were proposed in this pull request?
Thanks for @bahchis reporting this. It is more like a follow up work for #16581, this PR fix the scenario of Python UDF accessing attributes from both side of join in join condition.
How was this patch tested?
Add  regression tests in PySpark and BatchEvalPythonExecSuite.

Test build #95653 has started for PR 22326 at commit 23b1028.

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/testing-k8s-prb-make-spark-distribution-unified/2820/
Test PASSed.

Test build #95653 has finished for PR 22326 at commit 23b1028.

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/95653/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/2829/
Test PASSed.

Test build #95662 has started for PR 22326 at commit d58f3a5.

Test build #95662 has finished for PR 22326 at commit d58f3a5.

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/95662/
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/testing-k8s-prb-make-spark-distribution-unified/2840/
Test PASSed.

Test build #95676 has started for PR 22326 at commit a86a7d5.

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/testing-k8s-prb-make-spark-distribution-unified/2841/
Test PASSed.

Test build #95678 has started for PR 22326 at commit 82e50d5.

Test build #95676 has finished for PR 22326 at commit a86a7d5.

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/95676/
Test FAILed.

Test build #95678 has finished for PR 22326 at commit 82e50d5.

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/95678/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/2875/
Test PASSed.

Test build #95716 has started for PR 22326 at commit 63fbcce.

Test build #95716 has finished for PR 22326 at commit 63fbcce.

This patch fails Python style 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/95716/
Test FAILed.

Test build #95718 has started for PR 22326 at commit 777b881.

Test build #95718 has finished for PR 22326 at commit 777b881.

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/95718/
Test PASSed.

Gental ping @mgaido91 @HyukjinKwon @dilipbiswal, great thanks for advice, please have a look when you have time.

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/testing-k8s-prb-make-spark-distribution-unified/2922/
Test PASSed.

Test build #95789 has started for PR 22326 at commit 87440b0.

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/testing-k8s-prb-make-spark-distribution-unified/2923/
Test PASSed.

Test build #95790 has started for PR 22326 at commit 4d546e2.

Test build #95790 has finished for PR 22326 at commit 4d546e2.

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

Test build #95789 has finished for PR 22326 at commit 87440b0.

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/95790/
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/95789/
Test PASSed.

nit: attibutes in the title probably mean attributes

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/testing-k8s-prb-make-spark-distribution-unified/2945/
Test PASSed.

Test build #95830 has started for PR 22326 at commit 6875719.

@holdenk Thanks, sorry for the typo.

Test build #95830 has finished for PR 22326 at commit 6875719.

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

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

Merged build finished. Test PASSed.

IIUC, you are pulling out the join condition with python UDF and create a filter above join. Then the join become a cross join, which usually runs very slowly. I think we should keep the cross join check for this case.

@cloud-fan Thanks for your comment.
IIUC, you are pulling out the join condition with python UDF and create a filter above join. Then the join become a cross join, which usually runs very slowly.

Yes, that's right.
I think we should keep the cross join check for this case.

Yes, as Marco suggestion, the currently behavior is control cross join by crossJoinEnabled config, if crossJoinEnabled = false, it will throw AnalysisException.

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/testing-k8s-prb-make-spark-distribution-unified/3381/
Test PASSed.

Test build #96474 has started for PR 22326 at commit e7c1aee.

@cloud-fan Great thanks for your offline guidance, as our discussion, I reimplement this by adding a new rule HandlePythonUDFInJoinCondition in Analyzer, revert all changes in Optimizer before.

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/testing-k8s-prb-make-spark-distribution-unified/3384/
Test PASSed.

Test build #96482 has started for PR 22326 at commit caf6f94.

Test build #96482 has finished for PR 22326 at commit caf6f94.

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/96482/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/3389/
Test PASSed.

Test build #96488 has started for PR 22326 at commit caf6f94.

Test build #96488 has finished for PR 22326 at commit caf6f94.

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/96488/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/3397/
Test PASSed.

Test build #96498 has started for PR 22326 at commit caf6f94.

Test build #96498 has finished for PR 22326 at commit caf6f94.

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/96498/
Test FAILed.

retest this please

Test build #96503 has started for PR 22326 at commit caf6f94.

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/testing-k8s-prb-make-spark-distribution-unified/3402/
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/96503/
Test PASSed.

move this rule to optimizer, as the last batch (but before the UpdateAttributeReferences batch). Since we apply this rule after filter pushdown, we can simply pull out any python udf in join condition. Also add this rule to Optimizer.nonExcludableRules, since this is a special optimizer rule that can't be turned off.

Make sense, implement like this can also avoid breaking the assumption in Dataset.join of Join plan only return Join after analysis. I'll reimplement as this proposal soon.

Test build #96543 has started for PR 22326 at commit 3295ebf.

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/testing-k8s-prb-make-spark-distribution-unified/3429/
Test PASSed.

Test build #96543 has finished for PR 22326 at commit 3295ebf.

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/96543/
Test FAILed.

Test build #96548 has started for PR 22326 at commit 005bb3f.

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/testing-k8s-prb-make-spark-distribution-unified/3432/
Test PASSed.

Test build #96548 has finished for PR 22326 at commit 005bb3f.

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/96548/
Test FAILed.

Test build #96549 has finished for PR 22326 at commit b0dfab3.

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/96549/
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/testing-k8s-prb-make-spark-distribution-unified/3439/
Test PASSed.

Test build #96560 has started for PR 22326 at commit 306fcb9.

Test build #96560 has finished for PR 22326 at commit 306fcb9.

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/96560/
Test PASSed.

Does it make sense to have a test case where the PythonUDF referencing attributes from both legs of join in disjunction with a regular predicate ? i.e joinCond = pythonUDF(leftattr, rightattr) || leftattr > 5 ..

Test build #96600 has started for PR 22326 at commit 98cd3cc.

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/testing-k8s-prb-make-spark-distribution-unified/3465/
Test PASSed.

Test build #96600 has finished for PR 22326 at commit 98cd3cc.

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/96600/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/3482/
Test PASSed.

Test build #96621 has started for PR 22326 at commit d1db33a.

Test build #96617 has finished for PR 22326 at commit 98cd3cc.

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/96617/
Test PASSed.

Test build #96621 has finished for PR 22326 at commit d1db33a.

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/96621/
Test PASSed.

Test build #96630 has started for PR 22326 at commit 87f0f50.

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/testing-k8s-prb-make-spark-distribution-unified/3487/
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/testing-k8s-prb-make-spark-distribution-unified/3488/
Test PASSed.

Test build #96631 has started for PR 22326 at commit d2739af.

Test build #96635 has started for PR 22326 at commit 7f66954.

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/testing-k8s-prb-make-spark-distribution-unified/3491/
Test PASSed.

Test build #96630 has finished for PR 22326 at commit 87f0f50.

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/96630/
Test PASSed.

Test build #96635 has finished for PR 22326 at commit 7f66954.

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/96635/
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/testing-k8s-prb-make-spark-distribution-unified/3510/
Test PASSed.

Test build #96659 has started for PR 22326 at commit 2b6977d.

Test build #96659 has finished for PR 22326 at commit 2b6977d.

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/96659/
Test PASSed.

thanks, merging to master/2.4!

Thanks everyone for your review and advise.

late LGTM


error on actor hook setting

following error is thrown when setting the character sheet hook setting. I believe this is also causing that setting to not work. foundry 0.8.7

need more information than this, with only item-macro enabled the module seems to work fine on 0.8.7

So I used find the culprit to only test it with item macro and it still threw the error (occurs when enabling or disabling the setting and clicking save changes). on dhd 5e 1.3.5 and 0.8.7
anything else I can provide?

yea just tried it on a fresh world too and the error still occurred

believe i found the issue, going to be pushing it soon.

Glad to hear, sorry I couldn't provide more information!

Sorry for the wait, updating now.


Include tasks with modules in string format

For service tasks like:
service: name=httpd state=stopped enabled=no

 service


[Fix] Fix build installation

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@366e511). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #94   +/-   ##
=======================================
  Coverage        ?   76.21%
=======================================
  Files           ?       59
  Lines           ?     1455
  Branches        ?      199
=======================================
  Hits            ?     1109
  Misses          ?      276
  Partials        ?       70

Flag
Coverage Δ

unittests
76.21% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Documentation on Quarkus site has bad configuration table

Seems a double nats is happening

I can see you have fixed it


I managed to run the sample on Unity 2019.4

Many people in the Issues asked how to run the samples on Unity 2019.4, and I managed to run the example. I'll share what I did to get it to run so that others can hopefully avoid the time I spent experimenting and trying it out. Maybe someone with more knowledge can also fix my workarounds.
Maybe this isn't hard for experienced people, but it took a good long day to figure it all out.
To be clear, I ran the "Build2019Demo.Unity".
Relevant prerequisites
Below is the hardware and Unity version I use.
HoloLens 1st Gen
Unity 2019.4.34f1
Windows 10
Android OnePlus 5T
What I did

Clone master (commit 7796da6acb0ae41bed1b9e0e9d1c5c683b4b8374 when I did it)
Follow the repo's instructions

Run tools\scripts\CreateUnityPackage.bat in an administrator cmd window

Now, I can use Unity to open the project. I still had some compiling errors on Unity
Change packages used

I followed this link, which allows ASA to work with 2019.3
There is probably a way to update this package in the bat file process itself, but I didn't look into it
I updated the following libraries to these versions

AR Foundation: 3.1.3
ARCore XR Plugin: 3.1.3
ARKit XR Plugin 3.1.3

If your AR Foundation gives you compilation problems regarding obsolete API, check if it's something in 'Assets\AzureSpatialAnchors.SDK\Scripts\SpatialAnchorManager.cs'.

If it is, follow the same link in step 4, "Update the ASA SDK to ARFoundation 3 objects" part.

Now, the only problems should be coming from Text Mesh Pro. I removed references to ignoreRectMaskCulling
If Unity ever complains about upgrading API, just say yes.
After changing platforms for UWP/ Android, I did not have any compilation issues.
At this point, you can follow the instructions in the repo again.
I ran on HoloLens and managed to connect with my Android OnePlus 5T phone

I will close this since this isn't an actual issue, but more like documenting how I got this to run. Hopefully, this helps someone else.


Add support for account switch key ( ASK )

Please add support for account switch key support in CLI

this is not supported by Akamai. I do not see that you can enable ASK for siteshield for account.


Upgrade the version of HTML_codesniffer

Version used

AET 3.0

Context of the issue/feature
It looks like AET makes use of some old version of HTML_codesniffer (accessibility health check). We should upgrade it.

there is a separate task for updating the documentation at 420.


[RediSearch] Multi Index Search Support

Currently it is not possible to search over multiple indexes via RediSearch.

at current state we will not implement multi index search see #99 and #451


ZCL Electrum wallet no connection

Downloaded the new wallet and test receiving funds ok. But then it went offline i.e. showing no connection. Tried restarting the app to no avail.
At the time network showing none connected. Set server automatically was ticked. Only has zcl-electrum.com:50002 in the list
Platform: MacOS 10.13.3

@g-coxon Any T address will certainly be eligible

@ch4ot1c Are T address literally just an address tat starts with a 't'?

@ch4ot1c Seems I still have issues with the wallet. It looks like it does synchronized but I'm not receiving the coin I sent to my address.
This is the transaction that I made from mining.
http://explorer.zclmine.pro/address/t1fVfghBHc2cEVhz25NaH8fnNQUxwmAjcL6
Tx http://explorer.zclmine.pro/tx/e848f88f66122b70edcdf028dbf04fd6f777c00418138c310b48e9ffc78d5861
But electrum wallet did not show such transaction.


Twitter exercise

Create an exercise on rendering a Twitter map using the data already gathered here.
You need to do a branch, create the exercise and the propose a PR with the .carto file, the exercise and any other resource updated in the repository. Let me know if you need help on this.

@jsanz is there any criteria for this other than use the data provided?

It's in the visualization module so you shouldn't do any análisis or sql
hack, just a plain animated map with some meaningful widgets. Michelle can
give you some guidelines
--
Jorge Sanz
CARTO - Predict through location
Sent from my phone, excuse my brevity.
El 6 jul. 2017 20:08, "alexsroth"<EMAIL_ADDRESS>escribió:
@jsanz https://github.com/jsanz is there any criteria for this other than
use the data provided?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/carto-workshop/issues/55#issuecomment-313475215,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALfaFCTXC-yzWLeiWDdMHZ7tx8HmHgTks5sLSK3gaJpZM4OP3eY
.


node cli broken

Can you run npm list -g makecode for me and tell me what version it lists? I just ran this and 'worked on my machine', not sure what could be going on besides it grabbing 1.0.0 for some reason; do we need to unpublish that?
REDMOND+jowunder MSYS ~/test-dir
$ git clone https://github.com/microsoft/pxt-jacdac
Cloning into 'pxt-jacdac'...
remote: Enumerating objects: 54412, done.
remote: Counting objects: 100% (8216/8216), done.
remote: Compressing objects: 100% (1101/1101), done.
remote: Total 54412 (delta 7079), reused 8190 (delta 7063), pack-reused 46196
Receiving objects: 100% (54412/54412), 8.19 MiB | 11.42 MiB/s, done.
Resolving deltas: 100% (46073/46073), done.

REDMOND+jowunder MSYS ~/test-dir
$ cd pxt-jacdac/

REDMOND+jowunder MSYS ~/test-dir/pxt-jacdac (master)
$ npm install makecode

added 24 packages, and audited 25 packages in 4s

3 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

REDMOND+jowunder MSYS ~/test-dir/pxt-jacdac (master)
$ npx makecode
using project: C:\Users\jowunder\test-dir\pxt-jacdac/pxt.json
using config: C:\Users\jowunder\test-dir\pxt-jacdac\mkc.json
  include: C:\Users\jowunder\test-dir\pxt-jacdac\links.json
Download new webapp
Download https://makecode.microbit.org/
Download https://makecode.microbit.org/---manifest
Download https://pxt.azureedge.net/commit/621c40f952fe2c381892d49906e735eb774b0415/pxtworker.js
Download https://pxt.azureedge.net/commit/621c40f952fe2c381892d49906e735eb774b0415/target.json
Download https://pxt.azureedge.net/api/config/microbit/targetconfig/v5.0.12?cdn=20230202
Download https://trg-microbit.userpxt.io/---simulator
Download https://pxt.azureedge.net/blob/cf867ede4b0633c64dd32588ca6e2edc3aa67f98/pxtsim.js
Download https://pxt.azureedge.net/blob/a1a89319879eeabb01b4046263df0575fe619fd9/sim.js
using editor: https://makecode.microbit.org/ v5.0.12
Download https://pxt.azureedge.net/compile/bb8015207e408ac379e2a04df82b9a4d63d0cd7790079424e1835cbd7753c693.hex
Download https://pxt.azureedge.net/compile/7fb54a2f8854c250e22ce176d8b01448f8494b1aa1974f6e24c83fd4466ef1f5.hex
compiling C++; this can take a while
Download https://www.makecode.com/api/compile/extension
Download https://makecode.com/compile/7fb54a2f8854c250e22ce176d8b01448f8494b1aa1974f6e24c83fd4466ef1f5.json
Download https://makecode.com/compile/7fb54a2f8854c250e22ce176d8b01448f8494b1aa1974f6e24c83fd4466ef1f5.json
build log https://makecode.com/compile/7fb54a2f8854c250e22ce176d8b01448f8494b1aa1974f6e24c83fd4466ef1f5.log
Download https://makecode.com/compile/7fb54a2f8854c250e22ce176d8b01448f8494b1aa1974f6e24c83fd4466ef1f5.hex
write C:\Users\jowunder\test-dir\pxt-jacdac\built/yotta.json
write C:\Users\jowunder\test-dir\pxt-jacdac\built/codal.json
write C:\Users\jowunder\test-dir\pxt-jacdac\built/mbdal-binary.asm
write C:\Users\jowunder\test-dir\pxt-jacdac\built/mbdal-binary.hex
write C:\Users\jowunder\test-dir\pxt-jacdac\built/mbcodal-binary.asm
write C:\Users\jowunder\test-dir\pxt-jacdac\built/mbcodal-binary.hex
write C:\Users\jowunder\test-dir\pxt-jacdac\built/binary.hex
Build OK

@pelikhan ➜ /workspaces/pxt-jacdac (master) $ npm list -g makecode
/usr/local/share/npm-global/lib
└──<EMAIL_ADDRESS>

@pelikhan ➜ /workspaces/pxt-jacdac (master) $ npm install -g -u makecode

added 24 packages, and audited 25 packages in 397ms

3 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
@pelikhan ➜ /workspaces/pxt-jacdac (master) $ npm list -g makecode
/usr/local/share/npm-global/lib
└──<EMAIL_ADDRESS>
@pelikhan ➜ /workspaces/pxt-jacdac (master) $

1.1.0 is the latest release on npm


Errors at building a book

Have several errors in my  arch linux environment, including:

could not import cblas_* , while modules exist in libcblas.so
/home/username/getting-started/book/numerical_methods/interpolation.nim(108, 38) Error: undeclared identifier: 'Edge'
/home/username/.choosenim/toolchains/nim-1.6.14/lib/pure/strutils.nim(1073, 5) Error: unhandled exception: invalid integer: 0-DEV [ValueError]

Output file is attached.
Please help,
Thanks
book_out.txt

This looks similar to this issue? Can you try running the specific files using -d:blas=cblas?
You need the newest version of numericalnim for it to work. We should fix the version in the nimble file.
This looks like an error with the Julia pages. Have you set up Julia correctly? (see nimjl

Good that those were solved. For Julia, @Clonkk is the one with the most experience.

Thanks. It helps.

After removing cblas and using -d:blas=cblas these errors are gone
After installing the newest version of numericalnim that error is gone
Still have Julia integration errors. I switched from development Julia version 1.11.dev to stable 1.9 with my package manager. Now I have different errors: "copy relocation against non-copyable protected symbol `jl_main_module' in /usr/lib/libjulia.so"

I haven't tested Julia devel yet I am still on 1.9.x if they changed the way they build theur binaries it is possible they introduce regression in the C API (it has happened before).
Have you tried building julia from source by curiosity  ?

I've compiled julia-dev from source, while installed julia 1.9 by my package manager. I've found similar C-API problems mentioned on the web. What can you suggest? Like 2 ways:

Switch to Julia 1.7.x (but loose some performance)
Compile 1.9.x from source (but without proof of success) ?

I've compiled 1.9.2 from source, still same errors.
Can I generate a book and everything without interface to julia?

Can I generate a book and everything without interface to julia?

That's basically what you are currently doing. It will generate all pages of the book that don't error. They will still be in the sidebar but if you don't plan on reading them it doesn't matter.

Ok, after commenting nimjl I have all docs generated properly. Thanks!

Nice! Out of curiosity, why do you want to generate the book? Local documentation? Or are you planning on contributing a tutorial? 😉

Just nice local documentation, kind of perfectionism  ;)

Ok nice 😁


vélin

as suggested by @dcherian in https://github.com/pydata/xarray/pull/4867#issuecomment-774489458, ~this adds velin to the pre-commit hooks~ as suggested below, we'll leave that hook deactivated for now.
I think this would help enforcing the style of the docstrings a bit more. However, while most of the changes it proposes are great, the tool itself does not seem to be mature, so I'll mark this PR as a draft.

[x] Passes pre-commit run --all-files

I think the changes look great! Thanks for finding it!
I would probably start by applying it first and then as we gain more experience, later add it to pre-commit hooks; so we can learn whether it has any sharp edges. Because this affects all contributors — even those making a small docs PR for the first time — if it has any confusing behavior, it could be fairly frustrating for them.
We did this for isort IIRC and then later added it to pre-commit (though given its success, I wonder if that experience updates us to add tools slower or faster...)

I found a way to remove the extra newlines, so the missing , aside this actually does look really good.

Looks great! should we merge just before the next release to avoid conflicts with open PRs?

I don't think we have to worry about that: the number of lines this PR changes is pretty small, only touches the documentation and even if someone gets a merge conflict that would be a few lines at most.

Great. merging!

Nice! Thanks!


Please add an unsaved indicator to the Zen Mode.

In Zen Mode(hide tabs is on),there is no unsaved indicator:

Please add an unsaved indicator next to the file name:

Like the title bar in normal mode:

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks

Okay,that feature is not important,it should be just on backlog.
And thanks for your work in vscode.

Do I really have to go solve it with an extension? Isn't it against the "zen" to have to think about if I saved or not or spam the save button after every change one makes?
To me, it's just a dot. It should be there in the first place, it doesn't make any sense to hide it. We shouldn't even have to talk about it.
To me, it's a bug. A long-forgotten bug.
Please fix it.

To save someone the effort, if you enable the status line when in zen, this extension works well:
https://marketplace.visualstudio.com/items?itemName=esamatti.unsaved


avoidOptionals inputValue as false not producing optional fields

Describe the bug
In the following live demo:

observe an optional GQL query input, userId: ID
you can see that we are avoidOptionals: { inputValue: false } for that value
the output does avoid optionals, despite being instructed not to. you can see the following output:

export type FindUserQueryVariables = Exact<{
  userId: Maybe<Scalars['ID']>;
}>;

Interestingly, if you flip object: false as well, suddenly the correct behavior takes for   inputValue: false. Perhaps these are unintentionally coupled?
https://www.graphql-code-generator.com/docs/plugins/typescript-operations
To Reproduce
Codesandbox has been down for me. The schema, query, & config are listed below. The repro is quite simple :)

My GraphQL schema:

Stripped down version of the homepage demo:
schema {
  query: Query
}

type Query {
  user(id: ID): User
}

interface Node {
  id: ID!
}

type User implements Node {
  id: ID!
}

My GraphQL operations:

query findUser($userId: ID) {
  user(id: $userId) {
    id
  }
}

My codegen.yml config file:

generates:
  operations-types.ts:
    plugins:
      - typescript
      - typescript-operations
    config:
      avoidOptionals:
        defaultValue: true
        field: true
        object: true # toggle me!
        inputValue: false

Expected behavior
To produce an optional value for the userId?: Maybe<Int> field:
expected:
export type FindUserQueryVariables = Exact<{
  userId?: Maybe<Scalars['ID']>;
}>;

actual:
export type FindUserQueryVariables = Exact<{
  userId: Maybe<Scalars['ID']>;
}>;

Environment:
latest, deployed on graphql-code-generator website
Additional context
This PR https://github.com/dotansimha/graphql-code-generator/pull/5113, in the bottom comment, seems to suggest that this design is intentional. However, object seems orthogonal to inputValue for avoidOptionals configuration, and produces an unexpected result. If they are not orthogonal, then inputValue should take precedence over object

Hello, are there any updates here? Or other ways to solve this issue?


HA Core Error on removal of integration

On removing the O365 integration I receive these errors following any HA restart.
Logger: homeassistant.setup
Source: setup.py:269
2024-12-21 17:57:37.285 WARNING (MainThread) [homeassistant.helpers.translation] Failed to load integration for translation: Integration 'o365' not found.
Logger: homeassistant.helpers.translation
Source: helpers/translation.py:226
2024-12-21 17:57:40.648 ERROR (MainThread) [homeassistant.setup] Setup failed for 'o365': Integration not found.
Both adding and removing were performed through the HACS module
I tried re-adding and re-removing O365.  Errors stop when O365 is reinstalled, but reappear again on removal.
I migrated to your M365 Calendar integration in case that has any bearing on issue
Also -- Thank you for maintaining the office integrations!

Did you remove the o365 config from configuration.yaml?

That worked - thank you!


I got message "disconnected" from twilio-conversations what does it mean ?

async updateLastMsg(position) {
try {
await this.client.updateLastConsumedMessageIndex(position);
return true;
} catch (error) {
//func send error on prod
return false;
}
}
I'm using above code to updateLastMsgIndex but on my production i got error message "disconnected". Can you guy help me explain what does it mean ? The func above will be called when user focus the screen, when componentUnmount or when we have new message.
I'm using twilio-conversations 2.6.0 (newest)

@berkus Can you help me check this you need something to check please tell me.


randomly generate etcd tokens

the generate command uses a hardcoded etcd clusterToken, this will get us into trouble eventually.

#574 and #353 are both related and have more context. Closing this even though it is older. The labels are comparable.


pdflatex is very "geschwätzig"

I like to use the command "manage.py renderallpdf". In order to reduce the output of "pdflatex" as an option I use:
os.system(f'texfot pdflatex ../{note} svg')

Yeah, at the moment the renderallpdf command isn't very well implemented and there are better ways to interact with the system than os.system. For example it would be nice to check for errors in the scripts and also render the files in a sensible order, so I plan on changing this in the future, as well as adding in methods that render only the files that have changed.

I've changed renderallpdf in the latest commit


vkquake: <IP_ADDRESS> -> 1.31.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Novum/vkQuake/releases.
meta.description for vkquake is: Vulkan Quake port based on QuakeSpasm
meta.homepage for vkquake is: https://github.com/Novum/vkQuake
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.31.2 with grep in /nix/store/jwvfmr1lp0237kln3v609iansq8qa182-vkquake-1.31.2
found 1.31.2 in filename of file in /nix/store/jwvfmr1lp0237kln3v609iansq8qa182-vkquake-1.31.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

vkquake

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/3f55b10115faf5ca114e5d5b1aa0c0857b6e269f#vkquake

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

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

1 package built:

vkquake

Maintainer pings
cc @poperigby @ylh for testing.

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

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

@NixOS/nixpkgs-merge-bot merge


prepare the development environment to make code changes

Hello everyone, as part of the Manageiq project I am responsible for making changes to the service catalog by grouping the services created in a drop down menu to have a good user interface and easy to choose the services to create.
I am a Developer and I don't know how to prepare the development environment to make code changes, where can I start, i think that using an OVA is easy but still have no idea, i work alone, I searched on Documentation and I found to use an OVA Machine will be easier to prepare a development environment for Manageiq
I need some help please 🙏

Can yo give more details on what you're trying to accomplish?  It's hard to understand what exactly you're trying to solve.

Hello everyone, as part of the Manageiq project I am responsible for making changes to the service catalog by grouping the services created in a drop down menu to have a good user interface and easy to choose the services to create.
I am a Developer and I don't know how to prepare the development environment to make code changes, where can I start, i think that using an OVA is easy but still have no idea, i work alone, I searched on Documentation and I found to use an OVA Machine will be easier to prepare a development environment for Manageiq
I need some help please 🙏

The docs for setting up the dev environment are here in this repo under https://github.com/ManageIQ/manageiq-ui-service/blob/master/docs/developer_setup.md.  For ManageIQ in general, please refer to the Developer Setup guide here: https://github.com/ManageIQ/guides/blob/master/developer_setup.md

Yes indeed, I would just like to know how I'm going to prepare the Dev environment, as I'm working in a project with a freelance company, and I'm in charge of setting up an environment but I don't know where I'm starting from, what do you recommend as a trick whether it's an OVA machine already configured or a Docker container, Kubernetes...

The recommendation for development in the ManageIQ org is to use your local machine for development and follow the developer setup as documented.  I know there are some developers who like to develop on the appliance directly, but in our experience that makes development much more complicated.  There are docs for developing in a vagrant VM, but I personally have never done that, so I cannot attest to how well it works or not.


Add star at end to create complete bold.

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @alexlewiseburt, thank you for the PR! Unfortunately, your branch has some merge conflicts that are difficult to resolve. The painless way to fix this is to create a fresh PR.
So I'm going to close your PR now and looking forward to your new one. I'm so sorry about the inconvenience.


Error after vagrant up - Vagrant attempted to execute the capability 'configure_networks'

On OSX i get error after issuing vagrant up-command

Vagrant attempted to execute the capability 'configure_networks'
on the detect guest OS 'linux', but the guest doesn't
support that capability. This capability is required for your
configuration of Vagrant. Please either reconfigure Vagrant to
avoid this capability or fix the issue by creating the capability.```

? This is a docker image.


Added Service Scanner signatures and new info for wordpress version

Checklist

[x] I have followed the Contributor Guidelines.
[x] I have added the relevant documentation.
[x] My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request
Added Service Scanner signatures for:
PostgreSQL
PortMap
PPTP
Rsync
RTSP
ILC 150 GSM/GPRS (Pcworx)
Added New info for Wordpress version
Your development environment

OS: Kali linux
OS Version: 2.0
Python Version: 2.7.3

Hi,
thanks for your contribution, PR merged.
best regards.


[5.9] Propagate sanitizer arguments to the clang-linker-driver invocations for dynamic libraries

Cherry-pick of: https://github.com/apple/swift-driver/pull/1325
Unless we are building a static library, we need to propagate -fsanitizer= flag to the clang linker driver, otherwise sanitizers libraries will not be found at link-time.
Resolves rdar://107733898

@swift-ci test


Constraints Generation - Assert type count is 0

Description
Currently, constraints_generation will create constraints to assert that a type counter has non-zero values for a list of types. A suggestion for an extra check is, as pointed in the PR:

When the reference profile has a count of 0 for a type, assert that the production data also has 0 of these types.
That way if the data is sparse, we don't cause failure of the constraint (e.g. mixed columns containing strings, integers, fractions, but we only get 2 rows).

[x] I have reviewed the Guidelines for Contributing and the Code of Conduct.

Adressed on https://github.com/whylabs/whylogs/pull/1211


Pipe not support Mongo cursor as Collections

Now, I am following this tutorial: http://www.angular-meteor.com/tutorials/socially/angular2/search-sort-pagination-and-reactive-vars for sorting and paginate variable users: Mongo.Cursor<any> = null;
Although I have set totalItems = 100, but your plugin display nothing

I try to debug, and found that problem in file: node_modules/ng2-pagination/dist/paginate-pipe.js
PaginatePipe.prototype.transform = function (collection, args) {
    // When an observable is passed through the AsyncPipe, it will output
    // `null` until the subscription resolves. In this case, we want to
    // use the cached data from the `state` object to prevent the NgFor
    // from flashing empty until the real values arrive.
    if (args instanceof Array) {
        // compatible with angular2 before beta16
        args = args[0];
    }
    if (!(collection instanceof Array)) {
        var _id = args.id || this.service.defaultId;
        if (this.state[_id]) {
            return this.state[_id].slice;
        }
        else {
            return collection;
        }
    }
    var serverSideMode = args.totalItems !== undefined;
    var instance = this.createInstance(collection, args);
    var id = instance.id;
    var start, end;
    var perPage = instance.itemsPerPage;
    this.service.register(instance);
    if (!serverSideMode && collection instanceof Array) {
        perPage = perPage || LARGE_NUMBER;
        start = (instance.currentPage - 1) * perPage;
        end = start + perPage;
        var isIdentical = this.stateIsIdentical(id, collection, start, end);
        if (isIdentical) {
            return this.state[id].slice;
        }
        else {
            var slice = collection.slice(start, end);
            this.saveState(id, collection, slice, start, end);
            this.service.change.emit(id);
            return slice;
        }
    }
    // save the state for server-side collection to avoid null
    // flash as new data loads.
    this.saveState(id, collection, collection, start, end);
    return collection;
};

The main proplem in this code: It make function return and not run this.service.register(instance);
    if (!(collection instanceof Array)) {
        var _id = args.id || this.service.defaultId;
        if (this.state[_id]) {
            return this.state[_id].slice;
        }
        else {
            return collection;
        }
    }

Pagination pipe simply won't know how to calculate its size when collection not is Array
Can you support more type of collections ? Or tell me the way to set totalItems of PaginationService in this case !

Hi - I'm not sure I understand something - why would you want to generate pagination links for a "collection" which is null?
Another thought - would it work to have something like this?
<tr *ngFor="let item of (users || []) | paginate: config">...</ul>

I'm closing this issue because further data was needed but was not provided. If you want to continue the thread, feel free to re-open.

@michaelbromley Ok, I have been resolved this problem by inject paginationService in construct and register  custom config


Streamline views into observed styled-components

Maybe we should continue debating https://github.com/mobxjs/mobx-state-tree/issues/988 here?
https://github.com/andraz/mobx-styled-observers
demo: https://codesandbox.io/s/github/andraz/mobx-styled-observers/tree/master/
Where does the observers HOC belong into the MobX ecosystem and how could we improve some more on this pattern?
Point of this debate is:

to join all connected concerns
action -> update data -> recalculate -> view -> component -> <Jsx onClick={action} />
on the same spot while moving all concerns which do not belong in source of truth (css styling, etc ...) to external imports
to reduce the barrier to entry for newbies (programming and designers) as much as possible
eliminate big area prone to advanced-level subtle bugs by completely automating it

passing of stores via props
injecting stores with decorators
manual wrapping JSX with observers
reducing unneeded repaints (observed component avoids parent repaint)

use case:
// Button = imported StyledComponent (component with built in `css`)
// todo.textButtonText = calculated view used to generate button's label

.views(todo =>
   observers({ // wrap all "render-views" with a custom HOC
     TextButton: () => (  // component gets render(), inject('store') etc, all built in
       <Button onClick={todo.toggle}>{todo.textButtonText}</Button>
     ),
     CountButton: () => ( // is used without props or decorators: <CountButton />
       <Button onClick={todo.toggle}>{todo.countButtonText}</Button>
     )
   })
 )

I have cleaned up some of the code to simplify the use case.
Now the view is used only as a getter to provide a cached closure for store. The HOC now lands outside of the MST files.
import button from './button.component'
const { CountButton, TextButton } = button

...

    get TextButton() {
      // wrap store into the returned styled component via closure
      return () => <TextButton todo={todo} />
    },
    get CountButton() {
      // getter can use same name as the component
      return () => <CountButton todo={todo} /> // component name
    }

/src/button.component.js
import React from 'react'
import Button from './button.style'
import { observers } from './util'

const TextButton = ({ todo }) => (
  <Button onClick={todo.toggle}>{todo.textButtonText}</Button>
)

const CountButton = ({ todo }) => (
  <Button onClick={todo.toggle}>{todo.countButtonText}</Button>
)

export default observers({ TextButton, CountButton })

Can the observers HOC get added into mobx-react ?
It is a tiny wrapper for observer, but would be useful if support for objects came already built in.

If I am correct the pattern is just const observers = comps => lodash.map(comps, observer)? To me that sounds like a syntactic sugar that is so trivial that there is no real need to make that part of the library. I prefer to keep things that can easily be expressed in user land outside the core libraries, as this kind of features, although useful, tend to blow up the api surface and thereby the cognitive load of new folks learning a library.
Beyond that I think most people stick to one component per file, in which case observers couldn't be used anyway. (If I understand correctly what you are proposing)

I developed this idea out of an experiment you made on codesandbox.io on wrapping multiple actions so they return observable components.
The gist of this new pattern is to hide all "injecting and props passing" behind closures inside actions. This causes them to simply "work" out of the box, with high performance optimal repaints, virtually "without wiring", when simply "thrown anywhere" provided by store getters. Now this is a noobproof concept... ;)
Apparently we now also both came to the same conclusion about the non-need for HOC at about the same time.
I recently deployed new version of mobx-styled-observers software design pattern without it.
Latest version simplifies code so it does not need the HOC because refactoring to using a well known folder layout + splitting components to files, greatly reduces any need to wrap multiple components. Object literals would make code less DRY, so I threw that out and with it any need for HOC.
I still need to do some more research on nesting and how to optimize the wiring in MST map() or array() to make looping through all children (to get render components from them) as neat as possible.
Tnx for support.


fix all strict-casts and argument_type_not_assignable

fix all strict-casts and argument_type_not_assignable

You need to take care that you don't change the permissions on the flutterw.sh

If you are on Windows this change sometimes happens involuntarily because Windows doesn't know filemodes, everything is by default an executable. On Windows you can either do:

Tell git to ignore file modes: git config core.filemode false
Make a file executable again: git update-index --chmod=+x <file>


Support for keyboard/mouse events

Hi, just wanted to see if this something you would consider supporting.
My use case is that I wanted to use Streamsync to build a tool kit for managing datasets. Keyboard events, such as using arrow keys or the scroll wheel to move through entries, would make this much more usable.
Thanks!

Hi Darren, thanks for suggesting this.
In the next release, events will have a big upgrade. I'll make sure to support keyboard events.


Improving on the rate-limiting algorithm

Based on the comments on https://github.com/timokau/marvin-mk2/pull/101, it sounds like currently marvin's rate-limiting algorithm is based on “involves the person and was updated in the last N days”.
This attempts to limit the amount of notification, and thus makes sense.
However, IMO it would make more sense to instead limit the amount of comments a person is expected to make: we can't count notifications from the discourse anyway, etc. So IMO instead of trying to count the notifications a person received (and necessarily failing at it because github does not say who (un)subscribed), it might make sense to instead count the messages a person sent in the last N days (plus the tags marvin explicitly sent / reviews explicitly requested by PR authors, maybe).
This way, if one is passive in nixpkgs interactions (eg. due to commenting on one or a few high-traffic nixpkgs issues/PRs), they would get new marvin requests for things that are actually actionable.
What do you think?
(Note: this is just a prospective issue, for gathering ideas. IMO the current rate-limiting algorithm is good enough to go in production, and then it'll be possible to iterate on it — assuming that the reviewers list can somehow get pushed to nixpkgs so that people could easily adjust their reviewing settings)

Yes, I agree. For reference, here is my current view on rate limiting: https://github.com/timokau/marvin-mk2/issues/34#issuecomment-714386315

it might make sense to instead count the messages a person sent in the last N days (plus the tags marvin explicitly sent / reviews explicitly requested by PR authors, maybe).

That would require state. Until now I tried to avoid that. That's not a deal-breaker, but it would complicate things. So I think the "max-in-flight" (number of assigned PRs that have been updated in the last n days, possibly ignoring updates after merge) approach might work better.


Autoescape should be TRUE by default

Refer Doc: https://docxtpl.readthedocs.io/en/latest/#escaping-1
Block of source code:
# docxtpl\template.py
{...}
def render(
        self,
        context: Dict[str, Any],
        jinja_env: Optional[Environment] = None,
        autoescape: bool = False
    ) -> None:
{...}

I suggest this because I was new to python-docx and generating reports that contained < > & characters. At first, I was miserable; the report was generated successfully but was corrupted when opened in MS Word.

@HongYang01 this code is in the docxtpl project, not this one. You'll need to take it up with them :)
https://github.com/elapouya/python-docx-template


Email Alerts for Idle Account Suspension Doesn't Work

Describe the issue:
Warning email alerts that are sent after each predefined periods to the idle users are not working. It doesn't even send an alert after the allowed idle time span is over.
How to reproduce:

Configure the idle account suspension feature as documented in https://is.docs.wso2.com/en/latest/learn/user-account-suspension.
Set the 'allowed idle time span in days' and the 'alert sending time periods in days' for idle account suspensions through the IS management console as your preference.
Create a few user accounts in the primary user store and login to those accounts for the first time and logout to track the last active time.
Keep the server up and running to trigger account suspension alerts.
At the suspension trigger time (default is 20:00:00), none of the users gets warning email alerts or account suspension email alerts.

Expected behavior:
Users that didn't login to their account should get warning email alerts and idle account suspension alerts .
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 6.0.0-m1
OS: Mac OS 12.3.1
Database: PostgreSQL 14
Userstore: Tested with both LDAP and JDBC with PostgreSQL 14

Closing this issue as the flow is working fine and this issue is not reproducible based on https://github.com/wso2/product-is/issues/13919#issuecomment-1164300550.


Add multi-language documentation

By that I mean programming language. Translations can also be useful, but I can't do most of those.

I'm probably never going to get around to it. Don't need a ticket. Contributions welcome of course.


Re-implement reload button on the Staking screen

Describe the task
Re-implement reload button on the Staking screen
Actual: Do not reload data after perform Optimize staking performance (#1461)
Expect: Allow reload data

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-pr-1611-build-1.1.1-2023-07-06T10-42-10UTC.zip
Vaidation passed


EnumerateFiles() multiple filters

Hi Ralf
First: great work! Thanks a lot.
Now my "problem":
I have to find and grab all multimedia files (mp4, mp3, jpg, png, gif, wav, ...).
With the given method EnumerateFiles() I only able to search for one extension at a time. e.g. device.GetRootDirectory().EnumerateFiles("*.jpg"). That means I have to do that more or less manually for all files I want to grab.
It would be nice, if there is a way to do it like this:
device.GetRootDirectory().EnumerateFiles("*.jpg|*.png|*.mp4")
or
device.GetRootDirectory().EnumerateFiles(new string[] { "*.jpg", "*.png" })
Thanks for your service

Hello,
Did you try that? The first version should work. The search expression is put into a regurar expression and "|" is not changed. So it should work as an OR.
Greeting
Ralf

Hi :)
Thanks for your reply.
But in the comments before "EnumerateFiles()" it says: "...but doesn't support regular expressions".
Did that mean not to pass a "Regex"-object into the method?

That's correct.
It should have the same syntax as System.IO.Directory.EnumerateFiles.
Internally, however, the searchPattern is converted to a regular expression.
        internal static string FilterToRegex(string filter)
        {
            if (filter == null || filter == "*" || filter == "*.*")
            {
                return null;
            }

            StringBuilder s = new StringBuilder(Path.GetFileName(filter));
            s.Replace(".", @"\.");
            s.Replace("+", @"\+");
            s.Replace("$", @"\$");
            s.Replace("(", @"\(");
            s.Replace(")", @"\)");
            s.Replace("[", @"\[");
            s.Replace("]", @"\]");
            s.Replace("?", ".?");
            s.Replace("*", ".*");
            return s.ToString();
        }

So "*.png|*.gif" is converted to ".*\.png|.*\.gif"

ok, thanks.
that helped!

@ITaluone Out of curiosity: did this work for you? When I try to set different filters e.g. *.png|*.gif I receive the following error from within the Path.GetFileName(filter) part in FilterToRegex:
System.ArgumentException: 'Illegal characters in path.'
Without this command it still seems to work just fine?

Hi,
this is weird... the last time i tried it with the code above, this solution worked.
Now I get this same exception


Fix formatting token on Python 2.7

Pull Request Checklist
Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:

[x] 🧪 Added tests for changed code.
[x] 🛠 All tests pass when run locally (run tox or make test to find out!).
[x] 📚 Updated documentation for changed code.
[x] ⏩ Code is up-to-date with the master branch.

If you have any questions about your code changes or any of the points above, please submit your questions along with the pull request and we will try our best to help!
Description of Changes
Fix bug with Python 2.7 and the new "x" token.

Codecov Report

Merging #781 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #781   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines         1695      1680   -15
  Branches       289       289
=========================================
- Hits          1695      1680   -15

Impacted Files
Coverage Δ

arrow/formatter.py
100.00% <100.00%> (ø)

arrow/arrow.py
100.00% <0.00%> (ø)

arrow/parser.py
100.00% <0.00%> (ø)

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 13d4562...a88b745. Read the comment docs.


Tag formatting issue

YOS NEW UPDATE
The tags (background + font) don't seem to format properly in reading view:

Looking forward to font palettes!

Hi! i personally like minimal tags in reading mode, but i can see how that would have been confusing :) i went ahead and added a toggle in Style Settings for minimal tags! i also added a border to tags, but i'm not sold on it; let me know what you think.

Sorry I typed up a response and never hit send 🙈 I wanted to say that there should be a version live that splits the difference; rounder corners, thicc font, and no border :) if anyone wants the old background-less version, there should be a style settings toggle 💖

Looks fab!! dig the new tags :))


sometimes stucks at cloning if --repo-url passed

Describe the bug
I am not sure whether this is a bug or retry mechanism. In some cases or situations, gitleaks stuck at cloning... stage.
To Reproduce
Steps to reproduce the behavior:
$ gitleaks --report /tmp/report.json --repo-url GITHUB_REPO --leaks-exit-code 0 --no-git --access-token=ACCESS_TOKEN --clone-path=/tmp/foo --config-path /tmp/gitleaks.toml --debug --verbose

Expected behavior
It should not stuck at cloning stage?
I manually ran the $ git clone ... command to check if there are really any problem related to repo, but it cloned almost in a few seconds, whereas gitleaks awaits almost 5-6 mins in cloning stage.
Screenshots

Basic Info (please complete the following information):

OS: macOS 11
Gitleaks Version: 7.6.1

Additional context
Add any other context about the problem here.
Any ideas? Thanks! 🙏
cc @zricethezav

Gitleaks has been updated to v8.0.0 and this issue will not be fixed moving forward. Sorry for the inconvenience, feel free to fork older versions introduce a fix in your fork.
Gitleaks v8.0.0 does not support cloning anymore

I think the fact that gitleaks does not support cloning anymore and the rational behind it should be described in the Readme.
It is really confusing to figure out that quite recent documentation of the usage of this tool e.g.
https://computingforgeeks.com/gitleaks-audit-git-repos-for-secrets/
doesn't work anymore. To add to the confusion an option --version is not available anymore (see https://github.com/zricethezav/gitleaks/issues/773), so a user can only guess that the missing feature of cloning a repo  might be related to a newer version.


Scala Native and Scala 3

Hi @sideeffffect thanks for the contribution! Have you checked out https://github.com/scalameta/munit/pull/477 already?
Unless I'm missing something this is doing the same thing?

Yup, the same goal. How can I help to move it forward? Can we turn it from draft to a real PR, run tests and merge if green?

@kpodsiad is still working on it, so he can answer better, but I see there's only a failing test there so maybe there's not too much work left?

curious about the status of this. do we need to find a new volunteer to finish it?

I think this PR is basically deprecated by https://github.com/scalameta/munit/pull/477


Demo, error and aca-py

Hello, first I would like to know if there is any document / video of the project execution, demonstrating how to execute, establish a connection and receive credentials using the test networks indicated in the readme (Indicio TestNet / Sovrin StagingNet)?
Second, I am trying to execute the project, however I am having problems in responding to the AFJ mediator, as in the image below.

The ngrok address shown is correct in relation to that displayed by the AFJ when I run the mediator.
Thirdly, is it possible to carry out the project with aca-py as the issuing agent? before the error pointed out above, I couldn't connect to aca-py!
Finally, congratulations on the project!

Hello @dukmarques! I'm glad you've been able to try the project out!

As of right now the best documentation is on the project readme, but I'd be interested in creating a bit more of a walkthrough document to help on our documentation efforts. I'd also be happy to help answer questions/clarifying things here or on Rocketchat! We could also do a demo/walkthrough on the next Aries Bifold User Group call.

Since AFJ has had a bunch of updates recently, I am wondering if AFJ @ main for running a mediator is incompatible with where the Bifold project is right now. I've asked @TheTreek to investigate and see if he can replicate your issue. If my thoughts are correct then we just need to update our mediation instructions to tie it to the appropriate version.

And yes! This will work with ACA-Py as the issuer. I've primarily tested via the Aries Toolbox, which uses ACA-Py for its functionality. Given the above error it's more likely an issue with mediation vs ACA-Py.

Thank you! 😄

Hi @JamesKEbert , thanks for the replies!
1 - If possible, I ask that when performing the demonstration in the group call, that I can make it available afterwards, I cannot confirm my presence due to my schedule and would help me a lot if I had access to the content.
2- Okay, thank you very much, I'm waiting. A note, I believe that the mobile agent, in my execution, is able to communicate with the mediator and after a few messages he presents this error.
3 - Interesting, can you establish a connection and after issuing credentials normally? Is there a demo of the Aries Toolbox?
Thank you very much for your attention and answers!

And we definitely can make it available afterwards--we record every call actually to allow people to watch if they're unable to make it or anything. The recordings just get posted at the bottom of the meeting notes of every week.

I can confirm that my suspicion is unfortunately not the issue. Is there something in particular that triggers this error, or just after a couple of messages with the mediator it occurs? I would love to be able to recreate the issue. Also, we can  probably add additional logging around this to help give additional context.

We have done that as well. I don't know if there's a demo easily accessible on the Aries Toolbox actually. @dbluhm is the expert on the Toolbox, if you have additional insight.
AFAIK, you should be able to follow the documentation to set up ACA-Py with the Toolbox plugin here: https://github.com/Indicio-tech/aries-acapy-plugin-toolbox
And then the corresponding Toolbox UI:
https://github.com/hyperledger/aries-toolbox

Hello @JamesKEbert, sorry for the delay in answering you.
I have followed the steps present in the readme and the commands in the docs folder to upload the mediator.
After them, I realize that the mediator and the agent mobile make the connection, exchange some messages and stop, below I leave some logs that I believe can help.
I added to the gist due to the size, and added only the initial parts, where the connection is made between them, after that they just exchange messages until they stop.
Aries Mobile React Native logs: https://gist.github.com/dukmarques/67347d4f9c393da7fb5128ce96fca3f5
Mediator logs: https://gist.github.com/dukmarques/9c0839355b60e2b7d5773753250c8067
I realized this time that after presenting the error in the image that I attached above, they communicated again and then presented the error again and the communication ceased, when clicking on DISMISS, the error is presented right away.
I did not identify any error logs, either in the agent or in the mediator.

About him resuming communication with the mediator, I believe he has tried the connection again, when the cell phone locks the screen due to inactivity (with the agent app open), he ceases communications, by unblocking it, he returns to communication with the mediator for a period of time (just over 1 minute) and stops.
So I believe that in this process, when unlocking the screen, he starts communication with the mediator again, but shortly thereafter the agent is no longer able to communicate with the mediator and presents the image error.

We've had a discussion about this on the Aries Bifold WG call. ngrok has a limit on the polling interval and it may be the case that you're reaching the limit here.
We have a publicly hosted instance of an AFJ mediator at http://mediator.animo.id. Could you try to run with this and see if the problem persists?

We've had a discussion about this on the Aries Bifold WG call. ngrok has a limit on the polling interval and it may be the case that you're reaching the limit here.
We have a publicly hosted instance of an AFJ mediator at http://mediator.animo.id. Could you try to run with this url as mediatorUrl and see if the problem persists?

I have run with this url， problem as picture show， fromt the latest branch version ：

so why and how to solve ？

This is a bit unfortunate, but AFJ recently had a major overhaul to how mediation works. The public mediator hosted at mediator.animo.id has been updated since, but this repo hasn't.
So I think the solution for now will be to check out AFJ at the version before we made the changes to how mediation works: https://github.com/hyperledger/aries-framework-javascript/releases/tag/0.0.60-unstable.0

This is a bit unfortunate, but AFJ recently had a major overhaul to how mediation works. The public mediator hosted at mediator.animo.id has been updated since, but this repo hasn't.
So I think the solution for now will be to check out AFJ at the version before we made the changes to how mediation works: https://github.com/hyperledger/aries-framework-javascript/releases/tag/0.0.60-unstable.0

Thanks for your reply.  i want to  run the Aries Mobile and connect to the mediation,  which vesion is compatible， just for demo show。

As per todays Bifold WG call we're closing stale our otherwise dormant issues. Please re-open if its still relevant.


Delete detection-rules/shopify_infra_abuse.yml

Description
Seems that test-rules didn't show us the issues here, myshopify.com comes up a lot in benign email.

/mql-mimic-exempt 269531


fix: use instanceof elytra item check for compatability with modded elytras

Using .getItem() instanceof ElytraItem instead of .is(Items.ELYTRA) allows the swapping to function with modded elytra items that extend the vanilla ElytraItem class.
I also changed the Minecraft version dependency for the Fabric side to 1.20.x so that the mod can still be used with earlier 1.20 versions. I only tested that it's working on 1.20.2 and I didn't test Forge at all.

Thank you so much for the pull request!
I will update the forge mods.toml to reflect the change to the Minecraft dependency. This possibly also fixes #9.


When prehook is configured, it throws an exception when the api parsing the response from prehook lambda

There was an release last year that bump the version of aws sdk from version 2 to 3. But in version 3 the lambda response payload is now unit8array (look at the discussion here) , so we need to convert the response before doing JSON.parse()
.e.g.
import { toUtf8 } from '@aws-sdk/util-utf8-node';

const rawHookResult = JSON.parse(toUtf8(invocationResponse.Payload))

Thanks for the report! Feel free to create a PR that fixes this.


Date picker v3

reimplemented date-picker as React/Preact component based on flatpickr
Flatpickr only received modifications on the locales (Kw -> CW, Mon -> Mo, Tue -> Tu, ...)
Style is based on latest design

This is how it looks in the demo_app.js

@faboweb What do you mean with min max borders? There is nothing implemented and also not required.

@fragsalat With min/max-borders I mean defining a timeframe where you can pick a date from. Like: you can only select shipment between now and the end of the season. The older version had this implemented. I need this for the preorder recommendation frontend.

As far as I know this is not yet part of the styleguide. Can we merge this and place this as an enhancement?


Bug:  Timestamp is always '0000-00-00 00:00:00' in ci_sessions table when  $sessionDriver is set to the DatabaseHandler

Describe the bug
I am referring to CI 4.1.3
There is a bug in the system file Session\Handlers\DatabaseHandler.php in the function "public function write($sessionID, $sessionData): bool"
I added this line of code  (below) just above the 2 "return true;"  lines in this function (lines 225 and 251)
$fp = fopen('c:\temp\session.txt', 'a'); fwrite($fp,$this->db->getLastQuery()."\n");fclose($fp);
To write out the queries that are used to update the ci_sessions table when $sessionDriver is set to the DatabaseHandler.
The following is the sample SQL statements for both and INSERT and an UPDATE of ci_sessions generated by this function
//-------------------------
INSERT INTO ci_sessions (id, ip_address, timestamp, data) VALUES ('qtigi184f4bh2joblluam9t2a78sioeb', '<IP_ADDRESS>', 'now()', '__ci_last_regenerate|i:1625558640;session_id|s:32:"qtigi184f4bh2joblluam9t2a78sioeb";_ci_previous_url|s:57:"http://chios.ad.uow.edu.au/law_lip/public/index.php/login\";')
UPDATE ci_sessions SET timestamp = 'now()', data = '__ci_last_regenerate|i:1625558640;session_id|s:32:"qtigi184f4bh2joblluam9t2a78sioeb";_ci_previous_url|s:57:"http://chios.ad.uow.edu.au/law_lip/public/index.php/login\";status|s:2:\"OK\";user_id|s:2:\"la\";account_type|i:1;user_rec_id|s:1:\"1\";user_rec|s:0:\"\";menus|s:0:\"\";application_id|i:0;'
WHERE id = 'qtigi184f4bh2joblluam9t2a78sioeb'
//-----------------------------------
The issue causing  the Timestamp to be stored as '0000-00-00 00:00:00' is that the MySQL function NOW() is in quotes so it is treated as a string and timestamp is not assigned the value of NOW(), but the string 'NOW' .
This function will have to be adjusted so that NOW() is not quoted when the SQL is generated
CodeIgniter 4 version
I am referring to CI 4.1.3
Affected module(s)
Session\Handlers\DatabaseHandler.php in the function "public function write($sessionID, $sessionData): bool"
Expected behavior, and steps to reproduce if appropriate
$sessionDriver is set to the DatabaseHandler in the config file app.php
create the ci_session table as per the CI documentation
write a sample program that uses SESSIONS
All entries in the ci_sessions table - timestamp field will be '0000-00-00 00:00:00' because the SQL generated to update this field is a NOW() function in quotes.
The INSERT and UPDATE SQL code generated for SESSION control should not have quotes around the NOW() function call
Context

OS: [e.g. Windows 99] - Independent of this
Web server [e.g. Apache 1.2.3] - Independent of this
PHP version [e.g. 6.5.4] - Independent of this

Duplicate of #4807 and fixed by #4876


[Feature] Tesla P40不支持 flash_attn pipline 推理报错

Motivation
from lmdeploy import pipeline, TurbomindEngineConfig
from lmdeploy.vl import load_image

pipe = pipeline(
    '/home/workspace/model/internlm-xcomposer2d5-7b',
    backend_config=TurbomindEngineConfig(tp=4)
    ,model_name='yinhe-vl-chat'
)
image = load_image('/home/workspace/123.png')

response = pipe(('describe this image', image))
print(response.text)

我要怎样关闭它，不使用 flash_attn 加速
(.venv) root@e9c6a6e513d9:/home/workspace#  cd /home/workspace ; /usr/bin/env /home/workspace/.venv/bin/python /config/extensions/ms-python.debugpy-2024.8.0-linux-x64/bundled/libs/debugpy/adapter/../../debugpy/launcher 44773 -- /home/workspace/test.py
/home/workspace/.venv/lib/python3.10/site-packages/transformers/utils/generic.py:311: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  torch.utils._pytree._register_pytree_node(
2024-08-26 18:06:01,470 - modelscope - INFO - PyTorch version 2.2.1+cu118 Found.
2024-08-26 18:06:01,475 - modelscope - INFO - Loading ast index from /root/.cache/modelscope/ast_indexer
2024-08-26 18:06:01,553 - modelscope - INFO - Loading done! Current index file version is 1.15.0, with md5 678b20b607a9b7a970b699aea2b84212 and a total number of 980 components indexed
Set max length to 16384
Dummy Resized
Device does not support bfloat16. Set float16 forcefully
[WARNING] gemm_config.in is not found; using default GEMM algo
[WARNING] gemm_config.in is not found; using default GEMM algo
[WARNING] gemm_config.in is not found; using default GEMM algo
[WARNING] gemm_config.in is not found; using default GEMM algo
terminate called after throwing an instance of 'std::runtime_error'
  what():  [TM][ERROR]  Assertion fail: /lmdeploy/src/turbomind/kernels/attention/attention.cu:35

Related resources
No response
Additional context
No response

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

support sm70+ only

V100 也有这个问题，Assertion fail: /lmdeploy/src/turbomind/kernels/attention/attention.cu:35
如何解决？

which version of lmdeploy are you using?

which version of lmdeploy are you using?

Name: lmdeploy
Version: 0.5.3
Summary: A toolset for compressing, deploying and serving LLM
Home-page:
Author: OpenMMLab
Author-email<EMAIL_ADDRESS>License:
Location: /home/workspace/.venv/lib/python3.10/site-packages
Requires: accelerate, einops, fastapi, fire, mmengine-lite, numpy, nvidia-cublas-cu12, nvidia-cuda-runtime-cu12, nvidia-curand-cu12, nvidia-nccl-cu12, peft, pillow, protobuf, pydantic, pynvml, safetensors, sentencepiece, shortuuid, tiktoken, torch, torchvision, transformers, triton, uvicorn
Required-by:

flash_attn package doesn't support v100.
LMDeploy vision model inference reuses the upstream model repo's inference code.
It means if the upstream model repo uses flash-attn, LMDeploy will use it.


Use new Relay mutation API?

@Neitsch Swap migration API as described here when appropriate: https://github.com/lvarayut/relay-fullstack/issues/60 ?

Migrate once the new API is officially released 😄
On a side note, you might want to fork the project rather than simply copy pasting, since we can only merge it back in if it is forked.

@Neitsch progress copied to this repo https://github.com/Extra-lightwill/sql-demo-fork-rfs
Is this what you have in mind?


Add column 'last_login' to user model

The time of the last login is saved in the database upon successful login.

attribute is renamed and master is merged into branch

Rookie mistake. Forgot to do "git add" after renaming the file. Good that you have an sharp eye :)

Thank you!


Add configuration support for EXECUTOR_SHUTDOWN_MILLIS

PR #6822 is included.
Added few lines to connect this PR with WebServer configuration and tests to make sure it works.

Fixed main conflicts for the 2nd time.


Add support for custom functions

Hi !
It would be really useful to have the ability to define custom functions for improve the syntax.
e.g :
hello {anotherkey, functionName, arg1, arg2, arg3...}
function functionName(arg1, arg2, arg3){
   return arg1 + arg2 + arg3;
})

I'm facing this situation when i want to implement nested translation (translation that contains a link to another translation), but i don't want to rewrite the integrality of the compiler :)
Have a good day !

most templating languages will allow you to do so by executing helpers or arbitrary functions, and the result is what you pass into the message. If the value is a number and needs to be formatted, you have the custom formatters that allow you to customize some aspects of the formatting process, but I will be reluctant to add anything that is not part of ICU message format.


Make table writer less verbose in most common case.

Current coverage is 77.19% (diff: 50.00%)

Merging #76 into master will decrease coverage by 0.17%

@@             master        #76   diff @@
==========================================
  Files            24         24
  Lines           897        899     +2
  Methods           0          0
  Messages          0          0
  Branches        111        112     +1
==========================================
  Hits            694        694
- Misses          174        175     +1
- Partials         29         30     +1

Diff Coverage
File Path

•••••      50%
.../kafka/connect/bigquery/write/batch/TableWriter.java

Powered by Codecov. Last update a72c804...6d03f1a


[1.19.2] Origins GUI

So, DME affects huds in Origin's mod.
How need:
How it's look:
I guess this problem should have been addressed more to Origin's mod, but im not sure.

https://github.com/Buuz135/DarkModeEverywhere/commit/9082b06b699de342662c1f4baa63ef4721f99ca1


[UX] Add Manager / other Organisation / Link

Describe the bug
The link to add a user from another organisation is hidden behind the next input.
To Reproduce
Steps to reproduce the behavior:

Go to Organisation details
Click + in the `Managers Section

Expected behavior
No overlap
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 20.04.1 LTS (Focal Fossa)
Browser: Google Chrome
Version: 87.0.4280.88 (Official Build) (64-bit)

Additional context
None

This was fixed in https://github.com/caos/zitadel/releases/tag/v0.106.3 and should reach production end of this week


Android

Android importing example

Postponed as I don't see too much benefit at the moment, and it seems to make debugging unstable in Idea.


Add support for sphinx parallel read/write build

Add support for sphinx parallel read/write build

Thanks for the hint.
Added the needed flags for Sphinx.
Code is on main.


Add cut-n-paste-support

Apparently, if  you don't have a menu item, you can't paste using ctrl-v or cmd-v. This pull request adds the menu necessary.

Interesting, here the shortcuts work without this.
I tried this anyway, but the menu entries do not work. If I select Cut for example, nothing happens. Shortcuts keep working as usual though.
Out of curiosity, are using OSX?

Yes, selector and role work equally well on OS X. Sorry for the delay. Pushed a new commit using role instead :)


The flutter project reported an error when running in VSCode. error: Invalid Swift parseable output message (malformed JSON): `[123, 10,

Steps to reproduce
I created a Flutter project that used the pigeon plugin, but encountered an issue where UTF8 is not supported during runtime.
I  attempt
1: Clear/Users/XXX/. pub cache/hosted/pub.dev/cache
2: Clear/Users/XXXX/. cocoapods/repos
3: Clear/Users/XXX/Library/Developer/XCode/VerivedData
4：flutter  clean
Still unable to run successfully.
At the same time, other Flutter projects were running and were able to run normally.
Expected results
run successfully.

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

[✓] Flutter (Channel stable, 3.27.1, on macOS 14.6.1 23G93 darwin-arm64, locale zh-Hans-CN)
• Flutter version 3.27.1 on channel stable at /Users/gaoang/workPlace/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 17025dd882 (4 days ago), 2024-12-17 03:23:09 +0900
• Engine revision cb4b5fff73
• Dart version 3.6.0
• DevTools version 2.40.2
[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 16B40
• CocoaPods version 1.15.2
[✓] VS Code (version 1.96.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.102.0

Hi @nooto, please provide a complete minimal reproducible code sample (preferably without any third party packages) so that we can properly investigate this.
Thank you


Visual Studio Test Task: Add option to specify the Run Title

Currently, the Visual Studio Test Task generates default titles for the test runs, such as 1279:VSTest Test Run Release Mixed Platforms.
When you have multiple test runs running against the same build, it quickly becomes difficult to keep track of which which results relate to a specific run. For example, it would be easier if the test runs could be named UnitTests, IntegrationTests, and similar.
vstest.console.exe has a RunTitle parameter which can be specified as part of the /Logger switch (e.g. /Logger:trx,RunTitle=IntegrationTests), but currently the /Loggeris hard coded in theInvoke-VSTest` cmdlet.
Hence, a request to add an option to specify the Run Title in the Visual Studio Test Task.

We are looking to add this option in the next sprint or two. Will reply back once it is done
Thanks!

I believe this has been in for a while now.

Yes, thanks Jesse. You will find this setting in the Reporting Options for the tasks.

"Reporting Options for the tasks"
Can you elaborate on this please? As far as I can see I don't have fields or a section for "Reporting Options" on the Visual Studio Test task I'm using in my build.

@andydix
Are you on VSTS or using an On-Premise TFS? For On-Premise, this functionality will ship with Update2

Ah, ok. We're using on-premise. Thanks for the clarification.


Destroy doesn't work on update?

Hi!
Using gem for quite some time, but now the first time I tried to edit and update existing invoice model which has line_items associated with it, it has few and try to remove one of them.
When I hit Update invoice it doesn't work, the line item is still there.
In the server logs there is nothing about destroy with value "1"
DOM has one element for each line item:
<input autocomplete="off" type="hidden" value="false" name="invoice[line_items_attributes][0][_destroy]" id="invoice_line_items_attributes_0__destroy">
<input autocomplete="off" type="hidden" value="false" name="invoice[line_items_attributes][1][_destroy]" id="invoice_line_items_attributes_1__destroy">
but when I click remove item I can't see in the DOM hidden field for destroy with value "1" also.
invoice params has:
line_items_attributes: [:name, :unit, :tax_rate, :cost, :quantity, :amount, :_destroy, :id])
and invoice model has:
accepts_nested_attributes_for :line_items, reject_if: :all_blank, allow_destroy: true
Can you confirm that it actually working correctly?
Any hints for debugging this?
Thanks!

@czepesch It works correctly with my app and some other devs.
What version of the gem do you use?
Or did you follow this instruction:

data-nested-rondo-field-class-value is used to detect the element that needs to be removed. Its value must match an element's class name that wraps the partial. If you do not declare it, it will default remove the closest parent element.

@hungle00
I'm sorry, it was an issue on my side. It's indeed working well.
I have another stimulus controller for calculating and displaying the amount of the invoice depending on adding or removing line items which was conflicting with rondo_form controller, fixed that.
Thank you


cannot find function backup_entropy in this scope for wasm target even though source seed always provided

Apologies for asking a question by using an issue - I'm new to rust, and I wasn't sure how to deal with this myself.
I am wanting to reliably reproduce/shuffle vast quantities of data in wasm (yew-rs) by way of a seed provided by a server:
let mut rng = WyRand::new_seed(server_seed);
rng.shuffle(&renumber);

I don't think I need an additional source of entropy, since WyRand reproduces the same data for the same seed?
error[E0425]: cannot find function `backup_entropy` in this scope
  --> /home/krolaw/.cargo/registry/src/github.com-1ecc6299db9ec823/nanorand-0.6.1/src/entropy/mod.rs:51:2
   |
51 |     backup_entropy(out);
   |     ^^^^^^^^^^^^^^ not found in this scope

Cargo.toml entry:
nanorand = { version = "0.6.1", features = ["wyrand"] }

Thanks.

For 0.8, I'll likely add an entropy feature, and without it, only SeedableRng instances can be crafted.

I downgraded it to 0.5.2 and it worked for me. running in Cloudflare workers with WASM.

Hi @krolaw , how did you fix this issue?

I reimplemented WyRand that takes a seed. Code is very short, I'll post it there's interest.

@krolaw Would love to see your code and find out if it can be merged to main.

It's not really suitable to merge into main. However, it is simple enough to copy to where you want it.
pub struct WyRand {
    pub seed: u64,
}

const P0: u64 = 0xa076_1d64_78bd_642f;
const P1: u64 = 0xe703_7ed1_a0b4_28db;

impl WyRand {
    pub fn rand(&mut self) -> u64 {
        self.seed = self.seed.wrapping_add(P0);
        let r = u128::from(self.seed) * u128::from(self.seed ^ P1);
        ((r >> 64) ^ r) as u64
    }

    pub fn range(&mut self, max: u64) -> u64 {
        self.rand() % max
    }
}


Retry: Update to most recent VRF draft.

Previously reverted PR for VRF update can now be landed with updated clippy lints applied

Update to most recent VRF draft.

Apply same changes to testing ECVRF implementation.

Sorry for this! What should I have run to prevent it?

I don't think it was you actually! I think it's because Rust released a new version which came with a new Clippy instance. I prematurely removed the VRF update, but we're just checking if we want to merge it now or leave it on a "dev" branch until @kevinlewi is complete with a bunch of other code cleanups too. You're good!

I think this is ready to merge in now, as any Rust update identified problems have been resolved.


[discussion] How to choreography model's convert and extract

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
Should the extract python script be included in convert python script.
or run two pod in order, one running convert script, and then another running extract script.
@gaocegege WDYT
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

I think we can just push the converted model to model-registry.
Then model registry will file a new CRD request to K8s to create the extraction job.
WDYT

SGTM, If so, the status of convert job and extract job will manage independently.
And each job MUST to update ormbfile.yaml.

Yeah I think so.

/assign @simon-cj

I think we already solve it in model-registry.


Change regular expression in PathParms middleware to avoid replacing port in url

When PathParams middleware replaces the path params, it fails if the url contains the host with port (http://localhost: 7473/my_path/:whatever)
** (ArgumentError) argument error
    :erlang.binary_to_existing_atom("7473", :utf8)
    (stdlib) erl_eval.erl:680: :erl_eval.do_apply/6
    (stdlib) erl_eval.erl:888: :erl_eval.expr_list/6
    (stdlib) erl_eval.erl:411: :erl_eval.expr/5
    (stdlib) erl_eval.erl:273: :erl_eval.expr/5
    (elixir) lib/regex.ex:731: Regex.apply_list/5
    (elixir) lib/regex.ex:726: Regex.apply_list/5
    (elixir) lib/regex.ex:668: Regex.do_replace/4

Problem
It's unavoidable when using BaseUrl, Telemetron.Instrumenter.Tesla and PathParams. There is no order possible.
Telemetron needs BaseUrl to metric host, PathParams needs to be executed before BaseUrl because otherwise it would parse the port and Telemetron needs to be executed before PathParams because otherwise it would provoke high cardinality in metrics.
Solution
Change regular expression to avoid considering numbers

What about :my_param_123 ?

You're right. Would it be acceptable if we force to start by character?
Something like: :([a-zA-Z]{1}[\w_]+)
Otherwise we could opt by another approach:
defp build_url(url, nil), do: url

  defp build_url(url, params) do
    Enum.reduce(params, url, fn {k, v}, u ->
      String.replace(u, ":#{k}", to_string(v))
    end)
  end

Please see https://github.com/teamon/tesla/pull/270#issuecomment-447933951 on why String.replace is not a good solution.
:([a-zA-Z]{1}[\w_]+) will not match single letter param.

Please see #270 (comment) on why String.replace is not a good solution.

I see, thanks for this! Fair enough.

:([a-zA-Z]{1}[\w_]+) will not match single letter param.

Arg, right! I forgot this! So, what about :([a-zA-Z]{1}[\w_]*)?

Thanks! How about some tests? :)

Thanks! How about some tests? :)

Sure, sorry! 🤦♂
Now it's done! Let me know if you think it covers all the cases we've discussed :)

Note: After adding a few additional tests the pipeline failed because of a test not related (apparently) to my tests.

Well, after adding a last commit it passed the checks again 🤷♂

Thanks! 🙇


thiefmd: 0.2.5 -> 0.2.7

Description of changes
https://github.com/kmwallio/ThiefMD/compare/v0.2.5-stability...v0.2.7
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 235881 run on x86_64-linux 1

1 package built:

thiefmd


Question: How to disable automatic file redirect

Hello @SteamPixel
In fact what I want to do is the following:
Use you router to include my header / footer / sidebar files automatically without including them on every page. For this it is crucial that every URL request passes to the index and get the right route path that is configured.
Sadly somehow this is not working when I just remove your HTACCESS rewriteConditions.
Could you provide some explanations here?

Hi,
I hope I understand your problem :-) Could you provide a code example please?
I think you need a basic concept for themes, layouts and pages. This is not part of this router. You can see this problem in action by looking at the index.php file. There is a navi() function that gets called from every single route. So the navi will be displayed on every page.
I think this is a similar problem. Instead of calling nav(), footer(), header(), sidebar(), etc... in every callback function you should consider calling something like this:
Route::add('/',function(){   Page::render('home'); });
I have another project on GitHub I am working currently on. Checkout https://github.com/steampixel/simplePHPPages
This will give you an overview how you can render different pages with the same layout. It utilyzes the same router class. Its not very well documented at the moment. But its simple and basic. I think you will understand the source.

Hi Steampixel,
Thank you for your explanation! I will certainly check out your other project. For now, we just resolved the issue by putting all our pages in a 'view' directory, in this way the path is never triggered automatically. And by using functions with parameters we trigger the right header/footer like you did with the navi function.
We will upgrade once we have the skeleton ready, thank you again for your time to look at this issue.
Regards
Laurens


LWM2M: Fix check_authorization() for Access Control Objects [v2]

Changes from v1 (#2281):

Added error returning.

This patch fixes a bug when a given LWM2M Server who owns a given
Access Control Object Instance tries to modify it (for instance,
to add or remove an ACL for a given third-party LWM2M Server).
There was no way to modify an Access Control Object Instance, since
the check_authorization() would fail, not finding an Access Control
Object Instance related to the target instance - which should not
exist anyway. To fix it, a new condition was added in case the target
'obj_id' == 'ACCESS_CONTROL_OBJECT_ID', in which we should only look
for the target 'instance_id' itself and then inside the OWNER Resource,
not looking for ACLs or anything else.
Signed-off-by: Bruno Melo<EMAIL_ADDRESS>

+1


Review for "hidden requirements"

On the final review it seemed that there may be some "hidden requirements" buried in the various app notes.
Cases where the note says something like "if they say this, then they need to include some other SFR" but where that SFR isn't mentioned in the requirement itself, nor is it a dependency for that requirement, or linked in any other way. Most of these I think we handled, but on the final review a few showed up when looked at. These were not caught in the v1.0 review or the majority of the v2.0 review, so this is an explicit issue to do such a review for the subsequent version.

I read through most app notes (though I didn't focus on this issue specifically) and only found this:
FTP_CCMP_EXT.1: _Inclusion of this SFR requires inclusion of AES-CCM in FCS_COP.1/AEAD._
FTP_GCMP_EXT.1: `Inclusion of this SFR requires inclusion of AES-GCM in FCS_COP.1/AEAD.
This could be resolved by rewriting the SFR text of FTP_CCMP_EXT.1 (and FTP_GCMP_EXT.1) as follows:

The TSF shall implement CCMP using AES in CCM mode, as specified in FCS_COP.1/AEAD, and key size [...]

However that SFR is an extended SFR and the way it is defined right now does not allow such an insertion. Of course we could update the extended component definitions, but probably not for v2.0.