Skip to content

Fix example notebooks for current release zips - #21

Merged
0x17 merged 2 commits into
mainfrom
fix-example-notebooks
Oct 6, 2026
Merged

0x17 merged 2 commits into
mainfrom
fix-example-notebooks

Conversation

@0x17

@0x17 0x17 commented Oct 6, 2026

Copy link
Copy Markdown
Member
  • Bug fix: example notebooks failed on a fresh run because the release zip ships gamsconfig_cuopt.yaml, which the first cell never merged into gamsconfig.yaml (solve failed with unknown solver cuopt)
    • first cell now downloads fetch-cuoptlink.py from main and runs install -g <gamspy base> -c <cuversion> --cuda-runtime
    • skips the install if .cuopt_installed_files.txt already exists, so re-running the cell does not re-download ~1.5-2 GB
    • only the final state of each download progress line is kept in the cell output
  • Bug fix: _test_installation in fetch-cuoptlink.py needed gamslib, which gamspy_base does not ship
    • now writes a small embedded LP model instead and aborts unless the model status is Optimal
    • README note updated accordingly
  • Re-ran both notebooks (cu12, cu13) against v0.0.8e: GAMSPy 1.28.1 / GAMS 54.5.0, cuOpt 26.08.00, objective 153.697
  • Verified: install + self-test on a GAMSPy base for cu12 and cu13; self-test fails (rc 3) for an infeasible variant of the test model; arm64 not tested
  • Note: notebooks fetch the script from main, so they only work once this is merged

@0x17
0x17 merged commit 16c84d0 into main Oct 6, 2026
6 checks passed
0x17 added a commit that referenced this pull request Oct 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant