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

Filling man_made=pier with color #3459

Open
Commodoortje opened this issue Oct 19, 2018 · 27 comments
Open

Filling man_made=pier with color #3459

Commodoortje opened this issue Oct 19, 2018 · 27 comments

Comments

@Commodoortje
Copy link

man_made=pier does not get rendered anymore, it's looks like rendering blancs:
https://www.openstreetmap.org/way/451067705
Is that a conscious choice or did something go wrong?
2018-10-19_08-33-16

@Tomasz-W
Copy link

This was discussed in #2652, but I still don't find land-colour fill a proper one for this feature.

@polarbearing
Copy link
Contributor

It's also discussed in #3343 where I proposed to use the parking colour.
First, the pier bridges land and sea, and the land part is invisible when using land colour as it is.
Second, it's kind of a parking place for the boats, for reasoning the choice of colour.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 19, 2018

Hm, I see nothing wrong with a current color.

man_made=pier does not get rendered anymore, it's looks like rendering blancs:

What difference do you see? I'm not aware of anything like that, we just made linear piers thinner in #3267, but as far as I know nothing has changed for areas, especially the color.

@Tomasz-W
Copy link

I see nothing wrong with a current color

In my case it's like when I see land colour, I'm intuively treating it as "there is nothing here". Of course it's good to have at least land colour to see piers on the water, but I think we shouldn't use land colour for any other feature than a blank land.
As footway-like fill was rejected and I think parking fill is not a good idea because boats etc. park not on the pier but next to it, I propose to try with some darken shade of fill colour. They would be distinguished then, and some kind of brown would refer to wood colour which most of piers are build of.

@Adamant36
Copy link
Contributor

Adamant36 commented Oct 19, 2018

I like the idea of some kind of brown color. 4049 piers have the surface tag, of which 2981 are tagged as wood. There's 318,281 of them overall though. So it might not reflect the surface of piers in general, but I do think the default surface for most people when they think of piers is wood, whatever material any particular pier might be made out of in real life.

@kocio-pl kocio-pl changed the title man_made=pier Filling man_made=pier with color Oct 19, 2018
@kocio-pl
Copy link
Collaborator

My idea would be to start with something subtle, just to show there's not bare land, like different small x (like 1, 2, 5, 10...) in:

darken(@land-color, x%)

@Tomasz-W
Copy link

Tomasz-W commented Oct 19, 2018

When I'm doing my amateur test renderings in Photoshop, I'm using usually "round" numbers by moving each value by -5 or +5, so I suggest with trying 5, 10, 15% etc., otherwise it would be a straight way to go insane, because as we are mostly men here, we would propably have big problems with rating such subtle colour differences (current looking for minor/ major buildings proper shades is an exeption) ;)

@kocio-pl kocio-pl added new features Requests to render new features and removed cartography labels Oct 19, 2018
@polarbearing
Copy link
Contributor

I would not drive colour decisions from the surface material here. Probably my parking comparison wasn't a good argument either. But a pier is not land, it is a built feature.
Stepping back for a wider perspective, using a greyscale colour would fit into the colourspace of manmade features. It should be light so that the piers are recognisable but not too dominant.

(current @land-color is f2efe9, thus not neutral grey)

@Kogacarlo
Copy link

Kogacarlo commented Oct 20, 2018

Gimp exercise fff3d7:
fff3d7

@Adamant36
Copy link
Contributor

Adamant36 commented Oct 20, 2018

I just don't want another Beigeish/off yellowish color. And it would be cool if it was clear where the land stops and the pier starts. Its not so clear with fff3d7. At least in the example. What about a variation of the footway area or leisure colors? Either that, or we can go with something slightly lighter then parking like @polarbearing suggested.

@Kogacarlo
Copy link

Kogacarlo commented Oct 20, 2018

I like the idea of some kind of brown color.

Gimp exercise 543a00:
(please note OSM rendering would be slightly different because of no real anti-aliasing in my images)
543a00

@Adamant36
Copy link
Contributor

Thanks. I was imagining something a little lighter, but its to bad. People would definitely know they were there. Although it might scrue up rendering of amenity stuff due to also being brown.

@Kogacarlo
Copy link

Kogacarlo commented Oct 20, 2018

Adamant36, what do you mean with "I was imagining something a little lighter, but its to bad." ?

I find this funny, color 957890:
957890

@Kogacarlo
Copy link

Kogacarlo commented Oct 20, 2018

I was imagining something a little lighter,

Color b27b00, the poop color.
This may be the best color after all because usually piers are all shitted over by cormorants and ducks.
b27b00

@Kogacarlo
Copy link

Kogacarlo commented Oct 20, 2018

Last one.
Looking at an aerial picture of the marina my boat is in, I matched the color as close as possible.
Actually I like this a lot but I don't know what this would look like on other zoomfactors.
Color 848484:
848484

@Adamant36
Copy link
Contributor

Adamant36 commented Oct 20, 2018

I like the last one. Id be willing to test it when I have the time if know one beats me to it. Thanks for doing the mockups.

@Tomasz-W
Copy link

3% darken land colour:
pier2
3 2
light #faf7f0:
faf7f0 1
faf7f0 2
bridge fill re-use:
bridge 1
bridge 2

@meased
Copy link
Contributor

meased commented Oct 20, 2018

I don't think any of the above renderings are an improvement over what we have now.

Perhaps instead of changing the fill color the casing color could be tweaked? (like a bridge casing)

I would like to leave the pier color alone and allow combining it with landuse=retail/commercial/industrial/....

@Kogacarlo
Copy link

I find the last render shown by Tomasz-W called bridge fill re-use an improvement. It shows there is something rather than nothing.
Thanks Tomasz!

@polarbearing
Copy link
Contributor

leave the pier color alone and allow combining it with landuse

What would that mean exactly?

@Tomasz-W
Copy link

Basing on a visualisations above I think that only '3% darken' and 'the same fill with some outline' versions are worth further testing.

@meased
Copy link
Contributor

meased commented Oct 24, 2018

leave the pier color alone and allow combining it with landuse

Well, to explain with an example, I think Brighton Pier should be retail pink:

brighton

@kocio-pl
Copy link
Collaborator

I'm afraid that piers are so thin, that the outline would dominate them too much, especially at lower zoom levels. Bridges are typically wider.

But maybe it would be enough to move the piers layers (like piers-poly and piers-line) before landcover layer in project.mml, so they have their own, delicate color, but would be eclipsed by some other color if this area has some landuse tagged for example. Could somebody test if it would work this way?

@meased
Copy link
Contributor

meased commented Oct 24, 2018

bridge fill re-use an improvement. It shows there is something rather than nothing.

I think that's just the point. Piers are nothing. They are just an extension of land that goes out a little farther over the water. If there is "something" on them, that should be rendered separately.

While bridge casing may work where the water meets the pier, I do not think it would look good to have a casing where the pier meets the land (for piers mapped as an area).

@matthijsmelissen
Copy link
Collaborator

Well, to explain with an example, I think Brighton Pier should be retail pink:

That would require rendering piers below landuse colors. However, it would also mean that the parts of piers on top of beaches would become yellow (de facto invisible).

@polarbearing
Copy link
Contributor

Piers are nothing. They are just an extension of land

They are not nothing, they are man_made objects. The can be party over land and are partly over water.

@jeisenbe
Copy link
Collaborator

I've mentioned a possible way to render landcover over piers in #330 (comment) - this requires implementing #3854 first.

@jeisenbe jeisenbe added landcover and removed new features Requests to render new features labels Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants