Skip to content

Filter out elevator ways that are also implicit areas#3850

Merged
leonardehrenfried merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_fix_elevator_ways
Feb 1, 2022
Merged

Filter out elevator ways that are also implicit areas#3850
leonardehrenfried merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_fix_elevator_ways

Conversation

@hannesj

@hannesj hannesj commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

Summary

The PR #3750 did not consider elevator which are modeled as areas, without explicitly being defined as such. This adds filtering for these kind of elevators, eg. https://www.openstreetmap.org/way/503412863 or https://www.openstreetmap.org/way/187719215

Issue

Closes #3847

Unit tests

None changed

Code style

Documentation

None changed

Changelog

The changelog file
is generated from the pull-request title, make sure the title describe the feature or issue fixed.
To exclude the PR from the changelog add [changelog skip] in the title.

@hannesj hannesj added the !Bug Apply to issues describing a bug and PRs witch fixes it. label Feb 1, 2022
@hannesj hannesj requested a review from a team as a code owner February 1, 2022 10:06
@t2gran t2gran added this to the 2.1 milestone Feb 1, 2022
…nStreetMapModule.java

Co-authored-by: Leonard Ehrenfried <mail@leonard.io>
@leonardehrenfried leonardehrenfried merged commit 46f44a9 into opentripplanner:dev-2.x Feb 1, 2022
t2gran pushed a commit that referenced this pull request Feb 1, 2022
@hannesj hannesj deleted the otp2_fix_elevator_ways branch February 1, 2022 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

!Bug Apply to issues describing a bug and PRs witch fixes it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nullpointer when loading graph with horizontally moving elevators

3 participants