Skip to content

pandas.util.testing no longer imported with pandas #30869

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

Closed
TomAugspurger opened this issue Jan 9, 2020 · 8 comments · Fixed by #30973
Closed

pandas.util.testing no longer imported with pandas #30869

TomAugspurger opened this issue Jan 9, 2020 · 8 comments · Fixed by #30973
Labels
Deprecate Functionality to remove in pandas
Milestone

Comments

@TomAugspurger
Copy link
Contributor

0.25.3

In [1]: import pandas

In [2]: pandas.util.testing
Out[2]: <module 'pandas.util.testing' from '/private/tmp/pandas/util/testing.py'>

1.0

In [1]: import pandas

In [2]: pandas.util.testing
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-2-968e027d039f> in <module>
----> 1 pandas.util.testing

AttributeError: module 'pandas.util' has no attribute 'testing'
@TomAugspurger TomAugspurger added this to the 1.0.0 milestone Jan 9, 2020
@TomAugspurger TomAugspurger added the Deprecate Functionality to remove in pandas label Jan 9, 2020
@jreback
Copy link
Contributor

jreback commented Jan 9, 2020

hmm it i don’t think is a problem
and wouldn’t do anything about it

@jorisvandenbossche
Copy link
Member

It's perfect legitimate usage pattern of util.testing, and it's also easy to fix (we can still import it while hiding the warning)

@jreback
Copy link
Contributor

jreback commented Jan 10, 2020

It's perfect legitimate usage pattern of util.testing, and it's also easy to fix (we can still import it while hiding the warning)

missing the point - why should be supporting a non explicit import? i would simply break this

@jorisvandenbossche
Copy link
Member

why should be supporting a non explicit import?

Because we have always supported this?

@jreback
Copy link
Contributor

jreback commented Jan 10, 2020

why should be supporting a non explicit import?

Because we have always supported this?

i disagree

it doesn’t matter if it’s easy

@jorisvandenbossche
Copy link
Member

Because we have always supported this?

i disagree

I am assuming that your "I disagree" is about this being a good reason to keep supporting it? Because AFAIK, it's a fact that we have always supported this.

@jreback
Copy link
Contributor

jreback commented Jan 10, 2020

Because we have always supported this?

i disagree

I am assuming that your "I disagree" is about this being a good reason to keep supporting it? Because AFAIK, it's a fact that we have always supported this.

my point is that @TomAugspurger did enough on the current warning for an explicit import

an implicit import is not a reason to even have a depreciation

so -1 for fixing this and a hard break is fine

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Jan 10, 2020 via email

TomAugspurger added a commit to TomAugspurger/pandas that referenced this issue Jan 13, 2020
TomAugspurger added a commit that referenced this issue Jan 13, 2020
* Compat for util.testing import

Closes #30869
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants