Skip to content
Snippets Groups Projects
Commit d34b3ade authored by Mikhail Svechnikov's avatar Mikhail Svechnikov
Browse files

rm ALGORITHM_LIST

parent 0bb30b9f
No related branches found
No related tags found
1 merge request!1004Fit: catalogize algorithms, not minimizers
Pipeline #74476 passed
...@@ -32,7 +32,6 @@ const QMap<QString, QString> minimizer_names_map = { ...@@ -32,7 +32,6 @@ const QMap<QString, QString> minimizer_names_map = {
{"GSL Levenberg-Marquardt", GSLLMAMinimizerItem::M_TYPE}}; {"GSL Levenberg-Marquardt", GSLLMAMinimizerItem::M_TYPE}};
const QString default_minimizer = minimizer_names_map.key(MinuitMinimizerItem::M_TYPE); const QString default_minimizer = minimizer_names_map.key(MinuitMinimizerItem::M_TYPE);
#ifdef ALGORITHM_LIST
QMap<QString /*algorithm*/, QString /*minimizer*/> algorithm_minimizer_map; QMap<QString /*algorithm*/, QString /*minimizer*/> algorithm_minimizer_map;
void add_algorithm_from_minimizer_to_list_and_map(const QString& minimizer, void add_algorithm_from_minimizer_to_list_and_map(const QString& minimizer,
QStringList& common_algorithms_list, QStringList& common_algorithms_list,
...@@ -92,7 +91,6 @@ void create_algorithm_list_and_map(QString& default_common_algorithm, QStringLis ...@@ -92,7 +91,6 @@ void create_algorithm_list_and_map(QString& default_common_algorithm, QStringLis
default_common_algorithm = algorithms_list[1]; // after the first header: Minuit2 Migrad default_common_algorithm = algorithms_list[1]; // after the first header: Minuit2 Migrad
} }
#endif
} // namespace } // namespace
...@@ -113,7 +111,6 @@ MinimizerContainerItem::MinimizerContainerItem() ...@@ -113,7 +111,6 @@ MinimizerContainerItem::MinimizerContainerItem()
addProperty<SimAnMinimizerItem>(SimAnMinimizerItem::M_TYPE); addProperty<SimAnMinimizerItem>(SimAnMinimizerItem::M_TYPE);
addProperty<GSLLMAMinimizerItem>(GSLLMAMinimizerItem::M_TYPE); addProperty<GSLLMAMinimizerItem>(GSLLMAMinimizerItem::M_TYPE);
#ifdef ALGORITHM_LIST
QString default_common_algorithm; QString default_common_algorithm;
QStringList common_algorithms_list; QStringList common_algorithms_list;
QStringList common_algorithms_descriptions; QStringList common_algorithms_descriptions;
...@@ -123,7 +120,6 @@ MinimizerContainerItem::MinimizerContainerItem() ...@@ -123,7 +120,6 @@ MinimizerContainerItem::MinimizerContainerItem()
ComboProperty::fromList(common_algorithms_list, default_common_algorithm); ComboProperty::fromList(common_algorithms_list, default_common_algorithm);
common_algorithm_combo.setToolTips(common_algorithms_descriptions); common_algorithm_combo.setToolTips(common_algorithms_descriptions);
addProperty(P_COMMON_ALGORITHMS, common_algorithm_combo.variant()); addProperty(P_COMMON_ALGORITHMS, common_algorithm_combo.variant());
#endif
ComboProperty minimizer_combo = ComboProperty minimizer_combo =
ComboProperty::fromList(minimizer_names_map.keys(), default_minimizer); ComboProperty::fromList(minimizer_names_map.keys(), default_minimizer);
...@@ -190,7 +186,7 @@ ComboProperty MinimizerContainerItem::minimizerCombo() const ...@@ -190,7 +186,7 @@ ComboProperty MinimizerContainerItem::minimizerCombo() const
{ {
return getItemValue(P_MINIMIZERS).value<ComboProperty>(); return getItemValue(P_MINIMIZERS).value<ComboProperty>();
} }
#ifdef ALGORITHM_LIST
bool MinimizerContainerItem::algorithmHasMinimizer(const QString& name) bool MinimizerContainerItem::algorithmHasMinimizer(const QString& name)
{ {
return algorithm_minimizer_map.contains(name); return algorithm_minimizer_map.contains(name);
...@@ -231,7 +227,7 @@ void MinimizerContainerItem::applyAlgorithmToMinimizer(const QString& name) ...@@ -231,7 +227,7 @@ void MinimizerContainerItem::applyAlgorithmToMinimizer(const QString& name)
// GSL Levenberg-Marquardt // GSL Levenberg-Marquardt
// do nothing // do nothing
} }
#endif
QString MinimizerContainerItem::currentObjectiveMetric() const QString MinimizerContainerItem::currentObjectiveMetric() const
{ {
return objectiveMetricCombo().currentValue(); return objectiveMetricCombo().currentValue();
......
...@@ -39,8 +39,6 @@ class GeneticMinimizerItem; ...@@ -39,8 +39,6 @@ class GeneticMinimizerItem;
class SimAnMinimizerItem; class SimAnMinimizerItem;
class GSLLMAMinimizerItem; class GSLLMAMinimizerItem;
#define ALGORITHM_LIST
class MinimizerContainerItem : public MinimizerItem { class MinimizerContainerItem : public MinimizerItem {
private: private:
static constexpr auto P_MINIMIZERS{"Minimizer"}; static constexpr auto P_MINIMIZERS{"Minimizer"};
...@@ -66,12 +64,10 @@ public: ...@@ -66,12 +64,10 @@ public:
void setCurrentMinimizer(const QString& name); void setCurrentMinimizer(const QString& name);
ComboProperty minimizerCombo() const; ComboProperty minimizerCombo() const;
#ifdef ALGORITHM_LIST
bool algorithmHasMinimizer(const QString& name); bool algorithmHasMinimizer(const QString& name);
void setCurrentCommonAlgorithm(const QString& name); void setCurrentCommonAlgorithm(const QString& name);
ComboProperty commonAlgorithmCombo() const; ComboProperty commonAlgorithmCombo() const;
void applyAlgorithmToMinimizer(const QString& name); void applyAlgorithmToMinimizer(const QString& name);
#endif
// Objective metric to use for estimating distance between simulated and experimental data // Objective metric to use for estimating distance between simulated and experimental data
QString currentObjectiveMetric() const; QString currentObjectiveMetric() const;
......
...@@ -55,17 +55,7 @@ void MinimizerSettingsWidget::setMinItem(MinimizerContainerItem* minimizerItem) ...@@ -55,17 +55,7 @@ void MinimizerSettingsWidget::setMinItem(MinimizerContainerItem* minimizerItem)
if (!m_currentItem) if (!m_currentItem)
return; return;
#ifdef ALGORITHM_LIST
createGroupedAlgorithmsCombo(); createGroupedAlgorithmsCombo();
#else
m_mainLayout->addRow("Minimizer:",
GUI::Util::createComboBox([=] { return m_currentItem->minimizerCombo(); },
[=](const QString& t) {
m_currentItem->setCurrentMinimizer(t);
createMimimizerEdits();
},
&m_updaters));
#endif
auto* w = new QWidget(this); auto* w = new QWidget(this);
m_minimizerLayout = new QFormLayout(w); m_minimizerLayout = new QFormLayout(w);
...@@ -90,7 +80,7 @@ void MinimizerSettingsWidget::setMinItem(MinimizerContainerItem* minimizerItem) ...@@ -90,7 +80,7 @@ void MinimizerSettingsWidget::setMinItem(MinimizerContainerItem* minimizerItem)
createMimimizerEdits(); createMimimizerEdits();
updateUIValues(); updateUIValues();
} }
#ifdef ALGORITHM_LIST
void MinimizerSettingsWidget::createGroupedAlgorithmsCombo() void MinimizerSettingsWidget::createGroupedAlgorithmsCombo()
{ {
QComboBox* comboBox = new QComboBox; QComboBox* comboBox = new QComboBox;
...@@ -161,7 +151,7 @@ void MinimizerSettingsWidget::createGroupedAlgorithmsCombo() ...@@ -161,7 +151,7 @@ void MinimizerSettingsWidget::createGroupedAlgorithmsCombo()
m_mainLayout->addRow("Algorithm:", comboBox); m_mainLayout->addRow("Algorithm:", comboBox);
} }
#endif
void MinimizerSettingsWidget::createMimimizerEdits() void MinimizerSettingsWidget::createMimimizerEdits()
{ {
GUI::Util::Layout::clearLayout(m_minimizerLayout); GUI::Util::Layout::clearLayout(m_minimizerLayout);
...@@ -191,12 +181,6 @@ void MinimizerSettingsWidget::createMinuitEdits() ...@@ -191,12 +181,6 @@ void MinimizerSettingsWidget::createMinuitEdits()
{ {
MinuitMinimizerItem* minItem = m_currentItem->minimizerItemMinuit(); MinuitMinimizerItem* minItem = m_currentItem->minimizerItemMinuit();
#ifndef ALGORITHM_LIST
m_minimizerLayout->addRow(
"Algorithm:", GUI::Util::createComboBox(
[=] { return minItem->algorithmCombo(); },
[=](const QString& t) { minItem->setCurrentAlgorithm(t); }, &m_updaters));
#endif
m_minimizerLayout->addRow( m_minimizerLayout->addRow(
"Strategy:", "Strategy:",
GUI::Util::createIntSpinbox([=] { return minItem->strategy(); }, GUI::Util::createIntSpinbox([=] { return minItem->strategy(); },
...@@ -236,12 +220,6 @@ void MinimizerSettingsWidget::createGSLMultiMinEdits() ...@@ -236,12 +220,6 @@ void MinimizerSettingsWidget::createGSLMultiMinEdits()
{ {
GSLMultiMinimizerItem* minItem = m_currentItem->minimizerItemGSLMulti(); GSLMultiMinimizerItem* minItem = m_currentItem->minimizerItemGSLMulti();
#ifndef ALGORITHM_LIST
m_minimizerLayout->addRow(
"Algorithm:", GUI::Util::createComboBox(
[=] { return minItem->algorithmCombo(); },
[=](const QString& t) { minItem->setCurrentAlgorithm(t); }, &m_updaters));
#endif
m_minimizerLayout->addRow( m_minimizerLayout->addRow(
"Max iterations:", "Max iterations:",
GUI::Util::createIntSpinbox([=] { return minItem->maxIterations(); }, GUI::Util::createIntSpinbox([=] { return minItem->maxIterations(); },
......
...@@ -35,9 +35,7 @@ public slots: ...@@ -35,9 +35,7 @@ public slots:
void setMinItem(MinimizerContainerItem* minimizerItem); void setMinItem(MinimizerContainerItem* minimizerItem);
private: private:
#ifdef ALGORITHM_LIST
void createGroupedAlgorithmsCombo(); void createGroupedAlgorithmsCombo();
#endif
void createMimimizerEdits(); void createMimimizerEdits();
void createMinuitEdits(); void createMinuitEdits();
void createGSLMultiMinEdits(); void createGSLMultiMinEdits();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment