-
-
Notifications
You must be signed in to change notification settings - Fork 411
CuOpt Solver Integration #834
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
Comments
@Iroy30 @bbozkaya I have talked to Andrew Mason about this let me know if you need any extra discussion/ guidence @pchtsp is doing most of the work on pulp ATM but I would suggest something like the HiGHS interface https://github.com/coin-or/pulp/blob/d4763a873a799212949ebd138199c1f41c686042/pulp/apis/highs_api.py Would be ideal especially the use of a separate project highspy in that case to contain the binaries and the build. |
Thanks! Here is the PR #837 |
@Iroy30 how do we install cuopt for testing? |
The testing is a challenge. Getting the cuOPT binaries should be no problem, but having access to the right hardware is an issue. We're working on that. In the meantime, I indicated that if they can do extensive testing themselves and provide the logs from their own internal CI showing that everything seems to work, then I guess we could probably accept the PR. It's a low risk, since this interface will anyway only be used by people who have gotten cuOPT working and they will understand that it is "beta," hopefully reporting back any problems. It won't break anything for anyone else. |
I left some comments on the PR: #837 https://github.blog/changelog/2024-07-08-github-actions-gpu-hosted-runners-are-now-generally-available/ |
Now that you mention that, I remember the announcement of GPU-hosted runners. It seemed to me they were only available to paid accounts, but I can't find that anywhere. In any case, I suppose NVIDIA should have a paid account and should have access. In that case, I guess the setup could be added to the PR. @Iroy30 , what do you think? |
Indeed, it seems that the GPU Github runners are only for paying customers. I have access to a GPU cluster and also have a GPU that will work for testing on my laptop if it comes to it. |
I created a discussion in github actions to confirm we cannot get it for free: https://github.com/orgs/community/discussions/159224 |
Is it ok to go with @tkralphs' suggestion that we do the extensive testing on our end and share results and logs with you for your review and approval? |
Yes, it's okay. Although you first need to make the requested changes to the PR. |
What is your question
Nvidia cuOpt is looking forward to adding a cuOpt solver integration to pulp. We already have a PR developed and tested locally - needs some fine tuning and final updates. While we do that we were going through the contribution page and we have a few questions:
The text was updated successfully, but these errors were encountered: