-
Notifications
You must be signed in to change notification settings - Fork 13
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Pj/dta2 489/basic active volume costing (#162)
* Adding preliminary active volume architecture costing capability * Enabling running ex_1 and active volume costing improvements * Updated active volume accounting and cleaned up module * Major refactor to GraphEstimator and adding LogicalArchitectureModel classes * Adding a module to handle general magic state factory selection * Refactor to move LogicalArchitectureModels to own module and creating an AbstractLogicalResourceInfo class * Updated graph estimator interface and resource info usage throughout package, updated tests, and added tests for new magic state logging * Added tests to check active_volume compilation lowers graph state creation cost and to check that cycles for multi-T-state-distilleries get accounted for properly * Fixing style issues * Minor refactoring to reduce code duplication * Updated tests to include active volume spatial accounting * Added tests for substrate scheduler active volume functions and fixed bug in tocks accounting * Style fixes * Fixing bug that undercounted T and rotation measurements * Updated test and added test checking t measurement count * Fixing testing vestiges and some cleanup * Fixing issue about code distance not being updated in logical architecure model * Removing print statements * Updating usage of logical failure rate info and fixing minor style issues * Updating name from active volume to all to all * Updating setup with consistent flake8 style checks and shortening lines * Fixing handling of case where magic_state_factory is None * Removing python warning in decoder info that prevented tests from passing * Fixing typing incompatibility with python versions * Fixing bug in typing * Fixing further python3.9 incompatibilities that were missed in local testing * Removing support for computing T depth * Updating naming of all to all compiler in example * Fixing further style issues * Removing automatic estimator and fixing style issues * Improving type handling * Removing some type checks * Fixing style and type checking issues * Removing files that rely on Azure QRE and Orquestra. * Fixing minor black styling issues * Fixing import ordering * Fixing missed all-to-all name changes in one test module * Fixing attribute error from vestige * Populating total circuit failure rate in openfermion accounting
- Loading branch information
Showing
47 changed files
with
2,008 additions
and
1,891 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.