Conversation
cstr's recorded value was stale: BARON proves 3.062003537511728 at optcr=optca=0, where the README carried 3.0620145766. The other four models recorded no solution value at all; each now carries the best value found under a stated budget, with the provenance and the Examiner audit that backs it. Refs #152
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follows the solution-verification sweep in #152. No model code changes; READMEs only.
What changed
cstr: the recorded value was stale. The README carried 3.0620145766; GAMS/BARON atoptcr=optca=0proves 3.062003537511728 with lower bound equal to upper bound. Updated, and labelled proven optimal.ex1_linan_2023,grid,multiperiod_blending,pandemic: these recorded no solution value at all. Each now carries the best value found under a stated budget (GAMS/BARON ongdp.bigm, 120 s,optcr=1e-6), explicitly labelled not proven optimal.Every recorded point is backed by a GAMS/Examiner audit: primal and dual variable bounds and constraints at 1e-6, and both complementary-slackness conditions at 1e-7, with the discrete variables fixed. Each entry states its solver, tolerance, date, and audit status, so a later reader can tell a proven value from a best-found one and re-check either.
Two notes for reviewers:
grid's optimum is 0.0, which is meaningful rather than degenerate: the objective minimizes total capacity slack, and zero slack turns out to be attainable for the default instance, so no expansion is required.multiperiod_blendingships 60 instances underinstances_json/; the recorded value is for the instance built by the defaultbuild_model(), and the README now says so.Models with PRs in flight are deliberately untouched here; their values should be recorded as those PRs land.
water_networkandreverse_electrodialysisare also untouched, since their recorded values need the discussions on #151 and #149 respectively (scaling, and a units/value discrepancy) before they can be stated cleanly.Verification
pixi run pytest tests/— 335 passed, 1 skipped.Refs #152, #105