Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch

2017-02-15 Thread Mike Baggaley
HI Ticker, I've tried the patch without the --order-by-decreasing-area option and it does now run without crashing. However, it suffers from the same problem as I mentioned earlier when trying --order-by-decreasing-area without the patch, in that the coastline is corrupted. I haven't yet turned

Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch

2017-02-15 Thread Ticker Berkin
Hi Mike I think this is just a problem with the rules the tool you are using to show the image conflicting with what --order-by-decreasing-area aims to achieve on a Garmin device. For instance, observations of GPSMapEdit shows it renders smaller polygons on top of larger ones as the way of

Re: [mkgmap-dev] Performace mkgmap-3803

2017-02-15 Thread Gerd Petermann
OK, r3808 solves the performance problem in ShapeMergeFilter. Von: mkgmap-dev im Auftrag von Gerd Petermann Gesendet: Mittwoch, 15. Februar 2017 09:47:59 An: Development list for

[mkgmap-dev] Commit r3808: improve performance of ShapeMergeFilter

2017-02-15 Thread svn commit
Version mkgmap-r3808 was committed by gerd on Wed, 15 Feb 2017 improve performance of ShapeMergeFilter The old code didn't work well when many (e.g. 1000+) distinct similar shapes were processed (e.g. buildings). This typically happens only when style adds shapes at low resolutions.

Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch

2017-02-15 Thread Ticker Berkin
Hi Gerd / Mike I've just committed a fix for this to the split-shape branch Ticker On Wed, 2017-02-15 at 08:53 +, Ticker Berkin wrote: > Hi Mike > > OK - I think I understand the problem. > > Can you try with --order-by-decreasing-area and let me know if that > stops the unbound

[mkgmap-dev] Commit r3806: small performance improvement: Don't call Coord.toDegreeString() when result is not needed

2017-02-15 Thread svn commit
Version mkgmap-r3806 was committed by gerd on Wed, 15 Feb 2017 small performance improvement: Don't call Coord.toDegreeString() when result is not needed http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap=3806 ___ mkgmap-dev mailing list

Re: [mkgmap-dev] RE Commit r3801: merge split-shape branch

2017-02-15 Thread Ticker Berkin
Hi Mike OK - I think I understand the problem. Can you try with --order-by-decreasing-area and let me know if that stops the unbound recursion, and, in the meantime, I'll work on it. Thanks Ticker On Wed, 2017-02-15 at 00:48 +, Mike Baggaley wrote: > HI Ticker, having run this under the

Re: [mkgmap-dev] Performace mkgmap-3803

2017-02-15 Thread Gerd Petermann
Hi Ticker, the performance problem with that special style is in ShapeMergeFilter. I think the reason is that the new code in MapSplitter etc. puts a lot more shapes into one sub div, esp. at low resolutions, and ShapeMergeFilter doesn't work well in that case as it starts to do lots of seq.

Re: [mkgmap-dev] Commit r3804: remove echotags from defautl style (was for debugging only)

2017-02-15 Thread Gerd Petermann
Hi Steve, I tried the link but it doesn't work. I see the normal mkgmap layout with a windows titled "File does not exist" followed by "Sorry, the page or file could not be found. " Gerd Von: mkgmap-dev im Auftrag

Re: [mkgmap-dev] To do: If-Then-Else

2017-02-15 Thread Gerd Petermann
Hi Andrzej, I've fixed the initial problem with r3805, attached is a v3 of the patch which is simpler and handles null values. Allthough this is not related to if-then I'd prefer to add this to the branch as it is a step which adds new functionality. OK? Gerd

[mkgmap-dev] Commit r3805: correction: don't add null to usedTags collection for style functions like is_closed() or area_size()

2017-02-15 Thread svn commit
Version mkgmap-r3805 was committed by gerd on Wed, 15 Feb 2017 correction: don't add null to usedTags collection for style functions like is_closed() or area_size() http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap=3805 ___ mkgmap-dev