-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: DataFrame.groupby(., dropna=True, axis=0) incorrectly throws ShapeError #35751
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
Merged
Merged
Changes from 2 commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
0484244
handle dropna=False in _selected_obj, _set_result_index_ordered
arw2019 a335744
add whatsnew
arw2019 640ec38
rewrote _set_result_index_ordered
arw2019 099e30c
added dropna=False to tests reliant on that
arw2019 8a13d06
delete second reindexing in _transform_general
arw2019 394feb6
restore + change test_evaluate_with_empty_groups
arw2019 e1cafd4
remove calls to obj.dropna()
arw2019 0df329c
separate tests for dataframe/series slices
arw2019 77e7fc7
fix series indexing
arw2019 16544ea
fix DataFrameGroupBy._transform_general
arw2019 46e5f66
move DataFrameGroupBy._transform_general logic to _set_result_index_o…
arw2019 6fca785
handle edge case (datetime index, no NaNs)
arw2019 6341d97
merge with upstream/master
arw2019 269516b
move logic to BaseGrouper and _set_index_ordered
arw2019 6819ac6
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 2ec491d
add dropna args to tests
arw2019 249fc2a
add dropna=False arg to test
arw2019 b1cafad
feedback
arw2019 bef1437
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 9abc8c4
revert change to test
arw2019 c63a24c
feedback
arw2019 9791e1e
revert changes to test_transform
arw2019 21a6fbb
revert changes to test
arw2019 8afb6e2
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 239e16a
feedback
arw2019 0cdea22
add non-RangeIndex test cases
arw2019 ee73640
merge with master
arw2019 c07df76
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 ca2f898
add comments/clean up _set_result_index_ordered
arw2019 e15df1a
revert accidental change to _transform_general signature
arw2019 342540f
rewrite _set_result_index_ordered using auxiliary method
arw2019 90e687b
rewrite whatsnew note
arw2019 a10a933
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 bddfa81
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 2adba09
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 531414f
feedback
arw2019 10ee18a
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 2fcfda0
rename variables (possibly) for clarity
arw2019 1969bc4
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 2972ee4
merge with upstream/master
arw2019 62caeb6
simplify _set_result_index_ordered
arw2019 557903f
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 deb1b09
minimize diff
arw2019 3d579c5
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 ec85d7f
merge with master
arw2019 f6a9724
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 bfe6cde
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 b6fd41c
add type hints to _set_result_index_ordered
arw2019 983bb8e
revert type hints
arw2019 1884133
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 f207709
merge master
arw2019 4422a21
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 daa60a6
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 5658c12
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 4326b79
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 e12e8d9
merge with master
arw2019 9e6a130
BUG: fix merge mistake
arw2019 1770cc2
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 6a005dc
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 74dbe4f
Merge branch 'GH35612' of https://github.com/arw2019/pandas into GH35612
arw2019 0e9db9c
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 8be535c
feedback: rewrite _set_result_index_ordered
arw2019 91940c7
Merge branch 'GH35612' of https://github.com/arw2019/pandas into GH35612
arw2019 85d2165
CI: fix pd namespace usage
arw2019 c31b49f
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 21bfc82
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 7f67086
merge with master
arw2019 5555585
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 5ac7fbf
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 1e7ab91
DOC: fix use of pd namespace in whatsnew
arw2019 96b5af4
DOC: fix typo in whatsnew entry
arw2019 9cf9e05
Merge remote-tracking branch 'upstream/master' into GH35612
arw2019 f5a1635
REF (feedback): _set_result_index_ordered
arw2019 bd1abf9
DOC: @rhshadrach fix
arw2019 a789b6a
REF: restore casing on rows_dropped
arw2019 2ba8b44
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 95b86ba
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 7881134
merge with master
arw2019 15aa56e
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 08f0abd
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 8ab9baa
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 faf6570
move whatsnew to 1.3
arw2019 7bd2a9a
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 24bb112
minimize diff
arw2019 4377b63
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 de86144
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 9bc9ce4
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 1ea9d29
Merge branch 'master' of https://github.com/pandas-dev/pandas into GH…
arw2019 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
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
missing values in the grouper? with dropna?
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.
rewrote, mentioned grouper. Also separated this entry from #35014