Update to latest Sparkle.framework

Word on the street is that it handles Dark Mode.
Maybe it will also fix the bug where Check for Updates is disabled unexpectedly.

@vincode-io Want to handle this one? I think the current Sparkle is just a binary downloaded from the project’s website. Presumably we can get just get a new one and swap it in.

Yeah, I’ll take a look at it.

I updated mac-candidate with Sparkle 1.21.3.
Dark mode looks good on it now.  I was able to test up to the point that it tries to unarchive the app and fails.  This was because my dev cert didn't match the downloaded app signed with your cert.  This is how it should be, so I'm pretty confident updating still works.


QR code generator with a better UI

QR code generator
(Write project title over here)
Project Description
Generates QR code
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[] Html
[] CSS
[] JavaScript
[] ReactJS

https://user-images.githubusercontent.com/84259582/163225420-d47d7c61-ae37-4dad-846a-35baf6982cbd.mp4

/assign


N30: godot-rust

Part of #912

Added showcase image (44.4 kB) and moved description to Engine Updates category, as discussed on Discord.


JBDS-4299 Run detection on confirmation page after window gets focus

Codecov Report

Merging #603 into master will increase coverage by 0.11%.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##           master     #603      +/-   ##
==========================================
+ Coverage   85.64%   85.75%   +0.11%
==========================================
  Files          25       25
  Lines        2361     2514     +153
  Branches      206      207       +1
==========================================
+ Hits         2022     2156     +134
- Misses        339      358      +19

Impacted Files
Coverage Δ

browser/pages/confirm/controller.js
84.9% <82.35%> (+1.81%)
:white_check_mark:

browser/services/platform.js
86.71% <0%> (-1.75%)
:x:

browser/directives/breadcrumb.js
10% <0%> (-1.12%)
:x:

browser/services/request.js
20% <0%> (-0.84%)
:x:

browser/model/virtualbox.js
71.22% <0%> (-0.14%)
:x:

browser/pages/start/controller.js
44.79% <0%> (-0.04%)
:x:

browser/services/data.js
100% <0%> (ø)
:white_check_mark:

browser/model/cdk.js
100% <0%> (ø)
:white_check_mark:

browser/model/jdk-install.js
100% <0%> (ø)
:white_check_mark:

browser/model/devstudio-autoinstall.js
100% <0%> (ø)
:white_check_mark:

... and 14 more

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 eaa68e0...e4367b9. Read the comment docs.


moving back 41 commits from master

moving back 41 commits from master

Pull Request Test Coverage Report for Build 6581

175 of 238 (73.53%)  changed or added relevant lines in 31 files are covered.
43 unchanged lines in 17 files lost coverage.
Overall coverage decreased (-0.3%) to 45.074%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-common/src/main/java/azkaban/executor/ExecutionControllerUtils.java
4
5
80.0%

azkaban-common/src/main/java/azkaban/executor/ExecutionFlowDao.java
8
9
88.89%

azkaban-common/src/main/java/azkaban/executor/container/watch/AzPodStatusExtractor.java
9
10
90.0%

azkaban-common/src/main/java/azkaban/trigger/builtin/SlaAlertAction.java
0
1
0.0%

azkaban-common/src/main/java/azkaban/utils/EmailMessage.java
0
1
0.0%

azkaban-exec-server/src/main/java/azkaban/container/FlowContainer.java
2
3
66.67%

azkaban-common/src/main/java/azkaban/executor/AbstractExecutorManagerAdapter.java
4
6
66.67%

azkaban-common/src/main/java/azkaban/executor/JdbcExecutorLoader.java
1
3
33.33%

azkaban-common/src/main/java/azkaban/executor/container/ContainerCleanupManager.java
11
13
84.62%

azkaban-common/src/main/java/azkaban/imagemgmt/daos/ImageVersionDaoImpl.java
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

az-core/src/main/java/azkaban/Constants.java
1
21.43%

azkaban-common/src/main/java/azkaban/executor/AbstractExecutorManagerAdapter.java
1
59.53%

azkaban-common/src/main/java/azkaban/executor/container/KubernetesContainerizedImpl.java
1
61.68%

azkaban-common/src/main/java/azkaban/executor/ExecutionFlowDao.java
1
68.62%

azkaban-common/src/main/java/azkaban/metrics/DummyContainerizationMetricsImpl.java
1
21.05%

azkaban-common/src/main/java/azkaban/sla/SlaOption.java
1
81.29%

azkaban-exec-server/src/main/java/azkaban/execapp/FlowRunnerManager.java
1
16.67%

azkaban-web-server/src/main/java/azkaban/webapp/AzkabanWebServer.java
1
26.27%

azkaban-common/src/main/java/azkaban/executor/container/watch/FlowStatusManagerListener.java
2
75.32%

azkaban-common/src/main/java/azkaban/executor/ExecutorManagerException.java
2
83.33%

Totals

Change from base Build 6554:
-0.3%

Covered Lines:
17652

Relevant Lines:
39162

💛  - Coveralls

Closing as this was a debugging PR for a build error.


Add support for getting timestamp field out of nested structures

Hi,
with this PR support for timestamp fields in nested JSON structures is added.
This affects JsonMessageParser and DateMessageParser.

Simple question; is there any reason why NOT to always use . (dot) as separator ? I feel like this is kind of useless to leave the choice to the user to decide the separator for the configuration.

The reason for handling the separator was to also allow for non-nested keys that include . (dot).
{"foo.timestamp": "1405911096123", "bar.data":"test"}
Granted that's not a use case we have, but others might.

After you make the changes to the file, also make sure to use 'git squash' to squash them into one commit.

Just added a fix: String.split() expects a regex, so the separator needed to be escaped via Pattern.quote().
Anything else I should do here?

Looks fine, one small comment on when to initializing a few private variables.

Splitting of the timestamp field has been moved to the constructor of MessageParser with the implication, that the properties "timestamp.name" and "timestamp.name.separator" need to be accessed even if MessageOffsetParser is used. Exception handling also added to the initializer.

The exception handling in the constructor is questionable, I think it's probably better to have it fail (throw up that RuntimeException) if the configuration is not right rather than try to continue.  People usually don't notice a warning message up in the beginning of the log.
Also github says: 'This branch has conflicts that must be resolved', can you take a look?

If the exception is not handled, then

the test testCreateMessageParser in ReflectionUnitTest.java will fail because the empty mSecorConfig object does not contain the timestamp and separator fields which are then required during instantiation of every MessageParser subclass (here: OffsetMessageParser). This could be solved by adding empty fields to the test mSecorConfig object.
when using secor with MessageOffsetParser, an exception would be thrown if the config fields "timestamp.name" and "timestamp.name.separator" are not present (i.e. commented out) in the config. The point of MessageOffsetParser is to not read any timestamp fields inside the message, so it feels a bit off, that these options need to be present (even if not set to an actual value), I would think.
Any ideas for an elegant solution for this?

For 1) test can always be enhanced for the latest code behavior, adding empty fields in the test is fine with me;
For 2) The two new fields are present in the default properties file with empty value, so it should not get exception on that constructor.  If people comment out those two properties, they should be notified about the problem upfront and figure out a way to deal with it (either by adding those two properties or catch the exception from the constructor), I don't like the configuration issue/conflict lost in a big log file.

Adjusted the test, removed exception handling and resolved merge conflict.

Looks good, thanks for the patience on going through several iterations.


Disabling filepond encode leaves higher memory stack trace than if it was disabled

We ran into an issue where we are using filepond to upload a video without using the server capabilities of filepond. We are just adding the file in to show the user that it is there and we subsequently process the file.
We had the encode plugin turned on for encoding images but noticed a large memory spike when trying to add a video to filepond. Once we determined that we didn't need to use encode for video, we turned the allowFileEncode flag to false in the options which greatly decreased the memory usage.
That being said, in our testing we removed the encode plugin and saw that the memory usage was much lower again than if the allowFileEncode flag is set to false. Both of these tests were done adding a 500MB file to filepond.
With allowFileEncode flag set to false:

With fileEncode plugin removed:

Would you have any insight into why the memory usage is so different for when the encode plugin is in the main file but with the flag disabled and when the plugin isn't there at all?
Any help on this would be greatly appreciated!

Hi, just published version 2.1.3 which should address this issue.
Please note that encoding large files will indeed take up considerable amounts of memory.

Great, this fix worked perfectly! Thanks for the speedy turnaround on this!


Feature esc overlay disable enable

Example
 $('.open-paymentprogress-modal').leanModal({
        disableCloseOnOverlayClick: true,
        disableCloseOnEscape: true,
        closeButton: '.close-paymentprogress-modal'
});

Can you name a browser I can test in that that HREF fix affects?

Can you run this rebase command a delete the first commit so we have one, nice clean commit?
git rebase -i HEAD~2
    # pick 5d5c09c ... # will be deleted
    pick bbbe14d ... # will be preserved

Pushed as requested !


ImportError

Hi, if I use bing.py and downloader.py outside the package they work perfectly. But if I import as it is in the instructions it generates ImportError: cannot import name downloader from partially initialized module bing_image_downloader.

Hi, Can you please post how to reproduce the error.

This was done today on a PC where I did not have the module installed previously, then I will test again on the machine where I had tested it before.

It seems pip and python binary are from different distribution.  Can you please post the output of  python -m pip freeze


Incorrect usage stats when returning from app

When returning to the home screen from an app the usage will be very incorrect until the next update. I'm guessing that activity_paused hasn't been called yet and therefore it counts the usage for that app until midnight.

solved in #95


Recuirements for submission

From canvas submission page:
Please submit:

Name of the group participants and Canvas IDs.  It is important to confirm who worked in the group so the credit does not go to somebody else
Link to your GitHub repository. All group members must commit, in reasonable proportion. The repository must be accessible by the course TAs
README file explaining how to setup (we will NOT setup the project in most cases since we use the deployed version, see 4, but please document this briefly)
Link to running version of the web app (deployed). Please make sure that you do not deploy minified code <- click link for details
Text files (gladly with pictures) on documented user feedback and how it was acted upon
Other comments you wish to add

For us to submit:

Elin Blomquist, 123785<EMAIL_ADDRESS>Jonathan Erikson, 95347,
William Redgård, 141971,
Zakaria Sabbagh, 77092<EMAIL_ADDRESS>

https://github.com/zsabbagh/popify/

See README.md in repository.

https://popify.se

See userEvaluation1.md and userEvaluation2.md in repository.

