Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

restriction=only_straight_on not taken into account #1681

Open
1 task done
koebi opened this issue Feb 7, 2024 · 1 comment
Open
1 task done

restriction=only_straight_on not taken into account #1681

koebi opened this issue Feb 7, 2024 · 1 comment

Comments

@koebi
Copy link
Collaborator

koebi commented Feb 7, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Where did you encounter this issue?

live API

Request URL

https://classic-maps.openrouteservice.org/directions?n1=49.407492&n2=8.676386&n3=18&a=49.406436,8.676903,49.407978,8.673247&b=0&c=0&k1=de&k2=km

POST Request Body

{"preference":"recommended","language":"de","instructions":true,"geometry":true,"units":"m","attributes":["detourfactor","percentage"],"instructions_format":"html","elevation":true,"coordinates":[[8.676903,49.406436],[8.673247,49.407978]],"extra_info":["steepness","waytype","surface"]}

Response

Response
{"type":"FeatureCollection","metadata":{"attribution":"openrouteservice.org | OpenStreetMap contributors","service":"routing","timestamp":1707308272459,"query":{"coordinates":[[8.676903,49.406436],[8.673247,49.407978]],"profile":"driving-car","preference":"recommended","format":"geojson","units":"m","language":"de","geometry":true,"instructions":true,"instructions_format":"html","attributes":["detourfactor","percentage"],"elevation":true,"extra_info":["steepness","waytype","surface"]},"engine":{"version":"7.1.1","build_date":"2024-01-29T14:41:12Z","graph_date":"2024-02-04T14:27:02Z"}},"bbox":[8.673247,49.406436,113.64,8.676923,49.408006,117.0],"features":[{"bbox":[8.673247,49.406436,113.64,8.676923,49.408006,117.0],"type":"Feature","properties":{"ascent":0.0,"descent":3.4,"transfers":0,"fare":0,"segments":[{"distance":430.8,"duration":88.6,"steps":[{"distance":169.9,"duration":19.4,"type":11,"instruction":"Weiter nördlich auf <b>Mittermaierstraße, L 600a</b>","name":"Mittermaierstraße, L 600a","way_points":[0,2]},{"distance":260.9,"duration":69.2,"type":0,"instruction":"Biegen Sie links auf <b>Bergheimer Straße</b> ab","name":"Bergheimer Straße","way_points":[2,16]},{"distance":0.0,"duration":0.0,"type":10,"instruction":"Sie erreichen Bergheimer Straße (links)","name":"-","way_points":[16,16]}],"detourfactor":1.35,"percentage":100.0,"descent":3.362499999999997}],"extras":{"surface":{"values":[[0,16,3]],"summary":[{"value":3.0,"distance":430.8,"amount":100.0}]},"waytypes":{"values":[[0,2,1],[2,16,2]],"summary":[{"value":2.0,"distance":260.9,"amount":60.56},{"value":1.0,"distance":169.9,"amount":39.44}]},"steepness":{"values":[[0,16,0]],"summary":[{"value":0.0,"distance":430.8,"amount":100.0}]}},"way_points":[0,16],"summary":{"distance":430.8,"duration":88.6}},"geometry":{"coordinates":[[8.676923,49.406436,117.0],[8.676855,49.407827,117.0],[8.67685,49.407964,117.0],[8.676752,49.407966,117.0],[8.676671,49.407968,117.0],[8.676401,49.407967,116.9],[8.676148,49.407963,116.7],[8.676125,49.407963,116.6],[8.675317,49.407958,115.8],[8.674914,49.407951,115.5],[8.67471,49.407958,115.4],[8.674505,49.407968,115.2],[8.674375,49.407977,115.1],[8.674021,49.408,114.7],[8.673628,49.408005,114.3],[8.673568,49.408005,114.1],[8.673247,49.408006,113.6]],"type":"LineString"}}]}

Current behavior

The route is passing this restriction relation, but still turning to the left. This shouldn't happen.

Expected behavior

The route should go straight as per the restriction

Openrouteservice Version

7.1.1

Build date

2024-01-29T14:41:12Z

Graph date

2024-02-04T14:27:02Z

Forum Topic Link

No response

@takb takb added this to To do in ors general Feb 7, 2024
@koebi
Copy link
Collaborator Author

koebi commented Feb 7, 2024

This is an artifact of routing without any avoidables using CH, which doesn't take these restrictions into account.

Currently, CH is still needed for fast matrix (RPHAST).
The mid-term plan would be to finish up Core-Matrix and then drop/disable (support for) CH/RPHAST.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
ors general
  
To do
Development

No branches or pull requests

1 participant