Skip to content

Conversation

daxfohl
Copy link
Collaborator

@daxfohl daxfohl commented Jan 28, 2022

Fixes #2862 by making the unitary subspace configurable.

@Strilanc @dabacon do we want to make this explicit or does it just create extra confusion, especially since it can't do arbitrary subdimensions that can't be represented as a slice?

@daxfohl daxfohl requested review from cduck, vtomole and a team as code owners January 28, 2022 20:33
@daxfohl daxfohl requested a review from dabacon January 28, 2022 20:34
@CirqBot CirqBot added the size: M 50< lines changed <250 label Jan 28, 2022
@daxfohl daxfohl changed the title Make unitary subspaces configurable Allow selection of subdimension in cirq.unitary Mar 25, 2022
@daxfohl daxfohl changed the title Allow selection of subdimension in cirq.unitary Allow selection of subdimension in cirq.apply_unitary Mar 25, 2022
@daxfohl daxfohl changed the title Allow selection of subdimension in cirq.apply_unitary Allow choice of subdimension in cirq.apply_unitary Mar 25, 2022
@95-martin-orion
Copy link
Collaborator

Pinging reviewers - please reassign if necessary.

Copy link
Collaborator

@dabacon dabacon left a comment

Choose a reason for hiding this comment

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

What do you mean by:
since it can't do arbitrary subdimensions that can't be represented as a slice
?

I guess the main use of this is for mismatched qid cases?

@MichaelBroughton MichaelBroughton requested a review from dabacon June 6, 2022 17:47
Copy link
Collaborator

@dabacon dabacon 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 Jun 7, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jun 7, 2022
@CirqBot CirqBot merged commit abdd763 into quantumlib:master Jun 7, 2022
@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 Jun 7, 2022
@daxfohl daxfohl deleted the unitaryslice branch June 8, 2022 05:06
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Fixes quantumlib#2862 by making the unitary subspace configurable.

@Strilanc @dabacon do we want to make this explicit or does it just create extra confusion, especially since it can't do arbitrary subdimensions that can't be represented as a slice?
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
Fixes quantumlib#2862 by making the unitary subspace configurable.

@Strilanc @dabacon do we want to make this explicit or does it just create extra confusion, especially since it can't do arbitrary subdimensions that can't be represented as a slice?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: M 50< lines changed <250
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document in cirq.apply_unitary's docstring that it can operate on qudit subspaces
5 participants