-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
ENH: Add online operations for EWM.mean #41888
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 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
e195c58
Add scaffolding for online EWM
mroeschke 3d95167
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 9354bd0
Add online op and new methods and class
mroeschke 0ce197d
Make signatures match
mroeschke 8096cc6
Add some tests, rename some variables
mroeschke a5273b9
Add newline for readability
mroeschke bab78cc
Parameterize over adjust and ignore_na
mroeschke d72a03e
Test resetting in tests
mroeschke 0b7e773
Add test with invalid update
mroeschke 8444b42
Add docstring for mean
mroeschke 7847373
Add docstring for online
mroeschke df13b55
Parameterize over dataframe and series
mroeschke 57db06e
Generalize axis call for update_times
mroeschke 329dbd2
Remove comments
mroeschke 9594afe
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 28be18a
Add more test and ensure constructions
mroeschke 85025ff
Passing all the non-time tests
mroeschke 3345271
Add whatsnew and window.rst; xfail update_times
mroeschke 2186ea0
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 8024a7b
mypy
mroeschke 80c8b7f
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 8a5b0b9
Address comments
mroeschke e790947
Fix doctest
mroeschke 916e68b
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 175c4ca
Fix doctest
mroeschke f799a0f
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke c8b09b6
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 2cb4019
Cannot parallelize a loop
mroeschke fea8b0b
Trigger CI
mroeschke 04ea064
Merge remote-tracking branch 'upstream/master' into online/ewm
mroeschke 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
Add online op and new methods and class
- Loading branch information
commit 9354bd0d261580c02459f8ed71f641d2c227c116
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.
what is this?
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.
A user needs to call
mean()
first then can callmean(update=new_df)
This checks that
mean()
was called first.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.
ok, and test for this? (with good error message)
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.
Yes here: https://github.com/pandas-dev/pandas/pull/41888/files#diff-3d423ab20ad1177b0c1676fd48c54344c6fd3150ab6d4618301cd79b2942c21dR16