-
Notifications
You must be signed in to change notification settings - Fork 0
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
Require parallelio 2.5.2 #30
Labels
Comments
Good catch! It's a little depressing a newer version degraded performance. This is exactly the sort of situation ("upgrade" downgrading performance) we would want to catch with performance CI checks. |
harshula
added a commit
that referenced
this issue
Jan 23, 2024
Merged
harshula
added a commit
that referenced
this issue
Jan 23, 2024
Corrected the instructions at: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Looking at the COSIMA ACCESS-OM2 source and the git submodules:
https://github.com/COSIMA/access-om2/tree/master/src
https://github.com/COSIMA/cice5/tree/edcfa6f9c76ed05b63196ce4b5355fa5a8f4fe3a
https://github.com/NCAR/ParallelIO/tree/7e242f78bd1b4766518aff44fda17ff50eed6188
we find that COSIMA ACCESS-OM2 uses
parallelio 2.5.2
, https://github.com/NCAR/ParallelIO/blob/7e242f78bd1b4766518aff44fda17ff50eed6188/CMakeLists.txt#L11-L15:I did not require
^parallelio@2.5.2
during my testing. As a result Spack defaulted toparallelio@2.5.10
. We should be doing the following from now on:The text was updated successfully, but these errors were encountered: