In a run where OCN_NCPL=24 and time_mix_opt = 'robert'. Adding a debug print statement to build-namelist, I realized that time_mix isn't robert, it's 'robert' (with the single quotes); changing to
Produces the expected (though undesired) behavior that 3 matches > 2 matches so dt_count=24. We were getting the correct dt_count in out-of-the-box simulations because
<dt_count ocn_grid="gx1v7" > 24</dt_count>
I was confused about how this hasn't been noticed with
but apparently we don't test with this option because avg_mix is not a valid value for time_mix_opt (perhaps we should have a testcase for gx1v7 where we set time_mix_opt = "avgfit"?)
Anyway, I think the proper fixes are
Remove the quotes around 'robert' (or 'avgfit') in build-namelist (rather than adding single quotes to namelist defaults, which I just did as a quick / dirty check)
change avg_mix -> avgfit in dt_count definition attributes
And possibly add a leapfrog test to make sure dt_count=23?
Version:
CESM: 2.1.x (and probably 2.2, though I haven't verified)
POP2: cesm2_1_x_rel (and probably master, though I haven't verified)
Description of the issue:
While putting together #27, I was surprised that
Took precendence over
In a run where
OCN_NCPL=24
andtime_mix_opt = 'robert'
. Adding a debug print statement tobuild-namelist
, I realized thattime_mix
isn'trobert
, it's'robert'
(with the single quotes); changing toProduces the expected (though undesired) behavior that 3 matches > 2 matches so
dt_count=24
. We were getting the correctdt_count
in out-of-the-box simulations becauseI was confused about how this hasn't been noticed with
but apparently we don't test with this option because
avg_mix
is not a valid value fortime_mix_opt
(perhaps we should have a testcase forgx1v7
where we settime_mix_opt = "avgfit"
?)Anyway, I think the proper fixes are
'robert'
(or'avgfit'
) inbuild-namelist
(rather than adding single quotes to namelist defaults, which I just did as a quick / dirty check)avg_mix
->avgfit
indt_count
definition attributesAnd possibly add a leapfrog test to make sure
dt_count=23
?Version:
cesm2_1_x_rel
(and probablymaster
, though I haven't verified)Machine/Environment Description:
I was working on
cheyenne
when this cropped upAny xml/namelist changes or SourceMods:
Nope