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

Add common gate families as part of adopting gatesets across Cirq. #4517

Merged
merged 5 commits into from Sep 23, 2021

Conversation

tanujkhattar
Copy link
Collaborator

@tanujkhattar tanujkhattar commented Sep 22, 2021

This PR introduces new common gate families which will be useful to adopt the usage of Gatesets across Cirq. This is split off the larger PR #4513 and follow up PRs will use these new gate families.

Part of the roadmap item #3243

@tanujkhattar tanujkhattar requested review from cduck, vtomole and a team as code owners September 22, 2021 22:51
@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Sep 22, 2021
@CirqBot CirqBot added the size: M 50< lines changed <250 label Sep 22, 2021
cirq-core/cirq/ops/common_gate_families.py Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families.py Outdated Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families.py Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families.py Outdated Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families.py Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families_test.py Outdated Show resolved Hide resolved
cirq-core/cirq/ops/common_gate_families_test.py Outdated Show resolved Hide resolved
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

@MichaelBroughton MichaelBroughton added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Sep 23, 2021
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Sep 23, 2021
@CirqBot CirqBot merged commit 4d1b9c0 into quantumlib:master Sep 23, 2021
@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 Sep 23, 2021
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…uantumlib#4517)

This PR introduces new common gate families which will be useful to adopt the usage of Gatesets across Cirq. This is split off the larger PR quantumlib#4513 and follow up PRs will use these new gate families. 


Part of the roadmap item quantumlib#3243
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. size: M 50< lines changed <250
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants