Gilles created MATH-956:
---------------------------

             Summary: Hardcoded numbers in "LevenbergMarquardtOptimizer"
                 Key: MATH-956
                 URL: https://issues.apache.org/jira/browse/MATH-956
             Project: Commons Math
          Issue Type: Task
    Affects Versions: 3.1.1
            Reporter: Gilles
            Priority: Minor
             Fix For: 3.2


The class contains numbers "2.2251e-308" and "2.2204e-16".
The former is the "SAFE_MIN" defined in class "Precision".

The latter is documented as
{noformat}
2.2204e-16 is the machine epsilon for IEEE754
{noformat}

However, the "EPSILON" defined in class "Precision" is equal to 
"1.1102230246251565E-16" (i.e. half the above number). This requires 
clarification.
I would remove the seemingly incorrect comment (if we assume that the correct 
definition is in the comment for "EPSILON" in "Precision") and replace the 
hard-coded values with

{noformat}
2 * Precision.EPSILON
{noformat}

Do you agree?




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to