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

Update GSI hash #548

Closed
danielabdi-noaa opened this issue Jan 19, 2023 · 4 comments · Fixed by #800
Closed

Update GSI hash #548

danielabdi-noaa opened this issue Jan 19, 2023 · 4 comments · Fixed by #800
Labels
enhancement New feature or request

Comments

@danielabdi-noaa
Copy link
Collaborator

danielabdi-noaa commented Jan 19, 2023

Description

GSI has been updated to use ncdiag from hpc-stack instead of shipping it with the repository.
SRW should load ncdiag/1.0.0 in srw_common and make sure GSI builds correctly with it.
I have encountered some errors while triying to upgrade GSI hash because it is needed for RRFS_dev1 merge work.
If someone familiar with GSI can take a look at this, it would be very helpful!

Solution

Update GSI hash to latest and make sure SRW builds GSI properly.

Requirements**

None

Acceptance Criteria (Definition of Done)

GSI hash is updated and it builds correctly.

@danielabdi-noaa danielabdi-noaa added the enhancement New feature or request label Jan 19, 2023
@MichaelLueken
Copy link
Collaborator

@danielabdi-noaa Just a few questions:

  1. Are you wanting to update the GSI using NOAA-EMC/GSI or NOAA-GSL/GSI?
  2. Are you encountering the failure in the GSI or the EnKF?

Adding ncdiag/1.0.0 to srw_common.lua, I was able to build the GSI, but the EnKF is failing with the following message:

error #6405: The same named entity from different modules and/or program units cannot be referenced. [MPI_COMM_WORLD]

Cloning NOAA-EMC/GSI, I was able to get the GSI and EnKF to build without issue. However, when updating the compiler from Intel 18 (what the GSI is currently using) to Intel 2022.1.2, the FV3REG build of the EnKF failed with the exact same error. This will likely require an update to the GSI itself to correct this behavior.

@MichaelLueken
Copy link
Collaborator

@danielabdi-noaa Using the fix that @hu5970 made in the NOAA-GSL/GSI/rrfs_dev branch at 9b6bf7b, I was able to get the GSI to build using Intel 2022.1.2. I should also note that the GSI will likely not run using the Intel 2022.1.2 compiler. Please see Issue #447 for more details.

@hu5970 Would it be possible to add this fix to the authoritative NOAA-EMC/GSI repository?

@dshawul
Copy link
Contributor

dshawul commented Jan 19, 2023

@MichaelLueken Thanks a lot! I was encountering that same issue but thought I had made a mistake somewhere. I think latest EMC/GSL branch will work with RRFS_dev since they are almost in sync, but just to be safe we can set it to GSL's current hash. The current hash of GSI we have in SRW app has some issues with RRFS_dev workflow.

@christinaholtNOAA
Copy link
Collaborator

christinaholtNOAA commented Jan 19, 2023

I don't think that we want to use forks of authoritative repos in any case (for authoritative SRW). Changes really must be in the appropriate authoritative repositories/branches for all of this to work smoothly.

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

Successfully merging a pull request may close this issue.

4 participants