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 aggregation view for channel and georeferenced channel damages #531

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

cymed
Copy link
Contributor

@cymed cymed commented Jan 16, 2025

I found having a view vw_tww_channel practical, especially when planning maintenance events. The view aggregates all reach geometries linked to a channel. For now, the view is read-only

cymed and others added 8 commits January 16, 2025 09:22
I found having a read-only view vw_tww_channel practical, especially when planning maintenance events. The view aggregates all reach geometries linked to a channel.
@ponceta
Copy link
Member

ponceta commented Jan 17, 2025

Thank you for this new functionnality!

image

@cymed do we have a way to control the attributes order of the created view?

As it is an aggregation of compoundcurves should it be a multicompoundcurve geometry?

@cymed
Copy link
Contributor Author

cymed commented Jan 17, 2025

As it is an aggregation of compoundcurves should it be a multicompoundcurve geometry?

Yes. LineMerge aggregates to singlepart if the aggregated lines touch, but in a database this is not always the case - better to use multipart

@ponceta
Copy link
Member

ponceta commented Jan 17, 2025

Just as a small context comment, the LineMerge function aggregates to multilinestring and generalizes curved elements :

image

But as long as it is used in read only it should not break anything.

@ponceta ponceta added this to the TEKSI Wastewater 2025.0 milestone Jan 17, 2025
@ponceta ponceta self-requested a review January 17, 2025 13:33
@cymed
Copy link
Contributor Author

cymed commented Jan 17, 2025

Added a view tww_app.vw_tww_damage_channel to georeference TV inspection damages

@sjib
Copy link
Contributor

sjib commented Jan 17, 2025

@cymed
Copy link
Contributor Author

cymed commented Jan 20, 2025

New feature that could help in this :

https://www.qgis.org/project/visual-changelogs/visualchangelog340/#feature-add-linear-referencing-symbol-layer-type

The feature does allow using a fixed interval, a z or an M value, or all vertices. None of these options work for channel damages. The feature is mainly good for hydrology

@cymed cymed changed the title add vw_tww_channel [WIP] add aggregation view for channel and georeferenced channel damages Jan 20, 2025
@cymed cymed requested a review from ponceta January 20, 2025 13:47
@cymed cymed added the app_schema Concerns the application schema label Jan 20, 2025
@cymed cymed marked this pull request as ready for review January 21, 2025 14:04
@cymed cymed changed the title [WIP] add aggregation view for channel and georeferenced channel damages Add aggregation view for channel and georeferenced channel damages Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app_schema Concerns the application schema
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants