Refactoring #2234

Simplify minimizer constructor

Added by pospelov 11 months ago.

Status:BacklogStart date:29 Nov 2018
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:-
Target version:-

Description

For the moment minimizer initialization string has 3 parameters which allows to set minimizer type, minimizer algorithm and option string

minimizer.setMinimizer("Minuit2")
minimizer.setMinimizer("Minuit2", "Migrad")
minimizer.setMinimizer("Minuit2", "Migrad", "MaxFunctionCalls=500;Strategy=2")

The problem is that some minimizers has only one algorithm, so the string become

minimizer.setMinimizer("Genetic", "", "seed=1");

Empty string as second parameter looks not very natural and complicates tutorial writing.

Suggestion: merge algorithm and option string together.

minimizer.setMinimizer("Minuit2", "Algorithm=Migrad;MaxFunctionCalls=500;Strategy=2")

Also available in: Atom PDF