-
Notifications
You must be signed in to change notification settings - Fork 39
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
Remove CPhase + Tidy up src #717
Conversation
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 Ali!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #717 +/- ##
==========================================
+ Coverage 93.91% 95.35% +1.44%
==========================================
Files 107 91 -16
Lines 16244 14073 -2171
==========================================
- Hits 15255 13419 -1836
+ Misses 989 654 -335 ☔ View full report in Codecov by Sentry. |
b237ec4
to
dbeaaa3
Compare
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 @maliasadi ! LGTM!
pennylane_lightning/core/src/simulators/lightning_gpu/measurements/MeasurementsGPU.hpp
Show resolved
Hide resolved
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.
A couple questions, but ready to approve ... so I'll approve. Thanks @maliasadi , nice one.
* Remove CPhase; Tidy up NDPermuter, gh scripts, and doc * Add release_notes to docs * Auto update version from '0.37.0-dev0' to '0.37.0-dev2' * Update changelog * Auto update version from '0.37.0-dev2' to '0.37.0-dev3' * trigger ci * Revert change in config.h * Update * Update * Update * Auto update version from '0.37.0-dev3' to '0.37.0-dev4' * Fix _version * Auto update version from '0.37.0-dev6' to '0.37.0-dev7' * Update format * trigger ci --------- Co-authored-by: ringo-but-quantum <github-ringo-but-quantum@xanadu.ai>
CPhase
in favour ofCPhaseShift
in Lightning devices. qml.CPhase is an alias forqml.ControlledPhaseShift
and can be removed from the the list of supported ops and toml files.[sc-62667]
[sc-58634]