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

partition_circuit_qubits #140

Merged
merged 11 commits into from
May 12, 2023
Merged

partition_circuit_qubits #140

merged 11 commits into from
May 12, 2023

Conversation

caleb-johnson
Copy link
Collaborator

@caleb-johnson caleb-johnson commented May 6, 2023

Resolves #101
Resolves #138

This PR establishes a cutting module in the circuit_cutting package by implementing a partition_circuit_qubits function. It assumes cutqc has already been packaged in accordance with #138 , and it assumes the base QPD classes are available as in #137 .

This PR was created so we could discuss the arrangement of the packages, modules, and CI while we finalize the implementation internally.

@coveralls
Copy link

coveralls commented May 6, 2023

Pull Request Test Coverage Report for Build 4953717621

  • 27 of 27 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 86.547%

Totals Coverage Status
Change from base Build 4953083582: 0.2%
Covered Lines: 1782
Relevant Lines: 2059

💛 - Coveralls

caleb-johnson and others added 5 commits May 11, 2023 18:36
Co-authored-by: Jim Garrison <jim@garrison.cc>
Co-authored-by: Almudena Carrera Vazquez <acv@zurich.ibm.com>
Co-authored-by: Julien Gacon <jules.gacon@googlemail.com>
Co-authored-by: Daniel Egger <38065505+eggerdj@users.noreply.github.com>
Co-authored-by: Stefan Woerner <WOR@zurich.ibm.com>
Co-authored-by: Jen Glick <41485571+jenglick@users.noreply.github.com>
Co-authored-by: Pedro Rivero Ramirez <pedro.rivero.ramirez@gmail.com>
Co-authored-by: Bryce Fuller <bgf024@gmail.com>
garrison
garrison previously approved these changes May 11, 2023
Copy link
Member

@garrison garrison left a comment

Choose a reason for hiding this comment

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

Thanks. If the current version passes tests, I think it is good to merge.

@caleb-johnson caleb-johnson removed the on hold Let's wait for something before merging label May 11, 2023
@caleb-johnson caleb-johnson merged commit 89bf0cc into main May 12, 2023
8 checks passed
@caleb-johnson caleb-johnson deleted the cutting-module branch May 12, 2023 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cutting QPD-based circuit cutting code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a function for separating a quantum circuit along specified qubit boundaries
3 participants