From d5722b651835fd59941d1094aa78fc1a3b70103f Mon Sep 17 00:00:00 2001
From: ansons <ansonstewart@gmail.com>
Date: Wed, 24 Jan 2024 16:04:33 -0500
Subject: [PATCH] Ignore custom cost fields in CAR routing

This ignores custom costs (e.g. from elevation data) for the CAR street mode. Custom speeds for cars are handled separately.
---
 .../streets/MultistageTraversalTimeCalculator.java | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/main/java/com/conveyal/r5/streets/MultistageTraversalTimeCalculator.java b/src/main/java/com/conveyal/r5/streets/MultistageTraversalTimeCalculator.java
index 952bb603c..382f99727 100644
--- a/src/main/java/com/conveyal/r5/streets/MultistageTraversalTimeCalculator.java
+++ b/src/main/java/com/conveyal/r5/streets/MultistageTraversalTimeCalculator.java
@@ -40,12 +40,14 @@ public MultistageTraversalTimeCalculator (TraversalTimeCalculator base, List<Cos
     public int traversalTimeSeconds (EdgeStore.Edge currentEdge, StreetMode streetMode, ProfileRequest req) {
         final int baseTraversalTimeSeconds = base.traversalTimeSeconds(currentEdge, streetMode, req);
         int t = baseTraversalTimeSeconds;
-        for (CostField costField : costFields) {
-            t += costField.additionalTraversalTimeSeconds(currentEdge, baseTraversalTimeSeconds);
-        }
-        if (t < 1) {
-            LOG.warn("Cost was negative or zero. Clamping to 1 second.");
-            t = 1;
+        if (!streetMode.equals(StreetMode.CAR)) {
+            for (CostField costField : costFields) {
+                t += costField.additionalTraversalTimeSeconds(currentEdge, baseTraversalTimeSeconds);
+            }
+            if (t < 1) {
+                LOG.warn("Cost was negative or zero. Clamping to 1 second.");
+                t = 1;
+            }
         }
         return t;
     }