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

Use Mapillary vector tiles in order to get to show all images #3152

Open
peterneubauer opened this issue Jun 8, 2016 · 1 comment
Open
Labels
performance Optimizing for speed and efficiency streetlevel An issue with streetlevel photos

Comments

@peterneubauer
Copy link
Contributor

In the current implementation of creating a layer for every Mapillary image, we have a limit of 1000 images per request (tile) to the mapillary API, resulting in empty spots or images that are not shown, like in https://www.openstreetmap.org/node/244646675#map=16/-41.4369/147.1385 where there is a lot of coverage.

It would be better to use the Mapillary vector tiles, like e.g. https://d2munx5tg0hw47.cloudfront.net/tiles/14/8786/5127.mapbox that gives sequences and image location/compass angle data.

I don't know exactly how to integrate vector tiles with the D3 transforming scheme, so help from someone more skilled than me would be hugely appreciated!

@bhousel
Copy link
Member

bhousel commented Jun 8, 2016

Yeah, this is a good next step. We are currently fetching up to 10 pages of data per z14 tile from the paginated API.

Before f2cde0b we fetched all the data, but this ended up being too much around Knoxville, TN 😄 There are probably a few million images there. I've also noticed Mapillary JS viewer timeouts around there, maybe it is not able to efficiently cache nearby images.

But yeah, lets switch over to the vector tiles!

@bhousel bhousel added performance Optimizing for speed and efficiency and removed enhancement labels Nov 1, 2016
@bhousel bhousel mentioned this issue Jan 11, 2017
@bhousel bhousel added streetlevel An issue with streetlevel photos and removed new-feature A new feature for iD labels Oct 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Optimizing for speed and efficiency streetlevel An issue with streetlevel photos
Projects
None yet
Development

No branches or pull requests

2 participants