Skip to content

Commit

Permalink
Merge pull request #2472 from wwieder/b4b-neon
Browse files Browse the repository at this point in the history
B4b neon

Description
Fixes neon bugs related to calendar issue with transient cases created by run_neon and an odd error that can be fixed by removing redundant information in usermod_dirs.

CTSM Issues Fixed:
Fixes #2470
Fixes #2471
Fixes #2474

Are answers expected to change (and if so in what way)?
No, removes bug in NEON cases

Any User Interface Changes (namelist or namelist defaults changes)?
only changes to usermod_dirs

Testing:
OK: ./run_ctsm_py_tests -u
OK: ./run_ctsm_py_tests -s
PASS: ./run_sys_tests -s clm_pymods -c ctsm5.1.dev176 --skip-generate
  • Loading branch information
slevis-lmwg authored Apr 15, 2024
2 parents da2851a + ea4bcc6 commit 98fb232
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 2 additions & 0 deletions cime_config/usermods_dirs/NEON/defaults/shell_commands
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ TEST=`./xmlquery TEST --value`

# For a transient case run the whole length and don't cycle
if [[ $compset =~ ^HIST ]]; then
./xmlchange CALENDAR=GREGORIAN
./xmlchange DATM_YR_END=2022
./xmlchange RUN_STARTDATE=2018-01-01
# Number of months that can be run for the full transient case
Expand All @@ -29,6 +30,7 @@ if [[ $compset =~ ^HIST ]]; then
./xmlchange CLM_NML_USE_CASE="2018-PD_transient"
else
./xmlchange CLM_NML_USE_CASE="2018_control"
./xmlchange CALENDAR=NO_LEAP
fi

# If needed for SP simulations: & set history file variables
Expand Down
3 changes: 0 additions & 3 deletions cime_config/usermods_dirs/NEON/defaults/user_nl_clm
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,6 @@
! Set glc_do_dynglacier with GLC_TWO_WAY_COUPLING env variable
!----------------------------------------------------------------------------------

flanduse_timeseries = ' ' ! This isn't needed for a non transient case, but will be once we start using transient compsets
fsurdat = "$DIN_LOC_ROOT/lnd/clm2/surfdata_map/NEON/surfdata_1x1_NEON_${NEONSITE}_hist_78pfts_CMIP6_simyr2000_c230601.nc"

! h1 output stream
hist_fincl2 = 'AR','ELAI','FCEV','FCTR','FGEV','FIRA','FSA','FSH','GPP','H2OSOI',
'HR','SNOW_DEPTH','TBOT','TSOI','SOILC_vr','FV','NET_NMIN_vr'
Expand Down
2 changes: 2 additions & 0 deletions python/ctsm/site_and_regional/tower_site.py
Original file line number Diff line number Diff line change
Expand Up @@ -391,11 +391,13 @@ def run_case(
case.set_value("RUN_TYPE", "hybrid")

if run_type == "postad":
case.case_setup()
self.set_ref_case(case)
case.set_value("STOP_N", run_length)

# For transient cases STOP will be set in the user_mod_directory
if run_type == "transient":
case.case_setup()
if self.finidat:
case.set_value("RUN_TYPE", "startup")
else:
Expand Down

0 comments on commit 98fb232

Please sign in to comment.