For access, please send the email used for a spotify account, to any of us, and we will set it up.

Tog namn och canvasID från groups på canvas, stämmer dem? :)


Feat: Automated Greeting Workflow

Describe the feature
I propose to create a GitHub workflow named "Greetings". This workflow will automatically greet users who create new issues or pull requests in this repository.  The Greeting will look like "Hi there! Thanks for opening this issue. We appreciate your contribution to this open-source project. We aim to respond or assign your issue as soon as possible."
Please assign this issue to me.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

you can start

I would like to work on this . Please could you assign me this issue under GSSOC'24

Yes, you can do that.
On Wed, May 15, 2024, 7:21 PM KALYAN CHAKRABORTY @.***>
wrote:

I would like to work on this . Please could you assign me this issue under
GSSOC'24
—
Reply to this email directly, view it on GitHub
https://github.com/SiddharthBahuguna/NEWS-AGGREGATOR-PROJECT/issues/29#issuecomment-2112600846,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A24XZ7PV6PXPMZ4QNCNJCHLZCNR43AVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGYYDAOBUGY
.
You are receiving this because you commented.Message ID:
@.***
com>

Yes, you can do that.
…
On Wed, May 15, 2024, 7:21 PM KALYAN CHAKRABORTY @.> wrote: I would like to work on this . Please could you assign me this issue under GSSOC'24 — Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A24XZ7PV6PXPMZ4QNCNJCHLZCNR43AVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGYYDAOBUGY . You are receiving this because you commented.Message ID: @. com>

assign me please

You have to do it yourself
On Wed, May 15, 2024, 8:57 PM KALYAN CHAKRABORTY @.***>
wrote:

Yes, you can do that.
… <#m_6074623288034151116_>
On Wed, May 15, 2024, 7:21 PM KALYAN CHAKRABORTY @.> wrote: I would
like to work on this . Please could you assign me this issue under GSSOC'24
— Reply to this email directly, view it on GitHub <#29 (comment)
https://github.com/SiddharthBahuguna/NEWS-AGGREGATOR-PROJECT/issues/29#issuecomment-2112600846>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A24XZ7PV6PXPMZ4QNCNJCHLZCNR43AVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGYYDAOBUGY
https://github.com/notifications/unsubscribe-auth/A24XZ7PV6PXPMZ4QNCNJCHLZCNR43AVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGYYDAOBUGY
. You are receiving this because you commented.Message ID: @. com>
assign me please
—
Reply to this email directly, view it on GitHub
https://github.com/SiddharthBahuguna/NEWS-AGGREGATOR-PROJECT/issues/29#issuecomment-2112856530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A24XZ7P6S4MLWO3HSUHCA23ZCN5FTAVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSHA2TMNJTGA
.
You are receiving this because you commented.Message ID:
@.***
com>

Could you please assign this issue to me ?

Task completed any other suggestions?

The way I have labelled is it correct ?
On Fri, 17 May 2024 at 23:21, Sanmarg Sandeep Paranjpe <
@.***> wrote:

Task completed any other suggestions?
—
Reply to this email directly, view it on GitHub
https://github.com/SiddharthBahuguna/NEWS-AGGREGATOR-PROJECT/issues/29#issuecomment-2118115981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A24XZ7I7KW7B4KIZWLNFU4DZCY7RHAVCNFSM6AAAAABHTBPY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJYGEYTKOJYGE
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Actually, I think it is not correct because according to the mail I received it should be as follows:

I have changed format.Please check.

Sure sir


file hash doesn't coincide with expected while downloading artifact via https

Hi all,
While I'm trying to download an artifact from local repository via https I get the following error:
[ERROR] File : 'chromedriver_linux64.zip'.
[ERROR] Expected file hash : '025a098cde0a6ad8aef53d6734979c9845bf49b5'.
[ERROR] Actual file hash   : 'fe1f6c9fb1e46f147c2ba08322f182bedc3d4103'.
[INFO] Archive file 'chromedriver_linux64.zip' is valid : false
[INFO] Problem downloading 'chromedriver_linux64.zip'...
whereas downloading this zip via http is always successful.
I tried to find similar issues related to the plugin but failed what's more in the example of the repository mapping file there is https defined. So I may conclude that it should work.
Could you please help me to figure out what could be wrong and how I can try to solve an issue?
Thank you very much indeed,
Alex

If the hash is failing it means that the file you have downloaded is not the file that the plugin was expecting to download.
You can compare the files manually by doing the following:

Download the file from https://chromedriver.storage.googleapis.com/2.29/chromedriver_linux64.zip
Open a terminal and navigate to the directory in which the file has been downloaded
Type openssl sha1 chromedriver_linux64.zip

This is the SHA1 hash of the file downloaded from google, save this we will need it later.
Next do the following:

Download the file from your local repository
In the terminal navigate to the directory in which the file has been downloaded
Type openssl sha1 chromedriver_linux64.zip

This is the SHA1 hash of the file downloaded from your local repository.  If the SHA1 hashes are different you have 2 different files.  If this is the case you will either need to update your RepositoryMap.xml to expect the hash for the file you have downloaded from your local repository, or copy the file you have downloaded from google to your local repository.

It's a magic, not less than. :)
Didn't change anything but now it works. Thank you so much and really sorry for disturbing you.


Set to use new authentication system

See https://github.com/flarum/core/pull/2059

Closed in favour of fof/oauth.


Issue with tabs on IdP/SP detail page

Hi,
I've run into this somewhat complicated UI issue a few times.
When I consume the invite to become an admin of an IdP/SP, I get redirected to the same page with #tab-admins added to the URL.
When I then do any page-modifying action on other tabs (like adding an endpoint or even trying to search for users to add as contact), the UI jumps to the admin tab - which looks very confusing.  (And makes me forget what I was about to do).
Not sure how this could be fixed - avoiding the redirect/rewrite with #tab-admins when consuming the admin invite?
Cheers,
Vlad

PS: The same issue gets triggered (i.e., URL gets '#tab-admins' added) when trying to delete an admin.


[SPARK-20989][Core] Fail to start multiple workers on one host if external shuffle service is enabled in standalone mode

What changes were proposed in this pull request?
In standalone mode, if we enable external shuffle service by setting spark.shuffle.service.enabled to true, and then we try to start multiple workers on one host(by setting SPARK_WORKER_INSTANCES=3 in spark-env.sh, and then run sbin/start-slaves.sh), we can only launch one worker on each host successfully and the rest of the workers fail to launch.
The reason is the port of external shuffle service if configed by spark.shuffle.service.port, so currently we could start no more than one external shuffle service on each host. In our case, each worker tries to start a external shuffle service, and only one of them succeeded doing this.
We should give explicit reason of failure instead of fail silently.
How was this patch tested?
Manually test by the following steps:

SET SPARK_WORKER_INSTANCES=1 in conf/spark-env.sh;
SET spark.shuffle.service.enabled to true in conf/spark-defaults.conf;
Run sbin/start-all.sh.

Before the change, you will see no error in the command line, as the following:
starting org.apache.spark.deploy.master.Master, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.master.Master-1-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-1-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-2-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-3-xxx.local.out

And you can see in the webUI that only one worker is running.
After the change, you get explicit error messages in the command line:
starting org.apache.spark.deploy.master.Master, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.master.Master-1-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-1-xxx.local.out
localhost: failed to launch: nice -n 0 /Users/xxx/workspace/spark/bin/spark-class org.apache.spark.deploy.worker.Worker --webui-port 8081 spark://xxx.local:7077
localhost:   17/06/13 23:24:53 INFO SecurityManager: Changing view acls to: xxx
localhost:   17/06/13 23:24:53 INFO SecurityManager: Changing modify acls to: xxx
localhost:   17/06/13 23:24:53 INFO SecurityManager: Changing view acls groups to:
localhost:   17/06/13 23:24:53 INFO SecurityManager: Changing modify acls groups to:
localhost:   17/06/13 23:24:53 INFO SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users  with view permissions: Set(xxx); groups with view permissions: Set(); users  with modify permissions: Set(xxx); groups with modify permissions: Set()
localhost:   17/06/13 23:24:54 INFO Utils: Successfully started service 'sparkWorker' on port 63354.
localhost:   Exception in thread "main" java.lang.IllegalArgumentException: requirement failed: Start multiple worker on one host failed because we may launch no more than one external shuffle service on each host, please set spark.shuffle.service.enabled to false or set SPARK_WORKER_INSTANCES to 1 to resolve the conflict.
localhost:   	at scala.Predef$.require(Predef.scala:224)
localhost:   	at org.apache.spark.deploy.worker.Worker$.main(Worker.scala:752)
localhost:   	at org.apache.spark.deploy.worker.Worker.main(Worker.scala)
localhost: full log in /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-1-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-2-xxx.local.out
localhost: failed to launch: nice -n 0 /Users/xxx/workspace/spark/bin/spark-class org.apache.spark.deploy.worker.Worker --webui-port 8082 spark://xxx.local:7077
localhost:   17/06/13 23:24:56 INFO SecurityManager: Changing view acls to: xxx
localhost:   17/06/13 23:24:56 INFO SecurityManager: Changing modify acls to: xxx
localhost:   17/06/13 23:24:56 INFO SecurityManager: Changing view acls groups to:
localhost:   17/06/13 23:24:56 INFO SecurityManager: Changing modify acls groups to:
localhost:   17/06/13 23:24:56 INFO SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users  with view permissions: Set(xxx); groups with view permissions: Set(); users  with modify permissions: Set(xxx); groups with modify permissions: Set()
localhost:   17/06/13 23:24:56 INFO Utils: Successfully started service 'sparkWorker' on port 63359.
localhost:   Exception in thread "main" java.lang.IllegalArgumentException: requirement failed: Start multiple worker on one host failed because we may launch no more than one external shuffle service on each host, please set spark.shuffle.service.enabled to false or set SPARK_WORKER_INSTANCES to 1 to resolve the conflict.
localhost:   	at scala.Predef$.require(Predef.scala:224)
localhost:   	at org.apache.spark.deploy.worker.Worker$.main(Worker.scala:752)
localhost:   	at org.apache.spark.deploy.worker.Worker.main(Worker.scala)
localhost: full log in /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-2-xxx.local.out
localhost: starting org.apache.spark.deploy.worker.Worker, logging to /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-3-xxx.local.out
localhost: failed to launch: nice -n 0 /Users/xxx/workspace/spark/bin/spark-class org.apache.spark.deploy.worker.Worker --webui-port 8083 spark://xxx.local:7077
localhost:   17/06/13 23:24:59 INFO SecurityManager: Changing view acls to: xxx
localhost:   17/06/13 23:24:59 INFO SecurityManager: Changing modify acls to: xxx
localhost:   17/06/13 23:24:59 INFO SecurityManager: Changing view acls groups to:
localhost:   17/06/13 23:24:59 INFO SecurityManager: Changing modify acls groups to:
localhost:   17/06/13 23:24:59 INFO SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users  with view permissions: Set(xxx); groups with view permissions: Set(); users  with modify permissions: Set(xxx); groups with modify permissions: Set()
localhost:   17/06/13 23:24:59 INFO Utils: Successfully started service 'sparkWorker' on port 63360.
localhost:   Exception in thread "main" java.lang.IllegalArgumentException: requirement failed: Start multiple worker on one host failed because we may launch no more than one external shuffle service on each host, please set spark.shuffle.service.enabled to false or set SPARK_WORKER_INSTANCES to 1 to resolve the conflict.
localhost:   	at scala.Predef$.require(Predef.scala:224)
localhost:   	at org.apache.spark.deploy.worker.Worker$.main(Worker.scala:752)
localhost:   	at org.apache.spark.deploy.worker.Worker.main(Worker.scala)
localhost: full log in /Users/xxx/workspace/spark/logs/spark-xxx-org.apache.spark.deploy.worker.Worker-3-xxx.local.out

