fix(webapp): large maps load from a tile pyramid; rc2 review fixes (#1282, #1283) #1284

Merged

claude-bot merged 4 commits from fix/4.6.2-rc3 into main 2026-10-30 05:45:27 +00:00

Conversation 0 Commits 4 Files changed 16 +330 -32

claude-bot commented 2026-10-30 04:12:49 +00:00

Collaborator

Copy link

Closes #1282. Closes #1283.

For 4.6.2-rc3, on the owner's say (2026-10-30: "get issues opened up for what I found, fix them, and get rc3 cut").

#1282: large maps load from a pyramid

• map_tiles.lowest_zoom / cut_lower_levels / write_lower_levels: after zoom 0, the worker cuts the picture at half size, a quarter and so on (LANCZOS, each level made from the one above) until it fits in one tile. An 8192 × 6144 map goes down to z = -5. levels.json ({"min_zoom": -n}) is written last, atomically, to say the levels are complete.
• files_service.map_lowest_zoom reads it, returning 0 when it's missing or unreadable and clamping to -16..0. The map response carries tiles_min_zoom. The tile route refuses z outside -16..0 or a negative x/y before touching the disk.
• MapCanvas sets minNativeZoom to tiles_min_zoom, so a fitted view reads that level's few tiles, and draws the 1024 px thumbnail underneath, removed on error. In the canvas test, a 3200 × 2400 map fitted to 800 × 600 reads fewer than 20 tiles at z = -2 instead of the 130 native ones.
• backfill_map_thumbs also cuts the levels of any ready map without levels.json (it returns a levels count). It's already on the daily schedule, and the deploy runs it once.

#1283: from the owner's rc2 review

• A trail group's summary shows Show or Hide (CSS on [open]), and choosing a stop's place on the map opens every group holding its stops.
• The "Add a stop" marker searches list nothing until something is typed, except a portal's linked exits. The spot option reads "Or choose a spot on the map…".

Checked

• Backend on docker-test: map tiles, maps API and edit API, pictures and files, the EXIF/bomb guard, export files and maps, and one-job-per-import map tasks: 855 passed, after one tile-count expectation was updated. ruff format and check pass.
• Frontend: vitest over src/pages/wiki, src/wiki and src/pages/prep: 654 passed, with new tests for the pyramid level read and the thumbnail, nothing listed before typing, Show/Hide, and a place opening its session. eslint reports 0 errors.
• changelog.py check passes.

Not checked

• Exadia's real load time is measured on dev after rc3 is deployed and the backfill has run.

🤖 Generated with Claude Code

Closes #1282. Closes #1283. For 4.6.2-rc3, on the owner's say (2026-10-30: "get issues opened up for what I found, fix them, and get rc3 cut"). ### #1282: large maps load from a pyramid - `map_tiles.lowest_zoom` / `cut_lower_levels` / `write_lower_levels`: after zoom 0, the worker cuts the picture at half size, a quarter and so on (LANCZOS, each level made from the one above) until it fits in one tile. An 8192 × 6144 map goes down to z = -5. `levels.json` (`{"min_zoom": -n}`) is written last, atomically, to say the levels are complete. - `files_service.map_lowest_zoom` reads it, returning 0 when it's missing or unreadable and clamping to -16..0. The map response carries `tiles_min_zoom`. The tile route refuses z outside -16..0 or a negative x/y before touching the disk. - `MapCanvas` sets `minNativeZoom` to `tiles_min_zoom`, so a fitted view reads that level's few tiles, and draws the 1024 px thumbnail underneath, removed on error. In the canvas test, a 3200 × 2400 map fitted to 800 × 600 reads fewer than 20 tiles at z = -2 instead of the 130 native ones. - `backfill_map_thumbs` also cuts the levels of any ready map without `levels.json` (it returns a `levels` count). It's already on the daily schedule, and the deploy runs it once. ### #1283: from the owner's rc2 review - A trail group's summary shows Show or Hide (CSS on `[open]`), and choosing a stop's place on the map opens every group holding its stops. - The "Add a stop" marker searches list nothing until something is typed, except a portal's linked exits. The spot option reads "Or choose a spot on the map…". ### Checked - Backend on `docker-test`: map tiles, maps API and edit API, pictures and files, the EXIF/bomb guard, export files and maps, and one-job-per-import map tasks: **855 passed**, after one tile-count expectation was updated. ruff format and check pass. - Frontend: vitest over `src/pages/wiki`, `src/wiki` and `src/pages/prep`: **654 passed**, with new tests for the pyramid level read and the thumbnail, nothing listed before typing, Show/Hide, and a place opening its session. eslint reports 0 errors. - `changelog.py check` passes. ### Not checked - Exadia's real load time is measured on dev after rc3 is deployed and the backfill has run. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 2 commits 2026-10-30 04:12:50 +00:00

fix(webapp): large maps load from a tile pyramid; rc2 review fixes (#1282, #1283) ... 8e753dab27

#1282: the worker cuts each map at half size, a quarter and so on until it
fits one tile, and writes levels.json last to say the levels are complete.
The map response carries tiles_min_zoom (0 without levels.json), and
MapCanvas reads that level when zoomed out, over the thumbnail drawn at
once. backfill_map_thumbs also cuts the levels of maps cut before. The tile
route refuses a z outside -16..0 or a negative x/y before the disk.

#1283: a folded trail group says Show/Hide and opens when its stop's place
is chosen on the map; the Add a stop searches list nothing until typed,
apart from a portal's linked exits.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(webapp): an imported 300 px map is cut at two levels (#1282) ... 422f786d43

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot scheduled this pull request to auto merge when all checks succeed 2026-10-30 04:12:57 +00:00

rbrooks added 2 commits 2026-10-30 05:10:24 +00:00

test(webapp): a map's files include its lower zoom levels (#1282) ... 114b047951

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(webapp): a map stored upright is cut at its lower levels too (#1282) ... ddc2e50090

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot merged commit d9a8206322 into main 2026-10-30 05:45:27 +00:00

claude-bot deleted branch fix/4.6.2-rc3 2026-10-30 05:45:28 +00:00

claude-bot referenced this pull request from a commit 2026-10-30 05:45:28 +00:00

Merge pull request 'fix(webapp): large maps load from a tile pyramid; rc2 review fixes (#1282, #1283)' (#1284) from fix/4.6.2-rc3 into main

claude-bot referenced this pull request 2026-10-30 05:45:58 +00:00

chore(release): v4.6.2-rc3 #1285

claude-bot referenced this pull request 2026-10-30 18:32:59 +00:00

Map: what the plan has that the build lacks (audit, 2026-10-28) #1263

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

backend

Issue related to the backend

  

bot

Issue related to the bot

  

bug

Something is not working

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

frontend

Issue related to the frontend

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

recording

Issue related to the recording system

  

security

Security hardening / vulnerability work

  

wontfix

This won't be fixed

No labels

backend

bot

bug

duplicate

enhancement

frontend

help wanted

invalid

question

recording

security

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

rbrooks/Quest-Board!1284

No description provided.