Add ozone DATM streams #188
Conversation
…' into feature/ndep_stream
|
Redid some tests after adding extra compset logic: Ran a comparison against Tests ran:
Hashes used for testing: |
{{ message }}
Merged
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mvertens
approved these changes
Aug 31, 2022
uturuncoglu
approved these changes
Aug 31, 2022
jedwards4b
approved these changes
Sep 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
Add ozone DATM streams based on several input datasets.
Specific notes
config_componentto determine which stream goes with which compset by defaultdrv_flds_infile inbuldnmlpresndep.SSP2-4.5, and missing stream forpresaero.SSP5-8.5added backContributors other than yourself, if any: @billsacks @mvertens
CDEPS Issues Fixed: #187
Are there dependencies on other component PRs: CMEPS (309)
Are changes expected to change answers: Yes, when the ozone DATM data is used (e.g. in CTSM) instead of previous hard-coded ozone values
Any User Interface Changes (namelist or namelist defaults changes): New ozone datm streams added (
DATM_O3can beclim_2000,clim_2010,clim_1850,hist,SSP2-4.5,SSP3-7.0, orSSP5-8.5).General testing:
Performed tests to ensure the input ozone files matched reasonably well with output O3 from CTSM (see testing script /glade/u/home/afoster/analysis_scripts/Ozone_damage.ipynb). Example plots (global input/output difference at one time slice and then both input and output over one year at a single latitude and longitude)
System testing:
Ran a comparison against cesm2_3_alpha09d baselines created with
cdeps0.12.61and found only NLCOMP fails for extra variable:flds_preso3Tests ran:
ERS.T62_g37.G.cheyenne_gnu.pop-cice- /glade/scratch/afoster/cesm_tests/ERS.T62_g37.G.cheyenne_gnu.pop-cice.GC.20220830_120015_7f5l1oERS_IOP.T62_g16.GIAF.cheyenne_intel.pop-default- /glade/scratch/afoster/cesm_tests/ERS_IOP.T62_g16.GIAF.cheyenne_intel.pop-default.GC.20220830_120015_7f5l1oSMS_Ld2_P72_D.T62_g37.C1850ECO.cheyenne_intel.pop-ecosys_81blocks_100x116_spacecurve- /glade/scratch/afoster/cesm_tests/SMS_Ld2_P72_D.T62_g37.C1850ECO.cheyenne_intel.pop-ecosys_81blocks_100x116_spacecurve.GC.20220830_120015_7f5l1oHashes used for testing:
cime6.0.52cdeps0.12.62-26-g73bc63bcmeps0.13.70-3-g2a3f929