Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bicycle: treat use_sidepath as no access #5622

Merged
merged 1 commit into from
Dec 2, 2019

Conversation

jdhoek
Copy link
Contributor

@jdhoek jdhoek commented Dec 1, 2019

Issue

Fixes #5557; treat bicycle=use_sidepath as no access on the tagged way. When this tag is used, a parallel way that is legally suitable for bicycles will have been mapped, and the way marked with use_sidepath should be ignored.

Please see the OSM Wiki for details on the tag.

Tasklist

@jdhoek jdhoek force-pushed the bug/bicycle_use_sidepath branch from 30a4e31 to 92406da Compare December 1, 2019 17:35
@jdhoek
Copy link
Contributor Author

jdhoek commented Dec 1, 2019

Regarding update relevant Wiki pages; it doesn't look like there are any relevant wiki-pages for a fix like this. Is that correct?

@gardster
Copy link
Contributor

gardster commented Dec 2, 2019

Yes, its correct. There is no relevant wiki page were we describe all relevant tags for different profiles.

@gardster gardster merged commit 68dcab7 into Project-OSRM:master Dec 2, 2019
@gardster
Copy link
Contributor

gardster commented Dec 2, 2019

Thank you for your contribution and for the covering test without reminders 👍

@jdhoek
Copy link
Contributor Author

jdhoek commented Dec 2, 2019

@gardster My pleasure. The documentation proved sufficient to get started (appreciated!).

@jdhoek jdhoek mentioned this pull request Dec 9, 2019
datendelphin added a commit to fossgis-routing-server/osrm-backend that referenced this pull request Nov 19, 2020
  - Changes from 5.22.0
    - Build:
      - FIXED: pessimistic calls to std::move [Project-OSRM#5560](Project-OSRM#5561)
    - Features:
      - ADDED: new API parameter - `snapping=any|default` to allow snapping to previously unsnappable edges [Project-OSRM#5361](Project-OSRM#5361)
      - ADDED: keepalive support to the osrm-routed HTTP server [Project-OSRM#5518](Project-OSRM#5518)
      - ADDED: flatbuffers output format support [Project-OSRM#5513](Project-OSRM#5513)
      - ADDED: Global 'skip_waypoints' option [Project-OSRM#5556](Project-OSRM#5556)
      - FIXED: Install the libosrm_guidance library correctly [Project-OSRM#5604](Project-OSRM#5604)
      - FIXED: Http Handler can now deal witch optional whitespace between header-key and -value [Project-OSRM#5606](Project-OSRM#5606)
    - Routing:
      - CHANGED: allow routing past `barrier=arch` [Project-OSRM#5352](Project-OSRM#5352)
      - CHANGED: default car weight was reduced to 2000 kg. [Project-OSRM#5371](Project-OSRM#5371)
      - CHANGED: default car height was reduced to 2 meters. [Project-OSRM#5389](Project-OSRM#5389)
      - FIXED: treat `bicycle=use_sidepath` as no access on the tagged way. [Project-OSRM#5622](Project-OSRM#5622)
      - FIXED: fix table result when source and destination on same one-way segment. [Project-OSRM#5828](Project-OSRM#5828)
      - FIXED: fix occasional segfault when swapping data with osrm-datastore and using `exclude=` [Project-OSRM#5844](Project-OSRM#5844)
      - FIXED: fix crash in MLD alternative search if source or target are invalid [Project-OSRM#5851](Project-OSRM#5851)
    - Misc:
      - CHANGED: Reduce memory usage for raster source handling. [Project-OSRM#5572](Project-OSRM#5572)
      - CHANGED: Add cmake option `ENABLE_DEBUG_LOGGING` to control whether output debug logging. [Project-OSRM#3427](Project-OSRM#3427)
      - CHANGED: updated extent of Hong Kong as left hand drive country. [Project-OSRM#5535](Project-OSRM#5535)
      - FIXED: corrected error message when failing to snap input coordinates [Project-OSRM#5846](Project-OSRM#5846)
    - Infrastructure
      - REMOVED: STXXL support removed as STXXL became abandonware. [Project-OSRM#5760](Project-OSRM#5760)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bicycle and oneway, routing wrong
2 participants