add cuopt direct solver #3620
Open
add cuopt direct solver #3620
Conversation
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
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3620 +/- ##
==========================================
- Coverage 89.19% 85.24% -3.96%
==========================================
Files 896 896
Lines 103802 103846 +44
==========================================
- Hits 92589 88525 -4064
- Misses 11213 15321 +4108
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mrmundt
reviewed
Jun 17, 2025
|
@Iroy30 - We've been able to make cuopt available on our internal testing machines. Can you please add tests to this PR? |
Iroy30
force-pushed
the
add_cuopt_direct_solver_plugin
branch
from
5dbf9dd to
6f64094
Compare
|
@mrmundt Thanks! We have added tests by enabling testing cuopt with LP and MILP capabilities in tests/solvers.py. Let us know if:
The following is the testing output I get relevant to cuOpt |
mrmundt
reviewed
Jul 2, 2025
Comment on lines
60
to
62
| t0 = time.time() | ||
| self.solution = cuopt.linear_programming.solver.Solve(self._solver_model) | ||
| t1 = time.time() |
There was a problem hiding this comment.
This is fine, but just so you're aware, we have this lovely little utility called TicTocTimer that you may want to consider using: https://pyomo.readthedocs.io/en/latest/api/pyomo.common.timing.TicTocTimer.html
|
@mrmundt What do you think about including this solver interface in pyomo.contrib.solvers? Would it make sense to pull-in new solver interfaces there, since that's where the new solver API is evolving? |
@whart222 - I am evenly split. Because we are still messing with what the new solver interfaces are going to actually do / how they will handle input and present output, I don't know if we want to put "new" solvers there or just "well-established" ones that we can robustly test / really know what they are supposed to do and return. |
|
@Iroy30 - I forgot to post this last week, but all of the failures are of the variety: |
|
@Iroy30 - Two more things:
|
Co-authored-by: Miranda Mundt <[email protected]>
|
@mrmundt I don't think the fail we see is cuOpt related. Can you confirm? |
|
We are gearing up for launch of cuopt 25.10, it would be great if we could review this PR before the new release so that pyomo & cuopt are in sync |
|
This is very close to the top of my list. When is the cuopt release? |
|
Thanks @michaelbynum ! It is October 10th |
mrmundt
requested changes
Sep 25, 2025
There was a problem hiding this comment.
Thanks, @Iroy30 , for this PR! I have done an initial review. Some high-level things to consider:
- Logging -
loggeris created but then never used. Consider places it may be helpful - "Magic" numbers - there are quite a few instances of numbers that aren't defined being used. We love documentation!
Comment on lines
+213
to
+217
| if not flag: | ||
| raise RuntimeError( | ||
| "***The cuopt_direct solver plugin cannot extract solution suffix=" | ||
| + suffix | ||
| ) |
There was a problem hiding this comment.
It might be better to log loudly than to raise an error here (no strong feelings either way, just wanted to offer an alternative option).
jsiirola
requested changes
Sep 29, 2025
Comment on lines
+273
to
+274
| self.results.problem.upper_bound = solution.get_primal_objective() | ||
| self.results.problem.lower_bound = solution.get_primal_objective() |
There was a problem hiding this comment.
This implies convergence to a MIP gap of 0... I think this needs to be something like (assuming get_dual_objective() is a real thing):
Suggested change
| self.results.problem.upper_bound = solution.get_primal_objective() | |
| self.results.problem.lower_bound = solution.get_primal_objective() | |
| if self._solver_model.maximize: | |
| self.results.problem.upper_bound = solution.get_dual_objective() | |
| self.results.problem.lower_bound = solution.get_primal_objective() | |
| else: | |
| self.results.problem.upper_bound = solution.get_primal_objective() | |
| self.results.problem.lower_bound = solution.get_dual_objective() |
|
The errors we are seeing on Jenkins are segfaults: |
mrmundt
requested changes
Oct 14, 2025
|
@mrmundt Would it be possible to update CI to pull cuopt 25.10 |
|
@Iroy30 Yes (working on that now - it will take a couple hours to update everything and propagate it to the build machines)- but does that mean that this interface will enforce a minimum cuopt version? |
|
@jsiirola The segfaults you see in CI with 25.08 is concerning. Was the CI running the 25.08 release or the daily build of cuopt(this could cause the segfault). If 25.08 release was causing segfault then we might have to enforce minimum cuopt 25.10 and meanwhile I will debug 25.08 on my end because we were not seeing this previously |
The CI is running cuopt-cu12/25.8.0 from PyPI. I have added 25.10.0 (also from PyPI), but that won't make it to the build machines for testing until later tonight (it is waiting on finalizing the Pyomo 6.9.5 release) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Fixes #3626
Summary/Motivation:
Add cuOpt math optimization (includes LP and MILP) solver backend to Pyomo so users can solve pyomo models with cuOpt
Changes proposed in this PR:
Legal Acknowledgement
By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution: