-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: Cleanup tests from numeric_only / nuisance deprecation #50133
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
take |
Hello rhshadrach, this will be my first contribution to pandas. Do you have any example of test/file I can start with ? Maybe the work can be split if there is a lot to be done ? |
@AlexBuzenet - you can find examples by searching the tests for pandas/pandas/tests/extension/base/groupby.py Lines 145 to 147 in 5ee4dac
I'd recommend naming the files test_invalid_op.py (the name may be workshopped) that will go in various subdirectories - e.g. I'll plan to do one example PR in the next day or two in groupby. As for splitting this up, I think that's a good idea but it would require a list of tests. I don't have that at the moment, and it would take some time to create. |
@rhshadrach I just opened a PR extending your work in |
a lot of these exist in tests.groupby.test_raises |
This issue has been silent for a while, I can continue to finish it. |
@luke396 - that's be great. I've lost track of where things stand here. |
In enforcing the numeric_only / silent dropping of nuisance columns deprecations for 2.0, many of our tests now have the following pattern:
I think it would be better to consolidate these into a small group of tests, something like
test_op_raises_on_invalid_type
, and remove the invalid types from the remainder of the tests.cc @mroeschke
The text was updated successfully, but these errors were encountered: