Stormwater-Management-Model
Stormwater-Management-Model copied to clipboard
Maximum infiltration volume (Modified Horton method)
The Maximum Infiltration Volume feature works fine with the original Horton infiltration model (it correctly shuts off infiltration at 1-5 inches for the various subcatchments in the attached file). But it gives incorrect results when you switch to Modified Horton.
It seems as if Fmax is used for both Horton and Modified Horton in infil.c - is it being exported for the Modified Horton option? What does the inp file look like to you as you change Horton methods?
The only difference is the Infiltration method in the [OPTIONS] section of the input file.
This bug has now been fixed in PR #56 . Thanks @MikeGregory-Canada .
Yes, thanks @MikeGregory-Canada for finding this bug from Canada.
Closed in Release v5.2.0
Entirely possible I'm missing something here, but is there any chance this issue has somehow reverted?
In the 5.2.4 GUI, using a simple example model with six subcatchments incremented up on Max Vol. from 0 to 5", via traditional Horton I get:
But when I toggle the infiltration model to Modified Horton I get:
Model details:
[SUBCATCHMENTS]
;;Name Rain Gage Outlet Area %Imperv Width %Slope CurbLen SnowPack
;;-------------- ---------------- ---------------- -------- -------- -------- -------- -------- ----------------
MaxVol_0 RG1 9 10 10 500 0.01 0
MaxVol_1 RG1 10 10 10 500 0.01 0
MaxVol_2 RG1 13 10 10 500 0.01 0
MaxVol_3 RG1 22 10 10 500 0.01 0
MaxVol_4 RG1 15 10 10 500 0.01 0
MaxVol_5 RG1 23 10 10 500 0.01 0
[SUBAREAS]
;;Subcatchment N-Imperv N-Perv S-Imperv S-Perv PctZero RouteTo PctRouted
;;-------------- ---------- ---------- ---------- ---------- ---------- ---------- ----------
MaxVol_0 0.001 0.10 0.05 0.20 25 OUTLET
MaxVol_1 0.001 0.10 0.05 0.20 25 OUTLET
MaxVol_2 0.001 0.10 0.05 0.20 25 OUTLET
MaxVol_3 0.001 0.10 0.05 0.20 25 OUTLET
MaxVol_4 0.001 0.10 0.05 0.20 25 OUTLET
MaxVol_5 0.001 0.10 0.05 0.20 25 OUTLET
[INFILTRATION]
;;Subcatchment Param1 Param2 Param3 Param4 Param5
;;-------------- ---------- ---------- ---------- ---------- ----------
MaxVol_0 1.50 0.3 4.14 0.50 0
MaxVol_1 1.50 0.3 4.14 0.50 1
MaxVol_2 1.50 0.3 4.14 0.50 2
MaxVol_3 1.50 0.3 4.14 0.50 3
MaxVol_4 1.50 0.3 4.14 0.50 4
MaxVol_5 1.50 0.3 4.14 0.50 5
@dickinsonre Thanks Bob. Can I read your thumbs up as "issue confirmed"? I'm always hesitant to rule out the possibility of simple user error, especially when the user is @joelnc. If this functionality is truly broken in 5.2.4, then I can just move ahead without it for now.
@joelnc, it appears you are right. The fix was not propagated to 5.2.4 somehow. I am trying to figure out where this divergence came about. I will reopen this ticket in the meantime.
Thanks Caleb.
For some unknown reason the Modified Horton bug fix in PR #56 never actually got added to release 5.1.015 and all subsequent releases. The attached zip file contains the updates to infil.h and infil.c that fix the bug (tested on example1.inp by setting the max. infiltration volume to 2" for all subcatchments).
Thank you all. Just completed some runs with the bug_fix branch, and the functionality is now working as expected.