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

Multipolygon: handling of closed touching inner rings #57

Open
simon04 opened this issue Nov 25, 2012 · 4 comments
Open

Multipolygon: handling of closed touching inner rings #57

simon04 opened this issue Nov 25, 2012 · 4 comments

Comments

@simon04
Copy link
Contributor

simon04 commented Nov 25, 2012

According to https://wiki.openstreetmap.org/wiki/Relation:multipolygon#Touching_inner_rings, touching inner rings may be used if the ways are closed.

Osmium, however, fails on such multipolygons, e.g., http://www.openstreetmap.org/browse/relation/1517038:

Building multipolygon based on relation 1517038 failed: invalid ring

@joto
Copy link
Owner

joto commented Nov 25, 2012

"fails on such multipolygons" is a bit of a generalization. Osmium works fine in all other "touching inner rings" cases that I have seen. But you are right, in this case it doesn't work. Geometry of the multipolygon looks fine, so this should work. So this is a bug that needs looking into.

@simon04
Copy link
Contributor Author

simon04 commented Nov 25, 2012

"fails on such multipolygons" is a bit of a generalization.

It wasn't meant as an insult of Osmium nor as overly generalization, but I experienced this behaviour on several multipolygons of this type in my region … :-)

@joto
Copy link
Owner

joto commented Nov 25, 2012

Can you give some more examples? Maybe we can figure out what they have in common and this way figure out where the problem might be.

@simon04
Copy link
Contributor Author

simon04 commented Nov 25, 2012

Those are the errors reported by the OSM Inspector as Multipolygon/Touching inner rings. All examples I've seen have at least two inner rings with a common way segment.

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

2 participants