[SPARK-36217][SQL] Rename CustomShuffleReader and OptimizeLocalShuffleReader in AQE

What changes were proposed in this pull request?
This PR proposes to rename:

Rename *Reader/*reader to *Read/*read for rules and execution plan (user-facing doc/config name remain untouched)

*ShuffleReaderExec ->*ShuffleReadExec
isLocalReader -> isLocalRead
...

Rename CustomShuffle* prefix to AQEShuffle*
Rename OptimizeLocalShuffleReader rule to OptimizeShuffleWithLocalRead

Why are the changes needed?
There are multiple problems in the current naming:

CustomShuffle* -> AQEShuffle*
it sounds like it is a pluggable API. However, this is actually only used by AQE.

OptimizeLocalShuffleReader -> OptimizeShuffleWithLocalRead
it is the name of a rule but it can be misread as a reader, which is counterintuative

*ReaderExec -> *ReadExec
Reader execution reads a bit odd. It should better be read execution (like ScanExec, ProjectExec and FilterExec). I can't find the reason to name it with something that performs an action. See also the generated plans:
Before:
...
* HashAggregate (12)
   +- CustomShuffleReader (11)
      +- ShuffleQueryStage (10)
         +- Exchange (9)
...

After:
...
* HashAggregate (12)
   +- AQEShuffleRead (11)
      +- ShuffleQueryStage (10)
         +- Exchange (9)
..

Does this PR introduce any user-facing change?
No, internal refactoring.
How was this patch tested?
Existing unittests should cover the changes.

cc @cloud-fan and @maryannxue can you take a look please?

cc @ulysses-you too FYI

Test build #141297 has started for PR 33429 at commit f2b0bab.

let me rebase. seems like it couldn't detect my GitHub actions job.

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

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

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

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

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

BTW, looks like GA changed something .. all tests fail apparently because of OOM ..

Test build #141297 has finished for PR 33429 at commit f2b0bab.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AQEShuffleReadRule extends Rule[SparkPlan]
case class CoalesceShufflePartitions(session: SparkSession) extends AQEShuffleReadRule

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

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

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

Test build #141304 has finished for PR 33429 at commit 2840b47.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AQEShuffleReadRule extends Rule[SparkPlan]
case class CoalesceShufflePartitions(session: SparkSession) extends AQEShuffleReadRule

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

(rebased to resolve conflicts)

Test build #141335 has started for PR 33429 at commit d64ed95.

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

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

Test build #141335 has finished for PR 33429 at commit d64ed95.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AQEShuffleReadRule extends Rule[SparkPlan]
case class CoalesceShufflePartitions(session: SparkSession) extends AQEShuffleReadRule

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

I'll merge in few days if there are no more comments.

I am fine with "Rename OptimizeLocalShuffleReader rule to OptimizeShuffleWithLocalRead"
But can still use "reader" for the physical node name, i.e., AQEShuffleReaderExec ?

hmm I feel like that's inconsistent with the current SparkPlan naming rules. For example, there's no -ers. One example might be V2TableWriteExec. As far as I can tell, there's almost no plan named with actors. ScanExec, ProtectExec, ExchangeExec, AggregateExec, etc. I believe using Read is more consistent with other namimg.

Test build #141555 has started for PR 33429 at commit e0b8fa6.

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

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

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

Test build #141555 has finished for PR 33429 at commit e0b8fa6.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AQEShuffleReadRule extends Rule[SparkPlan]
case class CoalesceShufflePartitions(session: SparkSession) extends AQEShuffleReadRule

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

Will merge tomorrow if there's no more comment @maryannxue

Test build #141636 has started for PR 33429 at commit 19d6dee.

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

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

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

Test build #141636 has finished for PR 33429 at commit 19d6dee.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AQEShuffleReadRule extends Rule[SparkPlan]
case class CoalesceShufflePartitions(session: SparkSession) extends AQEShuffleReadRule

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

thanks, merging to master/3.2! (otherwise backporting AQE changes will be very hard)

Thanks Wenchen!


CCP-680: Updated snapshots of components which use the common Picker

Pull Request Test Coverage Report for Build 890

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

Totals

Change from base Build 889:
0.0%

Covered Lines:
1530

Relevant Lines:
3750

💛  - Coveralls


small crash

Hi there,
I wanted to share a small crash report I encountered. It happens when swapping an axe from the weapon slot:
Crash Report Link
Also, before anyone reports this as a false crash, I recommend updating the mod's description for version 1.21 to explicitly mention compatibility with 1.21.1. Otherwise, it might crash on startup with 1.21.1. (I manually edited it myself while testing the mod.)
By the way, will the keep inventory option for the backpack be added in a future update? I've always loved this feature and would be lovely to see it return!
Love your work, and have a great day!

Solved! The Shorthand redirects a lot of the visual code when its equipped but not much of the logical code. Better Combat redirects a lot of the logical code for what the player has in their hand and that was causing the game to crash.
Keep an eye out for 0.1-beta. I'll also add a config option to keep Back, Tool Bar, and Shorthand items on death.

Beta 0-1 is live on modrinth!


iOS build failed

SWIFT_VERSION '5.0' is unsupported, supported versions are: 3.0, 4.0, 4.2. (in target 'InputMask')

Maybe you should upgrade your Xcode to 10.2 or higher


Feature request: clone

My second (highly-anticipated!) feature request.
To be able to "clone" any part of a graph - and like my previous feature request, to be able to add a edge to it immediately.
I think it's pretty self explanatory -
Example:
addEdge(
    g.v(1),
    g.clone(g.v(1), my-clone-id),
    'ATTACH'
)

I guess the benefit of something like this instead of a Copy & Paste feature is that a Clone would occur immediately.

This is not a Gremlin request, but something that would probably go into Blueprints utilities. For example, in ElementHelper:
http://tinkerpop.com/maven2/com/tinkerpop/blueprints/blueprints/1.1/api/com/tinkerpop/blueprints/pgm/util/ElementHelper.html

Do you want me to make a feature request there?

please. better yet, make a pull request there.
On Jan 19, 2012, at 9:06 PM<EMAIL_ADDRESS>wrote:

Do you want me to make a feature request there?

Reply to this email directly or view it on GitHub:
https://github.com/tinkerpop/gremlin/issues/270#issuecomment-3578905


Added constellation data

I had a little more time to add more constellations :)

Codecov Report

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

@@           Coverage Diff            @@
##             main       #65   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        82
  Lines           ?       711
  Branches        ?         0
========================================
  Hits            ?       711
  Misses          ?         0
  Partials        ?         0

Impacted Files
Coverage Δ

qual_id/categories/constellation.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks so much for the contribution @GoldDust69 ! 🎉


add support for TLS Renegotiation

I am a developer of Azure Cognitive Services. When I sent a request using this client to our service, I encountered an error: Failure sending request: StatusCode=0 -- Original Error: Post xxx: local error: tls: no renegotiation
It seems Golang has supported renegotiation since version 1.7. (https://github.com/golang/go/issues/5742)
I have created a PR to enable renegotiation: https://github.com/Azure/go-autorest/pull/383
Please let me know if any questions.

Added in https://github.com/Azure/go-autorest/releases/tag/v11.8.0


Do not assume location property exists for list_all results for Azure.

It turns out that not necessarily every resource returned from a list_all method in the azure-armrest gem will necessarily have a location property. Specifically, network interface cards that are part of a scale set do not have this property set, and are effectively "private". These are not listed under the regular list of network interface cards that you can see in the azure portal, so skipping them matches the portal's current behavior.
So, this PR just checks to ensure that the resource has a location property first before attempting a comparison.

@djberg96 is there a specific BZ number for this one?

@blomquisg No, I just discovered it while attempting to investigate another bug. But, if a customer has scale sets in their inventory, they will hit this bug. Scale sets were only introduced in April, 2016, so this is why we hadn't encountered this particular issue before.

k, is there a test you could add for this, then?

@blomquisg I think I will need to create a scale set in one of our test groups, but I'm somewhat leary of doing that because it means using up more resources for our subscription to set that up. Which, at the moment, I don't know how to do.
I suppose what might be possible would be to fake out NetworkInterfaceService#list_all using rspec, but I will need some assistance there, as my rspec-fu is weak.

@djberg96 @blomquisg is there a BZ for this?

@chessbyte No, it was discovered in development. See above.


Undeclared dependency on cpphs

MFlow needs cpphs to compile, but that dependency is not declared in the Cabal file. This causes build failures on NixOS and probably on other distributions, too.

Hi Peter,
Do you mean the last version <IP_ADDRESS>?
It does not depend on cpphs. That dependency is in one of the examples and
cpphs is not a library but an executable preprocessor, so cabal can not
express such dependency.
It was required in some previous version of MFlow to compile the examples,
but for that reason the examples were suppressed from the installation
process in latest versions.
2015-08-29 20:01 GMT+02:00 Peter Simons<EMAIL_ADDRESS>
MFlow needs cpphs to compile, but that dependency is not declared in the
Cabal file. This causes build failures on NixOS and probably on other
distributions, too.
—
Reply to this email directly or view it on GitHub
https://github.com/agocorona/MFlow/issues/63.

--
Alberto.

Yes, I mean version <IP_ADDRESS>.

It does not depend on cpphs. That dependency is in one of the examples and cpphs is not a library but an executable preprocessor, so cabal can not express such dependency.

Huh? Of course it can! Isn't that exactly what the build-tool stanza is for?

Ok,
I mxed cpphs with monadloc_pp, which is the preprocessor that need he
examples.
How to use build_tool? I find no documentation about it . looking at
https://github.com/haskell/cabal/issues/1466 I think that it needs a line
somewhere in the cabal file like;
build_tool: cpphsProgram
But I don't know
2015-08-29 22:30 GMT+02:00 Peter Simons<EMAIL_ADDRESS>
Yes, I mean version <IP_ADDRESS>.
It does not depend on cpphs. That dependency is in one of the examples and
cpphs is not a library but an executable preprocessor, so cabal can not
express such dependency.
Huh? Of course it can! Isn't that exactly what the build-tool stanza is
for?
—
Reply to this email directly or view it on GitHub
https://github.com/agocorona/MFlow/issues/63#issuecomment-136045669.

--
Alberto.

https://www.haskell.org/cabal/users-guide/developing-packages.html describes the build-tools: stanza.

Uploaded <IP_ADDRESS>
2015-08-30 1:02 GMT+02:00 Peter Simons<EMAIL_ADDRESS>
https://www.haskell.org/cabal/users-guide/developing-packages.html
describes the build-tools: stanza.
—
Reply to this email directly or view it on GitHub
https://github.com/agocorona/MFlow/issues/63#issuecomment-136059246.

--
Alberto.

Thank you for the quick fix!

Thank you for the interest in MFlow!
2015-08-30 2:11 GMT+02:00 Peter Simons<EMAIL_ADDRESS>
Thank you for the quick fix!
—
Reply to this email directly or view it on GitHub
https://github.com/agocorona/MFlow/issues/63#issuecomment-136064958.

--
Alberto.

Why has the declaration of the build-tools: cpphs dependency vanished again from <IP_ADDRESS>'s Cabal file? Now we're back to the same build error that was already fixed before. :disappointed:

Hmm... I can't be certain, but I have never used (and even installed) cpphs on my OS X laptop, but my current MFlow projects compile just fine with stack tool.

It was due to a bad synchronization. Fixing it to drop cpphs as a
dependency. Sorry
2015-09-21 7:34 GMT+02:00, Peter Simons<EMAIL_ADDRESS>
Why has the declaration of the build-tools: cpphs dependency vanished
again from <IP_ADDRESS>'s Cabal file? Now we're back to the same build error
that was already fixed before. :disappointed:

Reply to this email directly or view it on GitHub:
https://github.com/agocorona/MFlow/issues/63#issuecomment-141880870

--
Alberto.

uploading 4.6.0
2015-09-21 11:18 GMT+02:00, Alberto G. Corona<EMAIL_ADDRESS>
It was due to a bad synchronization. Fixing it to drop cpphs as a
dependency. Sorry
2015-09-21 7:34 GMT+02:00, Peter Simons<EMAIL_ADDRESS>
Why has the declaration of the build-tools: cpphs dependency vanished
again from <IP_ADDRESS>'s Cabal file? Now we're back to the same build error
that was already fixed before. :disappointed:

Reply to this email directly or view it on GitHub:
https://github.com/agocorona/MFlow/issues/63#issuecomment-141880870

--
Alberto.

--
Alberto.


Add Java logo

Hi, I'd like to request to add the Java logo to the list :)
Thanks,

https://upload.wikimedia.org/wikipedia/commons/a/a4/Java_logo_and_wordmark.svg

Added 67287ea9e3214975c86bc0a61aaf65aeca8e8372


webapp:Need only Backup Command

az is missing in triggering already configured backup for webapp without SAS url. Currently we can trigger backup using SAS url "az webapp config backup create" or  "az webapp config backup update" but this command will also create schedule for backup which is not required to me.
I need an az command to trigger already configured webapp backup only.

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

ID: aae944b4-aa1b-4bc7-5f3c-1e645c792e86
Version Independent ID: 06e361cb-035c-0b20-1b85-6532e551a35a
Content: az webapp config backup
Content Source: latest/docs-ref-autogen/webapp/config/backup.yml
Service: app-service-web
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@Nking92 & @btardif this is a request for a new command, can you guys review & answer if this is something can & should support. Thank you.

Hi @developerprashantsinghal
I believe "az webapp config backup create" does what you would like. That command will create a backup on-demand, independent of your backup schedule. On the other hand "az webapp config backup update" is designed to configure the backup schedule, so please don't use that command for an on-demand backup.

thanks @Nking92 , @developerprashantsinghal can you try the command Nick suggested & let us know that it answers your question?

Closing this due to lack of response for 12 days

i can not use sas token due to some internal policy. For now as my webapp backup is already configured manually so i just reschedule the backup and this action also trigger backup of my webapp.
############################################################
$SchedulerDetails=az webapp config backup show --resource-group "myResourcegroup" --webapp-name "myWebapp" --subscription "mySub"
#parsing json data
$FrequencyOfBackup= $SchedulerDetails | findstr "frequencyInterval"
$Days=$FrequencyOfBackup |  ForEach-Object { $_ -replace '    ' } | ForEach-Object { $_ -replace '"' } | ForEach-Object { $_ -replace ':' } | ForEach-Object { $_ -replace ',' } | ForEach-Object { $_ -replace 'frequencyInterval ' }
Triggering backup of webapp by rescheduling
if($Days -eq "31"){
az webapp config backup update --resource-group "myResourcegroup" --webapp-name "myWebapp" --subscription "mySub" --frequency "30d" --retention "0"
echo "Backup frequency set for 30 days and indefinite retention"
}
if($Days -eq "30"){
az webapp config backup update --resource-group "myResourcegroup" --webapp-name "myWebapp" --subscription "mySub" --frequency "31d" --retention "0"
echo "Backup frequency set for 31 days and indefinite retention"
}
else
{
az webapp config backup update --resource-group "myResourcegroup" --webapp-name "myWebapp" --subscription "mySub" --frequency "30d" --retention "0"
echo "Backup frequency set for 30 days and indefinite retention under else condition"
}

Has anyone encountered "sizeInBytes": 0 issue while executing the az webapp config backup create ? which causes the backup to fail


Synthetic Data Generation With LLMs

#464 Here's a draft of how synthetic data generation might look like in autolabel. This PR introduces the autolabel.DatasetGenerator class which has the method generate allowing the user to specify the number of rows they want to generate and creating a csv that has that data.
from autolabel import DatasetGenerator

data_gen = DatasetGenerator(config)
df = data_gen.generate(25)

Here's the prompt generated by running this code on the civil comments few shot non chain-of-though example config:
You are an expert at generating synthetic data. You will generate a dataset that satisfies the following criteria:
1. The data should be diverse and cover a wide range of scenarios.
2. The data should be as realistic as possible, closely mimicking real-world data.
3. The data should not contain any sensitive or personal information.
4. The data should be generated in a csv format.

```csv
example,label
It's ridiculous that these guys are being called 'protesters'. Being armed is a threat of violence, which makes them terrorists.,toxic
This is so cool.   Who would read a book by a woman,toxic
It was a great show. Not a combo I'd of expected to be good together but it was.,not toxic
```

Create a CSV file with 25 rows and the columns ['example', 'label']. Each record should include the following fields:
- example: The column 'example' contains various statements or comments, some of which are toxic and others that are not toxic.
- label: The column 'label' categorizes the examples as either 'toxic' or 'not toxic'.

```csv

and here's the result
example,label
I can't believe how stupid some people can be.,toxic
This movie was amazing! I highly recommend it.,not toxic
I can't stand people who are always late.,toxic
I love spending time with my family. They bring me so much joy.,not toxic
This restaurant has the worst service. I will never come back.,toxic
I'm so grateful for all the opportunities I've been given.,not toxic
I can't believe how beautiful this sunset is.,not toxic
This song is terrible. I can't understand why people like it.,toxic
I had a great time at the party last night. It was so much fun.,not toxic
I can't believe how rude some people can be.,toxic
I'm so excited for my vacation next week. It's much needed.,not toxic
This book is so boring. I couldn't even finish it.,toxic
I love going for long walks in nature. It's so peaceful.,not toxic
I can't believe how expensive this car is. It's way overpriced.,toxic
I had a terrible experience at the hotel. The staff was rude and unhelpful.,toxic
I'm so proud of my sister for graduating with honors.,not toxic
I can't stand people who are always negative.,toxic
This painting is incredible. The artist is so talented.,not toxic
I can't believe how crowded this place is. It's impossible to find a seat.,toxic
I had a wonderful time at the concert. The band was amazing.,not toxic
I can't stand people who talk loudly on their phones in public.,toxic
I'm so happy for my friend who just got engaged. They make a great couple.,not toxic
This movie is so predictable. I knew exactly what was going to happen.,toxic
I love trying new foods and experiencing different cultures.,not toxic
I can't believe how lazy some people can be.,toxic

Design Doc here

lets add a test and update docs in a follow up PR


Allow users to clear selects with include_blank

Currently, even if a select dropdown has a blank option (include_blank: true), there is no way to clear the selection.
To enable this we have to specify allowClear: true in select2's config.
Fixes a particular problem whereby users cannot undo their "Primary topic" selection.
Note that when include_blank: false (the rails default), the clear button will not show up. Only selects where it was explicitly set will allow clearing after this commit.
Before

After

Trello

Once you've assigned a primary sub-topic to a piece of mainstream content, there's no way of getting rid of it. Where we've selected something from the drop-down in error or later realised it's not quite right, and there's no suitable alternative, we're having to assign a dummy topic that doesn't appear on the front end as a workaround. eg, on a document like https://publisher.preview.alphagov.co.uk/editions/5551c270ed915d2ec80002d1, if you add an entry to "Primary topic", there's no way to unset it.

https://trello.com/c/Y21ZBE9R/105-no-way-to-delete-primary-sub-topic-in-mainstream-publisher

If, like me, you were trying to look at the two images and weren't able to pick out the difference immediately... there's a tiny 'x' in the Primary Topic field.


[Bug]: Duplicated sub funnel ids may cause incorrect step rendering.

Package Scope
@use-funnel/browser
Bug description
Two identical sub-funnels within a main funnel cause step persistence issues.

Initial state: Main-A, Sub-A
After Sub step: Main-A, Sub-B
After Main step: Main-B, Sub-B (incorrect)

Expected behavior
No. 3 should show: Main-B, Sub-A
To Reproduce
No response
Possible Solution
That's because window.history.state of first sub funnel is alive even after that sub funnel has unmounted.
etc.
No response

This issue is same thing from (#85 )
Since all packages are affected, let's do a better job of thinking about it.

This issue is solved<EMAIL_ADDRESS>version at https://github.com/toss/use-funnel/pull/97.
Based on what you posted, I've listed you as a co-author on that PR.
Thanks to contribute!


Bell is audible when option is set to visual

Version: 1.0.113-nightly.0
Platform: win32 10.0.18363
Plugins: none
Maybe I'm not understanding how the "Terminal bell" option works, but I have the bell set to "Visual" but I still hear an audible bell. I just need to open a terminal and hit backspace to make it ring.

In fact, the bell is also audible even when set to "none".
My shell: WSL 1 with zsh
Terminus: 1.0.120
Reproduce:

Set Bell to off
Open WSL 1 shell
Type "cd" and press Tab a few times
-> A bell will be audible


Add supported / unsupported After Effects features

Example in lottie-ios

https://github.com/airbnb/lottie-ios#supported-after-effects-features
https://github.com/airbnb/lottie-ios#currently-unsupport-after-effects-features

@fdnhkj what lottie supports is a function of the ios and android projects, not this one. As a result, I think those projects are the right "sources of truth" for this information.  I think it's a good idea to add a link to these in the documentation though, and will do that...

Great, thank you!


Update for CUDA12.6.0

xref:

https://github.com/conda-forge/cuda-feedstock/issues/38
https://github.com/conda-forge/cuda-feedstock/issues/26

@conda-forge-admin, please restart ci

@conda-forge-admin, please restart ci

@conda-forge-admin, please lint

@conda-forge-admin, please rerender

I've been hitting that rerender error as well, doing it locally does work though.

The re-rendering is complaining it did not know how to define platform in this block
https://github.com/conda-forge/cuda-nvcc-impl-feedstock/blob/938b371b79b824c439485ac52b29553444d31835/recipe/meta.yaml#L7-L9

What version of conda-build do you have locally?

24.5.0

Ok so maybe this is a bug with the recently released conda-build version 24.7.1: https://github.com/conda-forge/conda-build-feedstock/pull/229

Just to confirm, are you using conda-smithy is 3.37.2?

Yes

Pulled your changes and local rerender had no further changes, adding automerge. Thanks John!

Regarding the re-rendering issue, filed upstream issue: https://github.com/conda-forge/conda-smithy/issues/2011


Check for iconv() TRANSLIT support

Some iconv() implementations don't support transliteration.
When iconv with //TRANSLIT is first attempted and fails,
try again without it, and remember the result.
Fixes #364
Fixes #174

Coverage increased (+0.04%) to 28.201% when pulling d41345f33b1533d7102682ed65e452b77422b986 on kobezda:check-translit-support into fcbe4fd110f91f9b973f652f5c181113cf507607 on akrennmair:master.

Coverage increased (+0.04%) to 28.204% when pulling d0cea1b4c1c0968eb4b78e5c4ce2b57e37fc9a83 on kobezda:check-translit-support into fcbe4fd110f91f9b973f652f5c181113cf507607 on akrennmair:master.

Coverage increased (+0.04%) to 28.204% when pulling d0cea1b4c1c0968eb4b78e5c4ce2b57e37fc9a83 on kobezda:check-translit-support into fcbe4fd110f91f9b973f652f5c181113cf507607 on akrennmair:master.

We don't have any tests for that part of Newsbeuter, so it's a bit early to call it a day.
Checklist of things I want to do before merging this:

[ ] build on FreeBSD, check if anything breaks
[ ] install on my machine, see if anything changes (I have UTF-8 locale so nothing should change for me)

If both of these checks fail to find any problems, I'm going to merge this PR on Sunday, October 1st.
@kobezda, thanks for your work!

Well, my testing didn't find any problems, so hopefully our users won't run into any either. Thanks, @kobezda!


[Bug] Module.Create(string, string, Address?) not implementing address

I tried using the Module.Create(string name, string catalogNumber, [Address? address = null]) method, but it looks like the address parameter is not implemented in the constructor:
public static Module Create(string name, string catalogNumber, Address? address = null)
{
    CatalogEntry catalogEntry = new ModuleCatalog().Lookup(catalogNumber);
    return new Module
    {
        Name = name,
        CatalogNumber = catalogEntry.CatalogNumber,
        Revision = catalogEntry.Revisions.Max(),
        Vendor = catalogEntry.Vendor,
        ProductType = catalogEntry.ProductType,
        ProductCode = catalogEntry.ProductCode,
        Ports = new LogixContainer<Port>(catalogEntry.Ports.Select((PortInfo p) => new Port
        {
            Id = p.Number,
            Type = p.Type,
            Address = ((p.Type == "Ethernet") ? Address.IP() : Address.Slot(0)),
            Upstream = !p.DownstreamOnly
        }).ToList()),
        Description = catalogEntry.Description
    };
}

Along with that (and I can put in a separate issue if you want), is there a reason the Module.IP property is read-only?  It would be really useful to be able to set the IP address of modules.

Good find. I must have just missed this when I created that method. I'll get it implemented for the next update.
IP was sort of a helper property to get the value configured on the Ethernet Port. But I agree that it would be valuable to set in the same way. I can also see about adding that in.

Okay this was fixed for 4.7.2 but have not had the chance to fully test it. If it works for you let me know and I will close this issue.


Dynamic name partials do not behave like static partials

Description:
Partial dynamic rendering doesn't behave in the same way as static.
Sometimes dynamic rendered partial ignores passed context using the one where it is rendered.
Background of the issue
In our application we have a table with headings / column that change dynamically based on external factors.
On first render everything works correctly but when changing the partial list, which is a computed property, the partials are rendered with the wrong context.
Versions affected:

1.3.0
1.2.0
1.1.1

Platforms affected:
All, tested on Chrome
Reproduction:
JSFiddle

Tables should have always the same content

<button on-click="@.toggle('changePartials')">Change partials</button>

With dynamic partials

  {{#each list}}

      List item is {{.}}

      {{#each ~/partialList}}
        {{>`${.}` ^^/}}
      {{/each}}

  {{/each}}

Static partials

    {{#each list}}

        List item is {{.}}

        {{>foo}}
        {{>bar}}
        {{#if ~/changePartials}}
          {{>another}}
        {{/if}}

    {{/each}}

const r = window.r = new Ractive({
  el: '#main',
  template: '#template',
  data: {
    list: [1,2,3],

    changeHeadings: true,
  },

  partials: {
    foo: 'I\'m foo and my . is {{.}}	',
    bar: 'I\'m bar . is {{.}}	',
    another: 'I\'m another . is {{.}}	',
  },

  computed: {
    partialList() {
      const changePartials = this.get('changePartials');

      const tableHeadings = [
        'foo',
        'bar'
      ];

      if (changePartials) {
        tableHeadings.push('another');
      }

      return tableHeadings;
    },
  }
});

This should now be fixed on edge. I'll wait on feedback for #3295 and backport to 1.1, 1.2, and 1.3 and publish.

When travis gets done churning, 1.1.3, 1.2.2, and 1.3.1 should be out with this fix. 1.3.1 also has Ractive.tick, which is flagged as experimental in the changelog.


Can't get SAMPLECOUNT from Python

When I try to get the new SAMPLECOUNT AOV from Python, I get this error:
RuntimeError: Unknown film output type in Film::GetOutput<float>(): 28
I compiled the latest sources.
I'm defining it like this:
film.outputs.1.type = SAMPLECOUNT
film.outputs.1.filename = samplecount.png

And I try to get it like this:
session.GetFilm().GetOutputFloat(pyluxcore.FilmOutputType.SAMPLECOUNT, channel_buffer)

Is there anything I'm missing?

I'm on the mobile but it should be an int aov not float so you need to call GetOuputInt or something like that.

Thanks, that was it.
I needed to use GetOutputUInt.


KEYBASE_DRIVE_LETTER environment

Otherwise users are stuck with hardcoded k:, whereas before watchdog2 they could at least modify the shortcut. @maxtaco @gabriel
BTW I built + tested this and it works for changing the drive letter.

retest this please

retest this please

Good to merge @maxtaco @gabriel, or should I take the command line one back out?

LGTM!


[WIP] [My Collection] Cleanup AddArtwork screen

WIP
The type of this PR is: TYPE

This PR resolves [MX-]
Description

Test Plan

Screenshots

Fails

:no_entry_sign:
Missing Test Files:

src/lib/Components/FancyModal/__tests__/FancyModalHeader-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS


Specs for show dependencies' feature #1012

Specs for show dependencies feature, see issue #1012.
This is a step towards managing dependencies #163.

Microsoft Reviewers: Open in CodeFlow

Additional note; please place this behind an experimental feature (yes, even just the information only phase).


built files is not matched with Vite's output

relative vite source code: packages/vite/src/node/plugins/reporter.ts

Rollup: https://github.com/rollup/rollup/blob/fecf0cfe14a9d79bb0eff4ad475174ce72775ead/src/rollup/rollup.ts#L183
Provides these "generated" assets to vite. Keep digging.


stencil version provides a broken acorn release

Updated my app to @ionic/angular v4.1.0. The app was originally started with v4.0.0-beta17 and with capacitor enabled.
Updated all the packages to the latest versions, but when running serve this error pops up:
[ng] ERROR in<EMAIL_ADDRESS>6:2160
[ng] Module parse failed: Unexpected token (6:2160)
[ng] You may need an appropriate loader to handle this file type.

Mike pointed me to this bug: https://github.com/ionic-team/ionic/issues/17225
The output of `npm ls acorn' with ionic-pwa-elements installed:
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>
    └─┬<EMAIL_ADDRESS>
      ├──<EMAIL_ADDRESS>
      └─┬<EMAIL_ADDRESS>        └──<EMAIL_ADDRESS>

After I remove it (and all the capacitor related packages) and run npm install again, it works.

can you try with 1.1.0?

Closing as latest version doesn't use @ionic/core


Dashboard Login Pass

The project source codes are made publicly viewable.
So it would be better if the admin panel entry is encrypted.
There may be something I missed, if there is a link to it, can you share it?
or
An idea to add a new feature to the project.
Thanks

I've been wanting this too, and in the Cloudflare workers documentation there are some examples on basic authentication. Haven't pieced together how to make it work for Counterscale yet though.

It's true that today Counterscale depends on being deployed on a publicly-accessible URL. There is no authentication or user/login today.
But I agree this is important and consider it a high priority.

Something to check https://dev.to/charca/password-protection-for-cloudflare-pages-8ma


Can't run tasks

Hello,
when I attempt to run example from readme file I received following messages:
./bin/gotask say-hello
can't load package: import cycle not allowed
package .
imports .
import cycle not allowed
package .
imports .
I'm using GO version 1.4

I think that this can be closed. This item 100 times was discussed.. sorry.

No worries. Yes, this is because your program has cycle dependencies, which is not allowed in Go.


Ongoing update issues

Date range not updating with the weekly update process.
Fix of the Contributing Factors filters not working again with the weekly update process.

Are you still adding new data to this application?

I loaded new data files on Friday.  Looks like the data updated on the
website, but the "Data available" date range notification at the top of the
page did not update.
Scott
On Mon, Feb 15, 2016 at 5:51 PM, steveoh<EMAIL_ADDRESS>wrote:

Are you still adding new data to this application?
—
Reply to this email directly or view it on GitHub
https://github.com/agrc/Crash-web/issues/53#issuecomment-184457054.

I'm going to close this since I revamped the update script. Please comment here and reopen after you upload new data or we see another issue.


Exception in thread Thread-1 (_run_pareto_accumulation)

Hi guys,
First, thank you for making this resource available, the GFlowNet seems a really promising method.
My question is that I've installed locally it following the instructions on the README, and now I'm trying the example in  sEH fragment-based MOO task in /gflownet/src/gflownet/tasks/seh_frag_moo.py. However, the calculation always dies with the error message below. I've also tried the seh_frag.py, but it dies with the same message.
$ python seh_frag_moo.py
[...]
18/06/2024 18:13:28 - INFO - logger - Final generation steps completed - sampled_reward_avg:0.39 igd:0.42 lifetime_igd_frontOnly:0.43 PCent:1.87 lifetime_PCent_frontOnly:0.69
Exception in thread Thread-1 (_run_pareto_accumulation):
Traceback (most recent call last):
  File "/blue/lic/seabra/.conda/envs/gflownet/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/blue/lic/seabra/.conda/envs/gflownet/lib/python3.10/threading.py", line 953, in run

Here is my configuration:
$ python
Python 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 15:36:39) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>> import torch_geometric as gn
>>> torch.__version__
'2.1.2+cu121'
>>> torch.version.cuda
'12.1'
>>> torch.cuda.get_device_name()
'NVIDIA A100-SXM4-80GB'
>>> torch.cuda.get_device_properties(0)
_CudaDeviceProperties(name='NVIDIA A100-SXM4-80GB', major=8, minor=0, total_memory=81050MB, multi_processor_count=108)
>>> gn.__version__
'2.4.0'

Am I missing something here? Thanks a lot!

@bengioe I have also encountered the same issue with a fresh install. have you had a chance to look into this? Thanks!


Is this still active

I've been using this library and it has been great. Unfortunately, it is missing a few features that I've added inside of my codebase. Should I spend the time to move them to the library itself and make a pull request? Will it be accepted?

Sure, every new feature is welcome

That was fast, thank! I'll get on it then 👍


[Bug] column miss

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.1.0 version
What's Wrong?
when i execute this sql , the column COLUMN_KEY is null.
select * from `information_schema`.`columns` ;

What You Expected?
I think when we use unique key table , the key column in the table should be apared as UNI on COLUMN_KEY.

It is temporarily not supported to obtain KEY information in this way. It can be realized by desc table first.

暂时不支持通过这种方式获取KEY信息。可以先实现desc table。

因为我这边是整库同步场景，想要获取所有表的元数据信息，这种情况下批量使用information_schema.columns更加方便一些。建议之后的版本可以加一下，不至于每个表都去做一个http请求或者去desc table。

I will solve the problem.


UI Tests for Messages V2

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
No response
Describe the feature you're interested in
Tests for Filters component
Tests for Select Component
Tests for offset and timestamp selection
Describe alternatives you've considered
No response
Version you're running
0.7
Additional context
No response

Hey @gimral!
Just a heads-up: we, the original team behind the project, have just announced the first release, where we fixed all the messages-related issues and other things. I kindly invite you to check our release notes here, also we'd be happy to see you as a contributor!


fix(tee-prover): simplify TeeProofGenerationDataResponse

This PR addresses Alex's code review comment:
https://github.com/matter-labs/zksync-era/pull/3017#discussion_r1795237929
and partially addresses this one too:
https://github.com/matter-labs/zksync-era/pull/3017#discussion_r1795240750
What ❔
Simplify TeeProofGenerationDataResponse.
Why ❔
Because it's ugly.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[x] Code has been formatted via zk_supervisor fmt and zk_supervisor lint.

Closing in favor of https://github.com/matter-labs/zksync-era/pull/3067


Add getMaxShortcutCount method for quick_actions.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
In quick_actions package, there is no method to getMaxShortcutCount.
Android.
You can get that count by calling getMaxShortcutCountPerActivity.
https://developer.android.com/reference/androidx/core/content/pm/ShortcutManagerCompat#getMaxShortcutCountPerActivity(android.content.Context)
iOS
Maximum: 4

People tend to expect every app to provide at least one useful quick action; you can provide a total of four.

https://developer.apple.com/design/human-interface-guidelines/home-screen-quick-actions#Best-practices
Proposal
Implement getMaxShortcutCount method for each platform based on above things.

I made a mistake on my account. I will re-create a new issue. Sorry...


adjust RNTester examples list for macOS

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[X] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
This PR introduces the following changes:

introduces custom tests list for macOS in RNTesterList.macos.js file rather than returning the iOS one
skips the examples that are iOS or mobile platform only
moves the macOS only tests out of iOS test list (this should also fix at least one issue in e2e CI in #403 PR)

The initial list of disabled tests is very subjective, I'm happy to tweak or change it completely if needed.
Changelog

[Internal] [Changed] - implement custom RNTester example list for macOS
Test Plan
I have testes those changes using local copy of react-native-macos repository and RNTester build in from this source. ESLint, yarn flow-check-ios and yarn flow-check-macos checks has been successful.
There is a probability that the CI fails so I'm going to investigate those errors too.
Preview

Microsoft Reviewers: Open in CodeFlow

In the state it's in now, it sort of feels like we're just kicking the can as to where we fork for macOS vs iOS. What if we forked up at the AppDelegate where we use RNTesterApp.ios for both macOS and iOS as that seems like the top most misnomer? From there we can then split down into respective platform specific test suites

@msftbot please behave, I have responded to the comments.


\u2013 character maps to undefined

soundscrape started crashing on me at the same point in a user's like list. I found the culprit to be one song: https://soundcloud.com/emiljoacmusic/emiljo-ac-go-hbd-j-dilla-february-7-1974-february-10-2006
The error I saw was a \i2013 unicode character undefined. This character should correspond to a special dash character that I suspect is used in the soundcloud song title "Emiljo A.C. - Go! - HBD J Dilla - (February 7, 1974 – February 10, 2006)".

Could you post the full traceback & the exact command you used? I didn't have any issue downloading the song on windows & linux.

Works for me (great track too)..

You should pip install soundscrape --upgrade to get the latest version.

Hi all,
Thanks for your help with this issue. Just knowing that the issue was on my end helped! In the end, the issue seems to have been something to do with passing the song titles thru to the console. In case anyone else has this issue in the future, here's what I did:

I'm now calling soundscrape direct from a batch file rather than via an os.system python call.
I switched over to Python 2 (which meant a fresh pip install). Previous fresh installs (and @Miserlou 's suggestion of pip --upgrade) had failed to solve the issue, so I'm betting there's something wrong with the way my Windows 7 system was handling special unicode characters in the console with python 3.

Anyway, thanks, and sorry this reply was so long coming (got busy!)


Multi instance scoped storage usage

What?
Ensure the SDK uses the new Device State Store and remove methods and tests that used the old style
...
Why?
We want people to be able to use multiple instances in the sdk
...

CC @pusher/mobile

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature-multi-instance@bb6476f). Click here to learn what that means.
The diff coverage is 100%.

@@                    Coverage Diff                    @@
##             feature-multi-instance     #132   +/-   ##
=========================================================
  Coverage                          ?   89.63%
=========================================================
  Files                             ?       53
  Lines                             ?     2991
  Branches                          ?        0
=========================================================
  Hits                              ?     2681
  Misses                            ?      310
  Partials                          ?        0

Impacted Files
Coverage Δ

Sources/Persistence/DeviceStateStore.swift
94.44% <ø> (ø)

Sources/Internal/ServerSyncProcessHandler.swift
81.95% <ø> (ø)

Sources/PushNotifications.swift
86.93% <100%> (ø)

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 bb6476f...6eeb034. Read the comment docs.


Race conditions on arrays

Turns out I didn't check thoroughly enough for race conditions.

fixed the race condition, but removed all results


Redeploying a deployment has no effect

When I click on "Redeploy" on a deployment

it looks like everything is working, but the operation has no effect. I would expect a rolling update to be triggered (like in the ember ui), but instead nothing happens and the Pods remain running (see age):

Seeing the same problem.

Seeing the same problem.

The label is getting set on the workload (metadata.labels) instead of the pod spec (spec.template.metadata.labels)

The label is getting set on the workload (metadata.labels) instead of the pod spec (spec.template.metadata.labels)

Verified on
Rancher | v2.5-b83b45c96e199dd0dc503131471ae94c8019a827-head
User Interface | v2.5-dev
Helm | v2.16.8-rancher1
Machine | v0.15.0-rancher50
Steps:
Create a cluster and deploy a workload from dashboard
Edit config and increase the replicas or just redeploy the workload.
The event history shows the pods are redeployed

Verified on
Rancher | v2.5-b83b45c96e199dd0dc503131471ae94c8019a827-head
User Interface | v2.5-dev
Helm | v2.16.8-rancher1
Machine | v0.15.0-rancher50
Steps:
Create a cluster and deploy a workload from dashboard
Edit config and increase the replicas or just redeploy the workload.
The event history shows the pods are redeployed


Use sizeof(*var) for malloc/realloc/memmove

This is more future-proof than using sizeof(type).
I did add a few sizeof(*var) multiplications when they appeared to be missing, so I'd recommend taking a second look at this before merging.

Yes! I've been meaning to do this for a while, but haven't gotten around to it. I'd love to merge this, but I'm new to this PR merging work. Your commit's currently broken (there's a typo in tran_s_form table's name). Once that's done, I'd be glad to merge this and then later work on rolling over my modules so they use the same form.

Should be fixed.

Much appreciated. I'll make sure to only use the new style henceforth.


Unable to install MediaPipe on Mac pip install -q mediapipe==0.10.0

OS Platform and Distribution
MAC Ventura 13.5 Intel Core 8
Compiler version
pip --version 20.3.4
Programming Language and version
python3 --version 3.12.0
Installed using virtualenv? pip? Conda?(if python)
No response
MediaPipe version
20.3.4
Bazel version
bazel 6.4.0-homebrew
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
None
OpenCV version (if running on desktop)
No response
Describe the problem
Can not install 0.10.0 Python
Complete Logs
pip install -q mediapipe==0.10.0

ERROR: Could not find a version that satisfies the requirement mediapipe==0.10.0 (from versions: none)
ERROR: No matching distribution found for mediapipe==0.10.0

This also fails, saw that there are a few new releases up to 0.10.7, etc
python3 -m pip install mediapipe
ERROR: Could not find a version that satisfies the requirement mediapipe (from versions: none)
ERROR: No matching distribution found for mediapipe

Ok had to downgrade python3 to 3.11.6 using pyenv.  wow. Hope this helps others.

Hi @jeff-orthober,
The build has failed due to the current lack of support for Python Version 3.12.*. Please note that MediaPipe supports Python versions 3.9 to 3.11 across all platforms, as outlined in the provided guide. We recommend ensuring the compatibility of Python, Bazel, and other dependencies before proceeding with the MediaPipe build.
For now, We are closing this issue as it has been resolved on your end. Thank you


Stop translating BOOL to bool

Fixes #362

I'd personally not add the implicit conversion operator, as it encourages the while (GetMessage(...)) kind of code, which is explicitly pointed out as wrong in the docs for GetMessage. Or maybe this special case is something an analyzer could handle.

That would hurt usability of all APIs that use BOOL for the sake of one or two that are this way, right?

That's why I suggested adding an analyzer instead, that would warn for such usages.

I imagine analyzers could be a huge win for users. But also a bottomless pit of work. If folks are interested in sending PRs that add analyzers, I will try to find time to add the first one as a template for how more can be added.


gapi.client.load(urlOrObject) should have an optional apiRoot parameter

I am loading my client library using

gapi.client.load(apiName, apiVersion, apiLoadCallback, apiRoot)

This uses an undocumented apiRoot parameter, which is needed when testing in my development environment (localhost).
Unfortunately, this way of calling gapi.client.load is deprecated. I would like to use gapi.client.load(urlOrObject) but there is no way to specify the apiRoot, so it does not work on localhost.
Please make it possible to use something like gapi.client.load(urlOrObject, apiRoot) and document it.
A related question is, if it is possible to use an apiRoot for gapi.client.init(args)? The documentation does not say so.

Thanks for your comment. The recommended way is to use URLs for everything. In local testing, just use gapi.client.load("http://localhost/...").

@bochunz: Thank you for your help. The problem turned out to be, that I had omitted the protocol-part from the URI of my discovery document. That caused gapi.client.load to look for the discovery document on Google's servers. Of course that did not work.
Once I had fixed that, gapi.client.load did exactly what I wanted, and I have stopped using the deprecated version.


Write an "installer" for Linux

Working on it - I think this goes hand in hand with #7

Pretty much done with #7 But should be included in our test matrix

neuro-conda-latest_linux.yml.txt

install.sh.txt
uninstall.sh.txt

Done this:
Check if script is run in POSIX mode
#if [[ -n "${POSIXLY_CORRECT+1}" ]]
#then
posix_abort 'Bash must not run in POSIX mode. Please unset POSIXLY_CORRECT and try again.'
#fi
and this:
if [[ "${OS}" == "Linux" ]]; then
info "Using Linux config..."
NeuroCondaLatestUrl="https://raw.githubusercontent.com/neuro-conda/neuro-conda/main/envs/neuro-conda-latest_linux.yml"
else
info "Using Default config..."
NeuroCondaLatestUrl="https://raw.githubusercontent.com/neuro-conda/neuro-conda/main/envs/neuro-conda-latest.yml"
fi
if curl --output /dev/null --silent --head --fail "$NeuroCondaLatestUrl"; then
info "$NeuroCondaLatestUrl exits"
else
error "$NeuroCondaLatestUrl does not exits!!!"
fi

@joschaschmiedt You need to figure out what it breaks on your end. :-)

Thanks, @davrot, I included your suggestions in #19. The platform-specific installation of the ML packages is now handled via Pip's platform_architecture and platform_system flags (like this).


feat: Add npm support

Adds npm support through use of yarn-or-npm.
Tested on Windows 10:
✅ WSL + Node 12
✅ WSL + yarn
✅ Powershell + Node 12
✅ Powershell + yarn
https://github.com/mweststrate/relative-deps/issues/5

Awesome, thanks a lot! Released as v1.0.2


[Enhancement] Add bonus to views and rename to type

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) locally.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

As we added bonus points to exercises, we want to display them as well in different views.
Description

Added bonus points to some views
Changed some translations (Exercise Type -> Type, Max Points -> Points)

Steps for Testing

Log in to Artemis
Navigate to Course Administration and
Go to Assessment/Instructor Dashboard of a course and then to Exercise Dashboard
Check if for exercises with bonus points the bonus points are displayed as well also (Exercise Type -> Type, Max Points -> Points)
Navigate to Course Overview and go into a course
Enter a Exercise and check if for exercises with bonus points the bonus points are displayed as well
Go to Course Administration and then to Exams
Check Exercise Groups and check that it says Points and not Max. Points
Participate in Exam (or do a test run) check if for exercises (modeling, text and programming) with bonus points the bonus points are displayed as well

Check also if I missed some other spots where this can be changed/added. Just let me know :)
Test Coverage

Nothing changed
Screenshots

Student Exercise Details View:
Bonus:

No Bonus:

Assessment Dashboard:

Instructor Dashboard:

Exam:

Client tests are failing because of formatting. I just runned prettier and got this back:

So do not wonder why it is formatted so strangely 😅

Imho

would look better as

Opinions?

I would like to avoid a third line and rather make the header a bit smaller. I think the current display is good

Do we want to display bonus points in the "View" exercise as well? Currently, there are only the normal points.
Yes, this would be good!
For exams (in this case a test run) i found "Max. Points" in the summary. Is this intended/ok?
Please rename to Points here as well.

Basically Max. Points should not be used any more in the whole application

In my exam summary, the bonus points are not shown. (Exercise 1 should have bonus points)

Mini remark: You could switch the white space to the second string, so without the second string, the square brackets are even:

The rest looked very good to me!

@FuchsDominik I added this now:

The issue with the whitespace comes from the formatting. I tried out to use prettier-ignore to fix the issue with the styling mentioned by @FrankeLukas.
@MaisiKoleni I added the bonus points also in the exercise view :

Would be nice if you could recheck :)


question on running the demo code

Hi , after installing different dependencies on ubuntu 16.04 , for your project . I come across this error . I am wondering if there is something i am missing in this regards ?? ?
snippet below . I am however using virtual machine for ubuntu
python webcam_demo.py
2019-03-31 13:52:55.633518: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
/home/ubuntu/repos/posenet-python/posenet/converter/config.py:9: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  cfg = yaml.load(cfg_f)
Cannot find model file ./_models/model-mobilenet_v1_101.pb, converting from tfjs...
Traceback (most recent call last):
  File "webcam_demo.py", line 63, in <module>
    main()
  File "webcam_demo.py", line 20, in main
    model_cfg, model_outputs = posenet.load_model(args.model, sess)
  File "/home/ubuntu/repos/posenet-python/posenet/model.py", line 42, in load_model
    convert(model_ord, model_dir, check=False)
  File "/home/ubuntu/repos/posenet-python/posenet/converter/tfjs2python.py", line 164, in convert
    variables = load_variables(chkpoint)
  File "/home/ubuntu/repos/posenet-python/posenet/converter/tfjs2python.py", line 60, in load_variables
    byte = open(os.path.join(base_dir, chkpoint, filename), 'rb').read()
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/_posenet_weights/mobilenet_v1_101/MobilenetV1_Conv2d_11_pointwise_biases'

I'm using windows 10 and I'm getting same error how can I solve this?

Hi , after installing different dependencies on ubuntu 16.04 , for your project . I come across this error . I am wondering if there is something i am missing in this regards ?? ?
snippet below . I am however using virtual machine for ubuntu
python webcam_demo.py
2019-03-31 13:52:55.633518: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
/home/ubuntu/repos/posenet-python/posenet/converter/config.py:9: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  cfg = yaml.load(cfg_f)
Cannot find model file ./_models/model-mobilenet_v1_101.pb, converting from tfjs...
Traceback (most recent call last):
  File "webcam_demo.py", line 63, in <module>
    main()
  File "webcam_demo.py", line 20, in main
    model_cfg, model_outputs = posenet.load_model(args.model, sess)
  File "/home/ubuntu/repos/posenet-python/posenet/model.py", line 42, in load_model
    convert(model_ord, model_dir, check=False)
  File "/home/ubuntu/repos/posenet-python/posenet/converter/tfjs2python.py", line 164, in convert
    variables = load_variables(chkpoint)
  File "/home/ubuntu/repos/posenet-python/posenet/converter/tfjs2python.py", line 60, in load_variables
    byte = open(os.path.join(base_dir, chkpoint, filename), 'rb').read()
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/_posenet_weights/mobilenet_v1_101/MobilenetV1_Conv2d_11_pointwise_biases'

I have the same issue anyone pls help


Fix serde for FragmentSearchQuerySpec and add some tests.

Having two JsonCreator constructors is bad because it makes deserialization not work.

:+1:


Fix logger TypeError for Python 3.8 API change

Fix logger error TypeError: findCaller() takes from 1 to 2 positional arguments but 3 were given when running with Python 3.8.

Thanks! We will supplement the version test actions.


Add ability to show curve scales and units to each curve box

Add ability to show curve scales and units to each curve box

Can I get axis to always label max and min?
Can we change font size by explicitly declaring it?
Where should units go?
How to handle units that are long or don't quite fit?


Adding FD HD full geometry gdmls of v6.

This PR only adds the gdmls. Configuration tables in geometry_dune.fcl will go in a separate PR.

trigger build with DUNE/dunecore#127

:heavy_check_mark: CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

:heavy_check_mark: CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

:x: CI build for DUNE Failed at phase ci_tests DUNE on slf7 for c14:prof - ignored warnings for build -- details available through the CI dashboard
:rotating_light: For more details about the failed phase, check the ci_tests DUNE phase logs
parent CI build details are available through the CI dashboard

:x: CI build for DUNE Failed at phase ci_tests DUNE on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard
:rotating_light: For more details about the failed phase, check the ci_tests DUNE phase logs
parent CI build details are available through the CI dashboard


[W.i.p] test_nixio.py

End of my internship. Here are the latest modification i made to test_nixio.py not everything is working.
Probably some part or also missed.

Hello @teogale! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file neo/core/imagesequence.py:

Line 196:73: W291 trailing whitespace

In the file neo/test/iotest/test_nixio.py:

Line 140:63: W504 line break after binary operator
Line 149:63: W504 line break after binary operator
Line 957:62: E231 missing whitespace after ','
Line 1067:100: E501 line too long (103 > 99 characters)
Line 1128:62: E203 whitespace before ','
Line 1129:25: E231 missing whitespace after ','
Line 1142:31: E231 missing whitespace after ','

Hi.  I had a look at the PR and everything looks good in general.  There's a couple of small issues that  are causing the tests to fail.  Here's a patch (and explanation) that should fix the issues.
One issue that appears in a few places is that the ImageSequence constructor ignores Quantities in the image_data attribute and converts that data to a dimensionless array.  Units for the data need to be specified with the units attribute.  Some tests were failing because when writing and then reading back an ImageSequence, the units were lost (not read) since the reader part of the NixIO was creating ImageSequence objects with a Quantity.  See the following patch for a fix:
diff --git a/neo/io/nixio.py b/neo/io/nixio.py
index b166b8fb..b6f749b4 100644
--- a/neo/io/nixio.py
+++ b/neo/io/nixio.py
@@ -465,7 +465,6 @@ class NixIO(BaseIO):
         neo_attrs["nix_name"] = metadata.name  # use the common base name
         unit = nix_da_group[0].unit
         imgseq = np.array([d[:] for d in nix_da_group]).transpose()
-        imgseq = create_quantity(imgseq, unit)

         sampling_rate = neo_attrs["sampling_rate"]
         del neo_attrs["sampling_rate"]
@@ -473,7 +472,8 @@ class NixIO(BaseIO):
         del neo_attrs["spatial_scale"]

         neo_seq = ImageSequence(image_data=imgseq, sampling_rate=sampling_rate,
-                                spatial_scale=spatial_scale, **neo_attrs)
+                                spatial_scale=spatial_scale, units=unit,
+                                **neo_attrs)

         self._neo_map[neo_attrs["nix_name"]] = neo_seq
         # all DAs reference the same sources
diff --git a/neo/test/iotest/test_nixio.py b/neo/test/iotest/test_nixio.py
index 982537fd..bb5f3a1c 100644
--- a/neo/test/iotest/test_nixio.py
+++ b/neo/test/iotest/test_nixio.py
@@ -896,9 +896,10 @@ class NixIOWriteTest(NixIOTest):
         seg = Segment()
         block.segments.append(seg)

-        imgseq = ImageSequence(image_data=self.rquant((19, 10, 15), pq.V),
+        imgseq = ImageSequence(image_data=self.rquant((19, 10, 15), 1),
                                sampling_rate=pq.Quantity(10, "Hz"),
-                               spatial_scale=pq.Quantity(10, "micrometer"))
+                               spatial_scale=pq.Quantity(10, "micrometer"),
+                               units=pq.V)
         seg.imagesequences.append(imgseq)
         self.write_and_compare([block])

@@ -1125,8 +1126,7 @@ class NixIOWriteTest(NixIOTest):
                 for imgseqdx in range(nimgseq):
                     imseq = ImageSequence(
                         name="{}:imgs{}".format(seg.name, imgseqdx),
-                        image_data=np.random.rand(20, 10, 10)
-                        ,units=pq.mV,
+                        image_data=np.random.rand(20, 10, 10), units=pq.mV,
                         sampling_rate=pq.Hz, spatial_scale=pq.micrometer
                     )
                     print(imseq.units)

The other issue was that ImageSequence objects weren't being added to the list of objects to check for naming conflicts when the use_obj_names flag was set.  The test for this feature was failing.  The following single line patch should suffice:
diff --git a/neo/io/nixio.py b/neo/io/nixio.py
index b6f749b4..13fa73a6 100644
--- a/neo/io/nixio.py
+++ b/neo/io/nixio.py
@@ -1424,6 +1424,7 @@ class NixIO(BaseIO):
             for seg in blk.segments:
                 signals.extend(seg.analogsignals)
                 signals.extend(seg.irregularlysampledsignals)
+                signals.extend(seg.imagesequences)
                 eests.extend(seg.events)
                 eests.extend(seg.epochs)
                 eests.extend(seg.spiketrains)

Finally, there's part of a test that retrieves the number of signals (channels) in an AnalogSignal or IrregularlySampledSignal by assuming the signals are 2-dimensional and getting the second dimension from the output of np.shape().  There's a change in this PR that does the same for ImageSequence by getting the third dimension.  This doesn't seem to generalise well for all the tests, so I changed it to just index the result of np.shape() with -1.
diff --git a/neo/test/iotest/test_nixio.py b/neo/test/iotest/test_nixio.py
index bb5f3a1c..e4350907 100644
--- a/neo/test/iotest/test_nixio.py
+++ b/neo/test/iotest/test_nixio.py
@@ -194,10 +194,7 @@ class NixIOTest(unittest.TestCase):
             for da in data_arrays:
                 if da.metadata.name == nixname:
                     dalist.append(da)
-            if nixname[:17] == "neo.imagesequence" or nixname[11:15] == "imgs":
-                _, _, nsig = np.shape(sig)
-            else:
-                _, nsig = np.shape(sig)
+                nsig = np.shape(sig)[-1]
             totalsignals += nsig
             self.assertEqual(nsig, len(dalist))
             self.compare_signal_dalist(sig, dalist)

Hi, thanks for the comment happy to see the modification was globally good  , i pushed the modification but ci/circleci is failling from test_read_all (neo.test.rawiotest.test_nixrawio.TestNixRawIO)

The failing test is unrelated, as far as I can tell.
I mentioned it in this PR https://github.com/NeuralEnsemble/python-neo/pull/767#issuecomment-554322399
but I haven't heard back about it so I'm not sure if that's the real cause.

@achilleas-k: Since #767 is merged now and the tests are succeeding, would you still like to add the proposed changes to the nixio?

Done.

replaced by #804


Set User-Agent header on both REST and graphql requests

In #256 we starting setting a custom User-Agent header on REST API calls, to help us understand the ways our API is used. It looks like this:
Terraform/ (+https://www.terraform.io) Terraform-Plugin-SDK/2.0.3 buildkite/0.15.0

This builds on that work and sets the same User-Agent on graphql API requests as well.
It drops the unnecessary use of golang.org/x/oauth2 to set a static bearer token header, and uses the tripperware pattern to set both headers we care about:

the static bearer token for auth
the user agent

The same http.Client is used for both REST and graphql, so we'll get identical behaviour on both APIs.

@jradtilbrook @mcncl absolutely no hurry on this one, I just thought I'd poke at it while the changes in #256 were fresh in our brains. If we merge it, it could wait for the next release you do for other reasons.

Happy to merge once @danstn suggestions are committed in. Also happy to just commit them from comment too.

Happy to merge once @danstn suggestions are committed in. Also happy to just commit them from comment too.

Oh these are not blocking, happy to land it as is. @yob up to you

They're good suggestions, I'll make them in a bit :heart:

I've confirmed that we see the Terraform useragent in our API access logs for both the REST and graphql calls


WIP: Hs deploy 1.4

This branch represents the changes needed to deploy the provider editing features to users. Currently it's hidden behind a flag.
This picks on scope from two Trello cards:

https://trello.com/c/JpFDqO3j/308-provider-portal-provider-listing-links-should-link-to-summary-view-not-old-portal
https://trello.com/c/SlW3wOhe/315-deploy-provider-editing-features-lift-provider-editing-flag

The short version of the changes needed here are:

[x] When a user has approved listings, and they are in their provider portal home, clicking on the title on the provider should take you to the summary view, not the old style portal.
[x] Removing the flag check for the provider edit_provider

Closing as we merged in #539 in December which includes these updates.


updating to support cloud9 dev environment

Adds a theme script, and changes the port to 8080 (to support c9 dev environment)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

no idea why the whole file looks like a diff/change -- I notated the only 2 changes.

Aah, these don't really do anything, but can be changed for consistency.
On Wed, Oct 24, 2018, 5:35 PM Ryan<EMAIL_ADDRESS>wrote:

@rnzsgh approved this pull request.
Other than the repo link question, LGTM.
In package.json
https://github.com/aws-samples/eks-workshop/pull/105#discussion_r227842034
:

"mermaid": "^8.0.0-rc.8"

}

"name": "eksworkshop",
"version": "1.0.0",
"homepage": "https://github.com/brentley/eksworkshop#readme",
"bugs": {
"url": "https://github.com/brentley/eksworkshop/issues"
},
"license": "ISC",
"main": "index.js",
"repository": {
"type": "git",
"url": "git+https://github.com/brentley/eksworkshop.git"
},
"scripts": {
"theme": "git submodule init && git submodule update",

Should these links point to the aws-samples repo?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/pull/105#pullrequestreview-167974685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO5EDHqVPY6A8z4ZxschK9GAXpQa_EEks5uoIjBgaJpZM4X3vlf
.


radeondrmfb unsupported?

Is this expected?   Installation will proceed
         but may produce broken setup, and/or some features may not be
         available or work as expect. Supported frame buffer types are:
         i915drmfb amdgpudrmfb svgadrmfb```

I should have mentioned...KMS/DRM appears to be running fine on this HW.


Can't send msg, There is no error

I have users list msg, and every user has name.
sometimes there is characters and telepot can't send msg. also there is no errors
i was replace strings and every things worked
but there is new chars and i can't catch the problem
is there any way to solve this?
def remove_bad_char(name):
    name = name.replace('�', '')
    name = name.replace('/', '')
    name = name.replace('@', '')
    name = name.replace('?', '')
    name = name.replace(':', '')
    name = name.replace('\n', '')
    name = name.replace('\\', '')
    name = name.replace('ـ', '')
    name = name.replace('-', '')
    name = name.replace('.', '')
    name = name.replace('', '')
    return name

updated this function and everythigns worked. is there any list of chars?

def remove_bad_char(name):
    name = name.replace('ई', '')
    name = name.replace('ह', '')
    name = name.replace('═', '')
    name = name.replace('°', '')
    name = name.replace('⇣', '')
    name = name.replace('↶', '')
    name = name.replace('/', '')
    name = name.replace('@', '')
    name = name.replace('?', '')
    name = name.replace(':', '')
    name = name.replace('\n', '')
    name = name.replace('\\', '')
    name = name.replace('ـ', '')
    name = name.replace('-', '')
    name = name.replace('.', '')
    name = name.replace('', '')
    return name

First, I don't understand your question.
Second, are you using Python 3?

no i user python 2.7
when msg has character like 'ई', msg can't deliver ... without any error

I am wondering, if you are using Python 2.7, should you put a u in front of unicode strings, e.g. u'ई', to make sure it's unicode? Not sure it can fix your issue. Just wondering.

received text is utf-8 .... no need no u

i didn't have any problem sending these characters in python 3 using this code:
import telepot
bot = telepot.Bot("xxx")
bot.sendMessage(xxx,"ईह═°⇣↶/@?:\n\\ـ-.")

i suspect your problem is with utf-8 handling, not these specific characters.
if you want to limit the names to ascii chars you can use the following code:
import re
def remove_nonalphanumeric(name):
    return re.sub('[\W_]+', '', name)

but i suggest that you upgrade to python 3, it has a much better utf-8 handling.

no this can't help, many ascii chars worked but some chars not work!
maybe upgarde to python 3 is better way


[Feature Proposal] More component examples

I love the idea of Tailwind and utility-based CSS frameworks but one thing that is stopping me from embracing it is that I'm a backend developer who at the moment still relies on frameworks like Bootstrap to build internal corporate tools and other apps.
Bootstrap feels like it's getting stale, at least to me, and I would love to try something new like Tailwind but right now it's still intimidating to make the switch. I think adding more component examples to the point that you almost have a full collection comparable to other CSS frameworks would allow more developers to feel like they could take the leap.
If this is out of scope of this project then please feel free to close this issue. Thanks.

Hey @ryanmortier! Definitely hoping to add more component examples over time but it's more of a documentation issue than a bug/feature request with the actual framework, so do you mind starting a new issue over at https://github.com/tailwindcss/docs with any component examples that would be helpful for you to see added?
Going to close this one here but again feel free to continue the discussion on the docs repo 👍

Sure, thank you.


ARXIVNG-407 Minimal submission API

Minor refactor of austere metadata submission API. Depends on #18 and #19.

@JaimieMurdock Were you able to get this working OK?

Merging per discussion.


Add GuildID to Multiple Structs

Discord has introduced the guild_id property into various guild related events and this PR aims to support them.

This is great :)


Video Player not working

Now, before anything...
No, I'm not in 1.2.0. I'm in 2.0.0. And this bug still happens? Gosh, this is terrible. Does anyone know a fix?

I'm having a similar issue too.

I'm having a similar issue too.

I'm making videos on FlashThemes until someone comments a solution.


Adding package features for Dlib and OpenCV

Dlib - Package feature for blas (dlib is compatiable with Intel MKL) and CUDA
OpenCV - Added option to integrate GDCM support (to read DICOM files)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: lp55 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Just signed the CLA

Looks good, thanks for the PR!
(future note: Most changes like this would require bumping the package version, but this particular case is ok because all existing builds should stay the same 😊)

I'll watch out for the need to bump the package version on future commits :)


copy-webpack-plugin

Version
3.0.5
Node and OS info
node 8.1.1, npm 3.10.10, windows 10
Steps to reproduce
project

    --src

        --main

            --webapp

    --target

        --www (vue.config.js  .outputDir)

    --package.json

My frontend project is in src/main/webapp, all files are in this directory, include vue.config.js and public files.  When i set VUE_CLI_SERVICE_CONFIG_PATH, my project can be built, but the public resources are not copied to the outputDir.  When I set the VUE_CLI_CONTEXT, i can't use jsx in render function,  the error info is follwing:
Module parse failed: Unexpected token (12:11)
You may need an appropriate loader to handle this file type.
What is expected?
build success & public files copied & jsx in render function is allowed
copy-webpack-plugin  source path can be configured
What is actually happening?
public files will not be copied or build failed

Hi
Thanks for filing this issue! Could you please provide a small reporduction on jsfiddle.net so we can debug this without trying to replicate this ourselves?`
That would be very helpful, thanks!

Closing due to inactivity for more than 1 week. Please open a new issue with a reference to this one if you can follow up with more information.


Pass response callback as block

The one thing that has bothered me about Scrapy is that callbacks can't be given inline to show the visual hierarchy of pages scraped. Ruby however has blocks. Could we do something like this?
request url: 'http://example.com' do |response|
    response.at_xpath("//a[@class='next_page']").each do |next_link|
        request url: next_link do |response2|
            #etc
        end
    end
end

@Tails Wow, this is interesting design! I think it's possible to extend request_to method to optionally receive a block just like on your example. I'll try to implement it soon and tell you how it goes.

I actually managed yesterday and it's not too difficult. There is only one major drawback which is that the inline callbacks cannot be called from the CLI (without some more complex promise-based lookup).
Here's how it works (in ApplicationSpider):
# use this class callback to bootstrap from start_urls
def self.request_start(&handler)
  define_method(:parse, &handler)
end

# parse page using inline block as callback
def request_in(handler_id, delay = nil, **opts, &handler)
  handler_name = "parse_#{handler_id.to_s}_page".to_sym
  self.class.send(:define_method, handler_name, &handler) unless self.methods.include? handler_name
  request_to handler_name, delay, **opts
end

# request multiple urls inline
def request_all(handler_id, delay = nil, urls:[], **opts, &handler)
  urls.each do |url|
    request_in(handler_id, delay, url:Linkable.parse(url), **opts, &handler)
  end
end

And this is what you can do (define at class-level in Spider):
class YourSpider < ApplicationSpider
  # start from start_urls
  request_start do |category_list, **opts1|
      request_all :product_list, urls: category_list.css(".css-selector1") do |product_list, **opts2|
        category_name = product_list.css('h1.main-title').text
        for link in product_list.css(".css-selector2")
          request_in :product, url: link, data: {category_name: category_name} do |product, **opts3|
            item = parse_entity(:product)
            save_to "results.json", item, format: :pretty_json
          end
        end
      end
    end
end


Httpclient 2044 reuse immutables

[HTTPCLIENT-2044] Don't create stateless/immutable factory objects.

For my understanding, every client instance  has its own factory instances, Client are thread-safe, so must be those factory instances. Therefore, they are safe to be shared among client instances?

All of the classes for which I created a DEFAULT or INSTANCE are stateless and defined as @Contract(threading = ThreadingBehavior.STATELESS). I think it would be better to doc some of these as immutable instead of stateless, but it's a detail.

Looking into...

Rebased, let's wait for the test results.

Thanks @michael-o


Example Notebook

Is your feature request related to a problem? Please describe
Not a major issue but would be nice to have a clear example of how to use the Jumanji API.
Describe the solution you'd like
An example notebook containing:

the basic usage of the Jumanji API
how to train an RL agent on a Jumanji Environment
rollout the agent and visualise its behaviour

Describe alternatives you've considered
N/A
Additional context
N/A

Misc

[ ] Check for duplicate requests.

Duplicate


Attach Handlers to Container Lifecycle Events

@chairraver Hi, Could you please share the details (here) on the issue you would like to report or the assistance you require? Kindly provide details in the description of the issue.
/triage needs-information

I'm very sorry, but this was a complete accident. I'll close this issue. I came here because I was researching some local K8s errors and just before actually closing the browser tab I must have accidentally clicked somewhere and that must have created this issue.
I apologize.


Update README.md to remove build badges

One badge is broken and the other relates to terraform which really isn't that interesting to 99% of people.  Removing the badges as they are not relevant and we get alerts anyway.

:tada: This PR is included in version 1.32.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[win10] Crash after running, exception: STATUS_ACCESS_VIOLATION

I tried to execute yarn tauri dev command, or yarn tauri build command and then run app_name.exe from target/release, both encountered the following issue: the window appears 2 seconds after the crash exit, the window appears during the white screen, no normal web page content is displayed.
$ yarn tauri dev
yarn run v1.22.10
$ tauri dev
 app:spawn Running<EMAIL_ADDRESS>tauri dev" +0ms

   Compiling app v0.1.0 (C:\Users\xxx\source\repos\tauri_demo\src-tauri)
    Finished dev [unoptimized + debuginfo] target(s) in 10.63s
     Running `target\debug\app.exe`
error: process didn't exit successfully: `target\debug\app.exe` (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION)

Environment info:
Operating System - Windows, version 10.0.19042 X64
Webview2 - 91.0.864.64

Node.js environment
  Node.js - 14.17.3
  @tauri-apps/cli - 1.0.0-beta.5
  @tauri-apps/api - 1.0.0-beta.4

Global packages
  npm - 6.14.13
  yarn - 1.22.10

Rust environment
  rustc - 1.53.0
  cargo - 1.53.0

App directory structure
/node_modules
/src-tauri
/src-web

App
  tauri.rs - 1.0.0-beta.4
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../src-web/index.html
  devPath - ../src-web/index.html

From the event viewer log of the win system, we can see that the exception from user32.dll.

Is your repo public? I need to see what apis are used.

@amrbashir I'm sorry to tell you: this crash issue cannot reproduce after I restarted my computer, but it was indeed 100% reproduce before I restarted the win system. I don't know what happened.
I didn't apply any api, I just ran the helloworld example. if I can reproduce it next time, I'll let you know.
By the way, I sometimes find that of course when I run any command, the following result appears and I have to wait a while before it works, but rustlang environment was installed.
$ yarn tauri dev (or yarn tauri init, yarn tauri info, yarn tauri build)
yarn run v1.22.4
$ C:\Users\xxx\source\github\tauri_demo\node_modules\.bin\tauri dev
Downloading Rust CLI...
Done in 0.53s.

Take a look at this issue, https://github.com/tauri-apps/tauri/issues/2079 , it should be fixed


c2pa.node: Module parse failed: Unexpected character '�' (1:0)

Hi,
When I install c2pa-node in my project (Next.js ^14.2.5, Typescript ^5.3.3), I have this error :
⨯ ./node_modules/c2pa-node/dist/generated/c2pa.node
Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)

Inquirey
Is it needed to assign the task to start working on this issue


[feature]: Add focus programmatically

Feature Request
I would like to propose the addition of a feature that allows programmatically setting focus on an OTP TextField within the SwiftUI package. Currently, there is no built-in way to set focus to a specific OTP TextField, which can be useful in various scenarios.

You can add this line
_ = textField.becomeFirstResponder()
in this class AEOTPViewRepresentable
please, add it with done button to dismiss the keyboard


boards: shields: Allow adding source code to shields

Add shield CMakeLists.txt if it exists.
Useful for one-off shields that require C code
This came up in a dev-review discussion from this PR: https://github.com/zephyrproject-rtos/zephyr/pull/69696#issuecomment-2195201613

No, shield should only be used to configure and describe real hardware. Snippets are introduced to have more extensible solutions.

@jfischer-no No, shields are still intended for physical shields.
Snippets are intended for more logical configurations, like cdc-acm.
In fact we have discussed that shields are in many ways a specific subset of snippets, so question more becomes, if shields starts to support adding extra source files (using CMakeLists.txt) should snippet allow the same ?
Main risk with allowing this is the risk of misuse, as shields or snippets suddenly could be doing more than adding source files.

@jfischer-no also take a look at the related discussion referenced in this PR:

This came up in a dev-review discussion from this PR: https://github.com/zephyrproject-rtos/zephyr/pull/69696#issuecomment-2195201613

No, shield should only be used to configure and describe real hardware. Snippets are introduced to have more extensible solutions.

@jfischer-no No, shields are still intended for physical shields. Snippets are intended for more logical configurations, like cdc-acm.

What I said. Also, technically speaking, cdc-asm snippet could be a shield.

In fact we have discussed that shields are in many ways a specific subset of snippets, so question more becomes, if shields starts to support adding extra source files (using CMakeLists.txt) should snippet allow the same ?

Shields are certainly not a subset of snippets, something that has been around much longer and has more capabilities cannot be a subset of something that is very limited in current state :smile:. I do not recall this discussion. Was there one in Architecture WG? Where is RFC?

Main risk with allowing this is the risk of misuse, as shields or snippets suddenly could be doing more than adding source files.

Yes, and I do not want shields to contain source code.
Maybe snippets, https://github.com/zephyrproject-rtos/zephyr/pull/53640#discussion_r1065718569

Shields, also known as “add-on” or “daughter boards”, attach to a board to extend its features and services for easier and modularized prototyping.

Shields are just reusable extensions to boards. Boards can contain source code, so why not shields 🤷

I think whether or not it makes sense to put code in a shield folder depends on the case, something like a simple SYS_INIT to send some required spi commands to initialize a daughterboard seems like it would make sense to put here, for example (which is the motivation of this PR).

Shields, also known as “add-on” or “daughter boards”, attach to a board to extend its features and services for easier and modularized prototyping.

Shields are just reusable extensions to boards. Boards can contain source code, so why not shields 🤷

A board may contain arch/board-specific code, unfortunately the documentation does not specify which code and to what extent. About every second shield PR does not even follow the shield documentation. Allowing shields to contain arbitrary code has meant the end of Board compatibility and long, unproductive discussions in PR reviews. If there is a legitimate interest in allowing code in shields, then the shield documentation should specify exactly what code and to what extent, and that this code should not be board/arch/vendor specific. The proper way to do this would be to create an RFC, commit to update the shield documentation, and present it to the Architecture WG.

Well of course, just like we don't want to put board specific code in the soc folders, and don't want to put soc specific code in the arch implementations. The code for a shield should be generic to be useful for any consumer of the shield, I feel like it goes without saying?

I do like to have the option as well, also for OOT shields.
A reviewer should comment on other PRs that would abuse this option, but that doesn't mean it can't be useful.


Crash when use setStyleSheet this function in subthread.

I use another thread to change the color of the widgets['result'] ， but I found that the programmer will crash when I change the stylesteet of 'result' 20~120 times.
Maybe this is a multiprocessing problem, the another thread possibly casue some memory error.
Here is a example code:
from guietta import _, Gui
import threading
import time
gui = Gui(
    ['result']
)

def startWS():
    while True:
        gui.widgets['result'].setStyleSheet('color:rgb(255, 0, 0);')
        time.sleep(0.1)
        gui.widgets['result'].setStyleSheet('color:rgb(0, 176, 52);')
        time.sleep(0.1)
threading.Thread(target=startWS).start()

gui.run()

guietta is built on top of PyQt, and PyQt does not allow changing widget properties from threads. This is mentioned in the tutorial:
https://guietta.readthedocs.io/en/latest/tutorial.html#working-with-threads
There is also a decorator, gui.execute_in_main_thread   (see https://guietta.readthedocs.io/en/latest/reference.html ) that can be used to make sure that a function runs in the main thread and can update widgets.

Thanks a lot ! I have already solved the problem.


ReferenceError: Can't find variable: _UIManager

Description
Getting an error every time I try to run my app on the android and ios simulator. It'll work for one but error out when i try to navigate on the other. tried cleaning cache, reinstalling nodemodules, cleaning project, etc.
Version
0.67.3
Output of npx react-native info
System:
OS: macOS 12.0
CPU: (8) x64 Apple M1 Pro
Memory: 70.20 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
Yarn: Not Found
npm: 8.1.4 - ~/.nvm/versions/node/v16.13.1/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7784292
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: 11.0.10 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: ^17.0.2 => 17.0.2
react-native: ^0.67.3 => 0.67.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
npm run ios and npm run android:devdebug
will run on ios but errors out on android with the error message ReferenceError: Can't find variable: _UIManager
Snack, code example, screenshot, or link to a repository
No response

Facing this error while testing simultaneously on Android and iOS

Output of npx react-native info
System:
    OS: macOS 12.3
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 147.21 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.1.2 - ~/.nvm/versions/node/v16.13.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK:
      API Levels: 23, 27, 28, 29, 30, 31
      Build Tools: 29.0.2, 29.0.3, 30.0.0, 30.0.2, 30.0.3, 31.0.0, 32.0.0, 33.0.0
      System Images: android-26 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom, android-Tiramisu | Google APIs Intel x86 Atom_64
      Android NDK: Not Found
  IDEs:
    Android Studio: 2021.1 AI-211.7628.21.2111.8193401
    Xcode: 13.3/13E113 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.13 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.66.2 => 0.66.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Logs from JS
ReferenceError: Property '_UIManager' doesn't exist
  at _onFsPressForward_Pressability(app:///main.jsbundle:1:313479)
  at _performTransitionSideEffects(app:///main.jsbundle:1:312328)
  at _receiveSignal(app:///main.jsbundle:1:311833)
  at onResponderRelease(app:///main.jsbundle:1:311018)
  at apply(native)
  at invokeGuardedCallbackImpl(app:///main.jsbundle:1:127019)
  at apply(native)
  at invokeGuardedCallback(app:///main.jsbundle:1:126951)
  at apply(native)
  at invokeGuardedCallbackAndCatchFirstError(app:///main.jsbundle:1:127141)
  at executeDispatch(app:///main.jsbundle:1:127103)
  at executeDispatchesAndReleaseTopLevel(app:///main.jsbundle:1:130596)
  at forEach(native)
  at forEachAccumulated(app:///main.jsbundle:1:128709)
  at anonymous(app:///main.jsbundle:1:130953)
  at batchedUpdatesImpl(app:///main.jsbundle:1:186627)
  at batchedUpdates(app:///main.jsbundle:1:130513)
  at _receiveRootNodeIDEvent(app:///main.jsbundle:1:130790)
  at receiveTouches(app:///main.jsbundle:1:180700)
  at apply(native)
  at __callFunction(app:///main.jsbundle:1:117206)
  at anonymous(app:///main.jsbundle:1:115689)
  at __guard(app:///main.jsbundle:1:116652)
  at callFunctionReturnFlushedQueue(app:///main.jsbundle:1:115647)

Seeing the exact same issue, but it seems fullstory related. I also see you've fullstory issues in your error logs.

@justinairbase we removed full story and things are back to normal with no android issues. We are still investigating though, it did seem to be scoped to expo OTA updates.

Hi all,
RC here from FullStory's support engineering team. Sorry to hear about the trouble you're experiencing. If you haven't already please open a ticket with<EMAIL_ADDRESS>to work with our team to find a solution.
Thanks so much,
RC

Hey all, for anyone having trouble with FullStory please reach out to<EMAIL_ADDRESS>and we'll work with you to find a solution. Fwiw, if you're using Metro Server while running into this that might be the root cause. Currently we don't support working with Metro Server and generally when running into _UIManager errors, they're Metro related.

@rcmaples  I do not understand the "we don't support working with Metro Server" part. We are using this official react-native package for FullStory. Which bundler does FS support on react-native?


add missing tdb_multi_cursor.c into build

when using autoconf to build traildb, tdb_multi_cursor.c is not compiled because it is not exist in Makefile.am
so I change Makefile.am to include tdb_multi_cursor into library.
but ./waf doesn't have this problem.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage remained the same at 77.953% when pulling c850f3f9edb9e61ac24fe1f05917650c41751645 on charsyam:feature/add_tdb_multi_cursor_file into 053ed8e5d0301c792f3ee703cd9936c49ecf41a1 on traildb:master.

Coverage remained the same at 77.953% when pulling c850f3f9edb9e61ac24fe1f05917650c41751645 on charsyam:feature/add_tdb_multi_cursor_file into 053ed8e5d0301c792f3ee703cd9936c49ecf41a1 on traildb:master.


google login implemented

-added frontend google sign in
-added backend logic of google sign in using oath2
issue no #27
https://github.com/user-attachments/assets/dfc25353-36e3-4b7b-8087-a574104cb913

@17arindam nice

remove the labtest link in navbar and add a profile icon somewhere in the navbar, to go to the profile page

@Luson045 can i do this by making another issue?

okay, but fix the conflicts here

@Luson045 i have resolved conflicts


Add lazy load flag

Close #4268

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

Project coverage is 76.87%. Comparing base (22380a4) to head (d9583fa).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4298      +/-   ##
==========================================
- Coverage   81.73%   76.87%   -4.86%
==========================================
  Files         107      107
  Lines       51787    51820      +33
==========================================
- Hits        42328    39837    -2491
- Misses       9459    11983    +2524