Remove duplicate useOrtools (#541)

Having 2 attributes results in OR-Tools not being used even if the flag --use-ortools is present.
AntaresSimulatorTeam · Feb 7, 2022 · 73e8d8d · 73e8d8d
1 parent a12b233
commit 73e8d8d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/libs/antares/study/load-options.h b/src/libs/antares/study/load-options.h
@@ -111,8 +111,6 @@ class StudyLoadOptions
     YString studyFolder;
     YString simulationName;
     std::string ortoolsSolver;
-
-    bool useOrtools = false;
 }; // class StudyLoadOptions
 
 } // namespace Data

diff --git a/src/solver/misc/options.cpp b/src/solver/misc/options.cpp
@@ -103,7 +103,7 @@ GetOpt::Parser CreateParser(Settings& settings, Antares::Data::StudyLoadOptions&
 
     // add option for ortools use
     // --use-ortools
-    parser.addFlag(options.useOrtools, ' ', "use-ortools", "Use ortools library to launch solver");
+    parser.addFlag(options.ortoolsUsed, ' ', "use-ortools", "Use ortools library to launch solver");
 
     //--ortools-solver
     parser.add(options.ortoolsSolver,
@@ -259,7 +259,7 @@ void checkAndCorrectSettingsAndOptions(Settings& settings, Data::StudyLoadOption
 void checkOrtoolsSolver(Data::StudyLoadOptions& options)
 {
     // ortools solver
-    if (options.useOrtools)
+    if (options.ortoolsUsed)
     {
         const std::list<std::string> availableSolverList
           = OrtoolsUtils().getAvailableOrtoolsSolverName();