Opened 7 years ago

Last modified 6 years ago

#17629 closed enhancement

complain about more descriptive names and propose to delete them [PATCH] — at Version 8

Reported by: mkoniecz Owned by: team
Priority: normal Milestone: 19.04
Component: Core validator Version:
Keywords: template_report Cc: stoecker

Description (last modified by mkoniecz)

Patch for the first batch in https://josm.openstreetmap.de/ticket/17629#comment:7

What steps will reproduce the problem?

  1. Create amenity=restaurant name=Restaurant
  2. Create amenity=place_of_worship religion=muslim name=Mosque
  3. Create amenity=toilets name=Toilet
  4. Run validator

What is the expected result?

Validator complains about descriptive names and offers to remove them

What happens instead?

Nothing

Please provide any additional information below. Attach a screenshot if possible.

​http://overpass-turbo.eu/s/I7p ​http://overpass-turbo.eu/s/I7r ​http://overpass-turbo.eu/s/I7t

Next part of https://josm.openstreetmap.de/ticket/17100

To be added in https://josm.openstreetmap.de/browser/josm/trunk/data/validator/unnecessary.mapcss#L93

Spotted in ​https://github.com/osmlab/name-suggestion-index/blob/master/dist/names_discard.json (note that this file list both generic and nonbrand names, not all of the should be purged)

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-04-03 00:33:43 +0200 (Wed, 03 Apr 2019)
Build-Date:2019-04-03 01:30:50
Revision:14954
Relative:URL: ^/trunk

Identification: JOSM/1.5 (14954 en) Linux Ubuntu 16.04.6 LTS
Memory Usage: 435 MB / 869 MB (102 MB allocated, but free)
Java version: 1.8.0_201-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (34867)
+ buildings_tools (34904)
+ continuosDownload (82)
+ imagery_offset_db (34867)
+ measurement (34867)
+ reverter (34961)
+ todo (30306)

Validator rules:
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/deprecated.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss

Last errors/warnings:
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet

Change History (8)

comment:1 by mkoniecz, 7 years ago

Summary: complain about more descriptive names → complain about more descriptive names and propose to delete them

comment:2 by mkoniecz, 7 years ago

Description: modified (diff)

comment:3 by mkoniecz, 7 years ago

BTW, I see also amenity=place_of_worship name=Church, Cami that seems to mean "mosque" in Turkish and used 1500+ in Turkey as a descriptive name - see ​http://overpass-turbo.eu/s/I7u

1500 Biserica meaning "church" in Romanian used for amenity=place_of_worship

3000+ amenity=pharmacy name=Аптека

comment:4 by Don-vip, 7 years ago

List of suspicious names used more than 2000 times to check:

comment:5 by mkoniecz, 7 years ago

​Cmentarz parafialny: 2479 "Parish cemeteries" in Polish?

This is a borderline thing. In some places this is a valid name (in places where there are multiple cemeteries), is some places it is on border of description and actual name, in some places it is a descriptive name.

Kwiatowa, Spacerowa, Zielona, Sportowa, Wiejska, Sosnowa, Wiejska, Łąkowa, Parkowa, Kolejowa, Ogrodowa, Kościelna are all popular street names in Poland. These names are not going to be used as decriptive names due to a grammar form. I am pretty sure that after excluding highway=* usage will drop to nearly 0.

Rynek is for market square, and many of them have it now as the official street name.

name=building is already caught on building=* - see https://josm.openstreetmap.de/browser/josm/trunk/data/validator/unnecessary.mapcss#L97
The same with house, rumah, jalan

Updated lists:

List of suspicious names used more than 2000 times to check:

ones not listed in the list above:

  • amenity=place_of_worship name=Church
  • Cami that seems to mean "mosque" in Turkish and used 1500+ in Turkey as a descriptive name - see ​​http://overpass-turbo.eu/s/I7u
  • 1500 Biserica meaning "church" in Romanian used for amenity=place_of_worship
  • amenity=restaurant name=Restaurant
  • amenity=place_of_worship religion=muslim name=Mosque
  • amenity=toilets name=Toilet
Last edited 7 years ago by mkoniecz (previous) (diff)

comment:6 by mkoniecz, 7 years ago

List of suspicious names used more than 2000 times to check:

And thank you very much for preparing this list!

comment:7 by mkoniecz, 7 years ago

​https://github.com/matkoniecz/josm/tree/descriptive has patch, currently at ​https://github.com/matkoniecz/josm/commit/48bf590c25a0e146921fd4306e4a4b9bfb8e13ce.patch

It removes entries that I initially reported and Pond, all only from objects where this name is both descriptive and provides no additional info.

Equivalent in iD - ​https://github.com/openstreetmap/iD/issues/6055#issuecomment-485131822

Last edited 7 years ago by mkoniecz (previous) (diff)

comment:8 by mkoniecz, 7 years ago

Description: modified (diff)
Summary: complain about more descriptive names and propose to delete them → complain about more descriptive names and propose to delete them [PATCH]
Note: See TracTickets for help on using tickets.