-
Notifications
You must be signed in to change notification settings - Fork 822
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
Reviewing layer stack (ground-based non-point features) #5041
Comments
There are a few general principles and specific issues that we would like to address that should be considered.
Regarding some of the oddities you observe:
|
Thanks for pulling together these considerations. I imagined that there would be some principles behind the current ordering. With that in mind, the queries refine to
P.S. I was carefully avoiding the roads layers. But a quick scan through #2046 and #3281 suggests that they might not be conflicting. #3281 ends with comment from pnorman "With what I was looking at for merging the layers was to have the areas as part of the road layer. Keeping areas as their own layer before other road layers would be much simpler." So there may be scope for incrementally simplifying the roads code? |
This should have broader input from people knowledgeable in the respective fields. In general i would be very careful with rendering different feature types in changing order relative to each other based on W.r.t. cliffs/line-barriers - keep in mind the general principle to render line features on top of polygon features unless there are specific reasons not to.
They are - you can either combine the road lines and polygons into common layers and commonly render them according to the sorting principles we have - or you move the polygons all to be rendered flatly before all linear roads. You cannot do both at the same time. Much of the past discussion of this can be found in #3872. See also the other issues mentioning #2046 and #3281. |
I have created a new
layering
|
Yes, I can see this. It only makes sense to merge layers where there is a positive case for doing so. For example, with In terms of "after roads layers", perhaps a sensible order would be
[Moved/new layers in italics. Addresses queries E/F.] In terms of
P.S. #4726 probably also needs a layering tag. |
Should IMO go immediately after the road casing/fill layers (as long as it is a separate layer) - bridges should go after guideways.
Yes, this needs to go after bridges and should be before entrances.
Yes, before roller-coaster is a sensible location - before bridges would also be possible.
That would definitely be an improvement relative to the status quo. The ultimate question - as indicated in #5021 (comment) - would be if to combine the polygons and lines to be both rendered after buildings. But that is a bigger question which relates also to #2046. So yes, for the moment this seems a sensible choice.
Done. |
To be absolutely clear, the proposed order is:
|
I would consider this a sensible order under the premise that the layers themselves are as they are. But i am not tied to this - any arguments why a different order is preferable would be welcome. |
As discussed in #5021, it would be useful to review layer contents rather than pushing
leisure=track
andattraction=water_slide
into the first suitable option.The current layer stack in
project.mml
is listed below ([] for layers / groups of layers not being considered). A, B labels for potential oddities that can be discussed / resolved.[landcover-low-zoom]
landcover - A This is a surprisingly busy layer ranging from landcovers (where ordering by descending area makes perfect sense) to "built" items such as
leisure=track
,leisure=pitch
,leisure=swimming_pool
where it is less obvious. This leads to odd situation whereleisure=track
mapped as an area is drawn much earlier thanleisure=track
as a line feature. Features where you couldn't sensibly stack another landcover item above may be better drawn later along with buildings?landcover-line - Currently just
man_made=cutline
. Seems OK. Features such as streams, tree symbols will go on top rather than being "cut" by a green stripe.[ice sheets, water layers]
landcover-symbols - Tree symbols etc.
bridge -
man_made=bridge
buildings - B These two layers could be potentially combined, as it is fairly common to find bridges that are over buildings or buildings that are on bridges. This might become a "built environment" layer incorporating
leisure=track
etc? This would allow layering e.g. sports pitches on the roof of a building.tunnels
landuse-overlay - military areas etc. Fine. Makes sense to "cover" buildings, but not roads.
barriers - D hedges, walls etc. This also seems late. Move forward with same logic as
cliffs
?cliffs - C This seems quite late, e.g. a bridge over a cliff edge will look strange. A more logical place would seem to be after
landcover-symbols
?road layers
waterway-bridges - Arguably best combined with road bridges, but fine for now. Logical place for
attraction=water_slide
?bridges (road bridges)
aerialways - E seems too early, given that these will be above all ground features.
guideways - [should be part of roads layers]
roller-coaster
entrances
aeroways - F seems too late. Belongs with "misc roadways"?
golf-line
admin
amenity-line - This is problematic for the two features, water-slide and track that are rendered as lines rather than point symbols.
power lines
tourism / protected areas boundaries - G These seem a bit late. Before power lines?
trees - H This seems very late given they are non-blocking features. Surely they belong better with landcover symbols?
[place names, symbols, text]
The text was updated successfully, but these errors were encountered: