Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude site relation multipolygon members from "Missing tag or role" #2343

Open
sammyhawkrad opened this issue Sep 24, 2024 · 4 comments
Open
Labels

Comments

@Marc-marc-marc
Copy link
Contributor

looking at the 1st exemple, it's a wrong usage of a type=site relation : a site relation is not a collection of all objets (relation is not a collection) in an area related to one tag (operator, owner or whatelse) but for objets unable to be describe with a (closed) area
from wiki A site [relation](https://wiki.openstreetmap.org/wiki/Relation) is used to group several objects which together have a single identity as a whole but cannot be accurately described by other data types
here the "site" is described with the closed way

@sammyhawkrad
Copy link
Author

I totally agree that the site relation is not correctly used in the examples I have shared. I have deleted a few of them with similar use recently. But the issue here is not really about "what" is mapped.

The real issue is that when a multipolygon is added to a site relation it gets flagged as not having a tag or role. The issue will still arise in cases where the site relation is correctly used.

@Famlam
Copy link
Collaborator

Famlam commented Sep 28, 2024

Item 1140, class 2
Can you give an example of such a valid case?

I can't think of many ways (no pun intended) how an untagged, outer way of a multipolygon is part of a site relation, but not the multipolygon (which contains the tags) itself.

@Marc-marc-marc
Copy link
Contributor

Marc-marc-marc commented Sep 30, 2024

by looking randomly, I've found at least 2 cases that shouldn't be causing a report but are doing so at the moment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants