mapbox / osm-navigation-map

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

Use mapillary js 2.x #104

Closed batpad closed 7 years ago

batpad commented 7 years ago

We probably need to do this at some point, and it maybe slightly non-trivial: https://github.com/mapillary/mapillary-js/blob/master/MIGRATING.md

@srividyacb can you mention a bit why we need this?

@ajithranka is it possible to get a sense of how much work this might be / how much is it worth doing / and if we need to prioritize / figure out team bandwidth to get this done?

srividyacb commented 7 years ago

@batpad @ajithranka

We have been seeing many incorrect detections in Mapillary Vector layer - as mentioned here. Per conversation with Mapillary folks in mapping ticket, these incorrect detections are due to older version of Mapillary JS in navigation-map - here.

To get the right detection on the navigation-map and to add right turn-restriction to OpenStreetMap, we might have to get this migrated to new Mapillary JS which will have better signage detection layer and reduce false detection during mapping.

cc @maning