Closed Zac-HD closed 2 years ago
Brilliant work, thanks @Zac-HD for taking my PR and making it better!
@crflynn - anything I can do to help get this merged?
@crflynn - ping, it's been a while but I'd still like to get this merged so that I can understand my download numbers. Anything I can do to help, or can you merge it as-is?
Closing this just to get it off my stack of open PRs 😕
Sad :(
Fixes #21 and closes #25 (with thanks to @scottgigante). The two differences to #25 are that
Instead of centering the smoothing window, we use the trailing seven days. This means that the effect of the shorter window does not impact the most recent days of data, which is where maintainers tend to be most concerned - otherwise there would be a big dip on weekends when the window was Thurs-Sun and not yet brought back up by Mon-Tues.
The proportional calculations have been refactored from "average of proportions" to "proportion of averages" - which is much less impacted by random variations, and the obvious interpretation when the plain averages are visible in the previous chart.
@crflynn - I'm keen to get this merged soon, is there anything I can do to help that?