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

CTSM5.3.0: New raw pft/lai/glc-behavior/soilcolor/fire datasets #2500

Merged
merged 106 commits into from
Sep 25, 2024

Conversation

slevis-lmwg
Copy link
Contributor

@slevis-lmwg slevis-lmwg commented Apr 30, 2024

Description of changes

Updating gen_mksurfdata_namelist.xml, initially with temporary paths to the new raw data.

New surface datasets because of updates in PFT/LAI/soil-color datasets, as well as two fire datasets (AG fire, pearland and the glacier behavior dataset. Also bring in an updated "PPE informed, hand-tuned" parameter file. And turn on Li2024 fire method. Also updates to allow creating transient landuse timeseries files going back to 1700. Fix an important bug on soil fields that was there since ctsm5.2.0. This results in mkaurfdata_esmf giving identical answers with a change in number of processors. Add in creation of ne0np4.POLARCAP.ne30x4 surface datasets.

Asides: Remove VIC fields from surface datasets and testing. Add version to the surface datasets. Update the fire emissions factor dataset for the fire emissions testing in CTSM. Remove the --hires_pft option from mksurfdata_esmf as we don't have the datasets for it.

TODOs:

Specific notes

Contributors other than yourself, if any:
@lawrencepj1 @ekluzek @wwieder @adrifoster @samsrabin @billsacks @lifang0209 @linniahawkins @olyson

CTSM Issues Fixed (include github issue #):
Fixes #2723
Fixes #2720
Partly addresses #2752
Partly addresses #2672
Fixes #2452
Fixes #2570
Fixes #2618
Fixes #423
Fixes #2734
Fixes #2744
Fixes #2502
Resolves #2748

Are answers expected to change (and if so in what way)?
Yes, in generated fsurdat/landuse files.

Any User Interface Changes (namelist or namelist defaults changes)?
gen_mksurfdata_namelist.xml points to new datasets.

Testing performed, if any: will do release testing
In the first iteration generated new fsurdat/landuse files and ran the model (NCAR/LMWG_dev#59).
Second iteration in progress.
Now have updated datasets and verified with the build-namelist tester

Merge b4b-dev

Some fixes for run_neon that came up in ctsm5.1.dev172.
Small bug fix for units of vector format hillslope hydrology.
Start adding PLUMBER2 users-mods (NOT functional)
@ekluzek

This comment was marked as outdated.

@slevis-lmwg
Copy link
Contributor Author

@lawrencepj1 I requested a review from you, but we could go over this together at our next "pft/landuse tools" meeting.

@wwieder wwieder changed the title New raw pft/lai/soilcolor datasets CTSM5.3: New raw pft/lai/soilcolor datasets May 23, 2024
@slevis-lmwg
Copy link
Contributor Author

slevis-lmwg commented Jun 6, 2024

UPDATE appears in #2570.

Also address #2617 with this PR.

@slevis-lmwg
Copy link
Contributor Author

Update submodule tags to pass runoff from cism to rof

- Update MOSART, CMEPS, and CISM so CISM runoff goes to ROF rather than CTSM
- Update RTM with fix needed for Paleo LGM work

Contributors:
@mvertens, @jedwards4b, @billsacks, @Katetc, @ekluzek, @slevis-lmwg

 Fixes ESCOMP#2590 Update CMEPS/MOSART/CISM/RTM tags
 Fixes ESCOMP/RTM#50 Likely wrong RTM river flux to MOM6 within cesm2_3_beta17

 Differences in namelist 'mosart_inparm':
  missing variable: 'do_rtmflood'
  missing variable: 'finidat_rtm'
  missing variable: 'frivinp_rtm'
  missing variable: 'rtmhist_fexcl1'
  missing variable: 'rtmhist_fexcl2'
  missing variable: 'rtmhist_fexcl3'
  missing variable: 'rtmhist_fincl1'
  missing variable: 'rtmhist_fincl2'
  missing variable: 'rtmhist_fincl3'
  missing variable: 'rtmhist_mfilt'
  missing variable: 'rtmhist_ndens'
  missing variable: 'rtmhist_nhtfrq'
  found extra variable: 'budget_frq'
  found extra variable: 'fexcl1'
  found extra variable: 'fexcl2'
  found extra variable: 'fexcl3'
  found extra variable: 'fincl1'
  found extra variable: 'fincl2'
  found extra variable: 'fincl3'
  found extra variable: 'finidat'
  found extra variable: 'frivinp'
  found extra variable: 'mfilt'
  found extra variable: 'mosart_euler_calc'
  found extra variable: 'mosart_tracers'
  found extra variable: 'ndens'
  found extra variable: 'nhtfrq'
  found extra variable: 'use_halo_option'

Changes answers
- what code configurations: mosart and rtm
- what platforms/compilers: all
- nature of change: mosart roundoff; rtm larger than roundoff due to bug fix; the latter also affects bgc variables

 We are ignoring strange diffs from baseline in two tests in variable
 FATES_TRANSITION_MATRIX_LULU as explained in issue ESCOMP#2656.
Copy link
Contributor Author

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

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

Reviewing my own PR to include some reminders.

python/ctsm/toolchain/gen_mksurfdata_namelist.py Outdated Show resolved Hide resolved
tools/mksurfdata_esmf/gen_mksurfdata_namelist.xml Outdated Show resolved Hide resolved
@slevis-lmwg slevis-lmwg added the priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations label Jul 25, 2024
@lawrencepj1
Copy link

Hi @slevis-lmwg

Just to let you know that the raw PFT data for the SSPs rebased to TRENDY 2024 starting in year 2015 are available at:

/glade/campaign/cesm/development/lmwg/landuse_source_data/CTSM53RawData
globalctsm53TRSSP119Deg025_240728
globalctsm53TRSSP126Deg025_240728
globalctsm53TRSSP245Deg025_240728
globalctsm53TRSSP370Deg025_240728
globalctsm53TRSSP460Deg025_240728
globalctsm53TRSSP585Deg025_240728

I have checked them and everything looks good.
Peter

image

image

image

@slevis-lmwg
Copy link
Contributor Author

Thank you @lawrencepj1

@ekluzek ekluzek added this to the ctsm5.3.0 milestone Aug 15, 2024
@ekluzek ekluzek changed the title CTSM5.3: New raw pft/lai/soilcolor datasets CTSM5.3.0: New raw pft/lai/soilcolor datasets Aug 15, 2024
@ekluzek
Copy link
Collaborator

ekluzek commented Aug 15, 2024

Same as mksrf_GlacierRegion_10x10min_nomask_cd5_c220131.nc but with
GLACIER_REGION=1 (Points outside Greenland itself but which might become
ice-covered in an evolving Greenland simulation) merged into
GLACIER_REGION 0, then 2 changed to 1 and 3 changed to 2. We are
removing the old GLACIER_REGION=1 even though this means we cannot
generate SMB in areas outside the initial CISM Greenland ice mask,
because it created problematic non-physical behavior. See
ESCOMP#423 for details.
@slevis-lmwg
Copy link
Contributor Author

slevis-lmwg commented Sep 24, 2024

Latest testing

izumi
PASS mosart
OK fates
OK aux_clm

derecho
PASS make black and make lint
PASS ./run_ctsm_py_tests -s and ./run_ctsm_py_tests -u
PASS ./build-namelist_test.pl test
OK mosart
OK rtm
OK aux_clm
OK fates
OK ctsm_sci (now in tests_0924-173947de, see notes below), RXCR* test in progress before baseline will be ready

I use this to check for failures in aux_clm:
./cs.status.fails | grep -v PASS | grep -v NLCOM | grep -v '28: DIF' | grep -v "fireemis' does not exist"

@ekluzek
Copy link
Collaborator

ekluzek commented Sep 24, 2024

Based on the previous work and the commit I made last night. I expect almost all testing will be as expected, other than possibly a couple 1979 tests. We can resubmit those 1979 tests that fail. If we have any other failing tests we can create an issue and move on.

The other thing is to finish the change files.

In talking with @briandobbins and @wwieder they would like us to make a tag today.

@slevis-lmwg
Copy link
Contributor Author

slevis-lmwg commented Sep 24, 2024

Failures that we need to open issues for.

  1. DONE @slevis-lmwg will open issue for aux_clm tests that fail in the RUN phase because they need c13 in finidat
SMS_Lm1.f10_f10_mg37.I1850Clm50BgcCropCmip6waccm.derecho_gnu.clm-basic
ERS_Ly3.f10_f10_mg37.I1850Clm50BgcCropCmip6.derecho_intel.clm-basic
SMS_Ld2_D_PS.f09_g17.I1850Clm50BgcCropCmip6.derecho_intel.clm-basic_interp
  1. DONE @ekluzek will open issue for ctsm_sci tests that give this error CLMBuildNamelist::setup_cmdl_fire_light_res() : fire_method is being set while use_cn and use_fates are both false in the SHAREDLIB_BUILD phase:
SMS_Ld5.f09_g17.IHistClm50Sp.derecho_intel.clm-nofire
SMS_Ld5.f09_g17.IHistClm60Sp.derecho_intel.clm-nofire
SMS_Ld5.f19_g17.IHistClm50Sp.derecho_intel.clm-nofire
SMS_Ld5.f19_g17.IHistClm60Sp.derecho_intel.clm-nofire
  1. DONE @ekluzek will open issues for fates test that failed with error build-namelist failed rc=255, err=Unknown option: i-clm_demand, Unknown option: flanduse_timeseries in the SHAREDLIB_BUILD phase:
ERS_D_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.derecho_intel.clm-FatesColdLandUse
ERS_D_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.izumi_nag.clm-FatesColdLandUse
  1. DONE @slevis-lmwg will correct typo in Expected Failures to Ld30 for this fates test that fails in the RUN phase for an unknown reproducible reason BUT passes on derecho:
    ERS_D_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.izumi_nag.clm-FatesColdLUH2
  2. DONE @slevis-lmwg will resubmit izumi tests not finding finidat
SMS_D.f10_f10_mg37.I2000Clm60BgcCrop.izumi_gnu.clm-crop
SMS_D.f10_f10_mg37.I2000Clm60BgcCrop.izumi_intel.clm-crop
SMS_D.f10_f10_mg37.I2000Clm60BgcCrop.izumi_nag.clm-crop
SMS_D_Ld3.f10_f10_mg37.I2000Clm60Bgc.izumi_nag.clm-HillslopeD
SMS_D_Ld65.f10_f10_mg37.I2000Clm60BgcCrop.izumi_nag.clm-FireLi2024GSWP
PEM_Ld1.f10_f10_mg37.I2000Clm60BgcCrop.izumi_intel.clm-till--clm-matrixcnOn
PEM_Ld1.f10_f10_mg37.I2000Clm60BgcCrop.izumi_intel.clm-till
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-matrixcnOn_ignore_warnings
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart
SMS_Ld5_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart
SMS_Ld10_D_Mmpi-serial.CLM_USRDAT.I1PtClm60Bgc.izumi_nag.clm-default--clm-NEON-HARV

@ekluzek ekluzek marked this pull request as ready for review September 25, 2024 01:32
@ekluzek ekluzek merged commit a3459c5 into ESCOMP:master Sep 25, 2024
2 checks passed
@ekluzek ekluzek deleted the new_rawpftlai branch September 25, 2024 07:23
@ekluzek
Copy link
Collaborator

ekluzek commented Sep 25, 2024

I figure I spent around a week to 10 days of engineering time here on ctsm5.3.0.

This was referenced Sep 26, 2024
@ekluzek ekluzek removed their assignment Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment