-
Notifications
You must be signed in to change notification settings - Fork 0
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
120 hydroobjecten exporteren damo #130
base: main
Are you sure you want to change the base?
Conversation
Deze doet het nu voor kleinere polders
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #130 +/- ##
==========================================
- Coverage 42.17% 41.64% -0.53%
==========================================
Files 52 55 +3
Lines 6571 6661 +90
==========================================
+ Hits 2771 2774 +3
- Misses 3800 3887 +87 ☔ View full report in Codecov by Sentry. |
…20-hydroobjecten-exporteren-DAMO
PR voor 116 (project directory) - al goedgekeurd door Wietse - en 120 (damo export) - moet nog reviewed worden - |
database_to_gdf, | ||
sql_builder_select_by_location, | ||
) | ||
from hhnk_threedi_plugin.local_settings import DATABASES |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import vanuit plugins, verkeerde kant op is hierargie modules?
|
||
|
||
def DAMO_exporter(model_extent, table_names, EPSG_CODE="28992"): | ||
# if instead of a model_extent, polder_name is given as input. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docstring
gdf_model = gpd.clip(bbox_gdf, model_extent) | ||
|
||
# add gdf of table export from DAMO to dictonary | ||
dict_gdfs_damo[table] = gdf_model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waarom schrijven we niet naar geopackage?
We willen sowieso de export opslaan naar een DAMO.gpkg. Die moet uit de folders.py te halen zijn denk ik.
Nadeel van deze dict is ook als één van de tabellen faalt je ook alles kwijt bent.
schema = "DAMO_W" | ||
|
||
# make bbox --> to simply string request to DAMO db | ||
bbox_model = box(*model_extent.total_bounds) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het lijkt me netter om de bounding box functionaliteit toe te voegen aan research tools. Dan zorgen we ervoor dat die functie altijd werkt.
No description provided.