You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Luc Maisonobe (JIRA)" <ji...@apache.org> on 2013/03/10 20:31:12 UTC

[jira] [Resolved] (MATH-936) RandomDataGenerator#nextLong violates bounds

     [ https://issues.apache.org/jira/browse/MATH-936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luc Maisonobe resolved MATH-936.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 3.2

Fixed in subversion repository as of r1454897.

We now use discrete raw elements to build the int/long rather than relying on floating point arithmetic.

Thanks for the report.
                
> RandomDataGenerator#nextLong violates bounds
> --------------------------------------------
>
>                 Key: MATH-936
>                 URL: https://issues.apache.org/jira/browse/MATH-936
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 3.1
>            Reporter: Ralf Wiebicke
>              Labels: random
>             Fix For: 3.2
>
>         Attachments: MATH-936.patch, RandomGeneratorLongTest.java
>
>
> I attached a test.
> If the underlying RandomGenerator returns 0.0, then nextLong returns Long.MIN_VALUE, although the lower bound is Long.MIN_VALUE+1.
> The javadoc of RandomGenerator#nextDouble does not clearly define, whether the result includes the lower border of 0.0 or not.
> In java.util.Random it clearly defined as included: "uniformly from the range 0.0d (inclusive) to 1.0d (exclusive)". And the existence of JDKRandomGenerator suggests, that RandomGenerator should have the same contract.
> I tested with version 3.1.1 from mvnrepository

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira