<div dir="ltr">Hi,<div><br></div><div>I'm agree with Anna. In my opinion if the input are correct the warning is only a warning and it is correct!</div><div>It is important in order to inform the user but the code is optimized in order to automatically change the computational timestep (lower or higher) in order to obtain the best performance without compromize the stability. </div><div><br></div><div>Perhaps you can do soem review n the html manual page in order to help users to insert correct input?</div><div><br></div><div>All the best and happy new year ;-) </div><div>R <br></div><div><br></div><div> <br>
<pre class="" cols="72">--
Eng. Roberto Marzocchi, PhD
R&D coordinator
Gter srl Innovazione in Geomatica, Gnss e Gis
Spin-off Progetto Uni.T.I. promoted by UNIGE
Via Greto di Cornigliano 6r - 16152 Genova
P.IVA/CF 01998770992
ph: 010-8694830 Fax: 010-8694737
mob: 349-8786575
E-mail: <a href="mailto:roberto.marzocchi@gter.it">roberto.marzocchi@gter.it</a>
skype: roberto.marzocchi84
<a href="http://www.gter.it">www.gter.it</a>
--
Gter social
<a href="http://www.twitter.com/Gteronline">www.twitter.com/Gteronline</a> - <a href="http://www.facebook.com/Gteronline">www.facebook.com/Gteronline</a> - <a href="https://plus.google.com/+GterIt/posts">https://plus.google.com/+GterIt/posts</a>
<a href="http://www.linkedin.com/company/gter-srl-innovazione-in-geomatica-gnss-e-gis">www.linkedin.com/company/gter-srl-innovazione-in-geomatica-gnss-e-gis</a>
-----------------------------------------------------------------
Please consider the environment before printing this email! </pre></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-12-31 1:16 GMT+01:00 Anna Petrášová <span dir="ltr"><<a href="mailto:kratochanna@gmail.com" target="_blank">kratochanna@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">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></span><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"><span class=""><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></span>_______________________________________________<br>
grass-user mailing list<br>
<a href="mailto:grass-user@lists.osgeo.org" target="_blank">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>
<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>