You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Phil Steitz (JIRA)" <ji...@apache.org> on 2011/06/12 21:44:51 UTC

[jira] [Closed] (MATH-587) AbstractIntegerDistribution cumulativeProbablitity(int, int) implementation does not match API documentation

     [ 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