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

Addresses should have higher priority than storage tanks #3435

Open
jidanni opened this issue Oct 6, 2018 · 25 comments
Open

Addresses should have higher priority than storage tanks #3435

jidanni opened this issue Oct 6, 2018 · 25 comments

Comments

@jidanni
Copy link

jidanni commented Oct 6, 2018

Addresses should cover storage tanks, just like they cover buildings. Not the other way around. #3429

@jidanni
Copy link
Author

jidanni commented Oct 6, 2018

If addresses didn't cover buildings there would be no addresses.

@kocio-pl kocio-pl added this to the Bugs and improvements milestone Oct 6, 2018
@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 6, 2018

Hi again:

  1. What is so special about storage tanks which does not apply to many other icons?
  2. How would you like to achieve that?

@jidanni
Copy link
Author

jidanni commented Oct 7, 2018

So if a building has a tank on its roof or side, its address gets eaten. Preposterous.

@jidanni
Copy link
Author

jidanni commented Oct 7, 2018

How about solar panels or fish tanks on the roof? Install them and ambulance services can no longer find your house!

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 7, 2018

We simply don't show them, so it's not a problem.

Special services need more data sources or at least their own server. I know that Polish fire and rescue services use it like this, here's a detailed presentation in Polish (I hope YT will enable automatic translation):

https://youtu.be/etFl6e7Ew00?t=3132

or at least something less sophisticated, like this:

https://wiki.openstreetmap.org/wiki/OpenFireMap

@jidanni
Copy link
Author

jidanni commented Oct 7, 2018

Fine, all the Whitehouse or Empire State Building has to do is install a water tank and its address disappears. Google should work that way too, I suppose.

@jidanni
Copy link
Author

jidanni commented Oct 7, 2018

Or even Trump Towers' neighbor pop stand pop tank "trumps" Trump Towers' address!

@HolgerJeromin
Copy link
Contributor

This is the same with all poi in and on buildings:

https://www.openstreetmap.org/?mlat=50.78733&mlon=6.07165#map=19/50.78733/6.07165

Instead of the important building name we see the name of the student bar and the radio sender.

I dont think we really can solve this overall issue.

@HolgerJeromin
Copy link
Contributor

Install them and ambulance services can no longer find your house!

I really hope an adress search tool (nominatim or osmand) does not skip adresses with a tank on top!

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 7, 2018

It looks that we already avoid showing all man_made things on the roof (which I did for small towers or chimneys):

'man_made_' || CASE WHEN (man_made IN ('mast', 'tower', 'water_tower', 'lighthouse', 'windmill', 'obelisk', 'communications_tower', 'chimney', 'storage_tank', 'silo')
AND (tags->'location' NOT IN ('roof', 'rooftop') OR (tags->'location') IS NULL)) THEN man_made ELSE NULL END,

For standalone storage tanks I don't think there's solution.

@jidanni
Copy link
Author

jidanni commented Oct 8, 2018

It may be a volunteer fire department, who have never heard of OSM themselves, relying on screenshots from public reporters.

Anyway in the original case, a 1 ㎡ tank nearby a 30 ㎡ house causes the house number to get zapped, even thought it is the only house in 100 meters.

Just simply first paint the house. Then paint the tank. Then paint the number, with a tiny border, to make sure it contrasts with whatever is piled up below it.

P.S., I hope cell towers are still rendered on top of houses.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

It may be a volunteer fire department, who have never heard of OSM themselves, relying on screenshots from public reporters.

Still address search tool works. I feel this example is very far stretched.

Anyway in the original case, a 1 ㎡ tank nearby a 30 ㎡ house causes the house number to get zapped, even thought it is the only house in 100 meters.

Such feature as "nearby" is simply not in our database, we can't use it.

P.S., I hope cell towers are still rendered on top of houses.

Only if there's no location=roof or location=rooftop.

@jidanni
Copy link
Author

jidanni commented Oct 8, 2018

Nearby: unrelated polygon then. Not even adjacent. Like Netherlands and Luxembourg.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

This code does not work like this, we don't check any distances. If you want to understand it better, you might start with reading general informations:

https://wiki.openstreetmap.org/wiki/Standard_tile_layer#Technical_details

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

There's a proposition to make this icon less solid: #588 (comment) - that might help.

This was referenced Oct 26, 2018
@jidanni
Copy link
Author

jidanni commented Oct 26, 2018

I need to have people see the address of this house, "1-3".

They can see it just fine on
https://www.openstreetmap.org/way/638198859#map=17/24.18167/120.86520

But zooming in causes a water tank to appear and the address to dissappear
https://www.openstreetmap.org/way/638198859#map=18/24.18166/120.86521

Zooming further one now gets two water tanks, and no address.
https://www.openstreetmap.org/way/638198859#map=19/24.18166/

I assume any new icons for water tanks will still cover up addresses.

It is a sad day tomorrow as I will be forced to move the tanks away from the
building enough to get the address to show.

@matkoniecz
Copy link
Contributor

matkoniecz commented Oct 26, 2018

It is a sad day tomorrow as I will be forced to move the tanks away from the

Threatening to damage correct data is not helpful. See https://wiki.openstreetmap.org/wiki/Tagging_for_the_renderer

See https://help.openstreetmap.org/questions/731/how-can-i-revert-a-changeset and https://wiki.openstreetmap.org/wiki/JOSM/Plugins/Reverter for tools to undo edits like this.

@jidanni
Copy link
Author

jidanni commented Oct 26, 2018

Well one of them has to be removed soon for repairs anyway. So might as well remove it from the map.

@polarbearing
Copy link
Contributor

@jidanni - could you please understand that OSM is a geo database, which can be used for different purposes. Rendering a map as a feedback loop for the mappers is just one of them. Evaluating the data for search and navigation is another one.

BTW I would consider micromapping a tiny tank of 70cm diameter with 19 nodes a waste of data.

@jidanni
Copy link
Author

jidanni commented Oct 26, 2018

So do I. That would be due no scale bar available when editing. openstreetmap/iD#5435

@matkoniecz
Copy link
Contributor

Can someone provide a photo of location where this is a problem? I suspect that someone incorrectly tagged something that is not a storage tank.

@jidanni
Copy link
Author

jidanni commented Nov 1, 2018

Can someone provide a photo of location where this is a problem? I suspect that someone incorrectly tagged something that is not a storage tank.

It's my house. I tagged the storage tank. #3435 (comment)

@jidanni
Copy link
Author

jidanni commented Nov 2, 2018

Can someone provide a photo of location where this is a problem? I suspect that someone incorrectly tagged something that is not a storage tank.

p_20181102_103115_vhdr_auto-459x816

@jeisenbe
Copy link
Collaborator

jeisenbe commented Nov 7, 2019

Related to #3880 - Symbol and icon prioritization. In this case the request is to have addr:housenumber text render at higher priority than man_made=storage_tank icons, while in #3904 it is requested that office= dots and text be rendered with higher priority than addr text.

@jeisenbe jeisenbe changed the title Addresses should cover storage tanks, just like they cover buildings Addresses should have higher priority than storage tanks Nov 10, 2019
@jidanni
Copy link
Author

jidanni commented Mar 24, 2021

A second address has arrived at my house, due to mass import of city data!
https://www.openstreetmap.org/node/8535565687
Alas, can you believe it, it also got knocked off the map by a different water tank!

Good thing my other house has no "dangerous water tanks" too nearby.
Both its hand edited address and its city import address survived to the renderer.
https://www.openstreetmap.org/?mlat=24.18170&mlon=120.86615#map=19/24.18170/120.86615

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

No branches or pull requests

6 participants