Modify

Opened 2 years ago

Closed 15 months ago

Last modified 15 months ago

#23298 closed defect (fixed)

Improper warning for aerialway=zip_line (Issue #2068) [osm-fr/osmose-backend]

Reported by: hugo.amann@… Owned by: team
Priority: normal Milestone: 24.08
Component: Core validator Version:
Keywords: osmose Cc: Schröcker

Description (last modified by taylor.smock)

Hello,
Forwarding my OSMOSE Issue #2068:
(item/class 9001001
missing tag usage without railway or waterway)

In order to stop this warning, they should change

way[usage][!railway][!waterway][route!=railway][man_made!=pipeline] {

to

way[usage][!railway][!waterway][route!=railway][man_made!=pipeline][!aerialway] {

In addition, although the use of usage=* is documented on the wikis of Key:aerialway, Tag:aerialway=zip_line and several others for many years with the intention to deprecate the unspecific tag aerialway=goods, it is not documented on the wiki of Key:usage itself

Thanks!
Hugo Amann

Attachments (0)

Change History (7)

comment:1 by taylor.smock, 2 years ago

Component: CoreCore validator
Description: modified (diff)
Milestone: 24.06

Please update the osmwiki:Key:usage page. It looks like osmwiki:Tag:usage=industrial and osmwiki:Tag:usage=tourism are the two usage tags I would expect on aerialways. osmwiki:Tag:usage=freight seems to be a duplication of the meaning for osmwiki:Tag:usage=industrial.

I'll revisit this once the wiki pages have been stable for awhile.

comment:2 by gaben, 2 years ago

Keywords: osmose added

comment:3 by stoecker, 16 months ago

Milestone: 24.0624.07

Milestone closed.

comment:4 by taylor.smock, 15 months ago

Milestone: 24.0724.08

Ticket retargeted after milestone closed

comment:5 by taylor.smock, 15 months ago

Resolution: fixed
Status: newclosed

In 19165/josm:

Fix #23298: Improper warning for aerialway=zip_line

Exclude aerialway from combinations check and add usage values to aerialway
presets (where someone has already used the key with the aerialway tag).

comment:6 by SimonPoole, 15 months ago

This change seems a bit problematic.

I could get over that the combination of aerialway with usage is undocumented and only occurs ~200 times, but shouldn't it be a multiselect and include a value for normal aka non-touristic transportation of people?

comment:7 by taylor.smock, 15 months ago

You might want to double check the undocumented bit -- see osmwiki:Key:usage#Aerialways .

Of specific note for my purposes, it has been documented there for >6 months. Maybe I shouldn't have added it to the presets, but I figured I might as well if I was going to add an exception to the validator rules.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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