Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

(1/2) Combine Pauli measurements and postselection#7746

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
ddddddanni wants to merge7 commits intoquantumlib:main
base:main
Choose a base branch
Loading
fromddddddanni:add_new_parameter

Conversation

@ddddddanni
Copy link
Collaborator

I split this#7492 giant PR into smaller pieces for easier review.
This PR:

  1. Adds a CircuitToPauliStringsParameters class which supports postselection_symmetries.
  2. Modify the validation method
  3. Allows measure_pauli_strings to take CircuitToPauliStringsParameters as a input.

@ddddddanniddddddanni requested review froma team andvtomole ascode ownersNovember 11, 2025 19:50
@github-actionsgithub-actionsbot added the size: L250< lines changed <1000 labelNov 11, 2025
Copy link
Collaborator

@eliottrosenbergeliottrosenberg left a comment•
edited
Loading

Choose a reason for hiding this comment

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

This PR doesn't actually apply the postselection, correct? If so, we should raise aNotImplementedError if the user tries to use postselection symmetries.

@ddddddanniddddddanni marked this pull request as ready for reviewNovember 19, 2025 01:10
@ddddddanni
Copy link
CollaboratorAuthor

This PR doesn't actually apply the postselection, correct? If so, we should raise aNotImplementedError if the user tries to use postselection symmetries.

Done!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@eliottrosenbergeliottrosenbergeliottrosenberg requested changes

@vtomolevtomoleAwaiting requested review from vtomolevtomole is a code owner

@MichaelBroughtonMichaelBroughtonAwaiting requested review from MichaelBroughtonMichaelBroughton is a code owner automatically assigned from quantumlib/cirq-maintainers

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

size: L250< lines changed <1000

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ddddddanni@eliottrosenberg
[8]ページ先頭
©2009-2025 Movatter.jp