-
Notifications
You must be signed in to change notification settings - Fork 586
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
Pauli sentence bug #5407
Pauli sentence bug #5407
Conversation
Thanks @Tarun-Kumar07 for this! I've tagged some reviewers who will take a look soon. |
Just for our internal tracking: [sc-58518] |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5407 +/- ##
==========================================
+ Coverage 99.63% 99.65% +0.01%
==========================================
Files 399 401 +2
Lines 37125 36950 -175
==========================================
- Hits 36990 36821 -169
+ Misses 135 129 -6 ☔ 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.
This is great, thanks @Tarun-Kumar07
Nothing to complain, looks good right off the bat 👌
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.
Looks great to me. Thanks for this 🚀
Context:
Earlier
PauliSentence.to_mat(wire_order)
failed with exception when it hadonly empty PauliWords
.This has been fixed under this PR.
Description of the Change:
Earlier when
PauliWord
didn't have wires the function_pw_wires
defaulted toPauliSentence
wires. Now rather thanPauliSentence
itself, it defaults towire_order
Benefits:
Possible Drawbacks:
Related GitHub Issues:
Fixes #5354