<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 26, 2015 at 5:04 AM, Nikos Alexandris <span dir="ltr"><<a href="mailto:nik@nikosalexandris.net" target="_blank">nik@nikosalexandris.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Disregard the previous one. This one without old 'rast' entries (instead, new 'raster').<span class=""><font color="#888888"><br>
<br></font></span></blockquote><div>Looks OK (I'm only looking at the source code). Just few things:<br><br>It seems you removed the option to add timestamps without registering to temporal database. I would just leave the else there. (You may want to create timestamps but not register or register later.)<br><br>It seems that tgis.open_new_stds is actually not according to PEP8. At least one contains whitespace at the end, rerun pep8.<br><br>I'm not sure if this is part of PEP8 but "comment graphics" is often discouraged; I personally don't see a reason for:<br><br># add timestamps either via temporal framework ----------------------<br># ------------------------------------------ outputs timestamped ---<br></div><div><br>It would be much better to include there actual sentence(s) which will be explanatory even for everybody. (I know what is behind "outputs timestamped" because I remember Soeren's commit but I don't think that this is well documented and generally known.)<br><br></div><div>Update Copyright year (I think it is OK to use 2013-2015 for simplicity - precise info is in Subversion).<br></div><div><br></div><div>When you are doing stylistic changes, you can also put spaces around assignment in r.mapcalc expressions. <br></div><br></div><div class="gmail_quote">For your todo: picture to the manual would be nice (perhaps two or three from a series).<br></div><div class="gmail_quote"><br><div>Nice work otherwise (e.g. option dependencies)!<br></div><div>Vaclav<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""><font color="#888888">
Nikos</font></span><br>_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br></blockquote></div><br></div></div>