#17199 closed defect (fixed)
Validator complains about "width" tags
| Reported by: | mueschel | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Milestone: | 19.01 |
| Component: | Core validator | Version: | tested |
| Keywords: | Cc: | GerdP |
Description
I recently (currently using 14620) see many warnings about the value of 'width' like
Value '12.8' for key 'width' looks like one of [1.5, 2.5]. (1) Wert '4.3' für Schlüssel 'width' sieht aus wie '4.5'. (1)
These values are perfectly fine and the validator should not complain about any numeric value for keys like 'width', 'height', 'length'... The only exception might be values with more decimals than reasonably possible (e.g. 2.23473189031) like they happen when importing float values.
Attachments (0)
Change History (10)
comment:1 by , 7 years ago
| Cc: | added |
|---|---|
| Milestone: | → 19.01 |
| Type: | enhancement → defect |
comment:2 by , 7 years ago
Hmm, seems that there is an external preset with those values as default. It doesn't happen with the default presets.
I guess I have to add these tags to an exclude list.
comment:4 by , 7 years ago
It looks like this is a more general problem, not limited to width and length. Some Examples:
Wert '1985' für Schlüssel 'start_date' sieht aus wie '1895'. (4)
-> I'm not aware 1895 was a particularly interesting year.
Wert 'К-150' für Schlüssel 'fire_hydrant:diameter' sieht aus wie '150'.
-> K-150 is a very common value used in Russia
comment:5 by , 7 years ago
Yes, I want to change the code to read common values from a list provided by taginfo.
In the meantime I might change the code to ignore vales when preset values are all numerical.
BTW: I'd like to know where this value 1895 comes from. It is not in the default presets.
comment:6 by , 7 years ago
1895 is one of the examples in the "Historical Objects" preset.
https://josm.openstreetmap.de/josmfile?page=Presets/Historical_Objects
Tagging presets: + http://zibi.openstreetmap.org.pl/kendzi/k/Simple3dPreset/current/roofspresets.xml + https://josm.openstreetmap.de/josmfile?page=Presets/LaneAttributes&preset&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/NewTags&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/PublicTransportOneClick&preset&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/OneClick&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/LaneAttributes&preset&zip=1 + http://www.country-linedance.de/daten/Verkehrszeichen-vorlage.zip + https://josm.openstreetmap.de/josmfile?page=Presets/hiking_routes_with_trail_marking&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/ParkingLanes&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/Crafts&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/Historical_Objects&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/Light_sources&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/NewParkingFeatures&zip=1 + https://josm.openstreetmap.de/josmfile?page=Presets/Towers&zip=1 + https://raw.githubusercontent.com/yopaseopor/traffic_signs_preset_JOSM/master/DE.zip + https://josm.openstreetmap.de/josmfile?page=Presets/Maxspeed-zones&zip=1
comment:7 by , 7 years ago
OK, thanks, I already wondered if there is an error in the program logic that extracts the default values...
comment:8 by , 7 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:9 by , 7 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
In 14696/josm:
fix #17199
- Don't suggest different spelling when all preset values for a key are numerical
- Add layer to ignore list, is checked in MapCSSTagChecker
- Add start_date end end_date to ignore list
- apply modified version of 17055-use-ignore.patch:
- the lines from ignoretags.cfg starting with K: are treated as "often used".
- instead of "... is not in the presets ..." use "... is unknown ..." in messages
comment:10 by , 7 years ago
Reg. fire_hydrant:diameter=K-150:
This is a bit special. The default presets say that the value should specify the "Diameter (mm)".
The wiki says: "# (numeric value) in mm, inches or letters. ".
Taginfo shows more values starting with K-
I assume that firefighters know what it means?



Related to the changes from #17055.