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

Corrected preset for xyz2 hexagonal stabilizer code #343

Merged
merged 7 commits into from
Sep 22, 2023
Merged

Conversation

dsvandet
Copy link
Collaborator

Summary

Corrected Issue #342 where the problem was that the preset for the weight 6 operators was incorrect as per the issue.

Details and comments

"cYZX2-hXX": [
            PauliList(["YZXYZX"], input_qubit_order="left-to-right"),  # 0
            PauliList(["YZXYZX"], input_qubit_order="left-to-right"),  # 1
            PauliList(["YZXYZX"], input_qubit_order="left-to-right"),  # 2
            None,  # 3
            PauliList(["XX"]),  # 4
            None,  # 5
            None,  # 6
            PauliList(["XX"]),  # 7
            None,  # 8
            None,  # 9
            PauliList(["XX"]),  # 10
            None,  # 11
        ],

Closes #342

@dsvandet dsvandet requested a review from a user March 13, 2023 14:05
Copy link
Collaborator

@grace-harper grace-harper left a comment

Choose a reason for hiding this comment

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

Hey Drew,
Do we already have a place where we're testing the preset?

@dsvandet
Copy link
Collaborator Author

Not for the Tile since the presets can be used for a range of codes of which some will be correct and others will not be correct. I guess that we could write a test for the builder that is using the tile and preset that would check that the code being generated is the code in question. That is probably a good idea.

@grace-harper grace-harper requested review from grace-harper and removed request for a user March 13, 2023 18:25
Copy link
Collaborator

@grace-harper grace-harper left a comment

Choose a reason for hiding this comment

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

LGTM

@grace-harper grace-harper merged commit e43f227 into main Sep 22, 2023
3 checks passed
grace-harper added a commit that referenced this pull request Oct 2, 2023
Co-authored-by: grace-harper <119029214+grace-harper@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct Hexagonal Tile preset for XYZ^2 Hexagonal Stabilizer code
2 participants