<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 18, 2016 at 2:53 PM, Moritz Lennert <span dir="ltr"><<a href="mailto:mlennert@club.worldonline.be" target="_blank">mlennert@club.worldonline.be</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 18/12/16 14:01, Rashad Kanavath wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
Hello,<br>
<br>
As promised, I had pushed code to G7 addons repo<br>
<br>
<a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/imagery/i.superpixels.slic" rel="noreferrer" target="_blank">https://trac.osgeo.org/grass/b<wbr>rowser/grass-addons/grass7/ima<wbr>gery/i.superpixels.slic</a><br></span>
<<a href="https://trac.osgeo.org/grass/browser/grass-addons/grass7/imagery/i.superpixels.slic" rel="noreferrer" target="_blank">https://trac.osgeo.org/grass/<wbr>browser/grass-addons/grass7/im<wbr>agery/i.superpixels.slic</a>><br>
</blockquote>
<br>
Great job, thanks a lot !<span><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I had tested with some small datasets and is working.<br>
<br>
More testing welcome :)<br>
</blockquote>
<br></span>
I get the following warning during installation:<br>
<br>
main.c: In function ‘main’:<br>
main.c:514:13: warning: implicit declaration of function ‘min’ [-Wimplicit-function-declarati<wbr>on]<br>
     seedx = min(g_width-1,seedx);<br>
             ^~~<br>
<br>
I think you have to explicitely define min() through a macro, or AFAIK you can use fmin() as elsewhere in the code.<br></blockquote><div><br></div><div>I will push a fix for that. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also:<br>
<br>
- The output as it is now is not very useful. What we would need is a map with each pixel containing the label of the superpixel it belongs to. In the original code, there are both outputs: i) an image of the superpixel limits overlayed over the original image, ii) the labeled pixels<br></blockquote><div><br></div><div>you need contour segments as seperate output and also the current one. right?<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- Linked to the above: currently the superpixel boundaries are two pixels wide. A one-pixel boundary would be enough.<br></blockquote><div><br></div><div>you can adjust no of superpixels with k. default now is 200 <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- I don't know the details of the algorithm well enough, but would it be possible to extend it beyond the use of r,g,b maps as input. Ideally, the use should be able to provide a group as input and all maps in the group are used in the definition of the superpixels. This should also include the case when you only have one band (e.g. black and white orthophoto).<br></blockquote><div><br></div><div>I need help in understanding this part. because in code, it operates on lab color space. first it takes RBG and convert to LAB. This is reason, I had to put red,green,blue parameters.<br><br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Again, many thanks for taking this first step, allowing us to test !<span class="m_6554721377225649978HOEnZb"><font color="#888888"><br>
<br>
Moritz<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="m_6554721377225649978gmail_signature" data-smartmail="gmail_signature"><div><font face="arial, helvetica, sans-serif">Regards,<br>   Rashad</font></div></div>
</div></div>