-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fix the failed observable serialization unit tests #683
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #683 +/- ##
==========================================
- Coverage 99.37% 97.99% -1.38%
==========================================
Files 51 207 +156
Lines 7639 29909 +22270
==========================================
+ Hits 7591 29309 +21718
- Misses 48 600 +552 ☔ View full report in Codecov by Sentry. |
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.
If that works, no blocker from me. Thanks @maliasadi !
Thanks @maliasadi ! Do we also want to turn on mgpu tests? |
Device tests are still failing with LGPU and LK. Let's figure out if the issue is on the lightning side or PL side before merging this. |
[sc-61588] |
PR PennyLaneAI/pennylane#5475 led to LQ and LK serialization unit tests failures. This PR updates them to pass on CIs.