<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 30, 2015 at 6:40 PM, Panagiotis Mavrogiorgos <span dir="ltr"><<a href="mailto:pmav99@gmail.com" target="_blank">pmav99@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>In r.damflood if you try to run a calculation using invalid input you get a warning WRT <a href="https://en.wikipedia.org/wiki/Courant%E2%80%93Friedrichs%E2%80%93Lewy_condition" target="_blank">Courant-Friedrich-Lewy stability condition</a>. If the wikipedia article is accurate and it is indeed a necessary condition for convergence wouldn't it make sense to raise a fatal error (or whatever is more appropriate) as soon as the condition isn't respected? I mean I don't see much point in running a potentially lengthy simulation that will give meaningless results. Am I wrong in this?</div></div></blockquote><div><br></div><div>I was getting this warning several times in the beginning of the simulation, but the simulation still produced reasonable results. Lowering the time step would probably eliminate it. I think the warning is appropriate and shouldn't be changed to fatal error in my opinion, unless the original authors or anyone else who fully understands it say otherwise.</div><div><br></div><div>Best,</div><div><br></div><div>Anna</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>If my suggestion has indeed merit, could anyone suggest the most appropriate way to do this?I don't mind submitting a patch but I am afraid that I am not really familiar with GRASS C APIs.</div><div><br></div><div>These are the relevant lines:</div><div><a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.damflood/SWE.c#L504" target="_blank">https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.damflood/SWE.c#L504</a><br></div><div><a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.damflood/SWE.c#L708" target="_blank">https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.damflood/SWE.c#L708</a><br></div><div><br></div><div>thank you,</div><div>Panos</div><div><br></div></div>
<br>_______________________________________________<br>
grass-user mailing list<br>
<a href="mailto:grass-user@lists.osgeo.org">grass-user@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-user" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-user</a><br></blockquote></div><br></div></div>