-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
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
Comments
hmm it i don’t think is a problem |
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 |
Because we have always supported this? |
i disagree it doesn’t matter if it’s easy |
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 |
I don’t think it’ll be hard. Similar to the np deprecation.
… On Jan 10, 2020, at 04:57, Jeff Reback ***@***.***> wrote:
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
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
* Compat for util.testing import Closes #30869
0.25.3
1.0
The text was updated successfully, but these errors were encountered: