Skip to content

In the handout for beginners, plt.subplots(2, 1) should be used. #117

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.

Sign up for GitHub

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

Jump to bottom
Closed
radarFudan opened this issue Oct 1, 2022 · 4 comments
Closed

In the handout for beginners, plt.subplots(2, 1) should be used. #117

radarFudan opened this issue Oct 1, 2022 · 4 comments

Comments

@radarFudan
Copy link
Contributor

I'm using matplotlib-3.6.0.

When I use plt.subplots((2, 1)), there is an error: ValueError: Number of rows must be a positive integer, not (2, 1)
@jklymak
Copy link
Member

jklymak commented Oct 1, 2022

Can you be more specific where this typo is on the cheat sheet?

@radarFudan
Copy link
Contributor Author

It's at the third column of the handout for beginners.

In the second code block of Organize.
fig, (ax1, ax2) = plt.subplots((2,1))

@rougier
Copy link
Member

rougier commented Oct 10, 2022

Thanks for the report. Can you make a PR?

radarFudan added a commit to radarFudan/cheatsheets that referenced this issue Oct 10, 2022
@radarFudan
Update the beginner tex file for the issue matplotlib#117
c1f9725
@radarFudan radarFudan mentioned this issue Oct 10, 2022
Merged
timhoffm added a commit that referenced this issue Oct 11, 2022
@timhoffm
Merge pull request #118 from radarFudan/master
56c1693 
Update the beginner tex file for the issue #117
@QuLogic
Copy link
Member

QuLogic commented Oct 11, 2022

Fixed by #118.

@QuLogic QuLogic closed this as completed Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@QuLogic @rougier @jklymak @radarFudan