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

Add support for transient UserDemand return factor #1230

Closed
SouthEndMusic opened this issue Mar 11, 2024 · 4 comments · Fixed by #1727
Closed

Add support for transient UserDemand return factor #1230

SouthEndMusic opened this issue Mar 11, 2024 · 4 comments · Fixed by #1727
Assignees
Labels
allocation Allocation layer core Issues related to the computational core in Julia improvement Improvements of the usability of existing functionality physics Physical process representation

Comments

@SouthEndMusic
Copy link
Collaborator

SouthEndMusic commented Mar 11, 2024

@Fati-Mon requested for one of her models that we support transient UserDemand return flow.

This is currently not supported, but it is already possible to put it in the UserDemand \ time table. #1229 is for avoiding confusion about this for time tables in general.

@SouthEndMusic SouthEndMusic added the improvement Improvements of the usability of existing functionality label Mar 11, 2024
@gijsber gijsber added physics Physical process representation allocation Allocation layer labels Apr 22, 2024
@SnippenE
Copy link

from planning session 16-5-2024: check if a flag is implemented in code.

@SnippenE SnippenE added core Issues related to the computational core in Julia validation Related to model validation allocation Allocation layer and removed allocation Allocation layer validation Related to model validation labels Jul 15, 2024
@Jingru923 Jingru923 linked a pull request Aug 14, 2024 that will close this issue
@Fati-Mon
Copy link
Collaborator

@Jingru923 how do you define the return flow over time?

I have this now:

# DEMAND (irrigation)
model.user_demand.add(
    Node(6, Point(-1.5, 1.0), name='IrrA'),
    [user_demand.Time(
        demand=[0.0, 0.0, 10, 12, 12, 0.0],
        return_factor=[0.0, 0.0, 0.2, 0.5, 0.5, 0.0],
        min_level=0,
        priority=1,
        time=[starttime, "2022-03-31", "2022-04-01", "2022-07-01", "2022-09-30", "2022-10-01"]
        )
    ]
)

When it irrigates 10 return 20% and when it irrigates 12 return 50%
But return over time remains zero.

@Jingru923
Copy link
Contributor

Hi @Fati-Mon the code looks correct to me. But we will have a look.

@visr
Copy link
Member

visr commented Aug 29, 2024

@Fati-Mon which version are you running with, what is the output of ribasim --version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
allocation Allocation layer core Issues related to the computational core in Julia improvement Improvements of the usability of existing functionality physics Physical process representation
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants