Fix issue with POA bound tightening

coin-or · Sep 16, 2021 · a146a13 · a146a13
1 parent 1dbf797
commit a146a13
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Tasks/TaskPerformBoundTightening.cpp b/src/Tasks/TaskPerformBoundTightening.cpp
@@ -87,7 +87,7 @@ void TaskPerformBoundTightening::run()
 {
     env->timing->startTimer("BoundTightening");
 
-    if(env->settings->getSetting<bool>("BoundTightening.InitialPOA.Use", "Model")
+    if(env->settings->getSetting<bool>("BoundTightening.InitialPOA.Use", "Model") && env->reformulatedProblem
         && (sourceProblem->properties.numberOfNonlinearConstraints > 0
             || sourceProblem->objectiveFunction->properties.classification
                 > E_ObjectiveFunctionClassification::Quadratic))

diff --git a/src/Tasks/TaskReformulateProblem.cpp b/src/Tasks/TaskReformulateProblem.cpp
@@ -245,9 +245,6 @@ TaskReformulateProblem::TaskReformulateProblem(EnvironmentPtr envPtr) : TaskBase
     reformulatedProblem->properties.numberOfAddedLinearizations = env->problem->properties.numberOfAddedLinearizations;
     reformulatedProblem->finalize();
 
-    auto taskPerformBoundTightening = std::make_unique<TaskPerformBoundTightening>(env, reformulatedProblem);
-    taskPerformBoundTightening->run();
-
     // Fixing that a quadratic objective changed into a nonlinear objective is correctly identified
     if(!(useConvexQuadraticObjective || useNonconvexQuadraticObjective)
         && reformulatedProblem->objectiveFunction->properties.classification
@@ -281,6 +278,9 @@ TaskReformulateProblem::TaskReformulateProblem(EnvironmentPtr envPtr) : TaskBase
         Utilities::writeStringToFile(filename.str(), problem.str());
     }
 
+    auto taskPerformBoundTightening = std::make_unique<TaskPerformBoundTightening>(env, reformulatedProblem);
+    taskPerformBoundTightening->run();
+
     env->timing->stopTimer("ProblemReformulation");
 }