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

Allow using plain qubits as keys for single-qubit heatmaps #4028

Merged
merged 2 commits into from
Apr 27, 2021

Conversation

maffoo
Copy link
Contributor

@maffoo maffoo commented Apr 16, 2021

It's a bit annoying that mypy seems to get confused unless I explicitly overload the __init__ type signature. If I just try to type value_map as a Union type, typechecks fail at callsites.

Also cleaned up a few types while I was in there, in particular we are passing around polygons as lists of (x, y) coords, not as matplotlib Polygon objects.

@maffoo maffoo requested review from cduck, vtomole and a team as code owners April 16, 2021 18:12
@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Apr 16, 2021
Copy link
Contributor

@balopat balopat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, after rebase

@maffoo maffoo added automerge Tells CirqBot to sync and merge this PR. (If it's running.) and removed pr/needs-rebase labels Apr 27, 2021
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Apr 27, 2021
@CirqBot CirqBot merged commit 58bebc8 into master Apr 27, 2021
@CirqBot CirqBot removed the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Apr 27, 2021
@CirqBot CirqBot deleted the u/maffoo/heatmaps branch April 27, 2021 19:45
@CirqBot CirqBot removed the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Apr 27, 2021
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…b#4028)

It's a bit annoying that mypy seems to get confused unless I explicitly overload the `__init__` type signature. If I just try to type `value_map` as a `Union` type, typechecks fail at callsites.

Also cleaned up a few types while I was in there, in particular we are passing around polygons as lists of (x, y) coords, not as matplotlib `Polygon` objects.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Makes googlebot stop complaining.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants