Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
2.2, 3.0, 3.1
-
None
Description
The way the density of the gamma distribution function is estimated can be improved.
It's much more stable to calculate first the log of the density and then exponentiate, otherwise the function returns NaN for high values of the parameters alpha and beta.
It would be sufficient to change the public double density(double x) function at line 204 in the file org.apache.commons.math.distribution.GammaDistributionImpl as follows:
return Math.exp(Math.log( x )*(alpha-1) - Math.log(beta)*alpha - x/beta - Gamma.logGamma(alpha));
In order to improve performance, log(beta) and Gamma.logGamma(alpha) could also be precomputed and stored during initialization.