Test build #77994 has started for PR 18290 at commit e22cd94.

Test build #77994 has finished for PR 18290 at commit e22cd94.

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

Test build #78056 has started for PR 18290 at commit e0eb92a.

Test build #78056 has finished for PR 18290 at commit e0eb92a.

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

@jiangxb1987 , I think using environment variable to check if there's number of same-node workers connected id potentially vulnerable, user can manually start worker one by one without setting this environment variables. So maybe if workers fail fast due to port collision and print out enough information, that should be enough to alarm user.

@jerryshao Thank you for your suggestion, since this PR is aimed to resolve the conflict of spark configuration, I think the user behavior you described is out of scope here.

cc @cloud-fan

LGTM, only one question: are we going to support it? To reuse the same shuffle service across workers or to allow multiple shuffle services on one host?

AFAIK there is no plan to support multiple external shuffle services on one host, and we mainly start only one worker on each host in standalone mode.

thanks, merging to master!


A question about code, please help!

I recently tried to run your code, encountered a problem, I hope I can get your help, can your code output the six dimensional coordinates of the key point or the center point of the forecast target??

I used the linemod dataset.

What're the six-dimensional coordinates mean? You can print the 3-dimensional coordinate in the camera coordinate system in the data preprocessing scripts: center point, keypoints.

What're the six-dimensional coordinates mean? You can print the 3-dimensional coordinate in the camera coordinate system in the data preprocessing scripts: center point, keypoints.

Shouldn't the result predicted by the code include the translation coordinate T and rotation coordinate R of the object?

For example, can I use your code to detect the actual coordinates of objects on the desktop, including translation coordinates (x, y, z) and rotation coordinates (Rx, Ry, Rz)?

