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

Merged
merged 6 commits into from
Jan 28, 2025

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
@j-fu
Copy link
Collaborator Author

j-fu commented Jan 28, 2025

Had another look - for me good to go...

@KristofferC
Copy link
Member

The CHANGELOG is a bit strange, I am not sure it is very useful for anyone. I think it is enough to just have a small note saying in perhaps the README saying that 1.0 dropped 32-bit support.

- Remove everything before 0.5
- Keep the logs of the most important developments
- Say what is breaking in 1.0
@j-fu
Copy link
Collaborator Author

j-fu commented Jan 28, 2025

I would like to keep the changelog (stripped down now, agree that the autorgenerated stuff was too much). And I want to refer to the CHANGELOG.md in the (now mandatory) release notes for package registration for the breaking change.

@ViralBShah
Copy link
Member

ViralBShah commented Jan 28, 2025

Looks good - but I am not a real user of Pardiso, so not up to date with the changes in MKL.

@j-fu j-fu merged commit e646342 into master Jan 28, 2025
9 checks passed
@j-fu j-fu deleted the jf/MKL2025 branch January 28, 2025 20:26
j-fu added a commit that referenced this pull request Jan 28, 2025
This complements #2797463 wherein loading libblas and libgomp has
been removed from the code as they are now distributed with the
library (confirmed by upstream in
#103 (comment))
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.

4 participants