BUG/VIS: fix Series.hist so that users can create hist subplots without the mpl API #4414
Conversation
ghost
assigned
| y.hist() | ||
| fig = gcf() | ||
| axes = fig.get_axes() | ||
| self.assertEqual(len(axes), 2) |
There was a problem hiding this comment.
not sure if there's a way to check for visually overlapping plots here
this test fails for the code b4 the change since there will be 3 axes instead of 2
|
it's too specific to |
|
@jreback u ok with this? fixes a couple of issues with overlapping subplots when ppl want to use the matlab-like pylab stuff |
|
also passing both color and style when style has a color symbol is not valid anymore, not sure it ever was, since the ts plot tests pass but that's b/c the error is somewhere deep in the mpl stack and it isn't propagated up to pandas |
|
so not an API change since i don't think it ever worked, but pandas won't try to create a plot if those two args are invalid, it will raise instead |
|
ok w me |
BUG/VIS: fix Series.hist so that users can create hist subplots without the mpl API
closes #4408.
closes #4402.