Skip to content

Commit

Permalink
Fix performance issue (#814)
Browse files Browse the repository at this point in the history
For now, outer loop is not implemented in Benders by batch, and the
current implementation in Benders is not efficient due to #813. For
benders by batch users, we do not want to introduce performance loss as
the algorithm has not changed, that is the goal of this PR
  • Loading branch information
tbittar authored May 16, 2024
1 parent 2ed62ff commit 170b15a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/cpp/benders/benders_by_batch/BendersByBatch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,7 @@ void BendersByBatch::GetSubproblemCut(
worker->fix_to(_data.x_cut);
worker->solve(subproblem_data.lpstatus, Options().OUTPUTROOT,
Options().LAST_MASTER_MPS + MPS_SUFFIX, _writer);
worker->get_solution(subproblem_data.solution);
// worker->get_solution(subproblem_data.solution);
worker->get_value(subproblem_data.subproblem_cost); // solution phi(x,s)
worker->get_subgradient(
subproblem_data.var_name_and_subgradient); // dual pi_s
Expand Down

0 comments on commit 170b15a

Please sign in to comment.