threedi / hhnk-threedi-tools

1 stars 0 forks source link

Error to warning #89

Closed wvanesse closed 6 months ago

wvanesse commented 8 months ago

In sommige situaties zijn de aangepaste schematisatiefouten geen fouten maar bewust: Voor integrale modellen is het perc. imp. surface niet altijd 14,4 of 11,5, ook voor de hydraulische toets zijn er nog meer smaken dan die twee Impervious surfaces die niet zijn gemapped zijn wel raar, maar N&S stopt het zo in het integrale model, het crashed niet Waarom is een connection node zonder imp surface fout?

TODO: deze lijst komt in een CSV 'general_sqlite_checks.csv', maar die zie ik nergens terug in de gui. Lastig als de splitter wel aangeeft dat er fouten zijn maar ik die nergens zie.

TODO2: De checks bij het integrale model van Castricum duren wel 5 minuten, geen 30s zoals in de melding staat.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4372ce4) 42.17% compared to head (0a7abb8) 42.17%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #89 +/- ## ======================================= Coverage 42.17% 42.17% ======================================= Files 52 52 Lines 6571 6571 ======================================= Hits 2771 2771 Misses 3800 3800 ```

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

wvangerwen commented 6 months ago

TODO1: https://github.com/threedi/hhnk-threedi-plugin/issues/162 aangemaakt TODO2: dit kan ik niet reproduceren. Duurt bij mij ook voor castricum maar iets van 30s