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

Change to calculation of compile time percent increase/decrease #21

Closed
Mark-Simulacrum opened this issue Aug 3, 2016 · 1 comment
Closed

Comments

@Mark-Simulacrum
Copy link
Member

Currently, ((previous - current) / current) * 100.0 is the calculation used to derive the percent change between the previous median time and the current median time for the summary page. This page on Wikipedia implies that the formula should instead be ((current - previous) / previous) * 100.0.

Wanted to open this for discussion since this change the results in my current branch from 11.4% increase in compile times to a 9.4% decrease.

@Mark-Simulacrum Mark-Simulacrum changed the title Percentage change calculation change Change to calculation of compile time percent increase/decrease Aug 3, 2016
@nrc
Copy link
Member

nrc commented Aug 3, 2016

I think I agree and we should do the latter, but it is not completely clear to me what current and previous are, so to be clear: ((week_n - week_n-1 / week_n-1) * 100.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants