[ 
https://issues.apache.org/jira/browse/MATH-936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13597988#comment-13597988
 ] 

Thomas Neidhart edited comment on MATH-936 at 3/9/13 4:07 PM:
--------------------------------------------------------------

If this can be adapted to uniformly provide numbers in the range of [lower, 
upper], also taking into account that lower can be negative, than this would be 
a better solution.
                
      was (Author: tn):
    If this can be adapted to uniformly provide number in the range of [lower, 
upper], also taking into account that lower can be negative, than this would be 
a better solution.
                  
> 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
>         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

Reply via email to