Skip to content

TilesRenderer: Make optimized traversal the default #1426

@gkjohnson

Description

@gkjohnson

Related to #1421

To Confirm / Fix

  • Consider adding option so the "optimized" path behaves like the original traversal approach. "loadParents" option, for example.
  • Test with a more complex additive tile set
  • Fix tile splitting for QuantizedMeshPlugin, ImageOverlaysPlugin
  • Fix auto-expanding children (QuantizedMeshPlugin, ImageFormatPlugins)
    • These do not work since they only expand the children once the tile itself is loaded and disposed of once the internal parents are disposed. That means we can get into an internal oscillation state when the parents are disposed, subsequently disposing of the children, causing that parent to render again.
    • Could implicitly add a tile with "unrenderable content" at some given number of layers so they re added / disposed in the same logic as internal tile sets. This won't work for tile splitting, though.
  • Expose variable indicating whether the tile set has full coverage or not
    • _allChildrenReady may almost be acceptable but it seems to switch to false even when all tiles are rendering.

Migration

  • Change "optimizedLoadStrategy" to "true" by default with a deprecation warning first
  • Remove previous load strategy option

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions