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

[WIP] Render salt ponds. #3905

Closed
wants to merge 5 commits into from
Closed

[WIP] Render salt ponds. #3905

wants to merge 5 commits into from

Conversation

StyXman
Copy link
Contributor

@StyXman StyXman commented Sep 24, 2019

Fixes #545.

Changes proposed in this pull request:

  • Render salt ponds with a flat old pink color. This reflects the fact that salt ponds tend to be colorful. Also the white salt dots recently introduced could be added.

Test rendering with links to the example places:

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

Before
Screenshot_20190924_231526

After
Screenshot_20190924_230907

This is closely related to #3901.

@StyXman StyXman changed the title 545 [WIP] Render salt ponds. Sep 24, 2019
@jeisenbe
Copy link
Collaborator

The location shown for testing may not be ideal, because there seems to be overlap between the wetland=saltmarsh and landuse=salt_pond areas.

The pink/purple color is somewhat similar to industrial and commercial areas. While salt ponds have some similarities to quarries and industry, we had discussed trying a shade of blue, since these features are intermittently covered with water, and adding a pattern in addition, in #545.

Perhaps we could try the triangular pattern that @imagico has used in his map style? See https://github.com/imagico/osm-carto-alternative-colors/blob/alternative-colors/symbols/patterns/salt_pond.svg and https://github.com/imagico/osm-carto-alternative-colors/blob/alternative-colors/symbols/patterns/salt_pond.png

@imagico
Copy link
Collaborator

imagico commented Sep 25, 2019

Generally a good idea but the color does not work for me either.

@StyXman
Copy link
Contributor Author

StyXman commented Sep 25, 2019

The location shown for testing may not be ideal, because there seems to be overlap between the wetland=saltmarsh and landuse=salt_pond areas.

So I'm looking for freestanding salt_ponds?

@StyXman
Copy link
Contributor Author

StyXman commented Sep 25, 2019

I'm going to have an issue here:

https://www.openstreetmap.org/way/108543652#map=14/47.4039/-2.4703

14-2068623-1468308

As you can see, the salt_pond contains a lot of basins. Whatever we choose here, it's going to be on top of everything else, like the pink blobs in the sshot above.

@jeisenbe
Copy link
Collaborator

https://www.openstreetmap.org/way/108543652 is not mapped according to the usual way that is documented at Tag:landuse=salt_pond - only the (intermittent) water areas should get this tag. Fortunately, most of the salt ponds in your screenshot are mapped correctly - all those east of Kercabellec have the tag landuse=salt_pond just on each pond, e.g. https://www.openstreetmap.org/way/142063430 and https://www.openstreetmap.org/way/142063459 - and rendering these similar to other water areas should help give feedback in support of consistent mapping.

@jeisenbe jeisenbe added new features Requests to render new features water labels Oct 10, 2019
@jeisenbe
Copy link
Collaborator

Whatever we choose here, it's going to be on top of everything else, like the pink blobs in the shot above.

That's only true if we render landuse=salt_pond with priority over natural=water. While I think that is the correct option, you could also render areas as water if double-tagged with landuse=salt_pond + natural=water - this is very common. In that case, we would need to use a pattern, like salt_pond.png, rendered on top of water areas, so that the landuse is still visible.

@jeisenbe
Copy link
Collaborator

Example with ocean-color and salt_pond.png pattern for landuse=salt_pond:

z16-salt-pond-pattern-oceancolor

z14-salt_pond-corse

Next to the facebook offices, in East Palo Alto:

Salt ponds in South Australia:

z14 after
z14-dry-creek-after-salt_pond

@jeisenbe
Copy link
Collaborator

jeisenbe commented Nov 12, 2019

@StyXman what do you think about the option with the pattern
shown in #3905 (comment) ?

Copy link
Collaborator

@pnorman pnorman left a comment

Choose a reason for hiding this comment

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

I don't think the pink works because it is too similar to industrial.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Mar 7, 2020

This needs rebasing for v5.0.0 - should I do this myself and take over this PR so that @StyXman gets credit for starting this, or would it be clearer I close this PR and open a new one with the designs in #3905 (comment) ?

@pnorman
Copy link
Collaborator

pnorman commented Mar 7, 2020

This needs rebasing for v5.0.0 - should I do this myself and take over this PR so that @StyXman gets credit for starting this, or would it be clearer I close this PR and open a new one with the designs in #3905 (comment) ?

Do a new PR because you're not going to be reusing any of the commits here

@jeisenbe jeisenbe closed this Mar 7, 2020
@StyXman StyXman deleted the 545 branch March 7, 2020 10:03
@StyXman
Copy link
Contributor Author

StyXman commented Mar 7, 2020

@jeisenbe thanks for thinking in terms of credit. I don't care much about it, really. Currently I have no time to work on these :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new features Requests to render new features water
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add rendering for landuse=salt_pond
4 participants