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.

Sign up for GitHub

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

Jump to bottom

Include method to setup retention structures #144

Open
2 tasks done
shartgring opened this issue Sep 25, 2024 · 0 comments · May be fixed by #170
Open
2 tasks done

Include method to setup retention structures #144

shartgring opened this issue Sep 25, 2024 · 0 comments · May be fixed by #170
Assignees
@shartgring
Labels
enhancement New feature or request

Comments

@shartgring
Copy link
Collaborator

Version checks

  • I have checked that this issue has not already been reported.
  • I have checked that this bug exists on the latest version.

Reproducible Example

Currently, retention basins cannot be included. These are supported by Delft3D-FM 1D2D in the form of storage nodes. So, it would be nice to include a method to add there retention nodes

Current behaviour

No storage nodes can be added to model

Desired behaviour

Include a method to add storage nodes to model

Additional context

No response
@shartgring shartgring added the enhancement New feature or request label Sep 25, 2024
@shartgring shartgring self-assigned this Sep 25, 2024
@shartgring shartgring linked a pull request Oct 2, 2024 that will close this issue
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shartgring shartgring

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

New method setup_retentions Deltares/hydromt_delft3dfm
1 participant
@shartgring