Project-OSRM / osrm-backend

Open Source Routing Machine - C++ backend
http://map.project-osrm.org
BSD 2-Clause "Simplified" License
6.29k stars 3.32k forks source link

Destinations with :forward / :backward are ignored #6283

Closed SamuelBrucksch closed 1 month ago

SamuelBrucksch commented 2 years ago

This already has been implemented a while ago and the code for that is here: https://github.com/Project-OSRM/osrm-backend/blob/06b1b980bbdfc838c4f8e8cb370cba6dfb0329b0/profiles/lib/destination.lua#L3-L13

However it seems, that this logic does not work. For example on this route: https://map.project-osrm.org/?z=18&center=49.225556%2C8.607002&loc=49.226146%2C8.605900&loc=49.225512%2C8.608330&hl=en&alt=0&srv=0

And here is the osm property: https://www.openstreetmap.org/way/979666549

The returned instruction object looks like this:

{
              "geometry": "klmkH{aps@Lq@He@f@kC",
              "maneuver": {
                "exit": 2,
                "bearing_after": 112,
                "bearing_before": 22,
                "location": [8.60718, 49.225815],
                "modifier": "right",
                "type": "exit roundabout"
              },
              "mode": "driving",
              "ref": "L 555",
              "driving_side": "right",
              "name": "Am Autobahnzubringer",
              "intersections": [
                {
                  "out": 1,
                  "in": 2,
                  "entry": [true, true, false],
                  "bearings": [0, 120, 210],
                  "location": [8.60718, 49.225815]
                }
              ],
              "weight": 5.7,
              "duration": 5.7,
              "distance": 90.1
            },

However it should contain the property from the destination, instead of the road name.

github-actions[bot] commented 2 months ago

This issue seems to be stale. It will be closed in 30 days if no further activity occurs.