Skip to content

Cleanup ugwp_driver_v0#1202

Open
matusmartini wants to merge 7 commits intoNCAR:mainfrom
matusmartini:cleanup-ugwp_driver_v0
Open

Cleanup ugwp_driver_v0#1202
matusmartini wants to merge 7 commits intoNCAR:mainfrom
matusmartini:cleanup-ugwp_driver_v0

Conversation

@matusmartini
Copy link
Contributor

@matusmartini matusmartini commented Mar 4, 2026

Description of Changes:

This PR does the following:

  1. Improve readability inspired by updates in GWD/cires_ugwpv1_oro.F90 (making comparison between v0 and v1 easier).
  2. Convert GWD/ugwp_driver_v0.F to GWD/ugwp_driver_v0.F90 and update related meta files.
  3. Remove unused variables and old code.
  4. Add errflg and errmsg.

Tests Conducted:

This PR does nothing, just cleanup. It gives bit-for-bit identical results in NEPTUNE.

Dependencies:

None.

Documentation:

None.

@matusmartini
Copy link
Contributor Author

Is anyone still using this scheme?

@dustinswales
Copy link
Member

Pinging our primary GWD developer @mdtoyNOAA

Copy link
Collaborator

@mdtoyNOAA mdtoyNOAA left a comment

Choose a reason for hiding this comment

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

Looks good

@matusmartini
Copy link
Contributor Author

Looks good

Thank you for checking @mdtoyNOAA. Do you know if this scheme still used at NOAA?

@yangfanglin
Copy link

It is still used in current operational models, including GFSv16, GEFSv12 etc. All upcoming implementations will not use this scheme anymore.

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Fine with me.

I will note that we should probably refrain from manual formatting changes for now until we had a discussion among the ccpp-physics developers about adopting a Fortran coding style.

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

Successfully merging this pull request may close these issues.

5 participants