-
Notifications
You must be signed in to change notification settings - Fork 23
Open
Description
Random.int contains a sanitization step at:
Lines 81 to 85 in c1c9da4
| ( lo, hi ) = | |
| if a < b then | |
| ( a, b ) | |
| else | |
| ( b, a ) |
As such, Random.int 100 -100 produces values in the range [-100, 100]
Random.float contains no such step, and Random.float 100 -100 produces values in the range [100, 300] due to this, which assumes the bounds were passed in the correct order:
Lines 180 to 182 in c1c9da4
| -- Scale it into our range | |
| range = | |
| abs (b - a) |
Either the same sanitization step should be added to Random.float or at the very least a prominent warning should be added to the documentation.
Metadata
Metadata
Assignees
Labels
No labels