Issue with PeTar when bridging with Fi and Nsph >~ 16000

Hey all,
I am trying to run a simulation of the evolution of SPH cloud interacting with a stellar cluster, coupling Fi and PeTar with Bridge
Apparently an error appears in PeTar when trying to describe the gas with more than ~16000 particles (2^14).
The error comes out at the moment of interaction, independently of the distance between the cloud and the cluster (initially set to 10^2 * Rcloud), whether is PeTar kicking Fi or the other way around, and all the physical parameters of the system.
When trying with less than 15000 sph particles the code works without errors.
here's the error with redirection='none' on petar when running with 5*10**4 sph particles and a cluster of 200 stars
Fi is synchronizing .. done
PETAR: recommit_particles start
PETAR: recommit_particles end
petar_worker: src/PeTar/src/soft_force.hpp:273: void CalcForcePPSimd<Tpi, Tpj>::operator()(const Tpi*, ParticleSimulator::S32, const Tpj*, ParticleSimulator::S32, ForceSoft*) [with Tpi = ParticleBase; Tpj = FPSoft; ParticleSimulator::S32 = int]: Assertion `n_ip<=pg.NIMAX' failed.
ni= 48793 NIMAX= 16384 nj= 200 NJMAX= 131072
Fi receives kick from petar --------------------------------------------------------------------------
MPI_ABORT was invoked on rank 0 in communicator MPI_COMM_WORLD
with errorcode -1.

You may or may not see output from other processes, depending on
exactly when Open MPI kills them.
--------------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/rfi/core.py", line 120, in __call__
    dtype_to_result = self.interface.channel.recv_message(call_id, self.specification.id, handle_as_array)
  File `"/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/rfi/channel.py",` line 1164, in recv_message
    raise exceptions.CodeException("Error in code: " + error_message)
amuse.support.exceptions.CodeException: Error in code: no error message - code probably died, sorry.
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "My_first_issue.py", line 108, in <module>
    system_gr_hyd.evolve_model(SysTime,timestep=timestep_gr_hyd)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/ext/bridge.py", line 152, in evolve_model
    return self.evolve_joined_leapfrog(tend,timestep)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/ext/bridge.py", line 170, in evolve_joined_leapfrog
    self.kick_systems(timestep/2)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/ext/bridge.py", line 295, in kick_systems
    kick_system(x,y.get_gravity_at_point,dt)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/ext/bridge.py", line 101, in kick_system
    ax,ay,az=get_gravity(parts.radius,parts.x,parts.y,parts.z)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/support/methods.py", line 168, in __call__
    result = self.method(*list_arguments, **keyword_arguments)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/support/methods.py", line 168, in __call__
    result = self.method(*list_arguments, **keyword_arguments)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/support/methods.py", line 168, in __call__
    result = self.method(*list_arguments, **keyword_arguments)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/support/methods.py", line 267, in __call__
    return self.method(*list_arguments, **keyword_arguments)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/rfi/core.py", line 123, in __call__
    raise exceptions.CodeException("Exception when calling function '{0}', of code '{1}', exception was '{2}'".format(self.specification.name, type(self.interface).__name__, ex))
amuse.support.exceptions.CodeException: Exception when calling function 'get_gravity_at_point', of code 'petarInterface', exception was 'Error in code: no error message - code probably died, sorry.'

Is there a way to get around the problem?
I am running on Ubuntu 20.04.1 LTS, Pyhon 3.8.5, Numpy 1.19.4, AMUSE 13.2.0

As a workaround (but also recommended practice), you could use an external code to do the kicking instead of Petar. Since you're kicking a tree code, this wouldn't really degrade your accuracy (and it will be faster).
You would set it up like this:
from amuse.couple.bridge import (
    Bridge, CalculateFieldForCodes,
)

def new_field_kicker(code):
    kicker = Fi(converter, mode="openmp")
    kicker.parameters.epsilon_squared = epsilon_squared
    result = CalculateFieldForCodes(
        new_field_gravity_code,
        [code],
    )
    return result

to_sph_kickers = [
    new_field_code(nbody_code),
]
to_nbody_kickers = [
    new_field_code(sph_code),
]

system = Bridge(
    timestep=timestep_bridge,
)
system.add_system(
    nbody_code,
    partners=to_nbody_kickers,
    do_sync=True,
)
system.add_system(
    sph_code,
    partners=to_sph_kickers,
    do_sync=True,
    h_smooth_is_eps=True,
)

Thank you so much for your reply
I tried to reproduce your snippet assuming new_field_code = new_field_kicker, but when to_nbody_kickers tries to kick the nbody code, bridge returns that this error:
petar receives kick from CalculateFieldForCodes Traceback (most recent call last):
   system_gr_hyd.evolve_model(SysTime)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 556, in evolve_model
    return self.evolve_joined_leapfrog(tend,timestep)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 571, in evolve_joined_leapfrog
    self.kick_codes(timestep/2.)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 712, in kick_codes
    de += x.kick(dt)
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 455, in kick
    self.kick_with_field_code(
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 491, in kick_with_field_code
    ax,ay,az=field_code.get_gravity_at_point(
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 137, in get_gravity_at_point
    code = self._setup_code()
  File "/home/paul/Scrivania/Amuse-env/lib/python3.8/site-packages/amuse/couple/bridge.py", line 166, in _setup_code
    return self.code_factory_function()
TypeError: 'Fi' object is not callable


[withStyles] [typescript] flexDirection column is not a valid type

Expected Behavior 🤔

It was expected that the "column" value would be accepted in the flexDirection property in a styles object of the withStyles function. As in the MDN definitions.
Current Behavior 😯

The Typescript is currently returning:
Argument of type '{ root: { flexDirection: string; }; }' is not assignable to parameter of type 'Styles<Theme, {}, "root">'.
Steps to Reproduce 🕹

Reproduction.

Tech
Version

Material-UI
v4.1.1

React
v16.8.6

TypeScript
v3.5.2

This is a common pitfall caused by TypeScript's type-widening by default. See Using createStyles to defeat type widening for more information on how to prevent those issues.

I have the exact same issue when using makeStyles or createStyles.


Gradle Plugin for picocli-codegen

Wouldn't it be nice to have a Gradle plugin which generates the config? Maybe using something like https://github.com/classgraph/classgraph to automatically discover classes annotated with @Command?
I would gladly help out with this.

A Gradle plugin sounds awesome. I have some work in progress on a branch to add an annotation processor to the picocli-codegen module. This would analyse the code at compile time and build a CommandSpec model. Subclasses of this annotation processor can then take this model and generate documentation, source code, configuration or other files. Planning to merge this to master in a few days.
I had not considered integrating this with Gradle as a plugin, that sounds pretty cool!

Such a plugin could be based on https://github.com/palantir/gradle-graal which greatly simplifies integration with Graal.

I have some work in progress on a branch to add an annotation processor to the picocli-codegen module.

does that remove the need for reflection? I have a picocli app that only compiles with graal if reflection isn't used.

does that remove the need for reflection?

Potentially, yes. Some early thoughts on that topic are here: https://github.com/remkop/picocli/issues/539

Is this theoretically resolved now with the annotation processor implementation?

@juddgaddie You may be right. The nice thing about the annotation processor is that it works for plain javac as well as Gradle and Maven.
That said, some more work is required for Groovy and Kotlin.
I need to take another look at using the Micronaut visitor API for this: https://github.com/micronaut-projects/micronaut-core/issues/739#issuecomment-430608376

Closing as this issue is superseded by the annotation processor.


Provide an API to remove all overridden status

Eureka has two instance states, one as overridden status and other as actual instance status published by the instance.
The override is generally used to manage instance state from an administration tool like asgard. It will be handy to have an API TO remove all overridden state which will start honoring instance status as published by the instance itself.

Fixed by PR #412


Container Creation failes

Hi, I realised today I cannot recreate local dev containers anymore.
It might be an issue with docker desktop v4.27.2
Error: No such object: failed to get console mode for stdout: The handle is invalid.
Run-AlPipeline Telemetry Correlation Id: 7881398b-4d32-4f8d-98ad-3676778da0d2
Error: Invalid JSON primitive: failed.
Stacktrace: at Test-BcContainer, C:\ProgramData\BcContainerHelper\6.0.6-preview1131\BcContainerHelper\ContainerInfo\Test-NavContainer.ps1: line 34
at <ScriptBlock>, C:\ProgramData\BcContainerHelper\6.0.6-preview1131\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 989
at <ScriptBlock>, C:\ProgramData\BcContainerHelper\6.0.6-preview1131\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 966
at <ScriptBlock>, C:\ProgramData\BcContainerHelper\6.0.6-preview1131\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 948
at Run-AlPipeline, C:\ProgramData\BcContainerHelper\6.0.6-preview1131\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 910
at CreateDevEnv, C:\Users\JonasWrede\AppData\Local\Temp\tmp785E.tmp.ps1: line 1894
at <ScriptBlock>, C:\Users\JonasWrede\Projects\byndit\App\.AL-Go\localDevEnv.ps1: line 128
at <ScriptBlock>, <No file>: line 1

I think the issue is that docker inspect can't find a container
$inspect = docker inspect $id | ConvertFrom-Json

Has anybody encountered it?

I reverted to 4.25.2 that solved it.

Yeah, this was reported here https://github.com/microsoft/navcontainerhelper/issues/3326 as well.
Nothing really we can do about that until docker releases 4.28.0
Personally, I use docker engine only, which doesn't have this problem.

Thanks for the tip. Feel free to close the ticket.


sdist not uploaded for last release

In checking why https://github.com/conda-forge/asv_runner-feedstock didn't receive an automatic PR, I realised that for the last release no sdist was uploaded. Could you add that, please?

Works! Thanks!


Pointer is shaking and lags when the player is moving

Environment

Source of VRTK (Github)
Version of VRTK (3.3.0-alpha)
Version of Unity3D (Unity 2017.3)
Hardware used (Oculus)
SDK used (Oculus Utilities)

Steps to reproduce

Attach the player (SDKManager root) to a moving object (e.g. car, platform, etc).
(this can be by making it a child of the other moving object, or by adding a VRTK_TransformFollow component)
While moving, press to show the pointer
See that the line rendered is shaking because its position is only updated every so often (fixedUpdate) instead of every frame

Expected behavior
Pointer should be smooth and attached to the hand always
Current behavior
Pointer origin is not always in the right position, which causes the pointer to show a ghost effect when moving.

Fixed update happens way more every frame than late update. The problem you're probably facing is the fixed update is in the physics time step but oculus is running at 90fps but the physics time step is only running at 60fps. Try setting your physics time step to 1/90 so it's also at 90fps

Pointer rendering should be independent of Physics update.
Increasing the physics update to match the rendering is not always feasible, as you maybe skipping frames, and different systems have different update frequencies (90Hz, 120Hz, etc).
In this case, what's happening is that the fixed update is not updating as often as the late update (which is great for performance) causing this out-of-sync.
Can you please, describe a situation where updating the pointer every frame could cause issues?

The fixed update is required for when the pointer is interacting with things, especially when using the pointer to grab.

You'll never skip frames if you match the physics timestep to the refresh rate, it's actually going to give you a better experience.
For example, the SteamVR plugin automatically sets the physics timestep to match the refresh rate.
If you want to have it so Oculus is always matching the refresh rate then just write a small script that sets the timestep based on the refresh rate.
Time.fixedDeltaTime = Time.timeScale / UnityEngine.XR.XRDevice.refreshRate;


PicoGK runtime as emscripten lib for webGL ?

Hello,
Is it possible to get access to a wasm/emscripten version of the lib, for using it through a web app, or at least know the technical feasibility (especially because of dependencies)?
Thank you

I do not have experience with wasm/emscripten. While the PicoGK runtime is fairly straightforward, as it's generally header-only, compiling openvdb and its dependencies is probably the biggest hurdle. It's not on our list of things to do, but the PicoGK runtime is open, so anyone can have a look whether it's possible.


(docs): update fern cli commands docs

remove fern register
update fern --version docs (showed the command as fern version, which doesn't work)
update fern --help docs to show current list of commands (was missing write-definition, token, etc) and show usage for help on specific commands

@minaelee please fix the failing CI Path ./docs/pages/overview/cli/cli-commands/fern-register.mdx does not exist

@minaelee please fix the failing CI Path ./docs/pages/overview/cli/cli-commands/fern-register.mdx does not exist

Done


feat: Migrate to bzlmod

Try migrating to bzlmod.
Fixes #51

@chenrui333 Could you please take a look? Thanks. I did not update README.md because I think it's better to make this into BCR first before updating any instruction.


fix failing pytest for config module

There are a number of pytests that are now failing due to recent code modifications. This PR fixes the tests for the invokeai.app.services.config module. It does not fix other failing lists.

LGTM - merging in


HTML Export with doFile produces error

When exporting to HTML and using the doFile function with JavaScript, the Web version complains about missing file even though the cartridge seem to have been exported. You can see a flicker of the game starting first.
It does not help copying the file to the same location either,

Temp workaround is to copy the code manually to the editor and avoid using doFile

html export is complaining because you use dofile, pls delete it before release, it's only for development stage and helps to use external editors

Fair enough


Video in docs still shows the planetary network in left menu.

https://manual.grid.tf/getstarted/TF_Connect/TF_Connect.html

Test Environment:
TFconnect version: 3.9.0-176
Android version: 12
kernel version: 4.14.113-25741823

@ramezsaeed @xmonader
Do we simply remove the video or we create another one?

@ramezsaeed @xmonader
We can remove the video and the picture with the planetary network being shown, but I think we could just let it all there, since the video does not focus on this aspect. It's outdated but not in the way of the video explanation.
If we want to remove it, I can do a quick PR with:

remove the planetary reference

remove the embedded video
crop the image showing the planetary network

Let me know if we keep as is and just don't mind the planetary network reference, or if we want to remove as explained above.

@Mik-TF
For me I recommend to remove it yes, so we avoid any miss leading to planetary network tab which is not available anymore in the current release.
lets see @xmonader opinion here too.

Good! Once, we get a confirmation from Thabet, I will remove what needs to be removed and make a PR.

Fix when #346 is merged.


chore: fixing running task in a proper zone preserving the active zone

Which problem is this PR solving?

Partly fixes this:
https://github.com/open-telemetry/opentelemetry-js/issues/1206

For full fix another PR is needed from sdk
https://github.com/open-telemetry/opentelemetry-js/pull/1209
Short description of the changes

it run the task in active zone preserving the spawned zone instead of root zone

@open-telemetry/javascript-maintainers  ^^

Thank you for getting this fixed @obecny. Can we release a new version for this package any time soon?

Yes I'll try to cut this today or tomorrow


BetaRegularized can have negative result

Example: MathNet.Numerics.SpecialFunctions.BetaRegularized(6985172036240.918,<PHONE_NUMBER>093.4609, 0.581395348837209); returns -472.91265397822207. The same in R pbeta(0.581395348837209,<PHONE_NUMBER>240.918,<PHONE_NUMBER>093.4609); returns 0.5.

Thanks for reporting this. I can reproduce the issue.
For reference, Mathematica returns 0.495 with these numbers.

Fixed in #497


Amplify can not even be used as a real product...

You have zero information about how to host things on a custom domain - just a cloudfront domain and nothing else.  No documentation what-so-ever and it doesnt work by just pointing to it. Wow.  All information has to do with AWS Amplify Console which has nothing included at all so I can't use that anyway.

403 ERROR
The request could not be satisfied.
Bad request.
Generated by cloudfront (CloudFront)
Request ID: qIq6obslqWpGJhKlNnXffSE73D6bUJ94oVCkrGv-ik-PcUSQDi7Y_A==

Clearly it needs cnames configuration but since its all CloudFormation if i modify all that manually it will break and the whole app will never work again at all...

Your upload time are ridiculous - I can deploy my app 5 times using a standard shell script by the time it gets finished with "Uploading Files"

Your AWS Amplify console doesn't even work in the first place... it shows like 10 out of 50+ branches and I can't search to find it or select it so its literally useless

Literally make it IMPOSSIBLE to use your products ALL THE TIME... this is almost as bad as the cognito js sdk you had which had 6 interfaces for doing async callbacks and didnt even provide enough information to properly reset tokens.

It appears you don't know how to use various elements of the AWS platform or the internet at large, which might be a part of why you're running into issues. If your domain name is in a Route 53 hosted zone, you can setup an Alias (AWS construct specific to AWS services like load balancers and cloudfront) pointing to your cloudfront distribution, add the relevant CNAMEs to CloudFront, and be good to go with your custom domain.
I have to say though, this post is so rude and unconstructive that I don't know what you can possibly expect to achieve with it.

Hello @bradennapier. Sorry you're having difficulties. We want to help make you successful, however we do want all community members to communicate with each other and ask for help in a respectful manner. Please do be aware we have a Code of Conduct that should be followed: https://github.com/aws-amplify/amplify-js/blob/bfa477662591bbd7a92bbb8486778924589e4adb/.github/CODE_OF_CONDUCT.md
Turning back to the issues at hand, they appear to be related to the Amplify Console (https://aws.amazon.com/amplify/console/) which is separate from the Amplify Framework, an Open Source project which this CLI is a part of. The place to ask questions for the Amplify Console is on the AWS forum (https://forums.aws.amazon.com/forum.jspa?forumID=314&start=0) as there are support engineers monitoring it who represent the service. However, I've also reached out to the Product Manager @swaminator who is happy to chat with you 1-1 and assist.

@undefobj thank you for your reply, i will redirect the amplify console issues there.
@jkeys-ecg-nmsu no we do not use route53 for handling our DNS, and we have many CNAMEs already setup and working - the issue is that it doesn't work and doesn't have any way to setup the CNAME properly in the cli itself.
In my experience making any modification to a CF built resource is basically a fatal mistake which can be difficult to come back from.  If this is a random edge case that doesn't break if modified than it should be noted somewhere / anywhere that if you want to use custom domains you have to manually adjust that value.

The upload time to upload the distribution is also an issue - taking around 10-15 minutes on every publish when I can do it many times in the same timeframe by just using the s3 api directly via bash.   It's not a small bundle, mind you, but the fact using the aws tools themselves to accomplish this have no issue with speed is my concern here.

@bradennapier: Sorry to hear that you cannot connect your repository to the Amplify Console. I have added a ticket in our engineers queue to look into why your branch isn't connecting. As a short-term remedy, can you try connecting one of your branches without a '/'? Also if you want to email me at<EMAIL_ADDRESS>I'd be happy to help you out.

@swaminator thank you for your reply, the issue is you only support the first like 10 branches - i cant even connect master -- i had to rename it to A-${branchName} so it would be the first just to select it.
Now it wont let me select domains and just sits there loading forever with tons of errors

@bradennapier are you calling amplify publish -c? If so the latency is because CloudFront is invalidating, and Amplify won't return successful until the invalidation succeeds. If not then that sounds like a valid issue.
I can tell you that I have Amplify websites in production that use custom domains that were manually configured once, and only once. I don't know if that makes you feel better about just adding the CNAMEs manually, but I can tell you that in my experience, it works.
In fact, the reason that adding CNAMEs isn't a part of the CLI configure is because it can't know your defaults, so every re-run of amplify hosting configure would break working URLs.

@bradennapier can you share the console log errors you're seeing?

@swaminator I know this isn't your team, but I'd like to note that the behavior that @bradennapier afflicts some other consoles, like the IAM Group console. I've had to prefix "AWS_" to Roles in the past in order to get them to appear because the IAM Group console only presented the first n options (in chrome at least).

Not too hard to get around that, and the custom domains that way make it so you can not use anything but amplify for your entire domain - all internal needs are completely lost and can't be done.  So I was hoping to try a fourth-level and point web.domain.com to Route53 and support *.web.domain.com -- although even the most basic features are not working without any of that atm.  From this I can easily edit the CloudFront with the CNAME's that i desire but that does take branch deploys off the table which leaves me with what i have today anyway.
@swaminator i sent you a far more detailed account of every failure along the way of trying to set this up and all the problems with the console (i am not joking not a single step worked even just default along the way).
We use github.  You show 30 branches and then no more - you could argue having more is crazy but we have a large team and a large repo.  Either way it shouldn't just fail to support it.

Did you try multiple browsers?

On that note, I truly don't mean to be so "rude" but when a product is provided as release-level I don't expect it to essentially not work and waste my entire day and/or teams day reading the documentation, learning if it fits our needs, and then getting to a proof of concept and it failing at every step.
@jkeys-ecg-nmsu yes as soon as you said that above I tried Firefox with the same effect.


fixing a typo

Thanks @drewda. Can you sign our Contributor License Agreement (CLA) and then we'll get this merged.

@JoeyZwicker sure, done.

CI passed here: https://github.com/pachyderm/pachyderm/pull/2524


In Public: a free or fixed-price Datastream asset displays both a price and Subscription Option prices

When creating a free datastream, as opposed to priced, the subscription options still appear. Apparently the options saved in the Publisher's local storage get used.
Once published, the Subscription Options appear, but procuring the asset still works. The “Copy Feed URL” button displays and does indeed copy the URL. But for the Consumer it’ll be confusing because it looks like they're paying.

Just to be sure, I checked the transactions and the Consumer is not paying for the free asset.
Fyi, when procuring an asset, different method names appear in Polygonscan:
Free: “Buy From Dispenser And Order”
Priced: “Buy From Fre And Order”

Example free asset: https://oceanmarket-datastreams.netlify.app/asset/did:op:235a2b8ec523d0fe7d085cc3fe169ba3877808819f0a076b3e597b5a4124d872

Also a one-time FIXED payment datastream asset displays the Subscription options.
The price displayed is correct, 5 OCEAN in the test asset: https://oceanmarket-datastreams.netlify.app/asset/did:op:738ebf67cef834cf94eeb4cfbac58109269e9db7376d10626e67957370cd32ee
But the Subscription Options should be there.

See issue #31, this is another result of the Local Storage being persistent.


bicep extention doesn't resolved modules names and version from private BR repo

I have created a repo with modules. When I reference them, the vscode bicep extension doesn't provide hints about names and versions, etc. It does it for the public bar. Why not for private br?
Is it a configuration issue, permissions, or it's 'not a bug, it's a feature' ;)?

Completions for private registries is being worked on but not yet implemented.


🐛 Unexpected error: Tried to cast node with kind JS_EXPRESSION_STATEMENT as "biome_js_syntax::generated::nodes::AnyJsExpression" but was unable to cast

Environment information
CLI:
  Version:                      1.4.1
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v18.15.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "pnpm/8.6.2"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 false

Workspace:
  Open Documents:               0

What happened?
I couldn't reproduce it in the playground, but here is my local code that crashes biome: https://biomejs.dev/playground/?code=aQBtAHAAbwByAHQAIABSAGUAYQBjAHQALAAgAHsARgBDACwAIABGAHIAYQBnAG0AZQBuAHQAfQAgAGYAcgBvAG0AIAAnAHIAZQBhAGMAdAAnAAoAaQBtAHAAbwByAHQAIAB7AHUAcwBlAFMAZQBsAGUAYwB0AG8AcgAsACAAdQBzAGUARABpAHMAcABhAHQAYwBoAH0AIABmAHIAbwBtACAAJwByAGUAYQBjAHQALQByAGUAZAB1AHgAJwAKAGkAbQBwAG8AcgB0ACAAewBJAGMAbwBuACwAIABUAG8AbwBsAHQAaQBwAH0AIABmAHIAbwBtACAAJwBAAG0AdQBpAC8AbQBhAHQAZQByAGkAYQBsACcACgBpAG0AcABvAHIAdAAgAHsAcAB1AHMAaAB9ACAAZgByAG8AbQAgACcAYwBvAG4AbgBlAGMAdABlAGQALQByAGUAYQBjAHQALQByAG8AdQB0AGUAcgAnAAoAaQBtAHAAbwByAHQAIAB7AFMAdABhAHQAZQB9ACAAZgByAG8AbQAgACcALgAuAC8ALgAuAC8AdAB5AHAAZQBzAC8AcwB0AGEAdABlACcACgAKAGkAbgB0AGUAcgBmAGEAYwBlACAAUAByAG8AcABzACAAewAKACAAIABjAGgAaQBsAGQAcgBlAG4APwA6ACAAUgBlAGEAYwB0AC4AUgBlAGEAYwB0AE4AbwBkAGUACgB9AAoACgBjAG8AbgBzAHQAIABMAGkAbQBpAHQAZQBkAFAAbABhAG4AVABvAG8AbAB0AGkAcAA6ACAARgBDADwAUAByAG8AcABzAD4AIAA9ACAAKAB7AGMAaABpAGwAZAByAGUAbgB9ACkAIAA9AD4AIAB7AAoAIAAgAGMAbwBuAHMAdAAgAGQAaQBzAHAAYQB0AGMAaAAgAD0AIAB1AHMAZQBEAGkAcwBwAGEAdABjAGgAKAApAAoAIAAgAGMAbwBuAHMAdAAgAHAAYQB5AG0AZQBuAHQAUABsAGEAbgAgAD0AIAB1AHMAZQBTAGUAbABlAGMAdABvAHIAKAAoAHMAdABhAHQAZQA6ACAAUwB0AGEAdABlACkAIAA9AD4AIABzAHQAYQB0AGUALgBhAGMAYwBvAHUAbgB0AFMAdABhAHQAZQAuAGEAYwBjAG8AdQBuAHQAPwAuAHAAYQB5AG0AZQBuAHQAUABsAGEAbgApAAoACgAgACAAYwBvAG4AcwB0ACAAaABhAHMAUwB0AGEAcgB0AGUAcgBQAGwAYQBuACAAPQAgAHAAYQB5AG0AZQBuAHQAUABsAGEAbgAgACYAJgAgAHAAYQB5AG0AZQBuAHQAUABsAGEAbgAuAHMAdABhAHIAdABzAFcAaQB0AGgAKAAnAG4AXwBzAHQAYQByAHQAZQByACcAKQAKAAoAIAAgAGkAZgAgACgAaABhAHMAUwB0AGEAcgB0AGUAcgBQAGwAYQBuACkAIAB7AAoAIAAgACAAIAByAGUAdAB1AHIAbgAgACgACgAgACAAIAAgACAAIAA8AFQAbwBvAGwAdABpAHAACgAgACAAIAAgACAAIAAgACAAdABpAHQAbABlAD0AIgBVAG4AbABvAGMAawAgAHQAaABpAHMAIABmAGUAYQB0AHUAcgBlACAAYgB5ACAAdQBwAGcAcgBhAGQAaQBuAGcAIAB0AG8AIAB0AGgAZQAgAEIAdQBzAGkAbgBlAHMAcwAgAHAAbABhAG4AIgAKACAAIAAgACAAIAAgACAAIABjAGwAYQBzAHMATgBhAG0AZQA9ACIAdAB4AHQALQBwAG8AaQBuAHQAZQByACIACgAgACAAIAAgACAAIAAgACAAbwBuAEMAbABpAGMAawA9AHsAKAApADoAIAB2AG8AaQBkACAAPQA%2BACAAewAKACAAIAAgACAAIAAgACAAIAAgACAAZABpAHMAcABhAHQAYwBoACgAcAB1AHMAaAAoACcALwBlAG4ALwBkAGEAcwBoAGIAbwBhAHIAZAAvAGEAYwBjAG8AdQBuAHQAJwApACkACgAgACAAIAAgACAAIAAgACAAfQB9AAoAIAAgACAAIAAgACAAPgAKACAAIAAgACAAIAAgACAAIAA8AEkAYwBvAG4APgBsAG8AYwBrAF8AbwBwAGUAbgA8AC8ASQBjAG8AbgA%2BAAoAIAAgACAAIAAgACAAPAAvAFQAbwBvAGwAdABpAHAAPgAKACAAIAAgACAAKQAKACAAIAB9ACAAZQBsAHMAZQAgAHsACgAgACAAIAAgAHIAZQB0AHUAcgBuACAAPABGAHIAYQBnAG0AZQBuAHQAPgB7AGMAaABpAGwAZAByAGUAbgB9ADwALwBGAHIAYQBnAG0AZQBuAHQAPgAKACAAIAB9AAoAfQAKAAoAZQB4AHAAbwByAHQAIABkAGUAZgBhAHUAbAB0ACAATABpAG0AaQB0AGUAZABQAGwAYQBuAFQAbwBvAGwAdABpAHAACgA%3D
The error specifically triggers on {children} on this line.
return <Fragment>{children}</Fragment>
This is the error report it spits out:
/usr/local/bin/pnpm biome:fix-file /Users/marcelpanse/Documents/Sources/dashboard/src/components/limitedPlan/LimitedPlanTooltip.tsx

unless@ biome:fix-file /Users/marcelpanse/Documents/Sources/dashboard
biome check --apply-unsafe --error-on-warnings --formatter-enabled=true --linter-enabled=true --organize-imports-enabled=true  "/Users/marcelpanse/Documents/Sources/dashboard/src/components/limitedPlan/LimitedPlanTooltip.tsx"

Biome encountered an unexpected error
This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:
Source Location: /Users/runner/work/biome/biome/crates/biome_rowan/src/ast/mod.rs:230:13
Thread Name: biome::worker_0
Message: Tried to cast node with kind JS_EXPRESSION_STATEMENT as "biome_js_syntax::generated::nodes::AnyJsExpression" but was unable to cast
/Users/marcelpanse/Documents/Sources/dashboard/src/components/limitedPlan/LimitedPlanTooltip.tsx internalError/panic  INTERNAL  ━━━━━━━━━━
× processing panicked: Tried to cast node with kind JS_EXPRESSION_STATEMENT as "biome_js_syntax::generated::nodes::AnyJsExpression" but was unable to cast
! This diagnostic was derived from an internal Biome error. Potential bug, please report it if necessary.
Fixed 1 file(s) in 1976µs
Process finished with exit code 0
Expected result
It should normally lint it
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

UPDATE workaround:
Changing the  for a  or <> solves the problem.

Hi! Thanks to report this bug! Do you encounter the bug when you disable the formatter? or the linter?


Fallback to use new discovery uri pattern when the old one fails.

Fallback to use new discovery uri pattern when the old one fails. Some Google API are now using the new Discovery url style.

Please address the CLA complaint by both authoring and committing from either your google.com account or an account that has signed Google's CLA.


text does not display the text as crossed out

Version 4.6.2

Sending ~~text~~. This text should be spanned as crossed out
I see what I send ("~~text~~", not crossed out).

For example, for "**" - "**bold**" after toHtml() on Spanned i see "
bold
" and this is correct
But After "~~" - "~~text~~" it is "
~~hello~~
"

I have used Strikethrough extension and it helped :)


1.0?

Hi @wojtekmach!
Req has become the go-to tool for me and some folks I know. We use it in production at work and it works well for us. With that said, do you know when a 1.0 will happen? Is there anything you'd like to do before a 1.0? I'm just curious, saw the 0.4 release and thought I'd reach out.
Thank you for your work ❤

I consider the 0.4 api stable, if your code works without warnings now it will work on and after 1.0. But stuff can still come up so I leave room for breaking changes but they will be carefully considered. Plan is to have 1.0 in a few months.

Awesome work, thank you!

Thank you for the kind words btw!

❤
Sometimes we don't celebrate the good as much as we should. The bad on the other hand, gets WAY more attention.
Gotta say the good when you see it! And, in your case, when it's earned. Great job!


Search not responding

I just put in a search for "fig|1121866.3.peg.99" at the production site and it never returned. While it was spinning I brought up a new tab and the search worked immediately.
New relic monitors showed issues too.

BTW how much do we pay them? The new relic site seems astonishingly slow.

It seems to be working now. Not sure what can be done here.
Assigning it to Dustin, in case he wants to comment on the new relic and/or the snapshot and close it.
-Maulik


possible regession in normalization of gradients

[x] Check that you are up-to-date with the master branch of keras-vis. You can update with:
pip install git+git://github.com/raghakot/keras-vis.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Hello!
In https://github.com/raghakot/keras-vis/commit/40b27dfa3ecb84cdde5ec6b44251923c3266cc40 this
https://github.com/raghakot/keras-vis/blob/14f3f61b2a07fa07164b2bb8ac1a6e5aed4936a6/vis/optimizer.py#L54-L55
was changed to this.
https://github.com/raghakot/keras-vis/blob/668b0e11dab93f3487f23c17e07f40554a8939e9/vis/optimizer.py#L62-L63
However those two expressions are not equivalent, the output they give is very different in results, possibly due to an additional an additional axis= parameter in l2_normalize that defaults to None. See https://github.com/petewarden/tensorflow_makefile/blob/master/tensorflow/g3doc/api_docs/python/functions_and_classes/tf.nn.l2_normalize.md
I will post examples to show the difference, but the current configuration gives slow convergence and high loss when using ActivationMaximization(Loss) only.

@ju-w , Thank you for your report!
As you said, it seems that the previous implementation better than currently it.
It's same as Keras's example implementation.
https://github.com/keras-team/keras/blob/7bee9a18d83899ce6dfd50c4883afc678139f4ad/examples/conv_filter_visualization.py#L57-L59
And it might be the way to fix issue / #109 ( Unfortunately, #143 was not fixed.) .
Could you contribute to fix this issue?
@raghakot , I belive we should choice your previous l2_normalize implementation.
But if you know the case which the currently implementation is better, please let us know.


can't use production mode when use React.lazy to import component, but the develoption mode is ok.

webpack.prod.config
{
mode: process.env.NODE_ENV,
module: {
rules: [
{
test: /.(js|mjs|jsx|ts|tsx)$/,
include: path.resolve("./src"),
loader: 'babel-loader',
exclude: /node_modules/,
},
......,
]
},
......,
optimization: { splitChunks: { chunks: "all" } },
}
.babelrc
{
"presets": [
"@babel/preset-react",
"@babel/preset-env",
"@babel/preset-typescript"
]
}
when i use cross-env NODE_ENV=development to build, it's work well, but cross-env NODE_ENV=production will get following error:

env:
"webpack": "^5.70.0",
"webpack-cli": "^4.9.2",
"babel-loader": "^8.2.3",
"@babel/preset-env": "^7.16.11",
"@babel/preset-react": "^7.16.7",
"@babel/preset-typescript": "^7.16.7",
Does anyone have this problem ?

I‘m sure that the problem is react.lazy(()=>import('./add.tsx')), when i use import Add from './add', this well be ok.

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us by recreating the issue using one of our templates.
Please fill out all fields, we can't help you without it. Also will be great to create reproducible repo.


Classes in Media Queries ignored

I have a clean project, i have installed bulma css which i import into a less file. When i run webpack, all the classes within media queries are ignored. I have googled it, and i found this bug in purifycss, but that bug is fixed in version 1.0.15
I am user purify-css version 1.2.6, purifycss-webpack version 0.7.0. Here is my code:
main.less
@import "../../node_modules/bulma/css/bulma.css";

webpack.config.js
var webpack = require("webpack")
var path = require('path')
var glob = require('glob');
var ExtractTextPlugin = require('extract-text-webpack-plugin');
var PurifyCSSPlugin = require('purifycss-webpack');
var CleanWebpackPlugin = require('clean-webpack-plugin');
var inProduction = (process.env.NODE_ENV === 'production');

module.exports = {
  entry: {
    main: [
        './src/js/main.js',
        './src/less/main.less'
      ]
  },
  output: {
    path: path.resolve(__dirname, './dist'),
    filename: '[name].[chunkhash].js'
  },
  module: {
    rules: [
      {
        test: /\.less/,
        use: ExtractTextPlugin.extract({
          use: ['css-loader', 'less-loader'],
          fallback: 'style-loader'
        })
      },

    ]
  },

  plugins: [
    new CleanWebpackPlugin(['dist'], {
      root: __dirname,
      verbose: true,
      dry: false
    }),

    new ExtractTextPlugin('[name].[chunkhash].css'),

    new PurifyCSSPlugin({
      paths: glob.sync(path.join(__dirname, 'index.php')),
      minimize: inProduction
    }),

    new webpack.LoaderOptionsPlugin({
      minimize: inProduction,
      debug: !inProduction
    }),

    function () {
      this.plugin('done', stats => {
        require('fs').writeFileSync(
          path.join(__dirname, 'dist/manifest.json'),
          JSON.stringify(stats.toJson().assetsByChunkName)
        );
      });
    }
  ]
};

if(inProduction) {
  module.exports.plugins.push(
    new webpack.optimize.UglifyJsPlugin()
  )
}

My index.php

        First column

        Second column

        Third column

        Fourth column

without the purifycss-webpack plugin i get the css code of bulma, but with the plugin i get only a couple of lines css, without any media query

Same happens for me using Bulma and Vue.js.

I tracked the problem down to purify-css not handling :not correctly. The problematic media query in Bulma that gets removed:
@media screen and (min-width: 769px), print
columns.sass:204
.columns:not(.is-desktop) {
    display: -webkit-box;
    display: -ms-flexbox;
    display: flex;
}

Upstream issue: https://github.com/purifycss/purifycss/issues/161

I think this issue is still not fixed, because PurifyCssPlugin still removes way too much Bulma CSS code.
I tried using the 'whitelilst' option but then my webpack build fails.


When using the option --use-babel-runtime the generated code will not require the runtime

I am trying to instrument a source that uses generators and iterators.Invoking
babel-istanbul instrument --use-babel-runtime src/ --output instrumented/

will produce instrumented code that does not include the required runtime.
Which might be due to the fact that
optional	[]	Array of transformers to optionally use. Run babel --help to see a full list of transformers. Optional transformers displayed inside square brackets.

and you simply pass in
lines<EMAIL_ADDRESS>            if (this.opts.useBabelRuntime) {
                babelConfig.optional = 'runtime';
            }

Yet, when patching the code so that it will pass in an array, it will still not work. Might this be related to the fact, that the babel specific options are never actually processed in lib/command/instrument.js?

Are you wanting the whole runtime included in your instrumented code? I'm not sure I see a use case there, as you'll be including a bunch of code that might not get covered.
--use-babel-runtime passes a flag to babel tells the compiler to use the babel-runtime instead of generating verbose code every time. The instrumented code should never include the actual runtime, but it will reference helper functions from babel-runtime instead.

No, of course not, it should still be optional. It seems though, as if the option is not being evaluated at all:
This is the this.opts from the instrumenter:
{"coverageVariable":"__coverage__","embedSource":false,"noCompact":false,"preserveComments":false}

As you can see, the instrument command will not properly configure the options from the command line...

Ah, I set up the commands for run-with-cover specifically. You are using the instrument command. I can update the instrument command to utilize the applicable arguments.

Okay, I see. This would be great since it actually makes more sense to have the instrumented code use the runtime, too!
TIA!

Please do not forget to also change the following lines in lib/instrumenter.js
 444             if (this.opts.useBabelRuntime) {
 445                 babelConfig.optional = ['runtime'];
 446             }

According to the spec, babel expects this to be an array, dunno if it will work with just a string.

Just published 0.2.5, it should be updated. Thanks!

Great, thanks.


Add a json configurable implementation that supports any of zenoh, mqtt and someip

We currently have a reference implementation for a zenoh-someip streamer. In addition to that there will be an implementation for a zenoh-mqtt streamer coming soon. These implementations can be great for people trying to understand a basic streamer setup or as a reference for people who want to build their own custom implementation using the streamer lib. However it might be nice to have a fully configurable streamer implementation as well, that supports all three transports and only needs to be configured by a config json.

I put some limited thoughts into what the config json should look like. Not very happy yet... I feel like some of the information is duplicate but maybe it can serve as a starting point.
{
  up_streamer_config: {
    ...
  },
  streamer_uuri: {
    ...
    },
  usubscription_config: {
    ...
  },
  host_transport: "zenoh",
  enabled_transports: [
    "zenoh",
    "mqtt",
    "someip"
  ],
  forwarding: [
    "ZenohToMqtt",
    "ZenohToSomeip",
    "MqttToZenoh",
    "MqttToSomeip",
    "SomeipToZenoh",
    "SomeipToMqtt"
  ],
  zenoh_config: {
    important: "zenohstuff"
  },
  someip_config: {
    important: "someipstuff"
  },
  mqtt_config: {
    important: "mqttstuff"
  }
}

The flow of the streamer implementation would then be something like this:

get config
set up the streamer
create the host transport
create all other transports
create all endpoints
add all of the selected forwardings to the streamer


Default top_k value (or something else) causes nonsensical outputs

Bug description
With the current default top_k setting (set to 50) we get nonsensical results most (but not all) of the time:
from litgpt import LLM

llm = LLM.load("mistralai/Mathstral-7B-v0.1")
print(llm.generate("What is 1+2?"))

#  In your job, what is the average time people spend using your app?
# These can have wildly different answers depending upon the dataset,
# the timeframe for the dataset, and how the data is being calculated or stated, disaggregated and re

print(llm.generate("What is 1+2?", top_k=1))
#  1+2 equals 3.

del llm
llm = LLM.load("microsoft/phi-2")

llm.generate("What do Llamas eat?")
# ' Curation Level:\n'

llm.generate("What do Llamas eat?")
# ' Peculiar\n'

llm.generate("What do Llamas eat?")
# ' Llamas eat grass, leaves, and shrubs.
# They prefer to graze on coarse vegetation and can
# consume as much as 30% of their body weight in a single day.\n'

llm.generate("What do Llamas eat?", top_k=1)
# ' Llamas are herbivores and primarily eat grass, hay, and other plant material.
# They have a unique digestive system that allows them to efficiently extract
# nutrients from fibrous plant material.\n'

However, don't remember it being that bad. It also seems to be only that bad in the Python API. I wonder if this was perhaps caused by the updated kv-cache settings via #1590
CC @Andrei-Aksionov
What operating system are you using?
Linux
LitGPT Version
0.4.5

Thank you for adding "(or something else)", this is the point I was trying to make in the PR.

A simple test to rule out kv cache can be to run with and without it (since kv cache should only affect speed).

@awaelchli  Thanks! I am fairly certain now that it was an incomplete kv-cache clearing (#1596)

We addressed this in #1596


MyApp.Secrets must implement the 'AshAuthentication.Secret` behaviour.

