Modify

Opened 3 months ago

Last modified 3 months ago

#22755 new defect

Inconsistent behavior of ∈ (building node part of building shape)

Reported by: Famlam Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: mapcss building inside Cc:

Description (last modified by skyper)

I spotted node 8793042989 where a (closed) building way contains a node with the same building tag, i.e. a node of the way equals the way itself.

Maybe it is interesting for JOSM to add the following rule to catch such cases?

way[building][building!=no]:closed > node[building][tag("building")==parent_tag("building")] {
  throwWarning: tr("{0} node part of {0} way", "{0.key}");
}

Attachments (1)

buildingnodeonbuildingway.osm (4.2 KB) - added by Famlam 3 months ago.
OSM data showing an example

Download all attachments as: .zip

Change History (6)

Changed 3 months ago by Famlam

OSM data showing an example

comment:1 Changed 3 months ago by taylor.smock

This is partly caught by the building inside building check. See source:trunk/resources/data/validator/geometry.mapcss@18667:186-190#L186 .

Try adding building=shed to node 8793042988 and run the validator.

I think this might be a bug with . I'd have to check the docs though.

comment:2 Changed 3 months ago by skyper

Description: modified (diff)
Keywords: inside added

fix link

comment:3 Changed 3 months ago by Famlam

Type: enhancementdefect

The behavior is indeed very inconsistent. If its the first (equaling last) node of the closed building way, it is caught by the ∈-rule. All other nodes are not matched by this rule. It should be either all nodes match, or none. (The description in the docs, "contains", doesn't specify whether that includes the border way or only the content within the bordering way)

comment:4 Changed 3 months ago by Famlam

Summary: building node part of building shapeInconsistent behavior of ∈ (building node part of building shape)

comment:5 Changed 3 months ago by skyper

Component: Core validatorCore
Keywords: mapcss added

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to Famlam
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.