Hi Gerd orderByDecreasingArea only has a param declaration and single use in MapSplitter and its value transforms to MapArea.splitPolygonsIntoArea so I think it is better as it is. I had some thoughts on a simple/efficient algo for MultiPolygonRelation hole cutting - It won't produce the optimized cuts that yours is aiming for. If you are interested, I'll formulate some pseudo-code. Merge the branch whenever is best for you. Ticker On Tue, 2017-02-07 at 13:12 +0000, Gerd Petermann wrote:
Hi Ticker,
thanks, I am still working a new algos to replace parts of the existing code in MultipolygonRelation, esp. that for createContainsMatrix(). I hope to finish the first patch tomorrow. If that takes much longer I plan to merge the branch into trunk first. Reg. refactoring: What do you think about adding orderByDecreasingArea as a (final) field in MapSplitter?
Gerd
________________________________________ Von: mkgmap-dev <mkgmap-dev-bounces@lists.mkgmap.org.uk> im Auftrag von Ticker Berkin <rwb-mkgmap@jagit.co.uk> Gesendet: Dienstag, 7. Februar 2017 13:45:25 An: mkgmap-dev@lists.mkgmap.org.uk Betreff: Re: [mkgmap-dev] r3784 produces large img files than r3773
Done
On Tue, 2017-02-07 at 12:17 +0000, Gerd Petermann wrote:
Hi Ticker,
that's what I expected. Most shapes do not have too many points after line simplification. If I got that right most of the code in PolygonSplitterFilter is now obsolete. I think it would be better to move the functionality of PolygonSplitIfNeededFilter into it so that the history is kept.
Gerd
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev