<div dir="auto">Yes deprecate next year probably fine. Keeping the option name simple is best I think. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 21 Jun 2021, 15:56 Gerd Petermann <<a href="mailto:gpetermann_muenchen@hotmail.com">gpetermann_muenchen@hotmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Felix,<br>
<br>
I like --simplify-lines / --simplify-polygons.<br>
What to do with the existing --reduce-... options? My plan is to introduce the new options and deprecate the old ones next year.<br>
<br>
Gerd<br>
<br>
<br>
________________________________________<br>
Von: mkgmap-dev <<a href="mailto:mkgmap-dev-bounces@lists.mkgmap.org.uk" target="_blank" rel="noreferrer">mkgmap-dev-bounces@lists.mkgmap.org.uk</a>> im Auftrag von Felix Hartmann <<a href="mailto:extremecarver@gmail.com" target="_blank" rel="noreferrer">extremecarver@gmail.com</a>><br>
Gesendet: Montag, 21. Juni 2021 10:29<br>
An: Development list for mkgmap<br>
Betreff: Re: [mkgmap-dev] low-res-opt branch ready to merge?<br>
<br>
if using reduce - then better call it --reduce-point-density-line and --reduce-point-density-polygon. Otherwise its a bit confusing. Or easiest - just call it --simplify-lines and --simplify-polygons (its not correcting an error, but just optimizing OSM data in my eyes. and how its achieved, dp-filter, whatever is not so relevant in the option name).<br>
I do not know of any errors  good if it will be merged.<br>
<br>
<br>
On Mon, 21 Jun 2021 at 11:20, Gerd Petermann <<a href="mailto:GPetermann_muenchen@hotmail.com" target="_blank" rel="noreferrer">GPetermann_muenchen@hotmail.com</a><mailto:<a href="mailto:GPetermann_muenchen@hotmail.com" target="_blank" rel="noreferrer">GPetermann_muenchen@hotmail.com</a>>> wrote:<br>
Hi all,<br>
<br>
I made no more progress to find a solution to detect parallel lines in the overview map. Too many special cases and almost no improvement with the simple cases.<br>
So, I think this should no longer block the merge to trunk.<br>
<br>
Mike suggested not to add new options --simplify-filter-line-errors / --simplify-filter-polygon-errors but instead enhance the syntax for the existing options --reduce-point-density and --reduce-point-density-polygon. I am open to that. What's better?<br>
<br>
Are there any other known problems with the branch? Any good ideas to improve something?<br>
<br>
Gerd<br>
_______________________________________________<br>
mkgmap-dev mailing list<br>
<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk" target="_blank" rel="noreferrer">mkgmap-dev@lists.mkgmap.org.uk</a><mailto:<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk" target="_blank" rel="noreferrer">mkgmap-dev@lists.mkgmap.org.uk</a>><br>
<a href="https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer noreferrer" target="_blank">https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</a><br>
<br>
<br>
--<br>
Felix Hartman - Openmtbmap.org & VeloMap.org<br>
<br>
_______________________________________________<br>
mkgmap-dev mailing list<br>
<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk" target="_blank" rel="noreferrer">mkgmap-dev@lists.mkgmap.org.uk</a><br>
<a href="https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer noreferrer" target="_blank">https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</a><br>
</blockquote></div>