Add a coloring how-to #30
Merged
Conversation
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
Co-authored-by: Max Rossmannek <[email protected]>
Pull Request Test Coverage Report for Build 10844302431Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
caleb-johnson
added
the
stable backport potential
|
@mrossinek I can probably get graphviz installed in the CI envs, but I'm a bit confused why OBP can use graphviz_draw in the notebooks using the same type of dependency specification of rustworkx[graphviz] that I do here. |
mrossinek
reviewed
Sep 12, 2024
mrossinek
reviewed
Sep 13, 2024
mrossinek
approved these changes
Sep 13, 2024
mergify bot
pushed a commit
that referenced
this pull request
Sep 13, 2024
* Add a coloring how-to Co-authored-by: Max Rossmannek <[email protected]> * style * lint * Add runtime to notebook deps for fake backends * Get graphviz with rustworkx * fix duplicate * Clean up tutorial * Skip graphviz cells for execution * style :) * Didn't save second metadata change * Evidently one of the tags wasn't checked? *sigh* * peer review * style * that/which pedantry * rephrase * Improve output * Use single quotes * docs: use slice_by_depth in colored circuit, too --------- Co-authored-by: Max Rossmannek <[email protected]> (cherry picked from commit 7c42e3c)
mrossinek
pushed a commit
that referenced
this pull request
Sep 13, 2024
* Add a coloring how-to Co-authored-by: Max Rossmannek <[email protected]> * style * lint * Add runtime to notebook deps for fake backends * Get graphviz with rustworkx * fix duplicate * Clean up tutorial * Skip graphviz cells for execution * style :) * Didn't save second metadata change * Evidently one of the tags wasn't checked? *sigh* * peer review * style * that/which pedantry * rephrase * Improve output * Use single quotes * docs: use slice_by_depth in colored circuit, too --------- Co-authored-by: Max Rossmannek <[email protected]> (cherry picked from commit 7c42e3c) Co-authored-by: Caleb Johnson <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a how-to showing how coloring can be used to improve the depth of circuits by more efficiently placing gates.