We output the transform matrix that can transform the object from its object coordinate system to the camera coordinate system, they are at lib/utils/pvn3d_eval_utils.py(link.

Thank you very much for your help！！
------ 原始邮件 ------
@.>;
发送时间:2021年3月22日(星期一) 凌晨0:06
@.>;
<EMAIL_ADDRESS>主题:Re: [ethnhe/PVN3D] A question about code, please help! (#73)
We output the transform matrix that can transform the object from its object coordinate system to the camera coordinate system, they are at lib/utils/pvn3d_eval_utils.py(link.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[Fix-5519] The t_ds_command table and t_ds_error_command table, process_definition_id change to process_definition_code

fix  #5519
from #5498

Codecov Report

:exclamation: No coverage uploaded for pull request base (json_split_two@d1a61f1). Click here to learn what that means.
The diff coverage is n/a.

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

@@                Coverage Diff                @@
##             json_split_two    #5937   +/-   ##
=================================================
  Coverage                  ?   46.27%
  Complexity                ?     3680
=================================================
  Files                     ?      605
  Lines                     ?    24605
  Branches                  ?     2795
=================================================
  Hits                      ?    11386
  Misses                    ?    12108
  Partials                  ?     1111

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 d1a61f1...d81227f. Read the comment docs.


Os update

Update the Swift version to 5.9 and the target platform to v16.0
Deleted View+TextStyle.swift as we are supporting v16.0 and above

@marliza3sc in Github you can add full teams as reviewers 🙌 .

Any other iOS 16 bits that come to mind @3SCSamDavis ? 👀


How to make autopilot vehicle take the same route at intersection every time?

I am required to capture 15-minute simulation images using 6 RGB cameras at 24fps. Many frames (~10%) are lost if I attach all 6 RGB cameras and record from them simultaneously.
So, I spawned the autopilot vehicle at the same position each time and captured it from 2 cameras at a time. I ran the simulation thrice to get images from all 6 cameras.
Since the paths of the vehicle are non-deterministic, the 3 simulations produce different results.
I looked into set_random_device_seed using the traffic manager to solve this problem, but it didn't work.

How could we make the car take the same route in every simulation?
How to use set_random_device_seed ? What are the expected values of seed_value?

Hey @snehanjalikalamkar. What version of CARLA are you using? I don't remember if this was added before or after 0.9.13, but the traffic manager has a set_path function, that can help you do that. Here are the docs on how to do it.
Closing the issue, but feel free to reopen it if this doesn't fully solve your question


chore(main): release 0.1.7

:robot: I have created a release beep boop
0.1.7 (2024-12-31)
Features

Add ci builds (449f7f4)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.7
:sunflower:


HARP-7808 Fix renderOrder not being reset

Reset the renderOrder of objects, because they may have been previously a fallback tile.

Codecov Report

Merging #1101 into master will increase coverage by <.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #1101      +/-   ##
==========================================
+ Coverage   46.13%   46.13%   +<.01%
==========================================
  Files         238      238
  Lines       22279    22281       +2
  Branches     5013     5015       +2
==========================================
+ Hits        10278    10280       +2
  Misses      12001    12001

Impacted Files
Coverage Δ

@here/harp-mapview/lib/MapView.ts
57.32% <75%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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


New scenarios for CI/CD - wait on pin value

Please consider to add the fallowing scenarios to the test environment:

Wait on PIN value: WaitPinValue
Wait on PIN change vale: WaitPinNewValue

This will allow a wide range verification's and tests
Src:

https://github.com/wokwi/wokwi-cli/tree/main/src/scenario

Hi @urish!
I'm currently in the process of enhancing an embedded system course for undergraduate computer engineering students. My objective is to leverage your tool for assessing specific skills through a pre-lab activity. By doing so, when the students arrive in the laboratory, they will already have acquired the essential skills, such as configuring peripherals.
For example, in the context of targeting the rp2040 with SDK, I aim to evaluate whether students can:

Configure a random pin as an output/input.
Successfully set a value on, or read from, a digital pin.
Utilize delay functions effectively.
Manage basic configuration of PWM/Timer peripherals.
Configure and read from an ADC pin (WaitSerialCommand can help with this).

However, it's worth noting that this approach does not address challenges related to I2C and SPI.
An alternative, more comprehensive solution (the gold standard) would involve enabling the use of two microcontrollers in the same simulation. This way, I could create a Device Under Test (DUT) controlled by another microcontroller, allowing me to emulate a sensors (I2C, SPI, UART, etc.) and verify students implementation. This approach would not only verify configuration but also assess functionality and simulate scenarios like how students' firmware behaves when a sensor malfunctions.

Thanks for explaining, a very interesting use case!
Regarding the gold standard - have you looked into Wokwi Custom Chips?  I noted you are also teaching about FPGAs and embedded Linux. I'm also working on TinyTapeout, a service which makes custom ASICs accessible and affordable. We are now working with several universities where faculty members and students send their digital / mixed signal designs into fabrication. For example, Jason from UCSC just had 31 students submit their designs last week. If this sounds interesting to you, we can discuss further.

@urish  I just sent you a LinkedIn request. Let's try to have a meeting.

Thanks, I don't remember seeing a request from you - could you please try once more?

hello, @urish .


add loader

Short description
add loader for page navigation and refreshing the page.
Proposed changes

add loading effect for changing pages and time-consuming operation, for example, add patients, add wards, and assign patients, etc.

Side effects

Resolved issues

Fixes: #104

Also, I just noticed that I missed this in the dark mode review:
Is there a reason to create the spinner "by hand" in JS instead of just having it as a hidden element in the _raw.html file? It looks like that would be way easier

Also, I just noticed that I missed this in the dark mode review:
Is there a reason to create the spinner "by hand" in JS instead of just having it as a hidden element in the _raw.html file? It looks like that would be way easier

Good point, I move it to _raw.html.

In principal I like this idea, but I think there's an easier way.
window.addEventListener("beforeunload", () => {
    showLoader();
});

This is all you need. However, I think we could make it even nicer by only showing the spinner after a short delay:
window.addEventListener("beforeunload", () => {
    const TIMEOUT = 200;
    setTimeout(showLoader, TIMEOUT);
});

In my testing, values as low as 100 worked so that for normal page loads, the spinner was never shown, but for heavy ones (like the patient list) it still looks to the user as if the spinner is shown instantly.
You can even change the theme switch button to be a normal a element with a href to your theme switcher view, and this will also show the spinner for that!
One other thing, the spinner is a bit hard to see, could you give it a nice background?

Done


0.4.2 - Added auxiliary values tooltip on Stacked Bar Chart

Feature

Added a serie that allow user pass a ghost array with auxiliary values to render only on tooltip;
It's necessary add the array in the last position of chart data.

Boa guri


[Adaptive Cards - Show Card Wizard] - While navigating on the 'Name' edit field, screen reader is not conveying the required information along with the label.

Target Platforms
Other
SDK Version
1.6
Application Name
Adaptive Cards
Problem Description
Test Environment:
URL: https://adaptivecards.io/
OS Version: Dev (OS Build 27695.1000)
Browser Version: Version 130.0.2849.1 (Official build) dev (64-bit)
Screen Reader: Narrator
Repro steps:

Open the above URL in the edge dev browser.
Navigate to the 'Sample' option and press enter key to activate it.
Press tab key and navigate to the 'Show Card Wizard' link and press enter key to activate it.
Press tab key and navigate the card and press tab key and navigate to 'Name' button.
Press enter key to expand it.
Now press tab key and navigate to the 'First Name' edit field and observe the issue.

Actual Result:
While navigating on the 'First Name' edit field screen reader is not announcing the required information. It is just announcing as 'First Name edit has a pop up.'
Also, it is announcing irrelevant information as has pop up for the edit field.
Expected Result:
Screen Reader should announce the required information while navigating on the 'First Name' edit field also it should not announce the has a pop information for the edit field. It should announce something like 'First Name edit required.'
Note:
Same issue is repro throughout the application for the required edit field.
Same issue is repro with NVDA.
User Impact:
Users rely on the screen reader to communicate whether a field is required or optional. If this information is not conveyed, users may leave required fields blank or spend time filling out optional fields unnecessarily.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/info-and-relationships
“ Have feedback to share on Bugs ? please tag bug as “A11yRCA” and add your feedback in the comment section”
Screenshots

https://github.com/user-attachments/assets/f9af82a3-2e6b-4c06-8a73-9cd1eb1def32
Card JSON
NA

Sample Code Language
No response
Sample Code
No response

Note: Input.Text is missing the aria-required option


feat: adds derive from cel query readiness policy

Description of your changes
Adds new readiness policy called "DeriveFromCelQuery". This policy allows the use of a cel query to check if an object is ready. This is handy to check conditions on objects with non-standard methods of determining if it ready such as the argocd applictation object.
Probably closes https://github.com/crossplane-contrib/provider-kubernetes/issues/192
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
I've written several test caes that cover the usage of the new feature.

@zach-source this looks great - do you have an example of how to reference a status condition with a specific type in a CEL query?  I have been looking at CEL examples and I haven't found an equivalent syntax to the JSONPath $.status.conditions[?(@.type=="Completed")].status syntax.

@zach-source this looks great - do you have an example of how to reference a status condition with a specific type in a CEL query? I have been looking at CEL examples and I haven't found an equivalent syntax to the JSONPath $.status.conditions[?(@.type=="Completed")].status syntax.

I think I figured it out:
status.conditions.exists(w, w.type=='Foo' && w.status == 'True')

Ah yea, exactly. And you can combine them if you want multiple. I've pushed up changes to make it pass all the reviewable checks. It's been a minute since I worked on a provider and I always have a hard time getting that submodule so sync correctly.

@zach-source Please run go mod tidy and make generate and commit the results.

Should be good to go now @bobh66

Had to break up the function as gocyclo got a little angry at me.

@turkenh can you take a look at this?  It would be great to get this into the next release if possible - thanks!

Looking good! A few minor comments

Thanks for the review - I'll do a bit of work on it this weekend and push it up. Probably tomorrow.

@phisco comments have been addressed.

@phisco Added your recommended changes/nits.


Make workflows run from PRs

Closes #61
Proposed Changes

Changes files for GitHub Actions (workflows) to make actions trigger not only on pushes but also on PRs

Do not worry, next events in this pull request are to check the PR.

Well, the workflows are triggered twice because two independent events happened. What are the thought about leaving the CI only with trigger on pull_request event?
Testing could be done locally before pushing, but it is essential to show that all checks were passed at PR.

Agree. One also can create draft PR and checks can still run remotely if needed. Removing hook of push


spl_object_hash() causing recorders to produce incorrect span data

I have been seeing strange behavior where some spans are not being reported and others are getting assigned tags from the wrong places.
This can be demonstrated with:
<?php
$reporter = new Zipkin\Reporters\Http(null, ['endpoint_url' => 'http://<IP_ADDRESS>:9411/api/v2/spans']);
$traceContext = \Zipkin\Propagation\TraceContext::createAsRoot(\Zipkin\Propagation\DefaultSamplingFlags::create(true));
$currentTraceContext = \Zipkin\Propagation\CurrentTraceContext::create($traceContext);
$tracer = \Zipkin\TracingBuilder::create()
        ->havingSampler(\Zipkin\Samplers\BinarySampler::createAsAlwaysSample())
        ->havingCurrentTraceContext($currentTraceContext)
        ->havingReporter($reporter)
        ->build()->getTracer();
$rootSpan = $tracer->newChild($tracer->getCurrentSpan()->getContext());
$rootSpan->setName('outerspan');
$rootSpan->start();

for ($i = 0 ; $i < 5; $i++) {
  $childSpan = $tracer->newChild($rootSpan->getContext());
  echo('childSpan spanId: ' . $childSpan->getContext()->getSpanId() . ' spl_object_hash(): ' . spl_object_hash($childSpan->getContext()) . PHP_EOL);
  $childSpan->setName('innerspan'.$i);
  $childSpan->start();
  sleep(1);
  $childSpan->finish();

}

$rootSpan->finish();
$tracer->flush();
?>

Screen shot attached of the trace visulaized in zipkins.

php -f test.php
childSpan spanId: 2d8ca211f7ca3e00 spl_object_hash():<PHONE_NUMBER>1e629e000000003f30782e
childSpan spanId: 3c3c583894079703 spl_object_hash():<PHONE_NUMBER>1e6291000000003f30782e
childSpan spanId: 5eda3ed654de34e5 spl_object_hash():<PHONE_NUMBER>1e629e000000003f30782e
childSpan spanId: a02da70f7a84e59c spl_object_hash():<PHONE_NUMBER>1e6292000000003f30782e
childSpan spanId: 6101793fb8afd412 spl_object_hash():<PHONE_NUMBER>1e629e000000003f30782e

Notice that on lines 1, 3 and 5 the result of spl_object_hash are the same despite the fact that they are different spans. This is caused by PHP running garbage collection and destroy the spans that were created in the for loop. Per http://php.net/manual/en/function.spl-object-hash.php, "once the object is destroyed, its hash may be reused for other objects." This causes the span recorders to overrite previous data because it seems them as the same object.
My suggestion is to change \Zipkin\Recording\SpanMap->getHash() https://github.com/jcchavezs/zipkin-php/blob/master/src/Zipkin/Recording/SpanMap.php#L88 to a hash of some combination of the spanId, parentSpanId and TraceId. This should produce an equally consistent hash for a span context, without relying on spl_obhect_hash().
hash('sha256', $context->getSpanId() . $context->getParentId() . $context->getTraceId());

Hi @Aramack,
Thanks for this awesomely explained issue.
Regarding this hashing, when wrote the library the first approach was to use something similar to what you mention:
md5($context->getSpanId() . ($context->getParentId() ?: "-") . $context->getTraceId())
Because $context->getParentId() con return null.
My only question is that wether it should be that order or the other way around (traceID - parentID - spanID) but maybe it does not matter.
Either way, given that you have a been a very nice issue reporter, would you mind to create a PR fixing this and a test with it?
Again, thanks for participating!

I've been running some tests with a few different hashing algorithms.

There's a small performance hit of about 3 microseconds, but I can confirm this fixes the problem I've been seeing:

Closed by https://github.com/jcchavezs/zipkin-php/pull/78. Thanks @Aramack!


Kubernetes cluster integration documentation

This is not a regular bug issue, more like a documentation request on how to seamlessly integrate an externally deployed Portus with an existing Kubernetes cluster. You have examples on how to deploy Portus inside a cluster but not outside the cluster, and instead pointing the cluster to that same external Portus, so deployments inside the cluster would actually use Portus with some cluster-wide imagePullSecrets setting (or simply inside their YAMLs, directly). I am not sure it would be possible or easy to deploy Portus inside this very cluster and make the cluster's default imagePullSecrets use it for other deployments afterwards, that would be freaking awesome if it is doable ☺

That's some very good points. Thanks for the feedback. During this release we will focus to make Portus work seamlessly in Kubernetes clusters (through a Helm chart). Thus, this issue comes in perfect timing. Whenever I'm done with the Helm chart, I'll get back to this issue :+1:


fix Update doris zh-CN doc hit-the-rollup.md  prefix index  link address open fail

Proposed changes
update dev version zh-CN doc hit-the-rollup.md  prefix index link address

Thank you for your contribution. Please add this link to the corresponding English document.

Thank you for your contribution. Please add this link to the corresponding English document.

i hive modified the doris en doc open prefix index link address
https://github.com/apache/doris/pull/24962#issue-1914907140,Could you please take a look

Thank you for your contribution. Please add this link to the corresponding English document.

i have modified the doris en doc open prefix index link address,pr address #24962 (comment) you please take a look

This is the same problem. Can it be merged into one PR?

Thank you for your contribution. Please add this link to the corresponding English document.

i have modified the doris en doc open prefix index link address,pr address #24962 (comment) you please take a look

This is the same problem. Can it be merged into one PR?

ok. i try merged


Improve projects

Some easy but convenient features:

Open Recent sessions : List of recent session files.
Ability to launch Spyder from command with a session file as input (we can create shortcut with this session).
Save session WITHOUT quitting Spyder....
Option Save session and SPYDATA  (into one file).

It allows easy but useful improvement.

Sessions were eliminated in Spyder 3.0 and replaced by the concept of Projects.
About your points:

This is solved. Once Spyder 3.0 is released, please look in the new Projects > Recent Projects menu.
Good point, it shouldn't be that hard to implement and we can do it in Spyder 3.1
This is solved now also. You can switch projects, and the list of open files for each project is saved automatically.
This is one is really hard to solve correctly and I think we won't work on it for quite some time (maybe a year or more).

Just wondering why it's difficult of combine SPYDATA file and Session into one zip file ?
One just need to unzip it and pick up separate SPYDATA and Session file.

The problem is translating "just need to" into code, while managing errors, corner cases, keeping coherency, and so on.
And the other problem of course is that our development resources are limited and we have to prioritize issues. That's why we gladly accept PRs from any contributors, if you see what I mean ! ;)

Hello,
Am fine to help (not only on this, but other items).
But, am not a 'professionnal programmer', so setuping the dev environnment (git, commit, compil,...) is usually very painful for me, because  I dont know any of the hacks to make it runs (not interest to remember all, because have to manager a ton of other issues in mutiple other language)...
That is the reason, your work on Spyder is really positive.
If you have time to describe (step by step, with exact copy/paste), or create directly a session file
for spyder contribution, am more than happy to spend coding on it.
But, am not a "hardcore programmer", it can be at odds with most of the projects like this since those projects require people how to setup devs environnment by themselves... and manager the command lines by themselves.
Am fine to code Python and contribute, but all others are a pain for me.

@arita37, thanks for your offering. The thing is this requires quite a bit of time and effort, and knowledge of the Spyder internals, to make it work right.
Saving the consoles namespace using spydata files is not the most efficient solution. If we want something that works really well, we need to use a Python package called dill, which can save and restore a full Python interactive session.
That's why this is for the a future release (perhaps 3.2 or even 3.3 :-)


Source map causes the source map file to appear in CSS

Now I just want to generate the sourcemap file for JavaScript files, but I don't want to move the CSS file
What should I do, thank you

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.


[BUG]  ">_ Launch Kubectl" does not work on windows

Describe the bug
">_ Launch Kubectl" does not work on windows:
level=error msg="Unknown error: not support windows"
To reproduce
Steps to reproduce the behavior:

In your path, download the latest autok3s for windows 64 and rename it: autok3s

Create a cluster.

In the UI, go to http://localhost:8080/ui/cluster-explorer/core/clusters, click on ">_ Launch Kubectl", select your cluster and get:
level=error msg="Unknown error: not support windows"

OS: Windows 11
AutoK3s Version: v0.6.0

@doublefx This is a known issue as autok3s uses https://github.com/creack/pty/blob/master/start_windows.go#L18 to establish pty connections and currently cannot support windows environment.

Maybe we can support Windows through conpty.


Enable Appveyor cache on 3.2 branch.

Further to https://github.com/godotengine/godot/pull/33736#issuecomment-588156933, enabling the cache on the 3.2 branch will speed up CI checks of PRs against the 3.2 branch, which are significantly delaying other CI checks caught in the queue.

Thanks!


cannot resolve symbol FragmentIndex()

i am a little bit new at this ,the problem is when i try to instantiate a section with this line
MaterialSection section = newSection("section 1",new FragmentIndex());
there is a problem in my build.gradle?

you have to replace FragmentIndex() with your own index


Score and timer,new board button(R) on the board [Enhancement]:

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Game enhancement
The current timer and score option on the board are confusing and don't make any sense during gameplay.
we get new board every time when we click on the R on the top left the colors and board change .so i would like to change it and add clear label to it.

I want to work on this issue .so i request the mentor to assign it to me and label the issue
Describe the solution you'd like
It would be better to integrate these into the game with automatic updates to improve overall experience.
Select program in which you are contributing
GSSOC'24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

i would like to work on this

@Sadhvika1 Could you please explain more what you want to do, if possible you can also add screenshot.


Deleted the necessary comma

The stylus code:
.project-item {
  border: none;
  box-shadow: 0 1px 3px 0 #d4d4d5, 0 0 0 1px #d4d4d5;
  border-radius: 0.28571429rem;
  transition: box-shadow 0.1s ease, transform 0.1s ease, -webkit-transform 0.1s ease;
}

After format will be:
.project-item {
  border: none;
  box-shadow: 0 1px 3px 0 #d4d4d5 0 0 0 1px #d4d4d5;
  border-radius: 0.28571429rem;
  transition: box-shadow 0.1s ease transform 0.1s ease -webkit-transform 0.1s ease;
}

It's error styles

Thank you for reporting this. I will take a look ASAP.

Thank you with your work, it's useful.

Hi @hstarorg,
This has been fixed at version 0.3.3. Please restart your VS Code and update the extension.
To prevent this further issues, I have added the test case.
Cheers!

@ThisIsManta Very fast, thank you.


How to keep some sort of ordering between joined streams?

Say I have two source streams on kafka, one updating at 1 message per second, and one updating at 5 messages per second. I am using a combine_latest to join the streams. What would be the most appropriate way to not allow the streams to get more than ~10s out of sync? I effectively want to push futures back up one stream, blocking until more data arrives on the second.
Or in another scenario, suppose one of the sources dies temporarily. I really would like to be able to hault any more emits until the source comes back to life.

Is there some code which says that the source died? If so then you could maybe push all the data into a buffer while waiting for the source to come back.
Similarly with the first scenario, you could have a conditional which depending on the time out of sync either pushes data through the normal route, or to a buffer until new data comes in.

Yeah I like your second idea @CJ-Wright.  I will whip something up - I wonder whether others might find it useful enough for a PR

I've used a combination of joining nodes, filter, and pluck to essentially gate certain paths. Maybe to gates would work for pushing data directly into the processing or into a buffer.


US206196  modal web component

https://rally1.rallydev.com/#/270861059696d/detail/userstory/293332135224
https://web-dev-drupal-cdoherty.dev.a1.vary.redhat.com/en/modal-demo

@TylerMartin5 did some work in the RHElements days on a modal.  I think he was using the new-ish <dialog> element.  Just mentioning it in case there's something to be gained by looking at what Tyler started.

@TylerMartin5 did some work in the RHElements days on a modal. I think he was using the new-ish <dialog> element. Just mentioning it in case there's something to be gained by looking at what Tyler started.

Thanks for the heads up, Michael! We reviewed the dialog element and decided against it because it's not very well supported at this time: https://caniuse.com/#feat=dialog

@kylebuch8 did you back out and fork your changes?

@starryeyez024 We worked though things in a code review and didn't need to back things out so we're good.


Allow stricter typed dispatch with redux-thunk

Unfortunately I failed using ng2-redux in combination with ThunkAction from redux-thunk using typescript.
The problem is, that dispatch expects an Action but ThunkAction does not extend Action.

You're right, typescript throws error if you try using the ThunkAction
type ThunkAction<R, S, E> = (dispatch: Dispatch<S>, getState: () => S, extraArgument: E) => R;

There are ways to fix this issue, not sure which one would be the best. An easy fix that wouldn't need to add the redux-thunk dependency is make it either action | function:
dispatch = (action: A | Function): any => {

It would also work with any other library that has a custom middleware which is able to understand different types of actions rather than Redux.Action

I agree with you. But wouldn't that soften the interface contract to much?
redux-thunk redeclares the Dispatch function of redux (see: https://github.com/gaearon/redux-thunk/blob/master/index.d.ts#L8) to accept ThunkAction.
I tried to do so with NgRedux
declare module 'ng2-redux' {
    export interface NgRedux<S> {
        dispatch<R, E, A extends Action>(action: ThunkAction<R, S, E> | A): any;
    }
}

but failed with

error TS2484: Export declaration conflicts with exported declaration of 'NgRedux'

I've used ng2-redux with redux-thunk in the past.However ultimately I felt that the ActionService pattern made it redundant for my needs (see docs.
Since then I've reverted to simple actions and side-effect management using redux-observable epics.
That said, you may have a different use case than me. Note that ngRedux.dispatch<T> had a generic argument type; I was able to dispatch thunks by calling ngRedux.dispatch<any>(thunkOrAction)

If you're uncomfortable with <any> you can probably specify a more specific type at the dispatch call site.

@SethDavenport: thanks. I already had a look at the docs and sure, it make sense to go for ActionService.
I thought it will not work to override the generic argument type as it says something extending Action.

Hmm - I seem to recall it working but I haven't tried in a while.  The typings should match the official redux typings, currently:
  /**
   * Dispatches an action. It is the only way to trigger a state change.
   *
   * The `reducer` function, used to create the store, will be called with the
   * current state tree and the given `action`. Its return value will be
   * considered the **next** state of the tree, and the change listeners will
   * be notified.
   *
   * The base implementation only supports plain object actions. If you want
   * to dispatch a Promise, an Observable, a thunk, or something else, you
   * need to wrap your store creating function into the corresponding
   * middleware. For example, see the documentation for the `redux-thunk`
   * package. Even the middleware will eventually dispatch plain object
   * actions using this method.
   *
   * @param action A plain object representing “what changed”. It is a good
   *   idea to keep actions serializable so you can record and replay user
   *   sessions, or use the time travelling `redux-devtools`. An action must
   *   have a `type` property which may not be `undefined`. It is a good idea
   *   to use string constants for action types.
   *
   * @returns For convenience, the same action object you dispatched.
   *
   * Note that, if you use a custom middleware, it may wrap `dispatch()` to
   * return something else (for example, a Promise you can await).
   */
  dispatch: Dispatch<S>;

Where Dispatch<S> is defined as
/**
 * A *dispatching function* (or simply *dispatch function*) is a function that
 * accepts an action or an async action; it then may or may not dispatch one
 * or more actions to the store.
 *
 * We must distinguish between dispatching functions in general and the base
 * `dispatch` function provided by the store instance without any middleware.
 *
 * The base dispatch function *always* synchronously sends an action to the
 * store’s reducer, along with the previous state returned by the store, to
 * calculate a new state. It expects actions to be plain objects ready to be
 * consumed by the reducer.
 *
 * Middleware wraps the base dispatch function. It allows the dispatch
 * function to handle async actions in addition to actions. Middleware may
 * transform, delay, ignore, or otherwise interpret actions or async actions
 * before passing them to the next middleware.
 */
export interface Dispatch<S> {
    (action: A): A;
}

In our case, since we're exposing something that is itself a wrapper to the middleware-wrapped native dispatch, it may make sense for us to loosen the type restriction a bit.

Tried using any and it works, but then you loose the ThunkAction type, trying to use ThunkAction type it complains since you can't override the Action type:
error TS2344: Type '(dispatch: Dispatch<any>, getState: () => any, extraArgument: any) => any' does not satisfy the constraint 'Action'.
  Property 'type' is missing in type '(dispatch: Dispatch<any>, getState: () => any, extraArgument: any) => any'

Maybe making the dispatch generic argument not extends from Action?

I am investigating this lib but I currently rely heavily on Thunk ... I am wondering if Thunk is not compatible with this lib?
regards
Sean

Redux thunk works fine with ng2-redux - you just have to dispatch like this: ngRedux.dispatch<any>(thunkOrAction).
I should rename this issue - it's more about whether we can do stricter typing than that.

Oh I see, tx for the reply, I will test it out shortly.
Angular 2 Kitchen sink: http://ng2.javascriptninja.io
and source@ https://github.com/born2net/ng2Boilerplate
Regards,
Sean

I'm wondering if it can be fixed as it's been a while and seems pretty basic

This has been a low priority since we've moved away from Thunk for side-effect handling in most of our projects in favour of epics.

ok but still can we please get it fixed, as some still use thunks

So there are a few reasons thunks are a low priority.  One is that for complex apps we're using Observable epics for side effect handling.
The second, however, is that even before we moved to epics, we realized that thunks are totally unnecessary with Angular2. Think about it: the only reason you make a thunk is so you can access getState and dispatch inside your action creator.
If you use the ActionCreatorService pattern instead, you can implement almost the exact same pattern simply by injecting NgRedux with Angular2's dependency injector.
Consider a component that uses a thunk for a conditional action creator:
import { Injectable } from '@angular/core';
import { NgRedux } from 'ng2-redux';
import { IAppState } from '../store';

@Injectable()
export class CounterActions {
  static INCREMENT_COUNTER: string = 'INCREMENT_COUNTER';

  incrementIfOddUsingThunk() {
    return (getState, dispatch) => {
      const { counter } = getState();

      if (counter % 2 !== 0) {
        return { type: CounterActions.INCREMENT_COUNTER };
      }
    }
  }
}

import { Component } from '@angular/core';
import { NgRedux, select } from 'ng2-redux';
import { Observable } from 'rxjs/Observable';
import { CounterActions } from '../actions/counter.actions';
import { IAppState } from '../store';

@Component({
  selector: 'counter',
  template: `

    Clicked: {{ counter$ | async }} times
    <button (click)="onClickThunkyButton()">Increment if odd using thunk</button>

  `
})
export class Counter {
  @select('counter') counter$: Observable<number>;

  constructor(
    public actions: CounterActions,
    private ngRedux: NgRedux<IAppState>) {}

  onClickThunkyButton() {
    this.ngRedux.dispatch(
      this.actions.incrementIfOddUsingThunk());
  }
}

Now consider the same action implemented using an Action Creator Service:
import { Injectable } from '@angular/core';
import { NgRedux } from 'ng2-redux';
import { IAppState } from '../store';

@Injectable()
export class CounterActions {
  constructor(private ngRedux: NgRedux<IAppState>) {}

  static INCREMENT_COUNTER: string = 'INCREMENT_COUNTER';

  incrementIfOdd() {
    const { counter } = this.ngRedux.getState();

    if (counter % 2 !== 0) {
      this.ngRedux.dispatch({ type: CounterActions.INCREMENT_COUNTER });
    }
  }
}

import { Component } from '@angular/core';
import { select } from 'ng2-redux';
import { Observable } from 'rxjs/Observable';
import { CounterActions } from '../actions/counter.actions';

@Component({
  selector: 'counter',
  template: `

    Clicked: {{ counter$ | async }} times
    <button (click)="actions.incrementIfOdd()">Increment if odd using action creator service</button>

  `
})
export class Counter {
  @select('counter') counter$: Observable<number>;

  constructor(public actions: CounterActions) {}
}

It's the same logic, and a bit more concise.

That said, if you still really really need to continue using thunks, give<EMAIL_ADDRESS>a try and let me know.

I totally understand. however I am deeply invested in thunks and would need it to transition... plus personally I enjoy working with thunks, it seems very natural to me... tx again and will check thunkexp..
Sean

If it fixes your issue I'll roll it into 4.2.0.

@SethDavenport good job!
I can confirm no more TS errors with<EMAIL_ADDRESS>

now if we just had support for operators inside decorators I would be golden ;)

Fixed and published in 4.2.0.
The decorator thing might take a little longer. Need to spec it out carefully.

tx so much.
have you had a chance to think about allowing operators inside the selector?
regards


[GridList] Add momentum scrolling

Closes #8527

@JeromeFitz Thanks for looking at our good first issue! I might start adding a complexity label per issues.


Run the ingestor with the new libraries

Closes wellcometrust/platform#3652. For wellcometrust/platform#3628.
I suddenly realised I don’t have to do them all at once! So just the ingestor for now.

@alexwlchan it'd be great to get some transparency on to what's going on here. Fancy running me through it? I know it's a bit of a hassle, but might hopefully mean you don't have to do it next time.

Sure, let’s go through it later.


Not detecting updates

I have installed wp-update-server as well as plugin-update-checker. Everything seems to be working without any errors. However, when I click on the 'Check for updates' link under Dashboard > Installed Plugins > My Plugin it says it's up to date, even though the version number installed is 1.0.4 (the latest version of my plugin is 1.0.5, which is correctly shown in the JSON file).
I should also note that the log correctly posts each request from the WordPress installation.

There's a number of things that could cause that. To help narrow it down, please install Debug Bar and take a look at the "PUC (your-plugin-slug)" section. Do all of the settings shown there look correct? Click the "Check Now" button - does it show the correct update details?
If everything seems to be fine there, it could be a configuration issue of some kind. Please double-check the filename that you're passing to the buildUpdateChecker.

It says, "Failed to retrieve plugin info from the metadata URL." However, I can view the update JSON file directly when I put the URL in a web browser. Upon close inspection I see the checker is prefeixing the slug of my plugin with 'plugin-', so the URL is not correct.


[ASK] Using datasets with a large number of users such as Gowalla in LightGCN model, RAM gets a lot involved

I'm using google colab for training the model. I used gowalla dataset and after a few seconds after calling
model = LightGCN(hparams, data, seed=SEED)
System RAM gets full and my session terminates. I think this happens when creating adjacency matrix
what should I do to fix this?

Hi @Sepiiiideh , you have passed your data set in chunk size for training . I think, your dataset is too large. so that, processor should not be perform task at time.


[azure arm] delete VM OS disk if it is a managed disk

Description
VMs on Azure can be launched with managed or unmanaged disks.
Unmanaged disks are typically launched from specific VHDs.
Managed disks are managed by Azure and they do not have a VHD
associated that we can remove (e.g., ubuntu image offered by Azure).
Deleting a VM on the Azure Web console allows you to choose whether
you want to delete the OS disk as well, independently of this is
managed or unmanaged. The original code would allow that only for
unmanaged disks via the 'ex_destroy_vhd' parameter but, as said,
this is incomplete as VMs can be launched from Azure managed disks.
This change proposes to introduce a new parameter 'ex_destroy_os_disk'
to delete the VM OS disk if it is a managed disk. Ideally, this
parameter should default to True as (i) that is the Azure's default
option and (ii) it would align with 'ex_destroy_vhd'. However, this
might pose backward compatibility issues if libcloud starts deleting
managed OS disks by default as well. I am not sure what the libcloud
maintainers' take is in this regard but this would be my proposal.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

Codecov Report

Merging #1957 (ac9a63c) into trunk (abba8c1) will increase coverage by 0.01%.
Report is 40 commits behind head on trunk.
The diff coverage is 96.30%.

@@            Coverage Diff             @@
##            trunk    #1957      +/-   ##
==========================================
+ Coverage   83.18%   83.20%   +0.01%
==========================================
  Files         352      353       +1
  Lines       81324    81406      +82
  Branches     8579     8588       +9
==========================================
+ Hits        67647    67726      +79
- Misses      10873    10874       +1
- Partials     2804     2806       +2

Files
Coverage Δ

libcloud/test/compute/test_azure_arm.py
98.97% <100.00%> (+0.04%)
:arrow_up:

libcloud/compute/drivers/azure_arm.py
60.29% <90.00%> (+0.46%)
:arrow_up:

... and 5 files with indirect coverage changes

@vicwicker It looks like some lint checks (black) failed. Can you please also fix that when you get a chance?
I think running tox -eisort,black should to the trick.
Thanks.

Sorry for the delay. It seems I forgot to run black after addressing the first comments. I have done now so and I have also added the release notes. I created a new section 3.9.0 but I am not sure if that's what you were asking. I have also run one last test against my use-case. Thanks a lot for you help :)

@Kami It seems the validation is still failing but I am not sure if this is related to the change directly. AFAIU, the error seems to be a 404 while uploading to codecov. Please let me know if I am reading this wrong and / or my PR requires some fixing. Thanks!

@vicwicker Sorry for the delay. I believe issue is related to secrets not being passed to forked repo builds for security reasons.
I think it should pass once I merge it into trunk.

PR has been merged into trunk. Thanks again for your contribution and patience :)


task32 in pkg/ddc/alluxio/transform_init_users.go

Ⅰ. Describe what this PR does
This pr is to add "Copyright 2023 The Fluid Authors." in pkg/ddc/alluxio/transform_init_users.go for task32.
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/assign @zwwhdls

/override Code scanning results / API

/override API


Add CBOR support

This is feature request for CBOR support. Binary format is important for battery-constrained devices.

I'd love it fi someone sent a patch to add CBOR. On some projects they have found that compressed JSON used less power and network bandwidth than CBOR but totally agree with you it would be great to have CBOR.

Maybe compressed JSON can be more economic, but SenML spec specifies only CBOR as a binary solution, AFAIK. So per my understanding we can not use compressed JSON and we must have CBOR implementation.

It now supports CBOR


IGNITE-15319 Set timeouts for tests

Issue is closed as invalid, cause it's already done elsewhere.


Referring to some types defined in Bridge's System namespace breaks generated code

Some types available on Bridge are defined on this file: Stubs.cs. If they are used, the resulting JavaScript code is broken. For example:
public class App
{
    [Ready]
    public static void Main()
    {
        var x = new Type();
        var y = new Enum();
    }
}

results in
Bridge.define('LiveApp.App', {
    statics: {
        config: {
            init: function () {
                Bridge.ready(this.main);
            }
        },
        main: function () {
            var x = new System.Type();
            var y = new System.Enum();
        }
    }
});

Example in bridge Live
Maybe just not allowing to use these types (or limiting their scope) would suffice. What do you think, @vladsch?

It is fixed by adding attribute  [Function] or [Object] for these types.

I think this might be related to #262


Customize settings > Startup > Startup actions - not marked as Preview feature

As I understand, startupActions is a preview feature for 1.6 https://github.com/microsoft/terminal/issues/9101#issuecomment-777034523
However it is not described as a preview feature in the docs

In contrast with other features like 'Center on launch'.

I'm sure 1.6 will be out soon, but for now it's a bit confusing :)
startupActions Docs PR: https://github.com/MicrosoftDocs/terminal/pull/217
startupActions Terminal PR: https://github.com/microsoft/terminal/pull/8770

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

ID: 4da2bad9-1b85-abf2-db4a-1d3d5a9a75f7
Version Independent ID: a12dfff0-f5e7-606d-421d-59e01186ff89
Content: Windows Terminal Startup Settings
Content Source: TerminalDocs/customize-settings/startup.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

Lucky timing on this! ;)


AUI-2058

Hey @thiago-rocha,
Attached is an update for http://issues.liferay.com/browse/AUI-2058.
Please let me know if you have any questions. Thanks!

Just started reviewing :) :octocat: Sent from GH.

