-
Notifications
You must be signed in to change notification settings - Fork 66
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
Rename QubitStateVector
to StatePrep
#326
Conversation
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.
will check again when it works : p
@@ -129,15 +129,15 @@ def execute_supported_operation(operation_name: str, parameters: list, wires: li | |||
"""Utility function that executes an operation that is natively supported by PennyLane. | |||
|
|||
Args: | |||
operation_name (str): wires specified for the template |
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.
lol
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.
Right LOL
Co-authored-by: Matthew Silverman <[email protected]>
@@ -1,4 +1,4 @@ | |||
pennylane>=0.30 | |||
git+https://github.com/PennyLaneAI/pennylane.git |
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.
Revert this before merge
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.
and by revert you mean bump to 0.32 : )
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.
Depends if it gets merged before the pre-release version bumps LOL 👍🏼
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.
change to the RC branch and I'd be happy to merge today 😄
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.
LOL, you just want me to do the pre-release bump for you
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.
I want you to not get experimental changes for 0.33 from master : )
[sc-43547] |
Codecov Report
@@ Coverage Diff @@
## master #326 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 7 7
Lines 314 314
=========================================
Hits 314 314
|
Following the work here, we make sure that the pennylane-qiskit plugin supports both operators and defaults to using
StatePrep
where appropriate until it is deprecated.