Skip to content

Round new_rating instead of flooring#12

Open
kevinreedy wants to merge 2 commits intoiain:masterfrom
kevinreedy:rounding-fix
Open

Round new_rating instead of flooring#12
kevinreedy wants to merge 2 commits intoiain:masterfrom
kevinreedy:rounding-fix

Conversation

@kevinreedy
Copy link

Using .to_i here will floor the result. Using .round over time will be more accurate. There are other PRs that change the output of the library to be a float, which works as well; however, there may be applications using this library expecting an integer.

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

Successfully merging this pull request may close these issues.

1 participant