<div dir="ltr">



<div>
<div style="color:black">
<p style="margin:0px 0px 1em;color:black"><br>
</p>
<p style="margin:0px 0px 1em;color:black;font-family:sans-serif"></p>
<p style="margin:0px 0px 1em;color:black;font-family:sans-serif">On 26
January 2017 11:53:30 pm Markus Neteler <<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>> wrote:</p>
<p style="margin:0px 0px 1em;color:black;font-family:sans-serif">>
On Thu, Jan 26, 2017 at 2:38 PM, Markus Metz<br>
> <<a href="mailto:markus.metz.giswork@gmail.com" target="_blank">markus.metz.giswork@gmail.com</a><wbr>> wrote:<br>
> ...<br>
>> It's a bug in the code [0], the algorithm is not working properly.
There is<br>
>> v.generalize, therefore I recommend to remove the prune tool from
v.clean.<br>
><br>
> Maybe keep (for consistency) and replace the functionality with an<br>
> G_fatal_error() pointing the user to v.generalize?<br>
></p>
<p style="margin:0px 0px 1em;color:black"></p><p style="margin:0px 0px 1em;color:black">For the GUI, and the 
manual page, I think it would be better if the functionality is removed 
completely. Presenting an option that is really not available would, 
i.m.h.o. be confusing. <br></p><p style="margin:0px 0px 1em;color:black"></p><p style="margin:0px 0px 1em;color:black">For command line use, it is good to get a error with pointer to v.generalize if users attempt to use 'prune' in v.clean. Especially important to warn people properly if using this in a script.<br></p>For legacy reasons, it is furthermore probably good to mention in the Note section of the help file that this function has been removed and that the user can use v.generalize for this.<br><br></div><div style="color:black">Just my 2ct.<br></div><div style="color:black"><p style="margin:0px 0px 1em;color:black"><br>
</p>
<p style="margin:0px 0px 1em;color:black;font-family:sans-serif"><br>
> markusN<br>
><br>
>> Markus M<br>
>><br>
>> [0]<br>
>> <a href="https://trac.osgeo.org/grass/browser/grass/trunk/lib/vector/diglib/prune.c" target="_blank">https://trac.osgeo.org/grass/<wbr>browser/grass/trunk/lib/<wbr>vector/diglib/prune.c</a><br>
</p>
<p style="margin:0px 0px 1em;color:black">______________________<br>
Send from my mobile<br>
Excuse my brevity<br>
</p>
</div>
</div>

</div>