coin-or / SHOT

A solver for mixed-integer nonlinear optimization problems
https://shotsolver.dev
Eclipse Public License 2.0
117 stars 25 forks source link

Reenable primal starting point functionality in Ipopt #172

Closed andreaslundell closed 7 months ago

andreaslundell commented 7 months ago

Does this fix the Valgrind errors in https://github.com/coin-or/SHOT/pull/171/files/db906f48a5086fcf698292d83133ff52e44c16e8#r1352778767 @svigerske?

svigerske commented 7 months ago

yes, that works for me

svigerske commented 7 months ago

The title of the pull request is a bit confusing, though. What the PR reenables is that you set values for the (primal) starting point of Ipopt. Without it, Ipopt would start from random values given by uninitialized memory. A warmstart would require values for the dual variables. You didn't enable this option in Ipopt, so not providing them is correct.