<div dir="ltr">Well,<div><br></div><div>about the tests - generally yes, I'm writing it. Are you referring to <a href="http://r3.info">r3.info</a> or to g.search.* modules?  I could not found any example to <a href="http://r3.info">r3.info</a> tests</div><div><br></div><div>I've rewritten <a href="http://r3.info">r3.info</a> so that the code is the same, as <a href="http://r.info">r.info</a> (with -e flag) - but I can not tell, I'm satisfied with the result (there are line breaks and quotes in the strings, what IMHO is really not good)<br></div><div><br></div><div>My time resources allocatable to this task are slowly reaching to their end. I would need clear decision from you guys (since you are in charge to mess around GRASS code base):</div><div><br></div><div>Where to put print_output & supporting functions?</div><div>Where to write tests for them? (I assume standard unittest package is being used)</div><div>Can I omit g.search.data for now? (I would like to put some better code structure, as I can see it more clear now) - so I would put only g.search.module for now to GRASS trunk?</div><div><br></div><div>Thanks</div><div><br></div><div>Jachym</div></div><br><div class="gmail_quote"><div dir="ltr">po 30. 11. 2015 v 5:10 odesílatel Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>> napsal:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 29, 2015 at 7:06 AM, Jachym Cepicky <span dir="ltr"><<a href="mailto:jachym.cepicky@gmail.com" target="_blank">jachym.cepicky@gmail.com</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">the code is in the same repository but packed in the g.search.module directory now, not sure, if the g.extension will work again</blockquote></div><br></div></div><div dir="ltr"><div class="gmail_extra">g.extension supports only directories which have Makefile, i.e. Makefile must be in the root directory of the repository. See r.modis (it might be more complicated then you need):<br><br><a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.modis" target="_blank">https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.modis</a><br><a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.green" target="_blank">https://trac.osgeo.org/grass/browser/grass-addons/grass7/raster/r.green</a><br></div><div class="gmail_extra"></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 29, 2015 at 4:06 PM, Jachym Cepicky <span dir="ltr"><<a href="mailto:jachym.cepicky@gmail.com" target="_blank">jachym.cepicky@gmail.com</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"><div>I've patched <a href="http://r3.info" target="_blank">r3.info</a> a bit (not sure, whether it's ok) <a href="https://trac.osgeo.org/grass/changeset/66983/" target="_blank">https://trac.osgeo.org/grass/changeset/66983/</a> for potential better g.search.data output</div></blockquote></div><br></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_extra">Not really looked to the source code but with the commands:<br></div><div class="gmail_extra"><br>r3.mapcalc "test1 = rand(0, 1)" -s<br>r3.flow in=test1 flowaccumulation=test2<br><a href="http://r3.info" target="_blank">r3.info</a> test2 -gh<br><br></div><div class="gmail_extra">I get:<br><br>...<br>description="generated by r3.flow"<br>comments_0=comments_1=""<br><br></div><div class="gmail_extra">(comments at the same row)<br><br></div><div class="gmail_extra">BTW, have you considered writing a test? :)<br></div></div></div><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 29, 2015 at 4:06 PM, Jachym Cepicky <span dir="ltr"><<a href="mailto:jachym.cepicky@gmail.com" target="_blank">jachym.cepicky@gmail.com</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"><br><div>i
 would like to move colorize, print_results from g.search.module to some
 reachable place within existing grass python code base - any hint?</div></blockquote></div><br></div></div><div dir="ltr"><div class="gmail_extra"></div><div class="gmail_extra">There are three options, not sure now what is better:<br><br></div><div class="gmail_extra">grass.script.utils (scripting related, grass specific)<br></div><div class="gmail_extra">grass.utils (more general functions, to be created)<br></div><div class="gmail_extra">grass.tools (compilation, html related, to be created)<br><br></div><div class="gmail_extra">We will need grass.tools at some point anyway to organize the code which now spreads over several files in tools and man directories.<br><br></div><div class="gmail_extra">Not that you can also keep some functions in a "package directory" inside the addon like in r.modis but at least print_results looks like it should go to trunk.<br></div></div>
</blockquote></div>