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

Reuse CanonicalTile in TilePyramid #10258

Closed
asheemmamoowala opened this issue Oct 20, 2017 · 5 comments
Closed

Reuse CanonicalTile in TilePyramid #10258

asheemmamoowala opened this issue Oct 20, 2017 · 5 comments
Labels
archived Archived because of inactivity Core The cross-platform C++ core, aka mbgl

Comments

@asheemmamoowala
Copy link
Contributor

asheemmamoowala commented Oct 20, 2017

Motivation

For over-zoomed and wrapped tiles, TilePyramid creates and maintains unique copies of the same canonical tile at multiple zooms and wraps. Each of these tiles needs a copy of the same raw data and has to re-parse it.

Reusing the tiles should allow caching more tiles and better performance when over-zooming.

For the ongoing custom vector sources, it would allow reusing the tile data without having to make a new fetchTile request.

Expectation

Raster tiles should be entirely reusable when over-zoomed. Once the raster data is parsed, the same bucket can be used for all zoom levels.

Vector tiles should reuse the raw data. The tile may hold separate buckets for different zoom levels, but may also be able to reuse the buckets when over-zooming.

cc @jfirebaugh @ChrisLoer @kkaefer

@asheemmamoowala asheemmamoowala added the Core The cross-platform C++ core, aka mbgl label Oct 20, 2017
@stale stale bot added the archived Archived because of inactivity label Nov 3, 2018
@stale
Copy link

stale bot commented Dec 3, 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 Dec 3, 2018
@asheemmamoowala
Copy link
Contributor Author

asheemmamoowala commented Jan 31, 2019

This is still useful to reduce memory consumption and improve performance

@stale stale bot removed the archived Archived because of inactivity label Jan 31, 2019
@stale stale bot added the archived Archived because of inactivity label Jul 30, 2019
@stale
Copy link

stale bot commented Jul 30, 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 Jul 30, 2019
@stale stale bot removed the archived Archived because of inactivity label Jul 31, 2019
@stale stale bot added the archived Archived because of inactivity label Jan 27, 2020
@stale
Copy link

stale bot commented Jan 27, 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 27, 2020
@alexshalamov alexshalamov reopened this Jan 27, 2020
@stale stale bot removed the archived Archived because of inactivity label Jan 27, 2020
@stale stale bot added the archived Archived because of inactivity label Jul 25, 2020
@stale
Copy link

stale bot commented Jul 26, 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 26, 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
Projects
None yet
Development

No branches or pull requests

2 participants