-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
TST: suppress deprecation warnings for compress #22064
TST: suppress deprecation warnings for compress #22064
Conversation
Copying from the other PR, how about we update the deprecation message to
We may also need something that Series.compress and np.compress(series, condition) are deprecated.
Use series[condition] or np.asarray(series).compress(condition) instead. |
can you rebase |
@TomAugspurger edited the deprecation message |
Codecov Report
@@ Coverage Diff @@
## master #22064 +/- ##
==========================================
- Coverage 92.08% 92.05% -0.04%
==========================================
Files 169 169
Lines 50691 50709 +18
==========================================
- Hits 46681 46679 -2
- Misses 4010 4030 +20
Continue to review full report at Codecov.
|
xref #21930 (comment)
In case we find it OK
np.compress
with Series is also deprecated, this PR makes sure the warnings are catched in the tests.