G-MAP mobility profile-based routing [Review only - DO NOT MERGE]#197
G-MAP mobility profile-based routing [Review only - DO NOT MERGE]#197binh-dam-ibigroup wants to merge 122 commits intoibi-dev-2.xfrom
Conversation
| */ | ||
| 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.
|
This branch seems to fail the graph build when no profile data is configured. |
leonardehrenfried
left a comment
leonardehrenfried
left a comment
There was a problem hiding this comment.
I've left a few comments but none of them seem very urgent to me, particularly because it's not intended to be merged. If this branch works for you then this is good to go.
…ng of CSV parsing outcome
2 participants
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