Skip to content

v12: Set default optics to RRTMGP - #389

Merged
mathomp4 merged 6 commits into
developfrom
feature/v12-rrtmgp-default
May 26, 2026
Merged

v12: Set default optics to RRTMGP#389
mathomp4 merged 6 commits into
developfrom
feature/v12-rrtmgp-default

Conversation

@mathomp4

@mathomp4 mathomp4 commented Mar 3, 2026

Copy link
Copy Markdown
Member

In GEOSgcm v12, we have moved from RRTMG to RRTMGP by default. However, GOCART has continued with RRTMG optics by default.

So, this PR updates it to be RRTMGP by default. This is zero-diff because, at the moment, we do sed shenanigans in gcm_run.j to change these lines at run time to RRTMGP.

NOTE: This will require changes in GEOSgcm_App for running with RRTMG. See GEOS-ESM/GEOSgcm_App#878

cc: @sdrabenh

@mathomp4 mathomp4 self-assigned this Mar 3, 2026
@mathomp4
mathomp4 requested a review from a team as a code owner March 3, 2026 15:18
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Mar 3, 2026
@mathomp4

mathomp4 commented May 21, 2026

Copy link
Copy Markdown
Member Author

When this PR merges and GOCART defaults to RRTMGP optics, the corresponding GEOSgcm_App changes are in GEOS-ESM/GEOSgcm_App#878. That PR removes the now-redundant @RRTMGP_RADIATION sed block and adds a @RRTMG_RADIATION block to downgrade RRTMGPRRTMG for L72 runs.

vbuchard
vbuchard previously approved these changes May 22, 2026
# Conflicts:
#	ESMF/GOCART2G_GridComp/CA2G_GridComp/CA2G_instance_CA.bc.rc
#	ESMF/GOCART2G_GridComp/CA2G_GridComp/CA2G_instance_CA.br.rc
#	ESMF/GOCART2G_GridComp/CA2G_GridComp/CA2G_instance_CA.oc.rc
#	ESMF/GOCART2G_GridComp/DU2G_GridComp/DU2G_instance_DU.rc
#	ESMF/GOCART2G_GridComp/NI2G_GridComp/NI2G_instance_NI.rc
#	ESMF/GOCART2G_GridComp/SS2G_GridComp/SS2G_instance_SS.rc
#	ESMF/GOCART2G_GridComp/SU2G_GridComp/AMIP/SU2G_instance_SU.rc
#	ESMF/GOCART2G_GridComp/SU2G_GridComp/SU2G_instance_SU.rc
@mathomp4
mathomp4 requested a review from vbuchard May 26, 2026 16:42
@mathomp4
mathomp4 merged commit d9f0ea0 into develop May 26, 2026
10 checks passed
@mathomp4
mathomp4 deleted the feature/v12-rrtmgp-default branch May 26, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants