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

Qgep swmm phase3 enhancements #190

Merged
merged 53 commits into from
Mar 2, 2023

Conversation

tproduit
Copy link
Contributor

@tproduit tproduit commented Nov 25, 2022

Messages and warning
• Most of the message regarding computation of values (roughness, dry weather flow) are migrated in the SWMM descriptions of the objects
• Info, warnings and errors are printed in the processing interface in QGIS

Catchment redirection
• See issue QGEP/QGEP#730 and QGEP/QGEP#560

Reach roughness:
• See issues QGEP/QGEP#727 and QGEP/QGEP#728

Special structures:
• See issue QGEP/QGEP#719
• Export pump curve
• Export tank storage curve
• Modelise prank weirs as outlet with curve

Selection
• See issue QGEP/QGEP#720 and QGEP/QGEP#729
• Export selected network (included secondary)

Add raingage location
• Related to issue QGEP/QGEP#646

Solve bug with the export of land use

tproduit and others added 30 commits October 11, 2022 16:14
force 3D geom vw_conduites
add X,Y location for raingages
change to export special structures
compute roughness using Strickler or Colebrook vw_conduites
@ponceta ponceta requested review from sjib and olivierdalang and removed request for sjib December 6, 2022 15:35
03_qgep_db_dss.sql Outdated Show resolved Hide resolved
03_qgep_db_dss.sql Outdated Show resolved Hide resolved
03_qgep_db_dss.sql Outdated Show resolved Hide resolved
03_qgep_db_dss.sql Outdated Show resolved Hide resolved
Copy link
Contributor

@sjib sjib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tproduit Can you check the proposed changes

@ponceta ponceta added the feature New functionnality or behaviour improving/extending user experience label Jan 23, 2023
Copy link
Contributor

@sjib sjib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets try it like this

@ponceta
Copy link
Member

ponceta commented Jan 24, 2023

@olivierdalang do we want to have it in 1.5.7 or is 1.5.8 fine (we should then publish a 1.5.7 release with the few fixes that we brought in the meantime)?

@ponceta ponceta added this to the 1.5.8 Swmm enhancements milestone Jan 27, 2023
@ponceta
Copy link
Member

ponceta commented Jan 27, 2023

We have to fix 1.5.7 and continuous integration and we will be ready to add this to 1.5.8

@ponceta ponceta closed this Feb 24, 2023
@ponceta ponceta reopened this Feb 24, 2023
@ponceta ponceta closed this Mar 2, 2023
@ponceta ponceta reopened this Mar 2, 2023
@ponceta ponceta merged commit df15d76 into QGEP:master Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionnality or behaviour improving/extending user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants