Refactoring #2234
Fit: Simplify minimizer constructor
| Status: | Backlog | Start date: | 29 Nov 2018 | |
|---|---|---|---|---|
| Priority: | Normal | Due 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")
Related issues
History
#2 Updated by wuttke 4 months ago
- Related to Refactoring #2476: Fit: catalogize algorithms, not minimizers added