Modify

Opened 22 months ago

Last modified 10 months ago

#22677 needinfo enhancement

[PATCH][RFC] Give hints as to what tags are being matched (was remove man_made=communications_tower)

Reported by: SherbetS Owned by: SherbetS
Priority: normal Milestone:
Component: Internal preset Version: tested
Keywords: communications_tower Cc:

Description

Hello.

I am proposing to remove the preset for the tag: man_made=communications_tower.

My reasoning is that there is a extremely high likelihood that there are not any objects in the world that are "a far visible landmark. An indication could be a height greater than 100 meters" that have not been added to the map already.

Preserving this tag in the JOSM presets is just asking for mappers to use it incorrectly, tagging objects that should be man_made=mast + tower:type=communication due to the nature of the english language.

I've spent several hours trying to clean up after mappers using this tag correctly, and I'm tired of having to fight the fast increasing rate at which objects are being tagged with this tag.

see: https://taginfo.openstreetmap.org/tags/man_made=communications_tower#chronology

The iD editor has already removed this tag from their presets for this reasoning.

Thanks,
--SherbetS

Attachments (9)

searchpresets.png (14.3 KB ) - added by Hufkratzer 22 months ago.
shows search result
searchpresets_tags.png (22.2 KB ) - added by taylor.smock 19 months ago.
Show tag matches
22677.patch (15.7 KB ) - added by taylor.smock 19 months ago.
Indicate what tags from a preset are causing the preset to be found when using the Search presets window
spanish.jpg (28.0 KB ) - added by Hufkratzer 19 months ago.
shows search result for "polideportivo" in Spanish presets
spanish2.jpg (35.3 KB ) - added by Hufkratzer 19 months ago.
another attempt with Spanish presets
22677.1.patch (10.9 KB ) - added by taylor.smock 10 months ago.
Update to current JOSM source
language_de.2.JPG (28.6 KB ) - added by GerdP 10 months ago.
--language=de
language_en.JPG (30.9 KB ) - added by GerdP 10 months ago.
--language=en
22677.2.patch (11.3 KB ) - added by taylor.smock 10 months ago.
Handle check groups

Download all attachments as: .zip

Change History (48)

comment:1 by skyper, 22 months ago

Keywords: communications_tower added
Owner: changed from team to SherbetS
Status: newneedinfo
Type: defectenhancement

Could you, please, share some example changesets created with JOSM where the tag was used incorrectly? Thanks. Did you comment on the changesets? Was the internal preset used?

We already named the preset Big Communication Tower and have additional text about the usual height of more than 100m plus links to the other presets. The name and the text is translatable therefore I wonder how the tags can be mixed up.

Why should we remove a well established tag? In addition the presets also offers several secondary tags for more details. This option would be lost. In my eyes this ticket is invalid.

comment:2 by SherbetS, 22 months ago

Sure. I can link some changesets.

https://osmcha.org/changesets/127234022/

https://osmcha.org/changesets/125200147/

https://osmcha.org/changesets/122885679/

https://osmcha.org/changesets/122229950/

North America has only one communication tower from what i’ve seen, but hundreds are added by users who don’t know the tagging scheme.

https://osmcha.org/changesets/108821814/

https://osmcha.org/changesets/115314437/

https://osmcha.org/changesets/81234420/

https://osmcha.org/changesets/72719984/

there are hundreds upon hundreds of changesets I can link all day of mappers using the tag incorrectly. The guidance in JOSM isn’t enough. Editors look for communication infrastructure, and pick the first preset that has communication in the name. Like I said before, the nature of this object means that the likelihood of one not presently being on the map is astronomically unlikely.

having the preset still in JOSM only is resulting in damage to OSM because mappers will continue to mistag objects. I’ve changed over a thousand of these objects to the correct tagging, but they’re still growing rapidly. https://taginfo.openstreetmap.org/tags/man_made=communications_tower#chronology

comment:3 by skyper, 22 months ago

