ESCOMP / CMEPS

NUOPC Community Mediator for Earth Prediction Systems
https://escomp.github.io/CMEPS/
24 stars 79 forks source link

Implementation of glc->ocn coupling via going through mosart #463

Closed mvertens closed 5 months ago

mvertens commented 5 months ago

Description of changes

Specific notes

Contributors other than yourself, if any: None

CMEPS Issues Fixed: None

Are changes expected to change answers? Only for fully coupled compsets with active CISM and a greenland ice sheet, otherwise bfb

Any User Interface Changes? The following namelist variables and associated xml variables have been removed ocn2glc_levels, glc2ocn_liq_rmapname, glc2ocn_ice_rmapname GLC2ICE_RMAPNAME, GLC2OCN_LIQ_RMAPNAME, GLC2OCN_ICE_RMAPNAME

The following xml variables have been removed from config_component.xml since the only allowed values are idmap or unset which are now set in the namelist_definition_drv.xml file

  <entry id="ROF2OCN_FMAPNAME">
  <entry id="GLC2ICE_RMAPNAME">
  <entry id="GLC2OCN_LIQ_RMAPNAME">
  <entry id="GLC2OCN_ICE_RMAPNAME">
  <entry id="OCN2WAV_SMAPNAME">
  <entry id="ICE2WAV_SMAPNAME">
  <entry id="WAV2OCN_SMAPNAME">

Testing performed

Using cesm2_3_beta17 checkout with the following changes:

Verified that the following was bfb with cesm2_3_beta17 baseline SMS_Ld2.f09_t232.B1850MOM.derecho_intel.mom-bcompset Verified that the following had answer changes with cesm2_3_beta17 baselines (as expected) SMS_Ld5.f09_g17.B1850G.derecho_intel.allactive-cism-test_coupling

Note that the

slevis-lmwg commented 5 months ago

just noting: I think this PR will close #437

billsacks commented 5 months ago

@mvertens - I just tried reproducing one of the tests you mentioned above. I got an error at preview_namelists time, saying that rtmhist_ndens (which appears in cime_config/testmods_dirs/allactive/defaultio/user_nl_mosart in the CESM repository) doesn't exist in MOSART's namelist definition any more. I can simply remove it for the test I'm doing, but is there a fix that should be put into CESM for this?

jedwards4b commented 5 months ago

@billsacks Those names have been shortened to just ndens etc (drop the rtmhist_ part) I have it in my cesm sandbox, but haven't commited it yet.

jedwards4b commented 3 months ago

@mvertens, @gustavo-marques has some questions/concerns about this PR. I will send an invitation to meet to discuss on Monday.