E3SM icon indicating copy to clipboard operation
E3SM copied to clipboard

Published 20 hours ago • verified publisher icon E3SM-Project

Removes few ELM fields that were added by default

Open bishtgautam opened this issue 10 months ago • 5 comments

The glacier-related fields should only be added when create_glacier_mec_landunit = .true..

Fixes #6365

bishtgautam avatar Apr 24 '24 17:04 bishtgautam

@golaz Could you please try this branch to see if it fixes the error reported in #6365?

bishtgautam avatar Apr 24 '24 17:04 bishtgautam

@rljacob While the answers will not change, the tests using this branch will report that the ELM restart file is missing a few fields. Should this PR be marked as BFB or non-BFB?

bishtgautam avatar Apr 24 '24 17:04 bishtgautam

restart files are not compared in testing. just history.

rljacob avatar Apr 24 '24 17:04 rljacob

I have updated the label and comment.

bishtgautam avatar Apr 24 '24 17:04 bishtgautam

@golaz please try this out.

rljacob avatar May 01 '24 21:05 rljacob

since @rljacob went ahead and approved, that I'll go ahead with the merge today unless there are any objections.

peterdschwartz avatar May 16 '24 18:05 peterdschwartz

Merged to next. Expect DIFFs due to FIELD LIST for any tests that compare elm.h0 files.

peterdschwartz avatar May 16 '24 22:05 peterdschwartz

Merged to master. hopefully got all the DIFFs

peterdschwartz avatar May 17 '24 18:05 peterdschwartz

@wlin7 will double-check that this fixes restart issue.

rljacob avatar Jun 13 '24 17:06 rljacob