mapbox / osm-navigation-map

A map to explore navigation data in OpenStreetMap
ISC License
28 stars 17 forks source link

Style OSM turn restrictions better #63

Open planemad opened 8 years ago

planemad commented 8 years ago

Showing existing OSM turn restrictions on the map help to easily compare with Mapillary restrictions and add/update the data onto the map.

screenshot 2016-08-17 14 30 26

Current styling of OSM turn restriction from via and to members in red

@rasagy can you review the issues with this styling from a reviewer perspective and post your notes on how it should ideally look?

cc @andreasviglakis

rasagy commented 8 years ago

Summarising the insights from the design sprint that I will try to take forward:

The proposed design that we can explore further, since data-driven styling is now available for icon-rotate:

final-view - 3

Specific feature requests in this design: (Will sync up with @ajithranka & @planemad on what to prioritise)

To do on this design:

You can play around with this map style here.

rasagy commented 8 years ago

I spent some time with @srividyacb in observing how she is mapping/verifying Turn Restrictions. Key insights:

I’ll be looking back at this again with fresh eyes to explore what can be the actionable steps from the observation.

Here are the details that I noticed during the shadow exercise:

  1. Sri finds a cluster of detected signages on the OSM Nav map, and checks the image thumbnail on one of them.
  2. Most often, the image in Nav map is opened in a new tab (by clicking on it) to quickly zoom into one part (verify TR is visible, and spot road name signage if possible), then JOSM is opened.
  3. On opening JOSM, she sometimes toggles to the OSM view (instead of satellite view) to confirm if she is on the correct junction (by verifying street names)
  4. She uses the Mapillary picture plugin and hovers on the photo locations to guess if the photo will have the TR & street info based on the thumbnail.
  5. She opens images (and switches next/back) to confirm which restrictions are where — changing (loading) images here takes a lot of time, and there is quite a bit of back and forth to find legible street names.
  6. The Mapillary pictures plugin tends to cover the map, so she sometimes drags the map around to keep a track of where the photo is taken on the map.
  7. The icons in JOSM generally help in quickly noticing if a TR is mapped. However, sometimes icons get overlapped (like No-U-Turn & No-Left-Turn on same street), so this isn’t always reliable.
  8. Even if the relation is already mapped (as above), she still double checks the relation details to confirm if the from, via, to are mapped correctly. She edits the relation and clicks on each member to see it visually highlighted (red blur) on the map.
  9. If the TR is not mapped, she adds them on JOSM.
  10. Sometimes, the TR is correctly detected but is for a one-way street (these are marked redundant).
  11. She removes the layers (for Mapillary, OSM etc.) from JOSM before continuing.
  12. She switches back to the browser, closes the image tab that was opened.
  13. She marks the particular node/photo as valid/redundant/invalid on Nav map. She doesn’t do that for every node that detects a signage, only the one that was the most clear (at times multiple).
planemad commented 8 years ago

Some prior art: http://restrictions.morbz.de/# which helps visualize OSM turn restriction. Its not very clear which are the from and to members at junctions with multiple restrictions.

screenshot 2016-08-17 18 35 02 screenshot 2016-08-17 18 34 57
rasagy commented 8 years ago

Based on voice w/ @planemad & @ajithranka.

We need to style the following restrictions visually to implement the original style, however the properties in bold aren’t available for data-driven gl-js:

way

node

For now, we’ll use two layers for way (one layer for from and one layer for to_no, to_only and via — differentiated by color) and one for node

Design tasks that I’ll be looking at today:

rasagy commented 8 years ago

Updated the style to a simpler one which has the same properties for all to & via ways as discussed above:

tr-v6

However, @ajithranka felt that we can retain individual layers for each case, so we’re sticking to the older style (with a few tweaks).

planemad commented 8 years ago

@rasagy we now have turn restrictions loaded dynamically after z14. The style is not really legible against the dark background.

screenshot 2016-08-29 16 16 00 Spot the OSM restrictions

cc @ajithranka

rasagy commented 8 years ago

@planemad Agreed, synced up with @ajithranka on the same. I’ll be reworking the style since I was using Mapbox Light as a base map & this map is a mix of dark + satellite view. Also need to sync up on prioritising some features, will put down my ideas and then chat further on this.

planemad commented 8 years ago

👍 on using light

rasagy commented 8 years ago

Based on the discussions, these were the action points:

rasagy commented 8 years ago

Based on discussion with @bsrinivasa, hiding the old TR style on zoom>14, and for zoom<14 showing only the via node styled in the new color scheme so we still get an overview.

PS: Once the new styles are pushed, we’ll share a documentation of the style with the team and add it to the wiki for the tool.

rasagy commented 8 years ago

Hi @mapbox/india-office,

We’ve been working on updating the osm-navigation-map style to reduce the clutter & noise.

Specifically:

Here’s how the new design is looking:

Overview:

screenshot 2016-09-01 16 58 55

Zoomed in view:

screenshot 2016-09-01 17 00 44

No left turn (from: yellow, via: red node, to: thin dashed red)

screenshot 2016-09-01 17 03 07

Only right turn (from: yellow, via: blue node, to: dashed blue)

screenshot 2016-09-01 17 03 44

Would love to hear if this is working or not, and what is hindering our workflow on this map. :)

planemad commented 8 years ago

@rasagy loving this, definitely makes it visually simpler.

rasagy commented 8 years ago

@planemad Thanks!

A big help will be a small graphic legend to help interpret turn restriction representation. Since it is new, the colors and symbology are not really intuitive for a mapper (This could be on the readme for now)

Makes sense. If there are no other changes by tomorrow, then I’ll update the instructions.

Did not notice the dashes in the blue, seems to have a smaller gap spacing than the red

The idea was that the red way is something you can not drive to, so it is dashed, however the blue way is something you can only drive to, so it has minor gap. I was hoping to keep all three way line styles different (not just color), hence this approach. If this is more confusing than helpful, then I can see if increasing the gap or removing gap altogether helps.

rasagy commented 8 years ago

@bsrinivasa pointed out how at high zoom levels, the validation orbs clash with the via nodes (red & blue). Updated the map so the via nodes are enlarged only after z12. Thanks Bharata!