Yes deprecate next year probably fine. Keeping the option name simple is best I think.
Hi Felix,
I like --simplify-lines / --simplify-polygons.
What to do with the existing --reduce-... options? My plan is to introduce the new options and deprecate the old ones next year.
Gerd
________________________________________
Von: mkgmap-dev <mkgmap-dev-bounces@lists.mkgmap.org.uk> im Auftrag von Felix Hartmann <extremecarver@gmail.com>
Gesendet: Montag, 21. Juni 2021 10:29
An: Development list for mkgmap
Betreff: Re: [mkgmap-dev] low-res-opt branch ready to merge?
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).
I do not know of any errors good if it will be merged.
On Mon, 21 Jun 2021 at 11:20, Gerd Petermann <GPetermann_muenchen@hotmail.com<mailto:GPetermann_muenchen@hotmail.com>> wrote:
Hi all,
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.
So, I think this should no longer block the merge to trunk.
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?
Are there any other known problems with the branch? Any good ideas to improve something?
Gerd
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk<mailto:mkgmap-dev@lists.mkgmap.org.uk>
https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
--
Felix Hartman - Openmtbmap.org & VeloMap.org
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev