-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
PERF: Use Indexers to implement groupby rolling #34052
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
jreback
merged 40 commits into
pandas-dev:master
from
mroeschke:rolling_groupby_indexer
May 25, 2020
Merged
Changes from 16 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
723d7e2
Prep rolling groupby
47a53d6
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
efc11a7
figure out how to get a similar groupby result
db9e134
Wrap result in list
e106b69
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
c5a2ab0
add custom code to produce resulting index
3d6468a
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
7856eac
Use _get_window_indexer instead of changing window in __init__
bc7a8c8
Remove unused import
462c7e5
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
67f938e
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
4f698ec
Create groupby_parent for groupbyrolling
741d79e
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
d77a96b
remove grouper parent and user the grouper object to get the key names
6b86936
remove comma
bfd485e
Don't dispatch to WindowGroupbyMixin._apply
e1eee66
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
8a680b1
Use variable algorithms only for groupby rolling
88b0b25
Add todo comments
0f45cd1
Flake8
a8c8d8d
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
32d4c49
rename some variables and handle timeseries case
9d886c3
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
6605f53
Fix non-monotonic case
1978098
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
dc8da47
Lint and fix windows dtype issue
e22eeba
Add asv benchmark
fd884c1
Add whatsnew
8ff7ab8
Fix more typing errors
ddddf69
Fix last of typing
d924f70
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
08abad3
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
7390a62
Add timeseries benchmark
41f8569
Improve whatsnew sphinx link
5517cc4
reuse _create_blocks
a62dba4
expand docstring
b64a321
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
b7b992b
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
dad3d0e
Add more commentary, typing
93772c0
Merge remote-tracking branch 'upstream/master' into rolling_groupby_i…
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
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.
can you add some comments here on what you are doing