Extend max duration to address the other part of opentripplanner#2148 …

…(biasing results by only including the trips with least waiting)
flibbertigibbet · Oct 19, 2015 · a435731 · a435731
1 parent 040258d
commit a435731
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/main/java/org/opentripplanner/profile/RaptorWorker.java b/src/main/java/org/opentripplanner/profile/RaptorWorker.java
@@ -33,7 +33,10 @@ public class RaptorWorker {
 
     private static final Logger LOG = LoggerFactory.getLogger(RaptorWorker.class);
     public static final int UNREACHED = Integer.MAX_VALUE;
-    static final int MAX_DURATION = 120 * 60;
+
+    // this should be somewhat longer than the cutoff in the UI because we don't want to take away, say,
+    // the longer half of the ways to reach a destination and take the average of the rest
+    static final int MAX_DURATION = 180 * 60;
 
     /**
      * The number of randomized frequency schedule draws to take for each minute of the search.

diff --git a/src/main/java/org/opentripplanner/profile/RepeatedRaptorProfileRouter.java b/src/main/java/org/opentripplanner/profile/RepeatedRaptorProfileRouter.java
@@ -241,7 +241,7 @@ public TIntIntMap findInitialStops(boolean dest, RaptorWorkerData data) {
         if (data == null) {
             // Non-transit mode. Search out to the full 120 minutes.
             // Should really use directModes.
-            rr.worstTime = rr.dateTime + 120 * 60;
+            rr.worstTime = rr.dateTime + RaptorWorker.MAX_DURATION;
             rr.dominanceFunction = new DominanceFunction.EarliestArrival();
         } else {
             // Transit mode, limit pre-transit travel.