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

Wrong routing after adding mutually exclusive car and truck restrictions #5925

Closed
ftrebien opened this issue Jan 5, 2021 · 6 comments
Closed
Labels

Comments

@ftrebien
Copy link

ftrebien commented Jan 5, 2021

After this change, OSRM stopped computing the correct route and started computing this incorrect route instead:

The new route leads drivers through a path that is only allowed for trucks.

Also, before this change from a single only_straight_on restriction to two no_u_turn restrictions here, OSRM was refusing to route through this interchange when crossing towards Brazil.

@mjjbell
Copy link
Member

mjjbell commented Jan 7, 2021

Sorry if I'm misunderstanding - the change applieshgv=no to the inner loop road, whilst the outer loop road has hgv=designated. Wouldn't this be the expected behaviour?

@mjjbell
Copy link
Member

mjjbell commented Jan 7, 2021

Yes, I was misunderstanding 😄
It looks like the roads have been edited a few times recently, so I'm not sure which version of the data is currently deployed.

Nevertheless, the reason cars are not routed via the inner loop is due to a height restriction of 2.4 metres, which is lower than that being used by the fossgis routing profile.

You can see in the debug tiles that the road is not accessible to cars with this profile.

@ftrebien
Copy link
Author

ftrebien commented Jan 8, 2021

Oh, I didn't notice that. That height limit is correct according to local signage, so I guess it can't be helped without causing problems elsewhere.

@GitBenjamin
Copy link
Contributor

@mjjbell
5.24.0
Where is the branch of this change FIXED: Reduce copying in API parameter constructors ?
The link in the changelog.md seems to be wrong !

@mjjbell
Copy link
Member

mjjbell commented Sep 14, 2022

@mjjbell 5.24.0 Where is the branch of this change FIXED: Reduce copying in API parameter constructors ? The link in the changelog.md seems to be wrong !

Off-by-one error: #5926

Copy link

github-actions bot commented Jul 8, 2024

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

@github-actions github-actions bot added the Stale label Jul 8, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants