Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set jax version to 0.4.28 in test workflow #98

Merged
merged 4 commits into from
Dec 31, 2024
Merged

Conversation

lillian542
Copy link
Contributor

@lillian542 lillian542 commented Dec 30, 2024

The Braket tests are failing because of an incompatibility between PennyLane and JAX. Add an environment variable for the JAX version in the Braket tests, and set it to the version of JAX that catalyst is currently pinned to.

This is intended as a temporary solution to get the tests working while we work out how to make our JAX dependencies clear and consistent across our various repos/packages. We don't want to have to keep track of updating this long term.

[sc-78774]

Copy link
Contributor

@PietropaoloFrisoni PietropaoloFrisoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! It would probably make sense to have JAX as a hard PL dependency (especially considering the program capture project)

@lillian542 lillian542 merged commit 4810fe5 into master Dec 31, 2024
30 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants