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

Add sampling overhead table and expand/update cutting explanation (backport #342) #553

Merged
merged 2 commits into from
Apr 17, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Apr 17, 2024

This adds a sampling overhead table. I should triple check it. I may push other improvements to the cutting explanation to this branch, too.

EDIT: I've added many other improvements, too.

Fixes #293.
Fixes #373.


This is an automatic backport of pull request #342 done by Mergify.

* Add sampling overhead table

* typos

* Fix heading level

* Rearrange table

* Fix some Sphinx cross references

* Fix Sphinx LaTeX expressions

* Add link to IBM blog post on LOCC cutting

* Tweaks to intro

* Lots of progress

* Update docs/circuit_cutting/explanation/index.rst

Co-authored-by: Ibrahim Shehzad <[email protected]>

* Rephrase LOCC vs dynamic circuits

following discussion at
#342 (comment)

* Removed `PauliList` limitation, lifted by #523

Follow up to
#342 (comment)

* Rephrase circuit cutting intro

#342 (comment)

* Remove circuit cutting graph

It's actually very similar to the one at the top of the explanation

#342 (comment)

* Update docs/circuit_cutting/explanation/index.rst

Co-authored-by: Ibrahim Shehzad <[email protected]>

* Update docs/circuit_cutting/explanation/index.rst

Co-authored-by: Ibrahim Shehzad <[email protected]>

---------

Co-authored-by: Ibrahim Shehzad <[email protected]>
(cherry picked from commit 299c849)

# Conflicts:
#	test/cutting/qpd/test_qpd.py
Copy link
Author

mergify bot commented Apr 17, 2024

Cherry-pick of 299c849 has failed:

On branch mergify/bp/stable/0.6/pr-342
Your branch is up to date with 'origin/stable/0.6'.

You are currently cherry-picking commit 299c849.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/circuit_cutting/explanation/index.rst
	modified:   releasenotes/notes/0.2/gate-cutting-workflow-ebbedbdb1313b258.yaml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   test/cutting/qpd/test_qpd.py

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify bot added the conflicts label Apr 17, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8728371529

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 93.541%

Totals Coverage Status
Change from base Build 8631862982: 0.0%
Covered Lines: 2346
Relevant Lines: 2508

💛 - Coveralls

@garrison garrison merged commit cd8adfb into stable/0.6 Apr 17, 2024
11 checks passed
@garrison garrison deleted the mergify/bp/stable/0.6/pr-342 branch April 17, 2024 21:17
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.

2 participants