Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to CatchCN GridComp so it can reproduce Fanwei's science-tested LDASsa version of CatchCN #311

Merged
merged 8 commits into from
Oct 29, 2020

Conversation

gmao-jkolassa
Copy link
Contributor

This branch contains a code modification needed to run GEOSldas with GEOSgcm_GridComp branch feature/jkolassa_#287_fixes that is currently submitted as draft pull request https://github.com/GEOS-ESM/GEOSgcm_GridComp/pull/349. The specific modification is:

GEOS_LdasGridComp.F90:
• Added LSM_CHOICE to SurfParams_init call in Initialize subroutine

This pull request should be approved shortly after GEOSgcm_GridComp pull request #349 is approved.

cc: @gmao-rreichle , @biljanaorescanin , @smahanam , @rdkoster , @gmao-jkolassa

@gmao-rreichle gmao-rreichle changed the title added LSM_CHOICE to SurfParams_init call Updates to CatchCN GridComp so it can reproduce Fanwei's science-tested LDASsa version of CatchCN Sep 18, 2020
@biljanaorescanin
Copy link
Contributor

Addressed with additional comments in PR #349.

@biljanaorescanin biljanaorescanin marked this pull request as ready for review October 23, 2020 13:35
@biljanaorescanin biljanaorescanin requested review from a team as code owners October 23, 2020 13:35
gmao-jkolassa and others added 2 commits October 28, 2020 14:50
Changed GEOSgcm_GridComp branch to 'develop' since PR#349 has been merged into develop.
@biljanaorescanin biljanaorescanin merged commit 11f5d7e into develop Oct 29, 2020
@biljanaorescanin biljanaorescanin deleted the feature/jkolassa_#287_fixes_match branch October 29, 2020 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants