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 inverse method for Clifford tableau #4111

Merged
merged 23 commits into from
May 21, 2021

Conversation

BichengYing
Copy link
Collaborator

Another preparation step for making general Clifford Gates #3639.

The implementation relies on the then method (composing two Clifford tableaux) check the code in #4096.
This branch is based on the branch used in #4096. Hence, there are a few then related codes, which should be orthogonal of this PR.

@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label May 16, 2021
@BichengYing BichengYing marked this pull request as ready for review May 16, 2021 22:38
@BichengYing BichengYing requested review from cduck, vtomole and a team as code owners May 16, 2021 22:38
@BichengYing BichengYing requested a review from mpharrigan May 16, 2021 22:38
@balopat balopat requested review from balopat and removed request for mpharrigan May 17, 2021 22:33
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 to me. Let's wait for #4096 to go in.

@BichengYing BichengYing requested a review from balopat May 21, 2021 00:46
@balopat balopat added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label May 21, 2021
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label May 21, 2021
@CirqBot CirqBot merged commit 0c0366b into quantumlib:master May 21, 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 May 21, 2021
NoureldinYosri pushed a commit to NoureldinYosri/Cirq that referenced this pull request May 25, 2021
Another preparation step for making general Clifford Gates quantumlib#3639.

The implementation relies on the `then` method (composing two Clifford tableaux) check the code in quantumlib#4096.
This branch is based on the branch used in quantumlib#4096. Hence, there are a few `then` related codes, which should be orthogonal of this PR.
@BichengYing BichengYing deleted the inv_tableau branch November 6, 2021 04:11
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Another preparation step for making general Clifford Gates quantumlib#3639.

The implementation relies on the `then` method (composing two Clifford tableaux) check the code in quantumlib#4096.
This branch is based on the branch used in quantumlib#4096. Hence, there are a few `then` related codes, which should be orthogonal of this PR.
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
Another preparation step for making general Clifford Gates quantumlib#3639.

The implementation relies on the `then` method (composing two Clifford tableaux) check the code in quantumlib#4096.
This branch is based on the branch used in quantumlib#4096. Hence, there are a few `then` related codes, which should be orthogonal of this PR.
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.

3 participants