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

latest gsi develop version has trouble reading hirs4_metop-a.SpcCoeff.bin from crtm/2.4.0.1 #752

Open
jack-woollen opened this issue Jun 7, 2024 · 12 comments
Assignees

Comments

@jack-woollen
Copy link
Contributor

@jswhit @ADCollard Error message below from reanalysis merge with latest develop. Is there a new crtm version for this?

GLBSOI: jiter,jiterstart,jiterlast,jiterend= 1 1 2 1
SpcCoeff_ReadFile(Binary)(FAILURE) : Error reading channel data. input statement requires too much data, unit 10, file /lfs/h2/emc/ptmp/Jack.Woollen/runda/cfsv2/cdas1.163176/./crtm_coeffs/hirs4_metop-a.SpcCoeff.bin
CRTM_SpcCoeff_Load(FAILURE) : Error reading SpcCoeff file #1, ./crtm_coeffs/hirs4_metop-a.SpcCoeff.bin; Process ID: 88
CRTM_Init(FAILURE) : Error loading SpcCoeff data; Process ID: 88
crtm_interface*init_crtm: ERROR crtm_init error_status= 3
TERMINATE PROGRAM EXECUTION
MPICH ERROR [Rank 88] [job id 7ebc1672-c444-42a5-af33-272592e0b982] [Fri Jun 7 10:23:17 2024] [nid001178] - Abort(71) (rank 88 in comm 0): application called MPI_Abort(MPI_COMM_WORLD, 71) - process 88

@dtkleist
Copy link
Contributor

dtkleist commented Jun 7, 2024

I don't thinks so. @emilyhcliu -- any insight?

@emilyhcliu
Copy link
Contributor

emilyhcliu commented Jun 7, 2024 via email

@dtkleist
Copy link
Contributor

dtkleist commented Jun 7, 2024 via email

@emilyhcliu
Copy link
Contributor

emilyhcliu commented Jun 7, 2024 via email

@emilyhcliu
Copy link
Contributor

CRTM team knows about the issue in HIRS coefficients.
I will contact CRTM team to get their plans about fixing HIRS coefficients or check if there is a newer version with HIRS coefficiets fixed.

Will repot back to here.

@ADCollard
Copy link
Contributor

@emilyhcliu @jack-woollen @dtkleist Sorry I have not been checking e-mails the last couple of days. As @emilyhcliu said, this is a versioning issue with the HIRS coefficient files that came to light when the code needed to be updated to use more recent ATMS files.

I actually mentioned it in this CRTM PR , but it needs an explicit issue (depending on whether Emily had any luck with the CRTM team).

It might be possible to edit the HIRS files ourselves using the (possibly obsolete) tools in the CRTM. I can take a look at that on Monday.

One question, is there a requirement for reanalysis to use the operational installation of the CRTM fix files or could be point to another place for HIRS? I ask as, even if we are able to update the fix files, NCO has been insisting on a complete re-install of CRTM just to update fix files and that takes months.

@ADCollard
Copy link
Contributor

ADCollard commented Jun 11, 2024

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)

@RussTreadon-NOAA
Copy link
Contributor

@jack-woollen , @emilyhcliu , and @ADCollard : May this issue be closed?

@jack-woollen
Copy link
Contributor Author

@jswhit Is this issue settled?

@RussTreadon-NOAA
Copy link
Contributor

@jack-woollen and @jswhit , what is the status of this issue?

@jack-woollen
Copy link
Contributor Author

@RussTreadon-NOAA The fix worked, but it's patchwork. I think the issue will be settled in the next CRTM release. I'd say the status is betwixt and between. The gits issue is probably moot now.

@RussTreadon-NOAA
Copy link
Contributor

Thank you @jack-woollen . Sounds like GSI issue #783 can / will resolve this issue.

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

No branches or pull requests

5 participants