Thanks, I get your point, so we need to add a better guidance.
Looking at the wiki page Tag:man_made%3Dcommunications_tower it definitely needs an update and translation in more languages. The sentence "Another indication is, that a man_made=communications_tower has stairs and a lift inside, whereas as man_made=tower, tower:type=communication has to be climbed on the outside." is completely misleading and should be removed. Maybe even a warning box that almost all communications towers are likely to be mapped already and if in doubt man_made=tower is the better choice is useful.

Sorry, I am probably not the best person to judge, why people have that many problems as I am familiar to man_made=communications_tower from my childhood on but I am still against removing this preset.

comment:4 by Klumbumbus, 22 months ago

I also don't like the idea of removing the preset. IMO the not clear tagging schema of man_made=communications_tower + man_made=tower (+man_made=mast) is the problem. Maybe a proposal to deprecate man_made=communications_tower in favor of man_made=tower + tower_type=... would be the better way to go.

comment:5 by SherbetS, 22 months ago

I think that man_made=communications_tower still has it's place, since it's a whole local monument thing in Europe from what I can tell. But I don't see why the preset has to be in the base preset set and not like an add-in preset because it's enabling even the most experienced mappers to use the wrong tag to describe regular masts and towers. I agree that the wiki pages do need work though.

There are tens of actual communications towers, but hundreds of thousands of towers and masts in the world.

comment:6 by skyper, 22 months ago

According to wikipedia these towers are called "telecommunications tower" but that is still ambiguous and I miss a page like the Spanish one, https://es.wikipedia.org/wiki/Torre_de_telecomunicaciones_(desambiguaci%C3%B3n).

I have commented on some CSs but still wait for some answers.

We could add another sentence to the preset but I still fear that won't help much as people seem to do not read carefully enough and to do not look at the other linked presets.

Last edited 22 months ago by skyper (previous) (diff)

comment:7 by Palolo, 22 months ago

This is an interesting discussion, I mostly work remotely in Africa and so cannot always see the details needed. I do know that while in the developed world almost all of these towers have been added to OSM, but in other places there are very prominent landmarks/towers/masts that have yet to be added. I find this tagging to be rather silly and outdated since aside from waterway= and natural= the remaining 99% of OSM feature were created by humans (women also make things). Maybe a more appropriate choice would be tower=communication, tower_type=mast, etc. I know this seems like a big effort, but just because a small group of people in Manchester or Amsterdam made some decisions many years ago, it doesn't mean that we have to continue doing the wrong thing.
It's fine to remove the preset, but that doesn't solve the problem of poor language.

Last edited 22 months ago by Palolo (previous) (diff)

comment:8 by skyper, 22 months ago

I have updated the external preset Mast and tower. Would a similar naming and individual presets per type solve the problem?

comment:9 by Woazboat, 22 months ago

People tagging a feature incorrectly isn't a good reason to remove a preset, it's a reason to improve the wording and description of the preset (if that's indeed the reason for the mistagging in the first place). A majority of objects (or even all of them) already being tagged is also no reason to remove a preset.

in reply to:  7 comment:10 by Woazboat, 22 months ago

Replying to Palolo:

I find this tagging to be rather silly and outdated since aside from waterway= and natural= the remaining 99% of OSM feature were created by humans (women also make things).
I know this seems like a big effort, but just because a small group of people in Manchester or Amsterdam made some decisions many years ago, it doesn't mean that we have to continue doing the wrong thing.
It's fine to remove the preset, but that doesn't solve the problem of poor language.

I assume you mean the use of man_made? Yes, the wording is unfortunate and a different term would be better. Changing it is not as easy as simply changing all man_made tags in the OSM database. That would be the trivial part. You'd have to change all software and data processing tools that use the affected tags as well as all documentation and that is pretty much impossible. "A big effort" is a bit of an understatement.
(I assume you are aware that the man in man_made does not and was never intended to refer to just male humans but to humanity in general as in 'mankind'. Use of the term has fallen out of favor in exchange for more overtly neutral terms. Avoiding it for new uses... good idea. Forcefully changing it with enormous effort and breaking lots of things along the way... maybe not.)

The JOSM issue tracker for a completely unrelated topic isn't the right place to discuss this however.

comment:11 by stoecker, 22 months ago

The JOSM issue tracker for a completely unrelated topic isn't the right place to discuss this however.

And in case you're not fluent in English please note that man in man_made comes from man (as in human, mankind, ...) and is translated to e.g. German as "Mensch" and not from man (as in the male human) translated to German as "Mann".

So it's not "poor language", but rather "poor understanding" of the language.

in reply to:  11 comment:12 by Palolo, 22 months ago

Replying to stoecker:

The JOSM issue tracker for a completely unrelated topic isn't the right place to discuss this however.

And in case you're not fluent in English please note that man in man_made comes from man (as in human, mankind, ...) and is translated to e.g. German as "Mensch" and not from man (as in the male human) translated to German as "Mann".

So it's not "poor language", but rather "poor understanding" of the language.

I have a fairly good understanding of the English language for many decades, but there is always much to learn. I live in the United States and we long ago started abandoning poor language usage like this. My point would even be true if the tag was people_made. It is really too bad that the "Open" part of OSM is not universal and doesn't apply to the system. stoecker If you would like to fix the incorrect tags in east Africa please feel free to do so. There are vast populations of this planet that have not map whatsoever. I very much appreciate all the work that people have done to build and maintain JOSM, it is of tremendous value.

in reply to:  2 comment:13 by Hufkratzer, 22 months ago

Replying to SherbetS:

The guidance in JOSM isn’t enough. Editors look for communication infrastructure, and pick the first preset that has communication in the name.

I assume this is the main problem.

If in JOSM I search for "communication" I currently get this list:
shows search result

If JOSM could look up how often these tags/tag combinations are used (e.g. in taginfo) and would sort the list by these numbers in descending order, the list would look like this:

NumbersTag/Tag combinationDisplay in search dialog
178.357man_made=tower + tower:type=communicationMan Made/Towers/Tower + Type communication
156.708man_made=mast + tower:type=communicationMan Made/Towers/Mast + Type communication
33.634office=telecommunicationOffices/Telecommunication
3.868man_made=communications_towerMan Made/Towers/Big Communication Tower

If the user picks the first one from this list, it is more likely than at present that this choice is not wrong.

This method probably does not always work, but maybe it can be improved somehow.

by Hufkratzer, 22 months ago

Attachment: searchpresets.png added

shows search result

comment:14 by stoecker, 22 months ago

The order is based on the presets file. Simply reordering the file manually would reach your goal.

comment:15 by Woazboat, 22 months ago

Notably the other preset options don't even have communication in the displayed title. That's probably the main issue

in reply to:  14 comment:16 by anonymous, 22 months ago

Replying to stoecker:

The order is based on the presets file. Simply reordering the file manually would reach your goal.

The current order in defaultpresets.xml is

  1. Mast line 6144 ff
  2. Tower line 6168 ff
  3. Big Communication Tower line 6202 ff

But (AFAIK) the presets that have the search string in their own name are always listed before the presets that have it only in some combo box or multiselect list with parameter

values_searchable="true"

Check/uncheck the checkbox Search in tags to see the two different groups.

in reply to:  8 comment:17 by skyper, 22 months ago

@all:
Please, try the external preset Mast and tower. It should solve most of the mentioned problems and an adaption would be quite easy.

Regarding finding the correct tags the OSM wiki pages would benefit from some rework and updates as I found same strange changes going back til 2018. Probably better photos of the range of each tag and better description of the differences (also from the engineering perspective) would help.

comment:18 by skyper, 21 months ago

Ok, all useful answers on CSs would appreciate finding Communication tower and Communication mast in preset's name. I could come up with a patch which adds own presets per tower:type similar to my external preset if this will be accepted.

comment:19 by SherbetS, 21 months ago

Yes. the main issue with this preset is that it has communications in it's name, which does a lot to guide users away from the correct tagging. I don't see any harm in just removing this preset from the ones that come with JOSM, and instead including it in a preset pack that the user can enable to work with these objects instead of continuing to have users continuing to get confused over the tagging scheme.

aplogies for the late response, I don't care to give this website my email address.

in reply to:  description ; comment:20 by Hufkratzer, 21 months ago

Replying to SherbetS:

The iD editor has already removed this tag from their presets for this reasoning.

Where is this discussion?

There is a related open issue for iD presets:
https://github.com/openstreetmap/id-tagging-schema/issues/92

in reply to:  20 comment:21 by anonymous, 20 months ago

Where is this discussion?

I don't think there was a discussion, but I figure someone at iD team would probably present similar reasoning to this issue.

comment:22 by SherbetS, 19 months ago

How do I get this removed?

All of us know these features are already mapped. I hate having to waste my time fighting this.

https://taginfo.openstreetmap.org/tags/man_made%3Dcommunications_tower#chronology

—SherbetS

by taylor.smock, 19 months ago

Attachment: searchpresets_tags.png added

Show tag matches

in reply to:  22 comment:23 by taylor.smock, 19 months ago

Replying to SherbetS:

How do I get this removed?

Short answer: you probably don't. The preset also provides rendering support for the already mapped communication towers.

Anyway, it would probably be a better idea to start showing the tag matches, like so:

Show tag matches

by taylor.smock, 19 months ago

Attachment: 22677.patch added

Indicate what tags from a preset are causing the preset to be found when using the Search presets window

comment:24 by taylor.smock, 19 months ago

Summary: remove man_made=communications_tower[PATCH][RFC] Give hints as to what tags are being matched (was remove man_made=communications_tower)

I don't know if showing the matching tags will help reduce the introduction of new man_made=tower, but I think it might help at least.

comment:25 by Hufkratzer, 19 months ago

As explained in comment:13, it would also be helpful if the more commonly used presets Mast and Tower were listed before the Big Communication Tower preset.

I thought that the presets that contain the search string would always be listed before the others (comment:16). But today I came across a case where this is not true:
shows search result for "polideportivo" in Spanish presets

I don't understand why this is happening here and not in other cases; but maybe someone who understands it can explain how this can be used to move the Mast and Tower presets up in the search results.

by Hufkratzer, 19 months ago

Attachment: spanish.jpg added

shows search result for "polideportivo" in Spanish presets

in reply to:  25 comment:26 by taylor.smock, 19 months ago

The sort order of that list depends upon several factors.

Here are the weightings:

  • Last used (CLASSIFICATION_IN_FAVORITES): 300
  • Name match (CLASSIFICATION_NAME_MATCH): 300
  • Group match (CLASSIFICATION_GROUP_MATCH): 200
  • Tags match (CLASSIFICATION_TAGS_MATCH): 100

For name, group, and tag matches, each word in the search term is compared. If it starts with the search term, 2 is added to the classification (so Polideportivo would be CLASSIFICATION_NAME_MATCH + 2 for a total of 302).

I don't know what is going on with Natacion, but I'd guess you have previously tagged something with it and it has a tag with polideportivo since it that search term occurs neither in the name (Natacion) nor the group (Deportes/Deporte). So it probably has a weighting of CLASSIFICATION_IN_FAVORITES + CLASSIFICATION_TAGS_MATCH + <some much smaller number> for something 400+.

comment:27 by Hufkratzer, 19 months ago

Probably right. I tried it on another computer an got
another attempt with Spanish presets

by Hufkratzer, 19 months ago

Attachment: spanish2.jpg added

another attempt with Spanish presets

comment:28 by Hufkratzer, 19 months ago

I wonder why there are so many of these man_made=communications_tower mapped in Germany (see taginfo map) although it was translated "Großer Kommunikationsturm" and I wouldn't call such a tower like that, rather I would search for "Funkturm" (compare DE:Tag:man_made=tower), and the search for "Funkturm" only finds the man_made=tower and man_made=mast presets.

Last edited 19 months ago by Hufkratzer (previous) (diff)

comment:29 by GerdP, 10 months ago

@Taylor: This issue poped up again here https://community.openstreetmap.org/t/tag-funkturm-funkmast/108599
I wanted to try the patch but it doesn't apply cleanly. Please can you link an updated version?

by taylor.smock, 10 months ago

Attachment: 22677.1.patch added

Update to current JOSM source

by GerdP, 10 months ago

Attachment: language_de.2.JPG added

--language=de

by GerdP, 10 months ago

Attachment: language_en.JPG added

--language=en

comment:30 by GerdP, 10 months ago

Thanks. I first thought it doesn't work at all, but then I noticed that I had the checkbox for "Search in tags" disabled.
One problem remains:
Why do I see different results for German and English language?
--language=de --language=en

Is the search only performed on the translated tags?

comment:31 by taylor.smock, 10 months ago

It looks like it is a partial search on translated tags.

It looks like the two missing ones are Man Made/Towers/{Mast|Tower}. This is probably from cms.getDisplayValues() (L313). If we change it to cms.getValues(), I think it will work, but we probably want to do both.

comment:32 by GerdP, 10 months ago

Yes, an additional line with

   list.addAll(cms.getValues());

seems to improve the results. Sort order is still different.
Is it intended that nothing is selected and the Select button is enabled? It seems to select the topmost entry.

comment:33 by taylor.smock, 10 months ago

From comment:26 (so I don't have to look at how order is calculated again):

  • Community center (en): CLASSIFICATION_NAME_MATCH + CLASSIFICATION_TAGS_MATCH
  • Community center (de): CLASSIFICATION_TAGS_MATCH

Is it intended that nothing is selected and the Select button is enabled? It seems to select the topmost entry.

I think this is the current behavior. And something that I kind of expect, although maybe we should be selecting the topmost entry or otherwise indicating that it is what will be selected when the user hits Select.

comment:34 by GerdP, 10 months ago

It would probably be bad to disable the button until something is selected. With the typical background colour blue for selected item the text is much harder to read. Probably the current behaviour is a good compromise.

Another possible problem:
When I search for "bridge" while a highway is selected I would expect to find presets for highway and railway and maybe others where bridge=yes could be applied, but I don't.

by taylor.smock, 10 months ago

Attachment: 22677.2.patch added

Handle check groups

comment:35 by GerdP, 10 months ago

OK, that works, but now the order really matters. With a highway=track selected I would want to find the preset Highays/Ways/Track much higher.

I rarely use presets, so maybe I am not the best candidate to suggest improvements.
I've normally disabled the preference properties.presets.visible, but for comparison I've enabled it. The selection of presets there is also confusing. When I have a way with two tags highway=track and bridge=yes the preset "Man made/Bridges/Bridge" is shown in addition to the "Highays/Ways/Track". Why?

comment:36 by skyper, 10 months ago

"Man made/Bridges/Bridge" is shown as bridge=yes is present and while the "Highways/Ways/Track" only has a checkbox the "Man made/Bridges/Bridge" allows to add other values plus lot more tags for bridges.

comment:37 by GerdP, 10 months ago

But a (unclosed) highway=track cannot also be a man_made=bridge. I may want to add a closed way with man_made=bride but it is very unlikely that I want to change the highway=* object to a man_made=bridge object.

comment:38 by skyper, 10 months ago

The preset for man_made=bridge is "Man made/Bridges/Bridge outline".

comment:39 by GerdP, 10 months ago

A OK, didn't try what happens when I select the preset. Probably just a thing one has to learn to be able to work with presets.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as needinfo The owner will remain SherbetS.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from SherbetS to the specified user. Next status will be 'new'.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from SherbetS to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.