You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't think PR #405 achieved the intended result. It was intended to make the maximum value occur more often, but instead, for larger values, it does not occur at all anymore. Large numbers are 1% too small, getting worse the larger the range is.
With commit 0371468, the parent of PR #405, we see this behaviour:
I picked the 4 because that is used in the PR, and the Int8 is from the Haddock for Hedgehog, with the Haddock still to this day noting this unsurprising result of (-128,127).
WIth commit 06eb474, that is the merge of the PR, this becomes:
Before PR #405, the code used to multiply by 99 and divide by 99.
The new code adds a constant, multiplies by 99 and divides by 100. But there is no constant additive value that can compensate for the larger division; as values get larger the division will dominate more and more. If an addition is wanted, it will need to be proportional to the values involved.
The text was updated successfully, but these errors were encountered:
I don't think PR #405 achieved the intended result. It was intended to make the maximum value occur more often, but instead, for larger values, it does not occur at all anymore. Large numbers are 1% too small, getting worse the larger the range is.
With commit 0371468, the parent of PR #405, we see this behaviour:
I picked the 4 because that is used in the PR, and the
Int8
is from the Haddock for Hedgehog, with the Haddock still to this day noting this unsurprising result of(-128,127)
.WIth commit 06eb474, that is the merge of the PR, this becomes:
Before PR #405, the code used to multiply by 99 and divide by 99.
The new code adds a constant, multiplies by 99 and divides by 100. But there is no constant additive value that can compensate for the larger division; as values get larger the division will dominate more and more. If an addition is wanted, it will need to be proportional to the values involved.
The text was updated successfully, but these errors were encountered: