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

Josm-check don't find an specific (tricky) Error #54

Closed
dex2000 opened this issue Aug 29, 2014 · 3 comments
Closed

Josm-check don't find an specific (tricky) Error #54

dex2000 opened this issue Aug 29, 2014 · 3 comments

Comments

@dex2000
Copy link

dex2000 commented Aug 29, 2014

Dunno about im right here with that:

following way: https://www.openstreetmap.org/way/125170567 (not changed by me)
Mo-Fr 05:00-21:00; Sa;Su 07:00-21:00; PH 07:00-21:00

opening_hours (in the qat-script) found right warning, josm-check not.

German:
Keine Ahnung, ob ich hier richtig bin.
Gerade obiges mit dem opening_hours-qat-script reingeholt, ich mache zusätzlich eine Josm-prüfung. Diese gibt mir keine Warnung aus. Sollte aber meines Erachtens (obwohl die Regel formal korrekt ist)

@ypid
Copy link
Member

ypid commented Aug 29, 2014

The version of opening_hours.js bundled and used in JOSM is a few months old. The qat_script relies on the current development version (same as the evaluation tool which is also pretty much bleeding edge 😄 ).

The test you are referring to was added recently. See #49.

I plan to make a release (v3.0.0) in the next few days and will ask the JOSM maintainers to update there copy of it then.

Die Version von opening_hours.js, die von JOSM benutzt wird, ist ein paar Monate alt. Das qat_script benutzt hingegen die aktuelle Entwicklerversion (genau so wie das Auswertewerkzeug, welches ich ebenfalls recht oft aktualisiere).

Den Test, den du ansprichst, habe ich erst kürzlich hinzugefügt. Siehe #49.

Ich werde voraussichtlich in den nächsten Tagen eine neue Version (v3.0.0) veröffentlichen und werde daraufhin die JOSM Entwickler fragen, auf diese Version zu aktualisieren.

@ypid ypid closed this as completed Aug 29, 2014
@don-vip
Copy link

don-vip commented Aug 29, 2014

We'll update, don't worry :)

@ypid
Copy link
Member

ypid commented Aug 30, 2014

That‘s what I was hoping for ;)

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

No branches or pull requests

3 participants