[KNITRO] Add warm start support#3833
Conversation
… into knitro-warm-start
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3833 +/- ##
==========================================
- Coverage 89.94% 89.93% -0.01%
==========================================
Files 902 902
Lines 106354 106369 +15
==========================================
+ Hits 95655 95664 +9
- Misses 10699 10705 +6
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:
|
michaelbynum
left a comment
There was a problem hiding this comment.
This PR looks great, but it raised a broader design discussion for solver configs (see #3933 ). We are trying to figure out how to have common config options among subsets of solvers. For this PR, I think we just rename use_start to be knitro_use_warmstart so that it is clear that this option is specific to knitro. With that change, I think this will be ready to go.
Fixes # .
Summary/Motivation:
This PR implements the warm-start procedure for the KNITRO solver.
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: