Allegro support can be possible？

Very Interesting work!
Wondering if it can be integrated with Allegro network?
Thanks!

Definitely! I'm planning to add support for Allegro and MACE in the coming weeks -- although there are some other features that receive priority for now. However, given the fact that Allegro is effectively just an extension to NequIP, I actually think you can combine an Allegro-enabled config yaml with the existing NequIPModel implementation!
If it does not end up working, let me know and I'll take a closer look.

@svandenhaute Sounds great! Can't wait to have a try! Are there any instructions for user or I should wait for some time? Thanks!

Awesome, looking forward to your feedback.
Yes, I'm currently still working on getting the documentation set up with some basic examples included. Within the next week or so, you should be able to get started!

Wow that's great:) Thank you!

@Hongyu-yu I've implemented Allegro as a separate model in psiflow. It is rather picky in terms of torch compatibility (essentially requiring 1.11.0) which makes is mutually exclusive with MACE as that requires torch==1.12.1 or later.
In any case; I've started working on the documentation, and the examples have been streamlined as well. Do not hesitate to report back any problems -- psiflow is still in development and feedback is extremely important at this stage.

@svandenhaute Hi, thanks for the update. For the torch compatibility, I only use Allegro so it should be good. I will try to run the examples and give feedback:) Thanks for your work!

Hi, @svandenhaute
I have been trying to use psiflow recently and during the installation, I meet an issue with ase. For ase of 3.22.1, it goes wrong with
from ase.calculators.calculator import BaseCalculator

After updating ase of the latest version from https://github.com/rosswhitfield/ase, it works well.
May you check the version of ase that should be right?
Also, as a user of VASP, examples with vasp will help a lot if provided. Thanks!
BTW, congratulations to your new publication of psiflow in npj :)

I am anticipating ASE 3.23.0, in which they renamed Calculator to BaseCalculator. If you install the latest version from the ASE gitlab repository, you won't experience that error. In any case, I'll ensure psiflow also works with 3.22.1, the latest version on pypi.
We still have to write the VASP interface. Would you need it soon?

Yes. It will be convenient for me to do with VASP interface.

Hi, @svandenhaute I have been trying to use psiflow recently and during the installation, I meet an issue with ase. For ase of 3.22.1, it goes wrong with
from ase.calculators.calculator import BaseCalculator

After updating ase of the latest version from https://github.com/rosswhitfield/ase, it works well. May you check the version of ase that should be right?

This has now been fixed.

Yes. It will be convenient for me to do with VASP interface.

This is technically low-priority, but we will try to get it done in the coming weeks!

This is technically low-priority, but we will try to get it done in the coming weeks!

Thanks!

Closing this as completed since Allegro is now officially supported. See #3 for tracking progress on the VASP interface


Instructions in Readme.md on how to include HF are incorrect

Instructions in Readme.md on how to include HF are incorrect.
I ran ./gradlew --refresh-dependencies -Dorg.gradle.jvmargs=-Xmx2G setupDecompWorkspace
This is the console log.
And this is my build.gradle.

Wrong repositories block. You put it in the Buildscript repositories, not the project repositories.

Yup, just like TehNut said :)


Fix memory leak due to unallocated tensor data

Fixes https://github.com/onnx/onnx-mlir/issues/2171. The Java OMTensor object was not taking full ownership duties of the backing data within the native OMTensor object, leading to a memory leak. This fix solves the issue by allocating a new ByteBuffer in the JNI OMTensor constructor with a copy, leaving the original data to be freed along with the jni OMTensor.
Some initial timings suggest that there isn't any meaningful difference in timing due to the copy in the YOLOv3-12 and MNIST models.

Can one of the admins verify this patch?

We made a lot of effort trying not to copy the input tensor data so I prefer we don't take the copy route as much as possible. In fact, I remember a while ago @AlexandreEichenberger said onnxruntime claimed that one of their advantages is that they don't copy the tensor data and asked if we can make the same claim and I told him yes. So we should keep that promise as much as possible. As mentioned in the slack conversation, I'm working on a solution for Java 9+ using the cleaner API that does not require copy. For Java 8, we will have to use the finalize() to avoid copy. This does have the unfortunate consequence that we will have to maintain two implementations. But the code involved is relatively simple and I'm hopeful that WMLz will eventually upgrade from Java 8.

I'm all for avoiding the extra copy if we can. However as a side note, don't we already do a copy whenever someone tries to call the get*Data() like at https://github.com/onnx/onnx-mlir/blob/7e49205dcd6c818bc92aba0abf13280f719fda54/src/Runtime/jni/src/com/ibm/onnxmlir/OMTensor.java#L392-L395?
If we use finalize()/Cleaner, do we need to do anything about https://github.com/onnx/onnx-mlir/pull/2174#issuecomment-1513486700? Do we even allow changing the data after it's initially set? If we don't support that, we could make it explicitly a one time write (ie throw an exception in setData if _data is not null) then we'll know we're safe.
I have no idea when WMLz will drop Java 8 support but it'll be Sept 2026 at the latest since that's when IBM's Java goes out of support for zOS (https://www.ibm.com/support/pages/java-sdk-lifecycle-dates)

Yes the fb.get(f) copy is unavoidable (at least I don't know how) since Java cannot simply convert a FloatBuffer into float[]. So we are trying to avoid a 2nd copy. And sadly, our claim of not copying tensor data currently only applies to C/C++ case.
We do have public setXXXData to allow changing tensor data. The issue mentioned in https://github.com/onnx/onnx-mlir/pull/2174#issuecomment-1513486700 isn't too hard to deal with. Basically, with our internal flag, we know if the data is backed by native buffer generated by model runtime. We just need to make the JNI call to free it before setting the new data.
And some good news and bad news from my testing so far. Good news is that finalize does appear to fix the leak for Java 8 (it does NOT work for Java 11 if I just ignore the compile time warning and the Leaker still leaks). Bad news is that cleaner doesn't appear to be working. The RES size of the Leaker still creeps up although very very slowly. I ran it for about an hour and it creeps up from roughly 50MB to 70MB, whereas the finalize-fixed version hovers around 53MB. So I must be doing something wrong with my cleaner implementation.

If we use finalize()/Cleaner, do we need to do anything about #2174 (comment)?

It'll be a minor bit of tweaking to ensure that all of the setters deallocate the data, if necessary. @gongsu832 there is a third option to support all releases. Utilizing a combination of WeakReference, RefrenceQueue and a cleanup task, one could approximate the Cleanup functionality and ensure it is compatible on Java8+.

I was looking into that but it appears to be overly complicated for what we want to do. And I seem to recall reading that things on the WeakReference (or is it Phantom Reference, can't remember) has changed between 8 and 9. So it doesn't inspire a lot of confidence for me that the same code will work for both 8 and 9+. I could be wrong though.

Not that watching top RES is a perfect measure, but here's what I got on a 120min mnist-12 loop run on LoZ Java 11:
For this bytbuffer branch version, the first 30 sec RES fluctuates and stabilizes at 58.1m. Between min 4-7 RES moves up to 61.5m. It stays stable until min 47 where it goes to 62.5m. After that RES doesn't move again and is still there at the 120min mark. In the end, mainGraph ran 1,951,483 times.
I did a similar test for the "reaper" version and the results here appear slightly better. Although it's hard to say how much of the RES difference is the the code vs JVM doing it's thing.

@gongsu832 So which version of the fix do you want to see go in? I'd like to start rallying around one so we can merge in a fix early next week if possible. I lean towards bytebuffer as it seems the simplist codewise and the most stable from my RES testing. However I'm not really set one way or the other other than the fix working for 1.8 and 11+

As I mentioned in the other PR https://github.com/onnx/onnx-mlir/pull/2182#issuecomment-1515631129, if we really don't want to have two separate versions for 8 and 11, then my vote would go to the copy fix in this PR for now. I'm still testing my cleaner fix to make sure it works correctly but only for the purpose of getting it ready so that when WLMz finally upgrades from Java 8 we can then switch to the cleaner fix.

Jenkins Linux amd64 Build #10782 [push] Fix memory leak due to u... started at 11:04

Jenkins Linux s390x Build #10792 [push] Fix memory leak due to u... started at 12:04

Jenkins Linux ppc64le Build #9839 [push] Fix memory leak due to u... started at 12:08

Jenkins Linux amd64 Build #10782 [push] Fix memory leak due to u... passed after 1 hr 17 min

Jenkins Linux s390x Build #10792 [push] Fix memory leak due to u... passed after 1 hr 29 min

Jenkins Linux ppc64le Build #9839 [push] Fix memory leak due to u... passed after 1 hr 52 min


Error fatal on line 49

When including this library on older versions of PHP (5.3.29), there is an error in line 49:
            \Dshafik\MySQL::$connections[$hash] = ['refcount' => 1, 'conn' => $conn];

The "[" to create an array is not allowed in this version. If you change it to the right "array(" such as:
            \Dshafik\MySQL::$connections[$hash] = array('refcount' => 1, 'conn' => $conn);

Everything work as intended (i.e., the code does nothing).
You must also make the same change in lines 77, 337, 628, 721 - 734, 736 and 748-781.

The lib doesnt support 5.3 right now

This library is for 5.6+ as stated in the README. I will amend the composer.json to reflect this.
Thanks for taking the time to report this issue.


NIFI-1488 fixes

Hmm... not sure why the previous commit got folded into mine.

Attached patch to the JIRA issue.  Will let this PR run to confirm successful build.


Proposal: Kubernetes wrapper

Similar to: https://github.com/spotify/luigi/blob/master/luigi/contrib/ecs.py
I think it should be possible create a wrapper using the kubernetes api. Either by creating a Pod or a Job and monitoring his status.

Feel free to send a pull request :)


add redis as centralised cache store

I am planning to use makara gem for rails application. I want to use redis as my cache store that will be shared between multiple server running the application.
the commands supported by makara cache are read and write. How can we add support for redis.

You would create a redis adapter and set it up in an initializer.
#config/initializers/makara.rb
class RedisAdapter
  def read(key)
    $redis.get(key)
  end
  def write(key, value, ttl)
    $redis.setex(key, ttl, value)
  end
end

::Makara::Cache.store = RedisAdapter.new


An unsupported batteryPowerLevel: location was provided

Here, it seems as if the parsed fields are mismatched:
2024-10-12T07:20:26+0000:INFO:vwsfriend_base:Updating data from WeConnect
2024-10-12T07:20:32+0000:WARNING:addressable:/vehicles/<FIN>/domains/readiness/readinessStatus/connectionState/batteryPowerLevel: An unsupported batteryPowerLevel: location was provided, known values are [comfort, emergency, off, unknown battery power level] please report this as a bug

Will be added in the next release


README and .env.example additions

@jv3nk @atlasctz @akostevich-atlassian please let me know what you think of the README and if you think it could be better organised or anything is missing.


[PL-52124] Add a Subscriptions section to KB FAQ

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: Add a Subscriptions section to KB FAQ
Jira/GitHub Issue numbers (if any): PL-52124
Preview links/images (Internal contributors only): building

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6690196145e0f11b8f4f22c3--harness-developer.netlify.app


Fix: Error on README generation if no objects in package

Fix: Error on README generation if no objects in package
Fix: Added npm ci for scratch org creation workflows


feat: allow to send a custom function to driver

Pull Request Test Coverage Report for Build 752515414

0 of 23 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 17.938%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.js
0
23
0.0%

Totals

Change from base Build 751157444:
-0.03%

Covered Lines:
2030

Relevant Lines:
11607

💛  - Coveralls


Mask Detection

Pull Request Template
script name - mask_detection
Brief about script
Implement a Mask detector that accepts feed from the live camera that determines whether a person has to wear a mask or not. It will send alert audio if the person did not wear a mask using open cv, python, and deep learning
Issue no.(must) - #540
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
[x] Just including required dependencies in requirements.txt (Don't include Python version too)

Please fix the linting

Please fix the linting

I have fixed it but Did not know why the checks are failing

Duplicate of #216

@pawangeek Can I work on another issue?


Better ui using bootstrap

Related to #3

Looks good to me, only thing missing would be to have some way of redirecting to a stream of a given language.
The changes that allow for having different language streams are on the staging branch, could you rebase onto it (and change the target of this PR)?


Major missing dimensions update

Forty missing dimensions added, some misplaced data removed, and ordering and headers modified to match the order in which they are displayed in CQD.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Org check failed recomplie apex classes on "batch tab"

Describe the bug
i clicked the recompile all classes (411) on batch tab and immediately the issue appeared
To Reproduce
Copy and past the information from the error dialog box here!!
Please go here and log an issue with the following information.
Salesforce Information
Salesforce API Version: 60
Installed on OrgId: 00DKN0000008aJK
Type of this org: Sandbox
Current running UserId:<PHONE_NUMBER>1r17JAAQ
Current Daily Api Requests: {"apiUsage":{"used":184811,"limit":32100000}}
Navigation Information
Page: /apex/OrgCheck_Batches_VFP
System Information
Language: en-US
Issue #0: undefined
Issue #1: undefined

Thanks for your ticket.
The new version Beryllium is going to resolve that issue.
It is going to be available on the AppExchange very very soon!

great news, thanks


Is there a way to identify that one is in a poetry shell after entering it?

[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I have searched the documentation and believe that my question is not covered.

Question

I was looking through the documentation and the issues, and even skimmed through the code some and could not find an answer to this. Out of the box if I install Poetry (either using curl or pip install) and then run poetry install, the shell that gets created doesn't have any sort of identifier indicating that I'm in the shell, e.g.
$ poetry install
$ poetry shell
Spawning shell within <snip>/Library/Caches/pypoetry/virtualenvs/myproject-py3.7
bash-3.2$

On the other hand, pipenv prefixes the shell with the name of the project, e.g.
$ pipenv install
$ pipenv shell
Launching subshell in virtual environment…
(myproject) bash-3.2$

Am I missing something?

Duplicate of #571.

@oakkitten thanks, but am I correct in my understanding that everything suggested in that thread seems to be an extra step that the individual installing poetry is responsible for adding? It wasn't clear when I was looking through it whether or not the lack of a shell name prefix was actually being fixed as part of poetry.

That issue is still open so it's a bug that hopefully will be eventually fixed; if you run the workaround as suggested by the top comment you should get the prefix in your shell.
The prefix is just a visual cue, anyway. You can see whether you are using the correct env by running which python

Thanks for the explanation @oakkitten! Even if the prefix is just a visual cue it is useful. I'll close this issue seeing as it is covered by #571.


Variables in Kotlin: Android Basics with Compose

URL of codelab
In which task and step of the codelab can this issue be found?
Describe the problem
Phone number of the sender (String)
It should be integer
Phone number of the sender (Int)
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @Rohit18112001
Thank you for writing in with your concern.
The team decided to use "String" as the field might contain non-numeric characters such as "()", "-",  or "+" to separate the digits and/or to indicate an international phone number.
Hope this helps and that you are enjoying the course!
Regards.


Merge users instead of deleting

At startup, instead of deleting user and adding new one, this PR is merging unexisting keys from old user to the new one before deleting it.
This permits to not lose stuff added to users. Like a lot of scripts do.

@ys, many, many thanks for your patch! Several people at https://github.com/hubot-scripts/hubot-auth/issues/14 and https://github.com/hipchat/hubot-hipchat/issues/213 are trying to figure out what's wrong.
I can confirm hubot-hipchat was removing roles assigned to users after connecting to hipchat, and the problem is fixed with this patch.
@mikehale @rbergman any chance one of you could update the package on npmjs? It would save many people quite some time I think! Thanks a lot!

@yannh heh, I just commented on this PR, I don't actually have access to npmjs.

I'd love to get the npm package updated, but have been really short on time to give this project much love, and feel like I need to successfully test the current state of master before rolling a new version out.  I'll see if I can find time for it soon, but it might take some time to get it done yet.


[Process Agent] Remove AgentConfig.Enabled

What does this PR do?

Removes AgentConfig.Enabled
Motivation

This PR is part of deprecating AgentConfig as part of the config refactor RFC
Additional Notes

This PR includes a fix for the container check being disabled if containers are unreachable.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Using this config file:
process_config:
  process_collection:
    enabled: false
  container_collection:
    enabled: false
  process_discovery:
    enabled: false

Make sure that the process agent starts up and then exits with this message:
Set env var DD_PROCESS_CONFIG_PROCESS_COLLECTION_ENABLED=true or add
process_config:
  process_collection:
    enabled: true
to your datadog.yaml file.

Also, make sure that process_config.enabled still works as expected.
Additionally, make sure that container checks are disabled if containers are not available. Keep in mind to test this you will have to either use inv process-agent.build or build the process agent with -tags "docker"
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

This commit fixes failing tests due to a 3 year old bug where container checks are not disabled if containers are unreachable. We move canAccessContainers to LoadProcessYamlConfig which is why so many files are being changed in that commit

DD_PROCESS_CONFIG_PROCESS_COLLECTION_ENABLED="true" env variable not working on windows servers
process agent is running successfully without DD_PROCESS_AGENT_ENABLED="true" env variable, API key is valid, everything fine with configuraiton.
live process is not showing up with env varible DD_PROCESS_CONFIG_PROCESS_COLLECTION_ENABLED="true"
but manually by editing the datadog.yml file working.
i need to do with powershell on windows server. please help me out.


Add a refresh command that rebuilds the snapshot and reloads it on th…

…e device
This provides a fast way to iterate on changes to a Flutter app that only
involve updates to Dart code and do not require a full build and install
of the FLX and APK

Can we do this automatically somehow? e.g. track the timestamps on our dependencies and if nothing changed, just do this instead?
LGTM

Eventually the flutter command will probably adopt the features of a full-fledged build system, but today we don't have robust dependency tracking.
This command is for the common scenario where you have run flutter start and deployed an APK to the target device, and you want to roll out new Dart code but haven't changed anything else in the package (resources, services, etc.)
In particular, this avoids the cost of building a full FLX and copying it to the device.


[Adding Tool]: React icons

What tool are you adding and to what category?
We need to add react icons . It contains popular icons which are largely used in react projects .It utilizes ES6 imports that allows you to include only the icons that your project is using.
Add Tool Link
Add tool link: https://react-icons.github.io/react-icons/
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] The tool is free for use

@JasonDsouza212  I have opened this issue because the previous one got closed soon after i got  the message that the pr title is wrong. Please assign this one so that I can make a correct pull request


x/pkgsite: examples missing playground links

What is the URL of the page with the issue?
https://pkg.go.dev/regexp?tab=doc#example-MatchString
What is your user agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.66 Safari/537.36

Screenshot

What did you do?
Visited the link above to view the example.
What did you expect to see?
Expected to see a link that takes me to the example in the Go Playground, like godoc.org

What did you see instead?
No link.

Yes thanks, I came here looking for this. If you visit the current site:
https://golang.org/pkg/io#Copy
You get complete example, with package main and everything. The example itself
is even editable without leaving the page, which is nice. You can run the
exmaple from the same page, or hit "Share" and it takes you to:
https://play.golang.org
Now if you visit the same page with the new site:
https://pkg.go.dev/io#Copy
You dont get a complete example, everything is stripped except for pretty much
the function call itself. The example is not editable. The example is not
runnable. No "Share" button is available to test with the playground. Overall
I would say its a worse experience, and I wont be updating my links.
I get that some might want to clean things up and remove cruft, but at a minimum
the playground links should be put back in. For myself, full, complete examples
are a critical part of the learning process. Please put them back in.

https://github.com/golang/go/issues/37819

This feature is now code complete and will be available soon on pkg.go.dev.The output should look like this:

This will work for all playable examples on pkg.go.dev.
If you would like to test it out locally, you can do so by:
Running go run ./devtools/cmd/experiment create executable-examples to enable the experiment flag
Setting this CSS class to display: flex;

https://github.com/golang/go/issues/41403

Why was this issue closed? Has it been fixed? Please specify when closing.

@mvdan have you checked? I do agree with you - but if you check playground links have been added to the new site

Why was this issue closed? Has it been fixed and deployed? Please specify when closing.

Sorry about that! 😅 As @nu8 mentioned The "play" button is deployed now and should appear in packages with examples.


Update and rename README.md to index.md

Why:
Closes [issue link]
What's being changed:

Check off the following:

[x] For workflow changes, I have verified the Actions workflows function as expected.
[ ] For content changes, I have reviewed the style guide.

updating README.md


Update singleton class locs on fast path

Motivation

The fast path would always set the loc for a class symbol to the
declLoc of the class node in the AST. But it would not also update the
singleton class's loc. This means that the old singleton class loc was
effectively an offset into a buffer state that didn't exist anymore. At
best this would show a red squiggle in a stale location and at worse it
would cause a crash.
We saw plenty of these crashes in practice, and multiple recent changes
landed in Loc.cc in an attempt to collect more logs to help us identify
where the problem was coming from. But because the crash manifests far
from where the bug happens (in error printing code, instead of in namer
code where the bug is), it was hard to use the logs to figure out what
had happened.
The test added in this test crashes with an ENFORCE failure in debug
builds and in a release/non-ENFORCE'ing build it trips the log lines
that we had added. It's possible that there are still other places that
exhibit other similar problems, so I'm going to keep the logs in place
for the time being, and circle back after observing the crash reports
for a while. Hopefully we can remove them finally.
Test plan

See included automated tests.

Current dependencies on/for this PR:

master

PR #6447   👈

PR #6422

This comment was auto-generated by Graphite.


Checkout Email Address Component

Guest or Account Email Display Component
Overview / Summary
Create a component that will display the email address associated with an order, either the guest email or if signed in, the account email address.
Expected use cases
This component will be used whenever displaying the customer's email address in the checkout
Where should this component live
This component should be created in the component library.
Images

// Pseudo-markup

  Signed in as

<EMAIL_ADDRESS>

Data
Props
propTypes: {
  emailAddress: PropTypes.string,
  isAccountEmail: PropTypes.bool
}

Render Criteria

This component should be rendered on the checkout page above the checkout actions when there is an email address associated with the account or cart

UI States
Normal
As a guest the email address should be rendered.
https://app.zeplin.io/project/5afc82f5e8d73e250ff9d855/screen/5b02f4fa90d3b184213318da
If the email address is associated with an account, the email address should be prefixed with "Signed in as "
https://app.zeplin.io/project/5afc82f5e8d73e250ff9d855/screen/5b02f470eefa9c371dd3c488
Empty
If the email address does not exist, this component should not be rendered.

:tada: This issue has been resolved in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


About upload

WARNING  Error during upload. Retry with the --verbose option for more details.
ERROR    HTTPError: 403 Forbidden from https://upload.pypi.org/legacy/
         The user '***' isn't allowed to upload to project 'tkterm'. See
         https://pypi.org/help/#project-name for more information.

Failed to upload. Maybe our project has the same name.

The badest way to solve the problem: rename our project in setup.py

Yeah, thats a slight issue. tkterminal is also taken so maybe tktermwidget? Its an ugly name for sure but there are not many options.

I just change it name to tktermwidget and I test it locally.
It shows me that:
C:\Program Files\Python39\lib\distutils\dist.py:274: UserWarning: Unknown distribution option: 'long_description_content_type'

And when I upload:
WARNING  Error during upload. Retry with the --verbose option for more details.
ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
         The description failed to render in the default format of reStructuredText. See
         https://pypi.org/help/#description-content-type for more information.

Almost drives me mad.

Why it still upload to tkterm?

Still failed upload now...
pypi github action (v0.0.2 upload)

The readme option is having an issue, I'll look into it tomorrow.

So, what happened to the readme option...?

I'm not sure to be honest.

IDK why it will say:
UserWarning: Unknown distribution option: 'long_description_content_type'

found it:
PS C:\Users\Administrator\Downloads\TkTerminal-main\TkTerminal-main> python setup.py sdist
C:\Program Files\Python39\lib\distutils\dist.py:274: UserWarning: Unknown distribution option: 'long_description_content_type'
  warnings.warn(msg)
running sdist
running check
warning: check: missing meta-data: if 'author' supplied, 'author_email' should be supplied too

warning: sdist: manifest template 'MANIFEST.in' does not exist (using default file list)

warning: sdist: standard file not found: should have one of README, README.txt, README.rst

writing manifest file 'MANIFEST'
creating tktermwidget-0.0.2
creating tktermwidget-0.0.2\tktermwidget
making hard links in tktermwidget-0.0.2...
hard linking setup.py -> tktermwidget-0.0.2
hard linking tktermwidget\__init__.py -> tktermwidget-0.0.2\tktermwidget
hard linking tktermwidget\tkterm.py -> tktermwidget-0.0.2\tktermwidget
Creating tar archive
removing 'tktermwidget-0.0.2' (and everything under it)
PS C:\Users\Administrator\Downloads\TkTerminal-main\TkTerminal-main>

What is sdist?

IDK why when I use bdist_wheel command to setup, it say there is no command named bdist_wheel, so I have to use this.

successed now, never mind about it.


Fix GH Actions cache

Updated:
There is a continue-on-error option in the GH Actions cache step. That should fix the issue of the the build failing if the cache-hit conditional fails. See: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#using-the-output-of-the-cache-action

Original:
Reading the example workflow, just using the syntax they use verbatim. I wouldn't be surprised if a cache miss does not make the cache-hit output be false 🤷

hmm reading through the docs more, though, I do see this:
      - if: ${{ steps.cache-npm.outputs.cache-hit == 'false' }}

https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#example-using-the-cache-action

Here's proof 🤦 https://github.com/talaia-labs/rust-teos/runs/7231503370?check_suite_focus=true

hmm reading through the docs more, though, I do see this:
      - if: ${{ steps.cache-npm.outputs.cache-hit == 'false' }}

https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#example-using-the-cache-action
Ah, might be missing this option: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#using-the-output-of-the-cache-action
continue-on-error: true

That makes more sense. Let me push that change up.

May it be then only necessary to add the continue-on-error?

Yeah, I think the following are actually equivalent:
if: ${{ steps.cache-cln.outputs.cache-hit == 'false' }}
if: steps.cache-cln.outputs.cache-hit != 'true'

So it's just a syntactic preference between the two (I personally have no preference either way).
I guess the change to != 'true' worked initially because it was just restoring the cache, and not creating a new one? Either way, happy to rebase this PR.

Sure, no rush at all. I'll work around it for now.
Thanks for taking care of this 🚀

So the output is true on cache hit, and nothing (not false) in cache miss.

Classic :) glad it's resolved!


ignore meta key events in keyDown handler

fix: ignore meta key events in keyDown handler
On macOS the meta key is often used for commands. Thus if doing an action such as undo the slider will not update until the cmd key is released.

Can you update the commit message to follow conventional commits guidelines.

Updated!

Sorry I thought you meant the PR description. The commit message should be good now.

<EMAIL_ADDRESS>

Thanks @zschreur for the contribution!

Thank you!


downgrade from 1.10 to 1.9 makes images go unnamed

BUG REPORT INFORMATION
Output of docker version:
Client:
Version:      1.10.3
API version:  1.22
Go version:   go1.4.2 gccgo (SUSE Linux) 5.2.1 20150721 [gcc-5-branch revision 226027]
Git commit:   9e83765
Built:
OS/Arch:      linux/ppc64le
Server:
Version:      1.10.3
API version:  1.22
Go version:   go1.4.2 gccgo (SUSE Linux) 5.2.1 20150721 [gcc-5-branch revision 226027]
Git commit:   9e83765
Built:
OS/Arch:      linux/ppc64le
Output of docker info:
Containers: 0
Running: 0
Paused: 0
Stopped: 0
Images: 6
Server Version: 1.10.3
Storage Driver: btrfs
Build Version: Btrfs v4.1.2+20151002
Library Version: 101
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
Volume: local
Network: null bridge host
Kernel Version: 3.12.51-60.20-default
Operating System: SUSE Linux Enterprise Server 12 SP1
OSType: linux
Architecture: ppc64le
CPUs: 8
Total Memory: 3.711 GiB
Name: ps64vt1093
ID: BG5I:42OA:3WKA:6IFH:Z5RE:WTMJ:UVHF:YIVB:4QJ4:DOE6:FVEY:4WO5
WARNING: No swap limit support
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:

Install docker 1.9.1
pull any image
Install docker 1.10.3
Install docker 1.9.1
pull another image
Install docker 1.10.3
run "docker images"

Describe the results you received:
You'll see the image pulled on 5 has no name.
Describe the results you expected:
the image keeps its name
Additional information you deem important (e.g. issue happens only occasionally):

Yes, this is expected; docker 1.10 has a new content-addressable store, and migrates all images the first time docker is started after upgrading. Once upgraded, this process will no longer be performed, so the images you downloaded after downgrading to 1.9.1 are not migrated when 1.10.3 is started, and won't be visible. Read about the migration to content-addressable store in this wiki; https://github.com/docker/docker/wiki/Engine-v1.10.0-content-addressability-migration
Note that we have never supported downgrading to older versions, although with other versions downgrading may have been less problematic and "worked" in many cases.
I'll close this issue, because this is expected, but feel free to comment here after I've closed


The requested module<EMAIL_ADDRESS>does not provide an export named 'Polyfill'

Describe the bug
With<EMAIL_ADDRESS>pixi.js working fine, but with<EMAIL_ADDRESS>it's not working.
Reproduction
src/main.js
import { createApp } from 'vue'
import App from './App.vue'
import './index.css'
import * as PIXI from 'pixi.js'

createApp(App).mount('#app')

package.json
{
  "name": "pixi-vite",
  "version": "0.0.0",
  "scripts": {
    "dev": "vite",
    "build": "vite build"
  },
  "dependencies": {
    "pixi.js": "^5.3.0",
    "vue": "^3.0.0-beta.15"
  },
  "devDependencies": {
    "@vue/compiler-sfc": "^3.0.0-beta.15",
    "vite": "^1.0.0-beta.11"
  }
}

System Info

required vite version: 1.0.0-beta.11
required Operating System: osx
required Node version: v14.3.0

Logs (Optional if provided reproduction)
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'Polyfill'

Because es6-promise-polyfill is not es module.You should add blew code inside vite.config.js.
optimizeDeps: {
    include: ['es6-promise-polyfill']
  }

Other @pixi/polyfill/lib/polyfill.es.js has error import with es6-promise-polyfill.
import { Polyfill } from 'es6-promise-polyfill';
// should be blew code.
import Polyfill  from 'es6-promise-polyfill';

Note: I also met the similar error with the pixi.js deps.You can though this way to resolve it and open a issue  inside corresponding repo.

I think I'm having a similar issue on #651
How can I check the esBuild output easily?


Fixes misspelled words

Hi. I was reading through your fantastic JavaScript Coding Style Guide and noticed a few misspelled words. I've provided my suggested changes.
Thanks,
Sean

You're welcome!


Adding ability to pass in CCID + molecule test

Closes #141
This PR introduces the FalconCid parameter to the powershell install script. It also adds a test case to ensure we have coverage in molecule.

CI run in forked repo:
https://github.com/carlosmmatos/falcon-scripts/actions/runs/4394746449/jobs/7696168633


Upgrade<EMAIL_ADDRESS>

Fixes https://github.com/atom/atom/issues/19480
Refs https://github.com/atom/find-and-replace/pull/1090
This pull request upgrades find-and-replace in an attempt to fix a couple of flaky tests that were causing CI to fail quite consistently. More details on the issue and pull request above.
I'll try building this 3 times and, if those tests don't fail anymore, I will go ahead and merge it.

Alright, it seems like this passed 3 times out of 3, so I'll go ahead and merge.


Can not open node template modal when create cluster

rancher/rancher:master 12/10
Steps:

Create a cluster
Select a node driver
Click add template

Results:
Can not open the add node template view

Version - 2.2 master
Verified fixed - I was never able to reproduce this.


Remove extra column in table header

Seems like the top example has 4 columns in the header and one in the body and doesn't render right.

Nice catch! I did a refactor of the columns and I miss it. Thanks!


Version inconsistencies, always pointing at ts-jest 26

Hello,
I'm trying to use this package with Jest 27. According to https://github.com/bencompton/jest-cucumber/pull/132 that should be supported.
I installed version 3.0.2 and looking at the package.json that came with that, it's still stuck on 26.1.4
{
  "name": "jest-cucumber",
  "version": "3.0.2",
  "description": "Execute Gherkin scenarios in Jest",
  "main": "dist/src/index.js",
  "typings": "dist/src/index.d.ts",
  "scripts": {
    "build": "tsc",
    "jest": "jest --verbose",
    "test": "npm run build & npm run lint & jest --color",
    "lint": "tslint --project ./"
  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/bencompton/jest-cucumber.git"
  },
  "keywords": [
    "jest",
    "gherkin",
    "cucumber"
  ],
  "author": "Ben Compton",
  "license": "MIT",
  "bugs": {
    "url": "https://github.com/bencompton/jest-cucumber/issues"
  },
  "homepage": "https://github.com/bencompton/jest-cucumber#readme",
  "devDependencies": {
    "babel-jest": "^24.1.0",
    "ts-jest": "^26.1.4",
    "tslint": "^5.10.0",
    "typescript": "^3.3.3"
  },
  "dependencies": {
    "@cucumber/gherkin": "^17.0.0",
    "@types/glob": "^7.1.3",
    "@types/jest": "^26.0.7",
    "@types/node": "^11.9.4",
    "@types/uuid": "^8.3.0",
    "callsites": "^3.0.0",
    "glob": "^7.1.6",
    "jest": "^26.1.0"
  },
  "jest": {
    "transform": {
      "^.+\\.tsx?$": "ts-jest"
    },
    "testMatch": [
      "**/*.steps.ts"
    ],
    "moduleFileExtensions": [
      "js",
      "ts",
      "tsx"
    ]
  }
}

Installing via npm by default gives me version 3.0.1 while 3.0.2 is the latest, perhaps because it's tagged as next?
https://www.npmjs.com/package/jest-cucumber/v/3.0.2?activeTab=versions

@bencompton I see this issue could have been resolved by https://github.com/bencompton/jest-cucumber/pull/154
I'm wondering, are you still committed to this project? One of our projects that utilizes jest-cucumber with great succes is starting to fall behind on depedency versions due to just-cucumber. Because of this we cannot really use this anymore on new projects, which is really a shame.


[app] decouple CASESession from OperationalDevice

Problem
There is an CASESession object in each OperationalDevice. The CASESession object is large and won't be used after authentication. For device-to-device communication such as binding, the object size will be a concern.
Change overview
This change decouples CASESession from OperationalDevice. Now the device will dynamically allocate from a session pool and release it after being authenticated.
Testing
Tested with chip-tool pairing and chip-tool onoff toggle

@gjc13 What are the next steps here? Agree the magic number of 2 seems not applicable for many SDK clients.

@gjc13 What are the next steps here? Agree the magic number of 2 seems not applicable for many SDK clients.

Was busy with internal stuffs, I'll make the number of outbound clients configurable.

/rebase

@pan-apple @bzbarsky-apple ?

@msandstedt PTAL


services.kresd.listenPlain says it supports ListenStream syntax from systemd.socket but doesn't

Describe the bug

Name: services.kresd.listenPlain
Description: What addresses and ports the server should listen on. For detailed syntax see ListenStream in man systemd.socket.

However, the following input is is rejected as "services.kresd.*: incorrect address specification":
services.kresd.listenPlain = [ "[fe80::1]:53%ofborg" ]

Expected behavior
I expected kresd to listen on fe80::1 on port 53 on the ofborg interface, as the documentation suggests should be possible.
I don't mind that it is impossible, but the documentation for this option isn't correct anymore.
Notify maintainers
@vcunat
nixos-unstable-small, rev 18c84ea816348e2a098390101b92d1e39a9dbd45

Well, it no longer uses systemd sockets anymore, and instead there are a few in-nix regexps that converts them.  It might not be perfect.
Either way, the inteface spec should surely be part of the IPv6 already, i.e. "[fe80::1%ofborg]:53" – and I think that should work (the generated config looks OK).

Ah, I see that systemd is weird here, at least in docs.  So I don't know.  Either shuffle it in the in-nix regexps or document as a deviation, I suppose.


init parallel batch processing

Handle parallel batch request
Divide RPCRequest as RpcRequestHandler & RpcRequest
Introudce waitgroup & go routine to handle the batch request in parallel

Please rebase against master.

@metachris  Thank you for the great review.

Nice, thanks 👍

LGTM 👍

@metachris squashed to single commit.

Awesome, thanks 👍


Why set request timeout just for OpenAI?

Hey @Dominastorm,
Why set the request timeout for just openai?
https://github.com/uptrain-ai/uptrain/blob/a663b2787a0bba54d4685b35a21e8774a7453288/uptrain/operators/language/llm.py#L54

Thanks for raising the issue. Do you have the list of error codes raised by LiteLLM for request timeout?

We raise openai-compatible error codes - here's the timeout one
https://github.com/BerriAI/litellm/blob/c59021d090000e639084737c3d88b93d32e8b126/litellm/exceptions.py#L74

Thanks, and do the this set of errors get raised for all LLMs - Anthropic, Azure, etc.?


[BUG]

Bug description:

How to reproduce:

Expected behavior:

Errors and logs:

[be sure to surround verbatim log lines with "~~~" lines]

Environment:

OS:
OS version:
TSDuck full version:
Installation type:

Additional information:

Probably not an intended issue creation.


Adds docs for Writing Office Hours

@jpotts244 thanks for the push on this, and sorry it took so long to get to! Let me know what I can clarify here.

@ashfurrow - this is great! Thanks so much for sharing context and for taking the time to write this up!


Center on object only if center property set

Currently map centers on every object drawing.
This eliminates the last map center save option....
normal use case is draw all objects after 'connect'  event, map center restore however is done on map object initialization.
This change centers the map on new object only if 'center' property is added to the object properties.

I think this is confused - the map only centres if the payload.command.lat and .lon are set... It won't centre for normal points being added ( payload.lat and .lon ) - if you don't want it to center don't add lat and lon to the command.

Agree, thanks


Use semicolon as an optional field separator for records

Example:
record point
    x: float; y: float
end

Codecov Report

Merging #100 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   91.45%   91.45%
=======================================
  Files          11       11
  Lines        2117     2117
=======================================
  Hits         1936     1936
  Misses        181      181

Impacted Files
Coverage Δ

titan-compiler/parser.lua
95.61% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This also be noted in the documentation (at the very least in the BNF section)

Done both changes, can someone verify and merge if everything is ok?

Done! :)


support field errors, fixed styles for input and select when invalid

PR Checklist
Check these before submitting your pull request:

[x] Visual and behavioral components are separated
[x] Exported components are documented with examples
[x] Props have JSDoc comments
[x] All relevant visual sub-components can be overridden via props

Changes to Existing Components

Added support for invalid <Select> components
Added support for errors in <Field>
Fixed invalid style for <Input> when required

What do you think about coordinating this and #137 for a patch release after the current one?

Works for me.

Ok, #137 was merged, let me know if you need any help reconciling the changes.


add support in ThingML to create a make file with includes for compiler ( '-I' option) for the C++ and C generators

should be supported if we want to use third party libs. It is better then carrying along all H files every time

@Lyadis what is the status here?

It is fixed.
You can use the annotation @add_c_libraries "myLib" on a thing. It will add -l myLib to the makefile.
You can also use @add_c_libraries_rep "path/to/my/rep" to indicate a path to include for .so (it will add -L path/to/my/rep to the makefile)  and @add_c_flags "myflag" to add flags, and @add_c_directives "myDirective" to add -D myDirective.


Clearly delimit which examples work with OpenCyc

From @nwinant on March 3, 2015 19:24
Some API functionality is not supported with OpenCyc; it should be easy to view only the examples which are supported by OpenCyc.
Copied from original issue: cycorp/example-code#1

From @nwinant on April 12, 2016 17:16
This has been added.


AP-5217: Test LFA changes

What
Link to story

[!WARNING]
This is a POC to get Apply using the new LFA endpoint
Do not merge

It adds a new service to POST to the new, test, LFA endpoint on a UAT branch
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
The standards in the Git Workflow document on Confluence should be followed
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

This will be replaced by AP-5235


Bug 1407623 - Add documentation for short-term projects.

@peiying2 How does that look?

@flodolo comments addressed. @peiying2, ping for the review. :)

@adngdb, I have added more strings to the existing project. Initially I added to the production side directly but on the status bar, it shows 100% completion, so I clicked on the link, at file level, it does show the strings waiting to be completed, but between the % icon and the status bar, a localizer won't know something new is added to an existing project at higher level. I went to the staging site to add these strings, it didn't fix the problem. The Sync button is for repo type of project, not database type, but I tried it too, :). It didn't work. We need to have the status feature fixed.

Can we not use a documentation PR to discuss feature issues? That belongs to a bug.
I'd also like to understand the status of this PR.

Periodic ping…

I think we should be able to merge this, for even if the feature could be improved, that should happen in bugs, and in my opinion it shouldn't block merging the documentation. @peiying2 what do you think?

Let's merge this.


When I run Windows Terminal as admin cannot start WSL

Environment
OS version: 10.0.18362 N/A Build 18362
Windows Terminal version (if applicable):0.9.433.0
Any other software?
WSL
Steps to reproduce
run windows terminal as admin
Expected behavior
ability run WSL in same window as powershell
Actual behavior
WSL is visible when i start Windows terminal as standard user but when I start it as Administrator WSL is not visible in menu.

OS version: 19041.113 (Insider Slow)
Windows Terminal version: 0.9.433.0
I am unable to reproduce this issue. When I run WT as administrator, Ubutnu-18.04 is visible and usable.

@svpudik Are you maybe using another user account to elevate the Terminal? Or are you using the currently logged-in user?
If you try running WSL manually in an admin Windows Terminal instance, does that still work? What's the output of wsl -l in both an unelevated and an elevated Windows Terminal window?

i can't launch it with Search > wt with run as administrator

I am running admin as logged user.
here is screen ad admin:

screen without admin:

screen of  wsl -l ad admin have not enough priviledges:

What I have to modify to provide there admin access?
here is my config
uid": "{2c4de342-38b7-51cf-b940-2309a097f518}",                 "hidden": false,                 "name": "Ubuntu",                 "source": "Windows.Terminal.Wsl"

That's certainly curious. @benhillis @craigloewen-msft any ideas why the user might not be able to see any linux distros while they're running as admin?

This may need to be filed on the WSL repository. Sorry!


Design for "create variable" button and dialog

Placeholders right now:

Scratch 2.0 - Creating a Variable Behaviors
Clicking 'Create a Variable' opens dialog

Ability to Name a Variable

Variables cannot have the same name (case sensitive), if the user tries to name a Variable with the same name an error will be thrown

A user can create two 'For this sprite only' variables that have same name across two different sprites

Ability to Choose if the Variable is 'For all sprites' or 'For this sprite only'

For all sprites (aka Global): The Variable that is created will be accessible in the 'Data' category on all sprites

For only this sprite (aka Local): The Variable that is created will only be accessible in the 'Data' category on the sprite it was created. (Unless manually dragged to another block)

Cloud Var checkbox
This will only appear for users who have been given access to the Cloud Variable feature and can only be checked 'On' for 'For all sprites' variables

Scratch 3.0 Spec coming soon...

For playtest don't worry about the variable scope or cloud variables. We will get to that in a future iteration.


Simplify the check if cent/ucent is supported.

The same check is used in Phobos in module std.traits.

Auto-merge toggled on


Store settings locally instead of via Chrome sync

While it's great to synchronise settings, it is not strictly necessary for an extension of this nature.
Perhaps this could be provided as an option, but as the only source of saving data it isn't necessary.

Doesn't seem to cause an issue with those not signed in. Seems like it might cause more issues than it's worth.
Could add as an optional selection if the future.


Question about the paper's comparison.

I've read the paper and have some small questions about the paper's comparison to other models like EWC, ER and DER++ because I've not gained  or maybe missed the information about the details of conducting these methods in the pretrained ViT model.
Which part of the ViT is trained or finetuned in Upper-bound and those methods and which parts are using pretrained weights? I guess only the classifier is trained but need some confirmation.

Great questions! We implement these baselines using the same ViT-B/16. For these methods, we tried to also freeze the backbone (except the classifier), however, the results is not as good as train the whole model. So we basically report the results with training the full model from the pre-trained checkpoint for the baselines.


X exchange <IP_ADDRESS>

Looks good to me. I will merge it after changing the version number.

Just pushed a new change. The schema version has been set back to <IP_ADDRESS>, and both instances of <IP_ADDRESS> that I had put in the documentation have now been changed to UNRELEASED.


API: Object PUT/GET func supporting CBC encryption

GetObject() with encryption will probably return io.Reader instead of a seekable object, not sure if it always possible to calculate which block we need to decrypt if we have a given offset.

Currently, this can work interchangeably with aws-sdk-java with symmetric key, tomorrow, I will finish the asymmetric and clean the code.

Currently, this can work interchangeably with aws-sdk-java with symmetric key, tomorrow, I will finish the asymmetric and clean the code.

Cool Thanks @vadmeste

This is the new API
1822     // Generate a symmetric key
1823     symKey := NewSymmetricKey([]byte("my-secret-key-00"))

1864     // Generate an asymmetric key
1865     asymKey, err := NewAsymmetricKey(privateKey, publicKey)

1902     securedObj := NewCBCSecureData(testCase.encKey)
1903     if err != nil {
1904         t.Fatal(err)
1905     }
1906
1907     // Put encrypted data
1908     _, err = c.PutSecuredObject(bucketName, objectName, reader, securedObj, nil, nil)
1909     if err != nil {
1910         t.Fatalf("Test %d, error: %v %v %v", i+1, err, bucketName, objectName)
1911     }
1912
1913     // Read the data back
1914     r, err := c.GetSecuredObject(bucketName, objectName, securedObj)
1915     if err != nil {
1916         t.Fatalf("Test %d, error: %v %v %v", i+1, err, bucketName, objectName)
1917     }

Are we okay with the new API ?

Are we okay with the new API ?

Not yet decided. @vadmeste  will get it reviewed from @abperiasamy

This API is approved @vadmeste - will review.. and we can merge it by tomorrow.

// Generate a symmetric key
symKey := NewSymmetricKey([]byte("my-secret-key-00"))

// Generate an asymmetric key
asymKey, err := NewAsymmetricKey(privateKey, publicKey)
if err != nil {
	t.Fatal(err)
}

// Secured object
cbcMaterials, err := NewCBCSecureMaterials(symKey)
if err != nil {
	t.Fatal(err)
}

// Put encrypted data
_, err = c.PutSecuredObject(bucketName, objectName, bytes.NewReader([]byte{"buf"}), cbcMaterials, map[string][]string{"Content-Type": []string{customContentType}}, nil)
if err != nil {
	t.Fatal(err)
}

// Read the data back
r, err := c.GetSecuredObject(bucketName, objectName, cbcMaterials)
if err != nil {
    t.Fatal(err)
}

@harshavardhana, This is the new API after some renaming that you suggested.

@harshavardhana, This is the new API after some renaming that you suggested.

This looks good @vadmeste

This looks good @vadmeste

I addressed all your comments but only one. Did you see it ?

Comments addressed again.

I've been planning to add envelope encryption to minio-go for a while as I thought the current encrypted object support wasn't envelope based. Just read the existing implementation, very elegant :ok_hand: nice work.


Exception: Account data length (0) does not match expected size (4296)

I've got "Exception: Account data length (0) does not match expected size (4296)" when trying to place an order.

