Opened 8 years ago
Closed 7 years ago
#13594 closed enhancement (fixed)
[PATCH] update landuse=industrial: add industrial=* combo
Reported by: | anonymous | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 17.08 |
Component: | Internal preset | Version: | |
Keywords: | landuse industrial | Cc: |
Description
This patch extends default preset for
landuse=industrial
by combo
industrial=*
as defined on wiki: http://wiki.openstreetmap.org/wiki/Key:industrial
Attachments (3)
Change History (24)
by , 8 years ago
Attachment: | defaultpreset.xml.patch added |
---|
comment:1 by , 8 years ago
Summary: | Default preset: add industrial=* combo to landuse=industrial → [PATCH] Default preset: add industrial=* combo to landuse=industrial |
---|
comment:2 by , 8 years ago
I do not like including both "wellsite" and "well_cluster". IMHO it means the same thing. I would choose only one, probably "wellsite".
comment:3 by , 8 years ago
I took these values from wiki.
Actually, according to taginfo (http://taginfo.openstreetmap.org/keys/industrial#values):
wellsite 1 920 13.81% well_cluster 1 179 8.48%
by , 7 years ago
Attachment: | defaultpresets.xml.patch added |
---|
Add combo key="industrial" to landuse=industrial
follow-up: 5 comment:4 by , 7 years ago
As per comment on wiki, wellsite is more general and well_cluster can contain more wellsites. I've updated the patch - combo values are sorted and spaces were removed.
ps: how can I remove the previous, incorrecly named attachement defaultpreset.xml.patch? Should be presets.
follow-ups: 6 7 comment:5 by , 7 years ago
I'd prefer to have the values get more accepted and better documented first. For example, the wiki page for industrial=well_cluster does not contain any sentences at all nor is there any differences between wellsite
and well_cluster
documented. timber
is quite common in use but not documented and the difference to sawmill
is unclear.
Can't we start with an individual preset and include it once we have better documentation (especially outlining the differences) and higher numbers of use.
Replying to Mkyral:
ps: how can I remove the previous, incorrecly named attachement defaultpreset.xml.patch? Should be presets.
If you login, you should be able to delete your attachments.
follow-up: 8 comment:6 by , 7 years ago
Well, I'm not the right person to document it. I'm not a native speaker and I don't know differences between these options :-(
I just want to see industrial=brewery in preset. This should help to increase numbers of use.
Maybe we should keep only some basic set and let rest to be added in future, when all will be clear.
Replying to Mkyral:
ps: how can I remove the previous, incorrecly named attachement defaultpreset.xml.patch? Should be presets.
If you login, you should be able to delete your attachments.
I'm logged in, but there is no option nor button to delete attachment. I can only add a new one (and replace one with the same name).
comment:7 by , 7 years ago
Replying to skyper:
Can't we start with an individual preset and include it once we have better documentation (especially outlining the differences) and higher numbers of use.
Well, it seems to be the only possibility.
Here we are: https://josm.openstreetmap.de/wiki/Presets/Industrial
comment:8 by , 7 years ago
Replying to Mkyral:
Replying to Mkyral:
ps: how can I remove the previous, incorrecly named attachement defaultpreset.xml.patch? Should be presets.
If you login, you should be able to delete your attachments.
I'm logged in, but there is no option nor button to delete attachment. I can only add a new one (and replace one with the same name).
You can't delete it becasue you need special rights to delete files at the josm website/wiki.
comment:9 by , 7 years ago
Keywords: | landuse industrial added |
---|---|
Summary: | [PATCH] Default preset: add industrial=* combo to landuse=industrial → [PATCH] update landuse=industrial: add industrial=* combo |
comment:10 by , 7 years ago
industrial=*
seems worth including in the preset, maybe leave out the values that are redundant or poorly documented?
by , 7 years ago
Attachment: | defaultpresets.xml.industrial.v3.diff added |
---|
Add industrial combo, third version
comment:12 by , 7 years ago
OK. Third version. I've removed "gas", "well_site", "wellcluster" as undocumented even there are frequently used. I also removed "furniture" and "ice_factory" as both are included in "factory" option.
comment:14 by , 7 years ago
Milestone: | → 17.07 |
---|---|
Version: | latest |
Beside the documentation in the wiki the tags should have a decent usage number (see DevelopersGuide/DefaultPresets). I just checked the first one on taginfo: industrial=aluminium_smelting. It is used only twice. Basically I agree to include industrial=* in the default preset, but not the ones with low usage numbers, I would say <100.
comment:15 by , 7 years ago
Isn't it chicken egg problem? In such case also bakery, shipyard, mine, etc would be omitted. And I don't think there is only 80 big bakeries worldwide.
comment:16 by , 7 years ago
You can tag it regardless of wheather there is a JOSM preset for this tag or not. But we don't need a preset if the users have no interest to tag it.
comment:17 by , 7 years ago
Yes I can tag it anyway. But I can tag it now as well and users are not doing it. Maybe because they don't know about it or because they don't want spent a too much time with it. If there no right option in preset, user have to search on wiki and maybe he have to use dictionary first, because he don't know the correct english word.
In many cases, user just skip this "additional" tagging to save time and map more.
From myself experience - if preset contains some additional options, I'll check them and try to fill them. But if this option is missing, I'll add additional tags only when I really want it.
comment:18 by , 7 years ago
It may seem silly in this case, but there are users inventing new tagging schemes and trying to get it into the editors before it is accepted by the community. To avoid controversial and later rejected tagging schemes in JOSM, we need to apply a common standard to all requests.
comment:19 by , 7 years ago
Key:industrial
Status: in use
I have nothing more to say.
If you want, you can eliminate more options (and later on add one by one back). But I'll not do it, I think it is useless.
Add industial=* key to landuse=industrial