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

Allow for MKL v2025 and remove superfluous loading of libblas and libgomp under linux #103

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

j-fu
Copy link
Collaborator

@j-fu j-fu commented Jan 23, 2025

No description provided.

@j-fu
Copy link
Collaborator Author

j-fu commented Jan 23, 2025

What do we bump here ? Patch version or minor version ? I guess patch version as minor would be breaking for now.

EDIT: I think due to @mipals remark in #104 it needs to be breaking as MKL drops 32bit support. So either it becomes 0.6, or, as I would prefer and we are breaking anyway, 1.0. Any comments on that ?

@j-fu j-fu mentioned this pull request Jan 23, 2025
@j-fu j-fu requested a review from ViralBShah January 23, 2025 17:02
@j-fu j-fu requested a review from KristofferC January 23, 2025 21:48
@j-fu
Copy link
Collaborator Author

j-fu commented Jan 23, 2025

@DmsPas can you confirm that Blas and openmp libs come bundeled with Pardiso if it is loaded via dlopen ?

@DmsPas
Copy link

DmsPas commented Jan 24, 2025

@j-fu

Yes, I can confirm that I confirm that for libpardiso.so in linux, BLAS and OpenMP are statically linked.

@j-fu j-fu changed the title Allow for MKL v2025 Allow for MKL v2025 and remove superfluous loading of libblas and libgomp under linux Jan 24, 2025
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

Successfully merging this pull request may close these issues.

2 participants