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

Add support for CADS IR cloud detection scheme in the GSI #2478

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

wx20jjung
Copy link
Contributor

@wx20jjung wx20jjung commented Apr 10, 2024

A new infrared cloud detection scheme "CADS" was added to the GSI. These changes allow the use of this new scheme within the global-workflow. Setting the namelist variables to .true. (e.g. cris_cads=.true.) will envoke the the new scheme for that instrument.

Resolves #2473

Type of change

  • New feature (adds functionality)

Change characteristics

  • Is this a breaking change (a change in existing functionality)? NO
  • Does this change require a documentation update? NO

How has this been tested?

Ran several cycles on Hera with the default cloud detection scheme and CADS.

…nd true/false flags to us CADS for airs, cris, and iasi instruments.
@WalterKolczynski-NOAA WalterKolczynski-NOAA changed the title changes needed to use a new infrared cloud detection scheme in the GSI Add support for CADS IR cloud detection scheme in the GSI Apr 10, 2024
@WalterKolczynski-NOAA
Copy link
Contributor

@wx20jjung Is there a GSI update that needs to be included in this PR? If so, please mark it as a draft until ready.

@wx20jjung
Copy link
Contributor Author

Walter,

The GSI develop branch being used by the global-workflow already has the necessary CADS code. Everything should be complete. I have tested both the default IR cloud detection scheme and the new CADS on Hera. Both work as expected.
Jim

@WalterKolczynski-NOAA
Copy link
Contributor

@wx20jjung Thanks for the confirmation. Please update your branch to the current develop and we will run CI tests.

Copy link
Contributor

@WalterKolczynski-NOAA WalterKolczynski-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conditionally approved pending successful completion of CI tests.

@wx20jjung
Copy link
Contributor Author

Walter,

CADS should be up to date now with global-workflow:develop

@WalterKolczynski-NOAA
Copy link
Contributor

Walter,

CADS should be up to date now with global-workflow:develop

Did you push them? I still have a button that says "Update branch" and I don't see any new commits.

@wx20jjung
Copy link
Contributor Author

Sorry. Please see if I did it properly this time.

@WalterKolczynski-NOAA WalterKolczynski-NOAA added CI-Hera-Ready **CM use only** PR is ready for CI testing on Hera CI-Orion-Ready **CM use only** PR is ready for CI testing on Orion and removed CI-Hera-Ready **CM use only** PR is ready for CI testing on Hera labels Apr 10, 2024
@emcbot emcbot added CI-Orion-Building **Bot use only** CI testing is cloning/building on Orion and removed CI-Orion-Ready **CM use only** PR is ready for CI testing on Orion labels Apr 10, 2024
@TerrenceMcGuinness-NOAA TerrenceMcGuinness-NOAA added CI-Orion-Passed **Bot use only** CI testing on Orion for this PR has completed successfully and removed CI-Orion-Building **Bot use only** CI testing is cloning/building on Orion labels Apr 11, 2024
@TerrenceMcGuinness-NOAA
Copy link
Collaborator

All scheduled CI test cases passed (updated label).

@WalterKolczynski-NOAA WalterKolczynski-NOAA merged commit 8edf94a into NOAA-EMC:develop Apr 11, 2024
7 of 9 checks passed
danholdaway added a commit to danholdaway/global-workflow that referenced this pull request Apr 15, 2024
* upstream/develop:
  Update fbwind for COM refactor (NOAA-EMC#2479)
  Update parm files for atm product (NOAA-EMC#2486)
  Fix paths in prepocnobs task (NOAA-EMC#2459)
  Add oceanalecn to workflow generator (NOAA-EMC#2409)
  Add support for CADS IR cloud detection scheme in the GSI (NOAA-EMC#2478)
  Improve error messaging to resolve bugzilla (NOAA-EMC#2468)
  Check the DO_JEDISNOWDA condition before adding snow DA analysis to the list (NOAA-EMC#2471)
@RussTreadon-NOAA
Copy link
Contributor

@WalterKolczynski-NOAA and @wx20jjung : The changes to scripts/exglobal_atmos_analysis.sh in this PR include three new fix files

${NLN} ${FIXgfs}/gsi/AIRS_CLDDET.NL   AIRS_CLDDET.NL
${NLN} ${FIXgfs}/gsi/CRIS_CLDDET.NL   CRIS_CLDDET.NL
${NLN} ${FIXgfs}/gsi/IASI_CLDDET.NL   IASI_CLDDET.NL

Where do these files reside? I do not see these fix files in GSI-fix or in EIB maintained FIX_DIR. Am I looking in the wrong places?

@wx20jjung
Copy link
Contributor Author

wx20jjung commented Apr 22, 2024 via email

@RussTreadon-NOAA
Copy link
Contributor

@wx20jjung , thank you for pointing me at the required CADS fix files. After looking at the CLDDET.NL files in /scratch1/NCEPDEV/jcsda/Jim.Jung/save/global-workflow/fix, these appear to be namelists, not fix files. Is this correct?

@RussTreadon-NOAA
Copy link
Contributor

@wx20jjung , thank you for stopping by my cubicle. Looking at gsi_enkf.fd/src/gsi/cads.f90 confirms that the CLDDT.NL files are fortran namelists. Subroutine CADS_Setup_Cloud in cads.fd90 reads the contents of these files as namelist Cloud_Detect_Coeffs.

A closer check of GSI-fix:develop shows that RTMA has a namelist file, rtma_parmcard.input in GSI-fix. Given this precedent, we could add the new CLDDET.NL files to GSI-fix.

That said, it's my impression that fix/ is for static or mostly static files. If rtma_parmcard_input or the new CLDDET.NL files change more frequently, they should either be part of the application script or the parm/ directory.

@wx20jjung , how often do you think users will need to modify the contents of the CLDDET.NL files. You described them as tuning files. This implies to me that these files are static once tuning is done. Is this true? How often is tuning needed? Do we need to retune when (a) change model resolution or physics change, (b) CRTM,is updated, (c) the instrument changes, or ...?

To keep things simple, I'm inclined to add the CLDDET.NL files to GSI-fix but I'd first like to get a sense of how often users will be changing these files.

@wx20jjung
Copy link
Contributor Author

wx20jjung commented Apr 23, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-Orion-Passed **Bot use only** CI testing on Orion for this PR has completed successfully
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Infrared cloud detection scheme
5 participants