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

Phil Steitz closed MATH-587.
----------------------------

    Resolution: Not A Problem

Once the javadoc API descriptions were reverted to last release versions, the 
documentation matches the implementation.  Sorry I was confused by the javadoc. 
 I added a standalone test class confirming that the API contracts match the 
default impls.

> AbstractIntegerDistribution cumulativeProbablitity(int, int) implementation 
> does not match API documentation
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: MATH-587
>                 URL: https://issues.apache.org/jira/browse/MATH-587
>             Project: Commons Math
>          Issue Type: Bug
>            Reporter: Phil Steitz
>
> The documentation for this method says it computes p(x0 < X < x1), but the 
> parameter documentation inconsistently states that the bounds are "inclusive" 
> and what it actually returns is
> {code}
> cumulativeProbability(x1) - cumulativeProbability(x0 - 1);
> {code}
> which is p(x0-1 <= X < x1).
> In the code above, x0 - 1 should be replaced by x0 + 1.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to