-
Notifications
You must be signed in to change notification settings - Fork 3
Impliment BusinessWindowIndexer for non-fixed offsets #24
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
Impliment BusinessWindowIndexer for non-fixed offsets #24
Conversation
@@ -395,6 +395,8 @@ def _get_window_indexer(self, index_as_array): | |||
------- | |||
VariableWindowIndexer or FixedWindowIndexer | |||
""" | |||
if isinstance(self.window, BaseIndexer): |
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.
so we ignore the arg when using a custom indexer? or should it still be passed (and can be ignored if needed inside the custom indexer?
imagine writing VariableIWindowIndexer and FixedWindowIndwxer as a BaseIndexer (which i think we should do at some point)
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.
All custom indexers should subclass BaseIndexer
so self.window
should be the custom indexer which will accept arguments from mean (_apply
).
VariableWindowIndexer and FixedWindowIndexer already subclass BaseIndexer
…ling_business_day
Question, @DiegoAlbertoTorres and @jreback, on the definition of a rolling business day. Given the example below:
Are values that fall on non-business days aggregated in the window? (Currently they are)
|
Will document this in the POC doc after this PR. |
Closing this example. This example requires a non trivial reworking of the internals for a feature that can be done today. |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff