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

rethink partial tiles #2851

Closed
jfirebaugh opened this issue Oct 28, 2015 · 6 comments
Closed

rethink partial tiles #2851

jfirebaugh opened this issue Oct 28, 2015 · 6 comments
Assignees
Labels

Comments

@jfirebaugh
Copy link
Contributor

The logic around partial tiles is complex and brittle. We need to rethink this, maybe do it in a totally different way.

cc @tmpsantos @kkaefer

@kkaefer
Copy link
Member

kkaefer commented Oct 29, 2015

What aspect of it is causing problems for you?

@jfirebaugh
Copy link
Contributor Author

Our current implementation is highly stateful, and it is difficult to reason about the order or timing of state transitions. This has given rise to pattern of issues where tiles either never finish parsing or are parsed ad infinitum:

These issues are often very difficult and time consuming to debug.

Yet despite its complexity, our current strategy is still inefficient:

I'm not sure what the right approach is yet, however.

@tmpsantos
Copy link
Contributor

We need a way of making tiles subscribe to the resources they need and get notified when they arrived. And I'm all in for getting rid of tile states as they are today. They will probably still exist, but IMO they could be inferred at runtime.

@kkaefer
Copy link
Member

kkaefer commented May 25, 2016

Partially addressed by #5139

@tmpsantos
Copy link
Contributor

Partially addressed by #5139

👍

@jfirebaugh
Copy link
Contributor Author

Lots of refactoring here lately. Fixing #667 will likely fix #2511 and then things will be pretty clean.

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

No branches or pull requests

4 participants