Attachments (1)
Change History (7)
by , 11 years ago
Attachment: | patch_8962_1.diff added |
---|
comment:1 by , 11 years ago
It looks like chain_link was added to fence_type in data/defaultpresets.xml but in commented out form (r5941).
Taginfo shows that fence_type=chain is still popular (11k), but barrier=chain (2k) and fence_type=chain_link (1k) are definitely being used.
Hedge and chain are full-fledged barrier types and should probably not be in this list.
I have attached a diff that tries to merge the state of the wiki and taginfo for consideration.
wiki: barbed_wire, chain_link, electric, railing, roundpole, split_rail, wood, wire, metal, pole
taginfo: chain, wood, split_rail, barbed_wire, pole, wire, hedge, chain_link, metal...
combined: barbed_wire, chain_link, electric, metal, pole, split_rail, wire, wood
comment:2 by , 11 years ago
Summary: | fence_type=chain should be changed to "chain_link" → [PATCH] fence_type=chain should be changed to "chain_link" |
---|
comment:4 by , 11 years ago
So why do we not support all the values from the wiki? E.g. in our city suburb, most fences I need to tag are 'railing'...
comment:5 by , 11 years ago
We like to see tags being used a bit before integrating them into internal presets. How often is it used ?
comment:6 by , 11 years ago
About 140 times (counting all typos) out of 41 654.
http://taginfo.openstreetmap.org/keys/?key=fence_type#values
I wonder why that is as no other type describes that fence type well. Maybe people just don't set any fence_type in that case. There are 834 364 uses of barrier=fence, so that means people do not add the fence_type in most cases. The high number of wood and split_rail indicates that mostly fences in forests/villages were tagged with the proper type. I think adding the more "city" types would promote the use of the fence_type tag.
Proposed diff.