Skip to content

Updated natural=rock and natural=stone presets#471

Open
bgo-eiu wants to merge 14 commits intoopenstreetmap:mainfrom
bgo-eiu:patch-3
Open

Updated natural=rock and natural=stone presets#471
bgo-eiu wants to merge 14 commits intoopenstreetmap:mainfrom
bgo-eiu:patch-3

Conversation

@bgo-eiu
Copy link
Copy Markdown

@bgo-eiu bgo-eiu commented May 27, 2022

These updates mostly reflect the discussion in #469 , with the objective of arriving at presets for natural=rock and natural=stone that are more clear in meaning and still relate to how they are used.

Included in this version:

  • "rock," "stone," and "boulder" as terms for both since enough people are using these interchangeably that it would help to see both options. Added some miscellaneous related terms for both that I found from searching ("cairns" appears to be a term that can be used to describe rocks used as trail markers for example).

  • "Glacial Erratic / Boulder" as the name of the natural=stone preset because as @westnordost clarified, this is being used more specifically than the name "stone" or "boulder" suggests alone. This would help make it more clear how people are using this tag.

  • Just "Rock" for natural=rock. This is the more common tag and people have been using it for a variety of types of rocks so the simple name seems OK as long as the terms are there.

  • Added "Exposed bedrock" as a checkbox field for natural=rock. Per @matkoniecz asking about this, I think that this has a much more clear meaning for adding geological=outcrop to natural=rock. Attached is too ambiguous and was confusing in the first place.

  • Added dimension field options (height by default; diameter, width, circumference in additional) since these are often used in combination and it would help to know what type of object people are actually mapping.

  • Added a "Lava field" checkbox field in the additional options for natural=bare_rock which adds geological=volcanic_lava_field. This tag combination is used a lot in Iceland for example. I noticed bare_rock didn't have very much in the presets so that could be a helpful detail to have in there.

Comment thread data/fields/geological_outcrop.json Outdated
{
"key": "geological",
"type": "defaultCheck",
"label": "Exposed bedrock",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Field names should be in Title Case.

Comment thread data/presets/natural/stone.json Outdated
"erratic"
],
"name": "Unattached Stone / Boulder"
"name": "Glacial Erratic / Boulder"
Copy link
Copy Markdown
Contributor

@1ec5 1ec5 May 28, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s been a long time since my geology appreciation class in school. “Glacial erratic” is a term I’ve never encountered before, despite hailing from in a region where terms like “terminal moraine” are well known. If there’s no colloquial term for this specific concept, then “Glacial Erratic” is a good name for the preset. But seeing “boulder” next to it might make me inclined to choose this obscurely named preset over “Rock” when tagging a decorative boulder. Maybe “Boulder” should be an alias but not part of the name? (“Decorative boulders”, marketed as such, are so common at garden stores that it could even be a term for that preset.)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was kind of on the fence since boulder is a more commonly known term. The German concept of "findling" that the tag is used for does seem to be a near exact match to "glacial erratic" though so boulder could just be an alias. Good point about decorative boulders / landscape boulders, I was thinking a lot of what people would describe as boulders count as glacial erratics anyway but those would not.

Thank you for the feedback, I will update 👍

