[mkgmap-dev] highway count not fixed yet... - merge-roads-branchFrom GerdP gpetermann_muenchen at hotmail.com on Sat Sep 28 14:18:08 BST 2013
WanMil wrote > Hi Gerd, > > in my opinion recalculating the highway counter after removing the short > arcs should fix all problems, shouldn't it? Don't think so. All routines that replace or add points after that must make sure that they maintain the correct highway count. WanMil wrote > I've added three changes to the patch: > 1. When calculating the highway count ways with duplicate id are not > considered. This avoid that all points of a duplicated way are preserved > by all filters. > I think this should be modified a bit. For the first and last point of > those ways the highway count should be increased and also all points > where another way is connected. I have no use case where this matters > but I think it is the "correct" counting? No, it just increases the possibility that the counter overflows. Maybe we should not use this counter at all. The information that it "hides" is that a point connects two or more roads. I think a better solution would be to have a method markRoutingNodes() which uses the counter and sets a bit flag in the coord instance. This routine would replace setHighwayCounts() and resetHighwayCounts(). WanMil wrote > 2. I have added the problematic point in the error message of the > MapBuilder in case a node is not a CoordNode. Just having the way id > might not be enough information and the way also might have been merged. > > 3. I have moved the recalculation of the highway counters after the > merge procedure. This should not change anything but it avoids a problem > with merging... This is a good idea. Maybe do it before and after? That also means that decHighwayCount() is obsolete, doesn't it? Gerd -- View this message in context: http://gis.19327.n5.nabble.com/highway-count-not-fixed-yet-merge-roads-branch-tp5778802p5779279.html Sent from the Mkgmap Development mailing list archive at Nabble.com.
- Previous message: [mkgmap-dev] highway count not fixed yet... - merge-roads-branch
- Next message: [mkgmap-dev] highway count not fixed yet... - merge-roads-branch
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
More information about the mkgmap-dev mailing list