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 to work with CRTMv3.1.1 #783

Open
emilyhcliu opened this issue Aug 18, 2024 · 3 comments
Open

Update GSI to work with CRTMv3.1.1 #783

emilyhcliu opened this issue Aug 18, 2024 · 3 comments
Assignees

Comments

@emilyhcliu
Copy link
Contributor

emilyhcliu commented Aug 18, 2024

Update GSI cde to with with CRTMv3.1.1
Users need to load Aerosol Coefficients when Cloud Coefficients are loaded
https://github.com/JCSDA/CRTMv3/blob/229ff8935976b152e8174473aa3d0250560b29e9/src/CRTM_K_Matrix_Module.f90#L710-L717

The above code block indicates that if the cloud coefficients are loaded, the aerosol coefficients need to be loaded as well. So, I modified GSI to load the aerosol coefficients for all-sky assimilation of ATMS and AMSU-A, for which the cloud coefficients are required. This resolved the issue of n_phase_elenemts difference between cloud and aerosol coefficients.

Note - Validate CRTMv3.1.1 with CRTMv2.4.0.1 (used in the operational system) with global GFS model background is documented in CRTMv3 repository Issue #167

@emilyhcliu emilyhcliu self-assigned this Aug 18, 2024
@emilyhcliu emilyhcliu changed the title Update GSI to work with CRTMv3.1.1 and Validate CRTMv3.1.1 with global GFS background Update GSI to work with CRTMv3.1.1 Aug 19, 2024
@RussTreadon-NOAA
Copy link
Contributor

@emilyhcliu , what is the status of this issue?

@CatherineThomas-NOAA , should we add this issue to the GFS v17 milestone?

@CatherineThomas-NOAA
Copy link
Collaborator

Added. Thanks @RussTreadon-NOAA.

@emilyhcliu
Copy link
Contributor Author

@emilyhcliu , what is the status of this issue?

@CatherineThomas-NOAA , should we add this issue to the GFS v17 milestone?

@RussTreadon-NOAA I will open a PR for this today. This will be very minor changes in crtm_interface.f90

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

3 participants