-
Notifications
You must be signed in to change notification settings - Fork 0
[WIP] MOI wrapper test (some failed) #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: MOI
Are you sure you want to change the base?
Conversation
| return MOI.NO_SOLUTION | ||
| end | ||
| return _DUAL_STATUS_MAP[optimizer.result.termination_reason] | ||
| return _PRIMAL_STATUS_MAP[optimizer.result.termination_reason] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It cannot be the same as the _PRIMAL_STATUS_MAP because for Lib.TERMINATION_REASON_PRIMAL_INFEASIBLE, the dual status should be MOI.INFEASIBILITY_CERTIFICATE
|
|
||
| using cuPDLPx_jll | ||
| # TODO: I have to add this in my local environment. This might be a issue in JLL. | ||
| const libcupdlpx = "/home/zdpeng/.julia/artifacts/4d407e51174c3bfe2f138e6e1db2531d0bc6240d/lib/libcupdlpx.so" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually had the same issue and it prevented me to run it locally, for me, it seems to be because I have CUDA v13.0
$ nvidia-smi
Thu Dec 18 08:27:35 2025
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 580.105.08 Driver Version: 580.105.08 CUDA Version: 13.0 |
+-----------------------------------------+------------------------+----------------------+
julia> CUDA.versioninfo()
CUDA toolchain:
- runtime 13.0, artifact installation
- driver 580.105.8 for 13.0
- compiler 13.1
and cuPDLPx is only available on CUDA up to v12.9:
https://github.com/JuliaBinaryWrappers/cuPDLPx_jll.jl/tree/main/src/wrappers
The issue seems to be this line:
https://github.com/JuliaPackaging/Yggdrasil/blob/96df50e83da84ec3a4c48b4d9360abaa9c5a843d/C/cuPDLPx/build_tarballs.jl#L51
If you can run it locally and show me test errors here I can help. Otherwise, if the jll is fixed for CUDA v13 then I will be able to run the tests locally and it should be more efficient
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! I’ll update build_tarballs.jl to add CUDA 13 support.
For the const libcupdlpx = "/home/zdpeng/.julia/artifacts/..." workaround: I asked Gemini and it suggested changing LibraryProduct("libcupdlpx", :libcupdlpx), to LibraryProduct("cupdlpx", :libcupdlpx), in build_tarballs.jl.
Do you think that makes sense here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not an expert in BinaryBuilder but looking at the docs, the libname should be libcupdlpx here:
https://docs.binarybuilder.org/stable/reference/#BinaryBuilderBase.LibraryProducthttps://docs.binarybuilder.org/stable/reference/#BinaryBuilderBase.LibraryProduct
Hi @blegat, thank you very much for your help with the MOI wrapper.
Following your instructions, I’ve tested the MOI wrapper. Could you please take a look at the implementation when you have a chance?
The tests are now able to run, although some currently fail due to issues in cuPDLPx itself. I’ll update the source code to fix these shortly.