Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Mapbox Satellite tiles should be overzoomed when not available #8151

Closed
ericrwolfe opened this issue Feb 21, 2017 · 1 comment
Closed

Mapbox Satellite tiles should be overzoomed when not available #8151

ericrwolfe opened this issue Feb 21, 2017 · 1 comment
Labels
bug Core The cross-platform C++ core, aka mbgl

Comments

@ericrwolfe
Copy link
Contributor

Satellite overzooming seems to be inconsistent, with areas outside of some metro areas returning empty black tiles when loading tiles z >= 18.

satellite overzoom

Example cities where overzooming works: San Francisco, Los Angeles, Austin, Mexico City, Paris, Venice.
Example cities where overzooming fails: Vancouver BC, Whistler BC, Lima Peru, Alamogordo NM, Mount Shasta CA, Truckee CA. (These same cities aren't an issue in gl-js)

A tile from an example failure area & zoom level returns a 404 and a json response:

GET https://api.mapbox.com/v4/mapbox.satellite/18/41541/[email protected]?access_token=ACCESS_TOKEN&events=true
HTTP/1.1 404 Not Found
Content-Type: application/json; charset=utf-8
{
    "message": "Tile does not exist"
}

In the major cities where overzooming does work, it seems the mapbox API does the overzooming itself:

GET https://api.mapbox.com/v4/mapbox.satellite/19/83865/[email protected]?access_token=ACCESS_TOKEN&events=true
HTTP/1.1 200 OK
Content-Type: image/jpeg
@friedbunny friedbunny added bug Core The cross-platform C++ core, aka mbgl labels Feb 21, 2017
@kkaefer
Copy link
Contributor

kkaefer commented Feb 22, 2017

These same cities aren't an issue in gl-js

gl-js has a slightly different, but also incorrect behavior: it retains the most recent available tiles when zooming in, but doesn't actually load overzoomed lower zoom level tiles when starting out or panning into regions with no tile availability: mapbox/mapbox-gl-js#4318

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Core The cross-platform C++ core, aka mbgl
Projects
None yet
Development

No branches or pull requests

3 participants