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

Icon colour categorization (Meta issue) #3395

Open
polarbearing opened this issue Sep 14, 2018 · 57 comments
Open

Icon colour categorization (Meta issue) #3395

polarbearing opened this issue Sep 14, 2018 · 57 comments

Comments

@polarbearing
Copy link
Contributor

polarbearing commented Sep 14, 2018

There are a lot of new icons being added recently, and often the colour has to be decided. Doing that on a case-by-case basis apparently leads to inconsistencies, thus this issue tries to collect definitions and decisions, which I can refine in the top post according to the discussion evolving. I focus on the principal colours and leave out fine tuning of labels etc.

Edits:
2018-09-15 gastro colour, headers current and change requests.
2019-01-11 health colour; minor amenities to man_made

Current situation

amenity-brown: = memorials:, culture:, public-service:

734a08; #734a08
financial services (bank, ATM, bureau_de_change, casino)
unattended shelters and installations (amenity_shelter, bbq)
unattended items (post_box, recycling, telephone, bench, waste_disposal, waste_basket, picnic_table)
artwork, memorials
culture (theatre, cinema, museum, community_centre, arts_centre
public_bath, shower, toilets, drinking_water, firepit
public service (fire station, embassy, courthouse, social_facility, town_hall, post_office, prison
tourism=office (don't see the rendering)
viewpoint
historic things (castle, fort, archaeological site etc)
problematic: car_wash, car_service

transportation-icon: = accommodation-icon:

0092da; #0092da
Transport related (public transport, vehicle rent, taxi, filling/charging station, toll, elevator, parking)
Touristic accommodation (hotel, motel, hostel, tourism-apartment, guest house, camping/caravan)
serviced shelters (alpine_hut,
wilderness_hut appears more to be an unserviced shelter?

shop-icon:

ac39ac; #ac39ac
Shop (wiki) A place selling retail products or services.
shops, markets
driving_school
hygiene services (hair dresser, nail studio)

health-color:

BF0000; #BF0000
changed from da0092; #da0092 in #3498
Hospital, Pharmacy, doctors, vet, clinic, now also dentist and lots of healthcare=* stuff

gastronomy:

C77400; #C77400 merged in #3360
Services where ready-to-eat food and drinks are served
restaurant, fastfood, pub, cafe, bar,
including those servings in a cultural context (nightclub)

religious black

000000 (needs a variable on top of file) #000000
Place of worship, man-made=cross, wayside_cross

power black

"black" (needs a variable on top of file) #000000
generator

water-color:

fountain
waterfall (as water-text)

office:

4863A0; #4863A0
office=*

man-made-icon: = advertising-grey:

555; #555555
mast, various towers, chimney, city_gate, ad column, lighthouse
bunker
tourism=board,guidepost (don't see the rendering)
wayside_shrine (should that be religious?)
windmill
hunting_stand
unattended amenities (moved from brown in #3586) (amenity_shelter, bench, waste_disposal, waste_basket, picnic_table)

leisure-green:

darken(@park, 60%); i.e. 0C8416 #0C8416
sport/play-related
water_park, beach_resort, fitness, dog_park, picnic_site
massage shop,
sauna
outdoor_seating
bird_hide

airtransport:

8461C4; #8461C4
helipad, aerodrome
ferry_terminal (why is that on air and not water or transport?)

landform-color:

d08f55; #d08f55
natural peak, volcano, saddle

undefined - marker-fill is missing

natural_cave_entrance (renders black)
mini_roundabout (renders 666666)
railway level crossing, thin line sampled as 4b4c4a probably from antialiasing with the halo
barrier=gate (also, 404040)

shades of grey

lift_gate, swing_gate 3f3f3f #3f3f3f
bollard, block, log 7d7c7c #7d7c7c

Open issues

@polarbearing
Copy link
Contributor Author

in discussion: craft #3126, bike repair station #3069 #3391

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 14, 2018 via email

@polarbearing
Copy link
Contributor Author

Thanks for the comments. I'd like to keep this discussion for the categories, not tuning a particular colour.

Wilderness_hut is similar to camping / caravan / alpine_hut

No it is particularly not. Wilderness hut is unstaffed and provides protection from the elements, much like amenity=shelter. An Alpine Hut is serviced and provides lodging and meals.

As for features using water, such as drinking_water, shower and public_bath, I'm not convinced to move them to water_colour, as the service character prevails over the medium. In car_wash the cars get wet as well, and it is more a service (which I'd put into shop colour).

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 14, 2018 via email

@Tomasz-W
Copy link

Tomasz-W commented Sep 14, 2018

@polarbearing Thanks for complex analysis of colours thing. I think it shows good why we need to recolour gastronomy key - as most of colours are predicted for 1-2 groups, amenity-brown is predicted for 4 groups of tags (!) at the moment. It's surely too much. I hope next step will be #3326 , as tags included in this issue are not memorials, not cultural-like and not a service.

@jeisenbe
Copy link
Collaborator

To consolidate the discussion here, on issue 3326, @Tomasz-W recommended changing unattended shelters and installations (shelter, bbq, bench, picnic_table, waste_basket, waste_disposal, recycling) to man-made-icon gray, because they are not cultural or public service related.

I see that outdoor_seating is under leisure-green. However, it is specifically used for outdoor seating at cafes, bars and restaurants: "A seating area, usually for the consumption of food and drink from neighbouring cafes and restaurants, often belonging to one or more of them, but not necessarily adjacent." wiki. Examples are beer gardens, restaurant terraces, food courts with shared seating.
So outdoor_seating should be gastronomy-color.

@Tomasz-W
Copy link

So outdoor_seating should be gastronomy-color.

It shouldn't. Outdoor seatings are parts of gastronomy points, not a gastronomy points itself. If we keep this logic, we should render trash can in front or a restaurant in garstronomy-orange.

@dieterdreist
Copy link

dieterdreist commented Sep 15, 2018 via email

@polarbearing
Copy link
Contributor Author

Started to update the top post.

@polarbearing
Copy link
Contributor Author

I'd like to discuss the shop definition "Shop (wiki) A place selling retail products or services" which we currently paint purple. There is discussion in #3391 about a self-service amenity, where I think that "selling services" includes:

Thus I conclude that the self-service amenity can be amenity-brown, while the car-repair needs to be shop-purple.

Another option is to bring all paid services to the colour where work is being done, which is dark-blue (currently for offices). Consequently, this would bring craft into dark-blue as well.

@Tomasz-W
Copy link

Tomasz-W commented Sep 15, 2018

Ok, I did a research of shop=* list (https://wiki.openstreetmap.org/wiki/Key:shop)

Here is the list of shops selling services (or goods + services): tailor, beauty, hairdresser, massage, tattoo, car_repair, copyshop, dry_cleaning, laundry

I like leisure-green for shop=massage, becuse it's intuitive. In shop violet, it would look like shop with massaging equipment.
I think we should be rather elastic, so I consider choosing icons colours basing only on certain tag key as a very bad path. We are making map for other people, not for robots. It all should be intuitive.
My vision is to extend amenity-brown to include not only public-services, but other services too. Then we could re-use some icons in different colour for shop with equipment and different colour for service-like shop/ craft. Examples: shop-violet shop=hairdresser_supply and amenity-brown shop=hairdresser ; shop-violet shop=photo and amenity-brown craft=photographer

@polarbearing
Copy link
Contributor Author

That does not take any load from the amenity-brown and would go in the opposite direction.
So what speaks against office-blue for those workplaces, offering paid services and crafts?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 16, 2018 via email

@polarbearing
Copy link
Contributor Author

We just assigned the gastronomy as a face-to-face interaction business their own colour, so I see no reason why other f2f services need to remain within amenity-brown.

@meased
Copy link
Contributor

meased commented Sep 16, 2018

Perhaps religious black, power black, man-made grey, and shades of grey should be merged into some sort of a "generic" category. These colors all look the same anyway.

@polarbearing
Copy link
Contributor Author

The categories should be driven semantically, not just grouped by having had a similar colour at some time. Religious is quite specific in my opinion and not generic. Grey and black look different to me.

@meased
Copy link
Contributor

meased commented Sep 16, 2018

Religious is quite specific in my opinion and not generic. Grey and black look different to me.

Perhaps the categories of grey and black should be defined? Regardless of intention, black seems to be used as a catch-all.

@polarbearing
Copy link
Contributor Author

Perhaps the categories of grey and black should be defined

Yes indeed. One outcome of the analysis was that there are a few hardcoded colours that should be grouped.

@jragusa
Copy link
Contributor

jragusa commented Sep 17, 2018

water-color should be unified: natural=waterfall uses water-text for both colour and icon whereas amenity=fountain uses marina-text.

@Tomasz-W
Copy link

Tomasz-W commented Sep 18, 2018

That does not take any load from the amenity-brown and would go in the opposite direction.
So what speaks against office-blue for those workplaces, offering paid services and crafts?

@polarbearing The thing is not to throw out of amenity-brown as much tags as possible, but to make a logical complex system. If we render public services in amenity-brown, it seems to be logical step to extend it also for non-public services (after deloading it by #3360 and #3326). When I see office-blue, I predict some place full of papers/ suits etc., and as I can understand your an argument that craft=* places are sometimes offices, they are rather an exeptions (please see: https://wiki.openstreetmap.org/wiki/Key:craft#Craft). That's the reason why I find office-blue very confusing for craft=* key.

@dieterdreist
Copy link

dieterdreist commented Sep 18, 2018 via email

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 18, 2018 via email

@polarbearing
Copy link
Contributor Author

@jragusa - the waterfall is natural, while the fountain is man-made like the marina.
Fountain even uses two colours, water for the basin and marina for the nozzles.

@Tomasz-W - I don't get why office-blue reminds you to paper and white-collar workers. Just because it was initially introduced for the office dots? It could easily be renamed in workplace-blue.

The general problem with the categorisation here is, that for any two objects A and B in the world you may find a criterion that binds them into one category AB. Then you add object C and find that you could create one category AC and one BC, so you end up with as many categories as objects. Adding more objects, your number of potential categories grows rapidly as (n*n-n)/2.

In consequence, it is good that we brainstorm about potential category definition, that's why I opened this thread, but we also need to consolidate them.

@Adamant36
Copy link
Contributor

Health & Safety sounds good to me.

@polarbearing, yeah true, but what's your point? I mean, are you saying that's a reason to not render them in the same category or something?

@kocio-pl
Copy link
Collaborator

We could simply have both categories and use the same color for them. 😄 It will make it easier to adapt if someone wants to.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 20, 2018 via email

@kocio-pl
Copy link
Collaborator

kocio-pl commented Sep 20, 2018

It all started with me trying just to make some sane internal categories for a clean code purposes. 😄 You can look at #2824, #2826, #2827 (leisure icon color extended), #2854 and #2855.

We can have more categories using the same color, it's even better to have @culture and @history variables than @culture_and_history, also because someone might use them for tuning colors.

@polarbearing
Copy link
Contributor Author

yeah true, but what's your point?

There's good brainstorming going on here currently. I don't have a fixed opinion for everything, I also like to listen :-)

@Adamant36
Copy link
Contributor

I agree. I just thought there might of been more behind it that I was missing for some reason.

On a related note, id love to see this issue help lead to the code being cleaner. The amenity file could really use some reorganzing and I wish there was some better documentation in the code about what things do.

@kocio-pl
Copy link
Collaborator

What exactly would you like to understand?

@Adamant36
Copy link
Contributor

Adamant36 commented Sep 20, 2018

Can any tag have its own stylesheet or do they have to be in the amenity-points file for some reason? Like if I wanted to clean up the amenity-points file by creating individual stylesheets for the none amenity tags in it, would that be doable or even have a benefit?

@Tomasz-W
Copy link

Tomasz-W commented Sep 20, 2018

I'm not sure about emergency-colour idea. We surely need separate colour for healthcare objects, and propably there will be a generic dots for healthcare=* tags in future - I'm worried that putting eg. healthcare=speech_therapist (see: #2938 (comment)) or amenity=dentist in the same category with amenity=fire_station or amenity=police_station would't be an improvement but rather a confusing mistake. Of course there are many categorisation methods, and these objects can be consider as more or less emergency-like if we want to categorise different aspects of human life, but for map categorisation purposes, I think they are not that close to each other to use the same colour for both of them.

@kocio-pl
Copy link
Collaborator

I don't think there will be great problems with splitting stylesheets, though I haven't tested it. You just need to include new ones in project.mml. It would make them editing stylesheets more clean, but I'm not sure how it would influence the performance - @pnorman, what do you think about it?

@Adamant36
Copy link
Contributor

Adamant36 commented Oct 10, 2018

@kocio-pl, that's what I was thinking. Is there anyone else that would know about the possible performance issues besides @pnorman? because I'd like to do it. It would make editing stylesheets easier to edit, especially for new contributors. Plus, Id like to add documentation at some point and it would help with that also.

@kocio-pl
Copy link
Collaborator

The problem is that we have no performance framework (see #1287), so no one really knows until production deployment shows if there are such problems or not. There are only some hints when somebody tries development deployment - for example @rrzefox volunteers to do that sometimes, but this is not permanent solution and results depend on the traffic.

@meased
Copy link
Contributor

meased commented Oct 10, 2018

@giggls made this script:

https://github.com/giggls/openstreetmap-carto-de/blob/master/scripts/render_single_tile.py

If we could get it work in the docker environment, we could hand select some tiles that exercise a feature, and then time how long the tiles take to render before and after the change. That should a very good indicator of performance.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 5, 2018

health-color is now #BF0000; (red), and includes dentists, vets, pharmacies, and many features under healthcare=*

@jeisenbe
Copy link
Collaborator

railway=tram_stop and railway=station use yet another shade of blue: @station-color: #7981b0;

@jeisenbe
Copy link
Collaborator

While the top post shows fountain and waterfall as "water-color", these icons currently use unique colors, as does the spring icon. There are 3 water-feature colors currently, and 3 other categories use blue (2 of the transportation-related icon classes, and offices):

  1. transportation-icon: #0092da - also used for the bus station icon, but this is set in the SVG file.
  2. office: #4863A0 - used for the office dots
  3. station-color: #7981b0 - used for train stations and tram stops
  4. marina-text: #576ddf - used for the fountain icon
  5. water-text: #4d80b3 - used for the waterfall icon
  6. The spring.svg file uses #7abcec

I would recommend creating a @water-icon: line in amenity-points.mss, and this color could be used for fountains, waterfalls and springs.

I'm also in favor of using the same color for station-color and transportation-icon, as discussed in #1127

@jeisenbe
Copy link
Collaborator

barrier=gate seems to be 3f3f3f with a ffffff halo, so same as lift_gate and swing_gate

<path d="M 0,0 6,0 6,5 0,5 z" fill="#ffffff" />
<path d="M 0,1 6,1 6,2 0,2 z" fill="#3f3f3f" />

Perhaps these and bollard, block, log (#7d7c7c) should all be combined with @man-made-gray.

@jragusa
Copy link
Contributor

jragusa commented Jan 14, 2019

@jeisenbe You forgot #6666ff for ferry routes. I agree to use a unique colour for all of these water features.

@kocio-pl
Copy link
Collaborator

I don't feel transport and accommodation should be decoupled. They are quite close to me and I would rather use separate color for craft for example.

@jeisenbe
Copy link
Collaborator

#6666ff for ferry routes

That's a linear features, more like roads or railways, so I wouldn't include it in the list of icon colors, but it's a good idea to keep it in mind while considering icon colors.

I would rather use separate color for craft for example.

I've got 2 new colors in #3641 that we can use, so one of them can be saved for craft=* and we can still add a new color for accommodation.

@jragusa
Copy link
Contributor

jragusa commented Jan 15, 2019

I agree with @kocio-pl I do not see the need to add a specific colour for accommodation but I guess that features with close colours could be simplified with a unique colour. Moreover I don't think that creating a complex colour palette is a good thing because it can disturb people looking at the maps. IMO I prefer creating a more uniform and simple colour scheme because it more easier to people to understand the features on the map.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 16, 2019

Another addition: #d40000 natural=volcano icon color

@jeisenbe
Copy link
Collaborator

Wind generators have been switched from black to man-made-icon
So the only black icons are natural=cave, places_of_worship icons and crosses.

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

10 participants