Add flex transit as a sandbox feature #3121

hannesj · 2020-06-25T07:40:55Z

This pull request adds the possibility of using flexible transit for access and egress routes. Currently only unscheduled trips are included.

To be completed by pull request submitter:

issue: Link to or create an issue that describes the relevant feature or bug. Add GitHub keywords to this PR's description (e.g., closes #45).
roadmap: Check the roadmap for this feature or bug. If it is not already on the roadmap, PLC will discuss as part of the review process.
tests: Have you added relevant test coverage? Are all the tests passing on the continuous integration service (Travis CI)?
formatting: Have you followed the suggested code style?
documentation: If you are adding a new configuration option, have you added an explanation to the configuration documentation tables and sections?
changelog: add a bullet point to the changelog file with description and link to the linked issue

To be completed by @opentripplanner/plc:

reviews and approvals by 2 members, ideally from different organizations
after merging: update the relevant card on the roadmap

t2gran · 2020-07-30T13:43:58Z

@hannesj Let me know if you when you want me to review this.

src/main/java/org/opentripplanner/model/Stoplike.java

src/main/java/org/opentripplanner/model/Location.java

src/main/java/org/opentripplanner/routing/vertextype/StreetVertex.java

hannesj · 2020-08-11T09:56:38Z

src/main/java/org/opentripplanner/routing/graph/GraphIndex.java

    private final Map<Station, MultiModalStation> multiModalStationForStations = Maps.newHashMap();
    private final HashGridSpatialIndex<TransitStopVertex> stopSpatialIndex = new HashGridSpatialIndex<>();
    private final Map<ServiceDate, TIntSet> serviceCodesRunningForDate = new HashMap<>();
+    private FlexIndex flexIndex = null;


could be final

src/main/java/org/opentripplanner/gtfs/mapping/GTFSToOtpTransitServiceMapper.java

src/main/java/org/opentripplanner/gtfs/mapping/LocationGroupMapper.java

src/main/java/org/opentripplanner/routing/core/TransferTable.java

src/ext/java/org/opentripplanner/ext/flex/FlexRouter.java

src/ext/java/org/opentripplanner/ext/flex/FlexLocationsToStreetEdgesMapper.java

src/ext/java/org/opentripplanner/ext/flex/template/FlexAccessEgressTemplate.java

src/main/java/org/opentripplanner/routing/algorithm/RoutingWorker.java

hannesj · 2020-09-03T13:10:57Z

src/main/java/org/opentripplanner/graph_builder/linking/SimpleStreetSplitter.java

+                    .createPoint(v0.getCoordinate());
+                Envelope env = p.getEnvelopeInternal();
+                for (FlexStopLocation flexStopLocation : graph.index.getFlexIndex().locationIndex.query(env)) {
+                    if (edge.canTraverse(new TraverseModeSet(TraverseMode.CAR))


check for WALK

hannesj · 2020-09-03T14:14:26Z

src/ext/java/org/opentripplanner/ext/flex/distancecalculator/DistanceCalculator.java

+    public int distance;
+    public int duration;


rename to distanceMeters and durationSeconds

src/main/java/org/opentripplanner/transit/raptor/api/transit/RaptorTransfer.java

leonardehrenfried · 2022-03-22T19:34:31Z

I think this can't be close, can't it?

hannesj · 2022-03-22T21:41:14Z

I think this can't be close, can't it?

Yes, everything has been implemented, or part of PRs #3456 or #3457

t2gran added the OTP2 label Jul 30, 2020

t2gran added the New Feature label Jul 30, 2020