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

VampGrowth: Rewrite calculation to avoid exponentiation #2

Closed
mooeypoo opened this Issue Aug 9, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@mooeypoo
Owner

mooeypoo commented Aug 9, 2018

Initially, the calculation dealt with long-term values, so using the exponential -- e^(growthFactor * time) made sense. However, "Buffy culling factor" was then introduced, and the calculator was set up to calculate per day which means that the exponentiation is not just not necessary, it gives wrong values at the end.

This should be fixed; since the calculation iterates over every day anyways, we can just use direct addition per day (which is how the original exponential was created to begin with)

@mooeypoo mooeypoo added the bug label Aug 9, 2018

@mooeypoo mooeypoo closed this in 804726e Aug 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment