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

Country specific highway access. fixes Project-OSRM / osrm-backend #6710 #7013

Closed
wants to merge 37 commits into from

Conversation

tombay
Copy link

@tombay tombay commented Aug 6, 2024

#6710
An issue #6710 has been logged (reopening #6701).
Address the issue of highway=trunk (and trunk_link) being blocked for foot and bicycle profiles. Discussed in https://wiki.openstreetmap.org/wiki/Key:motorroad and https://wiki.openstreetmap.org/wiki/OSM_tags_for_routing/Access_restrictions (esp #Alternative_ideas) proposed to address issue via
osrm-backend ... --location-dependent-data

Tasklist

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

Copy link
Author

@tombay tombay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initial load

@tombay tombay marked this pull request as draft August 12, 2024 01:01
@tombay tombay marked this pull request as ready for review August 12, 2024 01:03
@tombay
Copy link
Author

tombay commented Aug 18, 2024

Worldwide geojson file.
I have a geojson file containing the 32 countries involved. The file is 43Mb (13.5 gzipped). I have not uploaded it. The data was obtained from osm-boundaries.com. They source their data from openstreetmap.org, and prefer to have a link to them as the source of the data. I also assume that anybody making use of these profiles would only be serving one or two countries.

@tombay
Copy link
Author

tombay commented Aug 18, 2024

Issues:
1/ Ways that cross country boundaries only have one country name associated. This means that a way may be included (or dropped) inadvertently. It would overly complicate the solution to address a transient that should be rectified in the subsequent way in the route.
2/ Countries with intrastate differences. Australia has different rules for bicycles on footways. Two states (nsw and vic) have highway="footway" with the default bicycle="no". It is possible to make use of "bicycle_nsw" and "bicycle_vic" as 'Countries", but the assumption is that the correct "bicycle" tags have been addedd to OSM data.
Austria has a region (tyrol) that has different access for vehicle traffic. If a "countryvehicle" profile is desired this can also be covered by using "tryol" as a country.
3/ Incomplete and inconsistent data in the access tables. Some tables (e.g. Belarus) have footnotes to describe table entries. Other cases have caveats like "destination" or "service". The general assumption is that if does not say "no" then assume "yes".
As mentioned in "https://wiki.openstreetmap.org/wiki/OSM_tags_for_routing/Access_restrictions" -
As per 2024, no mainstream routing engine is known to use this page directly to influence routing behavior.

This exception should be caught but it is not happeing in my runs. So just testing without it.
put this back as error is expected. see why it is not being picked up
@tombay tombay changed the title Provide support for country specific highway access. Provide support for country specific highway access. Fixes #6710 Aug 27, 2024
@tombay tombay changed the title Provide support for country specific highway access. Fixes #6710 Provide support for country specific highway access. Aug 27, 2024
@tombay
Copy link
Author

tombay commented Aug 27, 2024

Fixes Project-OSRM/osrm-backend #6710

@tombay tombay changed the title Provide support for country specific highway access. Country specific highway access. fixes Project-OSRM / osrm-backend #6710 Aug 27, 2024
@tombay tombay marked this pull request as draft August 27, 2024 23:42
@tombay tombay marked this pull request as ready for review August 27, 2024 23:44
@tombay tombay marked this pull request as draft August 27, 2024 23:45
@tombay tombay marked this pull request as ready for review August 27, 2024 23:46
@tombay tombay closed this Aug 29, 2024
@tombay tombay deleted the tombay-country-profiles branch August 29, 2024 04:38
@tombay tombay restored the tombay-country-profiles branch August 29, 2024 04:38
@tombay tombay deleted the tombay-country-profiles branch August 29, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant