Use TileRenderer to render tiles from a VectorSource #351

Closed
ahocevar opened this Issue Mar 12, 2013 · 5 comments

Comments

Projects
None yet
5 participants
@ahocevar
Owner

ahocevar commented Mar 12, 2013

This may require some architectural changes in the way the tile renderers request tiles, but if we achieve this we can get rid of specific vector layer renderers.

@ahocevar ahocevar referenced this issue Mar 12, 2013

Merged

Vector work #218

@tschaub

This comment has been minimized.

Show comment Hide comment
@tschaub

tschaub Mar 15, 2013

Owner

One of the main reasons we didn't do this before was because tile renderers request a single tile at a time. For efficient vector rendering we want to render more than a single tile at a time. Going back to the idea of a tile source that can provide a range of tiles for requested bounds would make this work.

Owner

tschaub commented Mar 15, 2013

One of the main reasons we didn't do this before was because tile renderers request a single tile at a time. For efficient vector rendering we want to render more than a single tile at a time. Going back to the idea of a tile source that can provide a range of tiles for requested bounds would make this work.

@twpayne

This comment has been minimized.

Show comment Hide comment
@twpayne

twpayne Mar 15, 2013

Contributor

Suggestion: tile renderers give hints to their layers:

this.getTileSouce().hintWillRequestTliesInRange(tileRange);

This can be a no-op by default, but clever sources (like a vector tile source) can use it to be super-efficient.

Contributor

twpayne commented Mar 15, 2013

Suggestion: tile renderers give hints to their layers:

this.getTileSouce().hintWillRequestTliesInRange(tileRange);

This can be a no-op by default, but clever sources (like a vector tile source) can use it to be super-efficient.

@tschaub

This comment has been minimized.

Show comment Hide comment
@tschaub

tschaub Mar 15, 2013

Owner

We'd also have to give hints about animation. The source should return cached tiles when transitioning between zoom levels.

Owner

tschaub commented Mar 15, 2013

We'd also have to give hints about animation. The source should return cached tiles when transitioning between zoom levels.

@tsauerwein

This comment has been minimized.

Show comment Hide comment
@tsauerwein

tsauerwein Jul 3, 2014

Member

@ahocevar Can this issue be closed, now that there is ol.source.TileVector?

Member

tsauerwein commented Jul 3, 2014

@ahocevar Can this issue be closed, now that there is ol.source.TileVector?

@elemoine

This comment has been minimized.

Show comment Hide comment
@elemoine

elemoine Jul 3, 2014

Owner

Yes, this ticket is outdated. Closing.

Owner

elemoine commented Jul 3, 2014

Yes, this ticket is outdated. Closing.

@elemoine elemoine closed this Jul 3, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment