Upgrade Ipopt & CoinHSL #137
Conversation
jaelynlitz
force-pushed
the
upgrade-ipopt-coinhsl
branch
from
5975de1
to
578fba0
Compare
|
@jaelynlitz are you able to take a look at why coinhsl, ipopt and suite-sparse are failing to build? I am happy to take this off your plate if you're unable to get to it |
| require: '@3.12.10~metis+coinhsl~mumps' | ||
| require: '~metis+coinhsl~mumps' |
There was a problem hiding this comment.
I believe you need to build coinhsl with Metis support. Otherwise the linear solver and by extension Ipopt will have diminished performance.
| @@ -24,9 +24,9 @@ spack: | |||
| magma: | |||
| require: '@2.6.2' | |||
| coinhsl: | |||
| require: '@2019.05.21' | |||
| require: '@2023.11.17' | |||
There was a problem hiding this comment.
| require: '@2023.11.17' | |
| require: '@2023.11.17+metis' |
@pelesh per your suggestion
There was a problem hiding this comment.
Or is it sufficient just to build Ipopt with metis... Or do we need both? Should the spack package force coinhsl+metis if ipopt+metis for these later versions?
There was a problem hiding this comment.
Ipopt does not depend on Metis, only HSL does. I am not sure what ipopt+metis is supposed to do tbh.
Forcing coinhsl+metis is the right thing to do imho.
Merge request type
Relates to
This MR updates
Summary