Comment thread data/presets/natural/rock.json Outdated
"moreFields": [
"width",
"diameter",
"circumference",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This field is currently called “Circumference (at Breast Height)” because the natural=tree tagging scheme derives circumference from a diameter at breast height (DBH) measurement. (Go figure.) We’ll need to split out a different field for circumferences of other things, like rocks, for which “breast height” would be baffling.

/ref #455

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't realize it was that specific - I think there are already petrified trees on the map as natural=rock so it's not totally out of the question but diameter will do.

@bgo-eiu
Copy link
Copy Markdown
Author

bgo-eiu commented May 28, 2022

Updated to reflect feedback above

Comment thread data/presets/natural/stone.json Outdated
@bgo-eiu
Copy link
Copy Markdown
Author

bgo-eiu commented May 28, 2022

Updated to avoid duplication in terms/aliases

Comment thread data/fields/geological_outcrop.json Outdated
Comment thread data/fields/geological_volcanic_lava_field.json Outdated
@tyrasd tyrasd added the enhancement New feature or request label May 29, 2022
Comment on lines -18 to +31
"name": "Attached Rock / Boulder"
"name": "Rock"
Copy link
Copy Markdown
Contributor

@westnordost westnordost May 29, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just "Rock" for natural=rock. This is the more common tag and people have been using it for a variety of types of rocks so the simple name seems OK as long as the terms are there.

  1. The terms are not visible to people using the iD presets. Only aliases will be, but then only if one specifically searches those
  2. Well this would lead to natural=rock being tagged (a lot) on free-standing non-attached rocks which seems to be wrong according to the previous discussion/research.

If I understand your point correctly, you are saying that this is what is happening anyway already a lot, so renaming the preset name to just "Rock" is merely following the de-facto tagging situation and not redefining the meaning of a tag. The latter, we absolutely do not want for obvious reasons that I hope do not need explaining.

What I am missing is proof for the claim that this is indeed the de-facto tagging situation and if you have proof, it should first be documented in the wiki. How much percent of natural=rock is not really a rock attached to bedrock and how much percent is on the other hand tagged correctly according to the wiki definition? Without this, we cannot simply shrug this distinction away in an iD preset PR. After all, the very first two sentences of the wiki article mention this distinction:

natural=rock describes a notable rock feature or small group of rocks, attached to the underlying bedrock, mostly used on a single node element.

On the contrary, single boulders not attached to the bedrock are better tagged as natural=stone.

... and iD (presets), too, has been denoting the meaning of natural=rock correctly according to the wiki definition since the preset had been added.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point - I have a strong suspicion the de facto use is for any rock due to how the word is understood in English, but I will do some more formal queries on this for documentation first.

Copy link
Copy Markdown
Contributor

@westnordost westnordost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See https://github.com/openstreetmap/id-tagging-schema/pull/471/files#r884296415

As it stands, this would change the definition of a well-documented tag.

@westnordost
Copy link
Copy Markdown
Contributor

Also note that the wiki definition of natural=stone is

A large (whatever that might mean) natural boulder or rock fragment that is not attached to the underlying terrain.

So, this includes more than a glacial erratic - any boulder or rock that is not attached to the underlying terrain is a natural=stone. This is why I proposed that you use the aliases field.

@bgo-eiu
Copy link
Copy Markdown
Author

bgo-eiu commented May 29, 2022

I am going to look into:

  • How is natural=stone used outside of Europe, where mappers likely did not understand the idea of a findling? It does seem like it follows "glacial erratic" quite closely within Europe, but if there is a noticeable pattern in other uses that could complicate things. In any case, it does seem worth updating the wiki in some way to explain the findling concept and link to your research on this to clarify things.
  • How is natural=rock used generally/de facto?

And document the findings for this.

bgo-eiu and others added 2 commits May 29, 2022 13:12
Co-authored-by: Martin Raifer <martin@raifer.tech>
Co-authored-by: Martin Raifer <martin@raifer.tech>
@westnordost
Copy link
Copy Markdown
Contributor

Great! Though, in regards to this, just a disclaimer that I am not the maintainer of the iD presets, @tyrasd is.

In particular, I am not sure what happens if your finding is that indeed, many tagged natural=rocks are actually stones or the other way round: Whether the guideline for contributing to this repo would allow to disregard the (original, documented) definition of a tagging and replace it with a more general one. Maybe there is a precedent for that, I do not know.

@tyrasd
Copy link
Copy Markdown
Member

tyrasd commented May 30, 2022

Great that you're going to look into this in more detail @bgo-eiu !

In particular, I am not sure what happens if your finding is that indeed, many tagged natural=rocks are actually stones or the other way round: Whether the guideline for contributing to this repo would allow to disregard the (original, documented) definition of a tagging and replace it with a more general one.

In general I do abstain from modelling presets in a way that contradicts or otherwise mismatches the documentation on the wiki. If you find that the documentation on the wiki does in fact not match real-world usage of the tags, then we first need to find out why that is: If it is because of editor presets which are not conveying the intended meaning well enough, these presets should probably be improved/clarified. If it is because the tagging concept is not well enough defined or too complex for mappers, then the tagging needs to be redefined, optimally through a tagging proposal. It could also be the case that the wiki just contained some misleading information, then it would suffice to update the wiki.

"name"
"name",
"height",
"geological_outcrop"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something that irritates me a bit is that the proposed added field for geological=outcrop to the natural=stone preset currently only has very few usages (only 30 map features have this tag combination). The wiki also doesn't mention this particular usage.

Copy link
Copy Markdown
Collaborator

@matkoniecz matkoniecz Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bgo-eiu ping?

I have not looked much into PR in general, but I plan to close it as abandoned given no reaction to comment over 3 years. Just to confirm: is it abandoned?

{
"icon": "temaki-boulder3",
"moreFields": [
"geological_volcanic_lava_field"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the geological=volcanic_lava_field tag the combined usage with natural=bare_rock seems to be quite popular. Perhaps it could make sense to have this combination as a dedicated preset, so that it can be more easily found when mapping such features?

@tyrasd tyrasd force-pushed the main branch 2 times, most recently from 9d3204d to 49f529e Compare June 22, 2022 16:19
@matkoniecz matkoniecz added the new-field create a new field (see add-field for cases where field from presets is added to new entries) label Mar 13, 2026
@matkoniecz matkoniecz changed the title Updated natural=rock and natural=stone presets Updated natural=rock and natural=stone presets Mar 14, 2026
Comment thread data/fields/geological_outcrop.json
@matkoniecz matkoniecz added waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review. waitfor-pr-author-response and removed waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review. labels Mar 20, 2026
@matkoniecz
Copy link
Copy Markdown
Collaborator

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

Labels

enhancement New feature or request new-field create a new field (see add-field for cases where field from presets is added to new entries) waitfor-pr-author-response

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants