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

look for loaded parent tiles in cache #2158

Closed
wants to merge 1 commit into from

Conversation

ansis
Copy link
Contributor

@ansis ansis commented Feb 19, 2016

When the required tiles aren't loaded yet it tries to find parent tiles that area loaded that a cover the area. It used to just look for parents in the set of previously rendered tiles. There are many cases where the parents aren't in that set but they are in the cache.

This reduces the amount of the viewport not covered by tiles. It makes tile loading look a lot faster.

It helps a lot when you:

  • zoom in and the pan around
  • zoom in quickly and then zoom out more slowly

fixes #1918


This only loads parents, not children. I tried also loading children but it looked weird sometimes. Try the
load-children-from-cache branch to see what it's like.

If children are loaded from the cache you sometimes see flashes of high resolution tiles while zooming in. Seeing the area change from too-low-resolution to too-high-resolution to just-right-resolution is more jarring than just too-low-resolution to just-right-resolution.

👀 @mourner

This reduces the amount of the view not covered by tiles.
It makes tile loading look a lot faster.

It helps a lot when you:
- zoom in and the pan around
- zoom in quickly and then zoom out more slowly

ref #1918
@mourner
Copy link
Member

mourner commented Feb 19, 2016

👍 for this PR, will look at the children branch later, but I'd guess we can leave it — loading parent tiles is much more important for smooth user experience.

@ansis
Copy link
Contributor Author

ansis commented Feb 20, 2016

merged into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants