-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Revert "Simultaneous readout" #4406
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
Conversation
This reverts commit eb72fb5.
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, will resubmit once I figure out the flaky tests.
Automerge cancelled: A required status check is not present. Missing statuses: ['Typescript lint check', 'Typescript tests', 'Typescript tests coverage'] |
Automerge cancelled: A required status check is not present. Missing statuses: ['Build docs', 'Build protos', 'Changed files test', 'Coverage check', 'Doc test', 'Lint check', 'Notebook formatting', 'Pytest MacOS (3.7)', 'Pytest MacOS (3.8)', 'Pytest MacOS (3.9)', 'Pytest Ubuntu (3.7)', 'Pytest Ubuntu (3.8)', 'Pytest Ubuntu (3.9)', 'Pytest Windows (3.7)', 'Pytest Windows (3.8)', 'Pytest Windows (3.9)', 'Typescript lint check', 'Typescript tests', 'Typescript tests coverage'] |
Reverts quantumlib#4307 which has introduced CI flakes here here: https://github.com/quantumlib/Cirq/pull/4402/checks?check_run_id=3297425897 Fixes quantumlib#4404.
Reverts quantumlib#4307 which has introduced CI flakes here here: https://github.com/quantumlib/Cirq/pull/4402/checks?check_run_id=3297425897 Fixes quantumlib#4404.
Reverts #4307 which has introduced CI flakes here here: https://github.com/quantumlib/Cirq/pull/4402/checks?check_run_id=3297425897
Fixes #4404.