Is this variable supposed to be private, public, or keypair ?
wallet = mango.Wallet(bytes([1, 2, 3....32])

Is this one supposed to be the public key ( = wallet address) OR the SPL key (Serum dedicated address for SPL tokens) ?
account = mango.Account.load(context, PublicKey("..."), group)

Or is it completely something else ?
Thanks

in /mango/account.py @ line 506 Acccount.parse is called, with 3 non-null variable in it (ok so far):
return Account.parse(account_info, group, cache)
inside the parse function, {account_info} seems to be parsed properly (here's the variable content):
Owner:<PHONE_NUMBER>1111111111111111111111
Executable: False
Lamports: 435925934 / Sols: 0.435925934
Rent Epoch: 317
Data Length: 0

However this
data = account_info.data
return this (what i see if I print it):
b''
Leading to
len(data) = 0

Nevermind, i found the solution: I needed to use the mango account (instead of my own public key):
[(https://trade.mango.markets/account)]
works fine, now  :-)


error code sigsegv

OS/Platform
Debian, Ubuntu, and derivatives
Installed
Flatpak
Version
Version 123.0.6312.86 (Official Build, ungoogled-chromium) (64-bit)
Have you tested that this is not an upstream issue or an issue with your configuration?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[ ] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
error code sigsegv after flatpak update
How to Reproduce?
This happened after the flatpak update
Actual behaviour
Check screenshot https://imgur.com/a/JmFnKYa
Expected behaviour
Hi, two days ago I updated all the apps to flashpak, and I got this problem that pops up in ungoogled chrome, how to fix it, before that it was not there, that is, before the update of the flashpak version.
Relevant log output
No response
Additional context
No response

@rany2 Have you encountered such issue?

Nope, I don't have a clue about this. Sorry

@PF4Public The only thing I could think of is that the flatpak was updated and he didn't exit UGC and relaunch.


Name alignment not properly centered in 1st line of dialogue

When you use centered name alignment, in the1st dialogue line that is shown, the name label is not properly centered.
1st line:

2nd line:

EDIT: Managed to sort of fix this by just moving the name_label.visible = true at the end.

That simple? Want to make a PR for it if you're up to date? Otherwise I'll implement the fix

Yup, it's only that. I honestly am pretty new to GitHub, so I don't really understand how to make a pull request. Please feel free to do so!

Gotta love when a fix is easy, effective, and easy to test. Tested it out fixing the problem in my game too. #1127


Import leitan/zustand

Hi! Great job with the library, I've already seen the code and I'm sure it will help plenty of people like me.
Is it available for use? and if the answer is yes, how can I import it inside my project? Is it import 'leitan/zustand'
Again, congratulations and keep going!

Hi,  here
Hi. The library already comes with types if needed. here is an example how to use it.
And about function import
import {
leitenRequest,
leitenGroupRequest,
ILeitenGroupRequestParams,
leitenList,
leitenRecord,
leitenModal,
leitenNormalizedList,
leitenPrimitive,
leitenContext,
...
} from "leiten-zustand";
Hope I understood the question correctly.

Thanks for the prompt reply! I'll take a look to the example and try to import it the way you are showing me.


[ja] add Release History page for Japanese

This is a Feature Request
What would you like to be added
Create a Japanese page corresponding to https://kubernetes.io/releases/
Why is this needed
This page did not exist
Comments
/assign
/language ja
/area localization

/triage accepted


Fixed handling of json newline formatting and modify full sync interval

Change(s):

Fixes an issue affecting json file formatting in the inventory storage source
Modify default full sync interval from 14 to 7 days

Reason for Change(s):

Bugfix (privately reported)
Full sync interval modification to avoid connector status showing as disconnected

Testing Completed:

Yes

Hello @kingwil, Can you please share results/working images of function app

Hi @v-prasadboke  - please see screenshots

and


add isExcl param to CreateFile

KBFS-1217

First pass done.  Code looks good, but we should think about whether we should merge this as is, or if we should still allow EXCL creates to cause conflicts for now.


add ioBroker.vis-2-widgets-ovarious to latest repository

E201 invitation is send to bluefox
W184 as already reported if i remove common.noConfig then another error is shown
W400 yes, sure

Thanks for spending your time and providing a new adapter for ioBroker.
Your adapter will get a manual review as soon as possible. Please stand by - this might last one or two weeks. Feel free to continue your work and create new releases. You do NOT need to close or update this PR in case of new releases.
In the meantime please check any feedback issues logged by automatic adapter checker and try to fix them.
You will find the results of the review and eventually issues / suggestings as a comment to this PR. So please keep this PR watched.
If you have any urgent questions feel free to ask.

Some Info regarding checker messages:
❗ [E204] Version(s) "0.1.1, 0.1.0" listed at common.news at io-package.json do not exist at NPM. Please remove from news.
Releases not available at npm must not be listed at common.news. Please remove those release and eventually add the changes to next higher available release if desired. Admin uses news entries to configure user selectable release when doing an installation of a specific release. If this release is not abvailable at npm installation fails and users do not know why the see a release but get an error when trying to install.
👀 [W184] "common.noConfig" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI.config":"none".
Please remove common.noConfig and use adminUI.config:none. If there is still an other error please ignore this error - I'll check.
👀 [W523] "package-lock.json" not found in repo! Please remove from .gitignore!
package-lock.json should be present at github to speed up installation during tests. If you really want to do a full installation during test workflows you can ignore this warning. Maybe tha warning will be suppressed if build is done during every test anyway. I'll make a note.

Your adapter is named vis-2-widgets-ovarious and NOT ovarious. Please correct sources-dist.json.

reminder 4.8.2024

RE-CHECK!

Will be further processed by https://github.com/ioBroker/ioBroker.repositories/pull/3866
as alphabetic order is incorrect
broken commit (invalid reponame) blocks normal checks.
Please subscribe https://github.com/ioBroker/ioBroker.repositories/pull/3866 to get notifications

RE-CHECK!


Document more

Adds more documentation for how organisation works today

Merging since everything is accurate, just missing some info


Responsive Login/Sign up page

Description
The login and sign up page is not responsive on mobile phone. The following are screenshots attached of the problem.
@muhafiz5814 @Hamza20203064 @kindly assign this issue to me with label hacktoberfest amf gssoc
Etc.

Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@ParulPrashar You have raised a duplicate issue; it was already raised by someone else.

Look at all the issues, see which ones are available and which ones are not, and create issues accordingly.

This is not the way to work on open source :)

Don't forget to give our repository a star and let me know once you've done it.


Queue locking for concurrency = 1

This PR is the first draft of what a full queue locking PR might be. It does the following:

Adds a lock of with ttl 10 seconds before processing each job
Removes the lock when the job is done
If the lock fails, it calls process.nextTick and then tries again.

Known limitations that need to be address;

Most users of kue want concurrency greater than 1. This PR assumes everyone wants concurrency = 1. This needs to be configurable.
The lock ttl needs to be configurable. Currently it is hard coded for 10 seconds
The queue-lock key should be unique to the job type, and probably use the kue prefix, which defaults to q:. So, something like prefix:type:lock.

Testing
The redis_lock script in the test directory is what I've come up with so far as a way to test this. It uses node's cluster module to create multiple processes that try to pull jobs off the queue.
Run it like so export SIZE=2 && node test/redis_lock.js, changing SIZE from 1 to 2.
When the script is run WITHOUT the changes in lib/queue/worker.js, it should succeed with SIZE=1, but fail with SIZE=2.
When the script is run WITH the changes in lib/queue/worker.js, it should succeed no matter what SIZE is.

Apologies. This was meant to be into my own fork. Closing


Use esp idf v5.1.1

Update to latest version of ESP-IDF
Fixes golioth/firmware-issue-tracker#237

Currently blocked by #163. The new version of ESP-IDF enables additional compiler warnings, and that causes the build to fail on some uninitialized variables in libcoap. The compiler warnings are fixed in the version of libcoap that is brought in by #163. A passing test run can be seen in #172 which has both these changes and the libcoap changes.


Addresses #145, based off of @fmassa

This PR moves both find_classes and make_dataset under DatasetFolder.
The former is as suggested by fmassa in issue #145 . However, make_dataset relies on class names being the sub folders of the root directory. Hence, make_dataset was moved as well.

Actually I have a better idea. I'll suggest it in #145.


GitHub Actions: Examples for canceling in-progress jobs / runs with concurrency groups lead to confusing behavior

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#example-using-concurrency-to-cancel-any-in-progress-job-or-run
What part(s) of the article would you like to see updated?
👋🏽 I recently made a change to several of our GitHub Actions workflows that adds the config from https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#example-using-concurrency-to-cancel-any-in-progress-job-or-run to cancel any redundant in-progress run on the current PR – to recreate the PR behavior other CI providers offer out of the box. (e.g. Azure Pipelines, CircleCI)
However, it wasn't having the intended effect – it resulted in progress runs of these workflows to get canceled no matter what branch they're on, so long as they have an open PR.
From a lot of back and forth with GitHub Support, it turns out the issue was that the repo has multiple workflows that run on pull requests. Concurrency group names must be unique for each workflow across the whole repo, otherwise they will auto-cancel each other on a single PR run (because that would mean they share the same concurrency group).
In the example given in the GitHub Actions docs, github.head_ref evaluates to the same string on every workflow on a given PR, leading to these workflow runs were clobbering each other.
The fix is to ensure that each concurrency group name is unique. But, this wasn't clear to me from the docs at all – I only found this out from messaging GitHub Support and a lot of trial and error. I'd like to do what I can to help other folks avoid similar frustration in the future, with any luck. 🙏🏽
Expected
The docs gives examples for how multiple workflows with auto-canceling in-progress runs / jobs can coexist without auto-canceling each other.
Actual
The docs' only example for enabling auto-canceling on a PR sets a single concurrency group name that doesn't allow for multiple workflows to coexist without auto-canceling each other on a pull request (github.head_ref as the whole concurrency group name)
Let me know if there's any more information I could add that would be helpful! 🙇🏽 I'll have a PR up for this shortly.
Additional information

Azure Pipelines docs for auto-canceling a redundant build
CircleCI docs for auto-canceling a redundant build

super-octo-winner-J-magam.zip

fluffy-octo-tribble-J-magam.zip

auth_tokens.csv
category_preferences.csv
visits.csv
likes.csv
auth_token_logs.csv
queued_posts.csv
user_archive.csv
badges.csv
bookmarks.csv
flags.csv

(Issue #_19936!!!)
On Sun, Oct 17, 2021, 11:47 welcome[bot] @.***> wrote:

Thanks for opening this issue. A GitHub docs team member should be by to
give feedback soon. In the meantime, please check out the contributing
guidelines https://github.com/github/docs/blob/main/CONTRIBUTING.md.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/11243#issuecomment-945174835, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6J7CJCHFQOKNRRJVWTRTDUHMK2LANCNFSM5GFDONOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Begone stalebot - this is still an issue

@mnquintana Sorry about the stale bot! I made sure it is in the right review board for the writers review.
Thanks so much for your patience 💖


Use of s3() within parallel_map()

Consider this call:
with metaflow.S3() as s3i:
result = s3i.info_many(s3_path, return_missing=True)
Can this be put in a metaflow.multicore_utils.parallel_map ?
i.e. parallel_map(wrapper_for_s3_info_many, s3_paths)
When I try, I get this error:

2024-09-30 23:08:29.391 [261693/start/3201226 (pid 1400063)] metaflow.plugins.datatools.s3.s3.MetaflowS3URLException: Specify S3(run=self) when you use S3 inside a running flow. Otherwise you have to use S3 with full s3:// urls.
2024-09-30 23:08:29.391 [261693/start/3201226 (pid 1400063)] Internal error

However, s3_paths=["s3://path/to/something.jpg","s3://path/to/something_else.jpg", ...] and I know 100% that every path in s3_paths starts with "s3://"
Putting run=self in the S3 instantiation within the wrapper yields

2024-09-30 23:21:21.832 [261699/start/3201250 (pid 1405459)] S3 non-transient error (attempt #1): s3op failed:
2024-09-30 23:21:21.913 [261699/start/3201250 (pid 1405459)] Invalid url: /

@hasush the s3.xx_many calls are already parallelized behind the scenes so one shouldn't necessarily need parallel_map. regardless, the error that you highlighted looks like a bug that we will address.


unquote use switch

The switch statement replaces the series of if statements that check the quote character. The cases for single-quote, double-quote, pipe, and slash characters handle the same logic as before. The default case handles quote characters that are not supported.
By using a switch statement, the code becomes more concise and easier to read.

/retest

I need ok-to-test @Iceber

/retest

/retest

/cc @dmcgowan


fix regression of deleting all parameters in text mode

Summary

[x] Fix regression introduced https://github.com/finos/legend-studio/pull/2005 of deleting all parameters when parameters exits in text mode. We don't reset parameter state

How did you test this change?

[ ] Test(s) added
[ ] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

closing as this is addressed by https://github.com/finos/legend-studio/pull/1984


Able to skip to advanced stages

A student was able to go from the forest to the glacier by typing in the name of a level, which allowed the gray banner to show up, seeming as if they got there.

It's by design that you can skip ahead to levels if you know the name of the level you want to go to. If you are able to do the code and you have the gear, then why not skip the easier stuff?


Add params to Booster object for feature contribution computing

To implement the idea described at http://blog.datadive.net/interpreting-random-forests/, I think it's necessary to add params to Booster object (public or private?) so that we can scale down the raw prediction (e.g. log odds) of each node/leaf to get the real prediction (e.g. probability).
Does this sound a reasonable change to you @guolinke ? If so, where do you prefer to add the params to Booster? Thanks you!

Sorry, another question.
Is there anyway to get the learning_rate of each tree? The learning rate may change along training process if using reset_parameter.

to get the learning_rate of each tree:

the learning rate is multiplied to leaf value, so learning_rate = (left leaf value * left leaf count + right leaf value * right leaf count)/(interval value * interval count)
for figure above, learning_rate = (0.0585234093637455041666 + 0.087830416415208223314) / (4980*0.6002008032128514) = 0.13

@yanyachen
I think the solution provided by @wxchan can solve your problem without add additional parameters.

Yes, it solved my problem. Sorry, I forgot to close the issue.

I just realized that if the depth of tree is only 1, then this method can't be applied because interval_value is always 0. Is there anyway to walk around this? Thanks you!

@yanyachen I will open a new PR to output shrinkage_rate for each tree


treat cecil TypeReference as RuntimeTypeHandle

same as FieldReference and RuntimeFieldHandle
code and il:

output before:

output after:

Hi, can you also update ConstantDefinition's ToString to just dump the full name of the type, like it does for Method and Field definitions, in the case of TypeReferences? That would fix the argument to the GetTypeFromHandle call looking odd in your "after" screenshot.

Actually, never mind that. It's a little bit more complicated due to the fact it has to be wrapped in a typeof.


Visionos podspec misconfiguration resulting in build failure

Dear Owners / Contributors,
Since the last update, visionOS seems to be supported by default. While running a build against RN v0.73.8 it have failed as our project doesn't specify support for it.

Would it be possible to make visionOS optional? Thank you in advance!

@av185218 confirm the version you are running with pod --version.
After I updated to 1.15.2, this error is gone

It's by no means a good solution but I was running into the same problem and downgrading to 1.5.1 let me install pods. I'm running RN 73.9.

@av185218 confirm the version you are running with pod --version.
After I updated to 1.15.2, this error is gone

Thank you. it works for me.


running command with habitus fails due to Docker API changes.

update go-dockerclient
https://github.com/fsouza/go-dockerclient/issues/537


project_Details bug fix

Ajax call breaks when submitting a record for staff.

@flavour: Just a sec merging one more bug :p

The project_Details bug fix is missing ;)

Ah - it's already in trunk, so it doesn't appear in the diff anymore. Ok ok

@nursix : Fran is too fast for a second :D :D


Remove this Issue. Sorry

Please remove this issue. Wrong repo. Sorry for the mistake

Wrong Repo. Can remove

You can remove the history of previous edits on github of the issue if you published something secret.


[bug] Fix redirect after login to non root pages

Current Behavior
Redirect after login to non root pages are always redirecting to the root page
Expected Behavior
While trying to login in a non root page, redirect to the desired page after successful login.
Update RedirectUri within utils/Auth.ts -> concat origin with pathname

Is this issue still there?

Yes, it is. But is currently in development by @ldcss


running runs/parallel_ft_lora.shgives overflow

Hi, I am trying to parallel_ft_lora.sh without modification, but I get the overflow error from deepspeed. Here's the snippets of message:
[INFO|trainer.py:1721] 2024-02-01 16:50:44,694 >> ***** Running training *****
[INFO|trainer.py:1722] 2024-02-01 16:50:44,694 >>   Num examples = 117,404
[INFO|trainer.py:1723] 2024-02-01 16:50:44,694 >>   Num Epochs = 1
[INFO|trainer.py:1724] 2024-02-01 16:50:44,694 >>   Instantaneous batch size per device = 4
[INFO|trainer.py:1727] 2024-02-01 16:50:44,694 >>   Total train batch size (w. parallel, distributed & accumulation) = 128
[INFO|trainer.py:1728] 2024-02-01 16:50:44,694 >>   Gradient Accumulation steps = 4
[INFO|trainer.py:1729] 2024-02-01 16:50:44,694 >>   Total optimization steps = 917
[INFO|trainer.py:1730] 2024-02-01 16:50:44,696 >>   Number of trainable parameters = 7,733,248
  0%|                                                                                | 0/917 [00:00<?, ?it/s]
[2024-02-01 16:50:47,313] [INFO] [loss_scaler.py:183:update_scale] [deepspeed] OVERFLOW! Rank 0 Skipping step. Attempted loss scale:<PHONE_NUMBER>, reducing to<PHONE_NUMBER>
  0%|▎

all the way to
[2024-02-01 16:51:58,737] [INFO] [loss_scaler.py:183:update_scale] [deepspeed] OVERFLOW! Rank 0 Skipping step. Attempted loss scale: 2, reducing to 1
  5%|██████████████▊                                                                                | 42/917 [01:14<25:24,  1.74s/it]
Traceback (most recent call last):
  File "/mnt/task_runtime/ALMA/run_llmmt.py", line 223, in <module>
    main()
  File "/mnt/task_runtime/ALMA/run_llmmt.py", line 172, in main
    train_result = trainer.train(resume_from_checkpoint=checkpoint)
  File "/miniforge/lib/python3.10/site-packages/transformers/trainer.py", line 1539, in train
    return inner_training_loop(
  File "/miniforge/lib/python3.10/site-packages/transformers/trainer.py", line 1869, in _inner_training_loop
    tr_loss_step = self.training_step(model, inputs)
  File "/miniforge/lib/python3.10/site-packages/transformers/trainer.py", line 2777, in training_step
    self.accelerator.backward(loss)
  File "/miniforge/lib/python3.10/site-packages/accelerate/accelerator.py", line 1958, in backward
    self.deepspeed_engine_wrapped.backward(loss, **kwargs)
  File "/miniforge/lib/python3.10/site-packages/accelerate/utils/deepspeed.py", line 176, in backward
    self.engine.step()
  File "/miniforge/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 2157, in step
    self._take_model_step(lr_kwargs)
  File "/miniforge/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 2063, in _take_model_step
    self.optimizer.step()
  File "/miniforge/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1799, in step
    self._update_scale(self.overflow)
  File "/miniforge/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 2050, in _update_scale
    self.loss_scaler.update_scale(has_overflow)
  File "/miniforge/lib/python3.10/site-packages/deepspeed/runtime/fp16/loss_scaler.py", line 175, in update_scale
    raise Exception(
Exception: Current loss scale already at minimum - cannot decrease scale anymore. Exiting run. ```

Can anyone help? Also, I notice that the number of parameters is 7.7B instead of `--lora_rank 16` and `--use_peft`.

Hi, thanks for the interest!
Some points in my mind that could help you to fix the overflow issue:

Try bf16 rather than fp16
uninstall the transformers and reinstall it by pip install git+https://github.com/fe1ixxu/ALMA.git@alma-r-hf

As for number of parameters, I believe you may misread it:  7,733,248 is 7 million 😊
Let me know if you have further questions. Have a lovely day!

Thanks! Clean install works!

Simply change the training script to --bf16 worked for me. You can refer to this issue.


Build generic lab Panorama configuration XML

Build a complex lab configuration containing samples of every major object type in order to facilitate testing of the script.

This has been resolved.


Grpc fiber router integration

Description
Minor bugfix for grpc fiber
Changes

Handle unhandled error at config.go
Add negative test case for the above
Add omitempty to proxy struct's protocol, service, method field, as these json fields are not required in HTTP setting

Added some comments, the rest LGTM. Thanks!

thanks for the quick review!


chore: improve ci pipeline and validation

:tada: This PR is included in version 0.4.6-rc.1 :tada:
The release is available on:

npm package (@rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on npm package (@rc dist-tag)
Your semantic-release bot :package::rocket:

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

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


Failed assertion node.isRepaintBoundary is not true when combining LayoutBuilder, AnimatedSwitcher and ProgressIndicator

I have found an issue where a Failed assertion node.isRepaintBoundary is not true is thrown when having a tree consisting of LayoutBuilder, AnimatedSwitcher and CircularProgressIndicator.
The issue happens when the AnimatedSwitcher's animation duration (kAnimationDuration on the example) is bigger than the duration of the change rate of the variable controlling the AnimatedSwitcher's child (kTimerDuration on the example). The assertion fails when the AnimatedSwitcher's child changes for the second time (in the example I run the timer switching the child twice).
Removing the LayoutBuilder from the tree prevents the assertion to fail. Wrapping the CircularProgressIndicator with a RepaintBoundary widget.
Steps to Reproduce

Execute flutter run (debug mode) on the code sample.
Check the Console. You will see the following assertion exception 'package:flutter/src/rendering/object.dart': Failed assertion: line 1135 pos 20: 'node.isRepaintBoundary': is not true. (Complete logs on Logs section)

Expected results: Don't throw assert exception
Actual results: Assert  'node.isRepaintBoundary': is not true. thrown

Code sample
import 'dart:async';

import 'package:flutter/material.dart';

const kAnimationDuration = Duration(milliseconds: 1000);
const kTimerDuration = Duration(milliseconds: 900);
void main() {
  runApp(const IssueApp());
}

class IssueApp extends StatelessWidget {
  const IssueApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: IssueScreen(),
    );
  }
}

class IssueScreen extends StatefulWidget {
  const IssueScreen({Key? key}) : super(key: key);

  @override
  State<IssueScreen> createState() => _IssueScreenState();
}

class _IssueScreenState extends State<IssueScreen> {
  bool _showLoading = true;
  Timer? _timer;
  @override
  void initState() {
    int times = 0;
    _timer = Timer.periodic(kTimerDuration, (timer) {
      setState(() {
        _showLoading = !_showLoading;
      });
      times++;
      if (times > 2) {
        timer.cancel();
      }
    });
    super.initState();
  }

  @override
  void dispose() {
    _timer?.cancel();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Issue screen'),
      ),
      body: LayoutBuilder(builder: (context, constraints) {
        return AnimatedSwitcher(
          duration: kAnimationDuration,
          child: _showLoading
              ? const CircularProgressIndicator()
              : const SizedBox(),
        );
      }),
    );
  }
}

Logs
======== Exception caught by scheduler library =====================================================
The following assertion was thrown during a scheduler callback:
'package:flutter/src/rendering/object.dart': Failed assertion: line 1135 pos 20: 'node.isRepaintBoundary': is not true.

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

When the exception was thrown, this was the stack:
#2      PipelineOwner.flushPaint (package:flutter/src/rendering/object.dart:1135:20)
#3      RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:518:19)
#4      WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:865:13)
#5      RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:381:5)
#6      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1289:15)
#7      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1218:9)
#8      SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:1076:5)
#9      _invoke (dart:ui/hooks.dart:145:13)
#10     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:338:5)
#11     _drawFrame (dart:ui/hooks.dart:112:31)
(elided 2 frames from class _AssertionError)

flutter analyze
Analyzing issue.dart...
No issues found! (ran in 0.5s)

[✓] Flutter (Channel stable, 3.7.3, on macOS 13.2 22D49 darwin-arm64, locale en-CH)
    • Flutter version 3.7.3 on channel stable at /Users/mcorcuera/development/tools/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (8 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/mcorcuera/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.74.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] Connected device (3 available)
    • Mikel’s iPhone (mobile) • 00008110-000609D40C38401E • ios            • iOS 16.2 20C65
    • macOS (desktop)         • macos                     • darwin-arm64   • macOS 13.2 22D49 darwin-arm64
    • Chrome (web)            • chrome                    • web-javascript • Google Chrome 109.0.5414.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Thanks for the report. I am seeing same behavior using latest master and stable versions.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (4 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.8.0-13.0.pre.52, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.8.0-13.0.pre.52 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cc473e4f19 (73 minutes ago), 2023-02-15 22:51:19 -0500
    • Engine revision 3fc40ca5be
    • Dart version 3.0.0 (build 3.0.0-241.0.dev)
    • DevTools version 2.21.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Yes, I'm see the exact same as well.
Flutter 3.7.3 • channel stable
Thanks for the very useful report of it.

In my case we have a permanent Drawer with a list of a custom DrawerTile widgets, with each an AnimatedSwitcher inside it. The AnimatedSwitcher is used to animate the tile becoming active.  When using the Drawer to navigate between pages Failed assertion node.isRepaintBoundary is not true would show up, since changing the page would make another tile active and trigger the AnimatedSwitcher.
Removing the AnimatedSwitcher inside the DrawerTile fixed it (but now the UI looks worse😭)
I don't know if it is the interplay between Drawer and AnimatedSwitcher or another widget inside the widget tree either outside of the drawer or inside the drawer. Right now I can't investigate further. We don't use a LayoutBuilder but Row, Column, SizedBox, Expanded, SingleChildScrollView.
This error started occurred when I tried to upgrade our Flutter app from 2.10.5 to 3.7.3, it didn't occur before upgrading.

FYI, I bisected this regression to the following commit: https://github.com/flutter/flutter/pull/106962

CC @gspencergoog
https://github.com/flutter/flutter/pull/106962 was landed by @youssef-attia, looks like your LGTM.

Looks like this has already been fixed by https://github.com/flutter/flutter/pull/121408.

#121408 has since been reverted, in #121835, so reopening.
See also #121336, which was fixed by the same PR that fixed this one.

Same issue here, seems to happen when I press a button which is currently being animated to by an AnimatedSwitcher.
For example:
AnimatedSwitcher(
duration: const Duration(milliseconds: 400),
child: isLoading? Container(): Button(...)
);
if I press the button just after isLoading has been set to false the issue happens

same issue here, using a AnimatedSwitcher. Is there a fix coming or a workaround? For now, all I could do is lower the duration, which is not ideal.

Just wrapping the animation in a RepaintBoundary was the workaround for my case.

Just wrapping the animation in a RepaintBoundary was the workaround for my case.

When I wrapped the child of the AnimatedSwitcher with RepaintBoundary, the error did not occur, but the AnimatedSwitcher did not work.

is there any alternative solution other than RepaintBoundary?

Wrapping the child with a SizedBox fixed the issue for
AnimatedSwitcher(
              child: SizedBox(
                child: child,
                width: getScreenSize().width * 0.75,
              ),
              duration: const Duration(milliseconds: 500),
 )

Wrapping the child with a SizedBox fixed the issue for me
AnimatedSwitcher(
  child: SizedBox(
   key: UniqueKey(),
    child: child,
    width: getScreenSize().width * 0.75,
  ),
  duration: const Duration(milliseconds: 500),
);

This workaround works for me too. In my case I have an AnimatedAlign wrapped by a LayoutBuilder and with an animated shimmer as child.

Adding key: UniqueKey() to AnimatedSwitcher was the 'key' :)

Just wrapping the animation in a RepaintBoundary was the workaround for my case.

When I wrapped the child of the AnimatedSwitcher with RepaintBoundary, the error did not occur, but the AnimatedSwitcher did not work.

This problem seems to be related to the animated switcher not detecting that the child changed, because it is the same widget (RepaintBoundary). Providing a key that changes depending on the child or only wrapping the problematic child (CircularProgressIndicator) with the RepaintBoundary solved the issue for me.

Workarounds with unique or value key are problematic as they reset state which causes unexpected behavior when the value key changes, proceed with workarounds with caution, this may lead to an unexpected behavior

Also encountering this issue on Flutter version 3.23.0-0.1.pre
Cause is: Layout Builder -> Animated Switcher -> Some (Custom) Animated Widget

I have a rather complicated StatefulWidget: VisibilityDetector > ValueListenableBuilder > LayoutBuilder > [QR code scanner widget]
Wrapping the whole thing with a ColoredBox kills it.
Flutter stable / 3.22.2

I can no longer reproduce the issue on master (but I was able to repro this using 3.23.0-0.1.pre) using the provided sample. It looks like 8ce3bfb6ca4c2339b6eaed3a954547fc6498e8db fixed it.