nogis: introduce slope width

storm-fsv-cvut / smoderp2d

SMODERP2D - Distributed event-based model for surface and subsurface runoff and erosion

https://storm-fsv-cvut.github.io/smoderp2d-manual/

GNU General Public License v3.0

9 stars 4 forks source link

nogis: introduce slope width #113

Closed landam closed 3 years ago

landam commented 3 years ago

Config:

[domain]
slope_width: 1 # required

[x] Profile:
- maximalSurfaceFlow
- totalRunoff
times slope_width
[x] Hydrograph:
- @jerabekjak To be defined.

pesekon2 commented 3 years ago

Implemented into profile in c6da485dc7d79ce6db43a0055917f4ab0af025ac

jerabekjak commented 3 years ago

Should I do the same in hydrographs? As mention in the first comment?

pesekon2 commented 3 years ago

Yes, please, if you can.

But if you don't have time or something, just let me know what should be multiplied and I can try it.

landam commented 3 years ago

@pesekon2 @jerabekjak Please clarify what is need in order to close the issue. Thanks.

jerabekjak commented 3 years ago

I've done it in the commit cf7ff7adea9afa108fea9a2d43e8f7888e231a95 to a new branch. Please review if its a good solution.

jerabekjak commented 3 years ago

@kavkapet Should be the infiltration and rainfall also multiplied with the slope width? The multiplication is done aside from the te water balance computation so the water balance computation should not be affected.

jerabekjak commented 3 years ago

multiply the cumulative value which does to hydrograph by slope width (which differs from 1 only in case of nogis provider) see commit 8f182e28fd0561a36812c5d9fc8fc1303cd66ffc