Opened 12 years ago

Last modified 5 years ago

#9400 new enhancement

validate source:maxspeed — at Version 5

Reported by: mkoniecz Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: maxspeed Cc: gaben

Description (last modified by mkoniecz)

source:maxspeed=PL:living_street without maxspeed key or with any maxspeed value except 20 can be reported as "mismatch between source:maxspeed and maxspeed, for source:maxspeed=PL:living_street 20 is expected value of maxspeed"

report source:maxspeed=PL:living_street on any other object other than way with highway=living_street: "Living street should be tagged as highway=living_street and source:maxspeed=PL:living_street should not be placed on ways that are not a living street"

Change History (5)

comment:1 by skyper, 12 years ago

Description: modified (diff)

in reply to:  description comment:2 by Don-vip, 12 years ago

Summary: validate source:maxspeed=PL:living_streetvalidate source:maxspeed

Replying to Bulwersator:

source:maxspeed=PL:living_street without maxspeed key or with any maxspeed value except 20 can be reported as "mismatch between source:maxspeed and maxspeed, for source:maxspeed=PL:living_street 20 is expected value of maxspeed"

OK. I believe you, but is there an official website we could link to in documentation as well ? The limits in all countries are not eternal, it's good to check the official texts from time to time :)

report source:maxspeed=PL:living_street on any other object other than way with highway=living_street: "Living street should be tagged as highway=living_street"

OK to report a problem, but not with this text. It could also be the source:maxspeed value which is wrong.

highway=living_street with maxspeed=20, without source:maxspeed=PL:living_street: "For marking source of speed limit 20 as "Strefa zamieszkania" you may use source:maxspeed=PL:living_street"

No. Poland is not the whole world :p Think of other countries, I am sure we can find another one where living street are limited to 20 km/h as well :) We could however raise an informational message "maxspeed without source:maxspeed". It's up to the user to give the good value.

We should also make sure to validate other countries. Popular ones (at least 500 use) for this key are:

  • DE
  • PL
  • IT
  • AT
  • FR
  • DK
  • UK
  • CZ
  • BE
  • NL
  • HU
  • AU
  • LT
  • FI

But we should also look at other major OSM countries (US, RU, JA...)

comment:3 by mkoniecz, 12 years ago

Description: modified (diff)

comment:4 by mkoniecz, 12 years ago

http://en.wikipedia.org/wiki/Living_street#Around_the_world may be a good link, direct link to law is nearly impossible (171 pages of pdf in Polish, link http://isip.sejm.gov.pl/DetailsServlet?id=WDU19970980602 is likely to end dead once law is changed).

"Poland is not the whole world" - ops, it was stupid. I removed this case.

"We should also make sure to validate other countries." - I know, it is a good idea but I prefer to limit myself to area where my knowledge is not based solely on Wikipedia and googling. This probably may be added as a separate tickets (it is necessary to add all of this in one commit).

"OK to report a problem, but not with this text. It could also be the source:maxspeed value which is wrong." - good idea.

comment:5 by mkoniecz, 12 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.