-
Notifications
You must be signed in to change notification settings - Fork 1.2k
GridDeviceMetadata gate durations: docstring for looking up gate duration via searching through GateFamilies #5438
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
GridDeviceMetadata gate durations: docstring for looking up gate duration via searching through GateFamilies #5438
Conversation
…tion via searching through GateFamilies
MichaelBroughton
left a comment
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 after nits.
tanujkhattar
left a comment
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 % nit
Co-authored-by: Tanuj Khattar <tanujkhattar@google.com>
|
Automerge cancelled: A required status check is not present. Missing statuses: ['Build docs', 'Build protos', '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'] |
…tion via searching through GateFamilies (quantumlib#5438) Replaces quantumlib#5427 @MichaelBroughton cc @tanujkhattar
…tion via searching through GateFamilies (quantumlib#5438) Replaces quantumlib#5427 @MichaelBroughton cc @tanujkhattar
Replaces #5427
@MichaelBroughton
cc @tanujkhattar