Conversation
Merging this PR will degrade performance by 3.52%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ❌ | test_to_lp[merge_balance-severity=0] |
2.7 MB | 3.3 MB | -18.74% |
| ⚡ | test_to_lp[nodal_balance_sparse-severity=100] |
3.9 MB | 3.4 MB | +14.54% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing mcoughlin:finish-scip (ccc9f5c) with master (0770dfe)
Footnotes
-
181 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Replace the SCIP-internal aux denylist (prefixes + constraint handlers) with an allowlist matching linopy's own x<label>/c<label> names.
FabianHofmann
left a comment
There was a problem hiding this comment.
thanks @mcoughlin ; added a small change on how to reference optimization vars/cons.
SCIP was already reading indicator constraints, SOS constraints and semi-continuous variables; now it actually uses them.
Note
Code was checked by AI (Claude Code).
Changes proposed in this Pull Request
Plugs in SCIP indicator constraints, SOS constraints and semi-continuous variables such that they are used in the solver.
Checklist
AGENTS.md).doc.doc/release_notes.rstof the upcoming release is included.