ESCOMP / CDEPS

Community Data Models for Earth Prediction Systems
https://escomp.github.io/CDEPS/versions/master/html/index.html
20 stars 45 forks source link

Need to update presndep.cplhist stream to work with CMEPS output #230

Open billsacks opened 1 year ago

billsacks commented 1 year ago

In #228 @olyson updated most cplhist stream definitions to work with CMEPS output. However, he did not change the presndep.cplhist stream, in part because this isn't fully hooked up. Once this has been added to the CMEPS output (https://github.com/ESCOMP/CMEPS/issues/375), we'll need to change this block:

https://github.com/ESCOMP/CDEPS/blob/f2c4fe1cfad109d48d7c01250f48ed144d3abc56/datm/cime_config/stream_definition_datm.xml#L4691-L4720

similarly to the changes @olyson made in #228 .

billsacks commented 1 year ago

Once this is done, we should remove the setting ./xmlchange DATM_PRESNDEP=none from CTSM's cime_config/testdefs/testmods_dirs/clm/cplhist/shell_commands file. (See also https://github.com/escomp/ctsm/issues/1844#issuecomment-1536770855 .)