ESCOMP / ESCOMP/CISM-wrapper

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

Open
#75 24 comments 0 reactions 1 assignee Claimed by @Katetc View on GitHub
Dominant language
Fortran
Stars
4
Forks
18
PR merge metrics
No merged PRs in 30d

Description

These changes are being requested to improve the ease-of-access of CESM data and to normalize the look of the metadata across CESM components. See https://github.com/ESCOMP/CESM/issues/194 and especially the google doc referenced from https://github.com/ESCOMP/CESM/issues/194#issuecomment-961443965.

Current CISM time array output:

double time(time) ;
time:long_name = "Model time" ;
time:standard_name = "time" ;
time:units = "common_year since 0000-01-01 0:0:0" ;
time:calendar = "noleap" ;

Proposed CISM time array output:

double time(time) ;
time:long_name = "time" ;
time:units = "days since 0001-01-01 00:00:00" ;
time:calendar = "noleap" ;
time:bounds = "time_bounds" ;
double time_bounds(time, nbnd) ;
time_bounds:long_name = "time interval endpoints" ;
time_bounds:units = "days since 0001-01-01 00:00:00" ;
time_bounds:calendar = "noleap" ;

Changes made: Add time_bounds variable, alter time@units to match other components, add time@bounds, remove time@standard_name, change time@long_name to “time”.

@billsacks @phillips-ad @strandwg are filing similar issues for each component.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.