Hi Gerd My rationale was to see what highway= tags from various european countries the default style didn't handle and hence generated: [0x07 road_class=0 road_speed=0 resolution 23] and then decide to: a) explicitly ignore. b) handle as if they were a well defined highway type, with appropriate access control. c) generate a new line type, with routing as appropriate. d) allow mop-up as above. I removed escape/emergency_bay because I didn't think they added anything useful to routing or the resultant map. footpath/foot looked like they should be footway or path, but if you think they should be ignored, I have no problem with that. What remained after this exercise was a few rubbish tags and lots of highway={real name of street} and I'd rather have these on my map The {add horse=xxx} I just changed a bit to be in keeping with what was there already but happy to delete it. Ticker On Thu, 2018-12-13 at 09:18 +0000, Gerd Petermann wrote:
Hi Ticker,
I think it is not a good idea to remove highway=escape or highway=emergency_bay. The last time I looked at them most where correctly mapped. I'd also remove horse=yes or horse=no unless you want evaluate that somewhere in the style. Don't know why it is in the current default style. There is no code in mkgmap to evaluate it.
Reg. rules like highway=footpath | highway=foot {set highway=footway} # fix common bad tagging I think we don't need them. Most of those ways are mapped by HOT projects, it is very likely that they are not connected or self intersecting etc. I'd rather remove the mop up rule instead of adding a lot of "try to guess better" rules.
Gerd
________________________________________ Von: mkgmap-dev <mkgmap-dev-bounces@lists.mkgmap.org.uk> im Auftrag von Ticker Berkin <rwb-mkgmap@jagit.co.uk> Gesendet: Samstag, 8. Dezember 2018 18:19 An: Development list for mkgmap Betreff: Re: [mkgmap-dev] default style improvements
Hi Gerd
Here is revision to defaultStyleTidy3.patch. The changes are:
1/ change highway=trail to highway=path; add bicycle=no instead of track
2/ don't generate routable line for highway=rest_area
Regards Ticker