[mkgmap-dev] highway count not fixed yet... - merge-roads-branch
From Felix Hartmann extremecarver at gmail.com on Thu Oct 3 14:23:08 BST 2013
okay, it seems that my problems are fixed now. I don't have good enough internet to actually check the created maps (waiting for comments if broken within the next 2-3 days) - but the filesize vs the working older version of the merge-roads branch has increased ever so slightly (instead of breaking down by 20-50%) and no more errors are displayed on running mkgmap (at least for the first 10 countries compiled so far..).... best include the patch to the branch - without it I would say the branch is not safe/usable... On 03.10.2013 13:37, GerdP wrote: > Hi Felix, > > is fixed with r2726. > > Gerd > > > Felix Hartmann-2 wrote >> I'm having problems compiling mkgmap with this patch... (trunk1 is my >> folder for the merge-roads branch). >> dunno really what's going on here... >> I only have jdk 7 installed (update 40) and uninstalled java jdk7 and >> jre7... >> >> >> C:\garmin\mkgmap_trunk1>ant dist >> Buildfile: C:\garmin\mkgmap_trunk1\build.xml >> >> prepare: >> >> ivy-availability: >> >> download-ivy: >> >> init-ivy: >> [ivy:configure] :: Ivy 2.2.0 - 20100923230623 :: >> http://ant.apache.org/ivy/ :: >> [ivy:configure] :: loading settings :: file = >> C:\garmin\mkgmap_trunk1\ivysetting >> s.xml >> >> resolve-compile: >> >> compile: >> [javac] Compiling 474 source files to >> C:\garmin\mkgmap_trunk1\build\classes >> [javac] warning: [options] bootstrap class path not set in >> conjunction with >> -source 1.6 >> [javac] >> C:\garmin\mkgmap_trunk1\src\uk\me\parabola\mkgmap\osmstyle\StyledCon >> verter.java:1751: error: diamond operator is not supported in -source 1.6 >> [javac] List >> <Way> >> dupIdHighways = new ArrayList<>(); >> [javac] ^ >> [javac] (use -source 7 or higher to enable diamond operator) >> [javac] 1 error >> [javac] 1 warning >> >> BUILD FAILED >> C:\garmin\mkgmap_trunk1\build.xml:237: Compile failed; see the compiler >> error ou >> tput for details. >> >> On 28.09.2013 14:35, WanMil wrote: >>> Hi Gerd, >>> >>> in my opinion recalculating the highway counter after removing the >>> short arcs should fix all problems, shouldn't it? >>> >>> 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? >>> >>> 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... >>> >>> WanMil >>> >>>> Hi Felix, WanMil, >>>> >>>> attached is a patch that might solve the problem. I was still not able >>>> to reproduce it, so >>>> it's just a guess: In a special case, we create a new Coord instance to >>>> replace a CoordPOI >>>> instance. This new instance has highway count = 0. A very special case >>>> might be that this point is later used to split the way, in that case it >>>> would >>>> have highwaycount=1 for a first or last point of a road. >>>> The patch increments the count when the coord is created. >>>> >>>> Gerd >>>> >>>> >>>> ------------------------------------------------------------------------ >>>> Date: Fri, 27 Sep 2013 02:51:58 +0200 >>>> From: >> extremecarver@ >>>> To: >> mkgmap-dev at .org >>>> Subject: Re: [mkgmap-dev] highway count not fixed yet... - >>>> merge-roads-branch >>>> >>>> no, I don't use any roundabout command like options, but adjust turn >>>> headings?? -- see below for all commandline options. From style I don't >>>> call much except loads of continues and continue with action, as well as >>>> some link to pois stuff like reduce road_class/road_speed. >>>> >>>> On the old version the only very occasional problem note I get is the >>>> following - in this case for Bayern (Germany Bundesland) Geofabrik >>>> extract: >>>> start compilation 21:45:44 Velomap bayern this is run58 >>>> SEVERE (MapBuilder): c:\openmtbmap\maps\65260023.osm.pbf: possible >>>> routing problem: road end-points not both coordNodes: >>>> (http://www.openstreetmap.org/browse/way/156936823) >>>> SEVERE (MapBuilder): c:\openmtbmap\maps\65260023.osm.pbf: possible >>>> routing problem: road end-points not both coordNodes: >>>> (http://www.openstreetmap.org/browse/way/156936823) >>>> >>>> >>>> I'm using theese commandline options: >>>> start /low /b /wait java -jar -Xms6000M -Xmx10300M >>>> c:\openmtbmap\mkgmap.jar %max-jobs% %generate-sea% %precomp-seaxx% >>>> %style-file% --nsis %indx% %levels% --adjust-turn-headings >>>> --add-pois-to-areas --reduce-point-density=3.4 >>>> --reduce-point-density-polygon=6 --housenumbers --remove-short-arcs >>>> --link-pois-to-ways --ignore-turn-restrictions >>>> --polygon-size-limits="24:16, 23:14, 22:12, 21:11, 20:10, 19:9, 18:8, >>>> 17:7, 16:6, 15:5, 14:4, 13:3, 12:2, 11:0, 10:0" >>>> --description=openmtbmap_%abr% --show-profiles=1 %locationxx% --route >>>> --country-abbr=%abr% --country-name=%country% --mapname=%FID%0000 >>>> --family-id=%FID% --product-id=1 >>>> --series-name=openmtbmap_%country%_%date% >>>> --family-name=mtbmap_%abr%_%date% --tdbfile --overview-mapname=mapsetc >>>> --keep-going --area-name="%country%_%date%_openmtbmap.org" -c >>>> c:\openmtbmap\maps\template.%countryx% 7*.img >NUL >>>> >>>> with these variables in general: >>>> set generate-sea=--generate-sea --latin1 >>>> set precomp-seaxx=--precomp-sea=c:\openmtbmap\maps\sea.zip >>>> set levels=--levels="0:24, 1:23, 2:22, 3:21, 4:20, 5:19, 6:18" >>>> --overview-levels="7:17, 8:16, 9:15, 10:14, 11:13, 12:12 >>>> >>>> and for most countries: >>>> set indx=--index (not using index for Asia continent as asia continent >>>> with index was crashing in Basecamp/Mapsource very often, only few >>>> compiles actually worked) >>>> set max-jobs=--max-jobs=8 (for some countries 7 as I ran out of memory >>>> on them and server started to swap=slower) >>>> On 26.09.2013 21:57, Gerd Petermann wrote: >>>> >>>> Hi WanMil, >>>> >>>> at least we should know if options like frig-roundabout are used. >>>> Afaik the default style will never touch these >>>> routines. I guess Felix uses almost all. >>>> >>>> Gerd >>>> >>>> > Date: Thu, 26 Sep 2013 21:53:20 +0200 >>>> > From: >> wmgcnfg@ >> <mailto: >> wmgcnfg@ >> > >>>> > To: >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > Subject: Re: [mkgmap-dev] highway count not fixed yet... - >>>> merge-roads-branch >>>> > >>>> > Yeah, I guess it should be possible to simplify them be >>>> > reimplementation. But that's only a rough guess.... >>>> > >>>> > A test case would be great to find the missing incHighwayCount()! >>>> > >>>> > > Hi WanMil, >>>> > > >>>> > > yes, first and last node should be coordNode, so the assert >>>> is ok. >>>> > > Unfortunately, the data flow in StyledConverter is >>>> > > so complex that it is difficult to say why the assertion is >>>> triggered. I >>>> > > guess one of the split routines is still >>>> > > missing a call of incHighwayCount(). >>>> > > >>>> > > Gerd >>>> > > >>>> > > >>>> > > > Date: Thu, 26 Sep 2013 21:42:28 +0200 >>>> > > > From: >> wmgcnfg@ >> <mailto: >> wmgcnfg@ >> > >>>> > > > To: >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > > > Subject: Re: [mkgmap-dev] highway count not fixed yet... - >>>> > > merge-roads-branch >>>> > > > >>>> > > > Yes, it is meant to reduce the number of CoordNodes because >>>> that should >>>> > > > reduce the size of the routing network and might have a >>>> positive impact. >>>> > > > >>>> > > > The assertion reported by Felix seems to be a problem of the >>>> highway >>>> > > > count. The assertion checks if the first node of a MapRoad >>>> is a >>>> > > > CoordNode. I think this is required, isn't is? >>>> > > > While writing I am thinking of no exit roads. What about >>>> these roads? I >>>> > > > think the first and the last point should also be a >>>> CoordNode?!? >>>> > > > >>>> > > > WanMil >>>> > > > >>>> > > > > Hi WanMil, >>>> > > > > >>>> > > > > yes, it will not cause problems. On the other hand, if you >>>> do it to >>>> > > > > reduce the number of CoordNodes, we should try to have a >>>> correct >>>> > > > > counter. I think the short-arc-removal is not always >>>> correctly >>>> > > > > maintaining it. I'll have a look at it tomorrow. >>>> > > > > >>>> > > > > Gerd >>>> > > > > >>>> > > > > >>>> > > > > >>>> > > > > >>>> > > > > WanMil wrote >>>> > > > >> Hi Gerd, >>>> > > > >> >>>> > > > >> decHighwayCount() is called only on the node where two >>>> roads are >>>> > > merged. >>>> > > > >> So assuming that the highway count gives the number of >>>> connected roads >>>> > > > >> calling this method in such a case should be ok. >>>> > > > >> >>>> > > > >> WanMil >>>> > > > >> >>>> > > > >>> Hi WanMil, >>>> > > > >>> >>>> > > > >>> reg. the highway count: >>>> > > > >>> I guess you already noticed, but just to make sure: >>>> > > > >>> In trunk the absolute value of the counter does not >>>> really matter >>>> > > > >>> as long as it is > 1 for each point that should be >>>> converted to a >>>> > > > >>> node. I think a lot of routines are calling >>>> > > > >>> incHighwayCount() "just to make sure", so a node where >>>> two >>>> > > > >>> arcs meet might have a counter > 2. >>>> > > > >>> You have introduced decHighwayCount(), so now >>>> > > > >>> each place where this counter is incremented has >>>> > > > >>> to be double checked. >>>> > > > >>> >>>> > > > >>> Gerd >>>> > > > >>> >>>> > > > >>> >>>> > > > >>> WanMil wrote >>>> > > > >>>> Ok, but I need some food (style, data etc.) to reproduce >>>> it... >>>> > > > >>>> >>>> > > > >>>>> Just cannot find the topic on the merge-roads-branch. >>>> > > > >>>>> >>>> > > > >>>>> Is it known that the highway count error is not fully >>>> fixed yet? I >>>> > > > >>>>> still >>>> > > > >>>>> get loads of them. >>>> > > > >>>>> _______________________________________________ >>>> > > > >>>>> mkgmap-dev mailing list >>>> > > > >>>>> >>>> > > > >>> >>>> > > > >>>> mkgmap-dev at .org <mailto:mkgmap-dev at .org> >>>> > > > >>> >>>> > > > >>>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > > >>>>> >>>> > > > >>>> >>>> > > > >>>> _______________________________________________ >>>> > > > >>>> mkgmap-dev mailing list >>>> > > > >>> >>>> > > > >>>> mkgmap-dev at .org <mailto:mkgmap-dev at .org> >>>> > > > >>> >>>> > > > >>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > > >>> >>>> > > > >>> >>>> > > > >>> >>>> > > > >>> >>>> > > > >>> >>>> > > > >>> -- >>>> > > > >>> View this message in context: >>>> > > > >>> >>>> > > >>>> http://gis.19327.n5.nabble.com/highway-count-not-fixed-yet-merge-roads-branch-tp5778802p5779025.html >>>> > > > >>> Sent from the Mkgmap Development mailing list archive at >>>> Nabble.com. >>>> > > > >>> _______________________________________________ >>>> > > > >>> mkgmap-dev mailing list >>>> > > > >>> >>>> > > > > >>>> > > > >> mkgmap-dev at .org <mailto:mkgmap-dev at .org> >>>> > > > > >>>> > > > >>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > > >>> >>>> > > > >> >>>> > > > >> _______________________________________________ >>>> > > > >> mkgmap-dev mailing list >>>> > > > > >>>> > > > >> mkgmap-dev at .org <mailto:mkgmap-dev at .org> >>>> > > > > >>>> > > > >> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > > > >>>> > > > > >>>> > > > > >>>> > > > > >>>> > > > > >>>> > > > > -- >>>> > > > > View this message in context: >>>> > > >>>> http://gis.19327.n5.nabble.com/highway-count-not-fixed-yet-merge-roads-branch-tp5778802p5779102.html >>>> > > > > Sent from the Mkgmap Development mailing list archive at >>>> Nabble.com. >>>> > > > > _______________________________________________ >>>> > > > > mkgmap-dev mailing list >>>> > > > > >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > > > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > > > >>>> > > > >>>> > > > _______________________________________________ >>>> > > > mkgmap-dev mailing list >>>> > > > >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > >>>> > > >>>> > > _______________________________________________ >>>> > > mkgmap-dev mailing list >>>> > > >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> > > >>>> > >>>> > _______________________________________________ >>>> > mkgmap-dev mailing list >>>> > >> mkgmap-dev at .org >>>> <mailto: >> mkgmap-dev at .org >> > >>>> > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> >>>> >>>> _______________________________________________ >>>> mkgmap-dev mailing list >>>> >> mkgmap-dev at .org >> >>>> <mailto: >> mkgmap-dev at .org >> > >>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> >>>> >>>> >>>> _______________________________________________ mkgmap-dev mailing list >>>> >> mkgmap-dev at .org >>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> >>>> >>>> _______________________________________________ >>>> mkgmap-dev mailing list >>>> >> mkgmap-dev at .org >>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >>>> >>> >>> >>> _______________________________________________ >>> mkgmap-dev mailing list >>> >> mkgmap-dev at .org >>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev >> >> _______________________________________________ >> mkgmap-dev mailing list >> mkgmap-dev at .org >> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > > -- > View this message in context: http://gis.19327.n5.nabble.com/highway-count-not-fixed-yet-merge-roads-branch-tp5778802p5779914.html > Sent from the Mkgmap Development mailing list archive at Nabble.com. > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev at lists.mkgmap.org.uk > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
- 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