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

Require parallelio 2.5.2 #30

Closed
harshula opened this issue Jan 22, 2024 · 2 comments
Closed

Require parallelio 2.5.2 #30

harshula opened this issue Jan 22, 2024 · 2 comments
Assignees
Labels

Comments

@harshula
Copy link
Contributor

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:

# The project version number.
set(VERSION_MAJOR   2   CACHE STRING "Project major version number.")
set(VERSION_MINOR   5   CACHE STRING "Project minor version number.")
set(VERSION_PATCH   2   CACHE STRING "Project patch version number.")
mark_as_advanced(VERSION_MAJOR VERSION_MINOR VERSION_PATCH)

I did not require ^parallelio@2.5.2 during my testing. As a result Spack defaulted to parallelio@2.5.10. We should be doing the following from now on:

spack install access-om2 ^netcdf-c@4.7.4 ^netcdf-fortran@4.5.2 ^parallelio@2.5.2 ^nci-openmpi@4.0.2 %intel@19.0.5.281
@harshula harshula self-assigned this Jan 22, 2024
@aidanheerdegen
Copy link
Member

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
harshula added a commit that referenced this issue Jan 23, 2024
@harshula
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants