-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: fixing ewma() for adjust=False and ignore_na=False #7896
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
Conversation
side issue can you add a couple of examples in the doc-string of |
Happy to give it a shot, though to be honest I'm not quite sure what to put there. Can you point me to some other doc strings that have such examples? I can add more verbose notes about |
(just showing some examples, these are for users to copy-paste). if it doesn't add then its ok. but a bit more docs are always better. |
I don't think examples will add much, as it's just setting |
I've updated |
Actually, let me make a slight edit... |
that looks good. does this need to also be on the other (you can simply put this in a variable and use Appender to propgate this) |
I put it in |
OK, I'm done editing. |
The Travis CI build passed. |
BUG: fixing ewma() for adjust=False and ignore_na=False
The previous code in #7603 wasn't quite right for the case that both
adjust=False
andignore_na=False
. This PR fixes the handling of that case. Note thatignore_na=False
behavior is new in v0.15.0, so this bug and its fix don't affect any prior behavior.