make plt.subplot() act as plt.subplot(111) #1475
Conversation
Since plt.subplots() make an assumption of what you want when you call it without arguments, and as does plt.axes(), there's no reason why plt.subplot() shouldn't make the same assumption.
| @@ -1,3 +1,5 @@ | |||
| 2012-11-09 Make plt.subplot() without arguments act as subplot(111) - PI | |||
|
|
|||
There was a problem hiding this comment.
I don't think we are using CHANGELOG any more. I think this should go in whats_new.rst instead.
There was a problem hiding this comment.
I have no problem with updating CHANGELOG, although it is getting sparse to do so now. But this definitely should go into whats_new.rst.
There was a problem hiding this comment.
Ok, but on the first line (after the title) of whats_new.rst says:
This page just covers the highlights -- for the full story, see the
`CHANGELOG <http://matplotlib.org/_static/CHANGELOG>`_
EDIT: got my wires crossed and thought this comment was about a different PR. Now updated this PR as well
There was a problem hiding this comment.
Thanks.
My guess is that it will go into 1.3 (or whatever the new version is called), since 1.2.1 should be for critical bugfixes.
There was a problem hiding this comment.
My feeling is that "What's new" is for high-level user-visible changes (sort of like an advertisment for the new release), and CHANGELOG (which I believe we are still using) is for smaller details that users need to know about. We also have api_changes for things where we know users need to change their code. (I don't think this falls into that category because subplots() would have raised an exception before and now does something new).
|
I like this. How does everyone else feel? |
|
+1 on this harmless syntaxical sugar. |
|
+1 on the idea. |
|
Merged. |
| @@ -768,6 +771,10 @@ def subplot(*args, **kwargs): | |||
| .. plot:: mpl_examples/pylab_examples/subplot_demo.py | |||
|
|
|||
| """ | |||
| # if subplot called without arguments, create subplot(1,1,1) | |||
| if len(args)==0: | |||
There was a problem hiding this comment.
It's merged so it doesn't matter, but in future we are trying to standardise the code base towards PEP8 which recommends spaces between operators (and after commas).
I'm not worried about getting this sorted (it will get picked up eventually by an automatable tool).
Cheers,
Since plt.subplots() make an assumption of what you want when you call
it without arguments, and as does plt.axes(), there's no reason why
plt.subplot() shouldn't make the same assumption.