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

Add in "Salvage Yard" or "Junk Yard" area type #3387

Closed
rlerner opened this issue Aug 30, 2016 · 7 comments
Closed

Add in "Salvage Yard" or "Junk Yard" area type #3387

rlerner opened this issue Aug 30, 2016 · 7 comments
Labels
preset An issue with an OpenStreetMap preset or tag waitfor-consensus Waiting for OpenStreetMap consensus

Comments

@rlerner
Copy link

rlerner commented Aug 30, 2016

For designating automotive scrap yards, steel recycling scrap yards, "U-Pull-It" etc.

@pnorman
Copy link
Contributor

pnorman commented Aug 30, 2016

What tags are appropriate here?

@bhousel bhousel added the preset An issue with an OpenStreetMap preset or tag label Aug 30, 2016
@bhousel
Copy link
Member

bhousel commented Aug 30, 2016

Recent tagging thread here: https://lists.openstreetmap.org/pipermail/tagging/2016-January/028178.html
(that didn't really resolve the issue)

Reasonable tags include:
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dwaste_transfer_station

@bhousel bhousel added the good first issue Best for first-time contributors. No experience necessary! label Oct 14, 2016
bhousel added a commit that referenced this issue Oct 31, 2016
@bhousel bhousel removed the good first issue Best for first-time contributors. No experience necessary! label Oct 31, 2016
@bhousel
Copy link
Member

bhousel commented Oct 31, 2016

I added a Waste Transfer Station preset.

Let's hold off on amenity=scrapyard vs industrial=scrap_yard until there is better consensus.
(amenity=scrapyard has a few more uses and seemed to be the preferred tag in that tagging mailing list thread, but industrial=scrap_yard is close and has a wiki page)

@pnorman
Copy link
Contributor

pnorman commented Oct 31, 2016

Let's hold off on amenity=scrapyard vs industrial=scrap_yard until there is better consensus.

Is this a case where they should be added as non-searchable presets?

@bhousel
Copy link
Member

bhousel commented Oct 31, 2016

Is this a case where they should be added as non-searchable presets?

I don't think so, both tags have such low usage that there is no point at this time.

@bhousel bhousel added the waitfor-consensus Waiting for OpenStreetMap consensus label Nov 2, 2016
@bhousel
Copy link
Member

bhousel commented Jun 20, 2017

This has been open for a long time so I'm just adding the amenity=scrapyard preset to close it out.

The OpenStreetMap wiki suggests that landuse=industrial + industrial=scrap_yard is the tag to use. I actually tried for a while today to create the preset this way, but it was causing me a lot of problems:

  • industrial= is not a "toplevel" tag, so it does not get tag_classes unless the landuse=industrial tag is present. The tag classes are what allow us to adjust the rendering style of all the features and the preset icons.
  • All of the landuse=* presets are area-only features, not POI style things that can be mapped as either nodes or areas. A feature that shows fields like operator, address, opening_hours is an amenity/shop/office/etc, not a landuse.
  • landuse= areas do not render icons. If we did that, the map would be full of unnecessary icons. So to show an icon on this POI-like landuse would require extra hacky code.
  • Compound tags need special addTags and removeTags properties in order to recognize the primary tag industrial=scrap_yard even if the other tag landuse=industrial is missing.
  • There are only 150 uses of industrial=scrap_yard in OpenStreetMap as opposed to 91 uses of amenity=scrapyard. These are both very low numbers. This means that there is still time to "fix" the tag to be something more consistent with how other tags work in OpenStreetMap.

So amenity=scrapyard it is... 🔧

I don't normally write up a long post like this, but this is another situation where we're going against the recommendation of the OpenStreetMap wiki for good reasons, and I think it's useful to explain why in detail so that people who invent new tags or work on the wiki have a better understanding of some of the challenges we face when consuming and writing tools for OpenStreetMap.

@bhousel
Copy link
Member

bhousel commented Oct 25, 2017

per #4466 (comment)
We can do the same thing for industrial=scrap_yard, even though it might have some of the drawbacks listed above. I deprecated the amenity=scrapyard preset that I created last June, and made an industrial=scrap_yard one instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preset An issue with an OpenStreetMap preset or tag waitfor-consensus Waiting for OpenStreetMap consensus
Projects
None yet
Development

No branches or pull requests

3 participants