search

threedi / hhnk-threedi-tools

1 stars 0 forks source link

Calculation rasters #85

Closed d2hydro closed 3 months ago

d2hydro commented 1 year ago
  1. gebruiken van raster_verwekt folder voor het berekenen van schade: https://github.com/threedi/hhnk-threedi-plugin/issues/126
  2. hoogteraster 10cm opgehoogd t.b.v schadeberekening
codecov[bot] commented 1 year ago

Codecov Report

Attention: Patch coverage is 71.13402% with 28 lines in your changes missing coverage. Please review.

Project coverage is 41.37%. Comparing base (01af596) to head (8795f0b). Report is 4 commits behind head on main.

:exclamation: Current head 8795f0b differs from pull request most recent head 1a8d0f7

Please upload reports for the commit 1a8d0f7 to get more accurate results.

Files Patch % Lines
...eedi_tools/core/result_rasters/calculate_raster.py 12.50% 28 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #85 +/- ## ========================================== - Coverage 42.17% 41.37% -0.80% ========================================== Files 52 51 -1 Lines 6571 6477 -94 ========================================== - Hits 2771 2680 -91 + Misses 3800 3797 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

d2hydro commented 11 months ago

@wvangerwen callantsoog_test\Notebooks\03_nabewerking_klimaatsommen.ipynb heeft een verwijzing naar een niet-bestaande dem:

dem = folder.model.schema_1d2d_ggg.rasters.dem

Dit is blijkbaar default, zie: https://github.com/threedi/hhnk-threedi-tools/blob/manipulated_rasters/hhnk_threedi_tools/utils/notebooks/03_nabewerking_klimaatsommen.ipynb.

Wanneer ik dit zo laat staan, dan krijg ik bovenstaande foutmelding, omdat deze DEM niet bestaat. Wanneer ik deze regel vervang voor dem = folder.model.schema_base.rasters.dem, is deze foutmelding weg.

Vragen:

  1. In calculate_raster BaseCalculatorGPKG.from_folder() verwijzen we altijd naar schema_base.rasters.dem als dem voor schade-berekeningen. Deze notebook suggereert dat de keuze voor de dem vrij is. Moeten we BaseCalculatorGPKG.from_folder() hierop aanpassen?
  2. Het schalen van het raster niet in 03_nabewerking_klimaatsommen.ipynb coden, maar oppakken uit calculate_raster.py?
wvangerwen commented 11 months ago
  1. Ik pak die 1d2d_ggg dem omdat er technisch gezien meerdere DEM's gebruikt kunnen worden en dan heb je iig de DEM waarmee gerekend is. Maar volgens mij is dat niet echt relevant omdat het in de praktijk niet verandert. Ik kan me voorstellen dat bij scenarioberekeningen het wel nuttig is om hier een keuze in te kunnen maken. Volgens mij laten we in de klimaatsommen notebook sowieso al zien welke DEM gekozen is. Goed voorstel om in de from_folder wel de DEM ook mee te geven, en dan default naar schema_base
  2. Ja dat is logischer omdat je niet alleen bij klimaatsommen rasters wilt maken. Die kijkt dan nog steeds naar de raster_verwerkt map neem ik aan?
  3. een FileNotExists error zou netter zijn. Maar als we dat voor elk attribute moeten gaan checken dan krijgen we wel erg veel overhead, dus misschien maar laten.
DanielTollenaar commented 8 months ago

ToDo: