<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
BTW while it is still work in progress (now mainly because the
android UI isn't finished yet)
<a class="moz-txt-link-freetext" href="https://github.com/simonpoole/OpeningHoursParser">https://github.com/simonpoole/OpeningHoursParser</a> is a JavaCC based
parser which attempts to implement the full spec, undoubtedly I've
probably missed one or two special cases, but it is fairly complete.
<br>
<br>
It currently successfully parses 108'455 of 122'100 test strings
(with some relaxation of rules) of those that fail 10'569 seem to be
valid lexical errors and a large part of the remaining errors seem
to have other issues. The test strings were extracted from the OSM
database (nodes only).<br>
<br>
As you say the specification itself is overly complicated (the
"optional" colon is not really a problem, except that it is not
really clear where it is allowed, there is some further similar
fuzziness wrt comments) and definitely shouldn't have more stuff
added to it (with perhaps the exceptions of adding further variable
dates and similar things).<br>
<br>
Simon<br>
<br>
<div class="moz-cite-prefix">Am 21.08.2015 um 12:36 schrieb Ruben
Maes:<br>
</div>
<blockquote cite="mid:3532716.ARd3J7Uc1F@ood" type="cite">
<pre wrap="">Friday 21 August 2015 11:48:49, <a class="moz-txt-link-abbreviated" href="mailto:panieravide@riseup.net">panieravide@riseup.net</a>:
</pre>
<blockquote type="cite">
<pre wrap="">
Hello,
I recently released a new version of YoHours, a website which allows
everyone to create and view opening hours in the OSM syntax. It now
supports seasons-dependent hours (month, week, day, holiday selectors).
It's available here:
<a class="moz-txt-link-freetext" href="http://github.pavie.info/yohours/">http://github.pavie.info/yohours/</a>
The code is available on GitHub:
<a class="moz-txt-link-freetext" href="https://github.com/PanierAvide/panieravide.github.io/tree/master/yohours">https://github.com/PanierAvide/panieravide.github.io/tree/master/yohours</a>
[1]
If you have any suggestions, let me know :)
Cordially,
PanierAvide.
Links:
------
[1]
<a class="moz-txt-link-freetext" href="https://github.com/PanierAvide/panieravide.github.io/tree/master/yohours">https://github.com/PanierAvide/panieravide.github.io/tree/master/yohours</a>
</pre>
</blockquote>
<pre wrap="">
I opened an issue[1] on this GitHub project, because it puts a colon after week, month and monthday selectors.
PanierAvide replied that the specification allows an "optional separator for readability"[2]. Indeed, when you read the overly complicated and totally not mapper-focused specification, you can see
[ <year_selector> ] [ <month_or_monthday_selector> ] [ <week_selector> ] [ <separator_for_readability> ]
Whose idea was this? It's already complicated enough that you don't have to add *optional* separators for supposed readability.
IMO it's just fine without them.
PS: I always follow the time domains proposal[3]. It's clear and it's compatible with the other specification AFAIK.
[1] <a class="moz-txt-link-freetext" href="https://github.com/PanierAvide/panieravide.github.io/issues/1">https://github.com/PanierAvide/panieravide.github.io/issues/1</a>
[2] <a class="moz-txt-link-freetext" href="https://wiki.openstreetmap.org/wiki/Key:opening_hours/specification#separator_for_readability">https://wiki.openstreetmap.org/wiki/Key:opening_hours/specification#separator_for_readability</a>
[3] <a class="moz-txt-link-freetext" href="https://wiki.openstreetmap.org/wiki/Proposed_features/Time_domains">https://wiki.openstreetmap.org/wiki/Proposed_features/Time_domains</a>
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Tagging mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Tagging@openstreetmap.org">Tagging@openstreetmap.org</a>
<a class="moz-txt-link-freetext" href="https://lists.openstreetmap.org/listinfo/tagging">https://lists.openstreetmap.org/listinfo/tagging</a>
</pre>
</blockquote>
<br>
</body>
</html>