Jenn

Like @dickinsonre explained `MIN_RUNOFF_FLOW` is an internal value in the SWMM C engine. You can find the value here: https://github.com/OpenWaterAnalytics/Stormwater-Management-Model/blob/025337c026dc2b6d493fb23ab05016329f7547cb/src/solver/consts.h#L47

Correct, I think it is fair to interpret `total_periods` as the duration when outfall has inflow. `total_periods` is only incremented when flow exceeds MIN_RUNOFF_FLOW.

Thank you for looking into this @bemcdonnell! Should we create an issue for this in OWA/SWMM or should we correct the output in pyswmm? @MortenGrum thank you for writing this...

Ok, issue created and will be addressed in OWA/SWMM: https://github.com/OpenWaterAnalytics/Stormwater-Management-Model/issues/353

Hi @MortenGrum, I will keep this issue open until it has been resolved. Thanks for identifying this bug. The report is super helpful.

Hello! Can you share your model and code snippet? It will help to debug.

Should the overwrite flag be set to True as default to maintain previous behaviour?

This is fixed in pyswmm v1.0 release.

@tianmaxingkong12 Can you share your model inp file and the code snippet?

Interesting. There is no runoff from the subcatchment because all the flow goes to the green roof (occupies 100% of the subcatchment). In the .rpt file, the peak flow for...