Skip to content

Conversation

@mpharrigan
Copy link
Collaborator

Part 2 of #2781

Follows #3371

@googlebot googlebot added the cla: yes Makes googlebot stop complaining. label Sep 30, 2020
@mpharrigan mpharrigan force-pushed the 2020-10-operator-estimation-2-grouping branch from db77b75 to ec9d941 Compare September 30, 2020 21:02
@mpharrigan mpharrigan force-pushed the 2020-10-operator-estimation-2-grouping branch from ec9d941 to 500cb25 Compare October 6, 2020 03:49
@mpharrigan mpharrigan force-pushed the 2020-10-operator-estimation-2-grouping branch from 500cb25 to 581cc73 Compare October 13, 2020 05:37
@mpharrigan mpharrigan force-pushed the 2020-10-operator-estimation-2-grouping branch from 581cc73 to d89bc46 Compare October 20, 2020 15:46
@mpharrigan mpharrigan requested review from balopat and viathor October 20, 2020 15:46
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.

This looks good, I added a bunch of nits

mpharrigan and others added 5 commits October 20, 2020 20:31
Co-authored-by: Balint Pato <balopat@users.noreply.github.com>
Co-authored-by: Balint Pato <balopat@users.noreply.github.com>
…ping' into 2020-10-operator-estimation-2-grouping
@mpharrigan
Copy link
Collaborator Author

Your suggested change to the None handling broke mypy!

@mpharrigan
Copy link
Collaborator Author

Also the formatter insists on a formatting that breaks pylint (line too long)

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.

Awesome, thanks for the quick fixes!

@balopat balopat added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Oct 21, 2020
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Oct 21, 2020
@CirqBot CirqBot merged commit 6a151af into quantumlib:master Oct 21, 2020
@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 Oct 21, 2020
@mpharrigan mpharrigan deleted the 2020-10-operator-estimation-2-grouping branch October 21, 2020 04:51
CirqBot pushed a commit that referenced this pull request Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/expectation-value cla: yes Makes googlebot stop complaining.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants