-
Notifications
You must be signed in to change notification settings - Fork 746
{numlib}[foss/2024a] PETSc v3.22.0, ParMETIS v4.0.3, SuperLU_DIST v9.0.0 #21697
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
base: develop
Are you sure you want to change the base?
Conversation
removed |
you could take the tarball from this commit, which is the initial import of v4.0.3: i compared the contents with the original 4.0.3 tarball; the only important difference is that the commit does not contain the but the CP2K tarball has the same checksum as the original, so i'm not against just using that. let us know what you decide? |
I think it's better to use the static tarball for now, until KarypisLab/ParMETIS#27 and KarypisLab/ParMETIS#26 are resolved. |
@boegelbot please test @ jsc-zen3 |
@casparvl: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2977038463 processed Message to humans: this is just bookkeeping information for me, |
Hmmm, local build failed with:
|
Test report by @boegelbot |
Weird, the link https://web.cels.anl.gov/projects/petsc/download/release-snapshots/petsc-3.22.0.tar.gz looks ok, and if I use |
Test report by @casparvl |
Test report by @casparvl |
Updated software
|
|
Ah, that was causing the checks to fail, I was already surprised that a deprecation warning would cause a failure. I guess time has caught up with this PR :P I'll modify the versions, see if it still builds with the versions that are (by now) part of 2024a. |
@boegelbot please test @ jsc-zen3 |
@casparvl: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2996650328 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @casparvl |
# Author: Jasper Grimm (UoY) | ||
## | ||
name = 'PETSc' | ||
version = '3.22.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I realize this is an old PR, but I don't think we should be using a PETSc version that's older than what we're using in the 2023b
generation, see:
@maxim-masterov @casparvl Can we bump this version to latest (3.23.5)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably proceed with this PR instead:
easyblock = "EB_SuperLU" | ||
|
||
name = 'SuperLU_DIST' | ||
version = '9.0.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a 9.1.0
now, let's bump to that while we're also bumping the PETSc version?
(created using
eb --new-pr
)Note that ParMETIS was moved from glaros.dtc.umn.edu to GitHub. The former URL is not working anymore and the new GitHub repository contains no tags or releases (see this issue). After some browsing, I found the versioned tarball among the static downloads for CP2K, so I'm using it to get the package. Let me know if you have a better idea about where to fetch it from.
Depends on:mpi4py {lib}[gompi/2024a] mpi4py v4.0.0 #21563