bump existing SPH-EXA easyconfigs to v0.96.2#25606
Conversation
Updated software
|
|
The previous versions were only merged last week and are not in an EB release, let's remove those as part of this PR so we just retain the latest version? |
|
@ocaisa I have removed previous release as suggested |
ocaisa
left a comment
There was a problem hiding this comment.
LGTM these original versions were only merged last week so safe to replace
|
@boegelbot please test @ jsc-zen3-a100 |
|
@ocaisa: 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 4125432886 processed Message to humans: this is just bookkeeping information for me, |
|
@ocaisa is this one ok? I will update the EESSI PRs as soon as this one gets merged |
|
Just waiting on the bot to report back, I guess build jobs are backed up |
That's going to take a while, the only GPU node we have in I'll submit a test report from our GPU cluster so this can be merged... |
Thank you! |
|
Test report by @boegel |
am I missing something? |
|
@boegel I didn't try a local build because I was busy and I assumed such a small upgrade won't break anything but I guess I was wrong I will debug it locally and reply here as soon as I get rid of my boring meetings |
|
Test report by @boegelbot |
|
Seems the dependency |
|
I could reproduce the build error locally and I have reported it in sphexa-org/sphexa#587 |
|
We've had to patch for that issue in the past I wonder are they (or the h5hut dependency they pull in) vendoring hdf5? |
ocaisa
left a comment
There was a problem hiding this comment.
Let's see if we can just disable using static libraries
Co-authored-by: ocaisa <alan.ocais@cecam.org>
|
@boegelbot please test @ jsc-zen3-a100 |
|
@ocaisa: 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 4207075065 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Ah, it was only the |
|
Hmm, this looks like a smoking gun: H5hut/H5hut#5 |
|
That might actually be the fix that is needed: sphexa-org/sphexa#588 |
|
Ok, I found it. They are pulling in a branch at build time for their h5hut dependency https://github.com/eth-cscs/h5hut/commits/cmake/ , this should be (at least) pinned to a commit |
|
@pescobar you need a patch to pin https://github.com/sphexa-org/sphexa/blob/develop/CMakeLists.txt#L87 to a commit (probably best is eth-cscs/h5hut@74cd84c) |
|
The other option is to disable h5hut altogether |
Co-authored-by: ocaisa <alan.ocais@cecam.org>
|
@boegelbot please test @ jsc-zen3-a100 |
|
@ocaisa: 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 4207607569 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@ocaisa thanks for finding where the problem is, I was expecting a fix from the developers and I didn't look at it I have managed to build the application with I am going to create a new PR with the working easyconfig |
edit: follow-up to: