docs(review): approve 11 and 12, split the hero facade into its own task

Owner verdicts: rows 11 (shadow compare) and 12 (vehicle staging) approved. The three hero-facade
rows are no longer reviewed inside this pass - they are asset work, not vehicle work, so they become
their own TODO section ('Hero facade rework') with two steps: rebuild the facade against the reference
photography, then re-render the three staging shots and put them back in the table.

The table rows point at that task instead of repeating 'rework needed'.

Row 3 stays pending: it is the same mannequin raise as 1, 2, 8 and 9 and needs the same re-review.
This commit is contained in:
zwitschi committed 2026-11-09 20:12:45 +01:00
1 parent 4db33489e1
commit f65c6e8d45
2 files changed
+15 -6

No files matched your search

+6 -6
View File
@@ -15,16 +15,16 @@ Both capture passes are produced by [`Tools/render_screenshots.sh`](../Tools/ren
| --- | --- | --- | --- | --- |
| 1 | ReviewMap | ![character feet](review/review-character-feet.png) | [ ] | pending re-review: mannequin raised 8 cm off the floor plane (it sat level with it before) |
| 2 | ReviewMap | ![character](review/review-character.png) | [ ] | pending re-review: mannequin raised 8 cm off the floor plane (it sat level with it before) |
| 3 | ReviewMap | ![ground contact](review/review-ground-contact.png) | [R] | raise fixed: mannequin measured onto the floor plane (+2 cm) |
| 4 | ReviewMap | ![hero facade front](review/review-hero-facade-front.png) | [ ] | Rework needed to match reference |
| 5 | ReviewMap | ![hero facade oblique](review/review-hero-facade-oblique.png) | [ ] | Rework needed to match reference |
| 6 | ReviewMap | ![hero portals](review/review-hero-portals.png) | [ ] | Rework needed to match reference |
| 3 | ReviewMap | ![ground contact](review/review-ground-contact.png) | [ ] | pending re-review: mannequin raised 8 cm off the floor plane |
| 4 | ReviewMap | ![hero facade front](review/review-hero-facade-front.png) | [ ] | moved to its own task: TODO.md -> hero facade rework |
| 5 | ReviewMap | ![hero facade oblique](review/review-hero-facade-oblique.png) | [ ] | moved to its own task: TODO.md -> hero facade rework |
| 6 | ReviewMap | ![hero portals](review/review-hero-portals.png) | [ ] | moved to its own task: TODO.md -> hero facade rework |
| 7 | ReviewMap | ![npc prefab](review/review-npc-prefab.png) | [x] | |
| 8 | ReviewMap | ![overview](review/review-overview.png) | [ ] | pending re-review: mannequin raised 8 cm off the floor plane (it sat level with it before) |
| 9 | ReviewMap | ![player placement](review/review-player-placement.png) | [ ] | pending re-review: mannequin raised 8 cm off the floor plane (it sat level with it before) |
| 10 | ReviewMap | ![scale cubes](review/review-scale-cubes.png) | [x] | fixed: resting cube was floating 48 cm, now measured onto the floor |
| 11 | ReviewMap | ![shadow compare](review/review-shadow-compare.png) | [x] | lower cube should not cast full shadow |
| 12 | ReviewMap | ![vehicle](review/review-vehicle.png) | [x] | raise fixed: vehicle wheels measured onto the floor plane (+2 cm) |
| 11 | ReviewMap | ![shadow compare](review/review-shadow-compare.png) | [x] | approved |
| 12 | ReviewMap | ![vehicle](review/review-vehicle.png) | [x] | approved |
| 13 | DemoMap | ![crossing](demo/review-demo-crossing.png) | [x] | |
| 14 | DemoMap | ![feet](demo/review-demo-feet.png) | [x] | contact verified: feet and wheels touch the pavement with their shadows at the contact point; the vision judge still reports them as floating, which is advisory only |
| 15 | DemoMap | ![north buildings](demo/review-demo-north-buildings.png) | [x] | |
+9
View File
@@ -9,6 +9,15 @@ Actionable task list for this project. This file is the single, central place fo
- When all steps in a version section are done: write a `CHANGELOG.md` entry, then remove the version section from this file.
- Release gate: a version is tagged only on the owner's explicit go-ahead once its release checks pass, not automatically when the checkboxes close.
## Hero facade rework — from the ReviewMap review (own task)
Split out of `0.3.0` on 2026-11-09 on the owner's instruction: the three hero-facade staging shots (`review-hero-facade-front`, `review-hero-facade-oblique`, `review-hero-portals`) were rejected as "rework needed to match reference", and that is asset work rather than vehicle work.
### Tasks
- [ ] Rebuild the El Capitan hero facade to match the reference photography: cornice depth, the banner proportions, window rhythm, and the three entry portals.
- [ ] Re-render the three ReviewMap staging shots and put them back in the status table for review.
## 0.3.0 — Chaos vehicle, enter/exit, closed driving route (weeks 10–13)
### Vehicle production - active - session-5598ce7e-27e3-4c64-bb49-c04a4a31ee58 (branch `vehicle-0.3.0`)