Hey @thiago-rocha,
I just force pushed up this branch with the top styles removed.
Please let me know if you have any questions. Thanks!

Just started reviewing :) :octocat: Sent from GH.

Pull request submitted to https://github.com/mairatma/alloy-ui/pull/307. See changes here. :octocat: Sent from GH.


[iOS] Use cached image

Idea Description

Currently, iOS app uses AsyncImage that isn't cached downloaded image. Please apply cache for images 🙏🏼

Reference images and links

@HironobuIga
I missed..! Thanks! Assigned :bow:

@ry-itto
No problem! Thank you :)

@ry-itto
Could I ask one question?
I encountered this warnings and error when I try to build xcodeproj.

Could not find or use auto-linked framework 'FirebaseAuth': framework 'FirebaseAuth' not found
Could not find or use auto-linked framework 'FirebaseCore': framework 'FirebaseCore' not found
Could not find or use auto-linked framework 'FirebaseRemoteConfig': framework 'FirebaseRemoteConfig' not found
Undefined symbols:

Do you know how to resolve this issue...?
I use M2 chip mac and perhaps it makes problem...?

I succeed building iOS app. So the problem is resolved now.


Add Leagues of Adventure (french) character sheet based on Carl Tsui'…

…s Hollow Earth expedition

Hi Roll20 maintaner !
This is a modified Hollow Earth expeditions character sheet for Leagues of Adventure (as they both share Ubiquity system).
Cheers !

