Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Section change in the getting started section #417

Closed
jaygambetta opened this issue Aug 17, 2019 · 3 comments
Closed

Section change in the getting started section #417

jaygambetta opened this issue Aug 17, 2019 · 3 comments
Assignees
Labels
documentation Something is not clear or error in description

Comments

@jaygambetta
Copy link
Member

I think we should not have set output to mpl in the code (as it might break if they dont have the Matplotlib installed.

image

Could we remove this but have a note after that if your draw is text based and you want a nicer one then set your user config.

@jaygambetta jaygambetta added the documentation Something is not clear or error in description label Aug 17, 2019
@nonhermitian
Copy link
Collaborator

Could we have an user config setup for the build that sets mpl for the default, then the draw command would work in all cases and output still look good. We could still have a note of course

@jaygambetta
Copy link
Member Author

jaygambetta commented Aug 17, 2019

have a look at the pr I just did which is just a note but I agree we could put the user config in the install (for example) and in the getting started just have a smaller note linking to the user config section and then saying draw using whatever option you want. I think this is better than my pr but I let you decided. If you want to accept mine and then change later happy with that or close and do how you want im good as well.

@yesodharan1892
Copy link

hi,
i believe your are looking for a better representation of mpl,
may i suggest using
%matplotlib inline # just once in the project and then implement
circuit.draw(output='mpl') # i am sure you will see a better representation of mpl, till the time the qiskit team updates the user specific option.
hope you are okay with the resolution. if not please revert.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Something is not clear or error in description
Projects
None yet
Development

No branches or pull requests

3 participants