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

QGIS-TIM (transient) Headline Sink discharge with time #143

Open
jasper-hve opened this issue Mar 18, 2024 · 1 comment
Open

QGIS-TIM (transient) Headline Sink discharge with time #143

jasper-hve opened this issue Mar 18, 2024 · 1 comment

Comments

@jasper-hve
Copy link

For steady state: a discharge is displayed per line segment of a headline sink. Very conventient. I can imagine that this is difficult for transient calculations. Perhaps the attribute table can display discharge with time step as column header. Then at least you have an output.

Perhaps also applicable to other elements of a model

@Huite
Copy link
Contributor

Huite commented Apr 14, 2024

This is indeed a good suggestion, in general we'd like the capabilities of the transient elements to match the steady-state elements.

One of the things that makes this easier is that TTim now also has a method to give the discharge per segment:
https://ttim.readthedocs.io/en/latest/05api/generated/generated/generated/ttim.linesink.HeadLineSinkString.discharge_list.html#ttim.linesink.HeadLineSinkString.discharge_list

They can then be added with a time column, and QGIS temporal controller takes care of it, like it does for the contours.

The main annoyance with implementing this is that I need information somewhere to determine at which time the discharge should be computed. For the head results, I use values set in the "TTim Domain" entries. However, it doesn't seem like a good idea to use these to show the discharges: computing heads is expensive, so if you want to get detailed discharge info, you would be waiting on something you didn't care for.

I haven't figured out yet what kind of table would be the most appropriate to set these times values in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants