search

ESCOMP / CDEPS

Community Data Models for Earth Prediction Systems
https://escomp.github.io/CDEPS/versions/master/html/index.html
20 stars 45 forks source link

Update stream_definition_datm.xml to include longer time period in files #277

Closed TeaganKing closed 3 months ago

TeaganKing commented 4 months ago

Description of changes

This PR includes files containing longer time periods (1849-2101) in datm/cime_config/stream_definition_datm.xml for the following SSP370 streams: co2tseries.SSP3-7.0 and presaero.SSP3-7.0

Contributors other than yourself, if any: @wwieder , @ekluzek , @samsrabin

CDEPS Issues Fixed: #246

Are there dependencies on other component PRs (if so list): No

Are changes expected to change answers: No, the files are just extended but contain the same data for the later part of the time period

Any User Interface Changes (namelist or namelist defaults changes): No

Testing performed (e.g. aux_cdeps, CESM prealpha, etc):

Hashes used for testing:

TeaganKing commented 3 months ago

This is ready for review; @slevis-lmwg has already reviewed the changes, and I just updated the PR to ensure the branch is up to date. It looks like this may need a review from someone with write access in addition to the review from Sam.

billsacks commented 3 months ago

@TeaganKing (and @slevis-lmwg or others): Should we bring this in now, or is there some reason to wait to merge it?

TeaganKing commented 3 months ago

Thanks @billsacks . I'm not aware of any reason to wait to merge this.