Skip to content

mosart1.1.08: Standardize time metadata #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
slevis-lmwg merged 12 commits into from
Jan 15, 2025
Merged

mosart1.1.08: Standardize time metadata #66

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ac0f75e
Rename dimension hist_interval to nbnd.
samsrabin Jul 10, 2023
3bfc8bf
Add units to time_bounds.
samsrabin Jul 10, 2023
79f1088
Standardize time_bounds long_name.
samsrabin Jul 10, 2023
adbdcb5
Add calendar to time_bounds.
samsrabin Jul 10, 2023
87f09b6
Add calendar attribute to mcdate, mcsec, mdcur, and mscur.
samsrabin Jul 11, 2023
820798f
Merge remote-tracking branch 'origin/master' into standardize-time-me…
slevis-lmwg Nov 19, 2024
858248f
Bring this PR's mods from RtmHistFile.F90 to mosart_histfile.F90
slevis-lmwg Nov 19, 2024
d819640
Revert 692d183 and 7749459 from PR #70 for active cism test to pass
slevis-lmwg Nov 22, 2024
bb03b17
Correction to if-statement from #106
slevis-lmwg Nov 22, 2024
e11904e
Merge tag 'mosart1.1.07' into standardize-time-metadata
slevis-lmwg Jan 14, 2025
6f2327b
Draft ChangeLog
slevis-lmwg Jan 15, 2025
c023afe
Update ChangeLog
slevis-lmwg Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions src/cpl/nuopc/rof_import_export.F90
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,10 +113,8 @@ subroutine advertise_fields(gcomp, flds_scalar_name, rc)
call fldlist_add(fldsToRof_num, fldsToRof, 'Flrl_rofsub')
call fldlist_add(fldsToRof_num, fldsToRof, 'Flrl_rofi')
call fldlist_add(fldsToRof_num, fldsToRof, 'Flrl_irrig')
if ( ctl%rof_from_glc ) then

@ekluzek ekluzek Mar 17, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This was the commit that fixed an important bug in the coupled model. Coupled simulations done with cesm3_0_beta05 had a problem and then fixed in cesm3_0_beta06.

This fixed #103

call fldlist_add(fldsToRof_num, fldsToRof, 'Fgrg_rofl') ! liq runoff from glc
call fldlist_add(fldsToRof_num, fldsToRof, 'Fgrg_rofi') ! ice runoff from glc
end if
call fldlist_add(fldsToRof_num, fldsToRof, 'Fgrg_rofl') ! liq runoff from glc
call fldlist_add(fldsToRof_num, fldsToRof, 'Fgrg_rofi') ! ice runoff from glc

do n = 1,fldsToRof_num
call NUOPC_Advertise(importState, standardName=fldsToRof(n)%stdname, &
Expand Down