Thank you for the system information, I'll be sure to categorize them appropriately.


Updated Module API types and endpoints

Merge activity

Apr 29, 3:38 PM EDT: @tommylau523 merged this pull request with Graphite.


Windows Containers : fix typo and dead link

Partially duplicate with #1426

@mattbriggs Can you look at those PRs?

@Styxxy Thanks for your contribution. Can you make your PR against main rather than 'live'? Thanks! Matt

changed to : https://github.com/MicrosoftDocs/Virtualization-Documentation-Private/pull/783


storage: update TSDB

This upgrade addresses creation of empty blocks causing a lot of created files for large gaps between data points (#2725).
Block directories are no longer named after their time-based sequence but have a unique ULID.
Appender references are now strings. There can no longer be collisions between newly created series across transactions.
A config flag was removed.

:+1:


Icons Stop Rendering After Restart

Describe the bug
After restarting my computer/Obsidian, my folder icons stop rendering as icons and simply show up as their icon name. To get them to render again, I have to manually choose the icon again.
To Reproduce
Steps to reproduce the behavior:

Completely quit Obsidian
Restart

Expected behavior
Icons should render after a restart.
Screenshots

I should note that it doesn't always do this every single time, and it doesn't always fail to render every icon, sometimes it's just some of them fail to render.

same here. it shows after some time. I think it's because the async loading of the plugin.
Give it a min. also, if you have a lot of plugins, consider removing plugins you don't use or at least disabling them.
(Note: a lot => more than 10)

I've added a check which preloads your added icons which should appear mostly immediately. However, it can still take some time. I'll close this issue for now. Feel free to create a new one, if needed.


No Readme.md file available

Hello.
Please without a read me, we will have no idea what this project is about.
It Would be nice to add one.

Hesitant to use this package as our build backend without any documentation. Thanks!

So to clarify is this project solely for building poetry or can it be used by third parties?
Is it planned for instance to outline what functionality is covered here and what is covered by poetry 'full'?
Cheers

@FredHappyface I have added some information in #66. This package is intended to be a build backend fully compliant with PEP 517. Some shared functionality like configuration/pyproject.toml/poetry.lock parsing will also be available here. Poetry itself will use this package for everythign that is implemented here.
Right now, we do not intend formalise a public API other than what is required for PEP 517. But we do want to make things more accessible for downstream developers so they don't have to reinvent the wheel when building poetry plugins etc.


remove unecessary  element from maven-bundle-plugin

Current coverage is 90.44%

Merging #927 into master will not affect coverage as of 01b1070

@@            master    #927   diff @@
======================================
  Files           81      81
  Stmts         8741    8741
  Branches      1731    1731
  Methods          0       0
======================================
  Hit           7906    7906
  Partial        217     217
  Missed         618     618

Review entire Coverage Diff as of 01b1070

Powered by Codecov. Updated on successful CI builds.

Thx!


Is there anyway to hooking when a DisplayObject was created?

some thing like
PIXI.DisplayObject.on("created", displayObject => {
    // so programmer can do something to patching the displayObject
})

I think this is an user-level issue.
you can customize these events like:
class myclass extends PIXI.DisplayObject {
constructor(){
super();
this.emit('created');
}
}

Closing as I think the above solution is the best shout. Since the display objects are being created by you... it's your responsibility to track them. Whether that's via a customised  class, like jcyuan suggests, or via creating the objects via util functions, which you could then tap into; your choice :)


neovim-unwrapped: Fix 'disallowedRequisites' error

Description of changes
Tested on nix 2.24pre and nixVersions.minimum (2.3.18)

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

Not sure this is the correct answer, i can't get this build to fail with CA derivations which was the original reason for the postInstall patch at all

it looks like the disallowedRequisites came from https://github.com/NixOS/nixpkgs/commit/9305ed35ec7f38c6fed16e9d6d0dcd788f007b4d
But with removing the disallowedRequisites and the postInstall
it builds fine with nix-build  --arg config '{ contentAddressedByDefault = true; }' -A neovim-unwrapped
CC: @tie you add these bits originally can you look at this?

CC: @tie you added these bits originally can you look at this?

👀
I'll try to take a look at this tomorrow.
We should probably keep removeReferences though since IIRC neovim embeds the path to the compiler used to build it.

alright if that's the case https://github.com/NixOS/nixpkgs/commit/eedc7c098d7baeb03da5e610f429704852431770 should work, i'm not sure on the exact correct nix version i believe it's 2.18 that outputChecks.<output>.disallowedReferences was added but i'm not entirely sure

Maybe we can set __structuredAttrs = true and outputChecks unconditionally (without disallowedRequisites)? 🤔
I assume this should work as expected for recent Nix versions that support structured attributes and be no-op otherwise.
Converting {,dis}allowed{References,Requisites} to outputChecks based on the the Nix version should probably be done in mkDerivation and not on per-package basis. Or ideally shouldn't be done at all, I'm not really a fan of discarding string context in these attributes as is currently implemented since it breaks with CA derivations, hence the PR you've mentioned—and removeReferences/nukeReferences should be enough to avoid unwanted references in the derivation outputs.

your suggested solution works on

2.3
2.18
2.24-pre
2.24-pre CA

Sorry


Community helm chart?

Hello, everyone!
Does anyone have the latest version of the Helm chart for KeyDB? The current chart seems to be working, but it is quite outdated. You can find the current chart here.
Based on what I saw in another task, it appears that there is no up-to-date version of the chart available.
Are there any plans to add Community charts for KeyDB?

+1 Enapter KeyDB chart is Archived on Jun 6, 2023
Maybe is good to add KeyDB Managed Helm Chart?
@JohnSully

@ksemele Bitnami Release a KeyDB chart: https://github.com/bitnami/charts/pull/29025


python38Packages.ipyvuetify: 1.7.0 -> 1.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.ipyvuetify is: "Jupyter widgets based on Vuetify UI Components."
meta.homepage for python38Packages.ipyvuetify is: "https://github.com/mariobuikhuizen/ipyvuetify"
meta.changelog for python38Packages.ipyvuetify is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

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

found 1.8.1 with grep in /nix/store/d7i042qys4diz23whz4qirslqwdji8jd-python3.8-ipyvuetify-1.8.1

found 1.8.1 in filename of file in /nix/store/d7i042qys4diz23whz4qirslqwdji8jd-python3.8-ipyvuetify-1.8.1

directory tree listing: https://gist.github.com/436d871f216b35b21be9760bfcff344b

du listing: https://gist.github.com/1bf038b0d077e1b82dedc22d8f7f7915

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

24 total rebuild path(s)

6 package rebuild(s)

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

First fifty rebuilds by attrpath
python38Packages.ipyvuetify
python38Packages.qiskit
python38Packages.qiskit-ibmq-provider
python39Packages.ipyvuetify
python39Packages.qiskit
python39Packages.qiskit-ibmq-provider

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d7i042qys4diz23whz4qirslqwdji8jd-python3.8-ipyvuetify-1.8.1 \
  --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 python38Packages.ipyvuetify https://github.com/r-ryantm/nixpkgs/archive/30f794f7892e1e99bdf9bd63eb97203fd6a0d65e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d7i042qys4diz23whz4qirslqwdji8jd-python3.8-ipyvuetify-1.8.1
ls -la /nix/store/d7i042qys4diz23whz4qirslqwdji8jd-python3.8-ipyvuetify-1.8.1/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

6 packages built:

python38Packages.ipyvuetify
python38Packages.qiskit
python38Packages.qiskit-ibmq-provider
python39Packages.ipyvuetify
python39Packages.qiskit
python39Packages.qiskit-ibmq-provider

Maintainer pings
cc @drewrisinger for testing.

Result of nixpkgs-review pr 134646 at 30f794f7 run on aarch64-linux 1

4 packages failed to build:

python38Packages.qiskit
python38Packages.qiskit-ibmq-provider
python39Packages.qiskit
python39Packages.qiskit-ibmq-provider

2 packages built successfully:

python38Packages.ipyvuetify
python39Packages.ipyvuetify

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


Is Ruby 2.4 supported?

The simplecov dep requires 2.5+
simplecov (~> 0.19.0) was resolved to 0.19.0, which depends on
  Ruby (>= 2.5.0)

Perhaps the coverage could be done and used by higher-version Rubies?

I just loosened the simplecov dependency to allow building on 2.4. That's the easiest solution.

I like this gem, by the way.


Update getTitle of speaker with deleted user

Closes #2639
Needs pr https://github.com/OpenSlides/openslides-backend/pull/1921 to be closed

I am not sure about that. It could be that the user is not available because it was not loaded. In that case this would be misleading. On the other hand I don't have a better idea on how to do that. @luisa-beerboom do you have a better idea?

Just need to make sure he is loaded then

I looked at the os-autopilot-main subscriptions. Here the subscription seems to be in place.


[BUG] Default config name is used when saving config from napari plugin

