Skip to content

Make sure there are default values for some fields#681

Merged
billsacks merged 1 commit into
ESCOMP:mainfrom
billsacks:fix_noresm_coupling_mode
Jul 17, 2026
Merged

Make sure there are default values for some fields#681
billsacks merged 1 commit into
ESCOMP:mainfrom
billsacks:fix_noresm_coupling_mode

Conversation

@billsacks

@billsacks billsacks commented Jul 17, 2026

Copy link
Copy Markdown
Member

Description of changes

This is critical for histaux_wav2med_file1_flds (in case we're not running with ww3); for aofluxes_use_shr_wv_sat, this default isn't currently needed, but adding it is helpful for some future-proofing.

Specific notes

Contributors other than yourself, if any: @mvertens

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers? (specify if bfb, different at roundoff, more substantial) bfb

Any User Interface Changes (namelist or namelist defaults changes)? Sets a default for histaux_wav2med_file1_flds when not running with ww3 (to an empty list)

Testing performed

aux_cime_baselines from cesm3_0_alpha09d (SMS_Ld2.ne30pg3_t232.B1850C_LTso.derecho_intel.allactive-defaultio hasn't finished; all others pass and are bit-for-bit)

This is critical for histaux_wav2med_file1_flds (in case we're not
running with ww3); for aofluxes_use_shr_wv_sat, this default isn't
currently needed, but adding it is helpful for some future-proofing.
@billsacks
billsacks requested review from fischer-ncar and mvertens and removed request for mvertens July 17, 2026 19:07

@mvertens mvertens left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes.

@billsacks
billsacks merged commit 04526a1 into ESCOMP:main Jul 17, 2026
1 check passed
@billsacks
billsacks deleted the fix_noresm_coupling_mode branch July 17, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants