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

raster tiles blurry #963

Closed
incanus opened this issue Mar 7, 2015 · 11 comments
Closed

raster tiles blurry #963

incanus opened this issue Mar 7, 2015 · 11 comments
Assignees
Labels
bug iOS Mapbox Maps SDK for iOS
Milestone

Comments

@incanus
Copy link
Contributor

incanus commented Mar 7, 2015

Possibly related to #924, but raster tiles are blurry on device.

img_3099

The tile borders are 2x larger than they should be in debug mode.

img_3100

In addition, sometimes it looks like maybe two different zoom levels are being drawn together?

img_3101

/cc @1ec5 @jfirebaugh

@incanus incanus added bug iOS Mapbox Maps SDK for iOS labels Mar 7, 2015
@incanus incanus added this to the iOS Beta milestone Mar 7, 2015
@1ec5
Copy link
Contributor

1ec5 commented Mar 7, 2015

Interesting. Things look pretty sharp at z13+. Need to see which tiles are actually being fetched.

@1ec5
Copy link
Contributor

1ec5 commented Mar 7, 2015

@yhahn points out that 2× satellite tiles are currently upsampled at low zoom levels. @incanus, please compare with this online preview and close if you’re satisifed that it isn’t a software issue.

@incanus
Copy link
Contributor Author

incanus commented Mar 9, 2015

Weird. We didn't used to have this problem.

@incanus
Copy link
Contributor Author

incanus commented Mar 9, 2015

The tile borders are 2x larger than they should be in debug mode.

Here's specifically what I'm seeing:

https://dl.dropboxusercontent.com/u/575564/rasterblur.mov

You can see things looking ok at 15, 16, back out to 15, 14, 13, and then 12 looks like crap, as does anything lower-numbered out to 8, when I stop.

This is a retina iPad, so the screen is 768x1024 virtual (1536x2048 actual). However, based on the tile borders, these are 512 raster tiles being shown in 512 of virtual space. In reality, we should be seeing 4x as many tiles on screen as we are. In the old, 256 raster days, we would see 16x as many tiles.

@jfirebaugh
Copy link
Contributor

The current behavior is expected: for z in the interval [2, 3) we load z2 vector tiles and z3 raster tiles. At z2.0000 you see 512px @2x raster tiles in 256px of virtual space. At z2.9999, you see 512px raster tiles in ~512px of virtual space. Whether we want to change that is being discussed in https://github.com/mapbox/mapbox-gl-js/issues/1030.

@incanus
Copy link
Contributor Author

incanus commented Mar 12, 2015

Confirmed by:

  1. Setting starting zoom to an integer.
  2. Only using double-tap or two-finger single-tap to zoom, keeping integer zoom.

That things are not blurry when at integer zooms.

I put in my vote over in mapbox/mapbox-gl-js#1030.

@jfirebaugh
Copy link
Contributor

Per scrum discussion, we'll address this in a followup to the initial iOS Beta release.

@jfirebaugh jfirebaugh removed this from the iOS Beta milestone Mar 12, 2015
@incanus incanus added this to the iOS Beta 2 milestone Mar 12, 2015
@peterqliu
Copy link
Contributor

Main discussion is still over at mapbox/mapbox-gl-js#1030, but something more relevant for mobile:

Since this issue is really a tradeoff between image sharpness and tile loading time, can we detect the device's current connection quality/network speed and make the appropriate call? When slow, opt for blurry and stretched (Math.floor), so the lag wouldn't be terribad; when fast, try to get more sharpness.

@incanus
Copy link
Contributor Author

incanus commented May 26, 2015

A good way to evaluate how things are looking here as well is to use older, raster styles (not satellite) and evaluate the text legibility. We should be striving for this for backwards compatibility / use with other stacks and it can be easier to see blurriness / sharpness in these instances rather than straight-up satellite rasters.

@1ec5
Copy link
Contributor

1ec5 commented May 27, 2015

The approach taken in mapbox/mapbox-gl-js#1221 doesn’t seem to have any effect in mapbox-gl-native.

@incanus
Copy link
Contributor Author

incanus commented Jun 1, 2015

Raster tiles aren't a focus of b2; pushing.

@incanus incanus modified the milestones: iOS Beta 3, iOS Beta 2 Jun 1, 2015
1ec5 added a commit that referenced this issue Jul 7, 2015
@1ec5 1ec5 mentioned this issue Jul 7, 2015
1ec5 added a commit to mapbox/mapbox-gl-styles that referenced this issue Jul 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

No branches or pull requests

4 participants