Skip to content

perf: compute the tile cover once per camera - #2288

Open
greensopinion wants to merge 1 commit into
pitch-5-tile-level-of-detailfrom
pitch-6-tile-cover-once
Open

greensopinion wants to merge 1 commit into
pitch-5-tile-level-of-detailfrom
pitch-6-tile-cover-once

Conversation

@greensopinion

Copy link
Copy Markdown
Collaborator

The tile layer computed the cover, its buffer and its keep set for both
the build and the tile update event of each camera, and the separating
axis test allocated for every tile it tested. While rotating at 60° on a
Pixel 4a, the median build time per frame drops from 7.3 ms to 4.7 ms;
the tiles loaded don't change.

Depends on #2287.

I wrote this with Claude Code (Claude Opus 5.5). I directed the design,
reviewed the code, and tested it on a Mac, an iPhone 13, an iPhone 15 Pro
Max, a Pixel 4a (5G), and in Chrome.

The tile layer computed the cover, its buffer and its keep set for both
the build and the tile update event of each camera, and the separating
axis test allocated for every tile it tested. While rotating at 60° on a
Pixel 4a, the median build time per frame drops from 7.3 ms to 4.7 ms;
the tiles loaded don't change.

This branch has not been deployed

No deployments
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.

1 participant