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

Split out matplotlib and qcstyle as separate modules #1105

Merged
merged 2 commits into from Oct 16, 2018

Conversation

Projects
None yet
2 participants
@mtreinish
Copy link
Member

commented Oct 15, 2018

Summary

This commit starts the decomposition of the _circuit_visualization
module into having separate modules per output backend. The
_circuit_visualization module has is quite bulky containing the
implementation of 2 different visualization backends. This starts by
splitting out matplotlib and QCStyle. The latex backend will be done
soon, but it currently in flux with multiple open pull requests. Once
development calms on that backend it too will be spit into a separate
module.

Details and comments

Starts #1104

We'll also have to update _qcstyle.py here if #1100 merges first.

Split out matplotlib and qcstyle as separate modules
This commit starts the decomposition of the _circuit_visualization
module into having separate modules per output backend. The
_circuit_visualization module has is quite bulky containing the
implementation of 2 different visualization backends. This starts by
splitting out matplotlib and QCStyle. The latex backend will be done
soon, but it currently in flux with multiple open pull requests. Once
development calms on that backend it too will be spit into a separate
module.

Starts #1104

@mtreinish mtreinish force-pushed the mtreinish:split-circuit-backends branch from f5669bf to d46b84f Oct 15, 2018

@mtreinish mtreinish merged commit 30786bc into Qiskit:master Oct 16, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details

@mtreinish mtreinish deleted the mtreinish:split-circuit-backends branch Oct 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.