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
Revert "Support qubits in MeasurementKey
and update JSON serialization of keys"
#4277
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.
Add unit test if you want
Done: added |
gate=cirq.MeasurementGate(1, cirq.MeasurementKey(path=(), name='q0_1_0'), ()), | ||
qubits=[cirq.GridQubit(0, 1)], | ||
) | ||
assert op == eval(repr(op)) |
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.
cirq.testing.assert_equivalent_repr
won't work here?
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 point - updated to use that instead.
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.
@vtomole, PTAL - merge is blocked on your review.
Automerge cancelled: Merging is blocked (I don't understand why). |
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
…ion of keys" (quantumlib#4277) * Revert "Support qubits in `MeasurementKey` and update JSON serialization of keys (quantumlib#4123)" This reverts commit 81436fe. * Safeguard against repr-inequality. * assert_equivalent_repr
Reverts #4123 due to issues uncovered in #4273.