-
Notifications
You must be signed in to change notification settings - Fork 45
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
Use the U gate to translate Qiskit U-like gates #133
Use the U gate to translate Qiskit U-like gates #133
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.
Approving to merge post 3.8 deprecation
I think this is actually still worth finishing without waiting for the service; the service doesn't support the |
a16bb15
to
42baec9
Compare
I'm not sure the service handles more gracefully |
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'm not sure the service handles more gracefully
U
gates than control modifiers. We could try and fix it if it is a real issue.
The way I see it, since the service supports neither control modifiers nor
This can also help remind users of the full capabilities of the local simulator.
Summary
Replace
U
s decomposition by the application of a the Braket U gate.Details and comments
Since U has been added to the Braket SDK, we simplify the implementation here.
Bump qiskit-terra version to solve an occurence of Qiskit/qiskit#7326