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

Don't render house numbers for amenity=car_sharing #4912

Closed
dreua opened this issue Dec 6, 2023 · 5 comments
Closed

Don't render house numbers for amenity=car_sharing #4912

dreua opened this issue Dec 6, 2023 · 5 comments

Comments

@dreua
Copy link

dreua commented Dec 6, 2023

Expected behavior

It has been decided to not render car sharing (#1891) so I'd expect it to not render anything.

Actual behavior

amenity=car_sharing with an address shows up as a house number on the street. This is expected to be the address of a building close by so it is very unlikely to add any value and much more likely to cause confusion. In any case it looks bad.

Screenshots with links illustrating the problem

image

https://www.openstreetmap.org/node/9141226027

We have quite a few of those in Mainz: https://overpass-turbo.eu/s/1EmC

@HolgerJeromin
Copy link
Contributor

Perhaps this information should be a ref instead of an addr:housenumber. Not sure about this...

@dch0ph
Copy link
Contributor

dch0ph commented Dec 6, 2023

Yes, this seems a misuse of addr:housenumber. It's not a question of whether amenity=car_sharing is rendered (it isn't). Basically any node with addr:housenumber will show up with an "address".

This doesn't seem to be a ref number, just a duplication of the closest genuine address onto another feature (which would be poor/incorrect tagging). Or perhaps this is the way these sites are labelled locally?

@HolgerJeromin
Copy link
Contributor

HolgerJeromin commented Dec 7, 2023

Same with postboxes (at least in Germany):
image

https://commons.wikimedia.org/wiki/File:12-04-21-wildparkstrasze-ebw-by-RalfR-11.jpg

@dreua
Copy link
Author

dreua commented Dec 7, 2023

Thanks for the input, I'll check whether we can fix that by tagging differently.

@dreua dreua closed this as not planned Won't fix, can't repro, duplicate, stale Dec 7, 2023
@imagico
Copy link
Collaborator

imagico commented Dec 7, 2023

An additional note for clarification: The way we render housenumbers relies on the application of the One feature, one OSM element rule w.r.t. addresses. This is widely the case, but not universally, in particular the whole SEO sector is adamant at adding address information to each and every business under a certain address even if the building already has this or there is a separate address node. This does not show visually in the map as long as we render the feature in question (shop, office, etc.) in some way, because the housenumber labels have lower priority. But for those features we do not render which have a duplicate address information tagged it shows up of course.

Now we could try to de-duplicate those (though it would be fairly expensive - with an additional spatial query for every address). But the thing is that this tagging is inherently ambiguous, because it uses exactly the same address tagging for the actual address (the plot/building/building entrance that the address is verifiably assigned to, where the postal service would try to deliver a letter to that is posted with that address) and for features where the address data is just piggy-backed on to indicate the feature belongs to that address somehow. In other words: In case of duplication of the same address there is no distinction between the declarative use of the address tags (this feature represents that address - which is the consensus use among mappers) and the referential use (this feature belongs/is related to that address). Trying to hide this inherent ambiguity in tagging practice through some heuristics in de-duplication of addresses would not be in support of our mission.

See also #1746.

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

No branches or pull requests

4 participants