-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Recognize conditional restrictions #374
Comments
@matkoniecz do you have a real world example for conditional restrictions used for constructions? |
So tagging the access with the 'conditional' trick seems to be done not that often. Only maxspeed:conditional is rather often (44K), access:conditional and vehicle:conditional combined is only 10K. So I vote to implement only these 3 schemes for now and only if they contain dates, not only times. Hourly imports are possible but not often done in practise, at least not now. |
I've created the date parsing logic in a separate branch but still will postpone this. Also see this project for a more generic opening hours parser |
Some more examples: A major road that is closed during winter |
The PR by @boldtrn will be soon merged. Should we support 'pure' opening hours too? Is this tagging schema kind of official for nodes? http://www.openstreetmap.org/node/2519644036 |
We should definitely add the conditional access check for barriers. Here is a great example: https://graphhopper.com/maps/?point=46.491848%2C10.362371&point=46.49173%2C10.362607&locale=de-DE&vehicle=car&weighting=fastest&elevation=true&layer=Omniscale The gate can be found here: http://www.openstreetmap.org/node/2689473341 ´access:conditional no @ (Oct-Apr)´ |
Another case that would be nice to support is parsing of weeks like: An example way that uses this tagging scheme can be found here. |
are time (not date) based conditional restrictions supported yet? |
No, this is not possible yet. |
Constructions should be avoided and assuming a daily re-import we should recognize the conditional tag including the date format defined in opening_hours.
First major part is implemented already in #621. Still missing:
;
no @ week 1-5
The text was updated successfully, but these errors were encountered: