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

Create how-to optimize transpilation guide #573

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

garrison
Copy link
Member

@garrison garrison commented May 3, 2024

This is a draft. I will update the last section once #517 merges. This PR will probably wait on that to merge first.

Remaining action items

@garrison garrison added the documentation Improvements or additions to documentation label May 3, 2024
@garrison garrison added this to the 0.8.0 milestone May 3, 2024
@coveralls
Copy link

coveralls commented May 3, 2024

Pull Request Test Coverage Report for Build 8941516520

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.495%

Totals Coverage Status
Change from base Build 8913240803: 0.0%
Covered Lines: 3498
Relevant Lines: 3663

💛 - Coveralls

@garrison
Copy link
Member Author

garrison commented May 3, 2024

By the way, this how-to guide is essentially an expanded version of the workflow test added in #303.

@garrison
Copy link
Member Author

garrison commented May 3, 2024

Try to remove need for SparsePauliOp on following line: SparsePauliOp(subobservables[label]).apply_layout(transpiled_subcircuits[label].layout).paulis

This is tied to the following issues:

It is also related to Qiskit/qiskit#12221 but that one is not directly applicable to us because we are not using ObservablesArray (which is something from qiskit.primitives).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants