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

Load lower resolution tiles for tilted views #9037

Closed
kkaefer opened this issue May 18, 2017 · 6 comments · May be fixed by #15230
Closed

Load lower resolution tiles for tilted views #9037

kkaefer opened this issue May 18, 2017 · 6 comments · May be fixed by #15230
Assignees
Labels
archived Archived because of inactivity Core The cross-platform C++ core, aka mbgl feature needs discussion performance Speed, stability, CPU usage, memory usage, or power usage

Comments

@kkaefer
Copy link
Member

kkaefer commented May 18, 2017

We're currently loading tiles with a fixed zoom level for a particular view, regardless of whether it is tilted or not. This leads to too many tiles being loaded on the horizon, while potentially not enough tiles are loaded at the bottom of the viewport. This is especially true for raster images because they typically use smaller tile sizes.

screen shot 2017-05-18 at 17 52 41

Instead, we should be loading lower zoom level tiles for places that are further away from the camera, and potentially higher zoom level tiles that are closer to the camera, in particular when we implement #6908.

There are two ways for styles to handle this:

  • Render all tiles with the evaluated style of the current zoom level: This is what we do right now when showing parent/child tiles when zooming/panning while waiting for the actual tiles to load. It requires tiles to be somewhat coherent across zoom levels
  • Use a different zoom level throughout the screen: This means that we'll have to evaluate the stylesheet (based on zoom level) in the shader, and is probably out of scope for a first implementation.
@kkaefer kkaefer added feature needs discussion performance Speed, stability, CPU usage, memory usage, or power usage labels May 18, 2017
@kkaefer kkaefer added the Core The cross-platform C++ core, aka mbgl label May 19, 2017
@stale stale bot added the archived Archived because of inactivity label Nov 7, 2018
@stale
Copy link

stale bot commented Nov 28, 2018

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Nov 28, 2018
@1ec5 1ec5 removed the archived Archived because of inactivity label Dec 11, 2018
@1ec5 1ec5 reopened this Dec 11, 2018
@1ec5
Copy link
Contributor

1ec5 commented Dec 11, 2018

Needed for #6908. Reopening.

@stale stale bot added the archived Archived because of inactivity label Jun 9, 2019
@stale
Copy link

stale bot commented Jun 9, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Jun 9, 2019
@julianrex julianrex reopened this Jun 10, 2019
@stale stale bot removed the archived Archived because of inactivity label Jun 10, 2019
@1ec5
Copy link
Contributor

1ec5 commented Jul 18, 2019

Adaptive tile loading would be one way to address the runaway tile loading that occurs when increasing the top padding: #15163.

@stale stale bot added the archived Archived because of inactivity label Jan 20, 2020
@stale
Copy link

stale bot commented Jan 20, 2020

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Jan 20, 2020
@astojilj astojilj reopened this Jan 20, 2020
@stale stale bot removed the archived Archived because of inactivity label Jan 20, 2020
@stale stale bot added the archived Archived because of inactivity label Jul 18, 2020
@stale
Copy link

stale bot commented Jul 18, 2020

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Jul 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived Archived because of inactivity Core The cross-platform C++ core, aka mbgl feature needs discussion performance Speed, stability, CPU usage, memory usage, or power usage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants