-
Notifications
You must be signed in to change notification settings - Fork 6
Support storage column in basin profile #2278
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
base: main
Are you sure you want to change the base?
Support storage column in basin profile #2278
Conversation
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.
Overall this looks good! I left a few comments. I'm also missing an update of the Basin documentation in basin.qmd
.
core/src/validation.jl
Outdated
|
||
""" | ||
Validated the initialisation of basins. Each basin at least need a level-area or level storage relationship. | ||
We recommend to initialise all basins in the same way, which can be level-area, level-storage or level-area-storage. |
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.
It's good to have this here, but this is typically something that you also put into the documentation.
Co-authored-by: Bart de Koning <74617371+SouthEndMusic@users.noreply.github.com>
…https://github.com/Deltares/Ribasim into Ribasim#2254-Support-storage-column-in-Basin-profile
Fixes #2254
Notes: