From fb4941b301bead3f5cd42d5147a47a0efa07ac75 Mon Sep 17 00:00:00 2001 From: Erik Kluzek Date: Thu, 1 Aug 2024 00:59:41 -0600 Subject: [PATCH 1/2] Don't need to set coldstart temp or depth as these tests are starting up from finidat files --- .../testdefs/testmods_dirs/clm/ExcessIceStreams/user_nl_clm | 2 -- 1 file changed, 2 deletions(-) diff --git a/cime_config/testdefs/testmods_dirs/clm/ExcessIceStreams/user_nl_clm b/cime_config/testdefs/testmods_dirs/clm/ExcessIceStreams/user_nl_clm index aa2c9efa5b..f61ca32a79 100644 --- a/cime_config/testdefs/testmods_dirs/clm/ExcessIceStreams/user_nl_clm +++ b/cime_config/testdefs/testmods_dirs/clm/ExcessIceStreams/user_nl_clm @@ -1,4 +1,2 @@ use_excess_ice = .true. use_excess_ice_streams = .true. - excess_ice_coldstart_depth = 0.5 - excess_ice_coldstart_temp = -5.0 From 3ed874dd619f2d2af32e4292ae12a37fb765084f Mon Sep 17 00:00:00 2001 From: mvdebolskiy Date: Thu, 1 Aug 2024 05:40:03 -0600 Subject: [PATCH 2/2] add more to notes. --- doc/ChangeLog | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/doc/ChangeLog b/doc/ChangeLog index 049a075ad6..9bd2f3e86f 100644 --- a/doc/ChangeLog +++ b/doc/ChangeLog @@ -11,9 +11,13 @@ Changed the way soil temperature is initialized when excess ice is on and the mo Specific notes -Added 2 parameters: excess_ice_coldstart_depth and excess_ice_coldstart_temp which control top depth and soil temperature for soil layers in columns where excess ice is present. Other columns get their default soil temperature. +TemperatureType now has a private ReadNL subroutine that reads two new namelist options: excess_ice_coldstart_depth and excess_ice_coldstart_temp which control top depth (higher layers get default initial temperature) and soil temperature for soil layers ONLY in columns where excess ice is present. Other columns get their default soil temperature. + +New namelist options belong to a new group "clm_temperature_inparm" with its own logic routine. This is done so in the future hardcoded cold start temperatures can be moved to the namelists. Two namelist tests have been added to check for invalid values logic as well as a test for use_excess_ice_streams logic. use_excess_ice_streams has no default value (due to options for restart) and default value set in the CLMBuildNamelist.pm. + +excessicestream_type has been taken out of waterstate_type and its routines are called directly in clm_inst%Init. Checks for UseExcessIceStreams() are still in place in WaterStateType.F90 for double-checking. clm_inst%Init now has a new local variable with excess ice concentration that are read from streams (or zeros if use_excess_ice is false). Arguments for temperature_type%Init and water_type%Init (and children types) have been changed to include this new variable. Fortran unit tests are also updated to account for these new arguments. + -excessicestream_type has been taken out of waterstate_type and its routines are called directly in clm_inst%Init. Checks for UseExcessIceStreams() are still in place in WaterStateType.F90 for double-checking. Significant changes to scientifically-supported configurations