-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add get config functions #7565
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
base: main
Are you sure you want to change the base?
Add get config functions #7565
Conversation
…ssor-config' into add_get_config_snapshot
I'm working on fixing the errors. Some missing tests/logic for a few edge cases. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7565 +/- ##
========================================
Coverage 99.37% 99.37%
========================================
Files 1078 1080 +2
Lines 96162 96484 +322
========================================
+ Hits 95559 95881 +322
Misses 603 603 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cirq-google/cirq_google/cloud/quantum_v1alpha1/gapic_version.py
Outdated
Show resolved
Hide resolved
I've verified that this works fully end-to-end with a colab containing the following code:
|
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.
Looks good on my part. Looks like there are just a couple of comments left from Will.
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.
Overall LGTM % a few remaining nits.
…g functions from abstract classes.
Adds the ability to retrieve a
QuantumProcessorConfig
from a project. There are two ways to retrieve processor configs: from theEngine
or anEngineProcessor
.Here is an example using the processor:
Here is an example getting the config directly from the Engine: