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

mixed (ref-less and ref-full) out geom content leads to broken multipolygon rendering #38

Closed
tyrasd opened this issue Jul 14, 2015 · 2 comments
Labels

Comments

@tyrasd
Copy link
Owner

tyrasd commented Jul 14, 2015

See tyrasd/overpass-turbo#182

another test case: http://overpass-turbo.eu/s/apQ

@tyrasd tyrasd added the bug label Jul 14, 2015
@tyrasd
Copy link
Owner Author

tyrasd commented Jul 14, 2015

this is potentially related to #23

@tyrasd tyrasd changed the title mixed out geom content leads to broken multipolygon rendering mixed (ref-less and ref-full) out geom content leads to broken multipolygon rendering Jul 14, 2015
@tyrasd
Copy link
Owner Author

tyrasd commented Jul 14, 2015

The problem is that we have one part of the multipolygon's ways with node refs, while the rest is provided ref-less (just node coordinates, from the output of out geom on relations). This confuses the multipolygon assembly algorithm which relies on either all nodes having proper ids or all nodes being ref-less.

This is going to be quite tricky to solve. One possibility would be to greatly improve #23 together with a more clever multipolygon assembly. Another way would be to parse each full-geometry element independently from each the rest of the supplied content, which sounds like a reasonable thing to do.

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

1 participant