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

Render bare_rock pattern from z13, same as shingle and scree #4072

Merged
merged 1 commit into from
Apr 25, 2020

Conversation

jeisenbe
Copy link
Collaborator

Fixes #3862

Partial revert of #3458

Changes proposed in this pull request:

  • Renders rock_overlay.png pattern from z13 only, instead of z5, for natural=bare_rock
  • This is the same level when the similar features natural=scree and natural=shingle are first distinguished from natural=bare_rock by a pattern.
  • At mid and low zoom levels the great majority of bare_rock areas are too small for the pattern to be clearly seen, but the pattern adds noise and makes it harder to interprete the outline of these areas. For this reason, the pattern is not shown for scree or shingle before z13 (similarly, scrub, forest and other patterns are not shown before z13).
  • Currently the only very large areas of bare_rock are coarsely-mapped desert areas, which really consist of a mix of sand, scree, bare_rock, heath, grass, etc, and our showing the pattern early gives unintentionaly encouragement to mapping really large areas like this coarsely, rather than showing the detailed landcover/surface in each spot.

Test renderings with links to the example places:

Wales - NW

  • Most areas of bare rock in Wales are small patches on peaks or near quarries or along the ocean, as in most of Europe.

https://www.openstreetmap.org/#map=12/53.0707/-4.1940
z13 - unchanged
wales-bare_rock-scree-z13-after

z12 before
wales-bare_rock-scree-before-12:53 0707:-4 1940

z12 after
wales-bare_rock-scree-after-12:53 0707:-4 1940

z11 before
wales-bare_rock-scree-before-z11

z11 after
wales-bare_rock-scree-after-z11

Wales - Snowdonia National Park

z13 Unchanged
wales-bare_rock-scree-moors-after-13:52 9000:-4 0023 png

z12 before
wales-bare_rock-scree-moors-before-z12

z12 after
wales-bare_rock-scree-moors-after-z12

z11 before
wales-bare_rock-scree-moors-before-z11

z11 after
wales-bare_rock-scree-moors-after-z11

Corsica - South

z13 same
north-corsica-rock-scree-z13

z12 before
south-corsica-rock-scree-before-12:41 6185:9 1643

z12 after
south-corsica-rock-scree-after-12:41 6185:9 1643

z11 before
south-corsica-rock-scree-before-z11

z11 after
south-corsica-rock-scree-after-z11

Corsica - North

z13 same
north-corsica-rock-scree-z13

z12 before
north-corsica-rock-scree-before-12:42 5801:9 3068

z12 after
north-corsica-rock-scree-after-12:42 5801:9 3068

z11 before
north-corsica-rock-scree-before-z11

z11 after
north-corsica-rock-scree-after-z11

Renders rock_overlay.png pattern from z13 only, instead of z5. This is the same level when the similar features natural=scree and natural=shingle are first distinguished from bare_rock by a pattern.
@kocio-pl
Copy link
Collaborator

On the one hand I understand this as a rule, but in practice pure color areas look cryptic to me. For example in Snowdonia National Park z12 looks like industrial areas and on Corsica this looks similar to military area color nearby.

@jeisenbe
Copy link
Collaborator Author

in practice pure color areas look cryptic to me

That's how we render natural=scree and natural=shingle currently. I would be happy to adjust the low zoom @bare_ground color to improve it, however.

That's done in #3670, where unvegetated areas are rendered as #efe7d9.

The current rendering is lightened 20% at low zoom, so it looks a little too similar to some other features.

@imagico
Copy link
Collaborator

imagico commented Mar 12, 2020

I think this is fine - adjusting the low zoom color for the bare ground areas could be considered - but this makes relatively little sense as long as the fading is messing with the color relationships anyway.

@kocio-pl
Copy link
Collaborator

I would be happy to adjust the low zoom @bare_ground color to improve it, however.

Since brown looks more or less like a mud, I expect that might be hard without the patterns. But of course one should see test such assumptions.

@jeisenbe
Copy link
Collaborator Author

jeisenbe commented Mar 12, 2020

Since brown looks more or less like a mud...

We currently use earth-tones ("brown") for:
@farmyard: #f5dcba
@garages: #dfddce
@construction: #c7c7b4
@heath: #d6d99f

In addition to @bare_ground: #eee5dc (scree, shingle, bare_rock)
and
#e6dcd1 @mud

And before we had the earth-tone @societal_amenities color (previously the old @farmland color)

I don't think those ALL look like mud, do they? ;-)

@kocio-pl
Copy link
Collaborator

I have never seen a garage area being THAT big... We don't expect to see anything like this on these levels and there are relatively few types of objects, so it's much easier to find better fitting colors.

Another idea might be to mix background brown with some grey taken from the pattern. Mud could be more brown and less grey, since this is basically a soft (watered) ground. In general, it seems that bare ground and mud are currently "flipped", so maybe just flipping them would be enough.

@jeisenbe
Copy link
Collaborator Author

How about #efe7d9? This was tested in #3670 - I could show some examples from that branch.

(But any changes to the fill colors will need to be in another PR, whether in #3670 or a follow-up)

@jeisenbe jeisenbe requested a review from imagico March 21, 2020 13:04
Copy link
Collaborator

@imagico imagico left a comment

Choose a reason for hiding this comment

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

Works as intended and restores the previous state and with that the last remaining pattern use at low zoom levels (together with wetland - see #4094).

@jeisenbe jeisenbe merged commit 95a8f05 into gravitystorm:master Apr 25, 2020
@jeisenbe jeisenbe deleted the rock-pattern branch April 25, 2020 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Too early and inconsistent initial zoom level for natural area patterns
3 participants