Updates to add new Catchment-CN version: Catchment-CN5.1#1129
Updates to add new Catchment-CN version: Catchment-CN5.1#1129gmao-jkolassa wants to merge 711 commits intodevelopfrom
Conversation
… of CLM40 variables description
…tate.F90, GEOS_CatchCNGridComp.F90, CNCLM_init_mod.F90)
|
GCM 1 day AMIP, REPLAY and Inc Replay are zero diff. |
|
CatchCN CLM4.0 restart remapping compared from what I get using main GEOS_Util to what we get with GEOS-Util pr and this PR is zero diff for both m36 and c180 runs. Restart I used to remap from was one I created for Eunjee when she wanted to run v12 bcs. Also, I forgot to say regression was also run as part of GCM testing on the exp PR and it was a PASS. |
…mp/GEOScatchCN_GridComp/GEOS_CatchCNGridComp.F90)
|
Updates on my part:
|
…and bad Z0_FORMULATION=1 (GEOS_SurfaceGridComp.rc, lsm_routines.F90, GEOS_CatchGridComp.F90, GEOS_CatchCNCLM51GridComp.F90)
…GEOS_CatchGridComp.F90, GEOS_CatchCNCLM51GridComp.F90)
...GridComp/GEOSphysics_GridComp/GEOSsurface_GridComp/GEOSland_GridComp/Shared/lsm_routines.F90
Show resolved
Hide resolved
|
I think the error is on this line GNU fails to recognize that is a comment |
|
|
||
| do n = 1,ntiles | ||
| ta_count(n) = ta_count(n) + 1 | ||
| TA_MIN(n) = min(TA_MIN(n),TA(n)) |
There was a problem hiding this comment.
The minimum air temperature used here is the "lowest-model-level" temperature, which is typically at about 40-60 m height, and not the screen-level (2m) temperature. There is a comment above (just before the if block) that suggests as much, but it's still misleading to label this variable T2m and claim as much in the LONG_NAME of the Internal_Spec definition.
| TA_MIN(n) = min(TA_MIN(n),TA(n)) | ||
|
|
||
| if (ta_count(n) == n1d) then | ||
| T2MMIN5D(n) = ((accper-1)*T2MMIN5D(n) + TA_MIN(n)) / accper |
There was a problem hiding this comment.
Since init_accum is hardwired to .false., "accper" here is undefined. This seems to be a bug.
… (CatchmentCNRst.F90)
…p.F90, CNCLM_DriverMod.F90)
…tial moving averages (GEOS_CatchCNCLM51GridComp.F90)
…edundant "implicit none" (CNCLM_DriverMod.F90)
This PR introduces the new version Catchment-CN5.1 land model, based on the Community Terrestrial System Model version 5.1 (CTSM5.1). Most of the files related to Catchment-CN5.1 are contained in the subdirectory ./GEOScatchCNCLM51_GridComp.
Documentation for Catchment-CN5.1 was added here:
[..]/GEOScatchCN_GridComp/GEOScatchCNCLM51_GridComp/README_CN51.mdSource code for Catchment-CN4.5 was removed throughout, except in the make_bcs package where the generation of model parameters for Catchment-CN4.5 is still needed for Catchment-CN5.1.
It is expected that the "remap_restarts" package is zero-diff for Catchment-CN4.0. There were, however, several bugs in CatchmentCNRst.F90 for Catchment-CN4.5 that were fixed for Catchment-CN5.1.
Related PRs (in the order in which they should be merged):
Outstanding issues to be resolved before merge:
To be addressed in a future PR:
cc: @gmao-rreichle , @rdkoster , @weiyuan-jiang , @biljanaorescanin