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] Change retail and commercial area colors #3544

Closed
wants to merge 6 commits into from

Conversation

kocio-pl
Copy link
Collaborator

@kocio-pl kocio-pl commented Dec 1, 2018

Changes proposed in this pull request:

  • Replace retail color with something not aggressive (reusing light orange from what was a temporary societal area color)

We use few different reddish landcover colors. They are proper for military or police areas, kind of violet works for industrial areas and less intensive for commercial areas, but retail areas look like dangerous places. I think we should change that to make them not look scary, especially when downtown is tagged like this, for example Hannover:

Before

screenshot_2018-12-01 openstreetmap carto kosmtik 1

After

screenshot_2018-12-01 openstreetmap carto kosmtik 2

Some other examples:

screenshot_2018-12-01 openstreetmap carto kosmtik 3
screenshot_2018-12-01 openstreetmap carto kosmtik 4

screenshot_2018-12-01 openstreetmap carto kosmtik 5
screenshot_2018-12-01 openstreetmap carto kosmtik 6

screenshot_2018-12-01 openstreetmap carto kosmtik 7
screenshot_2018-12-01 openstreetmap carto kosmtik 8

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 1, 2018 via email

@Tomasz-W
Copy link

Tomasz-W commented Dec 1, 2018

I'm also not sure about current red colour of retail areas, because it makes them looking like some danger areas, and propably will be even more as we moved healthcare key to red. But proposed colour doesn't fit there, it looks too natural-landcover-like.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 2, 2018 via email

@Adamant36
Copy link
Contributor

I was just thinking farmyards could use a change now that farmland is different. #fbecd7 might work.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 8, 2018

What other color would you like to test instead?

We are making green areas more consistent (some natural + semi natural vegetation and leisure/sport), also we use blue for water and ice areas, and the rest is more or less free to use for any other needs (with some natural areas like yellow for sand). The proposed color is yellow with red tint, so it still is good visible, but we have too many red-like areas, which is associated with danger in general.

@Tomasz-W
Copy link

Tomasz-W commented Dec 8, 2018

@kocio-pl I propose to use #f2ece6 for retail and commercial areas.

@Adamant36
Copy link
Contributor

@Tomasz-W, same color for both huh? Gutsy. I was actually thinking there might be an advantage to that.

@Tomasz-W
Copy link

Tomasz-W commented Dec 8, 2018

They are very simillar to each other at the moment anyway, see: https://www.openstreetmap.org//#map=19/52.37146/9.73750. If there will be more voices to distinguish them I will look for some mix of pink (retail) + blue (offices) for commercial areas as they are a mix of retail and office functions.

@Adamant36
Copy link
Contributor

True. Both colors could use improvement to. Not just retail.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 9, 2018

@kocio-pl I propose to use #f2ece6 for retail and commercial areas.

Very similar to the bare ground:

ou8ccmji

@Tomasz-W
Copy link

Tomasz-W commented Dec 9, 2018

@kocio-pl Please try #f2e7e1 for retail and #e6ecf2 for commercial

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 9, 2018

Such retail would be close to the uniform service (police and fire departments), which are based on military background, and I feel grey for commercial is too pale:

https://www.openstreetmap.org/#map=18/52.21179/21.01202

yhpr9w6f

https://www.openstreetmap.org/#map=17/52.21371/21.04829

2t94wxw1

@Tomasz-W
Copy link

Tomasz-W commented Dec 9, 2018

@kocio-pl Next propositions due to your remarks:

  • retail -> #fcf7ed
  • commercial -> #e6eef0

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 9, 2018

They are also quite pale, retail is close to the ground (and a bit to societal amenities) and commercial grey is hard to notice:

https://www.openstreetmap.org/#map=15/52.2017/20.9341

rr02dsmn

@Adamant36
Copy link
Contributor

What about softer tones of something similar to the industrial color?

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 9, 2018

Everything is possible, it just needs to be tested.

