G-MAP mobility profile-based routing [Review only - DO NOT MERGE] #197
Conversation
src/main/java/org/opentripplanner/apis/gtfs/mapping/RouteRequestMapper.java
Outdated
Show resolved
Hide resolved
src/main/java/org/opentripplanner/graph_builder/module/osm/OsmModule.java
Outdated
Show resolved
Hide resolved
| @@ -85,6 +88,8 @@ public class StreetEdge | |||
| */ | |||
| private float bicycleSafetyFactor; | |||
|
|
|||
| public Map<MobilityProfile, Float> profileCost = new HashMap<>(); | |||
There was a problem hiding this comment.
If you notice that this causes a noticeable slowdown you can also create an array of and the use MobilityProfile.ordinal to access the values. This also saves some memory as arrays are extremely runtime- and space-efficient.
However, my gut feeling is that performance is not a huge priority so it's probably good enough as it is.
There was a problem hiding this comment.
Won't do at this time.
There was a problem hiding this comment.
@leonardehrenfried SonarLint suggests using EnumMap instead of HashMap, however, using EnumMap makes OTP unable to deserialize the graph. Have you seen this issue elsewhere?
There was a problem hiding this comment.
EnumMap is indeed faster than a HashMap. EnumMap also says that it is serializable.
In OTP we do use enum map sometimes but I cannot find an example where it is actually store in the graph file.
Can you post the stacktrace please?
There was a problem hiding this comment.
No need to post the stacktrace, I tried it out myself.
Serializing an enum map is indeed tricky as the default serialization uses JDK internals. However, if you want to do it anyway, here is a commit that shows you how to do it: 797d727
Of course you need to swap out the TransitMode with the enum that you want to serialize.
There was a problem hiding this comment.
Thank you, that looks useful.
src/main/resources/org/opentripplanner/apis/gtfs/schema.graphqls
Outdated
Show resolved
Hide resolved
|
This branch seems to fail the graph build when no profile data is configured. |
…ng of CSV parsing outcome
Summary
This PR adds support for mobility profile-based routing for the ITS4US G-MAP project, in which Arcadis IBI participates. A set of mobility profiles are predefined. For each mobility profile, special weights/costs are added, at graph build time, from a CSV file, to certain paths in the OSM street network. The CSV file and the predefined mobility profiles are sourced from a third-party proprietary server that provides costs for each mobility profile for select OSM paths. Profile-based routing is performed based on a new parameter in the
plangraphQL endpoint.Issue
None
Unit tests
TBA
Documentation
TBA
Changelog
N/A
Bumping the serialization version id
TBA