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

Quick fix for a typo in tutorials/dottest.py #556

Closed
wants to merge 1 commit into from

Conversation

alex-rakowski
Copy link
Contributor

changes:
gs = pltgs.GridSpec(1, 9) to gs = plt.GridSpec(1, 9)

@mrava87
Copy link
Collaborator

mrava87 commented Nov 29, 2023

I think is due to the import import matplotlib.gridspec as pltgs. Perhaps nowadays it is not needed anymore to import that specific submodule, but if we do the change you suggest we should also remove import matplotlib.gridspec as pltgs. Agree?

@alex-rakowski
Copy link
Contributor Author

alex-rakowski commented Nov 29, 2023

I made a mistake. I didn't notice the import when I was copying and working through the example.

I could edit the pltgs import and usage as I think it's not needed anymore, but it works so maybe simplest to just leave it unchanged.

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