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

Dennis Hendriks commented on MATH-957:
--------------------------------------

bq. Implementation and test case attached.

At line 63: "new  Well19937c()": there are two spaces after 'new'... Line 79 as 
well.

                
> Use analytical function for 
> UniformRealDistribution.inverseCumulativeProbability
> --------------------------------------------------------------------------------
>
>                 Key: MATH-957
>                 URL: https://issues.apache.org/jira/browse/MATH-957
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 3.2
>            Reporter: Evan Ward
>         Attachments: uniform.patch
>
>
> The inverse CDF is currently solved by a root finding function. It would be 
> much simpler (and faster) to use the analytical expression. This would save 
> the user from having to set the inverseCumAccuracy correctly.
> I've attached a patch that implements this.

--
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