Skip to content

TST: cleanup warnings on mpl 2.1 #17835

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

Merged
merged 1 commit into from
Oct 10, 2017
Merged

TST: cleanup warnings on mpl 2.1 #17835

merged 1 commit into from
Oct 10, 2017

Conversation

jreback
Copy link
Contributor

@jreback jreback commented Oct 10, 2017

No description provided.

@jreback jreback added Testing pandas testing functions or related to the test suite Visualization plotting labels Oct 10, 2017
@jreback jreback added this to the 0.21.0 milestone Oct 10, 2017
@jreback
Copy link
Contributor Author

jreback commented Oct 10, 2017

we have a deprecation warning on Series.reshape for quite a while, mpl >= 2.1 now tries to reshape in box-plot, causing some warnings.

its possible @tacaswell wants to do np.asarray() in the box plots before trying to reshape.

@@ -2000,7 +2000,7 @@ def maybe_color_bp(bp):

def plot_group(keys, values, ax):
keys = [pprint_thing(x) for x in keys]
values = [remove_na_arraylike(v) for v in values]
values = [np.asarray(remove_na_arraylike(v)) for v in values]
bp = ax.boxplot(values, **kwds)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tacaswell here, these are Series (in a particular case), converting to ndarray makes this work.

@jreback jreback merged commit 6ff8434 into pandas-dev:master Oct 10, 2017
@tacaswell
Copy link
Contributor

attn @phobson

ghost pushed a commit to reef-technologies/pandas that referenced this pull request Oct 16, 2017
alanbato pushed a commit to alanbato/pandas that referenced this pull request Nov 10, 2017
No-Stream pushed a commit to No-Stream/pandas that referenced this pull request Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite Visualization plotting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants