logo separator

[mkgmap-dev] highway count not fixed yet... - merge-roads-branch

From GerdP gpetermann_muenchen at hotmail.com on Thu Oct 3 14:58:40 BST 2013

Hi Felix,

a corrected version of the patch is already in the branch, so
you should test with an unpatched r2726.

Not sure what you mean with the filesize, why should it change that much?
Gerd


Felix Hartmann-2 wrote
> 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@
>>>   &lt;mailto:
>>> wmgcnfg@
>>> &gt;
>>>>>       > To:
>>> mkgmap-dev at .org
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > 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@
>>>   &lt;mailto:
>>> wmgcnfg@
>>> &gt;
>>>>>       > > > To:
>>> mkgmap-dev at .org
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > > > 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 &lt;mailto:mkgmap-dev at .org&gt;
>>>>>       > > > >>>
>>>>>       > > > >>>>>
>>>>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>       > > > >>>>>
>>>>>       > > > >>>>
>>>>>       > > > >>>> _______________________________________________
>>>>>       > > > >>>> mkgmap-dev mailing list
>>>>>       > > > >>>
>>>>>       > > > >>>> mkgmap-dev at .org &lt;mailto:mkgmap-dev at .org&gt;
>>>>>       > > > >>>
>>>>>       > > > >>>>
>>>>> 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 &lt;mailto:mkgmap-dev at .org&gt;
>>>>>       > > > >
>>>>>       > > > >>> http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>       > > > >>>
>>>>>       > > > >>
>>>>>       > > > >> _______________________________________________
>>>>>       > > > >> mkgmap-dev mailing list
>>>>>       > > > >
>>>>>       > > > >> mkgmap-dev at .org &lt;mailto:mkgmap-dev at .org&gt;
>>>>>       > > > >
>>>>>       > > > >> 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
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > > > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>       > > > >
>>>>>       > > >
>>>>>       > > > _______________________________________________
>>>>>       > > > mkgmap-dev mailing list
>>>>>       > > >
>>> mkgmap-dev at .org
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>       > >
>>>>>       > >
>>>>>       > > _______________________________________________
>>>>>       > > mkgmap-dev mailing list
>>>>>       > >
>>> mkgmap-dev at .org
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>       > >
>>>>>       >
>>>>>       > _______________________________________________
>>>>>       > mkgmap-dev mailing list
>>>>>       >
>>> mkgmap-dev at .org
>>>>>      &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>       > http://lists.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>>>>>
>>>>>
>>>>>      _______________________________________________
>>>>>      mkgmap-dev mailing list
>>>>>      
>>> mkgmap-dev at .org
>>>   
>>>>> &lt;mailto:
>>> mkgmap-dev at .org
>>> &gt;
>>>>>      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 .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-tp5778802p5779942.html
Sent from the Mkgmap Development mailing list archive at Nabble.com.


More information about the mkgmap-dev mailing list