-
Notifications
You must be signed in to change notification settings - Fork 643
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
[Strings2] Improve string repr of Sum #5138
Conversation
…o paulistrings
Hello. You may have forgotten to update the changelog!
|
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!
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.
🎉 🚀 Thank you so much!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5138 +/- ##
==========================================
- Coverage 99.68% 99.67% -0.01%
==========================================
Files 394 394
Lines 36298 36034 -264
==========================================
- Hits 36183 35918 -265
- Misses 115 116 +1 ☔ View full report in Codecov by Sentry. |
…tations of X, Y, Z and Identity (#5153) Updating docs and codebase with `qml.X/Y/Z/I(*)` instead of `qml.PauliX/Y/Z(*)` and `qml.Identity(*)` following #5116 and updating composite operation outputs in docs according to #5138 ToDo - [x] Improve https://docs.pennylane.ai/en/stable/introduction/operations.html#qubit-operators - [x] Update outputs in docs with PauliX(wires=*) - [x] Update outputs in docs with SProds and other composites (i.e. with extra spaces and removing unnecessary brackets) - [x] manually check & update all intro pages - [x] manually check & update pulse docs - [x] All tests pass - [x] changelog --------- Co-authored-by: Thomas R. Bromley <49409390+trbromley@users.noreply.github.com>
Sum
instances are now displayed multiline and without unnecessary brackets. Further, all outputs should be copy-and-paste-able. Some examples how this now looks:There are also some peculiarities that cannot be improved at the moment because they are due to the nesting and lazy execution of products and sums:
ToDo
Documentation will be updated together with the changes from #5116 in a follow-up PR
Builds on top of #5116
[sc-54959]
[sc-54955]