I followed the example here (https://ash-hq.org/docs/guides/ash_authentication/latest/github-quickstart) but ran into this error. It talks about password_reset which I'm not sure where it's coming from.
** (EXIT from #PID<0.104.0>) an exception was raised:
    ** (Spark.Error.DslError) [nil]
 password_reset:
  `MyApp.Secrets` must implement the `AshAuthentication.Secret` behaviour.
        lib/MyApp/accounts/resources/user.ex:1: anonymous fn/1 in MyApp.Accounts.User.__verify_ash_dsl__/1
        (elixir 1.14.1) lib/enum.ex:975: Enum."-each/2-lists^foreach/1-0-"/2
        lib/MyApp/accounts/resources/user.ex:1: MyApp.Accounts.User.__verify_ash_dsl__/1
        (elixir 1.14.1) lib/enum.ex:975: Enum."-each/2-lists^foreach/1-0-"/2
        (elixir 1.14.1) lib/module/parallel_checker.ex:239: Module.ParallelChecker.check_module/2
        (elixir 1.14.1) lib/module/parallel_checker.ex:78: anonymous fn/5 in Module.ParallelChecker.spawn/3

Sorry, it was an error on my side, typo in the module name :(


ERROR: Can't guess frontend for input file `-ql' (missing -f option)!

Steps to reproduce the issue
make test &> test.log

Expected behavior
No errors
Actual behavior
A bunch of log lines which ends with the following error (see also the full log here)
cd tests/aiger && bash run-test.sh ""
Checking and_.aag.
Checking buffer.aag.
Checking cnt1.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking cnt1e.aag.
Checking empty.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Warning: The current network has no primary outputs. Some commands may not work correctly.
Checking false.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking halfadder.aag.
Checking inverter.aag.
Checking notcnt1.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking notcnt1e.aag.
Checking or_.aag.
Checking symbols.aag.
Checking toggle-re.aag.
Checking toggle.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking true.aag.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking and_.aig.
Checking buffer.aig.
Checking cnt1.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking cnt1e.aig.
Checking empty.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Warning: The current network has no primary outputs. Some commands may not work correctly.
Checking false.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking halfadder.aig.
Checking inverter.aig.
Checking notcnt1.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking notcnt1e.aig.
Checking or_.aig.
Checking symbols.aig.
Checking toggle-re.aig.
Checking toggle.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Checking true.aig.
Warning: The new network has no primary inputs. It is recommended
to add a dummy PI to make sure all commands work correctly.
Running neg.ys.

 /----------------------------------------------------------------------------\
 |                                                                            |
 |  yosys -- Yosys Open SYnthesis Suite                                       |
 |                                                                            |
 |  Copyright (C) 2012 - 2020  Claire Wolf<EMAIL_ADDRESS>        |
 |                                                                            |
 |  Permission to use, copy, modify, and/or distribute this software for any  |
 |  purpose with or without fee is hereby granted, provided that the above    |
 |  copyright notice and this permission notice appear in all copies.         |
 |                                                                            |
 |  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES  |
 |  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF          |
 |  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR   |
 |  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES    |
 |  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN     |
 |  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF   |
 |  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.            |
 |                                                                            |
 \----------------------------------------------------------------------------/

 Yosys 0.9+3575 (git sha1 7affef7c, gcc 7.5.0_1 -fPIC -Os)

-- Executing script file `neg.ys' --

1. Executing Verilog-2005 frontend: <<EOT
Parsing Verilog input from `<<EOT' to AST representation.
Generating RTLIL representation for module `\top'.
Successfully finished Verilog frontend.

2. Shell command: rm -rf neg.out

3. Shell command: mkdir neg.out

4. Executing SIMPLEMAP pass (map simple cells to gate primitives).
Mapping top.$and$<<EOT:2$1 ($and).

5. Executing AIGER backend.

6. Executing AIGER frontend.
<suppressed ~385 debug messages>
Removed 0 unused cells and 192 unused wires.

7. Shell command: rm -rf neg.out

8. Shell command: mkdir neg.out

9. Executing SIMPLEMAP pass (map simple cells to gate primitives).
Mapping top.$and$<<EOT:2$1 ($and).

10. Executing XAIGER backend.
<suppressed ~5 debug messages>

11. Executing AIGER frontend.
<suppressed ~385 debug messages>
Removed 0 unused cells and 192 unused wires.
ERROR: Can't guess frontend for input file `-ql' (missing -f option)!
make: *** [test] Error 1

Ah, I see the issue, this is due to different behavior of getopt() in some systems (do you happen to use mac or BSD?) that doesn't allow you to put optional arguments after positional arguments.


autohide support for paging

Feature request:
It would be great if autohide would have option to autohide based on what is visible on a page, not just check if column is empty for whole page:
https://datatables.net/reference/option/paging

...to autohide based on what is visible on a page

Not sure what you mean. Can you give an example? And whats the DT paging link for?

paging was the feature I was referring to, therefore I have included link.
Basically if you have 100 rows with columns A, B and C. Paging is set for 20 rows.
Column C has values only in row 72-75.
So if page 0 is displayed (rows from 0 to 19) - C is hidden.
So if page 1 is displayed (rows from 20 to 39) - C is hidden.
So if page 2 is displayed (rows from 40 to 59) - C is hidden.
So if page 3 is displayed (rows from 60 to 79) - C is shown.
So if page 4 is displayed (rows from 80 to 99) - C is hidden.
With current plugin in all pages C column is shown event if its empty in 4 out of 5 pages.
I have sample code which does what I want, it just needs some "nice wrapping":
(function (window, document, $) {
    $(document).on('page.dt', function (e, dtSettings) {
        var api = new $.fn.dataTable.Api(dtSettings);
        var info = api.page.info();
        var emptyCount = 0;

        api.columns().every(function () {
            emptyCount = 0
            var data = this.data();
            for (var i = info.start; i < info.end; i++)
                if (data.toArray()[i] === null || data.toArray()[i].length === 0)
                    emptyCount++;

            if (emptyCount === info.length) {
                api.column(this.index()).visible(false);
            } else {
                api.column(this.index()).visible(true);
            }
        });

    });
})(window, document, jQuery);

This gives me an idea for another feature, something that will hide the column if its empty for the current page, and re-checks on every page change, to show populated columns and hide empty ones.
But regarding your issue, you're saying it should be shown the entire time, even if you dont see values in that specific page, due to the fact they do exist in column C, just in a separate page, correct?
If thats the case, It works just like that for me. I added an example of what I mean here, you can see the column Age is shown, but it only has values in the 2nd page.
If thats not what you mean, can you provide an example?

That feature what you have described - it's what I want.
I have added my code to my example here
You can see it hides "Age" in first page, but unhides in second page.

Ok, ill get this feature added, probably sometime this weekend.
Great idea! And thanks for using the plugin. And again, sorry for the late reply

Ive also started using ES6 a lot more... I may re-do a lot of this in ES6, and provide a transpiled version, maybe

I wish I knew what ES6 is :)

Haha. Newer version of JS - Take a look

I added the new feature you want, its in the hec121 branch if you wanna try it out first

http://demo.jsdigest.com/DataTables-Hide-Empty-Columns/examples/example-perpage.html Only page 3 has a value :) Try searching, ordering, etc

Thanks, I will definitely give it a try :)

The JS you posted was close, but I realized it wouldnt work if there was a filter in place. So I did it a little differently, src

Good job :)

Here is a question :) I have set example
What I want to achieve:
Column "Name" should be ignored by plugin and I don't want to see it, so I hide it. Paging should not make it visible.
What I did:
If I understood correctly I have set following options:
            hideEmptyCols: {               columns: [0],               whiteList: false,               perPage: true             }
From documentation: whiteList "false" will target all columns except those listed
But the result is opposite - as soon as I change page - "Name" is shown again.
BUT if I set whiteList to false - it seems working as I would want?
Confused :)

Well I think it should be reversed logic:
Now:
                        // If the current column is *not* found in the list..                         if( $.inArray( this.index(), colList ) === - 1 // Check column index #                             && $.inArray( api.column( this.index() ).dataSrc(), colList ) === - 1 )// Check column name (dataSrc)                         {                             // .. And the list type is whitelist, then skip this loop                             if( isWhiteList === true ) return                         }                         // If the current column *is* found in the list..                         else {                             // .. And the list type is blacklist, then skip this loop                             if( isWhiteList === false ) return                         }
Read last "else"
If the current column is found in the list And the list type is blacklist, then skip this loop
But documentation says: whiteList "false" will target all columns except those listed
It does reverse - it founds column and "skips loop", while it "should


Cannot run a snapshot with --preview-dart-2

If I create a snapshot without --preview-dart-2 and try to run it with --preview-dart-2 I get some long stack traces starting with:
Crash when formatting: [Encoding] Unable to decode bytes as UTF-8.

If I create a snapshot with --preview-dart-2 and try to run it without any VM arguments I get a sensible error:
Snapshot not compatible with the current VM configuration: the snapshot requires 'release strong' but the VM has 'release no-strong'

Do we need this fixed to unblock #32188 ?

@a-siva @mraleph

Do we need this fixed to unblock #32188 ?

We can make some progress by running from source always for Dart 2 mode but we'll be paying a performance cost.

@natebosch, in Dart 2 appjit snapshots is what works, no script snapshots. Did you try an appjit?

No we have not tried appjit snapshots. This will require some work in pub, I'm not sure how significant.
Is there reason we aren't supporting non-jit snapshots?

In dart 2, the kernel file is the equivalent of script snapshot. You can generate kernel files for the vm using the gen_kernel tool and then pass this to the VM instead of source.

@a-siva  @alexmarkov,  we are not shipping the gen_kernel tool in the sdk build. Could you add this?

cc @mit-mit we should think about whether gen_kernel is the right name for this tool.

So, um, why do we need a different tool? Why doesn't dart --preview-dart-2 --snapshot=<snapshot-file> <dart-file> produce a snapshot in the right format?

Script snapshotting through kernel _used_to work...
Now though I get this:
$ pkg/vm/tool/dart2 --snapshot=foo.snapshot hello.dart

$ ls -lha foo.snapshot
-rw-r----- 1 jensj primarygroup 847 Apr 20 12:22 foo.snapshot

$ out/ReleaseX64/dart foo.snapshot
Snapshot not compatible with the current VM configuration: the snapshot requires 'release strong' but the VM has 'release no-strong'

$ out/ReleaseX64/dart --strong foo.snapshot
Snapshot not compatible with the current VM configuration: the snapshot requires 'release no-strong' but the VM has 'release strong'

Big +1 for making it work again, and not introducing yet another tool.

Interestingly this still works:
$ pkg/front_end/tool/fasta compile hello.dart

$ out/ReleaseX64/dart --snapshot=foo.snapshot hello.dart.dill

$ ls -lha foo.snapshot
-rw-r----- 1 jensj primarygroup 452K Apr 20 15:39 foo.snapshot

$ out/ReleaseX64/dart hello.dart.dill
Hello, world!

@a-siva - assigning to you.  We need this to create and run Dart 2 snapshots for pub, test, etc.

@natebosch do we need this to work only when building SDK or do we need this to work from SDK itself too?

We need this to work externally to the SDK.
pub builds snapshots of binaries to run them, and if we're going to have to pay a ~10 second penalty for running from source we'll really want to keep doing that so we only pay the parse cost once.
Similarly in build_runner we currently rely on starting an Isolate on a dart source file we wrote. If there is a startup performance penalty we need to start using a snapshot there too.

Disclaimer: I'm not suggesting we do this exactly as such, but the following hack (or proof of concept or whatnot) at least works on a hello world script:

Tried on top of 1f7fa3e1ada893fde892e0127917d302eee3ef5a
Revert da14cbd1d7783c3b672f2e60dbe9188e3fec88d5.
Apply the following patch:

diff --git a/runtime/bin/dfe.cc b/runtime/bin/dfe.cc
index 6bccb8de40..c6387b8e1b 100644
--- a/runtime/bin/dfe.cc
+++ b/runtime/bin/dfe.cc
@@ -230,6 +230,30 @@ void* DFE::CompileAndReadScript(const char* script_uri,
       strong ? platform_strong_dill : platform_dill;
   intptr_t platform_binary_size =
       strong ? platform_strong_dill_size : platform_dill_size;
+
+  {
+    const uint8_t* buffer = NULL;
+    intptr_t buffer_length = -1;
+    const uint8_t** kernel_ir = &buffer;
+    intptr_t* kernel_ir_size = &buffer_length;
+
+    *kernel_ir = NULL;
+    *kernel_ir_size = -1;
+    void* script_file = DartUtils::OpenFileUri(script_uri, false);
+    if (script_file != NULL) {
+      const uint8_t* buffer = NULL;
+      DartUtils::ReadFile(&buffer, kernel_ir_size, script_file);
+      DartUtils::CloseFile(script_file);
+      if (*kernel_ir_size != 0 && buffer != NULL) {
+        if (DartUtils::SniffForMagicNumber(buffer, *kernel_ir_size) ==
+            DartUtils::kSnapshotMagicNumber) {
+          Dart_EnterScope();
+          return Dart_LoadScriptFromSnapshot(buffer, *kernel_ir_size);
+        }
+      }
+    }
+  }
+
   Dart_KernelCompilationResult result = Dart_CompileToKernel(
       sanitized_uri, platform_binary, platform_binary_size);
   switch (result.status) {
diff --git a/runtime/vm/dart_api_impl.cc b/runtime/vm/dart_api_impl.cc
index eff8af86ba..b80c021237 100644
--- a/runtime/vm/dart_api_impl.cc
+++ b/runtime/vm/dart_api_impl.cc
@@ -5129,9 +5129,7 @@ DART_EXPORT Dart_Handle Dart_LoadScript(Dart_Handle url,
   }
   Library& library = Library::Handle(Z, I->object_store()->root_library());
   if (!library.IsNull()) {
-    const String& library_url = String::Handle(Z, library.url());
-    return Api::NewError("%s: A script has already been loaded from '%s'.",
-                         CURRENT_FUNC, library_url.ToCString());
+    return Api::NewHandle(T, library.raw());
   }
   if (line_offset < 0) {
     return Api::NewError("%s: argument 'line_offset' must be positive number",

Try this:

$ time pkg/vm/tool/dart2 hello.dart
Hello, world!

real    0m0.401s
user    0m0.552s
sys     0m0.112s

$ rm foo.snapshot

$ time pkg/vm/tool/dart2 --snapshot=foo.snapshot hello.dart

real    0m0.512s
user    0m0.744s
sys     0m0.140s

$ ls -la foo.snapshot
-rw-r----- 1 jensj primarygroup 847 Apr 23 10:43 foo.snapshot

$ time pkg/vm/tool/dart2 foo.snapshot
Hello, world!

real    0m0.193s
user    0m0.276s
sys     0m0.080s

Again: I'm not saying we should do it like this, but maybe this can give someone else a starting point.

CL https://dart-review.googlesource.com/c/sdk/+/52740 out for review
sivalinuxmach[sdk]>out/ReleaseX64/dart --preview-dart-2 --snapshot=/tmp/hello.snapshot /tmp/hello.dart
sivalinuxmach[sdk]>out/ReleaseX64/dart /tmp/hello.snapshot
Hello World
sivalinuxmach[sdk]>

CL https://dart-review.googlesource.com/c/sdk/+/52740 has landed. Can this be closed?

Yes the CL has landed and stuck, closing issue.

https://github.com/dart-lang/sdk/commit/4b1180a39d5e4d0d46f2395508dae1b304daf104 – landed in -dev.53


aya: Truncate long program names

Signed-off-by: Dave Tucker<EMAIL_ADDRESS>

@alessandrod this is a quick fix :laughing: I know you had bigger plans for better name truncation.

What happens if two programs have the same 16 byte prefix? Is that a hard failure? If so I'm tempted to switch off this whole prog_name business until we have a plan 😅

I don't think names need to be unique. Let me check :)
On Sun, 16 Jan 2022, at 11:18, Alessandro Decina wrote:

What happens if two programs have the same 16 byte prefix? Is that a hard failure? If so I'm tempted to switch off this whole prog_name business until we have a plan 😅
—
Reply to this email directly, view it on GitHub https://github.com/aya-rs/aya/pull/171#issuecomment-1013856777, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA43JEKMJHYNHDDXGVLKVT3UWKSPXANCNFSM5MCUI6HQ.
You are receiving this because you authored the thread.Message ID: @.***>

194: xdp  name foo2  tag 3b185187f1855c4c  gpl
        loaded_at 2022-01-16T11:32:17+0000  uid 0
        xlated 16B  jited 18B  memlock 4096B
        btf_id 416
        pids foo2(272669)
200: xdp  name foo2  tag 3b185187f1855c4c  gpl
        loaded_at 2022-01-16T11:32:34+0000  uid 0
        xlated 16B  jited 18B  memlock 4096B
        btf_id 434
        pids foo2(272974)


test_development_guidelines_matches_ci fails from sdist

What happened:
The test fails when building from an sdist. This is because the GitHub workflows are not included in the sdist.
____________________ test_development_guidelines_matches_ci ____________________
[gw0] linux -- Python 3.10.1 /usr/bin/python3

    def test_development_guidelines_matches_ci():
        """When the environment.yaml changes in CI, make sure to change it in the docs as well"""
        root_dir = Path(__file__).parent.parent.parent

        development_doc_file = root_dir / "docs" / "source" / "develop.rst"
        additional_ci_file = root_dir / ".github" / "workflows" / "additional.yml"
        upstream_ci_file = root_dir / ".github" / "workflows" / "upstream.yml"
        latest_env = "environment-3.9.yaml"

        for filename in [development_doc_file, additional_ci_file, upstream_ci_file]:
>           with open(filename, encoding="utf8") as f:
E           FileNotFoundError: [Errno 2] No such file or directory: '/builddir/build/BUILD/dask-2021.12.0/.github/workflows/additional.yml'

dask/tests/test_docs.py:14: FileNotFoundError

What you expected to happen: Tests pass or are skipped if not relevant.
Environment:

Dask version: 2021.12.0
Python version: 3.10
Operating System: Fedora Rawhide
Install method (conda, pip, source): source

Should we include GitHub workflows in sdist?

I just want to point out that Dask is moving towards not packaging tests at all - see https://github.com/dask/dask/pull/8478 for that discussion.

Should we include GitHub workflows in sdist?

If you want, or maybe those tests should be skipped if not running on GitHub CI.

I just want to point out that Dask is moving towards not packaging tests at all - see #8478 for that discussion.

Seems like that's not happening.

Should we include GitHub workflows in sdist?

If you want, or maybe those tests should be skipped if not running on GitHub CI.

This seems like a reasonable solution as well if you are interested in opening a PR.

I just want to point out that Dask is moving towards not packaging tests at all - see #8478 for that discussion.

Seems like that's not happening.

Still planning to do it, but with a different approach and waiting for tests to be more stable on distributed before we do it https://github.com/dask/dask/pull/8662


Merge upstream

Placeholder PR for final merge upstream before release

Codecov Report

Merging #414 into master will increase coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #414      +/-   ##
==========================================
+ Coverage   56.34%   56.37%   +0.02%
==========================================
  Files         891      895       +4
  Lines       29143    29198      +55
  Branches     4217     4219       +2
==========================================
+ Hits        16421    16460      +39
- Misses      12722    12738      +16


Fix a typo in service container registration sample

You did ... I must have loaded the page before you clicked it.


Missing annotation in Chado v59 and v60

From pombase/curation#946:
These gene have no annotation in Chado, not even a product:
SPBC713.13
SPAC823.02
SPAPB1A11.05
There is annotation for these in v58.
Fix this and add a post load check that all genes have a product.

Update https://www.ebi.ac.uk/panda/jira/browse/PB-2633 when fixed.

These gene have no annotation in Chado, not even a product:

That's true, but due to poor querying that's not the full answer.  My dodgy query returned no annotation because those genes aren't in Chado.  In the contig files and in Chado all three IDs are synonyms of other genes.
There is one fix that is needed in the contig files.  The systematic_id here should be changed to "SPAC823.17":
FT   intron          2583903..2583940
FT                   /controlled_curation="term=misc, confirmed intron"
FT                   /systematic_id="SPAC823.02"
FT                   /gene="SPAC823.17"

Please don['t fix it just yet because when the current load finishes I'll look into why that didn't generate a warning.

I won't but we need to  careful here. The gene on the opposite strand is SPAC823.17
http://www.pombase.org/spombe/result/SPAC823.17
I think it might be correct in Art. This is the intron feature, not the CDS (no product on inton) ? Will take a look later...

Please don't fix it just yet because when the current load finishes I'll look into why that didn't
generate a warning.

I've added a warning to the logs.  I'll run a load soon now with the change.

This is how 'Art' looks
CDS
/systematic_id="SPBC713.14c"
/product="Schizosaccharomyces pombe specific protein"
/synonym="SPBC713.13"
pombase
http://www.pombase.org/spombe/result/SPBC713.14c
but there is also a page for the synonym?
http://www.pombase.org/spombe/result/SPBC713.13
I think the other 2 might be similar.....

This isn't a Chado problem so I'll close it.


Respect exact file encoding

Today I had a problem where I edited a config file of an application but after saving, that application failed to load the config file. After some investigation it turns out the problem was with encoding.
Originally the config file was in UTF-8 encoding but after editing it with Notepads it saved it in UTF-8-BOM.
It would be nice if Notepads would not change the encoding of an existing file.

This is something that I need to fix right away. I almost forget it, thanks for the reminder!

Fixed here: https://github.com/JasonStein/Notepads/commit/6ba9267 I have also changed naming for "UTF 16 LE and BE" to "UTF-16 LE BOM" and "UTF-16 BE BOM" since they are actually unicode with BOM implemented in Notepads. Cheers. Please test it in v<IP_ADDRESS> and leave me your comments.

Works great!


Find more suitable place for https://haveno-dex.github.io/haveno-ts/

Now it's at api.haveno.exchange. See #141


Please publish to the OpenVSX Registry

There are a growing set VSCode variations and other IDEs that use VSX extensions. Microsoft bars them from using Microsoft VSCode VSX Registry. The Eclipse Foundation hosts the OpenVSX Registry which enables people not using official VSCode or using other IDEs to use the extension. Please publish your excellent extension to the OpenVSX registry.

How to publish to the OpenVSX Registry
The OpenVSX extension repository

Thank you!

I'll look into it, thank you for the suggestion


fix: pass app object to open instead of string/array

Hello 👋
Thank you for this package ❤
It seems https://github.com/michaellzc/better-opn/commit/5a185c2d90c7a71548f8e7febb42d4e7d64ea849 broke the open invocation as it didn't update the structure of the arguments per https://github.com/sindresorhus/open/releases/tag/v8.0.0.
You can try this by writing a script that uses this library and set BROWSER=firefox to see.
This PR fixes the above issue.

hey @erezrokah
thanks a lot for the fix.
A new patch has been released https://www.npmjs.com/package/better-opn/v/3.0.2

Thank you for the quick review and release!


cppwinrt example

https://docs.microsoft.com/en-us/windows/uwp/xaml-platform/custom-attached-properties#code-example
bool IsMovable()     ----->   the accessors must be named GetIsMovable and SetIsMovable ?
Windows::UI::Xaml::DependencyProperty IsMovableProperty(){ return m_IsMovableProperty; }      not static ?
winrt::xaml_typename<bool>()  ------>  T must be WinRT type.

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

ID: 369c139c-69a6-8817-5cb2-6e93d64d8cce
Version Independent ID: 6feed647-1cd1-8971-eb36-f14aaccad151
Content: Custom attached properties - Windows UWP applications
Content Source: windows-apps-src/xaml-platform/custom-attached-properties.md
Product: uwp
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Hi, @Dylan-H, yes, the content and code is correct. For example, about the naming convention for accessors, and the actual DP getter is not static, by design. bool is a Windows Runtime type, in that it can be passed across the application binary interface (ABI). Please let me know if this doesn't address your questions. I'll leave the issue open for a while. ~Steve

@stevewhims
Sorry, my English is not very good. I'm using translation software.
Without Set (Get), I can't access DP as shown in the example above. In my code, I have to write SetIsMovable (GettIsMovable) to work properly.
winrt::xaml_typename() ------> “T must be WinRT type” is a compile-time error


Working on Building Code Components for Microsoft Power Apps with shadcn-ui Need little help with tailwind.

Hi Shadcn & team,
I really appreciate what you are doing for the community.
To introduce myself, I am a low code developer who mainly works on Microsoft Low code technologies such as Power Apps. I have been a big fan of your components ever since I saw it on twitter and I always wanted to bring them into Power apps as custom code components. Finally Yesterday I managed to Integrate very delicately and with care I was able to see the component actually rendering in the local component harness. However, for some reason only half of the tailwind classes are being applied, like initially non of the tailwind class were applied then I corrected content files location inside tailwind.config.js and finally the styles were applied but something is not correct as only half styling was done and it doesn't look good, example destructive didn't show any effect at all. I would really appreciate that not only you but any other kind contributor could come forward to look into it and help me with this request, it would mean a lot and it would allow me to achieve my goal. as of now it's still sitting on my local and as soon as someone reaches out I will push the project into github repo and share the link.
Looking forward to collaborate on this.
Thanks,
Chandra :)

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


[SPARK-49704][BUILD] Upgrade commons-io to 2.17.0

What changes were proposed in this pull request?
This PR aims to upgrade commons-io from 2.16.1 to 2.17.0.
Why are the changes needed?
The full release notes: https://commons.apache.org/proper/commons-io/changes-report.html#a2.17.0
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

Or, we can remove the benchmark code from this PR.

Or, we can remove the benchmark code from this PR.

I'm very sorry, I mistakenly submitted code /ArrayBinarySearchBenchmark, which is the logic of another PR.
I have deleted it. Thank you very much for your reminder! ❤


Frame.Navigate throws when clicking unselected item in GridView

Describe the bug
If a frame navigation is triggered by clicking a button in unselected item in GridView, it will throw with 'Child collection must not be modified during measure or arrange'.
This is a regression from 1.0.0-preview3
Steps to reproduce the bug

Create a page, with GridView inside it
In GridView items, put a button that triggers navigation to another page
When clicking the button without selecting the item first, Frame.Navigate throws
If select the item first then click the button inside it, it will not throw

Expected behavior
Navigate successfully without exception.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.0.0
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
May 2021 Update (19043)
Additional context
Minimal repro:
GridViewNavigationRepro.zip
Changing WindowsAppSDK to 1.0.0-preview3 and the problem disappears.

I experience the same issue. Did you find any workaround for this (apart from changing to 1.0.0-preview3)?

Thanks for the repro. We've done some debugging and the issue is that the button on the page that's being navigating away from has captured the pointer (normal for pointer press on a button), then that capture is being canceled as part of navigating away, which is triggering an attempt to modify a tree that's in the process of being torn down.
As a workaround for now you can release the button's capture:
private void Button_Click(object sender, RoutedEventArgs e)
{
    (sender as Button).ReleasePointerCaptures();
    Messenger.Navigate();
}

Thank you for the workaround. Hopefully you can fix it soon.

@JJBrychell FYI

The issue is not fixed in 1.0.1 or 1.1.0-preview.

I have also the same problem in version 1.1.3.

@ranjeshj can the controls team look at a global fix for this, instead of the workaround?

This can be repro'd on WinUI 2.

Do you have a date for a fix?
It is not possible to require users to first select a grid item before clicking on it.
Also, as I use a template for my grid, I haven't found a way to apply the workaround with the button clickl.

The bug is still present in WinAppSDK 1.3 and latest preview 1.4.230628000-preview1. Please reopen this.

@evelynwu-msft could we add a maui label to this?

@evelynwu-msft could we add a maui label to this?

I am facing this issue with WinUI3 App also.

I can confirm I'm receiving the same error in a UWP application, WinUI v2.8.5.
Having a command bound to my Button while the click event was doing this seemed to work for now (didn't need Messenger.Navigate() there though in my case, the command handles navigation):

private void Button_Click(object sender, RoutedEventArgs e)
{
    (sender as Button).ReleasePointerCaptures();
    Messenger.Navigate();
}

@llongley can you provide which version will include this fix? Thanks!

@huoyaoyuan The fix will be in WinAppSDK 1.5. It did not make it into the 1.5-experimental2 release today.

SelectionMode ="None"
This works Fine!
 Logically, this ain't a bug to fix.

I can still see the bug on a XamlIslands project with C++.
Microsoft.UI.Xaml 2.8.2.prerelease-220830001
The workaround from above works:
    if (!(sender.try_as<winrt::Windows::UI::Xaml::Controls::Button>(xBtnEdit))) {
    }
    else {
        //  \see    https://github.com/microsoft/microsoft-ui-xaml/issues/6350
        xBtnEdit.ReleasePointerCaptures();


fix(datalist): update compact font-size

part of https://github.com/patternfly/patternfly-next/issues/2653.
This just modifies the existing compact spacing and leaves the modifier and default behavior in place.
We're also not going to update the button alignment to center if the space below exceeds 8px.
Breaking changes

Changes compact data-list font-size to --pf-global--FontSize--sm

Also agree that that example looks cramped. the 14pt text next to the 16pt text in the button is a little visually jarring to me, but if its approved by design then im fine with it


[PROFILER] Fix launch metadata bug

Oh, so constants will have more values than constexprs? Just to confirm.
I found:
    new_constants = {k: True if k in tys and tys[k] == "i1" else 1 for k in attrs.equal_to_1}

Those are stored in self.constants

Yes. After the recent refactor, it seems like values equal to 1 are included in the arguments:
https://github.com/openai/triton/blob/cfa495f5a3c5057c1f5f2f14000c6869070c910b/python/triton/runtime/jit.py#L547


Transparency doesn't work in resizable windows

From docs:

Transparent window is not resizable, setting resizable to true may make transprent window stop working on some platforms.

This is a huge limitation. Is this set in stone (if so, what is the reason), or is there a possible fix incoming in a foreseeable future?

It is quite hard to fix because the way transparent window works is quite different to normal windows, but I think I will have a try in future.

I now don't think anyone would look into this, so I'm closing this now. We will still accept it if someone sends a pull request.


[BUG] High ping after installation

Acknowledgement of preliminary instructions

[ ] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
when i use bloxtrap my ping becomes very high (200+), without bloxtrap the ping is back to 80-90.
How do i fix this. ( i already tried putting 0 on the fps unlocker and nothing changed)

fps is not related to ping. Try 60 fps and see if it works.

Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

actually it got fixed automatically idk what happened but thank you for the reply and sorry for wasting your time


Select side dishes for recipies

As a user I want to be able to select one or multiple side dishes for a recipe in the recipe editor.
The side dish should have a read-only view about which main dishes it is assigned to.

Went for one side dish in the end to reduce the complexity.


Adds ecmascr. supp. for TextDecoder, TextEncoder

Support is available for both objects in Nodejs and Browsers.
Browsers make TextEncoder and TextDecoder available in the global
namespace.
In older versions of Nodejs (<11) TextEncoder and TextDecoder is only
available behind util module. But recent versions of Nodejs moved the
declaration to the global namespace too.
Spec, browser and Nodejs docs linked below.
[1] https://encoding.spec.whatwg.org/#textdecoder
[2] https://encoding.spec.whatwg.org/#textencoder
[3] https://developer.mozilla.org/en-US/docs/Web/API/TextDecoder
[4] https://developer.mozilla.org/en-US/docs/Web/API/TextEncoder
[5] https://nodejs.org/api/globals.html#globals_textdecoder
[6] https://nodejs.org/api/globals.html#globals_textencoder

It seems that the libraries that are npm installed in travis are not compatible with the node version travis is running. Is it a long standing issue?

Thank you for the merge othree :) !


/shell parameters not working

xfreerdp /shell:"C:\Windows\notepad.exe" does not work
I tried using the master branch, but the problem still persists. I suspect the problem is with Windows, but I don't know how to configure Windows to make this command work
This problem exists in Windows 10, 11, and Server 2016. Who can help me? I am not familiar with Windows at all

@zhuyaliang this option is equivalent to the alternate shell option of a rdp file.
how to enable that on server side I also don´t know though.


Fixed the distributed unit test failures in resetting exactness for InsertSelect.

Fixed the distributed unit test failures introduced in #155.
Assigned to @jianqiao.

LGTM. Thanks for the fix! Will merge after travis returns green.


Custom keras optimizer - learning rate changes each epoch

I'm writing a custom optimizer where the learning rate changes on runtime for each epoch,  is there a way to pass changing lr to the Optimizer class on runtime?
Is there a way I can do it?
def get_updates(self, loss, params):
        grads = self.get_gradients(loss, params)
        self.updates = [K.update_add(self.iterations, 1)]

        lr = self.lr
        # momentum
        shapes = [K.int_shape(p) for p in params]
        moments = [K.zeros(shape) for shape in shapes]
        self.weights = [self.iterations] + moments
        for p, g, m in zip(params, grads, moments):
            v = self.momentum * m - lr * g  # velocity
            print(lr, g)
            self.updates.append(K.update(m, v))

        return self.updates

A custom l_r for each epoch?

yes, you can use a learningratescheduler. https://github.com/keras-team/keras/issues/888 This issue has details about implementation. You can then use the scheduler to change the learning rate for each given epoch

Got it thanks! :smile:


[GHSA-cpx3-93w7-457x] A flaw was found in Ansible in the amazon.aws collection...

Updates

Affected products
Summary

Comments
ec2_instance was introduced in https://github.com/ansible/ansible/pull/35749, and had tower_callback.windows from the initial release. The issue was fixed in 3.5.1/4.4.0/5.1.0, and according to https://github.com/ansible-community/ansible-build-data/blob/main/6/CHANGELOG-v6.rst, ansible 6 never did update the version to 3.5.1.

Hi @xfix, thank you for your interest in GHSA-cpx3-93w7-457x. We can't review it under the ansible PyPI package because that package corresponds with https://github.com/ansible/ansible. https://github.com/ansible-collections/amazon.aws doesn't have a setup.cfg file that lists the package in PyPI, nor can I find a package corresponding with the repo when I search PyPI.

@xfix Thank you for the explanation. I looked for a relationship between https://github.com/ansible/ansible and https://github.com/ansible-community/antsibull and found this in the ansible repo and this in the antsibull repo. Are these the kind of connections you were thinking of?

PyPI ansible package combines a lot of things into one package using antsibull build script. The URL in the package (https://github.com/ansible/ansible) is defined in https://github.com/ansible-community/antsibull/blob/5d39db6407f4318cacde41c7f3d879be9fe9f541/src/antsibull/data/ansible-setup_py.j2#L173.
antsibull loads the list of ansible-collections repositories to include from ansible-build-data repository (https://github.com/ansible-community/antsibull/blob/5d39db6407f4318cacde41c7f3d879be9fe9f541/roles/build-release/defaults/main.yaml#L19).
Meanwhile https://github.com/ansible-community/ansible-build-data mentions amazon.aws repository (https://github.com/ansible-community/ansible-build-data/blob/31dfecc395cb1873de5389be2bfe6b3b78ef167c/8/collection-meta.yaml#L140-L141).

@xfix Thank you for the links! Now I understand. If you want credit on the advisory, you're welcome to make a new pull request.

Created https://github.com/github/advisory-database/pull/1634.


Allow clients to unregister their sessions

Currently, when a client/session is closed, its session is not removed from state machines until it expires. Clients should be able to explicitly unregister their session when closed. The protocol already supports an UnregisterRequest which commits an UnregisterEntry which removes the session quietly. Client sessions simply need to be updated to submit an UnregisterRequest when explicitly closed.

#100


Wrong information for Transit Gateway VPC Attachment after refresh

I have a Transit Gateway VPC Attachment defined as follow:
new aws.ec2transitgateway.VpcAttachment('general', {
  transitGatewayId: transitGateway.id,
  vpcId: vpc.id,
  subnetIds: [subnets.private.a.id, subnets.private.b.id, subnets.private.c.id],
  transitGatewayDefaultRouteTableAssociation: false,
  transitGatewayDefaultRouteTablePropagation: false,
  tags: {
    Name: environment,
  },
});

It creates the attachment correctly, but once I run a refresh, the transitGatewayDefaultRouteTableAssociation and transitGatewayDefaultRouteTablePropagation are set to true in the state, while still set to false in AWS:

It happens doing the refresh after a new update as well.

Hi @fernandocarletti
I have been able to recreate this both in Pulumi and the upstream Terraform provider. I am working on a fix right now
Paul

This has been fixed now and we no longer get this diff - sorry for not closing this out sooner @fernandocarletti


can not disable the view

the method "setEnabled" does not work, is there any way to disable it?

+1 I don't know how to disable it, would be useful to know.


fix(authentication): delete cookie (in chrome) on logout (DEV-325)

resolves DEV-325

works perfect on test-server 👍


Move captions

Move captions to a side and implement de ability of seeing them or not , would be comfortable. Just a suggestion . Thanks !

Thanks for the suggestion. This is a low value, high effort feature. So will consider it if more people ask for it.

great , thanks
*Rodrigo *
El lun, 6 may 2024 a las 12:22, Vivek G @.***>)
escribió:

Thanks for the suggestion. This is a low value, high effort feature. So
will consider it if more people ask for it.
—
Reply to this email directly, view it on GitHub
https://github.com/vivek-nexus/transcriptonic/issues/10#issuecomment-2096290563,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHDKH4TKGOCXIV6IDXNXL7TZA6N3TAVCNFSM6AAAAABHJAAREGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGI4TANJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Dimmed down the transcript opacity to 20%, so that it is less intrusive. Refer PR #11.


feat(security): implement spiffe token provider services

add spire server Dockerfile
add spire agent Dockerfile
add spire config Dockerfile
add spiffe token provider and gettoken restful endpoint

Closes: #3773
Signed-off-by: Jim Wang<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[ ] I am not introducing a new dependency (add notes below if you are)    spiffe/v2 and its related dependencies
[ ] I have added unit tests for the new feature or bug fix (if not, why?)   x509Source from spiffe APIs not testable directly as they don't have interface to mock on
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

With updated docker-compose files from this branch (https://github.com/jim-wang-intel/edgex-compose/tree/add-spiffe-samples) and do make run dev delayed-start from the command line , then follow the testcase.txt under security-spiffe-token-provider/ service: https://github.com/jim-wang-intel/edgex-go/blob/add-spiffe-token-provider/cmd/security-spiffe-token-provider/testcase.txt
to verify you can get a vault token.
New Dependency Instructions (If applicable)

spiffe dependencies:

GO111MODULE=on go mod graph | grep spiffe
github.com/edgexfoundry/edgex-go<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

squashed version of other PR #3892

Codecov Report

Merging #3897 (078dc65) into main (613ba3a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3897   +/-   ##
=======================================
  Coverage   45.78%   45.78%
=======================================
  Files         112      112
  Lines        9641     9641
=======================================
  Hits         4414     4414
  Misses       4844     4844
  Partials      383      383

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 613ba3a...078dc65. Read the comment docs.


Update iptables perf / KEP-3453 discussion for 1.27

This is an update for a feature going to beta, so I assume it belongs on dev-1.27, but that branch doesn't seem to be up-to-date? In particular, it doesn't have #38435, which this rewrites.
(I moved the feature-specific section because it seemed to make more sense to talk about that first now, since it's enabled by default.)
/cc @sftim @aojea

/hold
This PR needs to target dev-1.27 (it's OK to wait until dev-1.27 incorporates everything that's in main today).
OK to unhold once we fix the base branch.

For beta, please also revise the text that talks about Endpoints - refer to EndpointSlices if you can.

Changed one "Endpoints" to "EndpointSlices" and another to "endpoints"; we still use lowercase-e "endpoints" in many places to refer to endpoints as a general idea

/hold cancel
this was marked /hold because it was originally targeting main, but that's fixed now

/lgtm

/approve

/lgtm
again


fix: last message may be hidden by input + auto-scroll

https://dust4ai.slack.com/archives/C050SM8NSPK/p1690468010312419

Also fixes auto-scrolling. We might want to make it a bit agressive (each message) not each character. Not sure. But it's better with this than without

(The big diff is just a div removal)

(post-merge review welcome)


Webpage's responsive route does not work

Version
https://reacttraining.com/react-router/core/guides/philosophy
Test Case
When browsing the web page using a mobile device or test in 'toggle device model'  of Chrome, the back arrow does not work, and it will make the page hanging which is a quite bad example of responsive routing.

Steps to reproduce
1 Open https://reacttraining.com/react-router/core/guides/philosophy from a mobile device or use Chrome to stimulate a mobile device,  you could see the resolution I use in Chrome from snapshot I upload in Test Case.
2 Try to click the button I tag with a red rectangle.
Expected Behavior
Go back to Menu
Actual Behavior
No action.

Thanks for the report, @LeonAppDev. The issue is already reported in #6058, so I'm going to close this one.

Ok, thanks for quick response


Accounts with 0 ether generated with ganache

I asked about this on gitter: [ ]

Expected Behavior
Accounts generated by ganache-cli should have 100 ether in them.

Current Behavior
the account (coinbase) has 0 ether after running this command
ganache-cli --port 22001 --mnemonic --networkId 100  --db /root/ganachedb

Your Environment

Version used: Ganache CLI v6.1.0
Version of Truffle/Remix/Other tools used: Truffle v4.1.8
NodeJS Version: [ ] 6.x, [ ] 7.x (unsupported), [x] 8.x, [ ] 9.x
Operating System and version (include distro if Linux): OSX high Sierra

Thanks for reporting this! I haven't had the chance to reproduce this yet, but I'm guessing the lack of providing a mnemonic after specifying the flag is causing the issue

I'm not seeing this behavior in v6.1.6; I'm guessing this got fixed in v6.1.5. Below is the output I'm receiving:
ganache-cli --port 22001 --mnemonic --networkId 100 ~/ganachedb
Ganache CLI v6.1.6 (ganache-core: 2.1.5)

Available Accounts
==================
(0) 0x057e9718f7c14aa3dd054b92ad8a6b58e44abe97 (~100 ETH)
(1) 0xceb5802fcc9512f77a04dc2dbca035f98200f0a2 (~100 ETH)
(2) 0xfdd1baf494d956618cb5d3a315cf528a5af3cfc2 (~100 ETH)
(3) 0xeea8f8041197cbfc4d4f49ba34b471ace2770153 (~100 ETH)
(4) 0x11477acfaaa17ff038b88305d9ae2c445c8f0883 (~100 ETH)
(5) 0x667aa93d83a1440e26a73bcf4f27b8ef57e60b14 (~100 ETH)
(6) 0x425a77c9d792d1a97c9a66abac5313e89408258c (~100 ETH)
(7) 0x4d7facefefe5d23ac520d826c8961fc0bd9d6ad3 (~100 ETH)
(8) 0x90bdd08e695c61b223c40be2003ac730a255154e (~100 ETH)
(9) 0x73fe2d80731cab07e2fd229849c5f3bb6c90b84e (~100 ETH)