De-prioritize ways tagged with foot=use_sidepath

ratrun · Jul 6, 2024 · ce1235a · ce1235a
1 parent 6b1995c
commit ce1235a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/core/src/main/java/com/graphhopper/routing/util/parsers/FootPriorityParser.java b/core/src/main/java/com/graphhopper/routing/util/parsers/FootPriorityParser.java
@@ -104,6 +104,10 @@ void collect(ReaderWay way, TreeMap<Double, PriorityCode> weightToPrioMap) {
         if (way.hasTag("foot", "designated"))
             weightToPrioMap.put(100d, PREFER);
 
+        if (way.hasTag("foot", "use_sidepath")) {
+            weightToPrioMap.put(100d, VERY_BAD);
+        }
+
         double maxSpeed = Math.max(getMaxSpeed(way, false), getMaxSpeed(way, true));
         if (safeHighwayTags.contains(highway) || (isValidSpeed(maxSpeed) && maxSpeed <= 20)) {
             weightToPrioMap.put(40d, PREFER);

diff --git a/core/src/test/java/com/graphhopper/routing/util/parsers/FootTagParserTest.java b/core/src/test/java/com/graphhopper/routing/util/parsers/FootTagParserTest.java
@@ -333,7 +333,11 @@ public void testPriority() {
         way.clearTags();
         way.setTag("highway", "tertiary");
         assertEquals(PriorityCode.UNCHANGED.getValue(), prioParser.handlePriority(way, null));
+        way.setTag("foot","use_sidepath");
+        assertEquals(PriorityCode.VERY_BAD.getValue(), prioParser.handlePriority(way, null));
 
+        way.clearTags();
+        way.setTag("highway", "tertiary");
         // tertiary without sidewalk is roughly like primary with sidewalk
         way.setTag("sidewalk", "no");
         assertEquals(PriorityCode.AVOID.getValue(), prioParser.handlePriority(way, null));