Fix version number in HIRS binary SpcCoef files
Opened this issue · 3 comments
ADCollard commented
This is a standalone issue created following a comment in CRTM PR#42.
The HIRS SpcCoeff files have a range of version numbers, it is not clear what the numbers mean.
To be read in correctly the version number for these files should be 8.02, as larger numbers require the presence of the PolAngle. The current version numbers for HIRS cause the code to crash.
ADCollard commented
Here is the version numbers in the current fix files:
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n06.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.02
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n07.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.02
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n08.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.02
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n09.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n10.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n11.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n12.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_n14.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2_tirosn.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.02
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n06.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.01
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n07.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.01
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n09.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n10.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n11.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n12.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs2-UWS_n14.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3_n15.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3_n16.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3_n17.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3-UWS_n15.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3-UWS_n16.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs3-UWS_n17.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4_metop-a.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4_metop-b.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.02
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4_n18.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4_n19.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.04
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4-UWS_metop-a.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.06
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4-UWS_metop-b.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.01
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4-UWS_n18.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
/scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs4-UWS_n19.SpcCoeff.bin
SpcCoeff RELEASE.VERSION: 8.05
ADCollard commented
I used a modified version of SpcCoeff_Inspect.f90 to modify the version numbers that were causing the problems.
The modified files are on Hera at /scratch1/NCEPDEV/da/Andrew.Collard/git/CRTM_REL-2.4.0_emc/fix/SpcCoeff/Big_Endian/hirs_fix (big endian only).
BenjaminTJohnson commented
Currently in China, will review next week.
Thanks!