Skip to content
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

Failure loading libblastrampoline on julia v1.9 #11

Closed
baggepinnen opened this issue May 11, 2023 · 4 comments
Closed

Failure loading libblastrampoline on julia v1.9 #11

baggepinnen opened this issue May 11, 2023 · 4 comments

Comments

@baggepinnen
Copy link
Contributor

Hello Andreas, hope you're doing well!

A user of ControlSystems.jl recently reported an error calling lsminreal in Julia v1.9 running on Windows.
The error looks like this
error

(minreal is a thin wrapper around MatrixPencils.lsminreal)

I suspect the error stems from some manual management of the lapack installation in this package, a search came up with this line mentioning libblastrampoline explicitly

const liblapack = VERSION < v"1.7" ? Base.liblapack_name : "libblastrampoline"

@andreasvarga
Copy link
Owner

I already mentioned this issue here. My hope was that it will be automatically addressed with Julia 1.9, but apparently it was not. Any proposal how to proceed further with this?

@baggepinnen
Copy link
Contributor Author

This comment

It is not, that's the reason why we have Base.libblas_name: JuliaLang/julia#48435.

makes me think that this line

const liblapack = VERSION < v"1.7" ? Base.liblapack_name : "libblastrampoline"

should remove the version check and always use Base.liblapack_name, since this variable will have the correct value for all OS platforms, see here

@andreasvarga
Copy link
Owner

Thanks. I implemented your proposal and performed exhaustive tests for Julia 1.6-1.10 on both Windows and Linux. The version v1.7.7 will fix this problem for all Julia versions.

The same modification needs to be also performed to MatrixEquations.

@baggepinnen
Copy link
Contributor Author

Thanks for fixing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants