NextInt(min, max) slight incorrect?

Code (CSharp):

1. for (uint i = 1; i < 10000; i++) {
2.     var random = new Unity.Mathematics.Random();
3.     random.InitState(i);
4.     UnityEngine.Debug.Log($"Two ranges: {random.NextInt(0, 1)}, {random.NextInt(0, 1)}");
5. }

This will just spit out "0, 0" everytime. If if make the max 2 instead of 1, I will start to see some 1s as well as 0s. It seems like the max is non-inclusive, which the parameter name does not suggest.

 

Returns a uniformly random int value in the interval [min, max).
i.e. min <= result < max, and with range 0..1, only 0 is valid result.

 

While we're here:
> Returns a uniformly random double value in the interval [0, 1).
from https://docs.unity3d.com/Packages/c...ndom.html#Unity_Mathematics_Random_NextDouble
Seems like it should be "[0, 1]" after doing some wikipedia reading on intervals to make sure I was clear on the terminology.

Ultimately though I think the parameter name (the most visible documentation currently) of "max" is misleading, though it is handy because I don't have to subtract one from my array length.

 

This is the exact same behavior the existing UnityEngine.Random class displays.

 

And same behavior in standard C# Random class. (Next overload with min and max, non-inclusive max).

 

I have agree that is correct and always was done in that way but also agree that parameter name always was misleading and need to be renamed to something like maxExclusive so we can understand method logic from signature instead of from documentation.