@Tomasz-W
Copy link

Tomasz-W commented Dec 10, 2018

@kocio-pl Effects of my further amateur tests:

  • more intensive #f2ebdc for retail
  • darker #e1edf0 for commercial

Edit: please test also #fcf1e6

@jragusa
Copy link
Contributor

jragusa commented Dec 10, 2018

Based on the last screenshots of @kocio-pl, the increasing different colours in cities starts to be a bit confusing. That's why I would follow the suggestion of @Tomasz-W to merge retail and commercial colours. Please also notice we already use the same colour than industrial for railway and it's not shocking.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 10, 2018 via email

@Adamant36
Copy link
Contributor

For me it could go either way. While I think its important to be able to tell the difference between residential/industrial areas and retail/commercial areas, its less important tell the difference between retail and commercial. Although I agree maybe its an important distinction in places like silicon valley, in a lot of others it just adds extra noise. Especially in areas where the landuse planning is crappy, there is blended landuse, and the boundary between offices and stores isn't that clear cut. Its also not so clear cut in the tagging what exactly constitutes a retail business versus an office (or even somewhere that provides a service for a fee that isn't necessarily a store).

So there is ultimately no way to map it 100% accurately. Unless people have access to plot level zoning maps put out by their local government, of which most people don't have. So its a rendering difference without a purpose. For most people in most cases its enough to just know that any particular place is commerce, industrial, or residential. The icons for the specific tags make up whatever lack of clarity there be might be. Specific colors for retail and commercial areas where probably more important before all the tags related to them had specific icons. Now, the special rendering is not nearly as necessary.

@kocio-pl
Copy link
Collaborator Author

Although I agree maybe its an important distinction in places like silicon valley, in a lot of others it just adds extra noise.

BTW - there's also quite rarely used area (at least where I live) with its own color - I mean landuse=garages. It might be useful, but in overall design we just loose one color that might be useful for something more popular.

@kocio-pl
Copy link
Collaborator Author

I have united both retail and commercial under one color:

#f2ebdc (let's call it 1) looks promising - it's paler than my original proposition, but clearly different than bare ground, farmland and social colors:

gmkkf 7_

clywmezf

#fcf1e6 (2) looks a bit too pale to me:

vmwdomav

cd2dajw9

So maybe we could use similar colors for retail (bit stronger) and commercial (bit weaker) - like these both (1 & 2):

a8kjcgoi

...or 0 (my original proposition) & 1:

cd mrktf

@kocio-pl kocio-pl changed the title Change retail area color Change retail and commercial area colors Dec 14, 2018
@Tomasz-W
Copy link

Tomasz-W commented Dec 15, 2018

My remarks:

  • I would use one colour for both tags - using so simillar 2 shades does make no sense, because it wont't be saying anything and it would be hard to distinguish intuively, another thing is that adding every new colour makes map more complitated which we should generally avoid
  • I would use "1" (#f2ebdc)
  • labels should be set darker

PS. @kocio-pl The example which you mentioned earlier https://www.openstreetmap.org/way/436159213 is a tagging mistake - I presume you can't buy a verdict in this court, what makes it non-commercial ;)

@Adamant36
Copy link
Contributor

I really like color 1. @Tomasz-W's suggestions also seem sound. My only question is if there is a chance that rendering commercial and retail areas exactly the same will lead to people miss-tagging them because the difference becomes superficial on a rendering level. If not, them one color works fine. If so, maybe two close shades are better.

@kocio-pl
Copy link
Collaborator Author

Ok, so it looks like 1 is going to be used somehow. I have no clear idea if having one color or two similar is better, I hope we can find something convincing which approach we could use (or maybe which is better). I also agree that labels should be tuned once we have area colors set.

You're right with the court. It was my big misunderstanding of commercial tag definition - I thought that every kind of office area belongs here. Thanks for pointing it out!

@Adamant36
Copy link
Contributor

Adamant36 commented Dec 29, 2018

I think these are interesting examples. In the first one the retail area is really large, but it still looks really washed out. In the second, there's a lot of retail and commercial areas mapped in the center, but they aren't even visible. I don't think a color change would help either of them. I've tested the different colors from above for both at z13 and it didn't seem to help at all. Especially with the second example.
mall area
downtown
There's also a large retail area here that can hardly be made out. Again, changing the color to something above probably wouldn't help much.
third example
A multi-block retail area in down town SF. Again, almost impossible to see.
downtown sf
Notice from those examples that the grey areas are easy to make out, even small ones.

Largish retail/commercial area in Gilroy.
gilroy

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 30, 2018 via email

@Adamant36
Copy link
Contributor

@jeisenbe, good call. That could help. Im sure some of it is the parking lots spliting the areas up. Which makes them harder to see at that level.

@jeisenbe
Copy link
Collaborator

Here are some renderings at 4x (four times the usually resolution) at z13 and z14 of Redding, CA. Compare to the first image in the previous comment: #3544 (comment)

z13 at 4x resolution
z13-redding-4x

z14 at 4x resolution
z14-redding-4x

z15 at normal resolution
z15-redding-master

The parking lots and empty lots lead to many areas of land-color and parking color, which are both nearly the same neutral light gray, and this makes it hard to see the retail areas.

Compare to Kahului, which has only a few parking lots mapped around the central malls (I believe there are many smaller parking lots that have not been mapped).
The commercial and retail areas are much more clearly visible here.

z14 at 4x
z14-kahului-centre-4x
z16
z16-kahului

I think we should consider no longer rendering parking lots earlier than z14 (same as buildings).

@kocio-pl
Copy link
Collaborator Author

Nice idea, it was something I thought myself lately.

@matthijsmelissen
Copy link
Collaborator

I think we should consider no longer rendering parking lots earlier than z14 (same as buildings).

👍

@matthijsmelissen
Copy link
Collaborator

As far I can see, this is still under discussion so I'm not merging this yet.

@matthijsmelissen matthijsmelissen changed the title Change retail and commercial area colors [WIP] Change retail and commercial area colors Jan 2, 2019
@Adamant36
Copy link
Contributor

I did a brief look over of the problematic places with the new change to parking. It helped some but not completely. So I still think more needs to be done to z13 for any color to work good there (including the current one).

The two options I see are 1. Continue tweaking z13 and put this on hold until thats done 2. Just dont use the critism of the colors at z13 as a factor and go with what looks good at the other zoom levels (probably one of @Tomasz-W's latest colors).

Personally, I dont have a prefrence. With either option we are 99% probably going with one of @Tomasz-W colors in the end anyway. So theres really no difference with doing it now versus later. Except for testing the two latest ones he wanted tested (but thats about it).

@Tomasz-W
Copy link

Tomasz-W commented Jan 3, 2019

@Adamant36 Please test my latest proposition from #3544 (comment)

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 4, 2019

f2ebdc-bare_ground-ebe6dd-transportation_area

#f2ebdc is similar to bare_ground color (Δ: 3.6) and land-color (Δ: 5.0)

#ebe6dd is very similar to bare_ground color (Δ: 1.5) - lightness is the same, chroma is only 1 point different, and hue is only 14 points different. It's also quite similar to transportation_area (Δ: 2.3) and land-color (Δ: 3.5).

And #f2ebdc is close to #ebe6dd (Δ: 3.6) as well.

I don't think these will work. Test renderings:

Test areas:
z13-landcover

z14-landcover

z15-landcover

Redding, CA:

  • Can hardly see retail and commercial areas
    z13
    z13-redding-after
    z14
    z14-redding-after

Current Master: z13
z13-redding-master
z14 current
z14-master

@Adamant36
Copy link
Contributor

Its still hard to see even with the parking lots removed. At least in the retail/commercial area where the Redding name is. You also can't tell the difference between the retail and commercial areas on the right side in the same pictures.

I was planning on doing after tests of the places I originally used when I get some time. I'm thinking z13 will still be an issue though even without the parking lots. A couple of the examples didn't have parking lots to start with.

Ultimately I think the only solution is to make retail/commercial/whatever grey at z13, because your just going to be able to tell the difference at that level whatever the color is (unless you go by the few examples of larger places that are probably miss-tagged anyway).

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 4, 2019

It wouldn't make sense to unify industrial, commercial, retail and residential at z13 when we just started showing patterns at z13 to distinguish different areas, and removed buildings and parking to show the landcover/landuse better.

The problem in central Redding is that the retail and commercial areas are small, and the many large highways fill up the space. But there is no trouble seeing the retail areas by the motorway and Hilltop Drive, on the right in the current master images. Correct?

@Tomasz-W
Copy link

Tomasz-W commented Jan 4, 2019

@jeisenbe Both retail and commercial areas have an outlines helping distinguish them from bare ground, and you seem to didn't include them in squares visualisation (or if you included them, but they are actually so weak, we still can set them darker)

PS. You really don't need to provide some 'delta' calculations tables, if you want to show that some colour would be not proper in you opinion, just upload a test rendering/s showing possible problems with proposed colour ;)

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Jan 4, 2019

I agree that we need something actionable and I still think reddish is not an option, even if it's not really visible on z13. It's just a problem with area size - most of the nice colors on midzoom would look too strong on high zoom. And because OSM Carto is not for finding things, I would like them to look reasonable on high zoom, even if midzoom would be not so visible.

@jeisenbe Please say what do you think about it.

@Adamant36 I understand you're upset, but please make it shorter then, repeating arguments does not help.

@Adamant36
Copy link
Contributor

Adamant36 commented Jan 4, 2019

@kocio-pl, my bad. I didn't realize that I repeated myself that much. I was just trying to get my point across, but it could have been done way more succinctly.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 4, 2019 via email

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 4, 2019

Both retail and commercial areas have an outlines helping distinguish them from bare ground, and you seem to didn't include them...

The retail, commercial and industrial outlines first appear at z16. The tests I showed above were z13, z14 and z15, so the outline is not visible.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 4, 2019 via email

@Adamant36
Copy link
Contributor

What about #ffe7e4 for retail and #f3dddc for commercial? Both are fairly close to the original colors and still view-able zoomed out, but retail doesn't look as redish and both look a lot more subtle.

z19
z19
z15
z15
z13
z13

@kocio-pl
Copy link
Collaborator Author

@Adamant36 Would you like to make some before/after tests of this? It is close to the @jeisenbe proposition to merge them, but a subtle difference sounds preferable to me.

@Adamant36
Copy link
Contributor

Retail #ffe7e4 commercial #f3dddc

Z17 Before
Landuse Before z17 1
Z17 After
Landuse After z17 1
z19 different place before
Landuse Before z19 1
z19 different place after
Landuse After z19 1
Z18 different place before
Landuse Before z18 2
z18 different place after
Landuse After z18 2

I'll do more tests after some initial feedback.

@kocio-pl
Copy link
Collaborator Author

Thanks. How would it look like near some police/fire service areas? I'm worried that it would be too close.

@ankitmlive
Copy link

what about my instances ?
I have changed all the building, ways, retail, commercial areas and landcover colors for a better viewing experience. Brighter version of openstreetmap-carto

Screen Shot 2020-01-08 at 12 37 01 PM

Screen Shot 2020-01-08 at 12 37 21 PM

@pnorman pnorman marked this pull request as draft April 17, 2020 22:02
@pnorman
Copy link
Collaborator

pnorman commented Jun 20, 2022

@kocio-pl are you still working on this, or should it be closed?

@pnorman
Copy link
Collaborator

pnorman commented Jun 27, 2022

Closing as inactive. If you're interested in working on this in the future, feel free to open a new PR.

@pnorman pnorman closed this Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants