Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
Various examples updates.#10326
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| fig=plt.figure(1,figsize=(6,3)) | ||
| fig, (ax1,ax2)=plt.subplots(1,2) | ||
| fig.subplots_adjust(wspace=0.5) |
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.
Could be added tosubplots viagridspec_kw.
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.
Not convinced it's an improvement in legibility...
| importmatplotlib.pyplotasplt | ||
| fromnumpyimportnonzero | ||
| fromnumpy.randomimportrand | ||
| frommatplotlibimportcolorsasmcolors,path,pyplotasplt |
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.
Not sure if mixedimport andimport as are easy to read.
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.
disagree, but still split it out
| """ | ||
| frommatplotlib.pyplotimportfigure,show | ||
| frommatplotlibimportpyplotasplt |
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.
import matplotlib.pyplot as plt (also in various other locations, please search/replace)
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.
fixed
| # plot specifying units | ||
| ax2.plot(x,y,'o',xunits=2.0) | ||
| ax2.set_title("xunits = 2.0") | ||
| plt.setp(ax2.get_xticklabels(),rotation=30,ha='right') |
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.
Broadcasting explicitly to a list of tick labels feels a bit clumsy. Is there maybe a method to set this for all tick labels, e.g.tick_params or similar?
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.
tick_params can set labelrotation, but not horizontalalignment (and I don't think it's worth changing that, IMO setp actually does this job perfectly well...).
| label.set_ha('right') | ||
| ax1.plot(x,y)# uses default units | ||
| ax1.set_title('default units') | ||
| plt.setp(ax1.get_xticklabels(),rotation=30,ha='right') |
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.
see above
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.
ditto
| t=arange(0.0,3.0,0.01) | ||
| s=sin(2*pi*t) | ||
| t=np.arange(0.0,3.0,0.01) | ||
| s=np.sin(2*np.pi*t) |
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.
Do we have a convention, if this should include spaces around the operators? PEP8 would want them. Throughout the examples both variants are used.
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.
See discussion at#7562 (comment) which I now agree with: I still believe spaces are in general good, but in some cases no spaces read better.
See alsohttps://www.python.org/dev/peps/pep-0008/#other-recommendations ("use your own judgment", hehe).
jklymak left a comment
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.
These changes are all fine +/-@timhoffm suggestions. Probably could make more, but overall the movement away from the pyplot interface is helpful.
- don't import out of pyplot or numpy, import as plt/np.- move some examples to use subplots() instead of add_subplot.- set random seed for some examples.- fix some docstrings.
PR Summary
PR Checklist