Skip to content
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

DeviceSpecification serialization implementation #5375

Merged

Conversation

verult
Copy link
Collaborator

@verult verult commented May 18, 2022

  • Serialization function uses the signature proposed in PR DeviceSpecification serialization approach #5323
  • Can still populate a DeviceSpecification previously created from the old serialization method. This allows DeviceSpecification to be serialized in a backward-compatible way prior to the removal of old DeviceSpecification fields associated with SerializableDevice.
  • Refactored grid_device_test.py to reuse existing spec generation helper functions to proto serialization testing.

@dstrain115

@maffoo @tanujkhattar FYI

@verult verult requested a review from dstrain115 May 18, 2022 22:32
@verult verult requested review from wcourtney, a team, vtomole and cduck as code owners May 18, 2022 22:32
@CirqBot CirqBot added the size: L 250< lines changed <1000 label May 18, 2022
@verult verult force-pushed the cg-device-refactor/device-serialization-impl branch 2 times, most recently from aabeb10 to 2a5041d Compare May 18, 2022 23:11
@verult verult changed the title DeviceSpecification serialization DeviceSpecification serialization implementation May 18, 2022
@verult verult force-pushed the cg-device-refactor/device-serialization-impl branch from 2a5041d to 6bfd753 Compare May 18, 2022 23:17
@verult verult force-pushed the cg-device-refactor/device-serialization-impl branch 2 times, most recently from 8016228 to 6a2402e Compare June 3, 2022 23:07
verult added 2 commits June 3, 2022 23:09
* Serialization function uses the signature proposed in PR quantumlib#5323
* Can still populate a DeviceSpecification previously created from the old serialization method. This allows DeviceSpecification to be serialized in a backward-compatible way prior to the removal of old DeviceSpecification fields associated with `SerializableDevice`.
* Refactored grid_device_test.py to reuse existing spec generation helper functions to proto serialization testing.
@verult verult force-pushed the cg-device-refactor/device-serialization-impl branch from 6a2402e to 2c8da3b Compare June 3, 2022 23:10
@verult verult added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jun 6, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jun 6, 2022
@CirqBot CirqBot merged commit ed89b40 into quantumlib:master Jun 6, 2022
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Jun 6, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* Serialization function uses the signature proposed in PR quantumlib#5323
* Can still populate a DeviceSpecification previously created from the old serialization method. This allows DeviceSpecification to be serialized in a backward-compatible way prior to the removal of old DeviceSpecification fields associated with `SerializableDevice`.
* Refactored grid_device_test.py to reuse existing spec generation helper functions to proto serialization testing.

@dstrain115 

@maffoo @tanujkhattar FYI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: L 250< lines changed <1000
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants