Skip to content

chore: limit Cesium building tile cache - #1596

Open
rotwurstesser wants to merge 1 commit into
developfrom
chore-limit-cesium-building-cache
Open

chore: limit Cesium building tile cache#1596
rotwurstesser wants to merge 1 commit into
developfrom
chore-limit-cesium-building-cache

Conversation

@rotwurstesser

@rotwurstesser rotwurstesser commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

Limit the Cesium Swissbuildings3D tile cache to 256 MiB with 128 MiB overflow. This bounds retained building tile content without changing screen-space error or active rendering detail. Other Cesium tilesets keep their defaults.

Benchmark

  • Paired the deployed PR against the same build with Cesium's 512/512 MiB defaults restored at runtime
  • Covered 16 locations, repeated nationwide eviction/revisit passes, rapid combined navigation, seeded random navigation, and building-layer recreation
  • Peak building-tile payload: 255.99 MiB capped versus 511.97 MiB control (50% lower)
  • Screen-space error stayed at 10; 95/96 paired location checkpoints had identical selected building triangles and draw commands
  • The remaining checkpoint was a non-reproduced camera landing issue; a focused follow-up passed 60/60 landings with each cache setting
  • No Cesium render errors, page exceptions, browser crashes, WebGL context losses, or unhandled rejections
  • Mapviewer unit tests: 334 passed, 3 skipped; ESLint and existing CI pass

This verifies the Cesium building-tile payload reduction and functional equivalence for the tested scenarios. It does not claim a 50% reduction in total browser RSS.

Test link

Test link

@github-actions github-actions Bot added the skip-release-note Issue or PR excluded from the release notes label Aug 20, 2026
@cypress

cypress Bot commented Aug 20, 2026

Copy link
Copy Markdown

web-mapviewer    Run #6617

Run Properties:  status check passed Passed #6617  •  git commit 227266d5f5: chore: limit Cesium building tile cache
Project web-mapviewer
Branch Review chore-limit-cesium-building-cache
Run status status check passed Passed #6617
Run duration 04m 58s
Commit git commit 227266d5f5: chore: limit Cesium building tile cache
Committer Raphi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 222
View all changes introduced in this branch ↗︎

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

Labels

skip-release-note Issue or PR excluded from the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants