Merge pull request #2161 from ERGO-Code/fix-log-header

Now only recording written_log_header_ = true if output_flag is true …
ERGO-Code · Jan 30, 2025 · 52af45f · 52af45f
2 parents aa53030 + 6fb8f36
commit 52af45f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/Highs.h b/src/Highs.h
@@ -1471,7 +1471,7 @@ class Highs {
   bool called_return_from_run = true;
   HighsInt debug_run_call_num_ = 0;
 
-  bool written_log_header = false;
+  bool written_log_header_ = false;
 
   HighsStatus solve();
 

diff --git a/src/lp_data/Highs.cpp b/src/lp_data/Highs.cpp
@@ -1717,6 +1717,7 @@ HighsStatus Highs::getStandardFormLp(HighsInt& num_col, HighsInt& num_row,
                                      HighsInt& num_nz, double& offset,
                                      double* cost, double* rhs, HighsInt* start,
                                      HighsInt* index, double* value) {
+  this->logHeader();
   if (!this->standard_form_valid_) {
     HighsStatus status = formStandardFormLp();
     assert(status == HighsStatus::kOk);
@@ -4249,9 +4250,10 @@ HighsStatus Highs::callRunPostsolve(const HighsSolution& solution,
 
 // End of public methods
 void Highs::logHeader() {
-  if (written_log_header) return;
+  if (written_log_header_) return;
+  if (!*options_.log_options.output_flag) return;
   highsLogHeader(options_.log_options, options_.log_githash);
-  written_log_header = true;
+  written_log_header_ = true;
   return;
 }