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

Surface Tags On Nodes #17

Closed
jharpster opened this issue May 16, 2018 · 11 comments
Closed

Surface Tags On Nodes #17

jharpster opened this issue May 16, 2018 · 11 comments
Labels

Comments

@jharpster
Copy link
Contributor

The team has found >5000 improperly surface=* tagged nodes in Australia.
https://overpass-turbo.eu/s/ySQ

In each case we'll check to see if

  • The surface tag belongs on an adjacent way

  • The node is representing something that might be more accurately represented by a closed or unclosed way (ie. parking)

  • In other cases we'll just remove the surface tag from the node in alignment with guidelines on the wiki.

@mvexel
Copy link

mvexel commented May 16, 2018

I can see this making sense though @jharpster

Or does OSM wiki state that surface should only be on ways?

@jharpster
Copy link
Contributor Author

@mvexel Wiki says surface tags should only be on ways. But your example is a good argument for the contrary.

@andrewharvey
Copy link

The node is representing something that might be more accurately represented by a closed or unclosed way (ie. parking)

I suspect that's true for a lot of these, doesn't make it wrong, but if you're able to upgrade these to a way then 👍

In other cases we'll just remove the surface tag from the node in alignment with guidelines on the wiki.

I'd be cautious about that, always best to check with the original mapper before removing.

For example a lot of traffic_calming's should only be mapped as a node, and that node can have it's own surface.

@jharpster
Copy link
Contributor Author

Good point on the traffic_calming @andrewharvey. We'll check with local mappers before completely removing tags , the intent is just to clean up some of the mess.

@BrackoNe
Copy link

BrackoNe commented May 17, 2018

Hi!

I have to be honest to say that I didn't get @andrewharvey answer completely.

"For example a lot of traffic_calming's should only be mapped as a node, and that node can have it's own surface." Partially correct. traffic_calming can be mapped as way as well (especially for traffic_calming=hump or traffic_calming=island).

If you look at https://wiki.openstreetmap.org/wiki/Key:surface, it is clear that surface tag can't be used as Property of node. It can be used for ways and areas (polygons).

However, if you scroll to Values section, you will see that surface key has plenty of varieties’. Just after that, there is Element column that says for every single value which type of element can be used. Anything except that is not allowed. There are no nodes at all.

Please confirm.

Nemanja

@bhousel
Copy link

bhousel commented May 17, 2018

It sounds like the team is planning to check each node to verify whether it is correctly tagged - this is good!

The OSM Wiki is frequently wrong or offers contradictory advice, as it's maintained by volunteers who often aren't working with the actual OSM data or tooling. Anyway, it's important to view the wiki as a guide but not a spec. The only true spec in OSM is the data itself.

For example, things like tennis courts can be mapped as a point node with a surface tag. This is fine. Please don't remove the surface tags from them, just because you found a wiki page that says a wrong thing.

@andrewharvey
Copy link

The OSM Wiki is frequently wrong or offers contradictory advice, as it's maintained by volunteers who often aren't working with the actual OSM data or tooling. Anyway, it's important to view the wiki as a guide but not a spec. The only true spec in OSM is the data itself.

👍

@BrackoNe
Copy link

Hello, thanks for supporting us and for your feedback @bhousel !

However I just want to say that your example confirms OSM Wiki again.
Sport (tennis court) can be marked as NODE or as AREA (polygon).
Surface can be used for WAYs or AREAs. So if sport (tennis) is marked as AREA, useful combination is surface. But users often do not read it carefuly, and it is like: "Oh OK, I can mark as a node, and useful combination is surface. Great! I don't need anything else."

I just wanted to emphasize that. I will not proceed with this task until I do not get green light.

@bhousel
Copy link

bhousel commented May 17, 2018

Surface can be used for WAYs or AREAs.

@BrackoNe I am trying to be very nice here - the OSM wiki is not a spec. It does not work the way you think it does.

But users often do not read it carefuly, and it is like: "Oh OK, I can mark as a node, and useful combination is surface. Great! I don't need anything else."

Those users are correct. There are situations where you can put a surface tag on a node. The wiki page for surface is misleading. Whoever wrote it was not thinking about point tagged things. (parking lots, sports pitches, aerodromes, playgrounds). You will find plenty when you start this task.

Please do not treat the OSM wiki as a spec. Treat is as some person's opinion (and that person probably does not work on OSM tools or data).

@BrackoNe
Copy link

Thanks again for your reply!

@BrackoNe I am trying to be very nice here - the OSM wiki is not a spec. It does not work the way you think it does.

Sure, I'm not yelling, just emphasizing types of entities, but I see that it would be better if I used bold function. :)

Please do not treat the OSM wiki as a spec. Treat is as some person's opinion (and that person probably does not work on OSM tools or data).

OK, thanks for clarification. 👍

@andrewharvey
Copy link

The wiki has just been updated to document that use on a node is valid.

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

6 participants