Skip to content

FIX/TST: update tests for pandas 0.21 #9726

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.

Sign up for GitHub

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

Jump to bottom
Merged
story645 merged 1 commit into from
Nov 10, 2017
Merged

FIX/TST: update tests for pandas 0.21 #9726

story645 merged 1 commit into from
Nov 10, 2017

Conversation

@jklymak
Copy link
Member

@jklymak jklymak commented Nov 9, 2017 •
edited
Loading

This should fix #9610.

As of pandas 0.21, import pandas no longer calls

from pandas.tseries import converter
converter.register()

so it has been added to the tests.

Surprisingly no examples depend on pandas, so that made life simpler.

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant

@jklymak jklymak added this to the v2.1.1 milestone Nov 9, 2017
@jklymak jklymak added the Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. label Nov 9, 2017
dstansby
dstansby reviewed Nov 9, 2017
View reviewed changes
lib/matplotlib/tests/test_axes.py
pd = pytest.importorskip('pandas')
from pandas.tseries import converter
converter.register()

Copy link
Member

@dstansby dstansby Nov 9, 2017

Choose a reason for hiding this comment

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

PEP8 doesn't like this blank line, otherwise tests seem to be fine.

@jklymak jklymak mentioned this pull request Nov 9, 2017
Closed
@story645
Copy link
Member

story645 commented Nov 9, 2017 •
edited
Loading

so unclear if this code actually reduces coverage or if it's an artifact of a different PR.

@dstansby
Copy link
Member

dstansby commented Nov 9, 2017

Sometimes I feel like I need an undergraduate degree in codecov just to understand codecov...

I think the issue is that there are two tests both named test_pandas_bar_align_center() so only one is collected and run. @jklymak would you mind quickly changing the name of one of them here?

@jklymak
Copy link
Member Author

jklymak commented Nov 9, 2017

Huh, did I do that? Anyhow, should be fixed now barring other foolishness

dstansby
dstansby approved these changes Nov 9, 2017
View reviewed changes
Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

👍 happy for this to go in once tests pass - at least now we can start testing on latest version of pandas again whilst discussing what to do about unit support

@story645 story645 merged commit 91e4bee into matplotlib:master Nov 10, 2017
@tacaswell
Copy link
Member

tacaswell commented Nov 10, 2017

@meeseeksdev backport to v2.1.x

@lumberbot-app
Copy link

lumberbot-app bot commented Nov 10, 2017

There seem to be a conflict, please backport manually

tacaswell pushed a commit that referenced this pull request Nov 10, 2017
@story645 @tacaswell
Merge pull request #9726 from jklymak/fixpandas
ce45bea 
FIX/TST: update tests for pandas 0.21

Conflicts:
    .appveyor.yml
       conflicts due thrashing around pandas version pinning
@tacaswell
Copy link
Member

tacaswell commented Nov 10, 2017

manually backported via ce45bea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dstansby dstansby dstansby approved these changes

Assignees

No one assigned

Labels

Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. topic: testing

Projects

None yet

Milestone

v2.1.1

Development

Successfully merging this pull request may close these issues.

provide converters for datetime64 types

4 participants

@jklymak @story645 @dstansby @tacaswell