-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
DOC: Make pyplot import explicit in the 10 minutes to pandas page #30499
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
Conversation
Beginners don't know what is `plt`, thus it is better to show how to import it explicitly.
@datapythonista Could you please merge this PR? Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, thanks!
doc/source/getting_started/10min.rst
Outdated
@@ -717,6 +711,8 @@ of the columns with labels: | |||
|
|||
.. ipython:: python | |||
|
|||
import matplotlib.pyplot as plt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would leave this where it was, and maybe even add a comment (removing the suppress is ok)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jreback Thank you for your comment. I moved the import statement to the previous position and added a comment.
Also add a comment about the standard convention for referencing the matplotlib API.
pandas-dev.pandas (Web_and_Docs) is failing, but is it related to my change? |
I also have the same problem. See #30282 |
@yuseitahara can you merge master into your branch please? It should fix the CI. |
@jreback you're changes have been addressed, all green here. |
thanks @yuseitahara |
Beginners don't know what is
plt
, thus it is better to show how to import it explicitly.This is a continuation of #30274
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff