-
Notifications
You must be signed in to change notification settings - Fork 984
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
Move CliffordTableau and add unit test #4085
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.
Thank you!
I added some comments + we need to add a test for the deprecated behavior - see unit test examples where cirq.testing.assert_deprecated
is used.
Thanks. Added the assert_deprecated. PTAL. |
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.
LGTM, beautiful! Thanks for writing the great tests for it!
Fix quantumlib#4075. To avoid breakage, the original `cirq/sim/clifford/CliffordTableau` is untouched added with deprecated notation. All internal import of Cliffod Tableau is pointed to the new position under `cirq/qis`. (Unless people have strong opinions of putting it into other places.) New clifford_tableau is the same as the old one except in one place -- the `xs`, `zs`, and `rs` are protected by getter and setter pattern and do not expose the last scratch row.
Fix #4075.
To avoid breakage, the original
cirq/sim/clifford/CliffordTableau
is untouched added with deprecated notation.All internal import of Cliffod Tableau is pointed to the new position under
cirq/qis
. (Unless people have strong opinions of putting it into other places.)New clifford_tableau is the same as the old one except in one place -- the
xs
,zs
, andrs
are protected by getter and setter pattern and do not expose the last scratch row.