Skip to content

Commit f8cf3c9

Browse files
committed
Found source of lp_iterations blow-up: added comment with tag #2326
1 parent e77fd4d commit f8cf3c9

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

highs/mip/HighsPrimalHeuristics.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,12 @@ bool HighsPrimalHeuristics::solveSubMip(
159159
// (double)mipsolver.orig_model_->a_matrix_.value_.size();
160160
int64_t adjusted_lp_iterations =
161161
(size_t)(adjustmentfactor * submipsolver.mipdata_->total_lp_iterations);
162+
// The value of adjustmentfactor as the sub-MIP level decreases
163+
// can lead to lp_iterations blowing up - see original model in
164+
// #2326. Since lp_iterations is a pseudo-measure - for what
165+
// purpose? - is it misleading to report it? That said, the reason
166+
// why it blows up may be indicative of circumstances where
167+
// sub-MIPs being solved is inadvisable
162168
lp_iterations += adjusted_lp_iterations;
163169
total_repair_lp += submipsolver.mipdata_->total_repair_lp;
164170
total_repair_lp_feasible += submipsolver.mipdata_->total_repair_lp_feasible;

0 commit comments

Comments
 (0)