-
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
Add some more gates to CollectCliffords pass #13214
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Samantha Barron <samantha.v.barron@ibm.com>
One or more of the following people are relevant to this code:
|
Pull Request Test Coverage Report for Build 11013369525Details
💛 - Coveralls |
can collect :class:`.RZGate(np.pi/2)` gates and other unitary gates that are :class:`.Clifford()`gates | ||
for certain parameters. |
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.
can collect :class:`.RZGate(np.pi/2)` gates and other unitary gates that are :class:`.Clifford()`gates | |
for certain parameters. | |
can collect :class:`.RZGate(np.pi/2)` gates and other unitary gates that are :class:`.Clifford()` gates | |
for certain parameters. |
Note the extra space before "gates" -- I believe this is what's responsible for CI failures :).
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.
good catch!
The CI failure is not related. it's due to QPY backwards tests. should be fixed after #13202 is merged.
Summary
Close #13138
Add an argument
matrix_based
to theCollectCliffords
transpiler pass.If the new parameter
matrix_based=True
, theCollectCliffords
transpiler pass can collectRz(pi/2)
gates and other unitary gates that are Clifford gates for certain parameters.Details and comments
The PR is based on the suggestion of @samanthavbarron in #13138 and discussions with @alexanderivrii