-
Notifications
You must be signed in to change notification settings - Fork 35
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
Precompilation failure on v1.8 (related to MUMPS_seq_jll?) #203
Comments
cc @amontoison here's another MUMPS_seq_jll issue on Julia v1.8. x-ref: jump-dev/Ipopt.jl#327 For now a work-around is probably ] add Cbc MUMPS_seq_jll@5.4.1 There's some issue with the v5.5.1 version of MUMPS_seq_jll on Windows in Julia 1.8. |
@odow One issue comes from the PR of @blegat JuliaPackaging/Yggdrasil#4806 |
How does that PR affect |
@blegat |
@odow |
Actually, here's one from yesterday: https://github.com/jump-dev/Cbc.jl/runs/8060859973?check_suite_focus=true, which uses |
@thchr what happens if you just run |
I just had a chat to @amontoison and we think we have a patch to fix this issue: JuliaRegistries/General#67310 It's unrelated to Ipopt. |
That throws on the Stack trace```jl julia> using MUMPS_seq_jll ERROR: InitError: could not load library "C:\Users\tchr\.julia\artifacts\029cea9e5bf8cca60ec7023a09ff1c9a45091f9a\bin\libcmumps.dll" The specified module could not be found. Stacktrace: [1] dlopen(s::String, flags::UInt32; throw_error::Bool) @ Base.Libc.Libdl .\libdl.jl:117 [2] dlopen(s::String, flags::UInt32) @ Base.Libc.Libdl .\libdl.jl:116 [3] macro expansion @ C:\Users\tchr\.julia\packages\JLLWrappers\QpMQW\src\products\library_generators.jl:54 [inlined] [4] __init__() @ MUMPS_seq_jll C:\Users\tchr\.julia\packages\MUMPS_seq_jll\ziXhU\src\wrappers\x86_64-w64-mingw32-libgfortran5.jl:17 [5] _include_from_serialized(pkg::Base.PkgId, path::String, depmods::Vector{Any}) @ Base .\loading.jl:831 [6] _require_search_from_serialized(pkg::Base.PkgId, sourcepath::String, build_id::UInt64) @ Base .\loading.jl:1039 [7] _require(pkg::Base.PkgId) @ Base .\loading.jl:1315 [8] _require_prelocked(uuidkey::Base.PkgId) @ Base .\loading.jl:1200 [9] macro expansion @ .\loading.jl:1180 [inlined] [10] macro expansion @ .\lock.jl:223 [inlined] [11] require(into::Module, mod::Symbol) @ Base .\loading.jl:1144 during initialization of module MUMPS_seq_jll ```
This works without a hitch :) Thanks for looking at a fix for this so quickly! |
Okay, the problem is definitely that MUMPS_seq_jll v5.5.1 is broken on Windows. @amontoison has a plan to recompile things, but the patch to the General registry will prevent people installing Cbc from getting the broken version of MUMPS. Another work-around is to use HiGHS instead. At this point, I would encourage all users of Cbc to switch to HiGHS. |
@thchr can you try now in a fresh environment? The registry should be updated. |
Works now! Thanks! I tried the HiGHS solver per your suggestion, but it did not seem generically faster than Cbc for the problems I had (MILP via LayeredLayouts.jl): what's the rationale for switching? |
HiGHS is supported and actively developed. |
I'm having trouble precompiling this package on Julia v1.8 (everything works fine on v1.7); my troubles seem related to MUMPS_seq_jll.
Specifically, if I try to add Cbc (in a completely clean package environment), precompilation fails for several jll dependencies of Cbc:
Additionally MUMPS_seq_jll is marked as outdated:
The versions of Cbc, Clp_jll, Cgl_jll, and Cbc_jll are:
Full stack trace from precompilation below:
Precompilation stack trace
On the other hand, if I first install MUMPS_seq_jll manually, precompilation succeeds:
But this causes the previously failing Cbc jll dependencies to be marked as outdated:
System info (Windows)
The text was updated successfully, but these errors were encountered: