Skip to content

Commit

Permalink
Merge pull request ESCOMP#2624 from rgknox/fates-no-prescribedNP
Browse files Browse the repository at this point in the history
changing prescribed N and P uptake to 1 in the FATES PRT2 test
  • Loading branch information
samsrabin authored Aug 13, 2024
2 parents cc73d14 + ac85dbc commit a966cf6
Show file tree
Hide file tree
Showing 6 changed files with 83 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .gitmodules
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
[submodule "fates"]
path = src/fates
url = https://github.com/NGEET/fates
fxtag = sci.1.77.1_api.36.0.0
fxtag = sci.1.77.2_api.36.0.0
fxrequired = AlwaysRequired
# Standard Fork to compare to with "git fleximod test" to ensure personal forks aren't committed
fxDONOTUSEurl = https://github.com/NCAR/fates-release
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
SRCDIR=`./xmlquery SRCROOT --value`
CASEDIR=`./xmlquery CASEROOT --value`
FATESDIR=$SRCDIR/src/fates/
FATESPARAMFILE=$CASEDIR/fates_params_prt2_prescribed_np.nc

ncgen -o $FATESPARAMFILE $FATESDIR/parameter_files/fates_params_default.cdl

$FATESDIR/tools/modify_fates_paramfile.py --O --fin $FATESPARAMFILE --fout $FATESPARAMFILE --var fates_cnp_prescribed_nuptake --val 1.0 --allpfts

$FATESDIR/tools/modify_fates_paramfile.py --O --fin $FATESPARAMFILE --fout $FATESPARAMFILE --var fates_cnp_prescribed_puptake --val 1.0 --allpfts

Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
fates_paramfile = '$CASEROOT/fates_params_prt2_prescribed_np.nc'
fates_parteh_mode = 2
hist_fincl1 = 'FATES_L2FR','FATES_L2FR_CANOPY_REC_PF','FATES_L2FR_USTORY_REC_PF',
'FATES_NH4UPTAKE_SZPF','FATES_NO3UPTAKE_SZPF','FATES_NEFFLUX_SZPF',
Expand Down
68 changes: 68 additions & 0 deletions doc/ChangeLog
Original file line number Diff line number Diff line change
@@ -1,4 +1,72 @@
===============================================================
Tag name: ctsm5.2.021
Originator(s): rgknox (Ryan Knox,LAWRENCE BERKELEY NATIONAL LABORATORY,510-495-2153)
Date: Tue 13 Aug 2024 03:22:34 PM MDT
One-line Summary: Adding on-the-fly parameter settings for prescribed N and P in FATES

Purpose and description of changes
----------------------------------

This set of changes makes it so that the FATES PRT2 test is forced to have prescribed N and P uptake in FATES.

This allows the default to be coupled N and P, which is less confusing to new FATES-CNP users.

The FATES tag is also updated to sci.1.77.2_api.36.0.0 which has a bug-fix for the output variable FATES_TRANSITION_MATRIX_LULU.


Significant changes to scientifically-supported configurations
--------------------------------------------------------------

Does this tag change answers significantly for any of the following physics configurations? NO

Bugs fixed
----------


List of CTSM issues fixed (include CTSM Issue # and description) [one per line]:


Notes of particular relevance for users
---------------------------------------

CLM-FATES with N and cycling is not fully coupled yet. Therefor, these changes are only relevant to the testing infrastructure.
Users are not encouraged to use fates_parteh_mode = 2

Notes of particular relevance for developers:
---------------------------------------------

Changes to tests or testing: Existing scripting that FATES has used in other tests to update parameters in certain tests, is now
being called for FatesColdPRT2 tests. This forces parameters fates_cnp_prescribed_nuptake and fates_cnp_prescribed_puptake to be 1.


Testing summary:
----------------

regular tests (aux_clm: https://github.com/ESCOMP/CTSM/wiki/System-Testing-Guide#pre-merge-system-testing):

derecho ----- OK
izumi ------- OK

Answer changes
--------------

Changes answers relative to baseline: NONE, except:

FATES_TRANSITION_MATRIX_LULU reflects a BASELINE DIFF from a bug fix in two FATES tests.

All other tests are B4B.


Other details
-------------

FATES submodule was updated, pointing to tag sci.1.77.2_api.36.0.0

Pull Requests that document the changes (include PR ids): https://github.com/ESCOMP/CTSM/pull/2624


===============================================================
===============================================================
Tag name: ctsm5.2.020
Originator(s): HuiWangWanderInGitHub (Hui Wang,[email protected])
Date: Mon 12 Aug 2024 10:51:04 AM MDT
Expand Down
1 change: 1 addition & 0 deletions doc/ChangeSum
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
Tag Who Date Summary
============================================================================================================================
ctsm5.2.021 rgknox 08/13/2024 Adding on-the-fly parameter settings for prescribed N and P in FATES
ctsm5.2.020 slevis 08/12/2024 MEGAN updates (MEGAN-CLM6)
ctsm5.2.019 erik 08/11/2024 Add in an additional dust emission method Leung_2023, by default off
ctsm5.2.018 mvdebols 08/02/2024 Fix/excess ice cold start
Expand Down
2 changes: 1 addition & 1 deletion src/fates
Submodule fates updated 1 files
+1 −0 main/EDInitMod.F90

0 comments on commit a966cf6

Please sign in to comment.