-
Notifications
You must be signed in to change notification settings - Fork 47
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
Incorrect CRTM AMSU-A Spectral Coeff file on hera #1046
Comments
@DavidHuber-NOAA Could you take a look at this if you have time? |
Hercules check Log into Hercules. Load GSI
Thank you @ADCollard for reporting this problem. This issue prompted me to check GSI regression tests. The global_4denvar test sets |
@AlexanderRichert-NOAA @climbfuji @ADCollard I verified this does need to be fixed. The procedure outlined in #963 should work fine again here. This also needs to be fixed on Jet Rocky 8 installations. |
@RatkoVasic-NOAA Can you work with @DavidHuber-NOAA to fix this, please? |
@RatkoVasic-NOAA Apologies for not bringing this up in the spack-stack meeting. Is this something that can be addressed? |
@DavidHuber-NOAA does this mean we just have to copy amsua_metop-c.SpcCoeff.bin to all our installations? If yes, which one we know is correct? And I can use that one to populate to all environments/versions. |
@RatkoVasic-NOAA This only applies to spack-stack v1.6.0 Rocky 8 installations on Hera and Jet and only needs to be applied to the unified-env-rocky8 environment (the gsi-addon-rocky8 uses the unified-env-rocky8 installation of crtm-fix). A good copy of the fix file can be found here: |
@DavidHuber-NOAA thanks! I'm going to update it. |
@DavidHuber-NOAA BTW, I checked other files as well. Beside amsua_metop-c.SpcCoeff.bin, there are two more files that differ: |
@RatkoVasic-NOAA I'm not certain on this. |
@RatkoVasic-NOAA @DavidHuber-NOAA Thanks for doing this! This whole sorry saga started with atms_n21! Hopefully for 2.4.0.1 the cksums should be as follows:
|
@ADCollard thanks, it is correct in spack-stack-1.6.0/crtm-fix-2.4.0.1 |
The checksums and sizes are both correct now:
I think this can now be closed. Thanks @RatkoVasic-NOAA! |
Thank you both! |
This is a similar issue to that already addressed in Issue 963
The AMSU-A Metop-C spectral coefficient file needs special treatment as the one supplied with the default file name is not correct. The details on what to do are in that issue.
The correct file has the following
cksum
:This one is from orion at
/work/noaa/epic/role-epic/spack-stack/orion/spack-stack-1.6.0/envs/unified-env/install/intel/2022.0.2/crtm-fix-2.4.0.1_emc-ezbyzji/fix
On Hera at
/scratch1/NCEPDEV/nems/role.epic/spack-stack/spack-stack-1.6.0/envs/unified-env-rocky8/install/intel/2021.5.0/crtm-fix-2.4.0.1_emc-bm46d3q/fix
(which is the path given by $CRTM_FIX when the appropriate modules are loaded as specified in the develop version of GSI) thecksum
is:This causes a failure when trying to run the develop version of GSI.
The text was updated successfully, but these errors were encountered: