commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gilles (JIRA)" <>
Subject [jira] [Commented] (MATH-1014) Remove optimizer from constructor of "CurveFitter" subclasses
Date Wed, 20 Nov 2013 19:04:36 GMT


Gilles commented on MATH-1014:

Some time ago, it is has been noted on the ML that the Levenberg-Marquardt algorithm was not
the most efficient for fitting a polynomial. 
Another optimizer can be plugged into the code when available.

Deprecated classes must be removed when preparing the next major release.

> Remove optimizer from constructor of "CurveFitter" subclasses
> -------------------------------------------------------------
>                 Key: MATH-1014
>                 URL:
>             Project: Commons Math
>          Issue Type: Improvement
>    Affects Versions: 3.2
>            Reporter: Gilles
>            Assignee: Gilles
>            Priority: Minor
>              Labels: api-change
>             Fix For: 4.0, 3.3
>         Attachments:,
> In package "o.a.c.m.fitting", the constructor of the concrete subclasses of "CurveFitter"
(currently: "PolynomialFitter", "GaussianFitter", "HarmonicFitter") takes a "MultivariateVectorOptimizer"
> However, assuming that there is _one_ best choice for the optimizer (given the parametric
function), this argument should not be left to the user's choice (i.e. it should be hidden
within the class, and the best optimizer be transparently selected).
> Thus, I would propose to deprecate the non-default constructor.

This message was sent by Atlassian JIRA

View raw message