Re: [mkgmap-dev] why do we have mkgmap:road-speed-class ?

2013-12-28 Thread WanMil
Hi WanMil, I fear I don't understand the meaning of this new tag. Docu says: A value between 0 and 7. Overrides the road_speed definition in the element type definition if this tag is set. We have mkgmap:road-speed which allows to increment, decrement or override the road speed value in the

Re: [mkgmap-dev] why do we have mkgmap:road-speed-class ?

2013-12-28 Thread Gerd Petermann
Hi WanMil, I still don't understand. Please give an example where mkgmap:road-speed-class has to be used because mkgmap:road-speed will not work. Gerd Date: Sat, 28 Dec 2013 09:20:20 +0100 From: wmgc...@web.de To: mkgmap-dev@lists.mkgmap.org.uk Subject: Re: [mkgmap-dev] why do we have

Re: [mkgmap-dev] why do we have mkgmap:road-speed-class ?

2013-12-28 Thread WanMil
lines: include 'inc/roadspeed'; highway=* lanes 2 { set mkgmap:road-speed=+1 } The roadspeed include sets the road speed. After that you need a 2nd tag to modify the road speed. WanMil Hi WanMil, I still don't understand. Please give an example where mkgmap:road-speed-class has to be

[mkgmap-dev] Commit: r2914: changes in unit tests

2013-12-28 Thread svn commit
Version mkgmap-r2914 was committed by gerd on Sat, 28 Dec 2013 changes in unit tests - modify tests that check file sizes to match a range so that they print the actual size value - change the tested values to match exactly the actual values ___

Re: [mkgmap-dev] why do we have mkgmap:road-speed-class ?

2013-12-28 Thread Gerd Petermann
Hi WanMil, okay, now I got it. Would it be possible to print a warning when a style uses something like highway=secondary {set mkgmap:road-speed=3} highway=* surface=gravel {set mkgmap:road-speed=-1} highway=* tunnel=yes {set mkgmap:road-speed=-1} For a way with

Re: [mkgmap-dev] [patch v1] link-pois-to-ways and restrictions

2013-12-28 Thread Gerd Petermann
Hi WanMil, attached is a patch for r2914. It fixes the problem with --link-poi-to-ways that was introduced with r2790 in the branch. With this patch the option should again work like in trunk r2889. If you think it is okay please commit it and I'll post a 2nd version of the link-pois-to-ways

Re: [mkgmap-dev] why do we have mkgmap:road-speed-class ?

2013-12-28 Thread WanMil
Hi Gerd, it would be possible to print a warning (implemented in the addtagaction) but maybe it is better to add a new action like highway=secondary {set mkgmap:road-speed=3} highway=* surface=gravel { modroadspeed=-1} highway=* tunnel=yes { modroadspeed=-1} The modroadspeed action

[mkgmap-dev] mkgmap error

2013-12-28 Thread Jaromír Mikeš
Hi, I am trying adopt my style files to new release 2906 ... I succeed with points and lines files but having problem with polygons file. Getting error when trying build my map: $ java -ea -Xmx6144m -jar mkgmap-r2914/mkgmap.jar --precomp-sea=/media/mira/MEGA/MAPING/sea_20131101.zip

Re: [mkgmap-dev] mkgmap error

2013-12-28 Thread Gerd Petermann
Hi Mira, looks like a missing braket somewhere in polygons. If you don't have an editor that helps you, try binary search (remove last, if error persists, remove next half, else error is in 2nd half...) Gerd From: mira.mi...@seznam.cz To: mkgmap-dev@lists.mkgmap.org.uk Date: Sat, 28 Dec

Re: [mkgmap-dev] mkgmap error

2013-12-28 Thread Jaromír Mikeš
Od: Gerd Petermann looks like a missing braket somewhere in polygons. If you don't have an editor that helps you, try binary search (remove last, if error persists, remove next half, else error is in 2nd half...) Hi Gerd, good method :) ... missing ' in line 40 btw Thank you mira

Re: [mkgmap-dev] mkgmap error

2013-12-28 Thread Bernd Weigelt
Am Samstag, 28. Dezember 2013, 16:57:02 schrieb Jaromír Mikeš: Any idea what can be wrong? line 42 {add mkgmap:label:1='${name}' | 'Marina} missing ' ___ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk

Re: [mkgmap-dev] mkgmap error

2013-12-28 Thread Bernd Weigelt
Am Samstag, 28. Dezember 2013, 17:30:52 schrieben Sie: Am Samstag, 28. Dezember 2013, 16:57:02 schrieb Jaromír Mikeš: Any idea what can be wrong? line 42 {add mkgmap:label:1='${name}' | 'Marina} missing ' sorry error is in line 40 ernd ___

[mkgmap-dev] Commit: r2915: Fix the link-pois-to-ways handling

2013-12-28 Thread svn commit
Version mkgmap-r2915 was committed by wanmil on Sat, 28 Dec 2013 Fix the link-pois-to-ways handling This includes: * road speed and road class is calculated in a method that considers all modifying tags * road speed and road class is modified in a copy of the GType object. This makes the

[mkgmap-dev] Commit: r2916: Optimize logging statements in StyledConverter

2013-12-28 Thread svn commit
Version mkgmap-r2916 was committed by wanmil on Sat, 28 Dec 2013 Optimize logging statements in StyledConverter ___ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

[mkgmap-dev] routing seems to depend on order of ways

2013-12-28 Thread Gerd Petermann
Hi Steve, I think I found a very old bug in mkgmap, it seems to exist since at least r2210. I found it while testing a patch for r2914 and found it in all older versions that I tried. Attached are two test files and a gdb for MapSource. The command that I use: java -jar

[mkgmap-dev] Commit: r2917: Add undocumented option --no-mergeroads to disable merging of roads

2013-12-28 Thread svn commit
Version mkgmap-r2917 was committed by wanmil on Sat, 28 Dec 2013 Add undocumented option --no-mergeroads to disable merging of roads Usually this is used for debugging purposes only. ___ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk

Re: [mkgmap-dev] routing seems to depend on order of ways

2013-12-28 Thread WanMil
Hi, I have commited the new (undocumented) option --x-no-mergeroads to disable merging of roads. I think this is useful for chasing problems. WanMil Hi Steve, I think I found a very old bug in mkgmap, it seems to exist since at least r2210. I found it while testing a patch for r2914 and