-
Notifications
You must be signed in to change notification settings - Fork 5
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
[WIP] Add Tool to split reaches with wastewater nodes and points #142
base: main
Are you sure you want to change the base?
Conversation
and alter some messagebar stuff
for more information, see https://pre-commit.ci
when splitting a reach the user must have this options:
The tool has then to:
If new wastewater_node:
If no wastewater_node:
If split channel:
There are never automatic changes to connections of catchment_areas (also because catchment_areas are normally not connected to reaches). |
Do we need an option here to split a reach without adding a wastewater node? Or would it suffice to create a reach-reach connection when snapping on a reach while digitising a new one?
Imho snapping on the reach and digitising a wastewater node does just that (I'd need to check if my current solution allows using the construction mode on advanced digitising)
manually editing the reach point level is only needed if we split using reaches. Otherwise we can use (wn_)bottom_level
length_effective?
Hasn't the reach reached its end-of-life when we split it, so rather two new reaches, but still the old channel?
This is implemented
Good point, tbd.
TBD
Implemented
Implemented
Implemented
Implemented, can be altered in the dialog box
gross_costs? replacement_value? rv_base_year? subsidies?
TBD
|
Split without node: in secondary network is no need of nodes for reach to reach connections. Channel is not defined from manhole to manhole, if there is another primary channel connected in between without manhole. Channel is so long a channel, as all attributs are equal. The older the network gets, the more you have cases, where you have to document, that just a part of a channel was replaced. It's then not only a change in material, but also in construction year... and there are a lot of possiblities, where you want to split just reach or reach and channel. manually splitlevels are needed if the slope is not continues (e.g. you have measured the channel at this point). Otherwise you have to correct it manually afterwoods 2 or 3 times (if you don't forget it). length effective is calculated I want all attributes to be copied. The user has then normally to change at least one attribute (e.g. material and he should then think about the other attributes, that a PP channel will not have the same construction_year as the normal_concrete channel). If there are at the end two new reaches or an old and a new one does not matter. Do describe it's easier to take about old and new... |
|
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
This WIP will eventually add a tool to create a wastewater structure or a wastewater node on a reach and split the reach in two parts. The Reaches' corresponding Wastewater Structure and Channel attributes are preserved for wastewater node but re-created for Wastewater structure. The old reach is dropped.
A new button is added
that adds a popup in analogy to the wizard
and then lets you digitise a wastewater structure or wastewater node. The reach that is snapped upon is split by the node (there is a bug in that step for now, see below)
TBD:
Programming:
QgsGeometry.vertexIdFromVertexNr(vertexIndex)
seems to have problems whenvertexIndex==0
. The same problem exists for TwwMapToolAddReach. Due to these problems, the current code cannot be fully testedUsability (Tagging @urskaufmann and @tomep for input)
Note: This PR needs thorough testing and will not make it before the feature freeze. Aiming for 2024.1