You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Michael Sammeth (JIRA)" <ji...@apache.org> on 2010/03/18 22:08:27 UTC

[jira] Created: (MATH-354) compiler dependency in RandomDataImpl.getPoisson(double mean)

compiler dependency in RandomDataImpl.getPoisson(double mean)
-------------------------------------------------------------

                 Key: MATH-354
                 URL: https://issues.apache.org/jira/browse/MATH-354
             Project: Commons Math
          Issue Type: Bug
    Affects Versions: 2.0
         Environment: jre1.5/jre1.6
            Reporter: Michael Sammeth


Hi,

in RandomDataImpl.getPoisson(double mean) I got the following problem in the case mean>= 6.0:

in the branch if (u <= c1):

if (x < -mu)
    w = Double.POSITIVE_INFINITY;

implicits that (int) (mu+ x) < 0

I found that for some compiler/run-time environments the subsequent update of the "accept" value then fails, as by the right hand side of the comparison leads to an Exception in MathUtils.factorialLog((int) (mu + x)). Some compiler/jre combinations, however, skip evaluating the right side as by isInfinity(w).

To ensure stability, I currently worked around by an explicit if(Double.isInfinity(w)) branch, however, I would like to ask whether there is a more elegant way to ensure general functionality of that method.

Thank you, micha. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (MATH-354) compiler dependency in RandomDataImpl.getPoisson(double mean)

Posted by "Phil Steitz (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MATH-354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Phil Steitz resolved MATH-354.
------------------------------

       Resolution: Fixed
    Fix Version/s: 2.1

The method has been rewritten in version 2.1.  The current version no longer includes the compiler-dependent code referenced in this issue.

> compiler dependency in RandomDataImpl.getPoisson(double mean)
> -------------------------------------------------------------
>
>                 Key: MATH-354
>                 URL: https://issues.apache.org/jira/browse/MATH-354
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 2.0
>         Environment: jre1.5/jre1.6
>            Reporter: Michael Sammeth
>             Fix For: 2.1
>
>
> Hi,
> in RandomDataImpl.getPoisson(double mean) I got the following problem in the case mean>= 6.0:
> in the branch if (u <= c1):
> if (x < -mu)
>     w = Double.POSITIVE_INFINITY;
> implicits that (int) (mu+ x) < 0
> I found that for some compiler/run-time environments the subsequent update of the "accept" value then fails, as by the right hand side of the comparison leads to an Exception in MathUtils.factorialLog((int) (mu + x)). Some compiler/jre combinations, however, skip evaluating the right side as by isInfinity(w).
> To ensure stability, I currently worked around by an explicit if(Double.isInfinity(w)) branch, however, I would like to ask whether there is a more elegant way to ensure general functionality of that method.
> Thank you, micha. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (MATH-354) compiler dependency in RandomDataImpl.getPoisson(double mean)

Posted by "Phil Steitz (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MATH-354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Phil Steitz closed MATH-354.
----------------------------


> compiler dependency in RandomDataImpl.getPoisson(double mean)
> -------------------------------------------------------------
>
>                 Key: MATH-354
>                 URL: https://issues.apache.org/jira/browse/MATH-354
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 2.0
>         Environment: jre1.5/jre1.6
>            Reporter: Michael Sammeth
>             Fix For: 2.1
>
>
> Hi,
> in RandomDataImpl.getPoisson(double mean) I got the following problem in the case mean>= 6.0:
> in the branch if (u <= c1):
> if (x < -mu)
>     w = Double.POSITIVE_INFINITY;
> implicits that (int) (mu+ x) < 0
> I found that for some compiler/run-time environments the subsequent update of the "accept" value then fails, as by the right hand side of the comparison leads to an Exception in MathUtils.factorialLog((int) (mu + x)). Some compiler/jre combinations, however, skip evaluating the right side as by isInfinity(w).
> To ensure stability, I currently worked around by an explicit if(Double.isInfinity(w)) branch, however, I would like to ask whether there is a more elegant way to ensure general functionality of that method.
> Thank you, micha. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.