Skip to content

Implement some reductions for string Series #31757

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

Closed
wants to merge 30 commits into from
Closed

Implement some reductions for string Series #31757

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
c87ce47
Add reduction tests
Feb 6, 2020
5365fa1
Implement a few reductions
Feb 6, 2020
460030f
Fix test
Feb 6, 2020
6497e16
Add whatsnew
Feb 6, 2020
c052feb
skipna
Feb 6, 2020
4312b4f
Hardcode expected output
Feb 11, 2020
b67cc56
Merge branch 'master' into reduce-string
Feb 11, 2020
214995f
Skip tests
Feb 12, 2020
95c83fb
Merge branch 'master' into reduce-string
Feb 13, 2020
4082ae3
No else
Feb 13, 2020
12a2323
Modify nanminmax to better handle object dtype
Feb 14, 2020
4a61b81
Delegate to min and max methods
Feb 14, 2020
c6465e5
Fixup nanops
Feb 14, 2020
5c1a5fb
Import and clean up
Feb 14, 2020
ca99650
Update tests
Feb 14, 2020
18800e1
Update whatsnew
Feb 14, 2020
3eddb73
Limit to values.ndim == 1
Feb 14, 2020
0c1a2a3
Sort index in test
Feb 14, 2020
5714c06
Box in Series
Feb 14, 2020
5e01c3f
Add Series min / max test
Feb 14, 2020
0e4a4e2
Merge branch 'master' into reduce-string
Feb 14, 2020
ed01138
Move tests
Feb 14, 2020
1128950
Pass in mask and patch _maybe_get_mask
Feb 15, 2020
7c0f05d
Update whatsnew
Feb 15, 2020
d84f5bd
Merge branch 'master' into reduce-string
Feb 15, 2020
7db2052
Try to make mypy happy
Feb 15, 2020
e19bbe1
Update doc/source/whatsnew/v1.0.2.rst
dsaxton Feb 15, 2020
df99b4f
Add comment
Feb 15, 2020
ba20705
Merge branch 'master' into reduce-string
Feb 15, 2020
d69b587
Merge remote-tracking branch 'upstream/master' into reduce-string
dsaxton Feb 25, 2020
File filter

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
Move tests
  • Loading branch information
Daniel Saxton authored and Daniel Saxton committed Feb 14, 2020
commit ed01138bef57e03d0bda3aea5d83a85bec41dd7f
6 changes: 0 additions & 6 deletions pandas/tests/extension/base/reduce.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,6 @@ class BaseNoReduceTests(BaseReduceTests):

@pytest.mark.parametrize("skipna", [True, False])
def test_reduce_series_numeric(self, data, all_numeric_reductions, skipna):
if isinstance(data, pd.arrays.StringArray) and all_numeric_reductions in [
"min",
"max",
]:
pytest.skip("These reductions are implemented")

op_name = all_numeric_reductions
s = pd.Series(data)

Expand Down
14 changes: 13 additions & 1 deletion pandas/tests/extension/test_string.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,19 @@ class TestMissing(base.BaseMissingTests):


class TestNoReduce(base.BaseNoReduceTests):
pass
@pytest.mark.parametrize("skipna", [True, False])
def test_reduce_series_numeric(self, data, all_numeric_reductions, skipna):
if isinstance(data, pd.arrays.StringArray) and all_numeric_reductions in [
"min",
"max",
]:
pytest.skip("These reductions are implemented")
Copy link
Member

@jorisvandenbossche jorisvandenbossche Feb 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add here a comment saying that those are tested in tests/arrays/test_string.py ?

op_name = all_numeric_reductions
s = pd.Series(data)

with pytest.raises(TypeError):
getattr(s, op_name)(skipna=skipna)


class TestMethods(base.BaseMethodsTests):
Expand Down