8000 REF: implement _wrap_reduction_result by jbrockmendel · Pull Request #37660 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

REF: implement _wrap_reduction_result #37660

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.

Sign up for GitHub

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

Merged
merged 18 commits into from
Nov 8, 2020
Merged

REF: implement _wrap_reduction_result #37660

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
65fe4e4
REF: avoid special case in DTA/TDA.median, flesh out tests
jbrockmendel Oct 26, 2020
178f3e3
Merge branch 'master' of https://github.com/pandas-dev/pandas into re…
jbrockmendel Oct 26, 2020
c52b0bc
suppress warning
jbrockmendel Oct 26, 2020
434692a
checkpoint tests passing
jbrockmendel Oct 24, 2020
bdad92c
checkpoint tests passing
jbrockmendel Oct 25, 2020
5bbc5d1
follow-up clenaup
jbrockmendel Oct 26, 2020
a69f762
cln: no need to pass mask
jbrockmendel Oct 26, 2020
121108e
unnecessary mask arg
jbrockmendel Oct 26, 2020
d389884
lint fixup
jbrockmendel Oct 26, 2020
39afdfd
REF: avoid special-casing inside DTA/TDA.mean (#37422)
jbrockmendel Oct 26, 2020
857572d
TST/REF: misplaced tests in frame.test_dtypes (#37424)
jbrockmendel Oct 26, 2020
297b278
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Oct 30, 2020
a797447
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Oct 30, 2020
710fe30
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Oct 31, 2020
1d52aae
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Oct 31, 2020
3bca50c
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Nov 1, 2020
21fe9b9
Merge branch 'master' of https://github.com/pandas-dev/pandas into cl…
jbrockmendel Nov 6, 2020
198c4de
REF: implement _wrap_reduction_result
jbrockmendel Nov 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
suppress warning
  • Loading branch information
@jbrockmendel
jbrockmendel committed Oct 26, 2020
commit c52b0bce6f4f38aa1e1eccd1d6142ee1cb8f526b
13 changes: 11 additions & 2 deletions pandas/core/nanops.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
import itertools
import operator
from typing import Any, Optional, Tuple, Union, cast
import warnings

import numpy as np

Expand Down Expand Up @@ -643,7 +644,11 @@ def get_median(x):
mask = notna(x)
if not skipna and not mask.all():
return np.nan
return np.nanmedian(x[mask])
with warnings.catch_warnings():
# Suppress RuntimeWarning about All-NaN slice
warnings.filterwarnings("ignore", "All-NaN slice encountered")
res = np.nanmedian(x[mask])
return res

values, mask, dtype, _, _ = _get_values(values, skipna, mask=mask)
if not is_float_dtype(values.dtype):
Expand Down Expand Up @@ -671,7 +676,11 @@ def get_median(x):
)

# fastpath for the skipna case
return _wrap_results(np.nanmedian(values, axis), dtype)
with warnings.catch_warnings():
# Suppress RuntimeWarning about All-NaN slice
warnings.filterwarnings("ignore", "All-NaN slice encountered")
res = np.nanmedian(values, axis)
return _wrap_results(res, dtype)

# must return the correct shape, but median is not defined for the
# empty set so return nans of shape "everything but the passed axis"
Expand Down
0