screenshots: re-render the whole set after the 0.1.2 height calibration (review) #27

Merged
zwitschi merged 2 commits from screenshots-review-0.1.2 into main 2026-11-08 16:44:00 +01:00
Owner

Review request — all screenshots, re-rendered

Whole set regenerated from current main content via Tools/render_screenshots.sh
after the v0.1.2 building-height calibration was adopted.

What changed since the last approved set

Round Change
2026-11-08 0.1.1 street geometry: sidewalk upstand tapered flush at the building edge, crossing ramps graded ≤ 8 %; demo-north-buildings camera re-pointed (was 9.5 km west, 85 % sky)
2026-11-09 (this PR) 0.1.2 height calibration: Mapbox-derived 3.1 m per level (was 3.2), overrides for three rear outbuildings, blockout regenerated and frontage re-exported — 6922 42.1 → 39.1 m, El Capitan Theatre 20.0 → 20.5 m

12 of 14 PNGs changed. review-demo-spawn-view.png and review-demo-street-level.png are byte-identical to the previous round, so the height change reads mainly on the frontage silhouettes.

Please look at

  • Shots 11–14 (demo): the block from the north sidewalk, street level, the crossing, and the two north buildings. Confirm the six frontage buildings keep their relative height ordering, and that 6922 still reads as the tall west end while the El Capitan Theatre stays the lower cinema — compare with Art/reference/hollywood/el-capitan-theatre-blade-sign-day-wikimedia-2008-10.jpg.
  • Shots 1–10 (review map): unchanged content, but re-rendered — flag any regression in ground contact, scale cubes, hero portals or shadow comparison.

State

Screenshots/README.md has the earlier approvals cleared and a per-round change table; the set is marked pending owner review. Tick rows as they pass (AGENTS.md visual-review rule).

ReviewMap.umap and DemoMap.umap carry the re-staged shot cameras.

## Review request — all screenshots, re-rendered Whole set regenerated from current `main` content via `Tools/render_screenshots.sh` after the `v0.1.2` building-height calibration was adopted. ### What changed since the last approved set | Round | Change | | --- | --- | | 2026-11-08 | `0.1.1` street geometry: sidewalk upstand tapered flush at the building edge, crossing ramps graded ≤ 8 %; `demo-north-buildings` camera re-pointed (was 9.5 km west, 85 % sky) | | **2026-11-09 (this PR)** | `0.1.2` height calibration: Mapbox-derived **3.1 m per level** (was 3.2), overrides for three rear outbuildings, blockout regenerated and frontage re-exported — **6922 42.1 → 39.1 m**, **El Capitan Theatre 20.0 → 20.5 m** | **12 of 14 PNGs changed.** `review-demo-spawn-view.png` and `review-demo-street-level.png` are byte-identical to the previous round, so the height change reads mainly on the frontage silhouettes. ### Please look at - **Shots 11–14** (demo): the block from the north sidewalk, street level, the crossing, and the two north buildings. Confirm the six frontage buildings keep their relative height ordering, and that 6922 still reads as the tall west end while the El Capitan Theatre stays the lower cinema — compare with `Art/reference/hollywood/el-capitan-theatre-blade-sign-day-wikimedia-2008-10.jpg`. - **Shots 1–10** (review map): unchanged content, but re-rendered — flag any regression in ground contact, scale cubes, hero portals or shadow comparison. ### State `Screenshots/README.md` has the earlier approvals cleared and a per-round change table; the set is marked **pending owner review**. Tick rows as they pass (AGENTS.md visual-review rule). `ReviewMap.umap` and `DemoMap.umap` carry the re-staged shot cameras.
zwitschi added 1 commit 2026-11-08 16:42:16 +01:00
All 14 shots regenerated from current main content after adopting the Mapbox
calibration (3.1 m per level, overrides for three rear outbuildings, regenerated
blockout + frontage). 12 of 14 PNGs changed; demo-spawn-view and
demo-street-level are byte-identical to the 0.1.1 round, so the change reads on
the frontage silhouettes.

README: approval states from the earlier rounds cleared and replaced with a
per-round change table; the whole set is marked pending owner review; 'what to
look for' now asks to check 6922 / El Capitan Theatre heights against the
blade-sign reference and confirm the six-building height ordering.

ReviewMap/DemoMap carry the re-staged shot cameras.
zwitschi added 1 commit 2026-11-08 16:43:57 +01:00
Adds a per-row approval column to the shot table (AGENTS.md: one row per
screenshot with an approval state) and marks all 14 shots approved by the owner
on 2026-11-09 following PR #27.
zwitschi merged commit 640bff9631 into main 2026-11-08 16:44:00 +01:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: zwitschi/gig-line#27