When exporting the config from the napari plugin, the default config name is used, which means it cannot be loaded until it has been renamed.
167 if config_name in self.configs and not overwrite:
    168     _msg = (
    169         f"Config '{config_name}' already exists - "
    170         "config names must be unique."
    171     )
--> 172     raise ValueError(_msg)
        _msg = "Config 'cell' already exists - config names must be unique."
    174 self.configs[config_name] = config
    176 return config_name

I think it needs to be set in the actual config.json file too, rather than just the filename:
https://github.com/quantumjot/btrack/blob/2ec77286676c5fb65458f7d4fa59beb089906a2e/btrack/config.py#L21-L27

Sorry, I still don't follow

The name attribute of the TrackerConfig needs to be updated to match the new filename, before exporting it. If it's left as cell or particle the user can't load the saved config as those already exist as default options the plugin.

Okay I get you now

just linking to a related issue (though not the same) https://github.com/quantumjot/btrack/issues/243

it would probably be good to handle config name clashes in the plugin too - someone still might save cell.json, in which case they couldn’t load the config. Maybe a dialogue could popup asking whether the current config should be overwritten, or if the name of the loaded one should be modified. Or the name could be modified automatically


Mitigated files not detected

We have run into an issue where mitigated files are not detected as such.

Scan detecting the vulnerable JAR:

# /usr/java8_64/bin/java -jar logpresso-log4j2-scan-2.3.7.jar --silent --no-symlink --scan-log4j1  /homedata/icval22
Logpresso CVE-2021-44228 Vulnerability Scanner 2.3.7 (2021-12-20)
Scanning directory: /homedata/icval22
[?] Found CVE-2021-4104  (log4j 1.2) vulnerability in /homedata/icval22/Installer.jar, log4j N/A
[*] Found CVE-2021-44228 (log4j 2.x) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 2.11.0
[?] Found CVE-2021-4104  (log4j 1.2) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 1.2.17
[...]
Scanned 841 directories and 5696 files
Found 1 vulnerable files
Found 1 potentially vulnerable files
Found 0 mitigated files
Completed in 1.45 seconds

Scan with --fix parameter:

# /usr/java8_64/bin/java -jar logpresso-log4j2-scan-2.3.7.jar --silent --no-symlink --scan-log4j1 --fix /homedata/icval22
Logpresso CVE-2021-44228 Vulnerability Scanner 2.3.7 (2021-12-20)
This command will remove JndiLookup.class from log4j2-core binaries and remove JMSAppender.class, SocketServer.class, SMTPAppender.class, SMTPAppender$1.class from log4j1-core binaries. Are you sure [y/N]? y
Scanning directory: /homedata/icval22
[?] Found CVE-2021-4104  (log4j 1.2) vulnerability in /homedata/icval22/Installer.jar, log4j N/A
[*] Found CVE-2021-44228 (log4j 2.x) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 2.11.0
[?] Found CVE-2021-4104  (log4j 1.2) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 1.2.17

Fixed: /homedata/icval22/Installer.jar
Fixed: /homedata/icval22/Usage_Measurement.jar

Scanned 841 directories and 5696 files
Found 1 vulnerable files
Found 1 potentially vulnerable files
Found 0 mitigated files
Fixed 2 vulnerable log4j2 files and potentially vulnerable log4j1 files
Completed in 9.10 seconds

Scan not able to detect the Installer.jar mitigated JAR:

# /usr/java8_64/bin/java -jar logpresso-log4j2-scan-2.3.7.jar --silent --no-symlink --scan-log4j1  /homedata/icval22
Logpresso CVE-2021-44228 Vulnerability Scanner 2.3.7 (2021-12-20)
Scanning directory: /homedata/icval22
[*] Found CVE-2021-44228 (log4j 2.x) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 2.11.0 (mitigated)
[?] Found CVE-2021-4104  (log4j 1.2) vulnerability in /homedata/icval22/Usage_Measurement.jar, log4j 1.2.17 (mitigated)

Scanned 841 directories and 5698 files
Found 0 vulnerable files
Found 0 potentially vulnerable files
Found 1 mitigated files
Completed in 1.69 seconds

In fact, this is an enhancement issue. Original JAR file doesn't have pom.properties and detected by vulnerable .class file itself. SInce vulnerable class files are removed, scanner cannot detect it any more. Nontheless, I agree that mitigated mark is expected. I will fix it in the next release.

@bef11 Would you test v2.6.2 release?


[BUG] tests generation with TDVT installed as PIP fails

Describe the bug
Packaging TDVT as a Python PIP module, installing it and then attempting to generate the tests for a connector fails; the packaging&installation logs are attached, test generation run log inlined below. This has been observed both with the master version, as well as with the 2020.1 tag of TDVT (and with both Python 3.7 and 3.8 and both with or without wheel available).
PS C:\Users\[...]\Workspace> py -3 -m tdvt.tdvt action --add_ds elastic
Make sure you have already saved the appropriate TDS files in the tds directory.
Adding a new datasource [elastic] ...
Would you like to setup a password file? (y/n) This is suitable for a single connection per tds (standard).y
Enter the datasource password:<REDACTED>
Created file: tds/elastic.password
Enter the logical config to use or type 'list' to see the options or 's' to skip selecting one now:simple_lower
Created ini file: config/elastic.ini
Modifying C:\Users\[...]\Workspace\tds\cast_calcs.elastic.tds
Modifying C:\Users\[...]\Workspace\tds\Staples.elastic.tds
Generating test files from: C:\Program Files\Python37\lib\site-packages\tdvt\logicaltests\generate\input\calcs
Writing test files to: C:\Program Files\Python37\lib\site-packages\tdvt\logicaltests\setup\calcs
Traceback (most recent call last):
  File "C:\Program Files\Python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Program Files\Python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Program Files\Python37\lib\site-packages\tdvt\tdvt.py", line 756, in <module>
    main()
  File "C:\Program Files\Python37\lib\site-packages\tdvt\tdvt.py", line 725, in main
    generate_files(ds_registry, True)
  File "C:\Program Files\Python37\lib\site-packages\tdvt\tdvt_core.py", line 604, in generate_files
    generate_logical_files(logical_input, logical_output, ds_registry, force)
  File "C:\Program Files\Python37\lib\site-packages\tdvt\config_gen\gentests.py", line 292, in generate_logical_files
    process_test_file( os.path.join(input_root, input_filename), output_dir, staples_fields, calcs_fields, ds_registry )
  File "C:\Program Files\Python37\lib\site-packages\tdvt\config_gen\gentests.py", line 127, in process_test_file
    setup_file = open( os.path.join( output_dir, 'setup.' + test_name + '.' + ds + '.xml'), 'w', encoding='utf-8' )
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Program Files\\Python37\\lib\\site-packages\\tdvt\\logicaltests\\setup\\calcs\\setup.BUGS.B1713.TESTV1.xml'

The generation works fine if run from (tdvt) source.
Desktop (please complete the following information):

OS: Windows 10
Tableau Version: 2020.2.1
SDK version: 2020.1 and master

Hm, it looks like something went wrong when packaging TDVT, the source file for setup.BUGS.B1713 is definitely in the repo. On the TDVT doc page we have instructions for using the "live" version of TDVT, where it uses the exact files in your github repo. Run the following from the top-level tdvt directory:
py -3 -m pip install -e .

tfs 1197801


fix fix BUILD_CHECK_META do not take effect during compilation

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Fix missing input

@deepchoudhery this should fix the compilation issues, thanks!

Tested in VS and CLI. Works now.


Label / RichTextLabel \n not parsed when text is set via Text/Bbcode fields in inspector

Operating system or device:
Windows 7
Issue description (what happened, and what was expected):
Whenever I add \n to Labels/RichTextLabels (the standard text line break) via the Text/Bbcode fields in the inspector, it is never parsed... I can only get it to parse the linebreak if I set/add the text via code
This is especially confusing for newer users, as after trying to add it in using these fields when experimenting and it not working, I believed there was no linebreak character support for Labels/RichTextLabels
Steps to reproduce:
Attempt to add a linebreak (\n) to a Label/RichTextLabel via the Text/Bbcode fields in the inspector

Perhaps I'm wrong, but I don't think this is a bug: you can add newlines by simply pressing the 'enter' key on the keyboard, while entering the Text/Bbcode in the inspector editor, which you can open by clicking on the "text" icon on right side of the field.

The '\n' is a GDScript language-level escape sequence for strings constants.

I can understand the enter key argument but as I've said previously, it makes it confusing for newer users, especially with RitchTextLabel as other such codes are parsed (technically \n is not bbcode but you could see how it could be confusing)

This is not a bug, I honestly wouldn't add this functionality as I think it's not worth it, and we didn't have anyone else complaining about this so far.

I'd vote for closing this as inactive and probably a wontfix, WDYT @godotengine/bugsquad?

I agree.

@bojidar-bg totally agree


Use LazyloadImagesBlankUrl gives security alert in Firefox

Hello,
Use url given in doc,  https://modpagespeed.com/doc/filter-lazyload-images#lazyload-blank-url , gives a security alert in Firefox  50.1.0 .
I have tryed with pagespeed LazyloadImagesBlankUrl "https://www.gstatic.com/psa/static/1.gif";
Then it's OK.
Maybe Pagespeed doc should alert on this.
Thanks,
Eric

You're right, there's no reason to be listing an http-served image there.  https://github.com/pagespeed/mod_pagespeed/pull/1476 switches this to https.

Thanks

Fix deployed!


Using  a controller to show tooltip

Control the display of tooltip through controller instead of calling the show method directly on the Tooltip instance.

You mean because you could access the controller more easily?

Because I want to keep the build method more readable and minimize indirect widget references.😄
For example, in my opinion, something like this
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: SuperTooltip(
		  controller: _someController, // the controller is responsable for showing the tooltip
          content: Text('test'),
        ),
      ),
    );
  }

is more readable than this
  @override
  Widget build(BuildContext context) {
    final tooltip = SuperTooltip(content: Text('test')); // The tooltip instance itself is responsible for displaying the tooltip
	return MaterialApp(
      home: Scaffold(
        body: tooltip,
      ),
    );
  }

That makes totally sense. Would you like to make a PR?

ok, i will try it.

Hi Thomas,
I'm chasing PRs and saw this one.
I began to code it but soon realized that it is not possible to do it as @yohom proposes.
Since the SuperTooltip is an overlay it can't be placed in the widget tree during building so it must forcibly be created on the fly when needed, as you do for a PageRoute or an AlertDialog.
Cheers