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

Replace _PhasedFSimConverter in cirq_google with new transformer primitives #5095

Merged
merged 6 commits into from
Mar 17, 2022

Conversation

tanujkhattar
Copy link
Collaborator

@tanujkhattar tanujkhattar requested review from wcourtney, a team, vtomole, cduck and verult as code owners March 17, 2022 19:48
@tanujkhattar tanujkhattar requested a review from mpharrigan March 17, 2022 19:48
@CirqBot CirqBot added the size: S 10< lines changed <50 label Mar 17, 2022
Copy link
Collaborator

@MichaelBroughton MichaelBroughton 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 type fixes.

@tanujkhattar tanujkhattar added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Mar 17, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Mar 17, 2022
@CirqBot
Copy link
Collaborator

CirqBot commented Mar 17, 2022

Automerge cancelled: A required status check is not present.

Missing statuses: ['Pytest Windows (3.7)', 'Pytest Windows (3.8)', 'Pytest Windows (3.9)', 'Typescript lint check', 'Typescript tests', 'Typescript tests coverage']

@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 Mar 17, 2022
@tanujkhattar tanujkhattar merged commit a7df579 into quantumlib:master Mar 17, 2022
@tanujkhattar tanujkhattar deleted the phased_fsim_transformer branch March 17, 2022 23:28
tonybruguier pushed a commit to tonybruguier/Cirq that referenced this pull request Apr 14, 2022
…itives (quantumlib#5095)

* Replace _PhasedFSimConverter in cirq_google with new transformer primitives

* Fix mypy types

* Improve test coverage
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…itives (quantumlib#5095)

* Replace _PhasedFSimConverter in cirq_google with new transformer primitives

* Fix mypy types

* Improve test coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: S 10< lines changed <50
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants