Hi Gerd and followers of the default style

Here are some enhancements to default style "points" - patch attached.

I've been building up a list of changes to the default style for a
while, and, with the current discussion about multiple POI for a single
OSM object and the benefit of benches (I've just added this), it seemed
like a good time to put forward these changes. 

Some of these derive from:
http://www.mkgmap.org.uk/pipermail/mkgmap-dev/2019q4/030097.html

Some from other comments in this forum and the rest are from when I've
noticed something wrong or that could be improved.

Summary of changes:
- for amenity={atm/bar/cafe} allow additional POI from other tags,
  eg shop=bakery or any others: leisure/tourism/sport/...=...
- create POI 0x6605 for bench (Geographic>Land>Bench)
- don't create POI for private parking
- create parking POI for parking_entrance
- if parking unnamed, use access tag if exists, giving names like
  "Customer parking", "Permissive parking"...
- explicit shelter_type=basic_hut rather that trying to exclude
  bus shelters etc for Find>Lodgings
- marina code from 0x4300 to 0x2f09; searchable as "Others"
- POI for nature_reserve, even if not named
- don't create POI for pitch if no name or sport
- don't create POI for private swimming pools
- don't create POI for unnamed cliff
- add POI 0x6503 for named bay (Geographic>Water>Bay)
- for platforms, just 1 POI when --add-pois-to-lines
- name unnamed shops as category, regardless of category being matched
- add, commented out, extra shop categories supported by some devices
- add POI for shop=outdoor
- for {camp/caravan}_site have BOTH 0x2b03 AND 0x2b05;
  03 not shown on modern devices, 05 not shown on old devices
- have both Food & Lodging POI for same point if appropriate
- landuse=basin code from 0x650f (Geographic>Water>Reservoir)
                       to 0x6603 (Geographic>Land>Basin)
- water=reservoir as well as landuse=reservoir
- add POI 0x650d for named water=lake/pond (Geograhic>Water>Lake)

I'm happy to explain any of these change in more detail and, if anyone
considers a particular change incorrect, provide a revised patch.

Ticker
Index: resources/styles/default/points
===================================================================
--- resources/styles/default/points	(revision 4483)
+++ resources/styles/default/points	(working copy)
@@ -119,14 +119,15 @@
 aeroway=helipad [0x5904 resolution 23]
 aeroway=terminal [0x2f04 resolution 24]
 
-amenity=atm [0x2f06 resolution 24]
+amenity=atm [0x2f06 resolution 24 continue]
 amenity=arts_centre [0x2c04 resolution 24]
 amenity=bank [0x2f06 resolution 24]
-amenity=bar [0x2d02 resolution 24]
+amenity=bar [0x2d02 resolution 24 continue]
+amenity=bench [0x6605 resolution 24 continue]
 amenity=biergarten [0x2d02 resolution 24]
 amenity=border_control | barrier=border_control [0x3006 resolution 20]
 amenity=bus_station [0x2f08 resolution 23]
-amenity=cafe [0x2a0e resolution 24]
+amenity=cafe {delete cuisine} [0x2a0e resolution 24 continue with_actions]
 amenity=car_club [0x2f0d resolution 24]
 amenity=car_rental [0x2f02 resolution 24]
 amenity=car_sharing [0x2f02 resolution 24]
@@ -157,7 +158,7 @@
 amenity=library [0x2c03 resolution 24]
 amenity=nightclub [0x2d02 resolution 24]
 amenity=nursing_home [0x2f14 resolution 24]
-amenity=parking [0x2f0b resolution 24 default_name 'Parking']
+(amenity=parking | amenity=parking_entrance) & access!=private & access!=no {add name='${access} parking'| 'Parking'} [0x2f0b resolution 24]
 amenity=pharmacy [0x2e05 resolution 24]
 amenity=place_of_worship [0x2c0b resolution 24]
 amenity=police [0x3001 resolution 24]
@@ -172,7 +173,7 @@
 # amenity=shelter is ambiguous; when possible, consider using other tags:
 #  tourism=lean_to or tourism=picnic_site
 #  shelter=yes on highway=bus_stop or highway=tram_stop or railway=halt
-amenity=shelter & shelter_type!=public_transport [0x2b06 resolution 24 default_name 'Shelter']
+amenity=shelter & shelter_type=basic_hut [0x2b06 resolution 24 default_name 'Shelter']
 # amenity=supermarket is superceded by shop=supermarket
 amenity=supermarket [0x2e02 resolution 24]
 amenity=taxi [0x2f17 resolution 24]
@@ -202,15 +203,15 @@
 leisure=garden & name=* [0x2c06 resolution 24]
 leisure=golf_course [0x2d05 resolution 24]
 leisure=ice_rink [0x2d08 resolution 24]
-leisure=marina [0x4300 resolution 24]
-leisure=nature_reserve & name=* [0x6612 resolution 24]
+leisure=marina [0x2f09 resolution 24]
+leisure=nature_reserve [0x6612 resolution 24]
 leisure=park [0x2c06 resolution 24]
-leisure=pitch {name '${name} (${sport})' | '${sport}'} [0x2c08 resolution 24]
+leisure=pitch & (name=* | sport=*) {name '${name} (${sport})' | '${sport}'} [0x2c08 resolution 24]
 leisure=playground [0x2c06 resolution 24 default_name 'Playground']
 leisure=recreation_ground [0x2c08 resolution 24 default_name 'Rec.']
 leisure=sports_center | leisure=sports_centre {name '${name} (${sport})' | '${sport}'} [0x2d0a resolution 24]
 leisure=stadium {name '${name} (${sport})' | '${sport}'} [0x2c08 resolution 24]
-leisure=swimming_pool [0x2d09 resolution 24]
+(leisure=swimming_pool | amenity=swimming_pool) & access!=private & access!=no [0x2d09 resolution 24]
 leisure=track {name '${name} (${sport})' | '${sport}'} [0x2c08 resolution 24]
 leisure=water_park [0x2d09 resolution 24]
 
@@ -219,15 +220,30 @@
 # Edge 705 displays 0x650a,0x6511,0x6512,0x6513,0x6603,0x6614 as hollow white circles, no menu
 natural=cave_entrance [0x6601 resolution 24]
 natural=cape [0x6606 resolution 24]
-natural=cliff [0x6607 resolution 24]
+natural=cliff & name=* [0x6607 resolution 24]
 natural=peak {name '${name|def:}${ele|height:m=>ft|def:}'} [0x6616 resolution 24]
 natural=rock [0x6614 resolution 24]
 natural=volcano [0x2c0c resolution 24]
+natural=bay & name=* [0x6503 resolution 20]
 
 railway=station [0x2f08 resolution 22]
-(public_transport=platform & rail=yes) | railway=halt [0x2f08 resolution 23]
-public_transport=platform | highway=bus_stop | railway=tram_stop [0x2f17 resolution 24]
+railway=halt [0x2f08 resolution 23]
+# if option --add-pois-to-lines in effect, add just 1 POI
+(railway=platform | (public_transport=platform & railway=*)) & (mkgmap:line2poi!=true | mkgmap:line2poitype=mid) [0x2f08 resolution 23]
+public_transport=platform & (mkgmap:line2poi!=true | mkgmap:line2poitype=mid) [0x2f17 resolution 24]
+highway=bus_stop | railway=tram_stop [0x2f17 resolution 24]
 
+shop=* & name!=* & shop!=yes & shop!=no & shop!=none & shop!=vacant {set name='${shop|subst:"_=> "}'}
+# Uncomment the following lines to enable these extra POI that some devices support
+#shop=convenience [0x2e0e resolution 24]
+#shop=florist [0x2e0f resolution 24]
+#shop=gift | shop=art | shop=antiques [0x2e10 resolution 24]
+#shop=music [0x2e11 resolution 24]
+#shop=sports [0x2e12 resolution 24]
+#shop=wine | shop=alcohol [0x2e13 resolution 24]
+#shop=books [0x2e14 resolution 24]
+# to here
+
 shop=bakers [0x2e02 resolution 24]
 shop=bakery [0x2e02 resolution 24]
 shop=beauty [0x2f10 resolution 24]
@@ -255,11 +271,12 @@
 shop=hairdresser [0x2f10 resolution 24]
 shop=mall [0x2e04 resolution 24]
 shop=organic [0x2e0a resolution 24]
+shop=outdoor [0x2e08 resolution 24]
 shop=shoes [0x2e07 resolution 24]
 shop=supermarket [0x2e02 resolution 24]
 shop=tires [0x2f07 resolution 24]
 shop=tyres [0x2f07 resolution 24]
-shop=* & shop!=no & shop!=none {add name='${shop|subst:"_=> "}'} [0x2e0c resolution 24]
+shop=* & name=* [0x2e0c resolution 24]
 
 sport=9pin [0x2d07 resolution 24]
 sport=10pin | leisure=bowling_alley [0x2d07 resolution 24]
@@ -275,12 +292,14 @@
 tourism=artwork [0x2c04 resolution 24]
 tourism=aquarium [0x2c07 resolution 24]
 tourism=bed_and_breakfast [0x2b02 resolution 24]
-tourism=camp_site [0x2b05 resolution 24]
-tourism=caravan_site [0x2b05 resolution 24]
+# NB: different devices use different codes for camp/caravan_site, have both!
+tourism=camp_site [0x2b03 resolution 24] [0x2b05 resolution 24]
+tourism=caravan_site [0x2b03 resolution 24] [0x2b05 resolution 24]
 tourism=chalet [0x2b02 resolution 24]
 tourism=guest_house [0x2b02 resolution 24]
 tourism=hostel [0x2b02 resolution 24]
-tourism=hotel | tourism=motel [0x2b01 resolution 24]
+# Have both hotel &| restaurant POIs
+tourism=hotel | tourism=motel {set tmp:stopMopUp=yes} [0x2b01 resolution 24 continue with_actions]
 tourism=information [0x4c00 resolution 24]
 # tourism=lean_to replaces some uses of amenity=shelter
 tourism=lean_to [0x2b05 resolution 24 default_name 'lean-to']
@@ -292,8 +311,8 @@
 tourism=viewpoint {name '${name} - ${description}' | '${name}'} [0x2c04 resolution 24]
 tourism=wine_cellar [0x2c0a resolution 24]
 tourism=zoo [0x2c07 resolution 24]
-tourism=* & tourism!=yes & tourism!=no {add name='${tourism|subst:"_=> "}'}
-tourism=* & tourism!=no [0x2c0d resolution 24]
+tourism=* & name!=* & tourism!=yes & tourism!=no {set name='${tourism|subst:"_=> "}'}
+tourism=* & tourism!=no & tmp:stopMopUp!=yes [0x2c0d resolution 24]
 
 # amenity=restaurant/fast_food/cafe/pub and shop=* can use cuisine, so have cuisine section later than where the others
 # should take precedence
@@ -355,7 +374,8 @@
     barrier=stile | barrier=kissing_gate | barrier=lift_gate | barrier=swing_gate
     {add name='${barrier|subst:"_=> "}'} [0x3200 resolution 24]
 
-landuse=basin | landuse=reservoir [0x650f resolution 24]
+landuse=basin [0x6603 resolution 24]
+landuse=reservoir | water=reservoir [0x650f resolution 22]
 
 natural=beach [0x6604 resolution 24]
 natural=glacier [0x650a resolution 24]
@@ -362,6 +382,7 @@
 natural=spring [0x6511 resolution 24]
 natural=stream [0x6512 resolution 24]
 natural=water & (water=canal | water=lock) & name=* [0x6505 resolution 24]
+(water=lake | water=pond) & name=* [0x650d resolution 24]
 natural=water & name=* [0x6603 resolution 24]
 natural=waterfall | waterway=waterfall [0x6508 resolution 24]
 natural=wetland & name=* [0x6513 resolution 24]
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to