-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: update the pandas.DataFrame.clip docstring #20212
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
Changes from 23 commits
2448809
aaeb2dc
62afcab
33af7ce
cb8d7ab
e062a88
87f8b5b
1371a23
674ffb6
495847d
89a7690
131c052
089e882
1089460
60c34db
9aeda8e
75eecd1
504389f
694f9fb
af991cc
141e8c9
383c56d
3f1c785
90d8268
bd26d0f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5601,53 +5601,98 @@ def clip(self, lower=None, upper=None, axis=None, inplace=False, | |
""" | ||
Trim values at input threshold(s). | ||
|
||
Elements above/below the upper/lower thresholds will be changed to | ||
upper/lower thresholds. Clipping data is a method for dealing with | ||
out-of-range elements. If some elements are too large or too small, | ||
clipping is one way to transform the data into a reasonable range. | ||
|
||
Parameters | ||
---------- | ||
lower : float or array_like, default None | ||
upper : float or array_like, default None | ||
axis : int or string axis name, optional | ||
Align object with lower and upper along the given axis. | ||
lower : float, array-like or None, default None | ||
Lower threshold for clipping. Values smaller than `lower` will be | ||
converted to `lower`. | ||
upper : float, array-like or None, default None | ||
Upper threshold for clipping. Values larger than `upper` will be | ||
converted to `upper`. | ||
axis : {0 or 'index', 1 or 'columns', None}, default None | ||
Apply clip by index (i.e. by rows) or columns. | ||
inplace : boolean, default False | ||
Whether to perform the operation in place on the data | ||
.. versionadded:: 0.21.0 | ||
.. versionadded:: 0.21.0. | ||
*args, **kwargs | ||
Additional keywords have no effect but might be accepted | ||
for compatibility with numpy. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It was some change to the convention during the sprint, and args and kwargs should be in the same line Also, note that |
||
|
||
Returns | ||
------- | ||
clipped : Series | ||
`Series` or `DataFrame`. | ||
Original input with those values above/below the | ||
`upper`/`lower` thresholds set to the threshold values. | ||
|
||
See Also | ||
-------- | ||
DataFrame.clip : Trim values at input threshold(s). | ||
Series.clip : Trim values at input threshold(s). | ||
Series.clip_lower : Return copy of the input with values below given | ||
value(s) truncated. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. would just mention Series.clip here (not upper/lower) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why don't you find There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. going to deprecate lower/upper but that’s not the reason just a preference to not have too many things in See Also There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes sense to me. But if I'm not wrong, the title is misleading but this docstring is for the Series.clip method too. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ahh ok in that case this is ok |
||
Series.clip_upper : Return copy of input with values above given | ||
value(s) truncated. | ||
DataFrame.clip_lower : Return copy of the input with values below given | ||
value(s) truncated. | ||
DataFrame.clip_upper : Return copy of input with values above given | ||
value(s) truncated. | ||
DataFrame.quantile : Return values at the given quantile over requested | ||
axis, a la numpy.percentile. | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the prefix There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think you previously mentioned that those were generic. Did we want them included still? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry, I think I wasn't clear enough. You should check it, but as this docstring is in So, as your assigned docstring was So, summarizing, the see also should contain @jreback do you agree? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, yes I see, I was mistaken. Sure, I can add all those. |
||
Examples | ||
-------- | ||
>>> df = pd.DataFrame({'a': [-1, -2, -100], | ||
... 'b': [1, 2, 100]}, | ||
... index=['foo', 'bar', 'foobar']) | ||
>>> df | ||
0 1 | ||
0 0.335232 -1.256177 | ||
1 -1.367855 0.746646 | ||
2 0.027753 -1.176076 | ||
3 0.230930 -0.679613 | ||
4 1.261967 0.570967 | ||
|
||
>>> df.clip(-1.0, 0.5) | ||
0 1 | ||
0 0.335232 -1.000000 | ||
1 -1.000000 0.500000 | ||
2 0.027753 -1.000000 | ||
3 0.230930 -0.679613 | ||
4 0.500000 0.500000 | ||
|
||
>>> t | ||
0 -0.3 | ||
1 -0.2 | ||
2 -0.1 | ||
3 0.0 | ||
4 0.1 | ||
dtype: float64 | ||
|
||
>>> df.clip(t, t + 1, axis=0) | ||
0 1 | ||
0 0.335232 -0.300000 | ||
1 -0.200000 0.746646 | ||
2 0.027753 -0.100000 | ||
3 0.230930 0.000000 | ||
4 1.100000 0.570967 | ||
a b | ||
foo -1 1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. alignment doesn’t look right here |
||
bar -2 2 | ||
foobar -100 100 | ||
|
||
>>> df.clip(lower=-10, upper=10) | ||
a b | ||
foo -1 1 | ||
bar -2 2 | ||
foobar -10 10 | ||
|
||
You can clip each column or row with different thresholds by passing | ||
a ``Series`` to the lower/upper argument. Use the axis argument to clip | ||
by column or rows. | ||
|
||
>>> col_thresh = pd.Series({'a': -5, 'b': 5}) | ||
>>> df.clip(lower=col_thresh, axis='columns') | ||
a b | ||
foo -1 5 | ||
bar -2 5 | ||
foobar -5 100 | ||
|
||
Clip the foo, bar, and foobar rows with lower thresholds 5, 7, and 10. | ||
|
||
>>> row_thresh = pd.Series({'foo': 0, 'bar': 1, 'foobar': 10}) | ||
>>> df.clip(lower=row_thresh, axis='index') | ||
a b | ||
foo 0 1 | ||
bar 1 2 | ||
foobar 10 100 | ||
|
||
`Winsorizing <https://en.wikipedia.org/wiki/Winsorizing>`__ is a | ||
related method, whereby the data are clipped at | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. put this reference in the Notes section There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do you want me to move the entire part about winsorization or just the link? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just the link, you can see the format in the |
||
the 5th and 95th percentiles. The ``DataFrame.quantile`` method returns | ||
a ``Series`` with column names as index and the quantiles as values. | ||
Use ``axis='columns'`` to apply clipping to columns. | ||
|
||
>>> lower, upper = df.quantile(0.05), df.quantile(0.95) | ||
>>> df.clip(lower=lower, upper=upper, axis='columns') | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure if it's just because I'm too tired, but is There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Axis is required here since There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd find useful having this comment in the explanations before the test. :) |
||
a b | ||
foo -1.1 1.1 | ||
bar -2.0 2.0 | ||
foobar -90.2 90.2 | ||
""" | ||
if isinstance(self, ABCPanel): | ||
raise NotImplementedError("clip is not supported yet for panels") | ||
|
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.
I find the description of axis a bit complex.