Country specific highway access - Project-OSRM / osrm-backend #7038

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Unreleased
   - Changes from 5.27.1
     - Features
+      - ADDED: Routing profiles to support country differences [#7013](https://github.com/Project-OSRM/osrm-backend/pull/7013)
       - ADDED: Route pedestrians over highway=platform [#6993](https://github.com/Project-OSRM/osrm-backend/pull/6993)
       - REMOVED: Remove all core-CH left-overs [#6920](https://github.com/Project-OSRM/osrm-backend/pull/6920)
       - ADDED: Add support for a keepalive_timeout flag. [#6674](https://github.com/Project-OSRM/osrm-backend/pull/6674)

docs/profiles.md

@@ -375,3 +375,23 @@ There are a few helper functions defined in the global scope that profiles can u
 - `trimLaneString`
 - `applyAccessTokens`
 - `canonicalizeStringList`
+
+### Profiles for country differences
+Some countries have differing access rules for different transport types. (https://wiki.openstreetmap.org/wiki/OSM_tags_for_routing/Access_restrictions). Profiles countrybicycle and countryfoot may be used as starting points for country support. 
+
+To enable support add  'countryspeeds' to the uselocationtags Set. e.g.
+
+    uselocationtags = Set
+    {
+        'countryspeeds',
+    }
+
+Country Specific location data (geojson geometry) should be provided to the osrm-extract for example:
+
+osrm-extract --profile ../profiles/countryfoot.lua --location-dependent-data ../data/allthecountries.geojson
+
+osrm-extract --profile ../profiles/countrybicycle.lua --location-dependent-data ../mygeojson/Netherlands.geojson
+
+If you enable the tag but provide no location-dependent-data, the profile will use the 'Worldwide' settings. (https://wiki.openstreetmap.org/wiki/OSM_tags_for_routing/Access_restrictions#Worldwide)
+
+It is possible to to include multiple countries in the location data file through the use of a FeatureCollection.

features/bicycle/countrybikespeeds.feature

@@ -0,0 +1,89 @@
+@routing @bicycle @countrybikespeeds
+Feature:  Testbot - Country Profile Selection
+
+    # Check that country data is being selected
+    # CHE (a) does not support trunk on bike
+    # FIN (c) does not support trunk on bike
+    # GRC (m) and IRL (o) support bridleway
+    # LIU (s) is not a country in list so looks like worldwide
+    # so
+    # GRC (m) and LIU (s) support trunk
+
+    Scenario: Profile Selection - highway chosen for country
+
+        Given the extract extra arguments "--location-dependent-data test/data/countrytest.geojson"
+        And the partition extra arguments "--threads 1"
+        And the customize extra arguments "--threads 1"
+
+        And the profile file "bicycle" initialized with
+        """
+        profile.uselocationtags.countryspeeds = true
+        """
+
+        And the node locations
+            | node | lat        | lon      | 
+            | a    | 9.5        | 5.0      |
+            | b    | 9.5        | 10.0     |
+            | c    | 9.5        | 15.0     |
+            | d    | 9.0        | 5.0      |
+            | e    | 9.0        | 10.0     |
+            | f    | 9.0        | 15.0     |
+            | g    | 8.5        | 5.0      |
+            | h    | 8.5        | 10.0     |
+            | i    | 8.5        | 15.0     |
+            | j    | 8.0        | 5.0      |
+            | k    | 8.0        | 10.0     |
+            | l    | 8.0        | 15.0     |
+            | m    | 7.5        | 5.0      |
+            | n    | 7.5        | 10.0     |
+            | o    | 7.5        | 15.0     |
+            | p    | 7.0        | 5.0      |
+            | q    | 7.0        | 10.0     |
+            | r    | 7.0        | 15.0     |
+            | s    | 6.5        | 5.0      |
+            | t    | 6.5        | 10.0     |
+            | u    | 6.5        | 15.0     |
+
+        And the ways
+            | nodes | highway     | comment
+            | ab    | trunk       | CHE - FIN no
+            | bc    | trunk       |
+            | cf    | trunk       | FIN - BEL no
+            | fi    | trunk       | 
+            | fl    | trunk       | 
+            | or    | trunk       | IRL - world yes
+            | ru    | trunk       |
+            | mn    | bridleway   | GRC - IRL yes
+            | no    | bridleway   |
+            | mp    | trunk       | GRC - liu yes
+            | ps    | trunk       |
+
+        And the partition extra arguments "--threads 1"
+        And the customize extra arguments "--threads 1"
+
+
+        When I route I should get
+            | waypoints | route          | status | message                          |
+            | a,b,c     |                | 400    | Impossible route between points  |
+            | c,l       |                | 400    | Impossible route between points  |
+            | g,i       |                | 400    | Impossible route between points  |
+            | o,u       | or,ru,ru       | 200    |    |
+            | m,o       | mn,no,no       | 200    |    | 
+            | m,s       | mp,ps,ps       | 200    |    | 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+

features/bicycle/countrymotorroad.feature

@@ -0,0 +1,39 @@
+@routing @bicycle @countrymotorroad
+Feature:  Testbot - Country motorroad Selection
+
+    # Check that motorroad is avoided
+
+    Scenario: bicycle Profile Selection - motorroad avoided
+
+        Given the extract extra arguments "--threads 1"
+        And the partition extra arguments "--threads 1"
+        And the customize extra arguments "--threads 1"
+
+        And the profile file "bicycle" initialized with
+        """
+        profile.uselocationtags.countryspeeds = true
+        """
+
+        And the node locations
+            | node | lat        | lon      | 
+            | a    | 9.5        | 5.0      |
+            | b    | 9.5        | 10.0     |
+            | c    | 9.5        | 15.0     |
+            | d    | 9.0        | 5.0      |
+            | e    | 9.0        | 10.0     |
+            | f    | 9.0        | 15.0     |
+
+        And the ways
+            | nodes | highway  | motorroad |
+            | ab    | trunk    | yes       |    
+            | bc    | trunk    |           |
+            | ad    | trunk    |           |
+            | de    | trunk    |           |
+            | ef    | trunk    |           |  
+            | cf    | trunk    |           |
+
+        When I route I should get
+            | waypoints | route             | status | message                          |
+            | a,c       | ad,de,ef,cf,cf    | 200    |                                  |
+            | a,b       | ad,de,ef,cf,bc,bc | 200    |                                  |
+

features/bicycle/countryway.feature

@@ -0,0 +1,42 @@
+@routing @foot @countryway
+Feature: bicycle - Accessability of different way types with countryspeeds
+
+    Background:
+	Given the profile file "bicycle" initialized with
+        """
+        profile.uselocationtags.countryspeeds = true
+        """
+
+    Scenario: Countrybicycle - Basic access
+        Then routability should be
+            | highway        | forw |
+            | motorway       |      |
+            | motorway_link  |      |
+            | trunk          | x    |
+            | trunk_link     | x    |
+            | primary        | x    |
+            | primary_link   | x    |
+            | secondary      | x    |
+            | secondary_link | x    |
+            | tertiary       | x    |
+            | tertiary_link  | x    |
+            | residential    | x    |
+            | service        | x    |
+            | unclassified   | x    |
+            | living_street  | x    |
+            | road           | x    |
+            | track          | x    |
+            | path           | x    |
+            | footway        | x    |
+            | pedestrian     | x    |
+            | steps          | x    |
+            | pier           | x    |
+            | cycleway       | x    |
+            | bridleway      |      |
+
+    Scenario: Country bicycle - Routability of man_made structures
+        Then routability should be
+            | highway | man_made | bothw |
+            | (nil)   | (nil)    |       |
+            | (nil)   | pier     | x     |
+