-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add a rolling mean filter #454
Comments
hi @niksirbi i think i can work on this issue. could you please assign it to me? |
@niksirbi I would like to work on it! |
Hi, I’d like to work on this issue. Could you assign it to me? Thanks! |
Thanks all for expressing interest in solving this issue. Note that there is no need to be assigned to an issue to start working on it. Given that there are already two PRs proposing alternative solutions to this issue (#455 and #459), I would refrain from further duplicating effort on this one. |
We've had a rolling median filter for quite some time now, but we've neglected to also add a rolling mean filter that works the same way.
@stellaprins had a need for a rolling mean filter while working on #429, and ended up implementing it using
np.convolve
.It should be simple to either:
mean_filter
that works the same asmedian_filter
Once we implement this, we should also edit the eye movement example introduced by #429 to use our new shiny filter function.
The text was updated successfully, but these errors were encountered: