<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 27, 2016 at 9:05 PM, Rashad Kanavath <span dir="ltr"><<a href="mailto:mohammedrashadkm@gmail.com" target="_blank">mohammedrashadkm@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 dir="ltr"><div><br><br></div><div>I had pushed a updated version of code:<br><br></div><div>Done:<br></div><div>1. input is a imagery group from i.group (handles all raster in the group)<br></div><div>2. removed rgb2lab conversion<br></div><div>3. implementation of id output map<br></div><div>4. replaced  fmin with MIN and MAX macro<br></div><div>5. compactness is a new option to module? (default is 20 from paper)<br><br></div></div></blockquote><div>Wonderful, thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>TODO:<br></div><div>1. add option for SLICO<br></div><div>2. use segment library to avoid memory limitation when reading raster data<br><br></div><div>Hope this works now. Let me know If I missed something else.<br><br></div></div></blockquote><div>Code clean-up:<br>- please use tools/grass_indent.sh<br></div><div>- move variable declarations to the beginning of a code block<br></div><div>- convert C++ style comments to C style comments<br></div><div>- line 155<br>  int sz = nrows * ncols;<br><br></div><div>can cause integer overflow. Use off_t (large raster maps can only be handled with LFS, i.e. off_t being a 64bit integer).<br></div><div><br></div><div>- fix L157: must be<br></div><div>  int **pdata;<br>  <br>  pdata = G_malloc(sizeof(int) * group_ref.nfiles);<br><br></div><div>- Lines 317,318 are correct, line 320 is wrong<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>And...<br><br></div><div>Happy New Year to all :-)<br></div><div><br></div></div></blockquote><div>Happy new year to you too!<br><br></div><div>Markus M <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="gmail-h5">On Thu, Dec 22, 2016 at 10:12 AM, Markus Neteler <span dir="ltr"><<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="gmail-h5">On Thu, Dec 22, 2016 at 12:01 AM, Markus Metz<br>
<<a href="mailto:markus.metz.giswork@gmail.com" target="_blank">markus.metz.giswork@gmail.com</a><wbr>> wrote:<br>
...<br>
<span>> Adding<br>
><br>
> #undef MIN<br>
> #define MIN(a,b) ((a) < (b) ? (a) : (b))<br>
><br>
> as in other GRASS modules is a nice solution to keep things simple and<br>
> portable.<br>
<br>
</span>... how about a adding this and similar definitions to gis.h?<br>
<br>
markusN<br>
</div></div><span class="gmail-"><div class="gmail-m_-2673680542489267441HOEnZb"><div class="gmail-m_-2673680542489267441h5">______________________________<wbr>_________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org" target="_blank">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman<wbr>/listinfo/grass-dev</a></div></div></span></blockquote></div><span class="gmail-HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="gmail-m_-2673680542489267441gmail_signature"><div><font face="arial, helvetica, sans-serif">Regards,<br>   Rashad</font></div></div>
</font></span></div>
</blockquote></div><br></div></div>