Skip to content
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.

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

Standardize time metadata #35

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

samsrabin
Copy link

@samsrabin samsrabin commented Jul 13, 2023

Description of changes

Standardizes a dimension name of output variable time_bounds, as well as attributes for that plus mcdate, mcsec, mdcur, and mscur.

Specific notes

Contributors other than yourself: @phillips-ad, @ekluzek

RTM Issues Fixed:

Are answers expected to change (and if so in what way)? No.

Any User Interface Changes (namelist or namelist defaults changes)? No.

Testing performed

New metadata in a history file from a short test:

netcdf standardize-time-metadata_rtm.rtm.h0.1850-01-06-00000 {
dimensions:
	...
	nbnd = 2 ;
	...
variables:
	...
	int mcdate(time) ;
		...
		mcdate:calendar = "noleap" ;
	int mcsec(time) ;
		...
		mcsec:calendar = "noleap" ;
	int mdcur(time) ;
		...
		mdcur:calendar = "noleap" ;
	int mscur(time) ;
		...
		mscur:calendar = "noleap" ;
	double time_bounds(time, nbnd) ;
		time_bounds:long_name = "time interval endpoints" ;
		time_bounds:units = "days since 1850-01-01 00:00:00" ;
		time_bounds:calendar = "noleap" ;

An arbitrarily-chosen CTSM test, ERS_D_Ld5.f10_f10_mg37.I2000Clm50BgcCropRtm.cheyenne_intel.rtm-rtmOnFloodOnEffvelOn, passed bit-for-bit against tag ctsm5.1.dev130, despite the changed metadata.

  • Reminder to update ChangeLog

Eliminate the 0th time step from RTM simulations

slevis resolved conflicts:
src/riverroute/RtmHistFile.F90
@slevis-lmwg
Copy link
Contributor

slevis-lmwg commented Nov 18, 2024

@slevis-lmwg reminder to self:
DONE Merge to master (without making new tags) updates to the ChangeLog for rtm tags 81, 82, 83 and for mosart tags 03, 04, 05.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Modify naming and attributes of time variables on history files to be consistent with other CESM components
3 participants