Skip to content

Conversation

@theSubsurfaceGuy
Copy link
Contributor

@theSubsurfaceGuy theSubsurfaceGuy commented Dec 28, 2025

Checklist

  • I am familiar with the Contributing Guidelines
  • Tests added and all existing tests pass
  • Documentation updated

Summary

  • Regenerate C bindings with new QP functions (cuOptCreateQuadraticProblem, cuOptCreateQuadraticRangedProblem) and CUOPT_NUM_GPUS parameter
  • Update version bounds to support 25.08-25.12
  • Update README and Colab instructions
  • Update package version to 0.2.0

As a student, this is one of my earliest open source PRs so I appreciate any feedback :)

- Regenerate C bindings with new QP functions (cuOptCreateQuadraticProblem,
  cuOptCreateQuadraticRangedProblem) and CUOPT_NUM_GPUS parameter
- Update version bounds to support 25.08-25.12
- Update README and Colab instructions for 25.12
- Update package version to 0.2.0

Signed-off-by: Waquar Kaleem <[email protected]>
- Test cuOptCreateQuadraticProblem
- Test cuOptCreateQuadraticRangedProblem
- Test CUOPT_NUM_GPUS constant

Signed-off-by: Waquar Kaleem <[email protected]>
odow added 2 commits December 30, 2025 13:26
Refactor test functions to remove unnecessary return statements.

Signed-off-by: Oscar Dowson <[email protected]>
Signed-off-by: Oscar Dowson <[email protected]>
@odow
Copy link
Member

odow commented Dec 30, 2025

Looks good! Did you run the tests locally with the new version?

@theSubsurfaceGuy
Copy link
Contributor Author

Yes, I ran the tests on a NVIDIA L40S GPU. The summary is below and here is the full output.log. Thanks!

Test Summary: | Pass  Total  Time
C_wrapper.jl  |   20     20  1.2s
Test Summary:  | Pass  Total     Time
MOI_wrapper.jl | 1065   1065  6m13.7s
Test Summary: | Total  Time
datasets      |     0  0.0s
Test Summary: | Total  Time
runtests.jl   |     0  0.0s
     Testing cuOpt tests passed 

Signed-off-by: Oscar Dowson <[email protected]>
@odow odow merged commit aa06a3a into jump-dev:main Dec 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants