ctest passes with HIPO=off

Author: jajhall

highs/lp_data/HighsSolve.cpp

@@ -37,7 +37,12 @@ HighsStatus solveLp(HighsLpSolverObject& solver_object, const string message) {
     if (return_status == HighsStatus::kError) return return_status;
   }
   const bool use_only_ipm = useIpm(options.solver) || options.run_centring;
-  const bool use_hipo = useHipo(options, kSolverString, solver_object.lp_);
+  bool use_hipo = useHipo(options, kSolverString, solver_object.lp_);
+#ifndef HIPO
+  // Shouldn't be possible to choose HiPO if it's not in the build
+  assert(!use_hipo);
+  use_hipo = false;
+#endif
   const bool use_ipx = use_only_ipm && !use_hipo;
   // Now actually solve LPs!
   //

highs/mip/HighsLpRelaxation.cpp

@@ -1292,7 +1292,9 @@ HighsLpRelaxation::Status HighsLpRelaxation::run(bool resolve_on_error) {
 	// here. Later pass mip_ipm_solver and take action on failure in
 	// solveLp
 	bool use_hipo =
+#ifdef HIPO
 	  mip_ipm_solver == kHighsChooseString ||
+#endif
 	  mip_ipm_solver == kHipoString;
 #ifndef HIPO
 	// Shouldn't be possible to choose HiPO if it's not in the build

highs/mip/HighsMipSolverData.cpp

@@ -357,7 +357,9 @@ void HighsMipSolverData::startAnalyticCenterComputation(
     // here. Later pass mip_ipm_solver and take action on failure in
     // solveLp
     bool use_hipo =
+#ifdef HIPO
       mip_ipm_solver == kHighsChooseString ||
+#endif
       mip_ipm_solver == kHipoString;
 #ifndef HIPO
     // Shouldn't be possible to choose HiPO if it's not in the build