Fixed issues with unit tests

Author: mathgeekcoder

check/TestCallbacks.cpp

@@ -253,8 +253,6 @@ HighsCallbackFunctionType userkMipUserSetPartialSolution =
         // get every other index
         std::vector<HighsInt> index;
         std::vector<double> value;
-        index.reserve(callback_data.optimal_solution.size() / 2);
-        value.reserve(callback_data.optimal_solution.size() / 2);
 
         for (HighsInt i = 0; i < callback_data.optimal_solution.size(); i++) {
           if (i % 2 == 0) {

check/TestIpx.cpp

@@ -54,6 +54,10 @@ TEST_CASE("test-ipx", "[highs_ipx]") {
 
   highs::parallel::initialize_scheduler();
 
+  // set empty callback to avoid issues
+  HighsCallback callback(nullptr);
+  lps.SetCallback(&callback);
+
   Int status = lps.Solve();
   bool is_solved = status == IPX_STATUS_solved;
   REQUIRE(is_solved);

src/lp_data/HighsCallback.cpp

@@ -39,14 +39,15 @@ void HighsCallback::clearHighsCallbackOutput() {
 }
 
 void HighsCallback::clearHighsCallbackInput() {
-  this->data_in.user_interrupt = false;
+  size_t num_col = highs != nullptr ? highs->getNumCol() : 0;
 
   // make sure buffer size is correct and reset the contents if previously used
   if (this->data_in.user_has_solution ||
-      highs->getNumCol() != this->data_in.user_solution.size()) {
-    this->data_in.user_solution.assign(highs->getNumCol(), kHighsUndefined);
+      num_col != this->data_in.user_solution.size()) {
+    this->data_in.user_solution.assign(num_col, kHighsUndefined);
   }
 
+  this->data_in.user_interrupt = false;
   this->data_in.user_has_solution = false;
 }
 
@@ -261,17 +262,24 @@ HighsStatus HighsCallbackInput::repairSolution() {
     }
 
     // set callback to stop at first feasible solution
+    bool user_interrupt = false;
+
     HighsCallbackFunctionType mip_callback =
-        [](int callback_type, const std::string& message,
-           const HighsCallbackOutput* data_out, HighsCallbackInput* data_in,
-           void* user_callback_data) {
+        [&](int callback_type, const std::string& message,
+            const HighsCallbackOutput* data_out, HighsCallbackInput* data_in,
+            void* user_callback_data) {
           if (callback_type == kCallbackMipSolution) {
-            data_in->user_interrupt = true;
+            user_interrupt = true;
+          } else {
+            data_in->user_interrupt = user_interrupt;
           }
         };
 
     clone.setCallback(mip_callback);
     clone.startCallback(kCallbackMipSolution);
+    clone.startCallback(kCallbackMipInterrupt);
+    clone.startCallback(kCallbackSimplexInterrupt);
+    clone.startCallback(kCallbackIpmInterrupt);
     clone.run();
 
     auto status = clone.getModelStatus();