CESM icon indicating copy to clipboard operation
CESM copied to clipboard

Some refcase fields look inconsistent in CESM2.1

Open billsacks opened this issue 4 years ago • 1 comments

Brief summary of bug

From examining cime_config/config_compsets.xml in CESM2.1 (looking at what I think is the latest version: cesm2.1.4-rc.07), it looks like some of the refcase fields are inconsistent. This seems like a bug to me, and I think should be reviewed, though I'm not positive that it's actually a bug.

General bug information

CESM version you are using: cesm2.1.4-rc.07

Machine you are using: N/A

Have you modified the code? (If so, please point us to you changes.): No

Details of bug

I noticed this by looking at the xml entries for various _REF xml variables (and RUN_TYPE) that reference compsets with BDRD. RUN_REFDATE and RUN_TYPE each have 8 lines referencing compsets with BDRD, whereas RUN_REFCASE only has 2 lines referencing compsets with BDRD, and RUN_REFDIR has 4.

My impression was that the variables RUN_REFDATE, RUN_TYPE, RUN_REFCASE and RUN_REFDIR should typically all be in sync in terms of their matches - so should have the same number of entries, with a one-to-one relationship of their grid and compset matches. This is not currently the case.

It would be great if someone could review these entries for consistency and correctness. Note that CLM_NAMELIST_OPTS should also be considered if making any changes / corrections, though I don't think that there necessarily needs to be a CLM_NAMELIST_OPTS entry for each entry of the other variables.

@ekluzek @fvitt @fischer-ncar I'm assigning the three of you because it looks like you three have made most of the relevant changes to this file in the last couple of years. Feel free to work out among yourselves which of you should take the lead on this, and feel free to add others if you think others need to be involved in figuring this out.

billsacks avatar Feb 10 '21 18:02 billsacks

cesm2_init is the default value for RUN_REFDIR. So I don't believe there is a need have those entries for RUN_REFDIR which are all set to cesm2_init.

fvitt avatar Feb 10 '21 19:02 fvitt

I'm closing this as a wontfix, because our concern for CESM2.1 isn't large enough to look into this.

ekluzek avatar May 03 '24 16:05 ekluzek