-
Notifications
You must be signed in to change notification settings - Fork 821
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 relations 'Island within a hole' not rendering as expected #3347
Comments
This is a data issue: the water area doesn't have a hole in the data (it's not a multipolygon). |
Hi How comes the island renders the graphic? How come the other renders manage it? It's an outer within an inner within an outer. The water doesn't require a specific relation as it doesn't have a separate tag for the island - it should render as a wood. Can you confirm OSM-Carto has always done this, as I'm sure I've created island which render |
@DaveF63 I'm not sure exactly how you'd like @matthijsmelissen to phrase it, but https://www.openstreetmap.org/way/616701768 is an inner to the wood but is not itself a relation. https://www.openstreetmap.org/way/615985619 is not an inner to the water relation and hence is both "wood" and "water". This change was made here a while back to accommodate places where those things happen together (think mangroves). See e.g. #1754 for some previous discussion. |
sent from a phone
On 16. Aug 2018, at 17:56, SomeoneElseOSM ***@***.***> wrote:
This change was made here a while back to accommodate places where those things happen together (think mangroves).
+1, we have to look at the actual data, not whether other renderers show or don’t show the problems. And yes, this style should try to show the problems rather than hiding them in favor of a more pleasant map, because people will fix the problems much more likely if they suffer from them.
|
This is a follow up to this question: The only problem I see is that when water was properly tagged as a multipolygon with a hole inside, the island has the tree symbols, but not the green background. Which is interesting, but I would probably just tag the forest on the island instead of complicated two part multipolygon with forest-inside-a-forest shape. |
@DaveF63 Happy to expand on my answer. First: the current tagging indicates that the middle island is both trees and water, i.e. that the trees are growing in the water. Does this describe the situation correctly? If not, the tagging needs changing.
The current tagging describes the middle circle as both water and forest. Some renderers (such as the cycling map) decide to render it as forest, others as water.
This is a quote from link, which seems wrong to me. In the new situation, we also need two multipolygon relations. The quote seems to have been written years ago by @woodpeck, maybe he can comment?
No, OSM-Carto never rendered the island: http://bl.ocks.org/matthijsmelissen/raw/7d2b0538a8b64b9d31e2/#18.00/51.02482/-3.93383 |
Hi
Expected behaviour
https://www.openstreetmap.org/#map=18/51.02480/-3.93326&layers=C
Actual behaviour
https://www.openstreetmap.org/#map=18/51.02480/-3.93326
https://wiki.openstreetmap.org/wiki/Relation:multipolygon#Island_within_a_hole
Why isn't OSM-Carto rendering the inner island? I could have sworn it used to.
The text was updated successfully, but these errors were encountered: