{bio,numlib}[foss/2023b] openCARP v17.0, PETSc v3.22.5#22696
{bio,numlib}[foss/2023b] openCARP v17.0, PETSc v3.22.5#22696boegel merged 10 commits intoeasybuilders:developfrom
Conversation
Updated software
|
contributed through PR easybuilders#22696) as suggested by @boegel
Co-authored-by: Kenneth Hoste <kenneth.hoste@ugent.be>
Co-authored-by: ocaisa <alan.ocais@cecam.org>
| # Author: Jasper Grimm (UoY) | ||
| ## | ||
| name = 'PETSc' | ||
| version = '3.22.3' |
There was a problem hiding this comment.
@MarieHouillon Any particular reason you're sticking to PETSc 3.22.3 while 3.22.5 is already available for a while?
There was a problem hiding this comment.
It's the version we use in our Docker image and for packaging currently, but we could certainly also use 3.22.5
There was a problem hiding this comment.
I think if we are going to introduce a new PETSc into a toolchain, we should stick to including the latest version (particularly when it is patch release).
There was a problem hiding this comment.
In fact 3.23.1 is available at this stage, thoughts @boegel ?
There was a problem hiding this comment.
For us 3.22.5 is better, as 3.23 introduced changes that are not compatible with openCARP 17.0 without a patch.
There was a problem hiding this comment.
I've updated the PETSc version to 3.22.5 as suggested, please let me know if anything else should be done.
|
Test report by @boegel |
Co-authored-by: ocaisa <alan.ocais@cecam.org>
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2905874333 processed Message to humans: this is just bookkeeping information for me, |
|
CI fails due to missing dependency: @MarieHouillon I can't seem to find another PR that provides this, what am I overlooking? |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
Going in, thanks @MarieHouillon! |
…ncludes 3.22.5 version)
(created using
eb --new-pr)