-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Change 1q UnitaryGate definition to use U instead of U3 #10597
Conversation
Removing U3 from unitary and moving it to the U gate
One or more of the the following people are requested to review this:
|
Pull Request Test Coverage Report for Build 5814691289
💛 - Coveralls |
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 for pushing this. The code itself LGTM and is probably long overdue.
The only thing that I think is missing is probably a release note so that we can document the change as part of the next qiskit release. If you could add one that just documents the definition of UnitaryGate
for single qubit unitaries is now in terms of UGate
instead of the legacy U3Gate
class that either as an upgrade
or an other
section note. The process for adding a release note is documented here: https://github.com/Qiskit/qiskit-terra/blob/main/CONTRIBUTING.md#release-notes
should be done |
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 for the quick update adding the release note. This LGTM now
* Clean up to use U vs U3 Removing U3 from unitary and moving it to the U gate * fixing test * Create unitary-u-not-u3-0a32c2d968f7890e.yaml
Summary
Replaced U3 with U gate so that when I decompose it uses U not U3 as U is the gate going forward.
Details and comments