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

Tag natural=rock (node and area) is not rendered #2616

Open
mkyral opened this issue Apr 21, 2017 · 54 comments
Open

Tag natural=rock (node and area) is not rendered #2616

mkyral opened this issue Apr 21, 2017 · 54 comments

Comments

@mkyral
Copy link

mkyral commented Apr 21, 2017

I've changed incorrectly mapped natural=cliff area to natural=rock and it is not render at all.

https://www.openstreetmap.org/way/263176747

I thought it is only connected to area, but later on I've find that even node is not rendered.

http://www.openstreetmap.org/node/3841474372

JOSM uses following icon: https://josm.openstreetmap.de/export/11963/josm/trunk/images/presets/misc/rock.svg

@imagico
Copy link
Collaborator

imagico commented Apr 21, 2017

Yes, we currently do not render this. The numbers:

  • about 145k nodes
  • about 10k areas
  • nearly all of the nodes (about 135k) are from canvec imports
  • only about 3400 features have a name tag

We could render areas with the same styling as natural=bare_rock and an outline in addition. Nodes could be rendered with a small icon - design wise it would make sense to consider having two slightly different variants, one for natual=stone, one for natural=rock.

@HolgerJeromin
Copy link
Contributor

nodes could be rendered as a grey dot (similar to barrier)...

@mboeringa
Copy link

mboeringa commented Apr 21, 2017

nearly all of the nodes (about 135k) are from canvec imports

If that is the case, than probably almost all of them are obstructive dangerous rock formations on the coast and inland waterways and lakes, which also explains almost none having a name.

Look at this section of a scanned CanMatrix map: the symbol used is a "+" and notice how they show up near an island of an inland lake:

afbeelding

@mboeringa
Copy link

mboeringa commented Apr 21, 2017

Which also raises the question if they shouldn't be re-tagged according to OpenSeaMap:
seamark:type=rock

The "+" sign also seems in accordance with S100 / OpenSeaMap symbolization:
http://wiki.openstreetmap.org/wiki/Seamarks/Rocks

Actually, quite of a lot of these may even be underwater, at least temporarily based on tides. The "+" symbol even seems to be the sign for permanently submerged dangerous rock formations based on the OSM Wiki page.

@mkyral
Copy link
Author

mkyral commented Apr 21, 2017

But not all rocks are on water. E.g. Czech Republic has no sea ;-)

@imagico
Copy link
Collaborator

imagico commented Apr 21, 2017

Don't worry - there are no intentions here to base rendering primarily on use of a tag in imports which may or may not be in line with how the tag is used otherwise.

@nebulon42 nebulon42 added this to the New features milestone Apr 23, 2017
@aaronstar
Copy link

aaronstar commented Aug 24, 2017

As an example, recently I changed several features around Uluru, Northern Territory, Australia to natural=bare_rock from natural=rock. In this instance natural=bare_rock was more appropriate, however, this decision was also supported by the fact that natural=rock was not rendered. It would be good to see some form of rendering of this feature.

@Tomasz-W
Copy link

Do you think we should consider only icon/ dot rendering or area fill also?

@PontiacCZ
Copy link

I am for filling the areas as well, there are rocks tens of meters in diameter (or even hundreds).

Looking forward to see rocks and stones rendered! :-)

@Tomasz-W
Copy link

I propose:

  • for icon: edited memorial=stone icon in man-made-grey (but I'm open to any other ideas)

natural rock

  • for areas: #bfbab6 fill + rock_overlay.png

Here are some Photoshop mock-ups:
rock2
rock

@PontiacCZ
Copy link

I was thinking about a texture instead of a solid color for rocks too and this looks good!

@Adamant36
Copy link
Contributor

Rock area as how @Tomasz-W suggested
rock area

@Tomasz-W, do you want to provide an .SVG of the memorial=stone icon filled in so I can test it?

@Tomasz-W
Copy link

Tomasz-W commented Dec 2, 2018

@Adamant36 https://gist.github.com/Tomasz-W/e2bcbebf5f5ebb05c01272ba16325d24

Please add more examples (with different rock areas surroundings), I'm considering using a little bit darker shade for rock areas. What do you think?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 2, 2018 via email

@Adamant36
Copy link
Contributor

@jeisenbe, you mean natural=rock and natural=bare_rock? Also, how come? What's wrong with natural=rock being rendered different? I guess the difference between them is sort of superficial.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 2, 2018

Yes, sorry of the typo.

The wiki says "natural = rock describes a notable rock feature or small group of rocks, attached to the underlying bedrock" and natural=bare_rock can also be used for bedrock: "An area of bare rock [that] is sparsely vegetated or not vegetated at all, so that the solid bedrock becomes visible."

So I believe it would be best to render them the same, when tagged as an area feature.

Also, for the node rendering with an icon, take a look at natural=stone - although it's not approved.

@Adamant36
Copy link
Contributor

You don't think that since its the difference between a level feature versus one sticking out of the ground that you have to climb on that they should be rendered slightly differently? Like, what about natural=rock's that are mapped next to or inside natural=bare_rock areas? It would then look like one continuous rock. Which would be misleading.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 2, 2018

its the difference between a level feature versus one sticking out of the ground that you have to climb on

Bare_rock can also be used for vertical features. This is one of the example images on the natural=bare_rock page:

what about natural=rock's that are mapped next to or inside natural=bare_rock areas?

Good point. It might be a good idea to try an outline for natural=rock; this would be sufficient to show the extent of the feature. This would mainly be useful if the rock has a name, eg. used for bouldering / rock climbing. Perhaps something like:

    [zoom >= 17] {
      line-width: .3;
      line-color: darken(@bare_ground, 20%);
      [name != ''] {
        line-width: 0.5;
      }

@Adamant36
Copy link
Contributor

I think outlines work good, if even, for "imaginary" borders like administrative boundaries. Not so much for natural features though. For something like rocks it would be to abstract and there would be zero way for anyone to know that's it was. At least alone. Maybe there could be an outline with the pattern, but then it would be pointless because there's a pattern.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 3, 2018 via email

@Adamant36
Copy link
Contributor

only 4909 of them out of 170,279, or 0.01%, have names and who knows how many of those are ways versus nodes. Its a really small percentage to have a special rendering based off of.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 3, 2018 via email

@Adamant36
Copy link
Contributor

Just because they are from an import originally doesn't necessarily mean they shouldn't be rendered.

@Tomasz-W, what do you think?

@Tomasz-W
Copy link

Tomasz-W commented Dec 8, 2018

@Adamant36 I would go with #2616 (comment) or a little bit lighter shade + icon for rock nodes

@kocio-pl
Copy link
Collaborator

kocio-pl commented Dec 8, 2018

I guess area should be adjusted to natural=bare_rock - probably just less granular pattern (this is also how natural=bare_rock is different from natural=scree and natural=shingle).

@Adamant36
Copy link
Contributor

@Tomasz-W, you want to come up with a less granular pattern based on the one for natural=bare_rock? It makes sense to do it that way for me.

@Tomasz-W
Copy link

Tomasz-W commented Dec 8, 2018

@Adamant36 I've tried many times and I still can't use that Imagico's tool for patterns properly, so I can't help there.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 10, 2018

Here are a few options to consider. I'm not sure what direction to take this pattern.

I tried #d1c8be for the first 3 patterns, which is about 10% darker than bare_earth (#eee5dc). This is darker than the current bare_rock pattern, however.

1-d1c8be-rock-pattern

2-rock

3-rock

These use #dfd6cc for the pattern, more similar to the current bare rock pattern:
4)
4-rock

5-rock

6-rock

Zip with the 6 original svg files:
rock.zip

@Tomasz-W
Copy link

@jeisenbe We need some 'side by side' examples near bare_rock to compare.

@jeisenbe
Copy link
Collaborator

I was hoping that @Adamant36 might want to make some test images. I have not yet downloaded any areas that have natural=rock areas next to bare_rock.

Here's a sample of the current bare_rock to compare:
bare_rock

@Adamant36
Copy link
Contributor

Sure. I'll do it when I get some time. The first three patterns look interesting.

@kocio-pl
Copy link
Collaborator

I like 2 the most, because the difference is visible (not a general surface, there are single objects visible).

@jeisenbe
Copy link
Collaborator

More rock patterns to try, similar to 1), 2) and 3), which were popular.

I think they pattern may need to be smaller however; I remember that the large square patterns looked good alone for allotments, but on the map smaller patterns work better, especially at mid to low zoom levels where many area features are very small.

  1. 0.25 scale, closer together (distance 6), with darker pattern in #d1c8be
    http://www.imagico.de/map/jsdotpattern.php#x,256,jdp82238;g,6,32,32;rx,250,2,32,32;rd,1,0,1,rock,0.25,24,24,0,jdp3249,d1c8be,eee5dc;
    7-rock

  2. 0.5 scale, distance 18, #d1c8b3
    http://www.imagico.de/map/jsdotpattern.php#x,256,jdp83313;g,18,32,32;rx,250,2,32,32;rd,1,1,1,rock,0.5,24,24,0,jdp81566,d1c8be,eee5dc;
    8-rock

  3. 0.2 scale, distance 10, #d1c8b3
    http://www.imagico.de/map/jsdotpattern.php#x,256,jdp83313;g,18,32,32;rx,250,2,32,32;rd,1,1,1,rock,0.5,24,24,0,jdp81566,d1c8be,eee5dc;
    9-rock

Zip file with original svg files (which need conversion to png with transparency prior to use):
rock-7-8-9.zip

@Adamant36
Copy link
Contributor

I really like 8. It looks better then 1 did. 9 is good also.

@Adamant36
Copy link
Contributor

@jeisenbe, totally off topic but do you think we apply the same thing with parking lot areas to show their surface or do you think it would even be worth doing? A few of the patterns you created kind of make me think it would possibly be cool.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 12, 2018 via email

@Adamant36
Copy link
Contributor

Yeah exactly. Hhhhmmm OK. I was thinking about how rendering for unpaved roads should probably come first. Since it would look weird if they had to do different of a pattern and I agree roads should have the priority. So I'll wait to do an issue about the parking lot thing until after that's done.

@Adamant36
Copy link
Contributor

Zip file with original svg files (which need conversion to png with transparency prior to use):
rock-7-8-9.zip

@jeisenbe, you want to convert them to png and do the transparency thing so I can use them, since your the one that came up with them? Graphics aren't my thing and @Tomasz-W is busy so he's not going to do it.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 27, 2019 via email

@jeisenbe
Copy link
Collaborator

@Sjord
Copy link
Contributor

Sjord commented Jul 31, 2019

There is a filename clash: natural=bare_rock currently renders using symbols/rock_overlay.png. That filename would also be a obvious name for rendering natural=rock. So I propose to rename the current rock_overlay.png to bare_rock_overlay.png. An alternative would be to use render both using the same overlay.

Here are the images for what is called rock 8 above. These have transparency.

rock_overlay.png:
rock_overlay

rock_overlay@2x.png:
rock_overlay@2x

Also, remember to copy 8-rock.svg into symbols/generating_patterns/rock.svg.

Sjord added a commit to Sjord/openstreetmap-carto that referenced this issue Aug 2, 2019
Fixes gravitystorm#2616.

The used overlay is rock 8 as mentioned in gravitystorm#2616. Rock is rendered differently
than bare_rock, especially for cases when a notable rock feature is surrounded
by bare rock.
@Sjord Sjord mentioned this issue Aug 2, 2019
@jragusa
Copy link
Contributor

jragusa commented Mar 20, 2020

Since it's just a matter of size according to the wiki page, the pattern of natural=bare_rock could be applied with a latter rendering (z=16 or 17). An alternative would be to have a more spaced pattern.

@PontiacCZ
Copy link

Something went wrong with the commit from August 2019? Currently natural=rock is not rendered.

https://www.openstreetmap.org/node/7058306101#map=19/49.18632/17.36780

@imagico
Copy link
Collaborator

imagico commented Jun 24, 2020

This issue is open, natural=rock is not rendered (and has never been IIRC).

@Adamant36
Copy link
Contributor

Adamant36 commented Jun 24, 2020

Nothing went wrong with it. It was declined and closed without being merged (this issue should probably be closed also to reflect it).

@PontiacCZ
Copy link

PontiacCZ commented Jun 24, 2020

Oh, I am sorry. I saw the commit and thought it finally resolved this issue.

No problem with closing this isue but... maybe it would be good to get the natural=rock rendered first? At least nodes (currently over160k of them worldwide), areas can be tagged with bare_rock.

https://taginfo.openstreetmap.org/tags/?key=natural&value=rock#map

@Adamant36
Copy link
Contributor

No problem with closing this isue but... maybe it would be good to get the natural=rock rendered first? At least nodes (currently over160k of them worldwide), areas can be tagged with bare_rock.

There was a reason it ended up not getting rendered. If I recall correctly it was because the tag wasn't being used constantly enough as intended. I don't see that changing any time soon since its largely due to the ambiguous meaning of the word "rock." This isn't the place for a tagging discussion though.

@Pecita
Copy link

Pecita commented Sep 24, 2020

An area of bare rock is sparsely vegetated or not vegetated at all.
A rock, although significant, can be covered by trees.
For this reason I prefer a rock to be represented by a dark gray ovoid icon so that it can be inserted into a natural=wood or a landuse=forest as a single point.

@Mahabarata
Copy link

With more 226000 elements with the tag natural=rock, maybe this issue now is more relevant ?

The icon on iD seems great, no ? https://www.openstreetmap.org/edit?node=3183806161#map=19/53.05846/15.17789

Why not to use it on nodes and use the bare_rock rendering on areas ?

@PontiacCZ
Copy link

I wonder how many is enough... ;-)

I also like the iD icon.

@Nekzuris
Copy link

I've seen many rock areas changed to bare_rock only for the render, but according to the wiki, rock was the right tag.
I don't really understand why it's still not rendered yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.