Modify

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#9248 closed defect (invalid)

Wrong warning: crossing waterways (canal)

Reported by: RicoZ Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: Cc: RicoZ

Description

simply can not make any sense of it:
Id 241851378
Id 93043061

Attachments (0)

Change History (8)

comment:1 by RicoZ, 11 years ago

Cc: RicoZ added
Component: CoreCore validator

comment:2 by Don-vip, 11 years ago

It looks like the level=-1 is not taken into account.

comment:3 by Don-vip, 11 years ago

Resolution: invalid
Status: newclosed

Of course not it should be layer=-1. It works as expected.

comment:4 by RicoZ, 11 years ago

what should be layer=-1? I have one way that is without layer and one way with layer=-1, so why this warning??

comment:5 by Don-vip, 11 years ago

http://www.openstreetmap.org/browse/way/241851378 is tagged with level=-1 instead of layer=-1
See:
http://wiki.openstreetmap.org/wiki/Key:level
http://wiki.openstreetmap.org/wiki/Key:layer

You're confusing these two keys; they have absolutely not the same meaning. Level is for buildings while layer is for general mapping.

comment:6 by RicoZ, 11 years ago

sorry, totally stupid mistake from me.

However - :

  • technically two ways do not cross if either level or layer doesn't match, so this warning is still wrong even if using layer would have been the right thing in this case. I have seen indoor waterways where tagging with level might be correct.
  • it would be good to warn confused users like me tagging waterways (or other random features?) with level because it makes sense pretty rarely.

comment:7 by Don-vip, 11 years ago

No problem :) Do you have any example of indoor waterway ? I'm a bit curious about that.

comment:8 by RicoZ, 11 years ago

indoor water parks, aquariums, indoor waterfalls with artificial creeks and pools in some large shopping centers. Depending whether or not the waterways are bound to specific levels of the building either layer or level may be more appropriate to use.

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.