[
https://issues.apache.org/jira/browse/MATH-521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12997773#comment-12997773
]
Gilles commented on MATH-521:
-----------------------------
When the guess algorithm fails, it should nevertheless provide default initial
values (instead of bailing out).
I'll try to set up a unit test along the lines you indicated.
> Changes in "HarmonicCoefficientsGuesser"
> ----------------------------------------
>
> Key: MATH-521
> URL: https://issues.apache.org/jira/browse/MATH-521
> Project: Commons Math
> Issue Type: Improvement
> Reporter: Gilles
> Assignee: Gilles
> Priority: Minor
> Labels: api-change, documentation
> Fix For: 3.0
>
>
> (1) The "guess" method throws "OptimizationException" when the algorithm
> fails to determine valid values for amplitude and angular frequency.
> There are no test showing how such a situation can occur.
> Moreover, since this procedure is used to provide an initial guess to an
> optimizer, it is better to pick any values for those parameters (e.g. zero)
> and let the optimizer proceed from that initial point.
> (2) The class javadoc seems very thorough in explaining the algorithm, but is
> quite unreadable in the source code, making it fairly useless for checking
> how the code complies with the comments. I think that this explanation should
> go in the user guide (and leave a mostly "plain text" outline of the
> algorithm, referring to the guide for details). [Does the format of the user
> guide allow such tricky (ASCII "art") constructs?]
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
