-
Notifications
You must be signed in to change notification settings - Fork 121
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
Support for access:conditional #300
Comments
I bet there would be difficult implementation problem for coding it to RD5 files, and then evaluting them during route generation and real time route following. As it has highly variable syntax of the tag values. BRouter naturally prefers a list of enumerated tagvalues, often limited to the most frequent or grouped similar ones. Some tags have dozens or possibly even hundreds of possible values, often with just 1 of few usages. If there is time conditional restriction, it may be allowed at the time the route is calculated, but forbidden when the route is followed. Or vice versa. Other issue can be circumstanced or trave mode related restrictions. |
I had some thoughts on conditional restrictions a while ago: #193 however, that's the general case including intraday-changes conditions on a seasonal timeline could be processed at the preprocessor-level, which is much simpler. But still not high on the priority list currently |
I'm not saying the values need to be parsed and interpreted necessarily. After all, I might create a route in winter but actually use it in summer when the condition no longer applies. But IMHO the user should be made aware at the time of route creation that there are some restrictions on a route. It could be left to the client on what to do with the actual values (just display them or interpret them somehow). Komoot for instance pops up a generic warning. According to taginfo, access:conditional ion ways is used 21029 times right now... |
Yes, that is I think a good idea that can be implemented with less effort than handling it within the brouter App. It would be nice if also the OSM way/node id for the restriction is given. With that information for example brouter-web could retrieve those ways/nodes and present the user with the list of conditional restrictions to evaluate. |
Had a look on this again in the light of #416. First though was to encode :conditional=, i.e. all conditional keys with all possible values. First I realized that there is also "maxspeed:conditional" etc., so restriction on access/transport keys and then I had a look at Taginfo keys and filtered on ":conditional", tagged 318198 times on 20220410 The top 10:
So, yes, filtering on access/transport is needed. Filtering on access/transport and splitting out ways/nodes you get (top 25):
I would propose to take into account:
|
I would be nice if brouter would support the access:conditional tag. This tag is rather important when observing restrictions inside protection areas (one example are the German Wald- und Wild Schongebiete).
Typically, these areas are not rendered in standard map tiles, users build there routes and end up in areas they are not supposed to be at a given time, claiming that some app, probably bases on OSM "has sent them there without warnings".
One solution to this would be to at least pop up a user-friendly warning when a route is created, e.g. in brouter's web frontend and otherwise make the access information fully available in brouter's API so that other apps can pick it up, too. Typical values for that tag are discouraged or no with the respective timerange information as per the opening hours spec.
I haven't checked if access:conditional does not appear in brouter at all, but access:conditional: discouraged @ (Dec-May) - of which we have ~ 620 in Southern Bavaria alone at this time (amount growing) - does not appear at all in the web frontend's route data table way tags.
The text was updated successfully, but these errors were encountered: