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
remove (almost) all mentions of pylab from our examples #5066
Conversation
try: | ||
import IPython.core.shellapp as mod; | ||
for k in mod.InteractiveShellApp.pylab.values: | ||
for k in mod.InteractiveShellApp.matplotlib.values: |
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 didn't see you change the name of any InteractiveShell
attributes. Does this line assume that?
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.
This was done a while ago. Nothing to change in the actual code, just the examples.
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.
OK, thanks for clarifying.
Do we also want to rename our module and functions internally: Not nearly as big of a deal as it doesn't affect users. |
I suspect some folks reach into pylabtools for print_figure and/or figsize, so we would need to be careful about those. |
I think this looks good, merging. |
remove (almost) all mentions of pylab from our examples
The ipython devs discourage the use of pylab now (ipython/ipython#5066)
remove (almost) all mentions of pylab from our examples
I did a quick scan through our examples, and removed any mention of pylab I could find and reasonably remove.