-
Notifications
You must be signed in to change notification settings - Fork 57
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
Support the new precompiled Ipopt with LBT #369
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #369 +/- ##
=======================================
Coverage 94.02% 94.02%
=======================================
Files 4 4
Lines 921 921
=======================================
Hits 866 866
Misses 55 55
☔ View full report in Codecov by Sentry. |
Looks like an issue with spral:
I'd rather not need to set environment variables. have you tested spral? Perhaps we could just stick with mumps as the default? |
Is this going to fix our Windows binary issue? |
I thought that MUMPS was still the default linear solver: The plan is to keep MUMPS as the default linear solver. SPRAL is working if we define the following environment variables before that we start Julia:
but it's not user-friendly (ralna/spral#25 (comment)). This new artifact is compiled with LBT and without some link flags that were not relevant so It could solve the issue with the binary on Windows. |
Seems like we could also set the solver ourselves inside Ipopt.jl? |
Can we do that by default everytime that we create a JuMP model? |
You could do it here in Lines 274 to 292 in f66f888
|
Stefan released Ipopt 3.14.13 this morning. MUMPS will be the default linear solver now if SPRAL is installed. |
@odow