Skip to content

Conversation

gmao-jkolassa
Copy link
Contributor

@gmao-jkolassa gmao-jkolassa commented Jul 1, 2025

This PR introduces the new version Catchment-CN5.1 land model, based on the Community Terrestrial System Model version 5.1 (CTSM5.1).

CONTINGENCY: Requires GEOSgcm_GridComp PR 1129

Note: The GEOSldas branch associated with GEOSldas PR 829 can be used for testing.

cc: @gmao-rreichle , @rdkoster , @weiyuan-jiang , @biljanaorescanin

Copy link

github-actions bot commented Jul 1, 2025

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

Copy link
Collaborator

@gmao-rreichle gmao-rreichle left a comment

Choose a reason for hiding this comment

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

@gmao-jkolassa : Thanks for putting together the PR. I made a few very minor changes. Nothing set in stone

@weiyuan-jiang
Copy link
Contributor

Have you tried remap_restart package for the CNCLM51 ? @gmao-jkolassa

@weiyuan-jiang
Copy link
Contributor

I expect some changes in this block. At least we should say we don't support restart=0

elif (int(self.rqdExeInp['RESTART']) == 0) :
if (self.catch == 'catch'):
self.in_rstfile = '/discover/nobackup/projects/gmao/ssd/land/l_data/LandRestarts_for_Regridding' \
'/Catch/M09/20170101/catch_internal_rst'
self.in_tilefile = '/discover/nobackup/projects/gmao/ssd/land/l_data/geos5/bcs/CLSM_params' \
'/mkCatchParam_SMAP_L4SM_v002/SMAP_EASEv2_M09/SMAP_EASEv2_M09_3856x1624.til'
elif (self.catch == 'catchcnclm40'):
self.in_rstfile = '/discover/nobackup/projects/gmao/ssd/land/l_data/LandRestarts_for_Regridding' \
'/CatchCN/M36/20150301_0000/catchcnclm40_internal_dummy'
self.in_tilefile = '/discover/nobackup/projects/gmao/bcs_shared/legacy_bcs/Heracles-NL/SMAP_EASEv2_M36/SMAP_EASEv2_M36_964x406.til'
elif (self.catch == 'catchcnclm45'):
self.in_rstfile = '/discover/nobackup/projects/gmao/ssd/land/l_data/LandRestarts_for_Regridding' \
'/CatchCN/M36/19800101_0000/catchcnclm45_internal_dummy'
self.in_tilefile = '/discover/nobackup/projects/gmao/bcs_shared/legacy_bcs/Icarus-NLv3/Icarus-NLv3_EASE/SMAP_EASEv2_M36/SMAP_EASEv2_M36_964x406.til'

@gmao-jkolassa
Copy link
Contributor Author

I expect some changes in this block. At least we should say we don't support restart=0

Ok, sounds good. Also, we can add CN_CLM51 to this block once we have fully spun-up restarts for it.

@gmao-jkolassa
Copy link
Contributor Author

gmao-jkolassa commented Jul 7, 2025

Have you tried remap_restart package for the CNCLM51 ? @gmao-jkolassa

No, I haven't. See Rolf's earlier comment that this might need a separate PR.

See GEOS-ESM/GEOSgcm_GridComp#1129 (comment)

@weiyuan-jiang
Copy link
Contributor

I hope this line still works.

catch_obj.remap()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants