-
Notifications
You must be signed in to change notification settings - Fork 714
[Bug Fix Release] raise warning if jax>0.6.2 is installed
#7949
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
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v0.42.1-rc0 #7949 +/- ##
============================================
Coverage 99.68% 99.68%
============================================
Files 543 543
Lines 55314 55321 +7
============================================
+ Hits 55140 55147 +7
Misses 174 174 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
jax > 0.6.2 is installedCo-authored-by: Mudit Pandey <18223836+mudit2812@users.noreply.github.com>
Co-authored-by: Mudit Pandey <18223836+mudit2812@users.noreply.github.com>
Co-authored-by: Mudit Pandey <18223836+mudit2812@users.noreply.github.com>
PietropaoloFrisoni
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can confirm that importing PL with latest JAX results in a warning message:
>>> import pennylane as qml
RuntimeWarning: PennyLane is not yet compatible with JAX versions > 0.6.2. You have version 0.7.0 installed. Please downgrade JAX to <=0.6.2 to avoid runtime errors.
warnings.warn(
💯
Not sure about the sphinx failures (I had to struggle with it even last time, and I had to downgrade)
Co-authored-by: Mudit Pandey <18223836+mudit2812@users.noreply.github.com>
This reverts commit bbc1bdd.
I got around them by aliasing the imports as private (eb4a11d) |
Co-authored-by: Lee James O'Riordan <mlxd@users.noreply.github.com>
|
Tagging @jay-selby @Alex-Preciado as they were present on the last BF release PR. 👋🏼 |
DSGuala
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating the error/warning messages 👍
Looks good to me.
PietropaoloFrisoni
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇
Context:
Jax released a new version with a lot of breaking changes. Unfortunately our
v0.42release is not compatible with this version and so there are runtime errors.ℹ️ : This PR was created by branching the v0.42.0 tag.
Description of the Change:
jaxand if they do it will perform a version check to ensure compatibility.pennylane-lightningandpennylane-catalystinstead of the latest, because the latest version of these packages is now incompatible with this branch (they are using thepennylane.exceptionsmodule for some custom exceptions). This change will not be merged into master.Benefits: Better UI.
Possible Drawbacks: None.
[sc-96104]