commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MATH-699) inverseCumulativeDistribution fails with cumulative distribution having a plateau
Date Mon, 07 Nov 2011 15:12:51 GMT

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

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

My inclination would be to keep the implementation in the base class as simple as possible,
documenting what it does and pushing the responsibility for dealing with plateaus in the distribution
to the implementations that have these.  I don't think any of the currently implemented real
distributions have this problem.  Correct?  

The invariant you are proposing for when to return infinities for domain lower and upper bounds
would make sense if these were intended to be the bounds of support for the distribution,
but this is not what these properties represent.  They are initial guesses for where to start
when trying to bracket a root.  That means they have to be values that can be fed into the
cumulative probability function. Have you convinced yourself that we do not need to bracket
a root at the beginning?  What bounds would we then provide to the solver?  The code you reference
is trying to do the bracketing, starting with the domain upper and lower bounds as initial
guesses.  

Remember to consider convergence problems when the actual parameter to inverse cum is close
to or exactly equal to 0 or 1.  Per the comment in the code above the test that uses (or should
use) the function value absolute accuracy, if the distribution has bounded support and the
argument is 0 or 1, bracketing will fail.


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

       

Mime
View raw message