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

Phil Steitz commented on MATH-699:
----------------------------------

Wow, we can really improve the code when we put our heads together :)

Great observation by Christian - we can switch on plateau detection iff 
isSupportConnected is false.  Otherwise, I would say the "new" implementation 
is good to go.  I would also be happy to celebrate dropping all of the 
getXxx(p) methods, which are not needed any more.

We can all thank Mikkel, btw, for suggesting a while back that we add the 
numerical mean, variance and connected support properties.  Came in very handy 
here!

                
> inverseCumulativeDistribution fails with cumulative distribution having a 
> plateau
> ---------------------------------------------------------------------------------
>
>                 Key: MATH-699
>                 URL: https://issues.apache.org/jira/browse/MATH-699
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 3.0
>            Reporter: Sébastien Brisard
>            Assignee: Sébastien Brisard
>            Priority: Minor
>         Attachments: AbstractContinuousDistributionTest.java, 
> inv-cum-new-impl.zip
>
>
> This bug report follows MATH-692. The attached unit test fails. As required 
> by the definition in MATH-692, the lower-bound of the interval on which the 
> cdf is constant should be returned. This is not so at the moment.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to