-
Notifications
You must be signed in to change notification settings - Fork 8
River routing for GEOSldas #1023
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
Draft
zyj8881357
wants to merge
31
commits into
develop
Choose a base branch
from
feature/yujinz/Routing_GEOSroute
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
7faae9f
branch feature/yujinz/Routing_GEOSroute started, offline model was pu…
54950b8
change some paths for Discover
982bdd9
script build is replaced by run
f2a19da
readme.txt added
bf8b9a8
update K_str
71e44c5
Merge remote-tracking branch 'origin/develop' into feature/yujinz/Rou…
c6e341b
removing some Chinese characters in the comments
d2a0c95
more cleaning up
0d8ebcb
Merge remote-tracking branch 'origin/develop' into feature/yujinz/Rou…
5e9377a
move offline model to dir offline
7ceed5a
routing model was implemented to GEOSldas
17b7d4b
back to TAKI
431674c
back to Discover
5d98b30
a small bug fix for the offline model
abcf705
update the offline model to the coupled model
416a94f
Merge branch 'develop' into feature/yujinz/Routing_GEOSroute
gmao-rreichle 96cbcd6
reservoir module added
ebe5287
temporarily disable flood control for dams
a81bcea
preprocessing for routing model added
0c5d779
code clean up, and comment&documents added
af2ec8b
Merge remote-tracking branch 'origin/develop' into feature/yujinz/Rou…
9a08d7b
'==' changed to '.eqv.'
32fe955
'.eqv.' replaces '=='
1fb57bb
Merge branch 'develop' into feature/yujinz/Routing_GEOSroute
gmao-rreichle ef58210
changed variable name NUM_CATCH -> NUM_CATCH_ENS for clarity (GEOS_La…
gmao-rreichle 85af5a7
avoid duplicate hard-coding of #Pfaf catchments; fixed indent, alignm…
gmao-rreichle 4e3cfec
added comments to clarify connected "RUNOFF" variable (GEOS_LandGridC…
gmao-rreichle e367061
Update CMakeLists.txt
weiyuan-jiang f1c450d
copying river_input through make_bcs
c60c59b
river routing input data pointing to bcs
636cbf1
-diag-disable=10448 added in the building of routing_model preproc
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
5 changes: 3 additions & 2 deletions
5
...physics_GridComp/GEOSsurface_GridComp/GEOSland_GridComp/GEOSroute_GridComp/CMakeLists.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,11 @@ | ||
esma_set_this () | ||
|
||
set (srcs | ||
#GEOS_RouteGridComp.F90 | ||
GEOS_RouteGridComp.F90 | ||
routing_model.F90 | ||
reservoir.F90 | ||
) | ||
|
||
esma_add_library (${this} SRCS ${srcs} DEPENDENCIES MAPL ESMF::ESMF NetCDF::NetCDF_Fortran) | ||
esma_add_library (${this} SRCS ${srcs} DEPENDENCIES MAPL GEOS_LandShared ESMF::ESMF NetCDF::NetCDF_Fortran) | ||
|
||
install(PROGRAMS build_rivernetwork.py DESTINATION bin) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment is not about this particular line, but rather about the placement of GEOS_RouteGridComp inside GEOS_LandGridComp. Ultimately, we will need to route runoff from the land and the landice tiles. To do that, I think we need to place GEOS_RouteGridComp one level up, within GEOS_SurfaceGridComp (that is, next to GEOS_LandGridComp). Making this change will likely require extensive re-wiring. I wonder if it's best to tackle this re-wiring first or delay the move until later.