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.

Sign up for GitHub

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

Jump to bottom

[surface code] Organization #1183

Merged
merged 1 commit into from
Jul 24, 2024
Merged

[surface code] Organization #1183

merged 1 commit into from
Jul 24, 2024

Conversation

mpharrigan
Copy link
Collaborator

This is based off of #1157. See the one new commit for just the changes in this PR until that one gets merged and I rebase.

  • Use the names "Beverland et al" and "Gidney Fowler" everywhere
  • Split the CCZ2T factory from the functions associated with the model
  • rename to "QECScheme"
  • Remove global variables, which weren't pep8 names and the names would have been too many capital letters. Port them to consistently-named factory methods on their respective classes.
  • Relative imports in __init__.py

@mpharrigan mpharrigan mentioned this pull request Jul 24, 2024
Merged
@mpharrigan mpharrigan force-pushed the 2024-07/physicost-org branch from 883cca7 to abd8323 Compare July 24, 2024 18:49
@mpharrigan
Copy link
Collaborator Author

This is rebased and ready for review

@mpharrigan mpharrigan merged commit 3ef97f8 into quantumlib:main Jul 24, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NoureldinYosri NoureldinYosri NoureldinYosri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mpharrigan @NoureldinYosri