Update arcgis pro tool #39
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now uses an output gpkg template including styling. Saves calculation settings and statistics.
Nu gebruik maken van contextlib.contextmanager om het aantal keer dat de gpkg wordt geopend. Verder ook field mapping geoptimaliseerd.
QT designer gebruikt om de knop toe te voegen. Daarna in python class functie reset_parameters gemaakt om de functie van deze knop in te stellen.
Meer inputs. En mogelijk om te kiezen tussen opslaan naar gpkg of temp. laag QGIS.
…rharding Nu meer types verharding bij stats berekening.
To do: generiek maken zodat het ook gebruikt kan worden voor GWSW download.
Nu ook totale oppervlakken van verschillende types
Wordt geskipt als er sprake is van een eerste versie (geen bestaande resultaten gebruikt).
Checks worden nu ook goed weggeschreven in template gpkg
Foutmelding wordt nu gegenereerd wanneer gemeente geen GWSW data heeft. Werkt nu zowel voor BAG als voor GWSW. Ook wordt er rekening gehouden met gemeentenamen met spaties en -'s
Werkt nu voor wegschrijven van temp laag. Het houdt nog geen rekening met handmatige wijzigingen. Ook heeft de front-end nog geen fucntie. De optie leidingcodes koppelen aan-/uitzetten werkt nog niet. Als het wordt uitgezet, moeten de kolommen met leidingcode worden verborgen.
Styling werkt alleen nog niet goed. Komt dat door de aanpassingen die ik heb gedaan?
Nu alleen zowel fid als id in tabel. Dit later nog fixen. Daarnaast moet er nog een aanpassing komen aan de core dat de gebruiker kan kiezen of hij wel/niet leidingcodes wil koppelen.
Styling werkt nu weer naar behoren.
Bleek toch nog een fout in te zitten. Nu opgelost door niet vacuum toe te passen.
Bij de raw import wordt nu een spatial filter toegepast. Hierdoor worden niet alle feature geïmporteerd, maar alleen de features die intersecten met de extent (als dit is opgegeven). GML aanpassen (in het juiste formaat) wordt niet ondersteund door ogr. Daarom was dit geen optie meer.
Waterpasserende verharding BGT/BAG vergelijking Begin van relatieve hoogteligging vlakken
Handmatige wijzigingen vergelijken met BGT data
Front end aanpassingen
…' into update_arcgis_pro_tool
Mocht er sprake zijn van een invalid geometry (bijv. met multiple rings (gaten) of een MultiPolygonZ), dan wordt de geometry omgezet naar de bounding box ervan, om toch de BGT data te downloaden.
…' into update_arcgis_pro_tool
2_Controles, error_code van int64 naar int32
…' into update_arcgis_pro_tool
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JULLIE PR moet eerst. Daarna moet ik een merge vanuit de master uitvoeren en dan pas kan mijn PR.
Wijzigingen: