Skip to content
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 d3.movingAverage as a native d3-array function #176

Closed
rdmurphy opened this issue Sep 25, 2020 · 1 comment
Closed

Add d3.movingAverage as a native d3-array function #176

rdmurphy opened this issue Sep 25, 2020 · 1 comment

Comments

@rdmurphy
Copy link

rdmurphy commented Sep 25, 2020

I've borrowed the moving average function found in this notebook a few times now, and having just done it again 😁 it made me wonder if it belongs in d3-array proper? I feel like it has a similar vibe to d3.cumsum.

The only pause I have is whether there is an issue with its definition of what constitutes the "moving average." I think I know how to modify to be center-based ((N - 1) / 2 values on either side of V vs. V and N - 1 previous values), but don't know if alternative definitions would/should be on the d3-array namespace under different names or as options to pass to a d3.movingAverage itself.

Either way, thank you!

@Fil
Copy link
Member

Fil commented Sep 25, 2020

this or d3.blur would be great yes.

Closing as a duplicate of #56 (in particular #56 (comment)) & #151

@Fil Fil closed this as completed Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants