On Wed, Apr 18, 2012 at 07:23:04PM +0200, Luc Maisonobe wrote:
> Le 18/04/2012 14:21, Gilles Sadowski a écrit :
> > Hi.
> >
> > In "AbstractLeastSquaresOptimizer" (package "optimization.general"), the
> > method "guessParametersErrors" is defined as:
> > ---CUT---
> >     public double[] guessParametersErrors() {
> >         if (rows <= cols) {
> >             throw new
> > NumberIsTooSmallException(LocalizedFormats.NO_DEGREES_OF_FREEDOM,
> >                                                 rows, cols, false);
> >         }
> >         double[] errors = new double[cols];
> >         final double c = FastMath.sqrt(getChiSquare() / (rows -
> > cols)); // <--- HERE
> >         double[][] covar = getCovariances();
> >         for (int i = 0; i < errors.length; ++i) {
> >             errors[i] = FastMath.sqrt(covar[i][i]) * c;
> >         }
> >         return errors;
> >     }
> > ---CUT---
> >
> > In the above, could someone point me to a reference that would explain the
> > line marked "HERE" (i.e. the usage of "c" to compute the error
> > estimation)?
> 
> I have tracked the changes back to issue MATH-176

Thanks.
Looking at this patch, in the method in question, there is:
---CUT---
errors[i] = Math.sqrt(covar[i][i]) * c);
---CUT---

This looks like it couldn't have been compiled as is; maybe an indication
the original patch was not tested by the contributor.

Does anyone object to the estimated error be defined as

  errors[i] = FastMath.sqrt(covar[i][i]);

?


Best regards,
Gilles

> (<https://issues.apache.org/jira/browse/MATH-176>).
> I don't know the theory underlying the patch that was provided.
> 
> Luc
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to