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
dsaxton wants to merge 30 commits into from
Closed

Implement some reductions for string Series #31757

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

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
Sort index in test
  • Loading branch information
Daniel Saxton authored and Daniel Saxton committed Feb 14, 2020
commit 0c1a2a354fd2dc12f08dae17a905f9bf960b8557
4 changes: 2 additions & 2 deletions pandas/tests/frame/test_apply.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1406,8 +1406,8 @@ def test_apply_datetime_tz_issue(self):
@pytest.mark.parametrize("method", ["min", "max", "sum"])
def test_consistency_of_aggregates_of_columns_with_missing_values(self, df, method):
# GH 16832
none_in_first_column_result = getattr(df[["A", "B"]], method)()
none_in_second_column_result = getattr(df[["B", "A"]], method)()
none_in_first_column_result = getattr(df[["A", "B"]], method)().sort_index()
none_in_second_column_result = getattr(df[["B", "A"]], method)().sort_index()
Copy link
Contributor Author

@dsaxton dsaxton Feb 14, 2020

Choose a reason for hiding this comment

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

Previously the column with the missing value was getting dropped from the result so it only had a single row and the order didn't matter


tm.assert_series_equal(
none_in_first_column_result, none_in_second_column_result
Expand Down