-
-
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
ENH: removed median percentile to be always included in describe GH #60550 #60557
base: main
Are you sure you want to change the base?
Conversation
- fixes pandas-dev#60550 - median percentile is default when a blank list of percentiles is passed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks could you add a test for this?
@mroeschke I don't think a separate test is needed as the core functionality remains the same. |
We should have had a test that when you specify Since we're changing the behavior, I think a test needs to be added. |
@mroeschke @rhshadrach thanks for the feedback, the tests are now added. |
.describe(percentiles = [0.25])
returns 25th- and 50th-percentile #60550pandas/tests/reductions/test_describe_ndframe.py
Added type annotations to new arguments/methods/functions.NAdoc/source/whatsnew/v2.3.0.rst
file if fixing a bug or adding a new feature.