Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
73aca9db80 | ||
|
|
743a290ac9 | ||
|
|
71c7ca734b | ||
|
|
b8209ce82b | ||
|
|
a1d52aa583 | ||
|
|
5e2c5e29c0 | ||
|
|
d6d3d54810 |
No files matched your search
@@ -47,6 +47,10 @@ Thumbs.db
|
|||||||
/Geo/raw/mapbox-cache/
|
/Geo/raw/mapbox-cache/
|
||||||
/Geo/verification/*mapbox*
|
/Geo/verification/*mapbox*
|
||||||
|
|
||||||
|
# Visual-loop iterations: generated comparisons stay uncommitted until the owner
|
||||||
|
# confirms a shot. Only Screenshots/ is committed.
|
||||||
|
/Art/reference/iterations/
|
||||||
|
|
||||||
# Local, private reference material — never committed (see Docs/reference-notes-hollywood.md)
|
# Local, private reference material — never committed (see Docs/reference-notes-hollywood.md)
|
||||||
/Art/reference/
|
/Art/reference/
|
||||||
/Docs/data-inventory.md
|
/Docs/data-inventory.md
|
||||||
|
|||||||
@@ -67,6 +67,7 @@ Rules for `TODO.md`:
|
|||||||
- When **all** steps in a version section are done: write a short descriptive entry to `CHANGELOG.md`, remove the version section from `TODO.md`, and leave its goal in `ROADMAP.md`.
|
- When **all** steps in a version section are done: write a short descriptive entry to `CHANGELOG.md`, remove the version section from `TODO.md`, and leave its goal in `ROADMAP.md`.
|
||||||
- Release checks for each version are listed at the end of its `TODO.md` section; `Docs/release-checks.md` defines the checks themselves. If a check fails, do not tag the release; open a patch task in `TODO.md` and report the failing check in `CHANGELOG.md`.
|
- Release checks for each version are listed at the end of its `TODO.md` section; `Docs/release-checks.md` defines the checks themselves. If a check fails, do not tag the release; open a patch task in `TODO.md` and report the failing check in `CHANGELOG.md`.
|
||||||
- Finish each milestone's release checks **before** expanding scope. Cut map size and simulation complexity before compromising the core walking/driving experience.
|
- Finish each milestone's release checks **before** expanding scope. Cut map size and simulation complexity before compromising the core walking/driving experience.
|
||||||
|
- **Realism leads, performance is a floor.** When a realism improvement costs frame time, realism wins and the cost is paid — but 60 FPS at 1440p without frame generation must still be met at every release. If realism cannot fit the budget, reduce the realism scope deliberately and record it in `CHANGELOG.md`; never ship below the floor without the owner's approval. Walkability outranks realism in one direction only: a block the player falls through is not a demo.
|
||||||
|
|
||||||
## Git workflow
|
## Git workflow
|
||||||
|
|
||||||
|
|||||||
Binary file not shown.
Binary file not shown.
@@ -37,6 +37,15 @@
|
|||||||
],
|
],
|
||||||
"evidence": "6 columns at 4.96 m centres (sibling notes x measured frontage)"
|
"evidence": "6 columns at 4.96 m centres (sibling notes x measured frontage)"
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
"object": "Hero_Flute_x120",
|
||||||
|
"dims_m": [
|
||||||
|
0.064,
|
||||||
|
0.064,
|
||||||
|
6.53
|
||||||
|
],
|
||||||
|
"evidence": "photographs: fluted shafts; 20 grooves per column modelled as shallow cylinders in the shadow tone rather than boolean cut grooves"
|
||||||
|
},
|
||||||
{
|
{
|
||||||
"object": "Hero_ColumnBase_x6",
|
"object": "Hero_ColumnBase_x6",
|
||||||
"dims_m": [
|
"dims_m": [
|
||||||
@@ -104,10 +113,10 @@
|
|||||||
"object": "Hero_MottoBand",
|
"object": "Hero_MottoBand",
|
||||||
"dims_m": [
|
"dims_m": [
|
||||||
22.75,
|
22.75,
|
||||||
0.08,
|
0.1,
|
||||||
0.85
|
0.85
|
||||||
],
|
],
|
||||||
"evidence": "photographs: motto panel above the colonnade"
|
"evidence": "photographs: motto panel above the colonnade, now an inset panel with a moulded surround rather than a flat stripe"
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"object": "Hero_Medallion_x2",
|
"object": "Hero_Medallion_x2",
|
||||||
|
|||||||
+102
-1
@@ -8,6 +8,104 @@ Format: [Keep a Changelog](https://keepachangelog.com/) style, versions follow [
|
|||||||
|
|
||||||
Nothing yet.
|
Nothing yet.
|
||||||
|
|
||||||
|
## [0.1.5] — 2026-11-09
|
||||||
|
|
||||||
|
Façade finishing. The kit gets the hero detail it was missing, and a class of silent defect is found and gated: the map had been rendering with **604 of 620 mesh actors unmaterialed** without a single error.
|
||||||
|
|
||||||
|
### Added
|
||||||
|
|
||||||
|
- **Scene audit** (`Tools/ue_audit_scene.py`): fails when any staged mesh has no material, when a frontage building is more than ±1 m off its verified footprint, or when two actors share a label. Wired into `Tools/render_screenshots.sh`, which now runs the whole staging chain in the correct order and stops on an audit failure.
|
||||||
|
- **`Tools/ue_placement.py`**: one source of truth for staged positions and the verified frontage spans, with `span_error` / `describe_span_failure` validators.
|
||||||
|
- **Hero fluting**: 20 grooves per column as shallow shadow-tone cylinders (120 pieces) rather than boolean cuts.
|
||||||
|
- **Presence of references for the retail run**: `Art/reference/retail-neighbours/` (6800 Lee Drug and its neighbour). Local only, not committed.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
|
||||||
|
- **Motto band is a real recess** now — moulded surround 6 cm proud, panel 12 cm behind — instead of an 8 cm dark stripe. Hero: 148 parts / 3 044 tris → 271 / 5 492, still 14.50 m.
|
||||||
|
- **Kit layout for wide and tall buildings**: bays are capped at 9 with the module widened (6922 was 13 bays across 65.8 m — a window every 4.96 m on a 37 m block), and the storey spread is derived from the available height so the top band always lands inside the volume.
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
|
||||||
|
- **604 of 620 mesh actors rendered untextured.** `ensure_material` deleted and recreated each site material on every run, orphaning every reference held by the kit pieces and the theatre trim; the map looked only slightly flat. Materials are now updated in place.
|
||||||
|
- **The hero could be teleported 13 m east onto the wrong side of the boulevard.** `ue_hero_facade_import.py` carried its own copy of the pre-`0.1.0` hero position; running the importer alone moved the building, invisible in a render and visible only as a reference-score drop. Placement now has one definition and the import asserts the staged span (`span ok: X -14644..-12028`).
|
||||||
|
- **53 duplicated kit actor labels** (both shop piers of a bay shared one name).
|
||||||
|
|
||||||
|
### Verified
|
||||||
|
|
||||||
|
- Scene audit: **0 failures** — every mesh materialed, all 7 verified spans within tolerance, no duplicate labels.
|
||||||
|
- Packaged `-qaselftest`: walk, reset, pause/unpause, seam transect 1456 samples / 0 holes, wall probe 0/35, façade probe 0/9 — all PASS.
|
||||||
|
- **1440p re-measured with the kit**: 2.76 ms average (**363 fps**), p99 3.64 ms, against `0.1.0`'s 2.42 ms (414 fps) — the 625 kit actors and 8 textures cost about 12 % and leave roughly six times the 60 fps floor. Series at `Docs/perf/0.1.5-kit-walk-frametimes.csv`.
|
||||||
|
- 78 tool tests pass.
|
||||||
|
|
||||||
|
### Known limitations
|
||||||
|
|
||||||
|
- **Retail neighbours 6822 and 6806–6820 are not modelled**: Commons holds no usable reference for either, and substituting the El Capitan Theatre's photographs would misattribute their façades. Tracked in `TODO.md`.
|
||||||
|
- Cornice dentil spacing, abacus and plinth dimensions remain recorded assumptions.
|
||||||
|
- Textures are generated, not authored — no decals, signage text or weathering maps.
|
||||||
|
- The vision pre-check stays advisory; it has rejected shots the owner approved.
|
||||||
|
|
||||||
|
## [0.1.4] — 2026-11-09
|
||||||
|
|
||||||
|
Building façades. The block goes from flat extruded boxes with flat colours to coursed, textured surfaces assembled from a reusable module kit, and the El Capitan Theatre gets the marquee and blade sign that make it read as itself.
|
||||||
|
|
||||||
|
### Added
|
||||||
|
|
||||||
|
- **Façade module kit** at the measured **4.96 m bay** / **3.10 m storey**: pilaster, sill, lintel, window bay, shopfront bay, door bay, cornice, parapet and coping — 16 `SM_` meshes, each validated for manifold topology (12 tris each) and assembled onto **eight buildings, 625 pieces**, from each building's own world bounds.
|
||||||
|
- **Generated surface textures** — deterministic stdlib PNGs for ashlar stone, stucco, asphalt and concrete (base colour + roughness), sampled world-aligned with per-material tiling; byte-identical on re-run.
|
||||||
|
- **Materials**: masonry coursing and roughness variation in the material graphs; `M_FacadeStoneShadow`, `M_FacadeGlass` (specular 0.62 ≈ IOR 1.5) and `M_FacadeBanner`. Material assignment is now **per slot**, keyed on the imported slot material, instead of slot 0 only.
|
||||||
|
- **El Capitan Theatre trim**: projecting marquee (12.0 × 2.60 m canopy at a 4.60 m soffit), illuminated fascia, recessed soffit, and a vertical blade sign (1.30 × 5.20 m) with frame and wall bracket.
|
||||||
|
- **Visual review loop** (`Tools/visual_loop.py` + `visual_targets.json`): verifies every capture against an expectation (missing, blank, tone-less or wrong-size shots fail) and measures each render against its reference photograph, emitting feedback that names the discipline to consult. Iterations are written under `Art/reference/iterations/`, which is gitignored until a shot is confirmed.
|
||||||
|
- **`Tools/fbx_export_preset.py`** — one export preset replacing six copies, and it exports smoothing groups (`FACE`), so round geometry is no longer faceted.
|
||||||
|
- **`Tools/ue_common.py`** — clears the read-only bit an LFS `.umap` checks out with and makes a refused save loud.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
|
||||||
|
- **Elevation cameras** are levelled (pitch 0, so verticals stay parallel) and use a 42–55° lens to fill the frame from across the street.
|
||||||
|
- **`Screenshots/` holds demo-map captures only**; the review staging map is no longer rendered for review. Each shot carries its expected content and an approval state.
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
|
||||||
|
- **Blockout frame**: the `0.1.2` regeneration had run without `--origin`, shifting the whole frontage ≈119 m in X and 2.4 m in Y and dropping the hero out of the 6902/6838 gap. Regenerated with the affine fit (residual 0.0 cm) and every span re-verified against the block verification notes.
|
||||||
|
- **Kit placed ~12 km from its buildings** — it used the mesh-local bounding box, which excludes the actor offset.
|
||||||
|
- **Marquee mirrored across the boulevard** (Blender +Y is Unreal −Y) and **hanging 5 cm above the pavement** (absolute Z instead of pavement-relative).
|
||||||
|
- **Window bays invisible** inside the solid wall; the surround now projects with glazing inset.
|
||||||
|
- **Glazing was not glass**: kit pieces kept the grey material the FBX import creates.
|
||||||
|
- **Hero windows floated 1.44 m in front of the wall**; parapet medallions were flat discs on the coping.
|
||||||
|
- **Imports could report success while saving nothing** (read-only LFS map), and the north-buildings import failed silently on a missing `import sys`.
|
||||||
|
|
||||||
|
### Verified
|
||||||
|
|
||||||
|
- Packaged build `-qaselftest`: walk 761 cm, reset, pause/unpause, seam transect 1456 samples / 0 holes, wall probe 0/35, façade occlusion probe 0/9 — all PASS. Run twice; the harness is time-based and one loaded run reported a false failure.
|
||||||
|
- All eight buildings probed against the verified world spans.
|
||||||
|
- 67 tool tests pass.
|
||||||
|
|
||||||
|
### Known limitations
|
||||||
|
|
||||||
|
- Kit pieces are boxes; cornice dentil spacing, abacus and plinth dimensions are recorded assumptions.
|
||||||
|
- Textures are generated, not authored: no decals, signage text or weathering maps.
|
||||||
|
- The vision pre-check remains advisory — it is unreliable on this content.
|
||||||
|
- Retail neighbours, hero fluting and the blockout cleaning pass move to `0.1.5`.
|
||||||
|
- Performance has not been re-measured since `0.1.0`; 625 kit actors are unmeasured.
|
||||||
|
|
||||||
|
## [0.1.3] — 2026-11-09
|
||||||
|
|
||||||
|
Packaging and setup fixes. No gameplay or visual scope; this patch makes the pipeline and its documentation tell the truth about what `0.1.x` taught.
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
|
||||||
|
- **Map imports could report success while saving nothing.** `.umap` files are LFS-tracked and check out read-only, so `save_map()` returned `False` with no error and the staged actor vanished on the next load. New `Tools/ue_common.py` (`make_writable`, `require_saved`) clears the read-only bit and makes the failure loud; `ue_hero_facade_import.py` and `ue_import_north_buildings.py` now use it. Verified from a forced `0o444` map: chmod logged, `save_map: True`, exit 0.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
|
||||||
|
- `Docs/build.md`: content-pipeline table now lists every step as it exists (seven site FBXs, frontage, north buildings, hero, map) and the shared FBX preset; packaging archive corrected to `Builds/GigLineQA`; the packaged log location and `-qaselftest` usage are documented; new **traps** table covers the read-only save, the hero `--out`-is-the-FBX mistake, the MPFB add-on path, the shot actor quitting under QA, and the null-RHI trace trap.
|
||||||
|
- `Docs/setup.md`: the fresh-checkout build is marked verified (2026-11-08) and its blocker closed; a new section documents the two optional keys (`OPENROUTER_API_KEY`, `MAPBOX_ACCESS_TOKEN`) and that neither is needed by the build, the tests or a release.
|
||||||
|
- `Docs/Architecture/07_deployment_view.md` and the combined `architecture.md`: build-output path corrected and marked as living outside the repository.
|
||||||
|
|
||||||
|
### Verified
|
||||||
|
|
||||||
|
- 54 tool tests pass (5 new for `ue_common`).
|
||||||
|
- Packaged build still launches into `DemoMap` and `-qaselftest` passes every phase.
|
||||||
|
|
||||||
## [0.1.2] — 2026-11-09
|
## [0.1.2] — 2026-11-09
|
||||||
|
|
||||||
Scale and façade correction patch. Buildings and the hero elevation now match the reference photography and the cross-checked height data; no new gameplay scope.
|
Scale and façade correction patch. Buildings and the hero elevation now match the reference photography and the cross-checked height data; no new gameplay scope.
|
||||||
@@ -110,7 +208,10 @@ The first playable release: a walkable exterior demo of the **6840 Hollywood Blv
|
|||||||
- Kerb height (0.13 m as built vs 0.15 m assumed), kerb radii, driveway aprons and graded crossing ramps are deferred to `0.1.1`; crossing ramps are kerb flushes.
|
- Kerb height (0.13 m as built vs 0.15 m assumed), kerb radii, driveway aprons and graded crossing ramps are deferred to `0.1.1`; crossing ramps are kerb flushes.
|
||||||
- One block only; no vehicles, NPCs, audio, or settings UI yet.
|
- One block only; no vehicles, NPCs, audio, or settings UI yet.
|
||||||
|
|
||||||
[Unreleased]: https://git.allucanget.biz/zwitschi/gig-line/compare/v0.1.2...main
|
[Unreleased]: https://git.allucanget.biz/zwitschi/gig-line/compare/v0.1.5...main
|
||||||
|
[0.1.5]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.5
|
||||||
|
[0.1.4]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.4
|
||||||
|
[0.1.3]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.3
|
||||||
[0.1.2]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.2
|
[0.1.2]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.2
|
||||||
[0.1.1]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.1
|
[0.1.1]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.1
|
||||||
[0.1.0]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.0
|
[0.1.0]: https://git.allucanget.biz/zwitschi/gig-line/releases/tag/v0.1.0
|
||||||
@@ -20,11 +20,14 @@ Full requirements and progress are in [TODO.md](../../TODO.md); version directio
|
|||||||
|
|
||||||
| Priority | Quality Goal | Scenario |
|
| Priority | Quality Goal | Scenario |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| 1 | **Performance efficiency** | 60 FPS at 2560×1440 on RTX 5070 Ti, documented preset, no frame generation |
|
| 1 | **Visual realism** | Renders read as the reference photographs: façade identity, proportion, material and light — judged side by side, shot against photo |
|
||||||
| 2 | **Terrain walkability** | Zero raycast holes across both sidewalks and the carriageway (84-sample transect passes) |
|
| 2 | **Geographic fidelity** | Building placement and street scale match the measured data and the photographs |
|
||||||
| 3 | **Geographic fidelity** | Building placement, street scale, and façade identity match reference photography |
|
| 3 | **Terrain walkability** | Zero raycast holes across both sidewalks and the carriageway (84-sample transect passes) |
|
||||||
| 4 | **Build reproducibility** | Fresh checkout + documented engine path produces a runnable Linux build |
|
| 4 | **Build reproducibility** | Fresh checkout + documented engine path produces a runnable Linux build |
|
||||||
| 5 | **Provenance** | Every committed asset has recorded provenance |
|
| 5 | **Provenance** | Every committed asset has recorded provenance |
|
||||||
|
| 6 | **Performance efficiency** | 60 FPS at 2560×1440 on RTX 5070 Ti, documented preset, no frame generation |
|
||||||
|
|
||||||
|
**How the order is used.** It decides trade-offs. When realism and performance disagree, realism wins and the cost is paid — but never by silently shipping a frame rate below the floor. The 60 FPS figure is a **budget that must still be met**, so the response to "realism costs too much frame time" is to cut the realism feature deliberately, record the cut, and keep the floor — not to let frames slip. Walkability outranks realism in one direction only: a beautiful block the player falls through is not a demo.
|
||||||
|
|
||||||
## Stakeholders
|
## Stakeholders
|
||||||
|
|
||||||
|
|||||||
@@ -30,7 +30,8 @@ All modelling and geographic processing live outside the game project in `Art/`,
|
|||||||
|
|
||||||
## Quality achievement
|
## Quality achievement
|
||||||
|
|
||||||
- **Performance**: dynamic-only lighting (no build step, no static shadow maps), world-space-noise materials (cheap procedural variation), minimal overdraw (simple street-level geometry).
|
- **Realism first**: reference photography and measured data drive every façade decision; renders are compared against the photographs with `Tools/visual_loop.py`, and a façade is built from the shared 4.96 m module kit so the street wall reads consistently. Materials are authored in Unreal with world-aligned generated textures, and lighting is fixed daytime so a shot is comparable between runs.
|
||||||
|
- **Performance as a budget, not a driver**: dynamic-only lighting (no build step, no static shadow maps), world-space-noise materials (cheap procedural variation), minimal overdraw for street-level geometry, and a 60 FPS at 1440p floor that every release must meet. It closes the quality list because it constrains how much realism fits, not what the block should look like.
|
||||||
|
|
||||||
- **Walkability**: complex-as-simple collision on terrain (C++ helper forces the flag on the asset), block-all collision on every site mesh, swept transect QA verifies zero raycast holes.
|
- **Walkability**: complex-as-simple collision on terrain (C++ helper forces the flag on the asset), block-all collision on every site mesh, swept transect QA verifies zero raycast holes.
|
||||||
|
|
||||||
|
|||||||
@@ -7,7 +7,7 @@
|
|||||||
| Dev workstation | Bazzite Linux 45 (Fedora 45 base, immutable ostree), Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB, Vulkan RHI |
|
| Dev workstation | Bazzite Linux 45 (Fedora 45 base, immutable ostree), Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB, Vulkan RHI |
|
||||||
| Engine install | `/run/media/system/DATA/Engines/UE_5.8.4` (73 GiB, external to repo) |
|
| Engine install | `/run/media/system/DATA/Engines/UE_5.8.4` (73 GiB, external to repo) |
|
||||||
| Project repo | `/run/media/system/DATA/Documents/06-games/gig-line` (Git + LFS, remote at `git.allucanget.biz`) |
|
| Project repo | `/run/media/system/DATA/Documents/06-games/gig-line` (Git + LFS, remote at `git.allucanget.biz`) |
|
||||||
| Build output | `/run/media/system/DATA/Builds/GigLine` (packaged Linux build, ~1.1 GiB) |
|
| Build output | `/run/media/system/DATA/Builds/GigLineQA/Linux` (packaged Development build, ~1.1 GiB; outside the repository) |
|
||||||
|
|
||||||
Motivation
|
Motivation
|
||||||
Single-machine development. No build farm, no CI pipeline yet. The engine, toolchain, and GPU are all on the same host.
|
Single-machine development. No build farm, no CI pipeline yet. The engine, toolchain, and GPU are all on the same host.
|
||||||
|
|||||||
@@ -2,24 +2,34 @@
|
|||||||
|
|
||||||
## Quality Requirements Overview
|
## Quality Requirements Overview
|
||||||
|
|
||||||
| Category | Requirement | Priority |
|
Ordered by priority: realism first, performance last. The order decides design trade-offs; it does not license dropping a lower item silently (see the trade-off rule below the table).
|
||||||
|
|
||||||
|
| Priority | Category | Requirement |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| Performance efficiency | 60 FPS at 2560×1440, documented preset, no frame generation | 1 |
|
| 1 | Realism / Fidelity | Façade identity, proportion, material and lighting read like the reference photographs when compared side by side |
|
||||||
| Reliability / Terrain | Zero raycast holes across sidewalks and carriageway (84-sample transect) | 1 |
|
| 2 | Realism / Fidelity | Building placement matches the measured footprints and the photographs (within ~1 m) |
|
||||||
| Accuracy / Fidelity | Building placement matches reference photography within ~1 m | 2 |
|
| 2 | Realism / Fidelity | Street scale (lane width, sidewalk width) within ~10 cm of the measured profile |
|
||||||
| Accuracy / Fidelity | Street scale (lane width, sidewalk width) within ~10 cm of measured profile | 2 |
|
| 3 | Reliability / Terrain | Zero raycast holes across sidewalks and carriageway (84-sample transect) |
|
||||||
| Usability | WASD + mouse look, jump, reset (R), pause (Escape) — no tutorial needed | 2 |
|
| 3 | Usability | WASD + mouse look, jump, reset (R), pause (Escape) — no tutorial needed |
|
||||||
| Maintainability | Fresh checkout builds with documented engine installation | 2 |
|
| 4 | Maintainability | Fresh checkout builds with the documented engine installation |
|
||||||
| Portability | Linux build packages and runs; Windows build ships | 3 |
|
| 4 | Portability | Linux build packages and runs; Windows build ships |
|
||||||
|
| 5 | Provenance | Every committed asset has a recorded source and licence |
|
||||||
|
| 6 | Performance efficiency | 60 FPS at 2560×1440, documented preset, no frame generation |
|
||||||
|
|
||||||
|
**Trade-off rule.** Realism leads: when a realism improvement costs frame time, the improvement wins. Performance is still a **floor, not a preference** — 60 FPS at 1440p without frame generation must be met at every release. If realism cannot fit inside that budget, reduce the realism scope deliberately and record the reduction in `CHANGELOG.md`; never ship below the floor without the owner's explicit approval. Walkability outranks realism in one direction: a block the player falls through is not a demo, whatever it looks like.
|
||||||
|
|
||||||
## Quality Scenarios
|
## Quality Scenarios
|
||||||
|
|
||||||
| ID | Source/Stimulus | Response | Metric |
|
| ID | Source/Stimulus | Response | Metric |
|
||||||
| --- | --- | --- | --- |
|
| --- | --- | --- | --- |
|
||||||
| QS-1 | Player walks both sides of the demo | No fall-through, character stays at ground Z | 0 raycast holes in 84-sample transect |
|
| QS-1 | Screenshot is compared against its reference photograph | The render reads as the same building | Similarity score and visual match confirmed by the owner; `Tools/visual_loop.py` reports the deltas |
|
||||||
| QS-2 | Player holds W for 2 s on flat road | Character moves forward | ≥ 200 cm displacement, Z constant within ±5 cm |
|
| QS-2 | A façade is placed from measured data | It sits on its verified footprint | World span probed against `Geo/verification/` within ~1 m |
|
||||||
| QS-3 | Player presses R | Character teleports to spawn | < 100 cm from `StartLocation` |
|
| QS-3 | Player walks both sides of the demo | No fall-through, character stays at ground Z | 0 raycast holes in 84-sample transect |
|
||||||
| QS-4 | Player presses Escape | Game pauses, cursor appears | `IsGamePaused()` = true |
|
| QS-4 | Player holds W for 2 s on flat road | Character moves forward | ≥ 200 cm displacement, Z constant within ±5 cm |
|
||||||
| QS-5 | Player presses Escape while paused | Game unpauses, cursor hides | `IsGamePaused()` = false |
|
| QS-5 | Player presses R | Character teleports to spawn | < 100 cm from `StartLocation` |
|
||||||
| QS-6 | Game runs on dev workstation at 1440p | Frame rate stable | ≥ 60 FPS, 99th percentile ≥ 55 FPS |
|
| QS-6 | Player presses Escape | Game pauses, cursor appears | `IsGamePaused()` = true |
|
||||||
| QS-7 | Fresh checkout + engine install | Build succeeds | Exit code 0, runnable binary produced |
|
| QS-7 | Player presses Escape while paused | Game unpauses, cursor hides | `IsGamePaused()` = false |
|
||||||
|
| QS-8 | Fresh checkout + engine install | Build succeeds | Exit code 0, runnable binary produced |
|
||||||
|
| QS-9 | Game runs on dev workstation at 1440p | Frame rate holds the floor | ≥ 60 FPS, 99th percentile ≥ 55 FPS |
|
||||||
|
|
||||||
|
> QS-9 is last on purpose. Performance is a floor that every release must meet, not the goal that shapes the content: realism decides what gets built, the frame budget decides how much of it.
|
||||||
@@ -27,11 +27,14 @@ Full requirements and progress are in [TODO.md](../../TODO.md); version directio
|
|||||||
|
|
||||||
| Priority | Quality Goal | Scenario |
|
| Priority | Quality Goal | Scenario |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| 1 | **Performance efficiency** | 60 FPS at 2560×1440 on RTX 5070 Ti, documented preset, no frame generation |
|
| 1 | **Visual realism** | Renders read as the reference photographs: façade identity, proportion, material and light — judged side by side, shot against photo |
|
||||||
| 2 | **Terrain walkability** | Zero raycast holes across both sidewalks and the carriageway (84-sample transect passes) |
|
| 2 | **Geographic fidelity** | Building placement and street scale match the measured data and the photographs |
|
||||||
| 3 | **Geographic fidelity** | Building placement, street scale, and façade identity match reference photography |
|
| 3 | **Terrain walkability** | Zero raycast holes across both sidewalks and the carriageway (84-sample transect passes) |
|
||||||
| 4 | **Build reproducibility** | Fresh checkout + documented engine path produces a runnable Linux build |
|
| 4 | **Build reproducibility** | Fresh checkout + documented engine path produces a runnable Linux build |
|
||||||
| 5 | **Provenance** | Every committed asset has recorded provenance |
|
| 5 | **Provenance** | Every committed asset has recorded provenance |
|
||||||
|
| 6 | **Performance efficiency** | 60 FPS at 2560×1440 on RTX 5070 Ti, documented preset, no frame generation |
|
||||||
|
|
||||||
|
**How the order is used.** It decides trade-offs. When realism and performance disagree, realism wins and the cost is paid — but never by silently shipping a frame rate below the floor. The 60 FPS figure is a **budget that must still be met**, so the response to "realism costs too much frame time" is to cut the realism feature deliberately, record the cut, and keep the floor — not to let frames slip. Walkability outranks realism in one direction only: a beautiful block the player falls through is not a demo.
|
||||||
|
|
||||||
## Stakeholders
|
## Stakeholders
|
||||||
|
|
||||||
@@ -129,7 +132,9 @@ All modelling and geographic processing live outside the game project in `Art/`,
|
|||||||
|
|
||||||
## Quality achievement
|
## Quality achievement
|
||||||
|
|
||||||
- **Performance**: dynamic-only lighting (no build step, no static shadow maps), world-space-noise materials (cheap procedural variation), minimal overdraw (simple street-level geometry).
|
- **Realism**: reference photography drives every façade decision, measured data fixes placement and scale, and renders are compared against the photographs (`Tools/visual_loop.py`, `Screenshots/README.md`). One module kit at the 4.96 m bay keeps the street wall coherent; materials are world-aligned and lit by a fixed daytime rig so a shot is comparable between runs.
|
||||||
|
|
||||||
|
- **Performance holds the floor, it does not set the agenda**: dynamic-only lighting (no build step, no static shadow maps), cheap procedural materials, minimal overdraw for street-level geometry, and 60 FPS at 1440p as a budget every release must meet.
|
||||||
|
|
||||||
- **Walkability**: complex-as-simple collision on terrain (C++ helper forces the flag on the asset), block-all collision on every site mesh, swept transect QA verifies zero raycast holes.
|
- **Walkability**: complex-as-simple collision on terrain (C++ helper forces the flag on the asset), block-all collision on every site mesh, swept transect QA verifies zero raycast holes.
|
||||||
|
|
||||||
@@ -300,7 +305,7 @@ Source data: `Art/blender/` (`.blend` files), `Geo/` (OSM extracts, DEM, verific
|
|||||||
| Dev workstation | Bazzite Linux 45 (Fedora 45 base, immutable ostree), Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB, Vulkan RHI |
|
| Dev workstation | Bazzite Linux 45 (Fedora 45 base, immutable ostree), Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB, Vulkan RHI |
|
||||||
| Engine install | `/run/media/system/DATA/Engines/UE_5.8.4` (73 GiB, external to repo) |
|
| Engine install | `/run/media/system/DATA/Engines/UE_5.8.4` (73 GiB, external to repo) |
|
||||||
| Project repo | `/run/media/system/DATA/Documents/06-games/gig-line` (Git + LFS, remote at `git.allucanget.biz`) |
|
| Project repo | `/run/media/system/DATA/Documents/06-games/gig-line` (Git + LFS, remote at `git.allucanget.biz`) |
|
||||||
| Build output | `/run/media/system/DATA/Builds/GigLine` (packaged Linux build, ~1.1 GiB) |
|
| Build output | `/run/media/system/DATA/Builds/GigLineQA/Linux` (packaged Development build, ~1.1 GiB; outside the repository) |
|
||||||
|
|
||||||
Motivation
|
Motivation
|
||||||
Single-machine development. No build farm, no CI pipeline yet. The engine, toolchain, and GPU are all on the same host.
|
Single-machine development. No build farm, no CI pipeline yet. The engine, toolchain, and GPU are all on the same host.
|
||||||
@@ -450,27 +455,37 @@ Rule: place a building only against a neighbour or street mesh, never in isolati
|
|||||||
|
|
||||||
## Quality Requirements Overview
|
## Quality Requirements Overview
|
||||||
|
|
||||||
| Category | Requirement | Priority |
|
Ordered by priority: realism first, performance last. The order decides design trade-offs; it does not license dropping a lower item silently (see the trade-off rule below the table).
|
||||||
|
|
||||||
|
| Priority | Category | Requirement |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| Performance efficiency | 60 FPS at 2560×1440, documented preset, no frame generation | 1 |
|
| 1 | Realism / Fidelity | Façade identity, proportion, material and lighting read like the reference photographs when compared side by side |
|
||||||
| Reliability / Terrain | Zero raycast holes across sidewalks and carriageway (84-sample transect) | 1 |
|
| 2 | Realism / Fidelity | Building placement matches the measured footprints and the photographs (within ~1 m) |
|
||||||
| Accuracy / Fidelity | Building placement matches reference photography within ~1 m | 2 |
|
| 2 | Realism / Fidelity | Street scale (lane width, sidewalk width) within ~10 cm of the measured profile |
|
||||||
| Accuracy / Fidelity | Street scale (lane width, sidewalk width) within ~10 cm of measured profile | 2 |
|
| 3 | Reliability / Terrain | Zero raycast holes across sidewalks and carriageway (84-sample transect) |
|
||||||
| Usability | WASD + mouse look, jump, reset (R), pause (Escape) — no tutorial needed | 2 |
|
| 3 | Usability | WASD + mouse look, jump, reset (R), pause (Escape) — no tutorial needed |
|
||||||
| Maintainability | Fresh checkout builds with documented engine installation | 2 |
|
| 4 | Maintainability | Fresh checkout builds with the documented engine installation |
|
||||||
| Portability | Linux build packages and runs; Windows build ships | 3 |
|
| 4 | Portability | Linux build packages and runs; Windows build ships |
|
||||||
|
| 5 | Provenance | Every committed asset has a recorded source and licence |
|
||||||
|
| 6 | Performance efficiency | 60 FPS at 2560×1440, documented preset, no frame generation |
|
||||||
|
|
||||||
|
**Trade-off rule.** Realism leads: when a realism improvement costs frame time, the improvement wins. Performance is still a **floor, not a preference** — 60 FPS at 1440p without frame generation must be met at every release. If realism cannot fit inside that budget, reduce the realism scope deliberately and record the reduction in `CHANGELOG.md`; never ship below the floor without the owner's explicit approval. Walkability outranks realism in one direction: a block the player falls through is not a demo, whatever it looks like.
|
||||||
|
|
||||||
## Quality Scenarios
|
## Quality Scenarios
|
||||||
|
|
||||||
| ID | Source/Stimulus | Response | Metric |
|
| ID | Source/Stimulus | Response | Metric |
|
||||||
| --- | --- | --- | --- |
|
| --- | --- | --- | --- |
|
||||||
| QS-1 | Player walks both sides of the demo | No fall-through, character stays at ground Z | 0 raycast holes in 84-sample transect |
|
| QS-1 | Screenshot is compared against its reference photograph | The render reads as the same building | Similarity score and visual match confirmed by the owner; `Tools/visual_loop.py` reports the deltas |
|
||||||
| QS-2 | Player holds W for 2 s on flat road | Character moves forward | ≥ 200 cm displacement, Z constant within ±5 cm |
|
| QS-2 | A façade is placed from measured data | It sits on its verified footprint | World span probed against `Geo/verification/` within ~1 m |
|
||||||
| QS-3 | Player presses R | Character teleports to spawn | < 100 cm from `StartLocation` |
|
| QS-3 | Player walks both sides of the demo | No fall-through, character stays at ground Z | 0 raycast holes in 84-sample transect |
|
||||||
| QS-4 | Player presses Escape | Game pauses, cursor appears | `IsGamePaused()` = true |
|
| QS-4 | Player holds W for 2 s on flat road | Character moves forward | ≥ 200 cm displacement, Z constant within ±5 cm |
|
||||||
| QS-5 | Player presses Escape while paused | Game unpauses, cursor hides | `IsGamePaused()` = false |
|
| QS-5 | Player presses R | Character teleports to spawn | < 100 cm from `StartLocation` |
|
||||||
| QS-6 | Game runs on dev workstation at 1440p | Frame rate stable | ≥ 60 FPS, 99th percentile ≥ 55 FPS |
|
| QS-6 | Player presses Escape | Game pauses, cursor appears | `IsGamePaused()` = true |
|
||||||
| QS-7 | Fresh checkout + engine install | Build succeeds | Exit code 0, runnable binary produced |
|
| QS-7 | Player presses Escape while paused | Game unpauses, cursor hides | `IsGamePaused()` = false |
|
||||||
|
| QS-8 | Fresh checkout + engine install | Build succeeds | Exit code 0, runnable binary produced |
|
||||||
|
| QS-9 | Game runs on dev workstation at 1440p | Frame rate holds the floor | ≥ 60 FPS, 99th percentile ≥ 55 FPS |
|
||||||
|
|
||||||
|
> QS-9 is last on purpose. Performance is a floor that every release must meet, not the goal that shapes the content: realism decides what gets built, the frame budget decides how much of it.
|
||||||
|
|
||||||
# Risks and Technical Debts
|
# Risks and Technical Debts
|
||||||
|
|
||||||
|
|||||||
+50
-37
@@ -1,6 +1,6 @@
|
|||||||
# Build
|
# Build
|
||||||
|
|
||||||
Fresh-checkout build procedure. Completed 2026-11-07 after the pinned engine was installed and a blank `Game/GigLine` build was packaged and launch-tested.
|
Fresh-checkout build, content-pipeline and packaging procedure for the walkable demo. Begun 2026-11-07 from a blank build and extended through the `0.1.x` patches; the release checks that exercise it are in [release-checks.md](release-checks.md).
|
||||||
|
|
||||||
## Constraints
|
## Constraints
|
||||||
|
|
||||||
@@ -10,8 +10,7 @@ Fresh-checkout build procedure. Completed 2026-11-07 after the pinned engine was
|
|||||||
|
|
||||||
## Engine
|
## Engine
|
||||||
|
|
||||||
Pinned install: **Unreal Engine 5.8.4** (CL 58902052, promoted build) at
|
Pinned install: **Unreal Engine 5.8.4** (CL 58902052, promoted build) at `/run/media/system/DATA/Engines/UE_5.8.4`. The engine bundles **no** Linux clang; the system clang (23.1.4, Fedora 45) is the game-code toolchain. Game targets must use `BuildSettingsVersion.V7` to share the installed editor's build environment.
|
||||||
`/run/media/system/DATA/Engines/UE_5.8.4`. The engine bundles **no** Linux clang; the system clang (23.1.4, Fedora 45) is the game-code toolchain. Game targets must use `BuildSettingsVersion.V7` to share the installed editor's build environment.
|
|
||||||
|
|
||||||
## Fresh-checkout build
|
## Fresh-checkout build
|
||||||
|
|
||||||
@@ -42,20 +41,14 @@ cd /run/media/system/DATA/Engines/UE_5.8.4
|
|||||||
Engine/Build/BatchFiles/RunUAT.sh BuildCookRun \
|
Engine/Build/BatchFiles/RunUAT.sh BuildCookRun \
|
||||||
-project="/run/media/system/DATA/Documents/06-games/gig-line/Game/GigLine/GigLine.uproject" \
|
-project="/run/media/system/DATA/Documents/06-games/gig-line/Game/GigLine/GigLine.uproject" \
|
||||||
-noP4 -platform=Linux -clientconfig=Development \
|
-noP4 -platform=Linux -clientconfig=Development \
|
||||||
-cook -allmaps -build -stage -pak \
|
-cook -build -stage -pak \
|
||||||
-archive -archivedirectory="/run/media/system/DATA/Builds/GigLine" \
|
-archive -archivedirectory="/run/media/system/DATA/Builds/GigLineQA" \
|
||||||
-unattended
|
-unattended
|
||||||
# Expected: "BUILD SUCCESSFUL" (~1 min for the blank project; archive ~1.1 GiB)
|
# Expected: "AutomationTool exiting with ExitCode=0 (Success)" and "EXIT=0", ~90 s
|
||||||
# Launch test (plain, no -map): /run/media/system/DATA/Builds/GigLine/Linux/GigLine.sh
|
# Launch test (plain, no -map): /run/media/system/DATA/Builds/GigLineQA/Linux/GigLine.sh
|
||||||
```
|
```
|
||||||
|
|
||||||
**Launch test must open the DemoMap.** A plain `./GigLine.sh` should load
|
**Launch test must open the DemoMap.** A plain `./GigLine.sh` should load `/Game/Hollywood/DemoMap` (set via `DefaultEngine.ini` `GameMapsSettings`). If it instead opens `/Engine/Maps/Templates/OpenWorld`, the game shows an empty non-interactive "stuck" scene — check that `GameDefaultMap` is in `DefaultEngine.ini`, not `DefaultGame.ini`. Confirm the loaded map in the engine log: `.../Linux/GigLine/Saved/Logs/GigLine.log` (`LoadMap: /Game/Hollywood/DemoMap`). The archive directory is deliberately **outside** the repository; never archive into the checkout.
|
||||||
`/Game/Hollywood/DemoMap` (set via `DefaultEngine.ini` `GameMapsSettings`). If it
|
|
||||||
instead opens `/Engine/Maps/Templates/OpenWorld`, the game shows an empty
|
|
||||||
non-interactive "stuck" scene — check that `GameDefaultMap` is in
|
|
||||||
`DefaultEngine.ini`, not `DefaultGame.ini`. Confirm the loaded map in the engine
|
|
||||||
log: `.../Linux/GigLine/Saved/Logs/GigLine.log` (`LoadMap: /Game/Hollywood/DemoMap`).
|
|
||||||
The archive directory is deliberately **outside** the repository; never archive into the checkout.
|
|
||||||
|
|
||||||
## Headless content checks
|
## Headless content checks
|
||||||
|
|
||||||
@@ -76,9 +69,7 @@ Both content-acceptance tests run headless and are the template for all asset im
|
|||||||
|
|
||||||
## Content pipeline (current)
|
## Content pipeline (current)
|
||||||
|
|
||||||
The hero block and the demo level are built by a fixed sequence. Each step writes
|
The hero block and the demo level are built by a fixed sequence. Each step writes a result file under `/tmp/gigline/`; read it, because the commandlet's own stdout can be swallowed.
|
||||||
a result file under `/tmp/gigline/`; read it, because the commandlet's own stdout
|
|
||||||
can be swallowed.
|
|
||||||
|
|
||||||
| Step | Tool | Output |
|
| Step | Tool | Output |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
@@ -86,21 +77,28 @@ can be swallowed.
|
|||||||
| Terrain | `Art/blender/terrain/terrain_from_dem.py` | `Art/blender/terrain/terrain-*.blend` |
|
| Terrain | `Art/blender/terrain/terrain_from_dem.py` | `Art/blender/terrain/terrain-*.blend` |
|
||||||
| Street | `Art/blender/street/street_mesh.py` | `Art/blender/street/street-*.blend` |
|
| Street | `Art/blender/street/street_mesh.py` | `Art/blender/street/street-*.blend` |
|
||||||
| Site assembly | `Art/blender/site/site_assembly.py` | `Art/blender/site/site-*.blend` |
|
| Site assembly | `Art/blender/site/site_assembly.py` | `Art/blender/site/site-*.blend` |
|
||||||
| Site export | `Art/blender/site/export_site_fbx.py` | six FBXs, one per mesh |
|
| Site export | `Art/blender/site/export_site_fbx.py` | seven FBXs, one per mesh (terrain, road, both sidewalks, both kerbs, crossing) |
|
||||||
| Hero façade | `Tools/hero_facade_build.py` | `Art/blender/hero/hero-facade-*.blend` + JSON report |
|
| Hero façade | `Tools/hero_facade_build.py` | `Art/blender/hero/hero-facade-*.blend` + JSON part report |
|
||||||
|
| Block frontage | `Tools/export_block_frontage.py` | six frontage FBXs from the blockout |
|
||||||
|
| North buildings | `Tools/export_north_buildings.py` | TCL and Tussaud's FBXs from the blockout |
|
||||||
| Site import | `Tools/ue_import_site.py` | `/Game/Hollywood/Site/*` |
|
| Site import | `Tools/ue_import_site.py` | `/Game/Hollywood/Site/*` |
|
||||||
|
| Frontage import | `Tools/ue_import_block_frontage.py` | `/Game/Hollywood/Frontage/*` |
|
||||||
|
| North import | `Tools/ue_import_north_buildings.py` | `/Game/Hollywood/North/*` |
|
||||||
| Hero import | `Tools/ue_hero_facade_import.py` | `/Game/Hollywood/ElCapitan/SM_HeroFacade` |
|
| Hero import | `Tools/ue_hero_facade_import.py` | `/Game/Hollywood/ElCapitan/SM_HeroFacade` |
|
||||||
| Demo map | `Tools/ue_build_demo_map.py` | `/Game/Hollywood/DemoMap` |
|
| Demo map | `Tools/ue_build_demo_map.py` | `/Game/Hollywood/DemoMap` |
|
||||||
|
| Façade kit assembly | `Tools/ue_apply_facade_kit.py` | kit actors in `DemoMap` |
|
||||||
|
| Theatre trim | `Tools/ue_stage_theatre_trim.py` | marquee + blade sign actors |
|
||||||
|
| Scene audit | `Tools/ue_audit_scene.py` | materials, placement spans, labels (gate) |
|
||||||
| Review shots | `Tools/ue_build_review_map.py` + `Tools/render_screenshots.sh` | `Screenshots/` |
|
| Review shots | `Tools/ue_build_review_map.py` + `Tools/render_screenshots.sh` | `Screenshots/` |
|
||||||
|
|
||||||
|
**Order matters.** `ue_build_demo_map.py` rebuilds the site materials, so any actor staged by another tool must be (re)staged _after_ it, or it keeps a reference to a material that no longer exists — that is how 604 actors once ended up untethered from their materials while the map only looked slightly flat. `Tools/render_screenshots.sh` runs the full chain in the right order and gates on `ue_audit_scene.py`.
|
||||||
|
|
||||||
|
Every Blender exporter calls [`Tools/fbx_export_preset.py`](../Tools/fbx_export_preset.py) (`ue_metric_v1` / `ue_character_v1`) instead of repeating the FBX argument block — one place for the scale conversion, the axis mapping and `mesh_smooth_type="FACE"` (without which every Unreal import warns about missing smoothing groups and round geometry arrives faceted).
|
||||||
|
|
||||||
Two rules that the pipeline enforces rather than documents:
|
Two rules that the pipeline enforces rather than documents:
|
||||||
|
|
||||||
- **One conversion.** Metres to Unreal centimetres happens only in
|
- **One conversion.** Metres to Unreal centimetres happens only in `Geo/tools/osm_common.py::to_unreal_local`. Sidecars (`<asset>.geo.json`) record the result and `Geo/tools/asset_metadata.py audit` re-derives it.
|
||||||
`Geo/tools/osm_common.py::to_unreal_local`. Sidecars (`<asset>.geo.json`) record
|
- **Collision is decided at import,** not later: terrain, road and sidewalks are complex-as-simple; kerbs and props stay simple. A simple hull over a kerbed street ramps straight across the kerb.
|
||||||
the result and `Geo/tools/asset_metadata.py audit` re-derives it.
|
|
||||||
- **Collision is decided at import,** not later: terrain, road and sidewalks are
|
|
||||||
complex-as-simple; kerbs and props stay simple. A simple hull over a kerbed
|
|
||||||
street ramps straight across the kerb.
|
|
||||||
|
|
||||||
## Known engine quirks (2026-11-07)
|
## Known engine quirks (2026-11-07)
|
||||||
|
|
||||||
@@ -110,20 +108,35 @@ Two rules that the pipeline enforces rather than documents:
|
|||||||
|
|
||||||
## Packaged Linux build
|
## Packaged Linux build
|
||||||
|
|
||||||
- 2026-11-07: blank `Game/GigLine` Development build packaged via the command above; 1.1 GiB archive at `/run/media/system/DATA/Builds/GigLine/Linux`; `GigLine.sh` launched on the desktop session, Vulkan swapchain created, 759 frames rendered across a 60 s smoke test. Controls document to be written with the first playable (`0.1.0` Gameplay section).
|
Current QA archive: `/run/media/system/DATA/Builds/GigLineQA`. A plain `./GigLine.sh` opens `/Game/Hollywood/DemoMap`; controls are in [controls.md](controls.md) and shown in-game bottom-right. The packaged log is written **relative to the wrapper's working directory**, i.e. `<archive>/Linux/GigLine/Saved/Logs/GigLine.log` — not into the repository.
|
||||||
|
|
||||||
|
Behaviour check without a human at the keyboard:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cd /run/media/system/DATA/Builds/GigLineQA/Linux
|
||||||
|
./GigLine.sh -windowed -ResX=1280 -ResY=720 -nosplash -qaselftest
|
||||||
|
# then read <archive>/Linux/GigLine/Saved/Logs/GigLine.log for "GigLine QA:" lines
|
||||||
|
```
|
||||||
|
|
||||||
|
`-qaselftest` walks for 2 s, resets, pauses/unpauses, sweeps a 16-row seam transect, fires a ground-referenced 21 m wall probe along the street, fires 40 m façade-occlusion traces sideways from the road centre wherever a building is staged, and writes a frame-time series to `Saved/Profiling/`. Every phase logs `... PASS`/`FAIL` and the run ends with `RESULT`.
|
||||||
|
|
||||||
|
## Traps that cost real time in `0.1.x`
|
||||||
|
|
||||||
|
| Symptom | Cause | Fix |
|
||||||
|
| --- | --- | --- |
|
||||||
|
| A builder reports success but the asset is unchanged | `.blend`/`.umap` are LFS-tracked and check out **read-only**; Blender and Unreal fail the save quietly | `chmod u+w <file>` first — `ue_build_*`/`ue_hero_facade_import` and every Blender builder need it |
|
||||||
|
| `File format is not supported` when reopening a `.blend` | `hero_facade_build.py --out` writes the **FBX**; putting the `.blend` path there overwrites it with FBX bytes | `--out /tmp/…fbx --save-blend Art/…blend --report Art/…json` |
|
||||||
|
| `Add-on not loaded: "bl_ext.blender_org.mpfb"` | `mpfb_character_export.py` needs the MPFB add-on visible to Blender | export `BLENDER_USER_SCRIPTS` (see the invocation in [../Art/blender/README.md](../Art/blender/README.md)) |
|
||||||
|
| The packaged run exits after a couple of seconds during QA | `GigLineShotActor` runs its screenshot list and quits (`bQuitWhenDone`) | it skips its rounds when `-qaselftest` is on the command line — keep that guard |
|
||||||
|
| Headless `-nullrhi` traces report no hits for everything | the null RHI has no physics scene | verify traces in the **packaged** run (`-qaselftest`), never in a commandlet |
|
||||||
|
| Import logs "No smoothing group information" for every mesh | `mesh_smooth_type="OFF"` in the FBX export | use `Tools/fbx_export_preset.py` (`FACE`) |
|
||||||
|
|
||||||
## PIE input test (compare editor vs packaged input intake)
|
## PIE input test (compare editor vs packaged input intake)
|
||||||
|
|
||||||
Use to isolate whether the "no controls respond" bug is the packaged standalone's
|
Use to isolate whether the "no controls respond" bug is the packaged standalone's input intake or the Enhanced Input wiring generally.
|
||||||
input intake or the Enhanced Input wiring generally.
|
|
||||||
|
|
||||||
- `Tools/run_pie.sh` opens the editor on `/Game/Hollywood/DemoMap`. Press the
|
- `Tools/run_pie.sh` opens the editor on `/Game/Hollywood/DemoMap`. Press the green **Play** (PIE) button and walk / mouse-look.
|
||||||
green **Play** (PIE) button and walk / mouse-look.
|
|
||||||
- Interpretation:
|
- Interpretation:
|
||||||
- PIE **works** -> the Enhanced Input wiring is fine and the standalone
|
- PIE **works** -> the Enhanced Input wiring is fine and the standalone packaged build's SDL input intake is the environment-dependent bug.
|
||||||
packaged build's SDL input intake is the environment-dependent bug.
|
- PIE **fails** too -> the Enhanced Input setup itself is broken; rebuild the input config / character from scratch.
|
||||||
- PIE **fails** too -> the Enhanced Input setup itself is broken; rebuild
|
- The DemoMap is PIE-ready as committed: `GlobalDefaultGameMode` + `GameDefaultMap=/Game/Hollywood/DemoMap`, and the map stages a PlayerStart at `(-13336,-1150,600)` on the cut sidewalk with the walkable `AGigLineCharacter`.
|
||||||
the input config / character from scratch.
|
|
||||||
- The DemoMap is PIE-ready as committed: `GlobalDefaultGameMode` +
|
|
||||||
`GameDefaultMap=/Game/Hollywood/DemoMap`, and the map stages a PlayerStart at
|
|
||||||
`(-13336,-1150,600)` on the cut sidewalk with the walkable `AGigLineCharacter`.
|
|
||||||
+10
-16
@@ -1,36 +1,30 @@
|
|||||||
# Controls (`0.1.0` walkable demo)
|
# Controls (`0.1.0` walkable demo)
|
||||||
|
|
||||||
Keyboard and mouse. The same list is shown in-game, bottom-right of the frame
|
Keyboard and mouse. The same list is shown in-game, bottom-right of the frame (`AGigLineHUD`), and the packaged Linux build ships with these bindings.
|
||||||
(`AGigLineHUD`), and the packaged Linux build ships with these bindings.
|
|
||||||
|
|
||||||
| Action | Key | Notes |
|
| Action | Key | Notes |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| Move forward | `W` | 4 m/s walk |
|
| Move forward | `W` | 4 m/s walk |
|
||||||
| Move back | `S` | |
|
| Move back | `S` | |
|
||||||
| Strafe right | `D` | |
|
| Strafe right | `D` | |
|
||||||
| Strafe left | `A` | |
|
| Strafe left | `A` | |
|
||||||
| Look | Mouse | Captured on launch; `Escape` releases it |
|
| Look | Mouse | Captured on launch; `Escape` releases it |
|
||||||
| Jump | `Space` | |
|
| Jump | `Space` | |
|
||||||
| Reset to start | `R` | Returns to the grounded spawn point |
|
| Reset to start | `R` | Returns to the grounded spawn point |
|
||||||
| Pause / release mouse | `Escape` | Second press resumes and recaptures the mouse |
|
| Pause / release mouse | `Escape` | Second press resumes and recaptures the mouse |
|
||||||
|
|
||||||
## Notes
|
## Notes
|
||||||
|
|
||||||
- Movement is relative to where you are looking (controller yaw), so the capsule
|
- Movement is relative to where you are looking (controller yaw), so the capsule stays upright on the sloped street.
|
||||||
stays upright on the sloped street.
|
- The mouse is captured on launch, so looking works without clicking first. `Escape` toggles the capture as part of pause.
|
||||||
- The mouse is captured on launch, so looking works without clicking first.
|
- Bindings are Enhanced Input actions in `/Game/Input` built by `Tools/ue_create_input_assets.py`; the mapping context is `IMC_Default`.
|
||||||
`Escape` toggles the capture as part of pause.
|
|
||||||
- Bindings are Enhanced Input actions in `/Game/Input` built by
|
|
||||||
`Tools/ue_create_input_assets.py`; the mapping context is `IMC_Default`.
|
|
||||||
|
|
||||||
## Testing input without a physical keyboard
|
## Testing input without a physical keyboard
|
||||||
|
|
||||||
`Tools/input_test.py` injects OS-level keyboard/mouse events through a virtual
|
`Tools/input_test.py` injects OS-level keyboard/mouse events through a virtual `/dev/uinput` device (see `Tools/input_emulation.py`), e.g.:
|
||||||
`/dev/uinput` device (see `Tools/input_emulation.py`), e.g.:
|
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
python3 Tools/input_test.py --sequence move:2,look:3
|
python3 Tools/input_test.py --sequence move:2,look:3
|
||||||
```
|
```
|
||||||
|
|
||||||
It drives the focused game window exactly like real hardware, which is how the
|
It drives the focused game window exactly like real hardware, which is how the controls were regression-tested on the packaged build.
|
||||||
controls were regression-tested on the packaged build.
|
|
||||||
+9
-26
@@ -37,22 +37,12 @@ Important nuance: ODbL governs the **database** (the OSM data as a collection an
|
|||||||
|
|
||||||
## 4. Decision: posture for the derived geometry
|
## 4. Decision: posture for the derived geometry
|
||||||
|
|
||||||
The OSM-derived block geometry (streets, footprints, sidewalks) is treated as a
|
The OSM-derived block geometry (streets, footprints, sidewalks) is treated as a **Produced Work / derivative database** of the OpenStreetMap database.
|
||||||
**Produced Work / derivative database** of the OpenStreetMap database.
|
|
||||||
|
|
||||||
- The **share-alike obligation is accepted for the data layer only**. The
|
- The **share-alike obligation is accepted for the data layer only**. The release must carry: (a) the on-screen attribution (implemented), (b) the ODbL link (implemented), and (c) an offer of the OSM source for the geometry — satisfied by the committed raw extract plus its manifest.
|
||||||
release must carry: (a) the on-screen attribution (implemented), (b) the ODbL
|
- **Project creative work (code, shaders, artist-authored façades, textures) is NOT under ODbL.** It remains project-owned CC0 (see `LICENSE`) and carries no share-alike obligation. Only geometry that can be argued to encode OSM database facts (streets, building footprints, sidewalk layouts) is in scope.
|
||||||
link (implemented), and (c) an offer of the OSM source for the geometry —
|
- **No new obligation is created by the DEM**: 3DEP is US Government public domain and is independent of ODbL.
|
||||||
satisfied by the committed raw extract plus its manifest.
|
- **Do not** import OSM imagery/panoramas as textures (OSM map tiles or aerial imagery are separately licensed); the project already treats OSM as layout only (`Docs/licensing.md` rule 3).
|
||||||
- **Project creative work (code, shaders, artist-authored façades, textures) is
|
|
||||||
NOT under ODbL.** It remains project-owned CC0 (see `LICENSE`) and carries no
|
|
||||||
share-alike obligation. Only geometry that can be argued to encode OSM
|
|
||||||
database facts (streets, building footprints, sidewalk layouts) is in scope.
|
|
||||||
- **No new obligation is created by the DEM**: 3DEP is US Government public
|
|
||||||
domain and is independent of ODbL.
|
|
||||||
- **Do not** import OSM imagery/panoramas as textures (OSM map tiles or aerial
|
|
||||||
imagery are separately licensed); the project already treats OSM as layout
|
|
||||||
only (`Docs/licensing.md` rule 3).
|
|
||||||
|
|
||||||
Release checklist items this lands:
|
Release checklist items this lands:
|
||||||
|
|
||||||
@@ -63,20 +53,13 @@ Release checklist items this lands:
|
|||||||
|
|
||||||
## 5. Remaining (tracked separately)
|
## 5. Remaining (tracked separately)
|
||||||
|
|
||||||
- **Blosm add-on licence** is still `pending` in THIRD_PARTY_NOTICES (tool-only,
|
- **Blosm add-on licence** is still `pending` in THIRD_PARTY_NOTICES (tool-only, not distributed — must be checked once, not a release blocker for ODbL).
|
||||||
not distributed — must be checked once, not a release blocker for ODbL).
|
- Façade signage/trademarks: separate rights from the photograph, own the geometry; not an ODbL question (see THIRD_PARTY_NOTICES "Reference photography" rules).
|
||||||
- Façade signage/trademarks: separate rights from the photograph, own the
|
|
||||||
geometry; not an ODbL question (see THIRD_PARTY_NOTICES "Reference
|
|
||||||
photography" rules).
|
|
||||||
|
|
||||||
## 6. Sources checked
|
## 6. Sources checked
|
||||||
|
|
||||||
- ODbL text: <https://opendatacommons.org/licenses/odbl/1-0/>
|
- ODbL text: <https://opendatacommons.org/licenses/odbl/1-0/>
|
||||||
- OSM licence summary: <https://www.openstreetmap.org/copyright>
|
- OSM licence summary: <https://www.openstreetmap.org/copyright>
|
||||||
- ISPRS/ODbL practice notes on derived game geometry from OSM footprints
|
- ISPRS/ODbL practice notes on derived game geometry from OSM footprints (objective standard: attribution + produced-work offer, share-alike limited to the data layer).
|
||||||
(objective standard: attribution + produced-work offer, share-alike limited
|
|
||||||
to the data layer).
|
|
||||||
|
|
||||||
Not reviewed here (out of scope for this item): engine redistribution terms
|
Not reviewed here (out of scope for this item): engine redistribution terms (Epic), MPFB asset terms, street-prop reuse terms — all tracked in `THIRD_PARTY_NOTICES.md`.
|
||||||
(Epic), MPFB asset terms, street-prop reuse terms — all tracked in
|
|
||||||
`THIRD_PARTY_NOTICES.md`.
|
|
||||||
@@ -0,0 +1,728 @@
|
|||||||
|
frame,ms
|
||||||
|
0,2.599
|
||||||
|
1,2.877
|
||||||
|
2,2.709
|
||||||
|
3,2.646
|
||||||
|
4,3.168
|
||||||
|
5,2.729
|
||||||
|
6,2.524
|
||||||
|
7,2.726
|
||||||
|
8,2.639
|
||||||
|
9,2.814
|
||||||
|
10,2.819
|
||||||
|
11,2.708
|
||||||
|
12,2.822
|
||||||
|
13,2.703
|
||||||
|
14,2.365
|
||||||
|
15,2.796
|
||||||
|
16,2.899
|
||||||
|
17,2.972
|
||||||
|
18,2.908
|
||||||
|
19,2.356
|
||||||
|
20,2.781
|
||||||
|
21,2.574
|
||||||
|
22,3.148
|
||||||
|
23,2.306
|
||||||
|
24,2.668
|
||||||
|
25,2.855
|
||||||
|
26,2.588
|
||||||
|
27,3.321
|
||||||
|
28,3.419
|
||||||
|
29,3.121
|
||||||
|
30,3.391
|
||||||
|
31,2.355
|
||||||
|
32,2.679
|
||||||
|
33,3.463
|
||||||
|
34,2.614
|
||||||
|
35,2.922
|
||||||
|
36,2.429
|
||||||
|
37,2.830
|
||||||
|
38,2.842
|
||||||
|
39,2.782
|
||||||
|
40,3.252
|
||||||
|
41,2.731
|
||||||
|
42,2.577
|
||||||
|
43,2.363
|
||||||
|
44,2.786
|
||||||
|
45,2.547
|
||||||
|
46,2.594
|
||||||
|
47,2.652
|
||||||
|
48,2.836
|
||||||
|
49,2.792
|
||||||
|
50,2.552
|
||||||
|
51,2.792
|
||||||
|
52,2.718
|
||||||
|
53,3.333
|
||||||
|
54,2.165
|
||||||
|
55,2.880
|
||||||
|
56,2.613
|
||||||
|
57,2.460
|
||||||
|
58,2.919
|
||||||
|
59,2.770
|
||||||
|
60,2.551
|
||||||
|
61,2.622
|
||||||
|
62,2.712
|
||||||
|
63,2.818
|
||||||
|
64,3.155
|
||||||
|
65,2.472
|
||||||
|
66,2.861
|
||||||
|
67,2.651
|
||||||
|
68,2.573
|
||||||
|
69,2.892
|
||||||
|
70,2.689
|
||||||
|
71,2.729
|
||||||
|
72,2.668
|
||||||
|
73,2.753
|
||||||
|
74,2.812
|
||||||
|
75,2.606
|
||||||
|
76,2.686
|
||||||
|
77,2.842
|
||||||
|
78,2.660
|
||||||
|
79,2.903
|
||||||
|
80,2.486
|
||||||
|
81,2.591
|
||||||
|
82,3.626
|
||||||
|
83,2.455
|
||||||
|
84,2.475
|
||||||
|
85,2.922
|
||||||
|
86,2.603
|
||||||
|
87,2.728
|
||||||
|
88,3.057
|
||||||
|
89,2.731
|
||||||
|
90,3.632
|
||||||
|
91,1.950
|
||||||
|
92,2.694
|
||||||
|
93,3.020
|
||||||
|
94,2.230
|
||||||
|
95,2.796
|
||||||
|
96,2.707
|
||||||
|
97,2.807
|
||||||
|
98,2.658
|
||||||
|
99,2.697
|
||||||
|
100,2.453
|
||||||
|
101,3.051
|
||||||
|
102,2.758
|
||||||
|
103,2.662
|
||||||
|
104,2.599
|
||||||
|
105,2.733
|
||||||
|
106,2.751
|
||||||
|
107,2.894
|
||||||
|
108,2.501
|
||||||
|
109,2.753
|
||||||
|
110,2.873
|
||||||
|
111,2.439
|
||||||
|
112,2.634
|
||||||
|
113,2.682
|
||||||
|
114,2.913
|
||||||
|
115,2.692
|
||||||
|
116,2.735
|
||||||
|
117,2.758
|
||||||
|
118,2.726
|
||||||
|
119,2.907
|
||||||
|
120,2.568
|
||||||
|
121,2.764
|
||||||
|
122,2.815
|
||||||
|
123,2.530
|
||||||
|
124,2.684
|
||||||
|
125,2.719
|
||||||
|
126,2.846
|
||||||
|
127,2.879
|
||||||
|
128,2.647
|
||||||
|
129,2.770
|
||||||
|
130,2.477
|
||||||
|
131,2.657
|
||||||
|
132,2.675
|
||||||
|
133,2.917
|
||||||
|
134,2.700
|
||||||
|
135,2.493
|
||||||
|
136,2.678
|
||||||
|
137,2.786
|
||||||
|
138,3.004
|
||||||
|
139,2.697
|
||||||
|
140,2.642
|
||||||
|
141,2.624
|
||||||
|
142,2.684
|
||||||
|
143,2.717
|
||||||
|
144,2.743
|
||||||
|
145,2.740
|
||||||
|
146,2.825
|
||||||
|
147,2.849
|
||||||
|
148,2.991
|
||||||
|
149,2.297
|
||||||
|
150,3.084
|
||||||
|
151,2.459
|
||||||
|
152,3.214
|
||||||
|
153,2.260
|
||||||
|
154,2.550
|
||||||
|
155,2.621
|
||||||
|
156,2.660
|
||||||
|
157,2.739
|
||||||
|
158,2.574
|
||||||
|
159,3.078
|
||||||
|
160,2.735
|
||||||
|
161,2.607
|
||||||
|
162,2.793
|
||||||
|
163,2.798
|
||||||
|
164,2.584
|
||||||
|
165,2.637
|
||||||
|
166,2.627
|
||||||
|
167,2.726
|
||||||
|
168,2.656
|
||||||
|
169,2.788
|
||||||
|
170,2.674
|
||||||
|
171,2.782
|
||||||
|
172,2.766
|
||||||
|
173,2.613
|
||||||
|
174,2.724
|
||||||
|
175,2.770
|
||||||
|
176,2.709
|
||||||
|
177,2.700
|
||||||
|
178,2.600
|
||||||
|
179,2.686
|
||||||
|
180,2.747
|
||||||
|
181,2.841
|
||||||
|
182,2.601
|
||||||
|
183,2.666
|
||||||
|
184,2.846
|
||||||
|
185,2.599
|
||||||
|
186,2.806
|
||||||
|
187,3.126
|
||||||
|
188,2.521
|
||||||
|
189,2.634
|
||||||
|
190,2.716
|
||||||
|
191,2.584
|
||||||
|
192,2.714
|
||||||
|
193,2.923
|
||||||
|
194,2.277
|
||||||
|
195,2.706
|
||||||
|
196,2.606
|
||||||
|
197,2.758
|
||||||
|
198,2.747
|
||||||
|
199,2.958
|
||||||
|
200,2.798
|
||||||
|
201,2.387
|
||||||
|
202,2.596
|
||||||
|
203,2.861
|
||||||
|
204,2.801
|
||||||
|
205,2.521
|
||||||
|
206,2.869
|
||||||
|
207,2.884
|
||||||
|
208,2.515
|
||||||
|
209,2.592
|
||||||
|
210,3.723
|
||||||
|
211,2.474
|
||||||
|
212,2.736
|
||||||
|
213,2.774
|
||||||
|
214,2.769
|
||||||
|
215,3.007
|
||||||
|
216,2.768
|
||||||
|
217,2.716
|
||||||
|
218,2.694
|
||||||
|
219,2.722
|
||||||
|
220,2.862
|
||||||
|
221,2.587
|
||||||
|
222,2.833
|
||||||
|
223,2.792
|
||||||
|
224,2.746
|
||||||
|
225,2.585
|
||||||
|
226,2.741
|
||||||
|
227,2.585
|
||||||
|
228,2.802
|
||||||
|
229,2.674
|
||||||
|
230,2.742
|
||||||
|
231,2.718
|
||||||
|
232,2.881
|
||||||
|
233,2.707
|
||||||
|
234,2.585
|
||||||
|
235,2.810
|
||||||
|
236,2.948
|
||||||
|
237,2.718
|
||||||
|
238,2.773
|
||||||
|
239,2.571
|
||||||
|
240,2.788
|
||||||
|
241,2.702
|
||||||
|
242,2.634
|
||||||
|
243,2.788
|
||||||
|
244,2.678
|
||||||
|
245,2.914
|
||||||
|
246,2.527
|
||||||
|
247,2.637
|
||||||
|
248,2.877
|
||||||
|
249,2.693
|
||||||
|
250,2.644
|
||||||
|
251,2.670
|
||||||
|
252,2.700
|
||||||
|
253,2.877
|
||||||
|
254,2.638
|
||||||
|
255,2.617
|
||||||
|
256,2.774
|
||||||
|
257,2.660
|
||||||
|
258,2.731
|
||||||
|
259,2.671
|
||||||
|
260,2.838
|
||||||
|
261,2.726
|
||||||
|
262,2.671
|
||||||
|
263,2.744
|
||||||
|
264,2.735
|
||||||
|
265,2.684
|
||||||
|
266,3.850
|
||||||
|
267,2.120
|
||||||
|
268,2.669
|
||||||
|
269,2.729
|
||||||
|
270,2.712
|
||||||
|
271,2.817
|
||||||
|
272,2.926
|
||||||
|
273,2.678
|
||||||
|
274,2.786
|
||||||
|
275,2.927
|
||||||
|
276,2.493
|
||||||
|
277,2.846
|
||||||
|
278,2.610
|
||||||
|
279,2.644
|
||||||
|
280,2.577
|
||||||
|
281,2.939
|
||||||
|
282,2.727
|
||||||
|
283,2.595
|
||||||
|
284,2.819
|
||||||
|
285,2.778
|
||||||
|
286,2.774
|
||||||
|
287,2.628
|
||||||
|
288,2.711
|
||||||
|
289,2.699
|
||||||
|
290,2.715
|
||||||
|
291,2.763
|
||||||
|
292,2.710
|
||||||
|
293,2.892
|
||||||
|
294,2.721
|
||||||
|
295,2.575
|
||||||
|
296,3.608
|
||||||
|
297,2.811
|
||||||
|
298,2.739
|
||||||
|
299,2.751
|
||||||
|
300,2.608
|
||||||
|
301,2.913
|
||||||
|
302,3.052
|
||||||
|
303,2.862
|
||||||
|
304,2.653
|
||||||
|
305,2.727
|
||||||
|
306,2.646
|
||||||
|
307,2.888
|
||||||
|
308,3.338
|
||||||
|
309,2.743
|
||||||
|
310,2.639
|
||||||
|
311,2.681
|
||||||
|
312,2.846
|
||||||
|
313,2.766
|
||||||
|
314,3.184
|
||||||
|
315,2.834
|
||||||
|
316,2.728
|
||||||
|
317,2.581
|
||||||
|
318,2.770
|
||||||
|
319,3.351
|
||||||
|
320,3.157
|
||||||
|
321,2.508
|
||||||
|
322,2.835
|
||||||
|
323,2.754
|
||||||
|
324,2.411
|
||||||
|
325,3.398
|
||||||
|
326,2.802
|
||||||
|
327,2.611
|
||||||
|
328,3.058
|
||||||
|
329,2.407
|
||||||
|
330,2.640
|
||||||
|
331,3.645
|
||||||
|
332,2.729
|
||||||
|
333,2.580
|
||||||
|
334,2.902
|
||||||
|
335,2.523
|
||||||
|
336,3.005
|
||||||
|
337,3.173
|
||||||
|
338,2.840
|
||||||
|
339,2.610
|
||||||
|
340,2.576
|
||||||
|
341,2.706
|
||||||
|
342,3.405
|
||||||
|
343,2.849
|
||||||
|
344,2.678
|
||||||
|
345,2.796
|
||||||
|
346,2.782
|
||||||
|
347,2.636
|
||||||
|
348,3.475
|
||||||
|
349,2.798
|
||||||
|
350,2.745
|
||||||
|
351,2.564
|
||||||
|
352,2.792
|
||||||
|
353,2.714
|
||||||
|
354,3.496
|
||||||
|
355,2.841
|
||||||
|
356,2.465
|
||||||
|
357,2.708
|
||||||
|
358,2.692
|
||||||
|
359,2.934
|
||||||
|
360,3.304
|
||||||
|
361,2.561
|
||||||
|
362,2.689
|
||||||
|
363,2.749
|
||||||
|
364,2.780
|
||||||
|
365,2.940
|
||||||
|
366,3.025
|
||||||
|
367,2.706
|
||||||
|
368,2.881
|
||||||
|
369,2.619
|
||||||
|
370,2.716
|
||||||
|
371,3.133
|
||||||
|
372,2.874
|
||||||
|
373,2.816
|
||||||
|
374,2.414
|
||||||
|
375,2.737
|
||||||
|
376,2.797
|
||||||
|
377,2.651
|
||||||
|
378,2.920
|
||||||
|
379,3.032
|
||||||
|
380,2.606
|
||||||
|
381,2.710
|
||||||
|
382,2.649
|
||||||
|
383,2.700
|
||||||
|
384,3.469
|
||||||
|
385,2.224
|
||||||
|
386,2.567
|
||||||
|
387,2.733
|
||||||
|
388,2.511
|
||||||
|
389,2.950
|
||||||
|
390,3.979
|
||||||
|
391,2.798
|
||||||
|
392,2.487
|
||||||
|
393,2.740
|
||||||
|
394,2.741
|
||||||
|
395,2.718
|
||||||
|
396,3.291
|
||||||
|
397,2.769
|
||||||
|
398,2.697
|
||||||
|
399,2.796
|
||||||
|
400,2.641
|
||||||
|
401,2.652
|
||||||
|
402,3.151
|
||||||
|
403,2.673
|
||||||
|
404,2.741
|
||||||
|
405,2.606
|
||||||
|
406,2.665
|
||||||
|
407,2.859
|
||||||
|
408,2.739
|
||||||
|
409,2.586
|
||||||
|
410,2.910
|
||||||
|
411,2.745
|
||||||
|
412,2.698
|
||||||
|
413,2.670
|
||||||
|
414,2.797
|
||||||
|
415,2.829
|
||||||
|
416,2.643
|
||||||
|
417,2.776
|
||||||
|
418,2.772
|
||||||
|
419,2.530
|
||||||
|
420,2.567
|
||||||
|
421,2.826
|
||||||
|
422,2.782
|
||||||
|
423,2.738
|
||||||
|
424,2.640
|
||||||
|
425,2.967
|
||||||
|
426,2.600
|
||||||
|
427,3.020
|
||||||
|
428,2.502
|
||||||
|
429,2.642
|
||||||
|
430,2.625
|
||||||
|
431,2.770
|
||||||
|
432,2.659
|
||||||
|
433,2.717
|
||||||
|
434,2.698
|
||||||
|
435,2.780
|
||||||
|
436,2.992
|
||||||
|
437,2.686
|
||||||
|
438,2.560
|
||||||
|
439,3.048
|
||||||
|
440,2.721
|
||||||
|
441,2.507
|
||||||
|
442,2.694
|
||||||
|
443,2.695
|
||||||
|
444,2.892
|
||||||
|
445,3.443
|
||||||
|
446,2.633
|
||||||
|
447,2.908
|
||||||
|
448,2.823
|
||||||
|
449,2.346
|
||||||
|
450,2.599
|
||||||
|
451,2.914
|
||||||
|
452,2.774
|
||||||
|
453,3.391
|
||||||
|
454,2.384
|
||||||
|
455,2.585
|
||||||
|
456,2.667
|
||||||
|
457,2.704
|
||||||
|
458,2.674
|
||||||
|
459,2.755
|
||||||
|
460,2.718
|
||||||
|
461,2.804
|
||||||
|
462,2.583
|
||||||
|
463,2.694
|
||||||
|
464,2.797
|
||||||
|
465,3.079
|
||||||
|
466,2.434
|
||||||
|
467,2.723
|
||||||
|
468,3.671
|
||||||
|
469,2.211
|
||||||
|
470,2.567
|
||||||
|
471,2.701
|
||||||
|
472,2.717
|
||||||
|
473,2.657
|
||||||
|
474,2.581
|
||||||
|
475,2.889
|
||||||
|
476,3.006
|
||||||
|
477,2.595
|
||||||
|
478,2.703
|
||||||
|
479,2.498
|
||||||
|
480,2.694
|
||||||
|
481,2.915
|
||||||
|
482,2.655
|
||||||
|
483,2.686
|
||||||
|
484,2.833
|
||||||
|
485,2.717
|
||||||
|
486,2.723
|
||||||
|
487,2.451
|
||||||
|
488,2.913
|
||||||
|
489,2.784
|
||||||
|
490,2.684
|
||||||
|
491,2.651
|
||||||
|
492,2.809
|
||||||
|
493,2.819
|
||||||
|
494,2.676
|
||||||
|
495,2.851
|
||||||
|
496,2.775
|
||||||
|
497,2.638
|
||||||
|
498,2.520
|
||||||
|
499,2.824
|
||||||
|
500,2.930
|
||||||
|
501,2.757
|
||||||
|
502,2.608
|
||||||
|
503,2.808
|
||||||
|
504,2.508
|
||||||
|
505,2.819
|
||||||
|
506,2.654
|
||||||
|
507,2.748
|
||||||
|
508,2.705
|
||||||
|
509,2.798
|
||||||
|
510,2.559
|
||||||
|
511,2.694
|
||||||
|
512,2.821
|
||||||
|
513,2.706
|
||||||
|
514,2.612
|
||||||
|
515,2.928
|
||||||
|
516,2.475
|
||||||
|
517,2.830
|
||||||
|
518,2.658
|
||||||
|
519,2.784
|
||||||
|
520,2.572
|
||||||
|
521,2.935
|
||||||
|
522,2.688
|
||||||
|
523,2.600
|
||||||
|
524,2.675
|
||||||
|
525,2.828
|
||||||
|
526,2.774
|
||||||
|
527,2.848
|
||||||
|
528,2.764
|
||||||
|
529,2.379
|
||||||
|
530,2.962
|
||||||
|
531,2.989
|
||||||
|
532,2.528
|
||||||
|
533,2.639
|
||||||
|
534,2.906
|
||||||
|
535,2.543
|
||||||
|
536,2.582
|
||||||
|
537,2.949
|
||||||
|
538,2.813
|
||||||
|
539,2.504
|
||||||
|
540,2.754
|
||||||
|
541,2.638
|
||||||
|
542,2.695
|
||||||
|
543,2.662
|
||||||
|
544,2.922
|
||||||
|
545,2.616
|
||||||
|
546,2.810
|
||||||
|
547,2.675
|
||||||
|
548,2.551
|
||||||
|
549,2.894
|
||||||
|
550,2.754
|
||||||
|
551,2.632
|
||||||
|
552,2.748
|
||||||
|
553,2.827
|
||||||
|
554,2.751
|
||||||
|
555,2.759
|
||||||
|
556,2.684
|
||||||
|
557,2.766
|
||||||
|
558,2.948
|
||||||
|
559,2.461
|
||||||
|
560,2.621
|
||||||
|
561,2.772
|
||||||
|
562,2.985
|
||||||
|
563,2.582
|
||||||
|
564,2.897
|
||||||
|
565,2.574
|
||||||
|
566,2.661
|
||||||
|
567,2.742
|
||||||
|
568,2.797
|
||||||
|
569,2.632
|
||||||
|
570,2.755
|
||||||
|
571,2.878
|
||||||
|
572,2.759
|
||||||
|
573,3.812
|
||||||
|
574,2.651
|
||||||
|
575,2.862
|
||||||
|
576,2.577
|
||||||
|
577,2.634
|
||||||
|
578,3.173
|
||||||
|
579,2.418
|
||||||
|
580,2.714
|
||||||
|
581,2.746
|
||||||
|
582,2.794
|
||||||
|
583,2.774
|
||||||
|
584,2.583
|
||||||
|
585,3.659
|
||||||
|
586,3.222
|
||||||
|
587,2.489
|
||||||
|
588,2.868
|
||||||
|
589,2.558
|
||||||
|
590,2.586
|
||||||
|
591,2.599
|
||||||
|
592,2.930
|
||||||
|
593,2.401
|
||||||
|
594,2.540
|
||||||
|
595,2.889
|
||||||
|
596,2.733
|
||||||
|
597,2.664
|
||||||
|
598,2.877
|
||||||
|
599,2.588
|
||||||
|
600,2.677
|
||||||
|
601,2.897
|
||||||
|
602,2.672
|
||||||
|
603,2.780
|
||||||
|
604,2.740
|
||||||
|
605,2.780
|
||||||
|
606,2.493
|
||||||
|
607,2.794
|
||||||
|
608,2.631
|
||||||
|
609,2.768
|
||||||
|
610,2.870
|
||||||
|
611,2.626
|
||||||
|
612,2.862
|
||||||
|
613,2.602
|
||||||
|
614,2.868
|
||||||
|
615,2.627
|
||||||
|
616,2.638
|
||||||
|
617,2.702
|
||||||
|
618,2.904
|
||||||
|
619,2.918
|
||||||
|
620,2.487
|
||||||
|
621,3.225
|
||||||
|
622,2.350
|
||||||
|
623,2.820
|
||||||
|
624,2.837
|
||||||
|
625,2.408
|
||||||
|
626,2.593
|
||||||
|
627,3.526
|
||||||
|
628,2.554
|
||||||
|
629,2.746
|
||||||
|
630,2.684
|
||||||
|
631,2.729
|
||||||
|
632,2.762
|
||||||
|
633,2.768
|
||||||
|
634,3.034
|
||||||
|
635,2.514
|
||||||
|
636,2.732
|
||||||
|
637,2.739
|
||||||
|
638,2.692
|
||||||
|
639,2.832
|
||||||
|
640,2.810
|
||||||
|
641,2.484
|
||||||
|
642,2.632
|
||||||
|
643,3.284
|
||||||
|
644,2.311
|
||||||
|
645,3.079
|
||||||
|
646,2.656
|
||||||
|
647,2.683
|
||||||
|
648,2.622
|
||||||
|
649,2.668
|
||||||
|
650,2.779
|
||||||
|
651,2.796
|
||||||
|
652,2.874
|
||||||
|
653,2.427
|
||||||
|
654,2.644
|
||||||
|
655,2.872
|
||||||
|
656,2.867
|
||||||
|
657,2.404
|
||||||
|
658,2.917
|
||||||
|
659,2.773
|
||||||
|
660,2.857
|
||||||
|
661,2.789
|
||||||
|
662,2.443
|
||||||
|
663,2.651
|
||||||
|
664,2.839
|
||||||
|
665,2.814
|
||||||
|
666,2.575
|
||||||
|
667,2.796
|
||||||
|
668,2.709
|
||||||
|
669,2.873
|
||||||
|
670,2.670
|
||||||
|
671,2.936
|
||||||
|
672,2.496
|
||||||
|
673,2.608
|
||||||
|
674,2.641
|
||||||
|
675,2.793
|
||||||
|
676,2.765
|
||||||
|
677,2.736
|
||||||
|
678,2.679
|
||||||
|
679,2.987
|
||||||
|
680,2.884
|
||||||
|
681,2.469
|
||||||
|
682,2.774
|
||||||
|
683,2.948
|
||||||
|
684,2.598
|
||||||
|
685,3.478
|
||||||
|
686,2.856
|
||||||
|
687,4.623
|
||||||
|
688,3.038
|
||||||
|
689,2.507
|
||||||
|
690,2.717
|
||||||
|
691,2.682
|
||||||
|
692,2.807
|
||||||
|
693,2.778
|
||||||
|
694,2.699
|
||||||
|
695,2.802
|
||||||
|
696,2.605
|
||||||
|
697,2.650
|
||||||
|
698,2.789
|
||||||
|
699,2.791
|
||||||
|
700,3.015
|
||||||
|
701,2.539
|
||||||
|
702,2.790
|
||||||
|
703,2.698
|
||||||
|
704,2.637
|
||||||
|
705,2.657
|
||||||
|
706,2.800
|
||||||
|
707,2.720
|
||||||
|
708,2.702
|
||||||
|
709,2.845
|
||||||
|
710,2.583
|
||||||
|
711,2.780
|
||||||
|
712,2.782
|
||||||
|
713,2.593
|
||||||
|
714,2.695
|
||||||
|
715,2.747
|
||||||
|
716,2.871
|
||||||
|
717,2.671
|
||||||
|
718,2.676
|
||||||
|
719,2.991
|
||||||
|
720,2.727
|
||||||
|
721,2.576
|
||||||
|
722,2.539
|
||||||
|
723,2.770
|
||||||
|
724,2.712
|
||||||
|
725,2.818
|
||||||
|
726,2.615
|
||||||
|
+67
-100
@@ -13,7 +13,19 @@ Record results in the table at the end of this file, with the date, the exact bu
|
|||||||
3. **Licensing.** Every new asset has provenance in `THIRD_PARTY_NOTICES.md`, or is recorded as unresolved and kept out of the build.
|
3. **Licensing.** Every new asset has provenance in `THIRD_PARTY_NOTICES.md`, or is recorded as unresolved and kept out of the build.
|
||||||
4. **Version discipline.** Patch releases contain fixes, optimisation, docs, or tuning only — no new scope.
|
4. **Version discipline.** Patch releases contain fixes, optimisation, docs, or tuning only — no new scope.
|
||||||
|
|
||||||
## Performance check
|
## `0.1.0` checks
|
||||||
|
|
||||||
|
1. **Realism first.** Building placement, street width, sidewalk depth, material read and site identity match the reference photography. Compare side by side and note every deliberate deviation. "Looks right" is not evidence.
|
||||||
|
2. Player traverses both the south-side frontage and the north-side frontage of the demo without falling through geometry, including the designated crossing. Walk the route twice: once normally, once with collision debug visualisation enabled.
|
||||||
|
3. No blocking collision, no invisible walls, no gaps at sidewalk/road seams.
|
||||||
|
4. Fresh checkout builds against the documented external engine installation.
|
||||||
|
5. Linux build packaged and launched; controls documented in `Docs/` and shown in-game.
|
||||||
|
6. **Performance last.** 60 FPS at 1440p per the performance check below — a floor every release meets, not the goal that shapes the content.
|
||||||
|
7. Tag `v0.1.0`, write the `CHANGELOG.md` entry, then remove the completed section from `TODO.md` (its goal stays in `ROADMAP.md`).
|
||||||
|
|
||||||
|
## Performance check (last)
|
||||||
|
|
||||||
|
Performance is the **last** check on purpose: realism leads the design and the frame budget only decides how much of it fits. It is still a floor — a release is not tagged below 60 FPS at 1440p without the owner's explicit approval.
|
||||||
|
|
||||||
Measured at 1440p on the development workstation: Bazzite, Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB.
|
Measured at 1440p on the development workstation: Bazzite, Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB.
|
||||||
|
|
||||||
@@ -22,33 +34,20 @@ Measured at 1440p on the development workstation: Bazzite, Ryzen 7 7700X, 64 GB
|
|||||||
- Target: 60 FPS at 1440p without frame generation.
|
- Target: 60 FPS at 1440p without frame generation.
|
||||||
- Capture a frame-time graph over a fixed traversal route, not a single sample. State the route.
|
- Capture a frame-time graph over a fixed traversal route, not a single sample. State the route.
|
||||||
|
|
||||||
## `0.1.0` checks
|
|
||||||
|
|
||||||
1. Player traverses both the south-side frontage and the north-side frontage of the demo without falling through geometry, including the designated crossing. Walk the route twice: once normally, once with collision debug visualisation enabled.
|
|
||||||
2. Building placement, street width, sidewalk depth, and site identity match the reference photography. Compare side by side and note every deliberate deviation. "Looks right" is not evidence.
|
|
||||||
3. No blocking collision, no invisible walls, no gaps at sidewalk/road seams.
|
|
||||||
4. 60 FPS at 1440p per the performance check above.
|
|
||||||
5. Fresh checkout builds against the documented external engine installation.
|
|
||||||
6. Linux build packaged and launched; controls documented in `Docs/` and shown in-game.
|
|
||||||
7. Tag `v0.1.0`, write the `CHANGELOG.md` entry, then remove the completed section from `TODO.md` (its goal stays in `ROADMAP.md`).
|
|
||||||
|
|
||||||
## Check-record template
|
## Check-record template
|
||||||
|
|
||||||
| Version | Check | Result | Evidence | Date |
|
| Version | Check | Result | Evidence | Date |
|
||||||
| ------- | ----------------------- | ------ | -------- | ---- |
|
| --- | --- | --- | --- | --- |
|
||||||
| `0.1.0` | Traversal, both sides | PASS — Blender pre-check + in-engine swept transect, 0 holes, ground Z 317..395 | `-qaselftest`, TODO 0.1.0 notes | 2026-11-08 |
|
| `0.1.0` | Traversal, both sides | PASS — Blender pre-check + in-engine swept transect, 0 holes, ground Z 317..395 | `-qaselftest`, TODO 0.1.0 notes | 2026-11-08 |
|
||||||
| `0.1.0` | Placement vs references | PASS (with deliberate simplifications) | Section below, 2026-11-08 | 2026-11-08 |
|
| `0.1.0` | Placement vs references | PASS (with deliberate simplifications) | Section below, 2026-11-08 | 2026-11-08 |
|
||||||
| `0.1.0` | Collision and packaging | PASS — no blocking collision, no invisible walls, no seam gaps (see audit note) | `-qaselftest` on the packaged build, 2026-11-08 | 2026-11-08 |
|
| `0.1.0` | Collision and packaging | PASS — no blocking collision, no invisible walls, no seam gaps (see audit note) | `-qaselftest` on the packaged build, 2026-11-08 | 2026-11-08 |
|
||||||
| `0.1.0` | 60 FPS at 1440p | PASS — 2.42 ms avg (414 fps), p99 3.01 ms at 2560×1440, TSR on, VSync off, no FG | `-qaselftest` perf capture, `Docs/perf/0.1.0-walk-frametimes.csv` | 2026-11-08 |
|
| `0.1.0` | 60 FPS at 1440p | PASS — 2.42 ms avg (414 fps), p99 3.01 ms at 2560×1440, TSR on, VSync off, no FG | `-qaselftest` perf capture, `Docs/perf/0.1.0-walk-frametimes.csv` | 2026-11-08 |
|
||||||
| `0.1.0` | Fresh checkout build | PASS — clean clone of `main` (`4bc41e9`), `git lfs pull` + `check-lfs.sh` OK (130 LFS files), both `GigLineEditor` and `GigLine` targets built with the documented engine | `/tmp/fresh-check`, 2026-11-08 | 2026-11-08 |
|
| `0.1.0` | Fresh checkout build | PASS — clean clone of `main` (`4bc41e9`), `git lfs pull` + `check-lfs.sh` OK (130 LFS files), both `GigLineEditor` and `GigLine` targets built with the documented engine | `/tmp/fresh-check`, 2026-11-08 | 2026-11-08 |
|
||||||
| `0.1.0` | Packaged Linux build | PASS — packaged Development build launches straight into DemoMap, no missing-asset/linker errors; controls shown in-game and in `Docs/controls.md` | `Docs/perf/0.1.0-packaged-hud-controls.png`, 2026-11-08 | 2026-11-08 |
|
| `0.1.0` | Packaged Linux build | PASS — packaged Development build launches straight into DemoMap, no missing-asset/linker errors; controls shown in-game and in `Docs/controls.md` | `Docs/perf/0.1.0-packaged-hud-controls.png`, 2026-11-08 | 2026-11-08 |
|
||||||
|
|
||||||
## `0.1.0` — Placement vs references — record (2026-11-08)
|
## `0.1.0` — Placement vs references — record (2026-11-08)
|
||||||
|
|
||||||
Checked against the reference photography (`Art/reference/`) and the Geo measurements
|
Checked against the reference photography (`Art/reference/`) and the Geo measurements (`Geo/verification/*`). Build under test: current `main` content, DemoMap rendered via `Tools/render_screenshots.sh` (2026-11-08, Vulkan SM5, 1920×1080); engine positions probed from the DemoMap (commandlet, 2026-11-08).
|
||||||
(`Geo/verification/*`). Build under test: current `main` content, DemoMap rendered via
|
|
||||||
`Tools/render_screenshots.sh` (2026-11-08, Vulkan SM5, 1920×1080); engine positions
|
|
||||||
probed from the DemoMap (commandlet, 2026-11-08).
|
|
||||||
|
|
||||||
### Placement and scale (in-engine ↔ reference)
|
### Placement and scale (in-engine ↔ reference)
|
||||||
|
|
||||||
@@ -80,78 +79,43 @@ probed from the DemoMap (commandlet, 2026-11-08).
|
|||||||
- **Crossing ramps are kerb flushes, not graded ramps** — the `0.1.1` Crossing ramp gradient item.
|
- **Crossing ramps are kerb flushes, not graded ramps** — the `0.1.1` Crossing ramp gradient item.
|
||||||
- Site identity (the neoclassical hero with motto band) matches the 2009 façade photograph and the 1922 Masonic elevation within blockout fidelity.
|
- Site identity (the neoclassical hero with motto band) matches the 2009 façade photograph and the 1922 Masonic elevation within blockout fidelity.
|
||||||
|
|
||||||
Pre-check method: `Tools/judge_screenshot.py` (vision model) flagged the simplified
|
Pre-check method: `Tools/judge_screenshot.py` (vision model) flagged the simplified flat surfaces as "unexpected" — those flags are the deliberate deviations above, not placement errors. Final verdict is by human review of the shots in `Screenshots/demo/`.
|
||||||
flat surfaces as "unexpected" — those flags are the deliberate deviations above, not
|
|
||||||
placement errors. Final verdict is by human review of the shots in `Screenshots/demo/`.
|
|
||||||
|
|
||||||
## `0.1.0` — Collision and packaging — audit note (2026-11-08)
|
## `0.1.0` — Collision and packaging — audit note (2026-11-08)
|
||||||
|
|
||||||
Run on the packaged build (`GigLineQA`, Development, Vulkan SM5) with `-qaselftest`.
|
Run on the packaged build (`GigLineQA`, Development, Vulkan SM5) with `-qaselftest`.
|
||||||
|
|
||||||
- **Transect (holes/gaps):** 1456 samples over 16 rows — lane centres **plus the
|
- **Transect (holes/gaps):** 1456 samples over 16 rows — lane centres **plus the kerb/road/sidewalk seam Ys** (−1382, −1053, +718, +1047 and neighbours) and X −20000..−2000. Result **0 holes**. (The old 84-sample check covered only 3 lane centres; the seam rows are what this check adds.)
|
||||||
kerb/road/sidewalk seam Ys** (−1382, −1053, +718, +1047 and neighbours) and
|
- **Invisible-wall probe:** horizontal traces 30 m long, grounded at 160 cm above the local ground, along the road and both seam rows. Result **0/35 blocking hits inside 21 m** — no invisible walls or misplaced blockers in the street envelope.
|
||||||
X −20000..−2000. Result **0 holes**. (The old 84-sample check covered only 3
|
- **Facade occlusion probe:** from the road centre, 40 m traces sideways at walking height toward each frontage, required only where the map stages a building (the spans are listed in `AGigLineCharacter` and mirror `Tools/ue_build_demo_map.py`). Result **0/9 directions without a blocking wall** — the player cannot walk through the hero or the frontage. This probe found the hero façade had **no simple collision** at all (complex-only, so traces and capsule sweeps passed through it); it is now `UseComplexAsSimple`. Auto-generated convex hulls were tried first and rejected: on a 95-part, 26 m façade they bulged into the street (the wall probe went 0/35 → 2/35). (First pass used a fixed Z 160 start; the street is at Z ~366, so the probe started inside the earth and reported every row as a dist-0 blocker. Grounding the probe fixed that measurement bug.)
|
||||||
lane centres; the seam rows are what this check adds.)
|
- **Walk / reset / pause:** 761 cm in 2 s (Z steady 456), reset 0 cm, pause `IsGamePaused()=1`, unpause 0 — all PASS.
|
||||||
- **Invisible-wall probe:** horizontal traces 30 m long, grounded at 160 cm above
|
- Packaging: the packaged build launches, reaches gameplay, and the QA run quits cleanly. The `GigLineShotActor` screenshot rounds are suppressed under `-qaselftest` so they cannot quit the run before the audit phases.
|
||||||
the local ground, along the road and both seam rows. Result **0/35 blocking hits
|
- Caveat recorded: ground Z extremes (−12 m / 3000 cm) are trace-start/terrain-edge artifacts of the widened extent, not walkable holes — the walkable corridor is continuous (walk + transect both PASS).
|
||||||
inside 21 m** — no invisible walls or misplaced blockers in the street envelope.
|
|
||||||
- **Facade occlusion probe:** from the road centre, 40 m traces sideways at walking
|
|
||||||
height toward each frontage, required only where the map stages a building (the
|
|
||||||
spans are listed in `AGigLineCharacter` and mirror `Tools/ue_build_demo_map.py`).
|
|
||||||
Result **0/9 directions without a blocking wall** — the player cannot walk
|
|
||||||
through the hero or the frontage. This probe found the hero façade had **no
|
|
||||||
simple collision** at all (complex-only, so traces and capsule sweeps passed
|
|
||||||
through it); it is now `UseComplexAsSimple`. Auto-generated convex hulls were
|
|
||||||
tried first and rejected: on a 95-part, 26 m façade they bulged into the street
|
|
||||||
(the wall probe went 0/35 → 2/35).
|
|
||||||
(First pass used a fixed Z 160 start; the street is at Z ~366, so the probe
|
|
||||||
started inside the earth and reported every row as a dist-0 blocker. Grounding
|
|
||||||
the probe fixed that measurement bug.)
|
|
||||||
- **Walk / reset / pause:** 761 cm in 2 s (Z steady 456), reset 0 cm, pause
|
|
||||||
`IsGamePaused()=1`, unpause 0 — all PASS.
|
|
||||||
- Packaging: the packaged build launches, reaches gameplay, and the QA run quits
|
|
||||||
cleanly. The `GigLineShotActor` screenshot rounds are suppressed under
|
|
||||||
`-qaselftest` so they cannot quit the run before the audit phases.
|
|
||||||
- Caveat recorded: ground Z extremes (−12 m / 3000 cm) are trace-start/terrain-edge
|
|
||||||
artifacts of the widened extent, not walkable holes — the walkable corridor is
|
|
||||||
continuous (walk + transect both PASS).
|
|
||||||
|
|
||||||
## `0.1.0` — 60 FPS at 1440p — record (2026-11-08)
|
## `0.1.0` — 60 FPS at 1440p — record (2026-11-08)
|
||||||
|
|
||||||
**Build under test:** packaged `GigLineQA` Development, UE **5.8.4**, Vulkan SM5,
|
**Build under test:** packaged `GigLineQA` Development, UE **5.8.4**, Vulkan SM5, NVIDIA 615.78.08 (RTX 5070 Ti), Bazzite, Ryzen 7 7700X, 64 GB RAM.
|
||||||
NVIDIA 615.78.08 (RTX 5070 Ti), Bazzite, Ryzen 7 7700X, 64 GB RAM.
|
|
||||||
|
|
||||||
**Documented quality preset** (applied via `-ExecCmds`): all scalability groups at
|
**Documented quality preset** (applied via `-ExecCmds`): all scalability groups at 3 = Epic — `sg.ViewDistanceQuality 3, sg.ShadowQuality 3, sg.GlobalIlluminationQuality 3, sg.ReflectionQuality 3, sg.PostProcessQuality 3, sg.TextureQuality 3, sg.EffectsQuality 3, sg.FoliageQuality 3, sg.ShadingQuality 3`.
|
||||||
3 = Epic — `sg.ViewDistanceQuality 3, sg.ShadowQuality 3,
|
|
||||||
sg.GlobalIlluminationQuality 3, sg.ReflectionQuality 3, sg.PostProcessQuality 3,
|
|
||||||
sg.TextureQuality 3, sg.EffectsQuality 3, sg.FoliageQuality 3, sg.ShadingQuality 3`.
|
|
||||||
|
|
||||||
**Upscaling state (recorded from the run):** `r.AntiAliasingMethod=4` → **TSR on**,
|
**Upscaling state (recorded from the run):** `r.AntiAliasingMethod=4` → **TSR on**, default screen percentage, `r.VSync=0` (**off**), frame generation **off/unavailable** (`r.Streamline.FrameGeneration.Mode` absent → -999).
|
||||||
default screen percentage, `r.VSync=0` (**off**), frame generation **off/unavailable**
|
|
||||||
(`r.Streamline.FrameGeneration.Mode` absent → -999).
|
|
||||||
|
|
||||||
**Route:** the deterministic `-qaselftest` traversal — spawn
|
**Route:** the deterministic `-qaselftest` traversal — spawn `(-13336,-1150,456)`, walk forward (+Y) into the block for 2 s. Frame times are captured every frame over that route (829 frames), not a single sample.
|
||||||
`(-13336,-1150,456)`, walk forward (+Y) into the block for 2 s. Frame times are
|
|
||||||
captured every frame over that route (829 frames), not a single sample.
|
|
||||||
|
|
||||||
**Result at 2560×1440:**
|
**Result at 2560×1440:**
|
||||||
|
|
||||||
| Metric | Value |
|
| Metric | Value |
|
||||||
| --- | --- |
|
| --------------- | --------------------- |
|
||||||
| Frames captured | 829 |
|
| Frames captured | 829 |
|
||||||
| Average | **2.42 ms → 414 fps** |
|
| Average | **2.42 ms → 414 fps** |
|
||||||
| p50 | 2.38 ms |
|
| p50 | 2.38 ms |
|
||||||
| p95 | 2.71 ms |
|
| p95 | 2.71 ms |
|
||||||
| p99 | 3.01 ms (≈332 fps) |
|
| p99 | 3.01 ms (≈332 fps) |
|
||||||
| Max | 4.00 ms |
|
| Max | 4.00 ms |
|
||||||
|
|
||||||
**Verdict: PASS** — worst-case p99 is ~3 ms (≈330 fps), far above the 60 FPS
|
**Verdict: PASS** — worst-case p99 is ~3 ms (≈330 fps), far above the 60 FPS target without frame generation.
|
||||||
target without frame generation.
|
|
||||||
|
|
||||||
**Variance note:** two back-to-back runs measured ~9.2 ms (≈108 fps) when another
|
**Variance note:** two back-to-back runs measured ~9.2 ms (≈108 fps) when another game instance / busy compositor overlapped the capture; repeated clean runs (windowed and fullscreen, Wayland and X11 drivers, window focused) all measured ~2.4–2.5 ms. The high outlier is recorded here rather than hidden.
|
||||||
game instance / busy compositor overlapped the capture; repeated clean runs
|
|
||||||
(windowed and fullscreen, Wayland and X11 drivers, window focused) all measured
|
|
||||||
~2.4–2.5 ms. The high outlier is recorded here rather than hidden.
|
|
||||||
|
|
||||||
**Reproduce:**
|
**Reproduce:**
|
||||||
|
|
||||||
@@ -163,10 +127,25 @@ game instance / busy compositor overlapped the capture; repeated clean runs
|
|||||||
|
|
||||||
Committed frame-time series: `Docs/perf/0.1.0-walk-frametimes.csv`.
|
Committed frame-time series: `Docs/perf/0.1.0-walk-frametimes.csv`.
|
||||||
|
|
||||||
|
### `0.1.5` re-measure with the façade kit in the map
|
||||||
|
|
||||||
|
Same route, same preset, packaged Development build, **2560×1440**:
|
||||||
|
|
||||||
|
| Metric | `0.1.0` | `0.1.5` (kit + textures) |
|
||||||
|
| ------- | ----------------: | -----------------------: |
|
||||||
|
| Average | 2.42 ms (414 fps) | **2.76 ms (363 fps)** |
|
||||||
|
| p50 | 2.38 ms | 2.73 ms |
|
||||||
|
| p95 | 2.71 ms | 3.17 ms |
|
||||||
|
| p99 | 3.01 ms | 3.64 ms |
|
||||||
|
| Max | 4.00 ms | 4.62 ms |
|
||||||
|
|
||||||
|
The **625 kit actors and 8 generated textures cost about 12 %** of frame time and still leave **363 fps average, p99 3.64 ms** — roughly six times the 60 fps floor. Frame-time series: `Docs/perf/0.1.5-kit-walk-frametimes.csv`.
|
||||||
|
|
||||||
|
**Measurement trap, again:** runs that reported ~105 fps were two game instances overlapping (a previous launch had not exited). Confirm a single instance (`pgrep -af Binaries/Linux/GigLine`) before trusting a figure — 363 fps reproduced twice on a clean run.
|
||||||
|
|
||||||
## `0.1.0` — Fresh checkout build — record (2026-11-08)
|
## `0.1.0` — Fresh checkout build — record (2026-11-08)
|
||||||
|
|
||||||
Followed the documented steps in [`Docs/build.md`](build.md) exactly, into a clean
|
Followed the documented steps in [`Docs/build.md`](build.md) exactly, into a clean directory with no prior state:
|
||||||
directory with no prior state:
|
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
git clone https://git.allucanget.biz/zwitschi/gig-line.git /tmp/fresh-check
|
git clone https://git.allucanget.biz/zwitschi/gig-line.git /tmp/fresh-check
|
||||||
@@ -179,35 +158,23 @@ Engine/Build/BatchFiles/Linux/Build.sh GigLine Linux Development \
|
|||||||
-project=/tmp/fresh-check/Game/GigLine/GigLine.uproject -WaitMutex
|
-project=/tmp/fresh-check/Game/GigLine/GigLine.uproject -WaitMutex
|
||||||
```
|
```
|
||||||
|
|
||||||
- Clone of `main` at `4bc41e9`; LFS objects fetched (`DemoMap.umap` is real UE
|
- Clone of `main` at `4bc41e9`; LFS objects fetched (`DemoMap.umap` is real UE package content, not a pointer).
|
||||||
package content, not a pointer).
|
- `GigLineEditor` → **Result: Succeeded** (28.2 s); produced `/tmp/fresh-check/Game/GigLine/Binaries/Linux/libUnrealEditor-GigLine.so`.
|
||||||
- `GigLineEditor` → **Result: Succeeded** (28.2 s); produced
|
|
||||||
`/tmp/fresh-check/Game/GigLine/Binaries/Linux/libUnrealEditor-GigLine.so`.
|
|
||||||
- `GigLine` (game) → **Result: Succeeded** (26.9 s).
|
- `GigLine` (game) → **Result: Succeeded** (26.9 s).
|
||||||
- Engine: documented external install at `/run/media/system/DATA/Engines/UE_5.8.4`
|
- Engine: documented external install at `/run/media/system/DATA/Engines/UE_5.8.4` (UE 5.8.4), system clang; no machine-specific paths beyond the documented `-project=` argument.
|
||||||
(UE 5.8.4), system clang; no machine-specific paths beyond the documented
|
|
||||||
`-project=` argument.
|
|
||||||
|
|
||||||
**Verdict: PASS.**
|
**Verdict: PASS.**
|
||||||
|
|
||||||
## `0.1.0` — Packaged Linux build + documented controls — record (2026-11-08)
|
## `0.1.0` — Packaged Linux build + documented controls — record (2026-11-08)
|
||||||
|
|
||||||
**Package:** `RunUAT BuildCookRun … -platform=Linux -clientconfig=Development
|
**Package:** `RunUAT BuildCookRun … -platform=Linux -clientconfig=Development -cook -build -stage -pak -archive -archivedirectory=/run/media/system/DATA/Builds/GigLineQA` → `EXIT=0` (Success).
|
||||||
-cook -build -stage -pak -archive -archivedirectory=/run/media/system/DATA/Builds/GigLineQA`
|
|
||||||
→ `EXIT=0` (Success).
|
|
||||||
|
|
||||||
**Launch:** `./GigLine.sh -windowed` (plain, no map argument) →
|
**Launch:** `./GigLine.sh -windowed` (plain, no map argument) → `LogLoad: LoadMap: /Game/Hollywood/DemoMap?Name=Player` — the packaged game opens the walkable demo directly (the default-map fix holds in a package, not just the editor). No `LogLinker`/missing-asset/failed-to-find errors in the run.
|
||||||
`LogLoad: LoadMap: /Game/Hollywood/DemoMap?Name=Player` — the packaged game opens
|
|
||||||
the walkable demo directly (the default-map fix holds in a package, not just the
|
|
||||||
editor). No `LogLinker`/missing-asset/failed-to-find errors in the run.
|
|
||||||
|
|
||||||
**Controls documented and shown in-game:**
|
**Controls documented and shown in-game:**
|
||||||
- `Docs/controls.md` — full binding table (WASD, mouse look, Space, R, Esc) plus
|
|
||||||
how to test input headlessly.
|
- `Docs/controls.md` — full binding table (WASD, mouse look, Space, R, Esc) plus how to test input headlessly.
|
||||||
- `AGigLineHUD` now draws a control legend bottom-right every frame:
|
- `AGigLineHUD` now draws a control legend bottom-right every frame: "WASD move · Mouse look · Space jump · R reset · Esc pause".
|
||||||
"WASD move · Mouse look · Space jump · R reset · Esc pause".
|
- Verified in the packaged capture: the bottom-right text bbox is 426×12 px (a single right-aligned line), attributed text bottom-left. Screenshot: `Docs/perf/0.1.0-packaged-hud-controls.png`.
|
||||||
- Verified in the packaged capture: the bottom-right text bbox is 426×12 px
|
|
||||||
(a single right-aligned line), attributed text bottom-left. Screenshot:
|
|
||||||
`Docs/perf/0.1.0-packaged-hud-controls.png`.
|
|
||||||
|
|
||||||
**Verdict: PASS.**
|
**Verdict: PASS.**
|
||||||
@@ -0,0 +1,80 @@
|
|||||||
|
# `0.1.4` façade review — signoff
|
||||||
|
|
||||||
|
Four demo-map captures from the `facades-0.1.4` branch, after the module kit, the textured materials and the theatre marquee went in. **All seven items signed off 2026-11-09** (owner review of the local renders).
|
||||||
|
|
||||||
|
Reference photographs to compare against: `Art/reference/hollywood/` and `Art/reference/el-capitan-*` (local, not committed). Regenerate the set with `Tools/render_screenshots.sh --verify`; measure it with `python3 Tools/visual_loop.py --compare --iteration 5`.
|
||||||
|
|
||||||
|
## Signoff
|
||||||
|
|
||||||
|
- [x] **1. Block from the north sidewalk** — hero between its neighbours, sidewalk and road read correctly
|
||||||
|
- [x] **2. Street level** — colonnade, shopfronts and window rhythm hold up at eye height
|
||||||
|
- [x] **3. Crossing** — zebra bars, kerb line and the frontage behind
|
||||||
|
- [x] **4. North buildings** — TCL and Tussaud's framed across the boulevard
|
||||||
|
- [x] **5. Materials** — stone reads as coursed stone, glazing as glass, not flat colour
|
||||||
|
- [x] **6. Theatre trim** — marquee and blade sign proportions believable at street level
|
||||||
|
- [x] **7. Approve** `v0.1.4` (kit + materials + theatre trim); remaining geometry moves to the next version
|
||||||
|
|
||||||
|
## 1. Block from the north sidewalk
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
|
Expected: the neoclassical hero façade between its neighbours, both sidewalks and the road in the foreground. _Metric: 0.68 similarity, edges 0.28× reference._
|
||||||
|
|
||||||
|
## 2. Street level
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
|
Expected: the hero colonnade with adjacent frontage buildings either side, shopfronts at pavement level, 4.96 m bay rhythm. _Metric: 0.33, edges 0.27×._
|
||||||
|
|
||||||
|
## 3. The designated crossing
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
|
Expected: continental zebra bars on the asphalt at s=130 m, south frontage and colonnade behind. _Metric: 0.42, edges 0.34×._
|
||||||
|
|
||||||
|
## 4. North-side buildings
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
|
Expected: TCL Chinese Theatre and Madame Tussaud's across Hollywood Blvd, full height in frame. _Metric: 0.34, edges 0.14×._
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Appendix — what was checked, and what was broken
|
||||||
|
|
||||||
|
### Verified
|
||||||
|
|
||||||
|
- **Behaviour**, packaged build, `-qaselftest`: walk 761 cm in 2 s (Z steady 456), reset 0 cm, pause/unpause, seam transect 1456 samples / **0 holes**, wall probe **0/35**, façade occlusion probe **0/9** — all PASS. Run twice: the harness is time-based and one loaded run reported a false failure (`walk 246 cm`, `pause FAIL`), an immediate re-run passed — re-run before believing a single FAIL.
|
||||||
|
- **Placement**, probed from the live map against `Geo/verification/2026-11-06-el-capitan-block.md`: 6922 −24276…−17700, 6902 −17749…−14591, **hero −14644…−12028 between them**, theatre −12082…−8210, 6822 / 6806 / 6800 −7854…−1437, TCL −22889…−18442, Tussaud's −27071…−22799. Blockout affine residual **0.0 cm**.
|
||||||
|
- **Budgets**: hero 148 parts / 3 044 tris · kit 16 meshes × 12 tris · 625 kit actors ≈ 7 500 tris · 8 × 512² textures (196 KB) · 67 tool tests pass.
|
||||||
|
- **Topology**: every kit piece manifold, no loose geometry.
|
||||||
|
|
||||||
|
### Defects found this cycle
|
||||||
|
|
||||||
|
Every one of these rendered convincingly; the placement ones were caught by coordinate probes, not by looking.
|
||||||
|
|
||||||
|
| Sev | Defect | Root cause |
|
||||||
|
| --- | --- | --- |
|
||||||
|
| Blocker | Frontage shifted ≈119 m X / 2.4 m Y, hero outside its gap | Blockout regenerated without `--origin` |
|
||||||
|
| Blocker | Kit placed ~12 km from its buildings | Used mesh-local bbox; actors carry the offset |
|
||||||
|
| Blocker | Marquee on the far side of the boulevard | Blender +Y is Unreal −Y |
|
||||||
|
| Major | Marquee canopy 5 cm above the pavement | Absolute Z; pavement is 4.56 m above datum |
|
||||||
|
| Major | Window bays invisible, inside the solid wall | Piece modelled entirely at +Y |
|
||||||
|
| Major | Glazing wasn't glass | Grey `M_Kit_*` from the FBX import, never overridden |
|
||||||
|
| Major | Hero windows floated 1.44 m in front of the wall | Wrong Y in the builder |
|
||||||
|
| Major | Parapet medallions were discs on the coping | Built with the Z-axis cylinder helper |
|
||||||
|
| Major | Hero had no usable collision | Complex-only; traces use simple |
|
||||||
|
| Minor | "No smoothing group information" on every import | `mesh_smooth_type="OFF"` in six exporters |
|
||||||
|
| Minor | `save_map()` returned False silently | LFS `.umap` read-only, importer never chmod'd |
|
||||||
|
| Minor | North-buildings import silently no-op | Missing `import sys` |
|
||||||
|
| Minor | Kit folder held 24 assets (8 orphan materials) | No stale-asset sweep |
|
||||||
|
| Minor | Edge-density metric counted PIL's border ring | `FIND_EDGES` leaves a 128 ring |
|
||||||
|
| Minor | QA wall probe grazed the theatre wall | Scan row 1 cm from the building line |
|
||||||
|
|
||||||
|
### Limitations worth knowing
|
||||||
|
|
||||||
|
- Kit pieces are boxes; dentil spacing, abacus and plinth dimensions are recorded assumptions, not measurements.
|
||||||
|
- Textures are generated, not authored — no decals, signage text or weathering.
|
||||||
|
- The vision pre-check stays advisory: it has rejected shots the owner approved.
|
||||||
|
- The low edge ratios in the metrics are partly structural — a wide block view against a tight façade crop always scores low. Weigh the images, not the scores.
|
||||||
|
- Performance has not been re-measured since `0.1.0`; the 625 kit actors are unmeasured (tracked in `TODO.md` under `0.1.5`).
|
||||||
+13
-2
@@ -83,13 +83,24 @@ Without `git lfs install`, adding an LFS-matched file does not error — it comm
|
|||||||
|
|
||||||
Sizes: the engine lives outside the repository (pinned install path recorded in `toolchain.md`). The repository holds the Unreal project, Blender sources, geo sources, and tooling.
|
Sizes: the engine lives outside the repository (pinned install path recorded in `toolchain.md`). The repository holds the Unreal project, Blender sources, geo sources, and tooling.
|
||||||
|
|
||||||
**Fresh checkout not yet tested end to end.** The build procedure in [build.md](build.md) was produced from a working tree, not from a clean clone. Exercise it before `v0.1.0` is tagged — it is a release check on the board.
|
**Fresh checkout verified 2026-11-08.** A clean clone of `main` with `git lfs pull` built both `GigLineEditor` and `GigLine` against the documented external engine (recorded in [release-checks.md](release-checks.md)). Re-run it before a release tag if the engine pin or the module's dependencies change.
|
||||||
|
|
||||||
|
### Optional keys (local only)
|
||||||
|
|
||||||
|
Two tools talk to paid services and read their keys from the gitignored `.env`:
|
||||||
|
|
||||||
|
| Variable | Used by | Needed for |
|
||||||
|
| --- | --- | --- |
|
||||||
|
| `OPENROUTER_API_KEY` | `Tools/judge_screenshot.py`, `Tools/facade_detail_concepts.py` | vision pre-check and AI concept plates |
|
||||||
|
| `MAPBOX_ACCESS_TOKEN` | `Geo/tools/mapbox_buildings.py` | the second building-height source |
|
||||||
|
|
||||||
|
Nothing in the build, the tests or the release path needs either key. Never commit `.env`; `git check-ignore .env` should succeed.
|
||||||
|
|
||||||
## Known blockers
|
## Known blockers
|
||||||
|
|
||||||
| Blocker | Owner | Needed for |
|
| Blocker | Owner | Needed for |
|
||||||
| --- | --- | --- |
|
| --- | --- | --- |
|
||||||
| Delete the fallback 5.2.0 engine archive (`/run/media/system/DATA/Downloads/Linux_Unreal_Engine_5.2.0.zip`) once the 5.8.4 pin is trusted through asset production | agent | Disk hygiene (19.6 GiB), pin safety |
|
| Delete the fallback 5.2.0 engine archive (`/run/media/system/DATA/Downloads/Linux_Unreal_Engine_5.2.0.zip`) once the 5.8.4 pin is trusted through asset production | agent | Disk hygiene (19.6 GiB), pin safety |
|
||||||
| Fresh-checkout build exercised against the documented procedure | agent | `v0.1.0` release check |
|
| ~~Fresh-checkout build exercised against the documented procedure~~ | — | Done 2026-11-08 |
|
||||||
|
|
||||||
Closed 2026-11-07: Epic account and engine download, QGIS `lts` removal, Blosm and MPFB on Blender 5.2 (headless invocation fix in `Art/blender/README.md`), engine validation on Bazzite, full project bootstrap. Earlier closed: Bazzite update and reboot, driver and Vulkan verification, `git-lfs`/`clang`/`cmake`/`ninja` layering.
|
Closed 2026-11-07: Epic account and engine download, QGIS `lts` removal, Blosm and MPFB on Blender 5.2 (headless invocation fix in `Art/blender/README.md`), engine validation on Bazzite, full project bootstrap. Earlier closed: Bazzite update and reboot, driver and Vulkan verification, `git-lfs`/`clang`/`cmake`/`ninja` layering.
|
||||||
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
@@ -379,7 +379,11 @@ void AGigLineCharacter::StartQASelfTest()
|
|||||||
// trace at a fixed Z (e.g. 160) is wrong: the street is at Z ~366
|
// trace at a fixed Z (e.g. 160) is wrong: the street is at Z ~366
|
||||||
// there, so a 160 cm probe starts inside the earth (dist 0).
|
// there, so a 160 cm probe starts inside the earth (dist 0).
|
||||||
int32 Walls = 0, WallSamples = 0;
|
int32 Walls = 0, WallSamples = 0;
|
||||||
static const float TraceYs[] = {-1400.0f, -167.0f, 1000.0f, -1382.0f, 1047.0f};
|
// Sample the street envelope only: the carriageway plus the walkable
|
||||||
|
// part of each sidewalk. Y 1000 used to be in this list and sits ~1 cm
|
||||||
|
// from the frontage building line, so it grazed the theatre wall once
|
||||||
|
// the kit's proud shopfronts went in - a probe artefact, not a blocker.
|
||||||
|
static const float TraceYs[] = {-1400.0f, -167.0f, 800.0f, -1382.0f, 718.0f};
|
||||||
for (float WallY : TraceYs)
|
for (float WallY : TraceYs)
|
||||||
{
|
{
|
||||||
for (float X0 = -19500.0f; X0 <= -3000.0f; X0 += 2500.0f)
|
for (float X0 = -19500.0f; X0 <= -3000.0f; X0 += 2500.0f)
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
#include "GigLineShotActor.h"
|
#include "GigLineShotActor.h"
|
||||||
|
|
||||||
#include "Camera/CameraActor.h"
|
#include "Camera/CameraActor.h"
|
||||||
|
#include "Camera/CameraComponent.h"
|
||||||
#include "Engine/World.h"
|
#include "Engine/World.h"
|
||||||
#include "GameFramework/PlayerController.h"
|
#include "GameFramework/PlayerController.h"
|
||||||
#include "UnrealClient.h"
|
#include "UnrealClient.h"
|
||||||
@@ -83,6 +84,15 @@ void AGigLineShotActor::NextShot()
|
|||||||
Params.ObjectFlags |= RF_Transient;
|
Params.ObjectFlags |= RF_Transient;
|
||||||
CurrentCamera = GetWorld()->SpawnActor<ACameraActor>(
|
CurrentCamera = GetWorld()->SpawnActor<ACameraActor>(
|
||||||
ACameraActor::StaticClass(), ShotLocations[CurrentShot], ShotRotations[CurrentShot], Params);
|
ACameraActor::StaticClass(), ShotLocations[CurrentShot], ShotRotations[CurrentShot], Params);
|
||||||
|
if (ShotFieldOfView.IsValidIndex(CurrentShot) && ShotFieldOfView[CurrentShot] > 0.0f)
|
||||||
|
{
|
||||||
|
if (UCameraComponent* Cam = CurrentCamera->GetCameraComponent())
|
||||||
|
{
|
||||||
|
// Long lens for elevations: fills the frame from across the street
|
||||||
|
// without pitching the camera, so verticals stay parallel.
|
||||||
|
Cam->SetFieldOfView(ShotFieldOfView[CurrentShot]);
|
||||||
|
}
|
||||||
|
}
|
||||||
PlayerController->SetViewTarget(CurrentCamera);
|
PlayerController->SetViewTarget(CurrentCamera);
|
||||||
|
|
||||||
SettleCountdown = SettleSeconds;
|
SettleCountdown = SettleSeconds;
|
||||||
|
|||||||
@@ -36,6 +36,16 @@ public:
|
|||||||
UPROPERTY(EditAnywhere, BlueprintReadWrite)
|
UPROPERTY(EditAnywhere, BlueprintReadWrite)
|
||||||
TArray<FString> ShotNames;
|
TArray<FString> ShotNames;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Optional per-shot field of view (degrees), same index as ShotLocations.
|
||||||
|
* Zero or missing entries keep the engine default (90 deg). Elevation shots
|
||||||
|
* taken from across the street need a long lens: a level camera 45 m out with
|
||||||
|
* the default FOV leaves the building small in frame, and tilting it up to fill
|
||||||
|
* the frame is what breaks the verticals.
|
||||||
|
*/
|
||||||
|
UPROPERTY(EditAnywhere, BlueprintReadWrite)
|
||||||
|
TArray<float> ShotFieldOfView;
|
||||||
|
|
||||||
/** Seconds to let the view target settle before the capture. Resolution comes from the runner's -ResX/-ResY. */
|
/** Seconds to let the view target settle before the capture. Resolution comes from the runner's -ResX/-ResY. */
|
||||||
UPROPERTY(EditAnywhere, BlueprintReadWrite)
|
UPROPERTY(EditAnywhere, BlueprintReadWrite)
|
||||||
float SettleSeconds = 0.35f;
|
float SettleSeconds = 0.35f;
|
||||||
|
|||||||
@@ -239,271 +239,287 @@ Scope: facts and traps that cost time to establish. Planned work lives in [TODO.
|
|||||||
|
|
||||||
## 2026-11-08 — reference photography gaps closed (local-only)
|
## 2026-11-08 — reference photography gaps closed (local-only)
|
||||||
|
|
||||||
- Closed the two missing reference angles for the hero façade, both from Wikimedia
|
- Closed the two missing reference angles for the hero façade, both from Wikimedia Commons (gitignored deliverables under `Art/reference/`, provenance in sidecars):
|
||||||
Commons (gitignored deliverables under `Art/reference/`, provenance in sidecars):
|
- `el-capitan-entrance/` — `File:Hollywood El Capitan entrance.jpg` (+ detail), Andreas Praefcke, CC BY 3.0. This is the isolated hero (6840) entrance/door recess that the notes doc had claimed Commons didn't hold. Query that found it: `--query "El Capitan entrance"`.
|
||||||
- `el-capitan-entrance/` — `File:Hollywood El Capitan entrance.jpg` (+ detail),
|
- `el-capitan-night/` — `File:El Capitan theater with Tron marquee.jpg` (CC BY-SA 3.0) + West Side Story premiere (Adam Jones, CC BY 2.0). The theatre is the building NEXT to 6840; treat as proximity evidence only, not the hero.
|
||||||
Andreas Praefcke, CC BY 3.0. This is the isolated hero (6840) entrance/door
|
- Commons throttles hard: HTTP 429 after a batch. Fetch fewer files with `--delay`/separate calls and wait ~15-20s between runs (`queue.py` tool default 1 req/s is insufficient once rate limited).
|
||||||
recess that the notes doc had claimed Commons didn't hold. Query that found it:
|
- Task was claimed on branch `ref-photo-0.1.2`; delivers only the TODO closure (all actual reference files are gitignored, so no code branch content). Reported to coordinator for merge/cleanup.
|
||||||
`--query "El Capitan entrance"`.
|
|
||||||
- `el-capitan-night/` — `File:El Capitan theater with Tron marquee.jpg`
|
|
||||||
(CC BY-SA 3.0) + West Side Story premiere (Adam Jones, CC BY 2.0). The theatre is
|
|
||||||
the building NEXT to 6840; treat as proximity evidence only, not the hero.
|
|
||||||
- Commons throttles hard: HTTP 429 after a batch. Fetch fewer files with
|
|
||||||
`--delay`/separate calls and wait ~15-20s between runs (`queue.py` tool default
|
|
||||||
1 req/s is insufficient once rate limited).
|
|
||||||
- Task was claimed on branch `ref-photo-0.1.2`; delivers only the TODO closure (all
|
|
||||||
actual reference files are gitignored, so no code branch content). Reported to
|
|
||||||
coordinator for merge/cleanup.
|
|
||||||
|
|
||||||
## 2026-11-08 — security-game scan: concepts copied, licensing ignored (owner)
|
## 2026-11-08 — security-game scan: concepts copied, licensing ignored (owner)
|
||||||
|
|
||||||
- Scanned `/run/media/system/DATA/Documents/06-games/security-game` for reference
|
- Scanned `/run/media/system/DATA/Documents/06-games/security-game` for reference material vs our `Art/reference/`. Result: its **photo set** (`docs/screens/refs/ hollywood/`, 9 files) is already fully duplicated in our `Art/reference/hollywood/` (with SIDECAR/BOARD copies) — nothing new there.
|
||||||
material vs our `Art/reference/`. Result: its **photo set** (`docs/screens/refs/
|
- What we lacked: its **AI concept renders** (`docs/screens/concepts/ hollywood_boulevard/`, 12 PNGs + `.refgen.json`). Copied into `Art/reference/el-capitan-concepts/` with a folder README. **Owner explicitly said ignore licensing completely** — they are gitignored local development reference, never committed or shipped; `.refgen.json` still records the model and source photo per file for traceability.
|
||||||
hollywood/`, 9 files) is already fully duplicated in our `Art/reference/hollywood/`
|
- Use concepts for mood/lighting intent only; geometry truth stays with the photo refs (facade, entrance, night marquee, TCL/Tussauds, Masonic 1922, etc.).
|
||||||
(with SIDECAR/BOARD copies) — nothing new there.
|
- The security-game `realism/` A/B screenshots and `concepts/joke_shop/` are its own engine/venue renders — not relevant to our 6840 modeling.
|
||||||
- What we lacked: its **AI concept renders** (`docs/screens/concepts/
|
|
||||||
hollywood_boulevard/`, 12 PNGs + `.refgen.json`). Copied into
|
|
||||||
`Art/reference/el-capitan-concepts/` with a folder README. **Owner explicitly
|
|
||||||
said ignore licensing completely** — they are gitignored local development
|
|
||||||
reference, never committed or shipped; `.refgen.json` still records the model
|
|
||||||
and source photo per file for traceability.
|
|
||||||
- Use concepts for mood/lighting intent only; geometry truth stays with the photo
|
|
||||||
refs (facade, entrance, night marquee, TCL/Tussauds, Masonic 1922, etc.).
|
|
||||||
- The security-game `realism/` A/B screenshots and `concepts/joke_shop/` are its
|
|
||||||
own engine/venue renders — not relevant to our 6840 modeling.
|
|
||||||
|
|
||||||
## 2026-11-08 — ODbL review closed; remaining active TODO items finished
|
## 2026-11-08 — ODbL review closed; remaining active TODO items finished
|
||||||
|
|
||||||
- Wrote `Docs/odbl-review.md` (decision 2026-11-08): the shipped block geometry
|
- Wrote `Docs/odbl-review.md` (decision 2026-11-08): the shipped block geometry (streets/footprints/sidewalks) is a Produced Work of the OpenStreetMap database (ODbL 1.0). Obligations held: on-screen attribution via AGigLineHUD ("© OpenStreetMap contributors" + "Data © OSM, ODbL 1.0 — openstreetmap.org/copyright"), ODbL link, and the raw extract committed in Geo/ with a manifest (date, bbox, SHA-256). Share-alike is limited to the data layer; project code/art remain CC0 (Docs/licensing.md decision). 3DEP DEM is US Government public domain, unrelated to ODbL.
|
||||||
(streets/footprints/sidewalks) is a Produced Work of the OpenStreetMap database
|
- Closed both [A] items in TODO: (1) sidewalks/roads/collision — collision verified; remaining kerb radii/aprons/kerb-height explicitly tracked as 0.1.1 patch tasks (Sidewalk outer edge, Crossing ramp gradient, Kerb height); (2) ODbL review — recorded. The [D] Blosm-import (blockout cleaning) item stays deferred (tracked separately).
|
||||||
(ODbL 1.0). Obligations held: on-screen attribution via AGigLineHUD ("©
|
|
||||||
OpenStreetMap contributors" + "Data © OSM, ODbL 1.0 — openstreetmap.org/copyright"),
|
|
||||||
ODbL link, and the raw extract committed in Geo/ with a manifest (date, bbox,
|
|
||||||
SHA-256). Share-alike is limited to the data layer; project code/art remain CC0
|
|
||||||
(Docs/licensing.md decision). 3DEP DEM is US Government public domain, unrelated
|
|
||||||
to ODbL.
|
|
||||||
- Closed both [A] items in TODO: (1) sidewalks/roads/collision — collision
|
|
||||||
verified; remaining kerb radii/aprons/kerb-height explicitly tracked as 0.1.1
|
|
||||||
patch tasks (Sidewalk outer edge, Crossing ramp gradient, Kerb height);
|
|
||||||
(2) ODbL review — recorded. The [D] Blosm-import (blockout cleaning) item stays
|
|
||||||
deferred (tracked separately).
|
|
||||||
- Branch: odbl-review-0.1.0 (docs + TODO closure only).
|
- Branch: odbl-review-0.1.0 (docs + TODO closure only).
|
||||||
|
|
||||||
## 2026-11-08 — placement release check; north-buildings camera was 9.5 km off
|
## 2026-11-08 — placement release check; north-buildings camera was 9.5 km off
|
||||||
|
|
||||||
- Release check 2 (placement vs references): rendered demo, probed DemoMap world
|
- Release check 2 (placement vs references): rendered demo, probed DemoMap world positions, compared to Geo references. Matches: road 24.3 m (ref crossings 20.9-25.9), sidewalk centreline offsets 12.5/13.2 m (ref 12.6/13.6), hero frontage 26.0 m / setback ~13 m (ref 25.75/14.5), six south-frontage buildings
|
||||||
positions, compared to Geo references. Matches: road 24.3 m (ref crossings
|
- TCL/Tussaud's in verified OSM positions.
|
||||||
20.9-25.9), sidewalk centreline offsets 12.5/13.2 m (ref 12.6/13.6), hero
|
- **Bug found:** the `demo-north-buildings` camera sat at X -22800 (the buildings' own X), aiming at its own position -> 85%-sky screenshot. Probed real geometry: TCL bbox X -22889..-18442, Y -9431..-1802; Tussaud's X -27071..-22799, Y -7586..-1888; hero at X -13336, Y +1307 (frontage 26 m). Camera now at (X -22750, Y +1600) looking north; sky fraction 85% -> 41%.
|
||||||
frontage 26.0 m / setback ~13 m (ref 25.75/14.5), six south-frontage buildings
|
- Getting actor world bounds: spawn at loc(0,0,0) means static mesh bbox IS the world footprint (FBX carries the geo offset). Probe by loading the level explicitly (load_level) — a fresh commandlet starts on Untitled otherwise.
|
||||||
+ TCL/Tussaud's in verified OSM positions.
|
|
||||||
- **Bug found:** the `demo-north-buildings` camera sat at X -22800 (the buildings'
|
|
||||||
own X), aiming at its own position -> 85%-sky screenshot. Probed real geometry:
|
|
||||||
TCL bbox X -22889..-18442, Y -9431..-1802; Tussaud's X -27071..-22799,
|
|
||||||
Y -7586..-1888; hero at X -13336, Y +1307 (frontage 26 m). Camera now at
|
|
||||||
(X -22750, Y +1600) looking north; sky fraction 85% -> 41%.
|
|
||||||
- Getting actor world bounds: spawn at loc(0,0,0) means static mesh bbox IS the
|
|
||||||
world footprint (FBX carries the geo offset). Probe by loading the level
|
|
||||||
explicitly (load_level) — a fresh commandlet starts on Untitled otherwise.
|
|
||||||
|
|
||||||
## 2026-11-08 — collision/seam audit done; QA self-test extended
|
## 2026-11-08 — collision/seam audit done; QA self-test extended
|
||||||
|
|
||||||
- `-qaselftest` transect widened from 84 samples (3 lane centres) to **1456
|
- `-qaselftest` transect widened from 84 samples (3 lane centres) to **1456 samples / 16 rows** including kerb/road/sidewalk seam Ys (probed: N side -1912..-1053, kerb -1382..-1053, road -1382..+1047, kerb +718..+1047, S side +718..+1447). 0 holes.
|
||||||
samples / 16 rows** including kerb/road/sidewalk seam Ys (probed: N side
|
- **New wall probe:** ground-referenced horizontal 30 m trace at +160 cm. First attempt used a fixed Z 160 and started INSIDE the earth (street is at Z ~366) -> every row reported a dist-0 blocker. Always ground the probe first.
|
||||||
-1912..-1053, kerb -1382..-1053, road -1382..+1047, kerb +718..+1047, S side
|
- **Guard:** GigLineShotActor must skip rounds under `-qaselftest`, else bQuitWhenDone closes the game before the transect phase (observed: game exited at t=2s with "ShotActor: done (4 shots)" + RequestExit).
|
||||||
+718..+1447). 0 holes.
|
- Results on packaged build: walk 761cm/2s, reset 0cm, pause/unpause, transect 0 holes, wall 0/35 -> all PASS. Evidence recorded in Docs/release-checks.md.
|
||||||
- **New wall probe:** ground-referenced horizontal 30 m trace at +160 cm. First
|
|
||||||
attempt used a fixed Z 160 and started INSIDE the earth (street is at Z ~366)
|
|
||||||
-> every row reported a dist-0 blocker. Always ground the probe first.
|
|
||||||
- **Guard:** GigLineShotActor must skip rounds under `-qaselftest`, else
|
|
||||||
bQuitWhenDone closes the game before the transect phase (observed: game exited
|
|
||||||
at t=2s with "ShotActor: done (4 shots)" + RequestExit).
|
|
||||||
- Results on packaged build: walk 761cm/2s, reset 0cm, pause/unpause, transect
|
|
||||||
0 holes, wall 0/35 -> all PASS. Evidence recorded in Docs/release-checks.md.
|
|
||||||
|
|
||||||
## 2026-11-08 — setup logs gated behind a cvar (diagnostics item)
|
## 2026-11-08 — setup logs gated behind a cvar (diagnostics item)
|
||||||
|
|
||||||
- The remaining spawn/input setup logs (settle-timer "start captured", "GigLine
|
- The remaining spawn/input setup logs (settle-timer "start captured", "GigLine IN:" pushed/bound/errors) are now gated behind `gigline.SetupLogs` (TAutoConsoleVariable, default **0** via `DefaultEngine.ini [ConsoleVariables]`). Re-enable at runtime: `gigline.SetupLogs 1`.
|
||||||
IN:" pushed/bound/errors) are now gated behind `gigline.SetupLogs`
|
- The `-qaselftest` QA logs stay ungated (deliberate release-check feature per Docs/release-checks.md). Verified packaged build: 0 setup-log lines, QA suite still all PASS (walk/reset/pause/transect 0 holes/wall 0-35).
|
||||||
(TAutoConsoleVariable, default **0** via `DefaultEngine.ini [ConsoleVariables]`).
|
- Branch qa-diagnostics-0.1.0. SIM/GROUND probes were removed earlier in the DefaultKeyMappings fix (PR #8).
|
||||||
Re-enable at runtime: `gigline.SetupLogs 1`.
|
|
||||||
- The `-qaselftest` QA logs stay ungated (deliberate release-check feature per
|
|
||||||
Docs/release-checks.md). Verified packaged build: 0 setup-log lines, QA suite
|
|
||||||
still all PASS (walk/reset/pause/transect 0 holes/wall 0-35).
|
|
||||||
- Branch qa-diagnostics-0.1.0. SIM/GROUND probes were removed earlier in the
|
|
||||||
DefaultKeyMappings fix (PR #8).
|
|
||||||
|
|
||||||
## 2026-11-08 — v0.1.0 released
|
## 2026-11-08 — v0.1.0 released
|
||||||
|
|
||||||
- All six `0.1.0` release checks pass and are recorded in `Docs/release-checks.md`:
|
- All six `0.1.0` release checks pass and are recorded in `Docs/release-checks.md`: traversal (seam transect 0 holes), placement vs references, collision+packaging (wall probe 0/35), 60 fps at 1440p (2.42 ms avg; TSR on, VSync off, no FG), fresh-checkout build, packaged Linux build with controls shown in-game.
|
||||||
traversal (seam transect 0 holes), placement vs references, collision+packaging
|
- `CHANGELOG.md` gets the `0.1.0` entry; the completed `0.1.0` section was removed from `TODO.md` (goal stays in `ROADMAP.md`); tag `v0.1.0` cut on `main`.
|
||||||
(wall probe 0/35), 60 fps at 1440p (2.42 ms avg; TSR on, VSync off, no FG),
|
|
||||||
fresh-checkout build, packaged Linux build with controls shown in-game.
|
|
||||||
- `CHANGELOG.md` gets the `0.1.0` entry; the completed `0.1.0` section was removed
|
|
||||||
from `TODO.md` (goal stays in `ROADMAP.md`); tag `v0.1.0` cut on `main`.
|
|
||||||
- Owner approved the regenerated screenshot set (PR #15) on 2026-11-08.
|
- Owner approved the regenerated screenshot set (PR #15) on 2026-11-08.
|
||||||
- THIRD_PARTY_NOTICES switched to checkbox statuses (`[x]`/`[P]`/`[ ]`); all tools
|
- THIRD_PARTY_NOTICES switched to checkbox statuses (`[x]`/`[P]`/`[ ]`); all tools incl. Blosm, QGIS and Git LFS are now verified as tool-only.
|
||||||
incl. Blosm, QGIS and Git LFS are now verified as tool-only.
|
- Next work: the `0.1.1` collision-detail patch (sidewalk outer edge 15.8 cm step, crossing ramp gradient 10 %, kerb height assumption, traversal regression).
|
||||||
- Next work: the `0.1.1` collision-detail patch (sidewalk outer edge 15.8 cm step,
|
|
||||||
crossing ramp gradient 10 %, kerb height assumption, traversal regression).
|
|
||||||
|
|
||||||
## 2026-11-08 — v0.1.1 released (collision patch)
|
## 2026-11-08 — v0.1.1 released (collision patch)
|
||||||
|
|
||||||
- Sidewalk upstand now tapers full-at-kerb → zero at the building edge (removes the
|
- Sidewalk upstand now tapers full-at-kerb → zero at the building edge (removes the 15.8 cm outer ledge). Crossing ramp run derives from `--max-ramp-grade` (8 % ⇒ 1.63 m) instead of a fixed 1.5 m/8.7 %. Street + site rebuilt, site re-imported, DemoMap rebuilt, packaged build verified.
|
||||||
15.8 cm outer ledge). Crossing ramp run derives from `--max-ramp-grade` (8 % ⇒
|
- Pre-check tool fixes: coverage no longer shrinks with `--sample` (it used `range(int(block_len))`), and it reports grade + location of the worst step.
|
||||||
1.63 m) instead of a fixed 1.5 m/8.7 %. Street + site rebuilt, site re-imported,
|
- Residual, documented not hidden: 13.4 cm surface step at s≈198 (south) / 8.0 cm at s≈251 (north) — a strip discontinuity near a centreline vertex, below the 45 cm MaxStepHeight, so not blocking.
|
||||||
DemoMap rebuilt, packaged build verified.
|
- Kerb height finalised at 0.15 m modelled / 0.13 m built (6 in LA standard; DEM and photos can't resolve finer).
|
||||||
- Pre-check tool fixes: coverage no longer shrinks with `--sample` (it used
|
|
||||||
`range(int(block_len))`), and it reports grade + location of the worst step.
|
|
||||||
- Residual, documented not hidden: 13.4 cm surface step at s≈198 (south) / 8.0 cm
|
|
||||||
at s≈251 (north) — a strip discontinuity near a centreline vertex, below the
|
|
||||||
45 cm MaxStepHeight, so not blocking.
|
|
||||||
- Kerb height finalised at 0.15 m modelled / 0.13 m built (6 in LA standard; DEM
|
|
||||||
and photos can't resolve finer).
|
|
||||||
|
|
||||||
## 2026-11-09 — Mapbox building data as a second height source
|
## 2026-11-09 — Mapbox building data as a second height source
|
||||||
|
|
||||||
- `Geo/tools/mapbox_buildings.py` (new, stdlib `urllib`) Tilequeries
|
- `Geo/tools/mapbox_buildings.py` (new, stdlib `urllib`) Tilequeries `mapbox.mapbox-streets-v8` layer `building` at every building centroid in the extract and merges the answer with our OSM height + source. Token comes from `MAPBOX_ACCESS_TOKEN` in the ignored `.env`; responses cache to `Geo/raw/mapbox-cache/`.
|
||||||
`mapbox.mapbox-streets-v8` layer `building` at every building centroid in the
|
- **Mapbox terms restrict persisting/redistributing their data**, so `.gitignore` now covers `/Geo/raw/mapbox-cache/` and `/Geo/verification/*mapbox*`. The tool also refuses to cache a response that contains the token.
|
||||||
extract and merges the answer with our OSM height + source. Token comes from
|
|
||||||
`MAPBOX_ACCESS_TOKEN` in the ignored `.env`; responses cache to
|
|
||||||
`Geo/raw/mapbox-cache/`.
|
|
||||||
- **Mapbox terms restrict persisting/redistributing their data**, so `.gitignore`
|
|
||||||
now covers `/Geo/raw/mapbox-cache/` and `/Geo/verification/*mapbox*`. The tool
|
|
||||||
also refuses to cache a response that contains the token.
|
|
||||||
- **First run (72 buildings): 58 match, 5 close, 3 disagree, 6 non-building.**
|
- **First run (72 buildings): 58 match, 5 close, 3 disagree, 6 non-building.**
|
||||||
- It independently classifies our roof/canopy/shelter volumes as `roof`/
|
- It independently classifies our roof/canopy/shelter volumes as `roof`/ `shelter` — corroborating the `0.1.2` finding that they are not buildings.
|
||||||
`shelter` — corroborating the `0.1.2` finding that they are not buildings.
|
- Remaining disagreements are untyped `building=yes`/`apartments` rear structures where our fallback guess (2 × 3.2 m = 6.4 m) meets Mapbox's 3.0 m.
|
||||||
- Remaining disagreements are untyped `building=yes`/`apartments` rear
|
- Every "close" case is a `building:levels × 3.2 m` building where Mapbox is **0.6 m lower per 6 levels** → Mapbox implies **3.1 m per level**, not 3.2. Worth revisiting `LEVEL_HEIGHT` if a measured storey height ever lands.
|
||||||
structures where our fallback guess (2 × 3.2 m = 6.4 m) meets Mapbox's 3.0 m.
|
|
||||||
- Every "close" case is a `building:levels × 3.2 m` building where Mapbox is
|
|
||||||
**0.6 m lower per 6 levels** → Mapbox implies **3.1 m per level**, not 3.2.
|
|
||||||
Worth revisiting `LEVEL_HEIGHT` if a measured storey height ever lands.
|
|
||||||
- Its query API surface: `height`, `min_height`, `type`, `extrude` per polygon.
|
- Its query API surface: `height`, `min_height`, `type`, `extrude` per polygon.
|
||||||
|
|
||||||
## 2026-11-09 — Mapbox calibration adopted into the blockout
|
## 2026-11-09 — Mapbox calibration adopted into the blockout
|
||||||
|
|
||||||
- Owner decision: adopt Mapbox. Applied:
|
- Owner decision: adopt Mapbox. Applied:
|
||||||
- `LEVEL_HEIGHT` 3.2 → **3.1 m** in `Geo/tools/block_height_report.py`, and the
|
- `LEVEL_HEIGHT` 3.2 → **3.1 m** in `Geo/tools/block_height_report.py`, and the Blosm `--level-height` default in `Art/blender/blockout/blosm_blockout.py`.
|
||||||
Blosm `--level-height` default in `Art/blender/blockout/blosm_blockout.py`.
|
- `HEIGHT_OVERRIDES` in block_height_report: ways 1490910971, 1492058875, 1492747608 → 3.0 m (Mapbox single-storey; the 2-level guess gave 6.4 m).
|
||||||
- `HEIGHT_OVERRIDES` in block_height_report: ways 1490910971, 1492058875,
|
- `classify_height(tags, level_height, way_id)` now takes the way id so overrides apply; `mapbox_buildings.py` passes it, so the cross-check agrees with the height table.
|
||||||
1492747608 → 3.0 m (Mapbox single-storey; the 2-level guess gave 6.4 m).
|
|
||||||
- `classify_height(tags, level_height, way_id)` now takes the way id so
|
|
||||||
overrides apply; `mapbox_buildings.py` passes it, so the cross-check agrees
|
|
||||||
with the height table.
|
|
||||||
- Re-running the cross-check: **66 match, 6 non-building, 0 close/disagree**.
|
- Re-running the cross-check: **66 match, 6 non-building, 0 close/disagree**.
|
||||||
- Geometry: blockout regenerated at 3.1 (object names preserved — the frontage
|
- Geometry: blockout regenerated at 3.1 (object names preserved — the frontage exporter maps `element.020`/`element`/`element.019`/named buildings, all still present, 6/6 exported), frontage re-imported, DemoMap rebuilt, packaged build `-qaselftest` all PASS.
|
||||||
exporter maps `element.020`/`element`/`element.019`/named buildings, all still
|
- **Blosm adds roof mass** on top of `levels × levelHeight`: 6922 39.1 m vs table 37.2, El Capitan Theatre 20.5 m vs 18.6 — both ~+1.9 m. That is a roof, not a height error; the table is the tagged height, the mesh is storey + roof.
|
||||||
present, 6/6 exported), frontage re-imported, DemoMap rebuilt, packaged build
|
- Gotcha: regenerating the blockout is safe for names, but any future change to the import must re-verify the 6/6 frontage export and the `element.*` mapping.
|
||||||
`-qaselftest` all PASS.
|
|
||||||
- **Blosm adds roof mass** on top of `levels × levelHeight`: 6922 39.1 m vs table
|
|
||||||
37.2, El Capitan Theatre 20.5 m vs 18.6 — both ~+1.9 m. That is a roof, not a
|
|
||||||
height error; the table is the tagged height, the mesh is storey + roof.
|
|
||||||
- Gotcha: regenerating the blockout is safe for names, but any future change to
|
|
||||||
the import must re-verify the 6/6 frontage export and the `element.*` mapping.
|
|
||||||
|
|
||||||
## 2026-11-09 — hero colonnade proportion fixed (47.6 % → 53.1 %)
|
## 2026-11-09 — hero colonnade proportion fixed (47.6 % → 53.1 %)
|
||||||
|
|
||||||
- The hero's colonnade (columns 2.00–7.95 + capitals 7.95–8.90) was 6.90 m of the
|
- The hero's colonnade (columns 2.00–7.95 + capitals 7.95–8.90) was 6.90 m of the 14.50 m elevation = **47.6 %**, below the photographed 50–60 %, because the entablature (3.40 m) and parapet (2.20 m) used 5.60 m of the height.
|
||||||
14.50 m elevation = **47.6 %**, below the photographed 50–60 %, because the
|
- Thinned ENTABLATURE_H 3.40 → 3.00 and PARAPET_H 2.20 → 1.80 in `Tools/hero_facade_build.py`; the colonnade grows to **7.70 m = 53.1 %** while the envelope, motto band and medallions keep their relationships (all positioned relative to those two constants).
|
||||||
entablature (3.40 m) and parapet (2.20 m) used 5.60 m of the height.
|
- **Gotcha (cost a bad build):** `hero_facade_build.py` takes `--out` for the **FBX** and `--save-blend` for the `.blend`. Passing `--out <file>.blend` writes FBX bytes over the blend and Blender then reports "File format is not supported". Correct invocation: `--out /tmp/gigline/hero-facade.fbx --save-blend Art/...blend --report Art/...json`.
|
||||||
- Thinned ENTABLATURE_H 3.40 → 3.00 and PARAPET_H 2.20 → 1.80 in
|
- Rebuilt hero → re-imported (`SM_HeroFacade`, bounds 2604 × 1257 × 1450 cm) → DemoMap rebuilt → screenshots re-rendered. Note `ue_hero_facade_import.py` saves the map, so `chmod u+w` the `.umap` first or the import silently does not persist.
|
||||||
`Tools/hero_facade_build.py`; the colonnade grows to **7.70 m = 53.1 %** while the
|
|
||||||
envelope, motto band and medallions keep their relationships (all positioned
|
|
||||||
relative to those two constants).
|
|
||||||
- **Gotcha (cost a bad build):** `hero_facade_build.py` takes `--out` for the
|
|
||||||
**FBX** and `--save-blend` for the `.blend`. Passing `--out <file>.blend` writes
|
|
||||||
FBX bytes over the blend and Blender then reports "File format is not supported".
|
|
||||||
Correct invocation: `--out /tmp/gigline/hero-facade.fbx --save-blend Art/...blend
|
|
||||||
--report Art/...json`.
|
|
||||||
- Rebuilt hero → re-imported (`SM_HeroFacade`, bounds 2604 × 1257 × 1450 cm) →
|
|
||||||
DemoMap rebuilt → screenshots re-rendered. Note `ue_hero_facade_import.py` saves
|
|
||||||
the map, so `chmod u+w` the `.umap` first or the import silently does not persist.
|
|
||||||
|
|
||||||
## 2026-11-09 — AI detail plates from reference photos; floating windows fixed
|
## 2026-11-09 — AI detail plates from reference photos; floating windows fixed
|
||||||
|
|
||||||
- `Tools/facade_detail_concepts.py` generates architectural detail plates from one
|
- `Tools/facade_detail_concepts.py` generates architectural detail plates from one local reference photograph via OpenRouter image models (default `google/gemini-2.5-flash-image`; the sibling project uses the same route in `tools/generate_textures.py` — chat/completions with a text + image_url content part, image returned as `message.images[].image_url.url`). Eight studies, `--list` free, `--all` generates, `-vN` never overwrites, `.refgen.json` sidecar per image, all output under gitignored `Art/reference/facade-details/`.
|
||||||
local reference photograph via OpenRouter image models (default
|
- **Prompt gotcha:** the model refuses when the prompt describes features the reference photograph does not show ("the image you sent appears to be a decorative architectural detail, not an entrance with steps"). Prompts must describe the _reference subject_ and only restage light/people/surface.
|
||||||
`google/gemini-2.5-flash-image`; the sibling project uses the same route in
|
|
||||||
`tools/generate_textures.py` — chat/completions with a text + image_url content
|
|
||||||
part, image returned as `message.images[].image_url.url`). Eight studies,
|
|
||||||
`--list` free, `--all` generates, `-vN` never overwrites, `.refgen.json` sidecar
|
|
||||||
per image, all output under gitignored `Art/reference/facade-details/`.
|
|
||||||
- **Prompt gotcha:** the model refuses when the prompt describes features the
|
|
||||||
reference photograph does not show ("the image you sent appears to be a
|
|
||||||
decorative architectural detail, not an entrance with steps"). Prompts must
|
|
||||||
describe the *reference subject* and only restage light/people/surface.
|
|
||||||
- **Output is real:** 1184×864 to 1344×768 PNGs, ~2 MB each.
|
- **Output is real:** 1184×864 to 1344×768 PNGs, ~2 MB each.
|
||||||
- **Measured defect found and fixed:** hero window panes sat at Y −1.98 while the
|
- **Measured defect found and fixed:** hero window panes sat at Y −1.98 while the mass front is Y −0.50 and the columns reach −1.02 — i.e. glazing floating 1.44 m in front of the wall and ahead of the colonnade. Now `WINDOW_Y = −0.56` (6 cm proud of the wall, behind the columns). Visible as dark openings in `review-hero-facade-front`.
|
||||||
mass front is Y −0.50 and the columns reach −1.02 — i.e. glazing floating 1.44 m
|
- Reminder: `.blend` is LFS read-only on checkout — `chmod u+w` or Blender fails with "Cannot save blend file … not writable" (this bit again here).
|
||||||
in front of the wall and ahead of the colonnade. Now `WINDOW_Y = −0.56`
|
|
||||||
(6 cm proud of the wall, behind the columns). Visible as dark openings in
|
|
||||||
`review-hero-facade-front`.
|
|
||||||
- Reminder: `.blend` is LFS read-only on checkout — `chmod u+w` or Blender fails
|
|
||||||
with "Cannot save blend file … not writable" (this bit again here).
|
|
||||||
|
|
||||||
## 2026-11-09 — UE export hygiene: smoothing groups + facade collision
|
## 2026-11-09 — UE export hygiene: smoothing groups + facade collision
|
||||||
|
|
||||||
- **All our FBX exporters used `mesh_smooth_type="OFF"`**, which is why every Unreal
|
- **All our FBX exporters used `mesh_smooth_type="OFF"`**, which is why every Unreal import logged "No smoothing group information was found for this mesh". The hero now exports `"FACE"` and its round parts are shaded with `shade_smooth_by_angle(35°)` before export: cylinders read as round instead of faceted, and the import warning is gone (6 → 0).
|
||||||
import logged "No smoothing group information was found for this mesh". The hero
|
- **The hero façade had no simple collision** (complex-only), so line traces and capsule sweeps passed straight through it — the player could walk through the wall of the demo. Caught by a new `-qaselftest` **facade occlusion probe** (40 m sideways traces from the road centre at walking height).
|
||||||
now exports `"FACE"` and its round parts are shaded with
|
- **Auto-generated convex collision on a 95-part 26 m façade is unusable**: it produced coarse hulls that bulged into the street and the wall probe went 0/35 → 2/35. Rejected in favour of `UseComplexAsSimple` via `UGigLineAssetTools.set_complex_collision_as_simple` (same reason the terrain uses it), saved on the asset.
|
||||||
`shade_smooth_by_angle(35°)` before export: cylinders read as round instead of
|
- Probe expectations must mirror the staged buildings: the facade probe now carries explicit X spans per frontage (and the TCL/Tussaud's spans on the north side) so a gap between buildings is not reported as a defect.
|
||||||
faceted, and the import warning is gone (6 → 0).
|
|
||||||
- **The hero façade had no simple collision** (complex-only), so line traces and
|
|
||||||
capsule sweeps passed straight through it — the player could walk through the
|
|
||||||
wall of the demo. Caught by a new `-qaselftest` **facade occlusion probe** (40 m
|
|
||||||
sideways traces from the road centre at walking height).
|
|
||||||
- **Auto-generated convex collision on a 95-part 26 m façade is unusable**: it
|
|
||||||
produced coarse hulls that bulged into the street and the wall probe went
|
|
||||||
0/35 → 2/35. Rejected in favour of `UseComplexAsSimple` via
|
|
||||||
`UGigLineAssetTools.set_complex_collision_as_simple` (same reason the terrain
|
|
||||||
uses it), saved on the asset.
|
|
||||||
- Probe expectations must mirror the staged buildings: the facade probe now carries
|
|
||||||
explicit X spans per frontage (and the TCL/Tussaud's spans on the north side) so
|
|
||||||
a gap between buildings is not reported as a defect.
|
|
||||||
|
|
||||||
## 2026-11-09 — one FBX export preset instead of six copies
|
## 2026-11-09 — one FBX export preset instead of six copies
|
||||||
|
|
||||||
- The `bpy.ops.export_scene.fbx(...)` argument block was duplicated in six scripts
|
- The `bpy.ops.export_scene.fbx(...)` argument block was duplicated in six scripts (hero, block frontage, north buildings, site, scale test, MPFB character), and all six carried `mesh_smooth_type="OFF"` — hence the "No smoothing group information was found" warning on every import.
|
||||||
(hero, block frontage, north buildings, site, scale test, MPFB character), and
|
- Now `Tools/fbx_export_preset.py` holds `ue_metric_v1` and `ue_character_v1`: `preset_kwargs()` (bpy-free, unit-tested) plus `export_static()` / `export_character()` wrappers that import `bpy` lazily. Every caller imports it and passes `selection=True/False`; the seven duplicated blocks are gone.
|
||||||
all six carried `mesh_smooth_type="OFF"` — hence the "No smoothing group
|
- Callers in `Tools/` resolve the module with `sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))`; the Blender-side `Art/blender/site/export_site_fbx.py` computes the repo root (`..`, `..`, `..`) and joins `Tools`. Getting that wrong silently inserts `Tools/Tools` — check the import path when adding a caller.
|
||||||
information was found" warning on every import.
|
- Verified end to end: scale test, hero, frontage (6/6), site (7/7) and north buildings all re-exported through the preset, re-imported with **0 smoothing warnings**, DemoMap rebuilt and `-qaselftest` all PASS (walk, transect 0 holes, wall probe 0/35, facade probe 0/9).
|
||||||
- Now `Tools/fbx_export_preset.py` holds `ue_metric_v1` and `ue_character_v1`:
|
- `mpfb_character_export.py` was not re-run: it needs the MPFB add-on on `BLENDER_USER_SCRIPTS` ("Add-on not loaded: bl_ext.blender_org.mpfb"), which is pre-existing and unrelated; the character is not in `0.1.x` gameplay.
|
||||||
`preset_kwargs()` (bpy-free, unit-tested) plus `export_static()` /
|
|
||||||
`export_character()` wrappers that import `bpy` lazily. Every caller imports it
|
|
||||||
and passes `selection=True/False`; the seven duplicated blocks are gone.
|
|
||||||
- Callers in `Tools/` resolve the module with
|
|
||||||
`sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))`; the Blender-side
|
|
||||||
`Art/blender/site/export_site_fbx.py` computes the repo root
|
|
||||||
(`..`, `..`, `..`) and joins `Tools`. Getting that wrong silently inserts
|
|
||||||
`Tools/Tools` — check the import path when adding a caller.
|
|
||||||
- Verified end to end: scale test, hero, frontage (6/6), site (7/7) and north
|
|
||||||
buildings all re-exported through the preset, re-imported with **0 smoothing
|
|
||||||
warnings**, DemoMap rebuilt and `-qaselftest` all PASS (walk, transect 0 holes,
|
|
||||||
wall probe 0/35, facade probe 0/9).
|
|
||||||
- `mpfb_character_export.py` was not re-run: it needs the MPFB add-on on
|
|
||||||
`BLENDER_USER_SCRIPTS` ("Add-on not loaded: bl_ext.blender_org.mpfb"), which is
|
|
||||||
pre-existing and unrelated; the character is not in `0.1.x` gameplay.
|
|
||||||
|
|
||||||
## 2026-11-09 — v0.1.2 released
|
## 2026-11-09 — v0.1.2 released
|
||||||
|
|
||||||
- Cornice finished: projecting corona over a **52-dentil course** (0.18 × 0.30 ×
|
- Cornice finished: projecting corona over a **52-dentil course** (0.18 × 0.30 × 0.22 m at 0.45 m centres, z 12.12–12.34, 0.25 m clear of the corona). Hero is now 3 044 tris across 148 parts, still 14.50 m. Verified: packaged `-qaselftest` all PASS (transect 0 holes, wall probe 0/35, facade probe 0/9).
|
||||||
0.22 m at 0.45 m centres, z 12.12–12.34, 0.25 m clear of the corona). Hero is now
|
- `CHANGELOG.md` gets the `0.1.2` entry (added/changed/fixed/verified/limitations), the `0.1.2` section is removed from `TODO.md`, `ROADMAP.md` status updated, tag `v0.1.2` cut after the merge.
|
||||||
3 044 tris across 148 parts, still 14.50 m. Verified: packaged `-qaselftest` all
|
- Assumptions recorded rather than hidden: dentil spacing, base and abacus dimensions are not resolvable in our reference photography.
|
||||||
PASS (transect 0 holes, wall probe 0/35, facade probe 0/9).
|
|
||||||
- `CHANGELOG.md` gets the `0.1.2` entry (added/changed/fixed/verified/limitations),
|
|
||||||
the `0.1.2` section is removed from `TODO.md`, `ROADMAP.md` status updated, tag
|
|
||||||
`v0.1.2` cut after the merge.
|
|
||||||
- Assumptions recorded rather than hidden: dentil spacing, base and abacus
|
|
||||||
dimensions are not resolvable in our reference photography.
|
|
||||||
|
|
||||||
## 2026-11-09 — elevation cameras levelled (straight verticals)
|
## 2026-11-09 — elevation cameras levelled (straight verticals)
|
||||||
|
|
||||||
- archviz MUST: an elevation must not lean. `hero-facade-front` looked up ~4° from
|
- archviz MUST: an elevation must not lean. `hero-facade-front` looked up ~4° from 4.2 m at a 7.6 m target; `demo-spawn-view` and `demo-street-level` were pitched too. All three now put the camera at the target's height, so `FindLookAtRotation` yields pitch 0 and roll 0 and verticals are parallel by construction.
|
||||||
4.2 m at a 7.6 m target; `demo-spawn-view` and `demo-street-level` were pitched
|
- Do not try to _measure_ verticality from the rendered PNG with a naive edge scan: banners, windows and the parapet produce different edge sets at different heights and the pairwise match is meaningless (it reported a 418 px "drift"). Verify it geometrically in the camera list instead.
|
||||||
too. All three now put the camera at the target's height, so
|
|
||||||
`FindLookAtRotation` yields pitch 0 and roll 0 and verticals are parallel by
|
## 2026-11-09 — 0.1.3 packaging/setup fixes
|
||||||
construction.
|
|
||||||
- Do not try to *measure* verticality from the rendered PNG with a naive edge
|
- **`Tools/ue_common.py`**: `make_writable()` clears the read-only bit an LFS `.umap` checks out with (otherwise `save_map()` returns False with no error and the staged actors vanish), and `require_saved()` makes that failure visible. `ue_hero_facade_import.py` and `ue_import_north_buildings.py` never did this — which is why importing the hero appeared to succeed while saving nothing. Verified by forcing `chmod u-w` on DemoMap first: the run logged `map writable: chmod u+w … (was 0o444)`, then `save_map: True`, exit 0.
|
||||||
scan: banners, windows and the parapet produce different edge sets at different
|
- Gotcha while adding it: `ue_hero_facade_import.py` had **no** `import os`/`sys` (only `unreal`), so the sys.path line raised `NameError: name 'sys' is not defined` inside the commandlet — an exit code of 255 with no obvious cause.
|
||||||
heights and the pairwise match is meaningless (it reported a 418 px "drift").
|
- Docs: `build.md` gained the current pipeline table (7 site FBXs, frontage, north, hero, map), the `Builds/GigLineQA` archive, the packaged-log path, `-qaselftest` usage and a **traps** table; `setup.md` marks the fresh-checkout build verified and documents the two optional API keys; architecture 07 + combined doc corrected the build-output path.
|
||||||
Verify it geometrically in the camera list instead.
|
|
||||||
|
## 2026-11-09 — 0.1.4 materials + architecture-camera pass
|
||||||
|
|
||||||
|
- **Material graphs**: `ensure_material()` now supports masonry coursing (`courses_m`) and roughness variation. **UE's `WorldPosition` is in CENTIMETRES**, so a course height in metres must be ×100 — the first attempt divided by 0.55 and produced coursing 100× too fine to see.
|
||||||
|
- **`connect_material_expressions` returns False instead of raising**, so an unknown pin name leaves an input unconnected and the material silently compiles with a default (a mask of 0 is invisible). All links now go through a `link()` helper that logs failures. There is also **no `MaterialExpressionLess`**; use `Frac → Divide → OneMinus → Clamp` for a soft threshold band, or `MaterialExpressionStep`/`SmoothStep` if a hard one is wanted.
|
||||||
|
- **Material assignment was slot-0-only**, which on the hero is the _banner_ — the stone, shadowed stone and glazing kept their flat Blender materials. Assignment is now per slot, keyed on the imported slot material's name (`glass`/`banner`/`stoneshadow`/`stone`/`roof`/`wall`). Hero: 4/4 slots.
|
||||||
|
- New materials: `M_FacadeStoneShadow` (same coursing, a stop darker), `M_FacadeGlass` (roughness 0.14, specular 0.62 ≈ IOR 1.5 expressed without a refraction pass), `M_FacadeBanner` (the one saturated surface).
|
||||||
|
- **Camera FOV**: levelling the elevation cameras left the building small in frame (39 % sky). Added optional per-shot `ShotFieldOfView` to `AGigLineShotActor` and a 4th element on the shot tuples; a long lens (42–55°) fills the frame from across the street while the camera stays level, so verticals stay parallel. `review-hero-facade-front` sky 39 → 20 %, `demo-spawn-view` 23 → 16 %.
|
||||||
|
|
||||||
|
## 2026-11-09 — facade texture maps (0.1.4)
|
||||||
|
|
||||||
|
- `Tools/generate_facade_textures.py` writes deterministic 512² base-colour and roughness maps (ashlar stone, stucco, asphalt, concrete) with a pure-zlib PNG encoder and hash-based value noise — no numpy/Pillow, ~14 s for all eight, and byte-identical on a re-run (checked by md5sum).
|
||||||
|
- `ue_build_demo_map.py` imports them to `/Game/Hollywood/Textures` and samples them **world-aligned**: WorldPosition (cm) → ×(1/tile_cm) → ComponentMask(R,G) → TextureSample UVs. `tile_cm` per material: stone 440, stucco 260, asphalt 900, concrete 250.
|
||||||
|
- Gotchas: `MaterialExpressionTextureSample`'s UV input pin is **"UVs"**, and the output pin for colour is **"RGB"** / roughness **"R"**. `ComponentMask` pins are R/G/B/A booleans. A textured material skips the graph coursing on purpose — overlaying both would double the course pattern.
|
||||||
|
- The script needs `ROOT` in `ue_build_demo_map.py` and `sys` imported; forgetting either fails with `NameError` and an exit code of 0 from the commandlet (always read the result file under /tmp/gigline).
|
||||||
|
|
||||||
|
## 2026-11-09 — façade module kit (0.1.4)
|
||||||
|
|
||||||
|
- `Tools/facade_kit_build.py` builds the kit at the measured **4.96 m bay** and **3.10 m storey**: pilaster, sill, lintel, window bay, shopfront bay, door bay, cornice, parapet + coping = 16 meshes, exported per-object with `fbx_export_preset` plus a JSON part report (dims + evidence per piece).
|
||||||
|
- `Tools/ue_import_facade_kit.py` → `/Game/Hollywood/Kit/*`, complex-as-simple collision (a convex hull would fence off the shopfront glazing).
|
||||||
|
- `Tools/ue_apply_facade_kit.py` lays the kit out from each building's own bbox: `bays = round(width / 496 cm)`, storeys from height minus the 310 cm ground floor. **638 pieces** over six frontage buildings + both north buildings.
|
||||||
|
- **Bug worth remembering:** the first window piece was modelled entirely at +Y (into the wall), so every window sat _inside_ the solid blockout wall and was invisible in every render while the placement maths looked right. The kit now projects the surround 3 cm proud with the glazing 6 cm behind its face — a solid-wall blockout cannot carry a real recess, so the piece has to project.
|
||||||
|
- Blender gotcha: clearing the scene _including materials_ invalidates handles already taken ("StructRNA of type Material has been removed"); clear objects and orphan meshes only, and create the material after the clear.
|
||||||
|
- The kit meshes are spawned at the world origin with their position baked into the mesh (like the frontage exports), so placement uses the bbox directly and must not add the actor location.
|
||||||
|
|
||||||
|
## 2026-11-09 — CRITICAL: blockout must be regenerated with --origin
|
||||||
|
|
||||||
|
- Regenerating the blockout in 0.1.2 (for the 3.1 m storey height) was run **without `--origin`**, so Blosm centred on the extract instead of fitting the project frame. The whole frontage shifted about **119 m in X and 2.4 m in Y**, and the hero ended up outside the 6902/6838 gap — the "verified placement" was undone silently, and every render still looked like a plausible block.
|
||||||
|
- Correct invocation (README documents it, and the affine fit prints `frame correction residual: max 0.0 cm`): `blosm_blockout.py --osm … --bounds 34.1006,-118.3422,34.1020,-118.3370 --level-height 3.1 --origin 34.101545,-118.338718 --assets-dir … --out …`
|
||||||
|
- After re-export + re-import, the frontage spans are back to the verified values: 6922 −24276..−17700, 6902 −17749..−14591, **hero −14644..−12028 between them**, theatre −12082..−8210, and the north buildings at −22889..−18442 / −27071..−22799.
|
||||||
|
- **`ue_import_north_buildings.py` also lacked `import sys`**, so its sys.path line raised NameError and the import silently did nothing (exit 255). Same trap as the hero importer: these scripts imported only `unreal`.
|
||||||
|
- Lesson: after any blockout regeneration, probe the staged buildings' world X spans and compare them with `Geo/verification/2026-11-06-el-capitan-block.md` before trusting a render. A shift of this size is invisible in a screenshot.
|
||||||
|
|
||||||
|
## 2026-11-09 — kit naming + topology validation (blender-modeler checklist)
|
||||||
|
|
||||||
|
- Kit assets now follow the Unreal convention: the Blender object keeps its readable name (`Kit_Pilaster`), the exported asset and file are `SM_Kit_Pilaster`. The kit loader strips the `SM_` prefix so the layout code still addresses pieces by short name.
|
||||||
|
- `facade_kit_build.py::validate()` runs the blender-modeler cleanup checklist on every piece before handoff — non-manifold edges, loose vertices, triangle count — and records the result in the part report. Current kit: 16 pieces, **all manifold, no loose geometry, 12 tris each**.
|
||||||
|
- `ue_import_facade_kit.py` now deletes kit assets that the current build no longer produces, so a rename cannot leave two generations of the same piece in the folder (the rename from `Kit_*` to `SM_Kit_*` did exactly that until it did).
|
||||||
|
|
||||||
|
## 2026-11-09 — visual comparison loop; screenshots are demo-only
|
||||||
|
|
||||||
|
- `Tools/visual_loop.py` closes the reference → render → feedback loop. `--verify` gates captures against `Tools/visual_targets.json` (blank, tone-less, wrong-size fail); `--compare` measures luma, RMS contrast, edge density, palette distance and a 3x3 luma grid against the reference and prints a 0–1 score; `--iteration N` writes `Art/reference/iterations/N/{metrics.json,feedback.md}`. The feedback names the skill to consult, so the next modelling pass starts from a measurement rather than an opinion.
|
||||||
|
- **Iterations are gitignored** (`/Art/reference/iterations/`); only confirmed shots are promoted into `Screenshots/`. A test asserts the ignore rule and that every target is a demo-map shot.
|
||||||
|
- `Screenshots/` now holds **demo-map captures only** — the review staging map is no longer rendered for review. `render_screenshots.sh --verify` renders the demo map, collects the shots and runs the verification gate in one step.
|
||||||
|
- **PIL gotchas:** `FIND_EDGES` leaves its own border ring at 128 (236/3600 px on a 60x60 frame = 6.5% "detail" on a perfectly flat image), so crop **after** filtering — cropping before does not help, the new border gets the same treatment. Palette shares must divide by the full-image area, not the cropped one.
|
||||||
|
- First comparison (iteration 2): spawn view 0.70, street level 0.35, crossing 0.41, north buildings 0.34. Edge density is the dominant gap everywhere (0.04–0.11 against 0.20–0.32 in the references) — the render still lacks the relief the photographs carry, which is exactly what the `0.1.4` façade work is for.
|
||||||
|
|
||||||
|
## 2026-11-09 — theatre marquee + blade sign
|
||||||
|
|
||||||
|
- `Tools/theatre_trim_build.py` builds the El Capitan Theatre's signature features in **world coordinates**: marquee canopy 12.0 × 2.60 m at a 4.60 m soffit, illuminated fascia, recessed soffit, vertical blade sign 1.30 × 5.20 m with frame and bracket. `Tools/ue_stage_theatre_trim.py` imports to `/Game/Hollywood/Theatre` and stages one actor per piece (single spawn at the origin — no placement arithmetic).
|
||||||
|
- **Two placement bugs, both invisible in a render:**
|
||||||
|
1. **Axis sign.** Blender +Y is Unreal −Y, so a face at Unreal Y +1001 is Blender Y −10.01. Using +10.01 put the whole marquee on the far side of the boulevard.
|
||||||
|
2. **Absolute Z.** Trim heights were absolute while the pavement at the theatre is 4.56 m above the project datum, so the canopy hung 5 cm above the pavement — the QA wall probe caught it as a knee-height blocker. `GROUND_Z` now offsets every height.
|
||||||
|
- Wall probe: a scan row at Y 1000 sat ~1 cm from the frontage building line and grazed the theatre wall once the kit's proud shopfronts went in. Rows now sample the street envelope only (−1400, −167, 800, −1382, 718).
|
||||||
|
|
||||||
|
## 2026-11-09 — 0.1.4 signed off; 0.1.5 created for the carried geometry
|
||||||
|
|
||||||
|
- Owner reviewed the four demo captures locally and approved the set. Signoff recorded in `Docs/reviews/0.1.4-review.md` (7/7 items) and in the per-shot `Approved` column of `Screenshots/README.md`.
|
||||||
|
- The three geometry items that `0.1.4` did not finish — retail neighbours from their own references, hero fluting/motto band, blockout cleaning — move to a new **`0.1.5` Façade finishing** patch rather than growing this one, plus a much-needed **1440p performance re-measure** (625 kit actors have never been measured; `0.1.0` measured 414 fps).
|
||||||
|
- Decision recorded because it sets a precedent: finish and sign off a version when its _stated_ scope is done, and carry leftover scope into the next patch — do not let a patch accumulate unmeasured geometry.
|
||||||
|
|
||||||
|
## 2026-11-09 — quality priority flipped: realism first, performance last
|
||||||
|
|
||||||
|
- Owner decision: realism leads; performance closes the quality list. Applied in
|
||||||
|
`Docs/Architecture/01_introduction_and_goals.md`, `10_quality_requirements.md`,
|
||||||
|
`04_solution_strategy.md`, the combined `architecture.md`, `Docs/release-checks.md`
|
||||||
|
(performance section now last, with the reason), `ROADMAP.md` (a "Quality priority"
|
||||||
|
section) and `AGENTS.md` (the tie-break rule next to the scope-cutting rule).
|
||||||
|
- Ordering used: 1 visual realism, 2 geographic fidelity, 3 walkability/usability,
|
||||||
|
4 build reproducibility/portability, 5 provenance, 6 performance.
|
||||||
|
- **Both revisions carry the same guard so the order cannot be used as an excuse:**
|
||||||
|
performance is a *floor*, not a preference — 60 FPS at 1440p without frame
|
||||||
|
generation must be met at every release, and if realism cannot fit, the realism
|
||||||
|
scope is cut deliberately and recorded in `CHANGELOG.md`, never the frame rate.
|
||||||
|
Walkability outranks realism in one direction: a block the player falls through is
|
||||||
|
not a demo.
|
||||||
|
|
||||||
|
## 2026-11-09 — 0.1.5 start: retail reference + kit layout maths
|
||||||
|
|
||||||
|
- Commons holds exactly **one** usable reference for the retail run:
|
||||||
|
`Art/reference/retail-neighbours/01-lee-drug-and-neighbor.jpg` (Gb321, CC BY-SA
|
||||||
|
4.0, 1920×953) — 6800 Lee Drug **and** its neighbour at street level, shopfronts
|
||||||
|
and parapets visible (vision pre-check: yes). Nothing for 6822 or 6806–6820, so
|
||||||
|
those two façades still have no reference and stay blocked.
|
||||||
|
- **Kit layout was wrong for tall/wide buildings**, not just ugly: bays were
|
||||||
|
`round(width / 496)`, so 6922 (65.8 m wide, 37.1 m tall) got **13 bays** — a
|
||||||
|
window every 4.96 m across the whole frontage — and the storey count came out at
|
||||||
|
11 × 310 cm = 34.1 m against a 35.7 m window zone, leaving bare wall under the
|
||||||
|
cornice. Fixed: `MAX_BAYS = 9` (module widens instead of subdividing), and the
|
||||||
|
storey spread is derived from the available height (`usable / storeys`) so the top
|
||||||
|
band always lands inside the building. Result: 6922 9 × 11, theatre 8 × 6,
|
||||||
|
6806 7 × 3, 6800 4 × 3, TCL 9 × 5.
|
||||||
|
|
||||||
|
## 2026-11-09 — hero fluting + motto band; and a silent placement divergence
|
||||||
|
|
||||||
|
- Fluting: 20 grooves per column as thin shadow-tone cylinders set into the shaft
|
||||||
|
(120 pieces) rather than boolean cuts — the eye reads them in silhouette and
|
||||||
|
grazing light, booleans would cost far more for that. Motto band is now a real
|
||||||
|
recess: surround 6 cm proud, panel 12 cm behind (was an 8 cm dark stripe).
|
||||||
|
Hero 148 parts/3 044 tris -> 271/5 492, still 14.50 m.
|
||||||
|
- **The real find:** `ue_hero_facade_import.py` had its own hero position
|
||||||
|
`(-12024.5, -1450, 0)` — the pre-`0.1.0` value, before the frontage placement fix
|
||||||
|
moved the hero to `(-13336, +1307)`. Running the importer on its own therefore
|
||||||
|
teleported the hero **13 m east and onto the north side of the boulevard**, and
|
||||||
|
nothing complained: a displaced building still renders. The only symptom was the
|
||||||
|
reference score dropping 0.69 -> 0.56 with the 3x3 composition correlation
|
||||||
|
flipping +0.54 -> -0.08.
|
||||||
|
- Fix: `Tools/ue_placement.py` is now the single source of truth (hero position,
|
||||||
|
verified frontage spans, `span_error`/`describe_span_failure`). Both
|
||||||
|
`ue_hero_facade_import.py` and `ue_build_demo_map.py` import it, the import asserts
|
||||||
|
the staged span and logs `PLACEMENT ERROR` when it does not match, and tests assert
|
||||||
|
the stale literal cannot come back. Spawn-view score back to **0.72** (best yet).
|
||||||
|
- Lesson to keep: **duplicated placement constants drift silently.** The metric that
|
||||||
|
caught it was the composition correlation, not a visual review.
|
||||||
|
|
||||||
|
## 2026-11-09 — 1440p re-measured with the kit: 363 fps, ~12% cost
|
||||||
|
|
||||||
|
- 2560×1440, Epic preset, TSR, VSync off, no frame generation: **2.76 ms average
|
||||||
|
(363 fps)**, p50 2.73, p95 3.17, p99 3.64, max 4.62 — against `0.1.0`'s 2.42 ms
|
||||||
|
(414 fps). The 625 kit actors + 8 generated textures cost about **12 %** and leave
|
||||||
|
~6× the 60 fps floor.
|
||||||
|
- **Variance trap confirmed a third time:** runs reporting ~105 fps (9.5 ms) were
|
||||||
|
two overlapping game instances — an earlier launch had not exited. `pgrep -af
|
||||||
|
Binaries/Linux/GigLine` should show one wrapper + one binary; 363 fps reproduced
|
||||||
|
twice on a clean run. Always count instances before believing a perf number.
|
||||||
|
- Series committed at `Docs/perf/0.1.5-kit-walk-frametimes.csv`; comparison table in
|
||||||
|
`Docs/release-checks.md`.
|
||||||
|
|
||||||
|
## 2026-11-09 — 604 unmaterialed actors: material rebuild was delete+recreate
|
||||||
|
|
||||||
|
- The "blockout cleaning pass" turned out to need **material and placement hygiene,
|
||||||
|
not more geometry**. `Tools/ue_audit_scene.py` (new) found **604 of 620 mesh actors
|
||||||
|
with no material**: kit pieces and theatre trim all untextured.
|
||||||
|
- **Cause:** `ensure_material` in `ue_build_demo_map.py` deleted and recreated each
|
||||||
|
`/Game/Hollywood/Site/M_*` asset every run ("so a look change cannot be masked by a
|
||||||
|
stale asset"). That orphane every reference: actors staged by other tools pointed at
|
||||||
|
deleted assets and fell back to the default material. The map looked only *slightly*
|
||||||
|
flat — no error, no warning.
|
||||||
|
- **Fix:** update the material **in place** — load it, `MaterialEditingLibrary.
|
||||||
|
delete_all_material_expressions(mat)`, rebuild the graph, recompile, save. Note
|
||||||
|
`Material.expressions` is protected from Python; use the library's deleter.
|
||||||
|
- **Ordering rule now documented and enforced:** the demo builder rebuilds materials,
|
||||||
|
so kit and theatre staging must run *after* it. `render_screenshots.sh` now runs
|
||||||
|
demo map → kit assembly → theatre trim → scene audit → shot cameras → capture, and
|
||||||
|
**fails on a non-zero audit count**.
|
||||||
|
- Second find: 53 duplicated kit labels (both shop piers of a bay shared one name).
|
||||||
|
- Scene audit after the fixes: **0 failures** (all materialed, all spans verified, no
|
||||||
|
duplicate labels). Packaged QA still PASS; spawn-view reference score 0.72.
|
||||||
|
|
||||||
|
## 2026-11-09 — v0.1.5 released
|
||||||
|
|
||||||
|
- All actionable `0.1.5` work is done: hero fluting + recessed motto band, kit layout
|
||||||
|
for wide/tall buildings, placement single-source-of-truth, 1440p re-measure
|
||||||
|
(363 fps), and the material/placement hygiene that the "blockout cleaning" task
|
||||||
|
actually needed (604 unmaterialed actors, 53 duplicate labels) with a new scene
|
||||||
|
audit gate.
|
||||||
|
- Only item carried forward is the **retail frontage (6822 / 6806–6820 / 6800)**,
|
||||||
|
blocked because Commons holds no usable reference for two of the three buildings
|
||||||
|
and substituting the theatre's photographs would misattribute the façades. It now
|
||||||
|
lives in a `0.1.6` Retail frontage patch.
|
||||||
|
- Precedent applied again: tag when the stated scope is done, carry blocked scope
|
||||||
|
into the next patch rather than letting a release wait indefinitely on missing
|
||||||
|
reference photography.
|
||||||
+5
-1
@@ -2,9 +2,13 @@
|
|||||||
|
|
||||||
Development roadmap for this project: milestone goals, scope, decisions, and release gates. Actionable tasks, claims, and progress marks live in [TODO.md](TODO.md).
|
Development roadmap for this project: milestone goals, scope, decisions, and release gates. Actionable tasks, claims, and progress marks live in [TODO.md](TODO.md).
|
||||||
|
|
||||||
|
## Quality priority
|
||||||
|
|
||||||
|
**Realism leads; performance is a floor.** Façade identity, proportion, material and light are judged against the reference photographs and drive design decisions. Performance closes the quality list: 60 FPS at 1440p without frame generation is a budget every release must meet, and if realism cannot fit inside it the realism scope is cut deliberately and recorded — never the frame rate.
|
||||||
|
|
||||||
## Status at a glance — 2026-11-08 15:30
|
## Status at a glance — 2026-11-08 15:30
|
||||||
|
|
||||||
Workstation, geo pipeline, and engine validation are done. The art pipeline has a first blockout, terrain, and street in metres, in one shared coordinate frame. **Project bootstrap is complete**: `Game/GigLine` builds, packages, and both content-acceptance tests (static-mesh scale, MPFB skeletal character) pass. **`v0.1.0` released 2026-11-08**: the walkable demo, hero façade, verified block frontage, first-person controls, ODbL attribution, and every release check pass (traversal, placement, collision, 60 fps at 1440p, fresh-checkout build, packaged build with controls). Next: the `0.1.1`–`0.1.4` patch series (collision detail, scale/façade corrections, packaging fixes, real building façades), then `0.2.0` (player body, third-person camera, animation).
|
Workstation, geo pipeline, and engine validation are done. The art pipeline has a first blockout, terrain, and street in metres, in one shared coordinate frame. **Project bootstrap is complete**: `Game/GigLine` builds, packages, and both content-acceptance tests (static-mesh scale, MPFB skeletal character) pass. **`v0.1.0` released 2026-11-08**: the walkable demo, hero façade, verified block frontage, first-person controls, ODbL attribution, and every release check pass (traversal, placement, collision, 60 fps at 1440p, fresh-checkout build, packaged build with controls). Next: the `0.1.1`–`0.1.4` patch series (collision detail, scale/façade corrections, packaging fixes, real building façades), **Next:** `0.1.6` retail frontage (6822, 6806–6820, 6800 — blocked on reference photography for two of the three), then `0.2.0` (player body, third-person camera, animation).
|
||||||
|
|
||||||
| Area | State |
|
| Area | State |
|
||||||
| --- | --- |
|
| --- | --- |
|
||||||
|
|||||||
+33
-45
@@ -1,63 +1,51 @@
|
|||||||
# Screenshots — visual review
|
# Screenshots — demo-map visual review
|
||||||
|
|
||||||
Rendered in-game captures of the current project content, generated by [`Tools/render_screenshots.sh`](../Tools/render_screenshots.sh). The PNGs are committed alongside this README so a reviewer sees exactly what the build rendered.
|
Rendered captures of **the demo map only** — what a player sees when the demo runs. Generated by [`Tools/render_screenshots.sh`](../Tools/render_screenshots.sh); the PNGs are committed so a reviewer sees exactly what the build rendered.
|
||||||
|
|
||||||
|
The review staging map (`/Game/Test/ReviewMap`) is not part of this set: it exists to hold the camera list and the hero detail rig, and it is not what ships.
|
||||||
|
|
||||||
## How to regenerate
|
## How to regenerate
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
Tools/render_screenshots.sh
|
Tools/render_screenshots.sh --verify
|
||||||
```
|
```
|
||||||
|
|
||||||
Rebuilds `Game/GigLine/Content/Test/ReviewMap.umap` from current content, runs the game on it (and on the DemoMap for the walkable views), walks the fixed camera list via `AGigLineShotActor`, captures each shot, copies the PNGs here and exits — about a minute. The camera list lives in `Tools/ue_build_review_map.py`; add new shots there, remove retired ones here.
|
Stages the demo shot cameras, runs the game on `/Game/Hollywood/DemoMap`, copies the PNGs into `Screenshots/demo/` and then runs [`Tools/visual_loop.py`](../Tools/visual_loop.py) `--verify` to check that each shot still shows the content its expectation describes. About a minute.
|
||||||
|
|
||||||
> Naming: shots use their stable camera name (no index prefix), so adding or reordering cameras never renumbers files or orphans the old ones. Click an image for the full-size PNG.
|
## The shot list
|
||||||
|
|
||||||
## Shot list
|
Each shot has an **expectation** in [`Tools/visual_targets.json`](../Tools/visual_targets.json) — the content the capture must show — and a verification state produced by `--verify`: it fails if the frame is blank, the wrong size, or has no tonal range.
|
||||||
|
|
||||||
**Status: whole set ☑ approved 2026-11-09** (owner review of PR #27). Every shot was re-rendered from current `main` content after the building-height calibration was adopted (`v0.1.2`): Mapbox-derived **3.1 m per level** (`Geo/tools/block_height_report.py`), explicit overrides for three rear outbuildings, and the regenerated blockout and frontage that follow — 6922 is now 39.1 m (was 42.1 m) and the El Capitan Theatre 20.5 m (was 20.0 m).
|
| # | File | View | Expected content | Verified | Approved |
|
||||||
|
| --- | --- | --- | --- | --- | --- |
|
||||||
|
| 1 | `demo/review-demo-spawn-view.png` | north sidewalk | The whole south block: the neoclassical hero façade between its neighbours, both sidewalks and the road in the foreground | ☑ 2026-11-09 | ☑ 2026-11-09 |
|
||||||
|
| 2 | `demo/review-demo-street-level.png` | eye height, north side | Street level: the hero colonnade with the adjacent frontage buildings either side, shopfronts at pavement level | ☑ 2026-11-09 | ☑ 2026-11-09 |
|
||||||
|
| 3 | `demo/review-demo-crossing.png` | along the crosswalk | The designated mid-block crossing (s=130 m, UE X −14956): zebra bars on the asphalt, south frontage and colonnade behind | ☑ 2026-11-09 | ☑ 2026-11-09 |
|
||||||
|
| 4 | `demo/review-demo-north-buildings.png` | across the Boulevard | The two north-side buildings (TCL Chinese Theatre, Madame Tussaud's) from the south side, full height in frame | ☑ 2026-11-09 | ☑ 2026-11-09 |
|
||||||
|
|
||||||
| Round | What changed |
|
Approval is per shot, in the table above (owner review of the `0.1.4` set, recorded in [`Docs/reviews/0.1.4-review.md`](../Docs/reviews/0.1.4-review.md)). Add a row when a shot is added; remove the file in the same change when a shot is retired.
|
||||||
| --- | --- |
|
|
||||||
| 2026-11-07 | First approved set; block frontage pass corrected hero and north-building placement |
|
|
||||||
| 2026-11-08 | Re-rendered after `0.1.1` street geometry (sidewalk upstand 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 | Re-rendered after the `0.1.2` building-height calibration (3.1 m per level). 12 of 14 PNGs changed; `review-demo-spawn-view` and `review-demo-street-level` are byte-identical to the `0.1.1` round, so the height change reads mostly on the frontage silhouettes |
|
|
||||||
|
|
||||||
| # | File | Preview | Map | View | What it shows | ☑ |
|
## The iteration loop (uncommitted)
|
||||||
| --- | --- | --- | --- | --- | --- | --- |
|
|
||||||
| 1 | `review-overview.png` | [<img src="review-overview.png" alt="overview" width="180">](review-overview.png) | Review | 7 m up, NE | Whole review layout: character + both cubes on the floor | ☑ 2026-11-09 |
|
`Tools/visual_loop.py` measures each capture against its reference photograph and turns the deltas into modelling actions for the Blender skill packs:
|
||||||
| 2 | `review-character.png` | [<img src="review-character.png" alt="character" width="180">](review-character.png) | Review | eye height, south | MPFB mannequin, idle pose + warm skin; feet meet the plane | ☑ 2026-11-09 |
|
|
||||||
| 3 | `review-scale-cubes.png` | [<img src="review-scale-cubes.png" alt="scale cubes" width="180">](review-scale-cubes.png) | Review | south | Floating vs grounded cube: cast vs contact shadow | ☑ 2026-11-09 |
|
```bash
|
||||||
| 4 | `review-player-placement.png` | [<img src="review-player-placement.png" alt="player placement" width="180">](review-player-placement.png) | Review | spawn | The `AGigLineCharacter` pawn, capsule height and landing | ☑ 2026-11-09 |
|
python3 Tools/visual_loop.py --compare --iteration 3 # writes Art/reference/iterations/3/
|
||||||
| 5 | `review-npc-prefab.png` | [<img src="review-npc-prefab.png" alt="npc prefab" width="180">](review-npc-prefab.png) | Review | mid, west | The `AGigLineNPCPerson` prefab: collision capsule + visible MPFB body, feet on Z 0 | ☑ 2026-11-09 |
|
```
|
||||||
| 6 | `review-ground-contact.png` | [<img src="review-ground-contact.png" alt="ground contact" width="180">](review-ground-contact.png) | Review | knee height, feet | Floating/sinking character is obvious here | ☑ 2026-11-09 |
|
|
||||||
| 7 | `review-shadow-compare.png` | [<img src="review-shadow-compare.png" alt="shadow compare" width="180">](review-shadow-compare.png) | Review | low angle | Detached (floating) vs touching (grounded) shadows | ☑ 2026-11-09 |
|
That directory is **gitignored**: iterations stay uncommitted until a shot is confirmed, and only confirmed shots are promoted into `Screenshots/demo/`. The feedback file names the affected skill (`blender-modeler`, `archviz`, `materials`, `lookdev`, `camera-cinematography`) so the next modelling pass has a starting point rather than an opinion.
|
||||||
| 8 | `review-hero-portals.png` | [<img src="review-hero-portals.png" alt="hero portals" width="180">](review-hero-portals.png) | Review | 8 m, eye height | Three genuine 1.2 m portal recesses with flights and plinth | ☑ 2026-11-09 |
|
|
||||||
| 9 | `review-hero-facade-front.png` | [<img src="review-hero-facade-front.png" alt="hero facade front" width="180">](review-hero-facade-front.png) | Review | road side, straight-on | Hero façade elevation and column rhythm | ☑ 2026-11-09 |
|
Note the comparison is only meaningful when the shot and the reference frame the same subject — a wide block view measured against a tight façade crop will always report low edge similarity.
|
||||||
| 10 | `review-hero-facade-oblique.png` | [<img src="review-hero-facade-oblique.png" alt="hero facade oblique" width="180">](review-hero-facade-oblique.png) | Review | north-west | Body depth and column line standing proud of the mass | ☑ 2026-11-09 |
|
|
||||||
| 11 | `demo/review-demo-spawn-view.png` | [<img src="demo/review-demo-spawn-view.png" alt="spawn view" width="180">](demo/review-demo-spawn-view.png) | Demo | north sidewalk | The whole block: hero façade between its neighbours, both sidewalks and the road | ☑ 2026-11-09 |
|
|
||||||
| 12 | `demo/review-demo-street-level.png` | [<img src="demo/review-demo-street-level.png" alt="street level" width="180">](demo/review-demo-street-level.png) | Demo | eye height, north side | The hero colonnade with the adjacent frontage buildings either side | ☑ 2026-11-09 |
|
|
||||||
| 13 | `demo/review-demo-crossing.png` | [<img src="demo/review-demo-crossing.png" alt="crossing" width="180">](demo/review-demo-crossing.png) | Demo | along the crosswalk | The designated mid-block crossing at s=130 m (UE X -14956): three zebra bars on the asphalt, with the south frontage and the hero colonnade behind | ☑ 2026-11-09 |
|
|
||||||
| 14 | `demo/review-demo-north-buildings.png` | [<img src="demo/review-demo-north-buildings.png" alt="north buildings" width="180">](demo/review-demo-north-buildings.png) | Demo | across the Blvd | North-side buildings (TCL, Tussaud's) seen from the south side across the road. **2026-11-08:** camera re-pointed to the buildings' mass centre (was 9.5 km west → 85% sky); now frames both north buildings. Flat surfaces = `0.1.4` façade scope. | ☑ 2026-11-09 |
|
|
||||||
|
|
||||||
## Scene conventions
|
## Scene conventions
|
||||||
|
|
||||||
- **Review map** (`/Game/Test/ReviewMap`): 160 m floor, top face at Z 0; the character measures Z 0..83 cm. One warm key light plus sky fill.
|
- **Demo map** (`/Game/Hollywood/DemoMap`): fixed daytime — sun pitch −62 / yaw 120 (intensity 4, warm), sky light 2.6, sky atmosphere, height fog, fixed exposure.
|
||||||
- **Cubes**: `Cube_ScaleTest_A` rests on the floor, `Cube_ScaleTest_B` floats at Z 2 m; both are exactly 1 m — the scale reference.
|
- **Cameras**: elevation shots sit at their target height (pitch 0, so verticals stay parallel) and use a 42–55° lens to fill the frame from across the street.
|
||||||
- **Demo map** (`/Game/Hollywood/DemoMap`): fixed daytime — sun at pitch −50 / yaw 120 (intensity 4, warm), sky light 2.6, sky atmosphere, height fog, fixed exposure.
|
- **Materials**: every surface carries a look-developed material from `Tools/ue_build_demo_map.py` — noise-blended base colour, masonry coursing, and a generated texture set (`Tools/generate_facade_textures.py`) sampled world-aligned. Explicit roughness and specular per surface type.
|
||||||
- **Materials**: every exported surface carries a look-developed material built in `Tools/ue_build_demo_map.py` (`ensure_material`) because Blender's flat diffuse colours do not survive the FBX → Unreal import. Each has an explicit roughness and a subtle world-space noise blend between two shades: `M_Asphalt` (0.92), `M_Concrete` (0.88), `M_KerbConcrete` (0.85), `M_Ground` (0.98), `M_RoadPaint` (0.62), `M_FacadeStone` (0.72) and `M_Stucco` (0.86) for the north-side buildings.
|
|
||||||
- **Lighting is dynamic-only** (`r.AllowStaticLighting=False`, all lights Movable), so no lighting build is ever required.
|
|
||||||
|
|
||||||
## What to look for
|
|
||||||
|
|
||||||
- **Ground contact**: feet meet the plane with no gap and no shadow seam.
|
|
||||||
- **Scale**: everything checks against the 1 m cubes.
|
|
||||||
- **Façade rhythm**: six columns at 4.96 m across 25.75 m should read as the photographs in silhouette.
|
|
||||||
- **Frontage heights (new in this round)**: 6922 and the El Capitan Theatre were re-cut to the 3.1 m storey height. Check they still read as the taller west end and the lower cinema respectively against `Art/reference/hollywood/el-capitan-theatre-blade-sign-day-wikimedia-2008-10.jpg`, and that the ordering of the six frontage buildings is unchanged.
|
|
||||||
|
|
||||||
## Known cosmetic issues
|
## Known cosmetic issues
|
||||||
|
|
||||||
- Floor speckle from the `Constant3Vector` wiring in `Tools/ue_build_review_map.py`.
|
- Hero façade renders pale: the stone material is a graph and a texture set, not a scanned surface (the `0.1.4` façade pass is in progress).
|
||||||
- Hero façade renders pale — Blender flat colours with no normal/roughness, so it reads as untextured stone (the `0.1.4` façade patch, not a lighting bug).
|
- Banners sit on the column line rather than flush to the wall (they hang on the columns in the photographs, so this is acceptable).
|
||||||
- Banners sit on the column line rather than the wall (acceptable: they hang on the columns in the photographs).
|
- Between kerb and building line the walkable surface is the terrain mesh, not a modelled pavement.
|
||||||
- ~~Banners and window panels float detached from the colonnade wall~~ — **window panes fixed 2026-11-09**: they sat at Y −1.98, 1.44 m in _front_ of the wall and ahead of the column line; they now sit on the wall plane at Y −0.56, behind the columns, and read as dark openings (see `review-hero-facade-front`).
|
- The El Capitan Theatre has no marquee or blade sign yet — the feature that makes it read as itself.
|
||||||
- Between kerb and building line, the walkable surface is the terrain mesh, not a modelled pavement.
|
|
||||||
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
@@ -9,30 +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.
|
- 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.
|
- 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.
|
||||||
|
|
||||||
## 0.1.3 — Packaging and setup fixes (patch)
|
## 0.1.6 — Retail frontage (patch)
|
||||||
|
|
||||||
Suggested blender skills: **export-pipeline** and **unreal-export** (scale/pivot/normals/materials/collision verification) if any packaging defect traces back to the Blender export presets.
|
Suggested blender skills: **references** (reference-image-match), **archviz**, **historical-worlds** for the 1920s–1990s retail mix on this run.
|
||||||
|
|
||||||
### Patch tasks
|
### Patch tasks
|
||||||
|
|
||||||
- [ ] Fix any packaging or build issues encountered in `0.1.x`.
|
- [ ] **Model 6822 Budget, 6806–6820 and 6800 Lee Drug from their own references.** `Art/reference/retail-neighbours/` holds 6800 and its neighbour (Gb321, CC BY-SA 4.0); Commons has nothing usable for 6822 or 6806–6820, so those two need photography before they can be modelled honestly. Do not substitute the El Capitan Theatre's 18-frame set — it would misattribute the façades.
|
||||||
- [ ] Update `Docs/` setup instructions for anything that changed.
|
- [ ] Review every façade change with `Tools/render_screenshots.sh --verify` before tagging.
|
||||||
- [ ] Tag `v0.1.3` and write `CHANGELOG.md` entry; then remove this section.
|
- [ ] Tag `v0.1.6` and write `CHANGELOG.md` entry; then remove this section.
|
||||||
|
|
||||||
## 0.1.4 — Proper building façades (patch)
|
|
||||||
|
|
||||||
Suggested blender skills: **procedural-modeling** + **geometry-nodes** (the façade module kit — bay, pilaster, cornice, parapet, window, shopfront, door — as reusable node groups), **archviz** (real-world scale and façade composition), **materials** (procedural brick, concrete, stone, render, glass — the ported generators), **texture-workflow** (AO/curvature/normal bakes and decals for the motto band and medallions), **uv-workflow** (texel density and modular UV layouts for the kit), **scene-assembly** (assembling kit pieces onto the fifteen block buildings), **lookdev** (screenshot-comparison loop until shading reads correctly), **historical-worlds** or **realistic-style** (1920s Hollywood Blvd neoclassical direction), **asset-optimization** + **unreal-export** (pre-export validation, `ue_metric_v1` preset), **qa-review** (final gate before the `Tools/render_screenshots.sh` review).
|
|
||||||
|
|
||||||
### Patch tasks
|
|
||||||
|
|
||||||
- [ ] **Finish the Blosm blockout cleaning/detailing (moved from the deferred `0.1.0` item).** Import state: `Art/blender/blockout/blockout-2026-11-06.blend` (86 meshes, EPSG:26911, 0.0 cm affine residual) with cleaning tooling in `Geo/tools/` and `Art/blender/blockout/blosm_blockout.py`. To do here: (a) untextured roofs/facades → **Replace placeholder materials** below; (b) Blosm `setupScript` façade styles → **Give OSM buildings a façade style** below; (c) the 3 of 15 buildings lacking height data are the `0.1.2` placeholder-heights item.
|
|
||||||
- [ ] **Define the façade module.** One set of reusable pieces (bay, pilaster, cornice, parapet, window, shopfront, door) at a fixed module, exported with the `ue_metric_v1` preset so every building assembles from the same kit.
|
|
||||||
- [ ] **Replace placeholder materials.** Every blockout building currently carries Blender flat colours. Port the procedural generators from `Tools/sibling_asset_audit.py`'s sibling project (`brick_facade`, `seamless_concrete`, `asphalt`) and add stone, render, and glass, then wire them through Unreal material instances.
|
|
||||||
- [ ] **Give OSM buildings a façade style.** Blosm exposes `scene.blosm.setupScript` (`setup/neoclassical.py` is the example) so a building can carry a real elevation instead of a plain extrusion. Build one style script for the south frontage and one for the north.
|
|
||||||
- [ ] **Hero façade detail pass.** Ionic capitals and fluting, the parapet motto band and medallions as geometry or decal, window rhythm behind the colonnade, and the three portal flights at their photographed proportions.
|
|
||||||
- [ ] **El Capitan Theatre (6838) and the retail neighbours.** Model from the 18-frame 5472×3648 reference set; the theatre shares the measured 4.96 m bay rhythm with the temple.
|
|
||||||
- [ ] Review every façade change with `Tools/render_screenshots.sh` before tagging.
|
|
||||||
- [ ] Tag `v0.1.4` and write `CHANGELOG.md` entry; then remove this section.
|
|
||||||
|
|
||||||
## 0.2.0 — MPFB player body, third-person camera, basic animation (weeks 7–9)
|
## 0.2.0 — MPFB player body, third-person camera, basic animation (weeks 7–9)
|
||||||
|
|
||||||
|
|||||||
+12
-1
@@ -16,6 +16,15 @@ Geographic tools live in [`Geo/tools/`](../Geo/README.md) and the Blender-side b
|
|||||||
| Tool | Purpose |
|
| Tool | Purpose |
|
||||||
| --- | --- |
|
| --- | --- |
|
||||||
| `check-lfs.sh` | Fails if the `lfs` filter is missing or an LFS-tracked file was committed as a plain blob |
|
| `check-lfs.sh` | Fails if the `lfs` filter is missing or an LFS-tracked file was committed as a plain blob |
|
||||||
|
| `facade_kit_build.py` | Builds the reusable façade **module kit** at the measured 4.96 m bay: pilaster, sill, lintel, window bay (surround + glazing), shopfront bay (piers, glass, fascia), door bay (jambs, door, threshold), cornice and parapet + coping. Exports each piece with the shared preset plus a JSON part report (dims + evidence per piece). |
|
||||||
|
| `ue_import_facade_kit.py` | Imports the kit into `/Game/Hollywood/Kit/*` as static meshes with complex-as-simple collision; writes `/tmp/gigline/kit-import.txt`. |
|
||||||
|
| `theatre_trim_build.py` | El Capitan Theatre trim: projecting marquee canopy, illuminated fascia, recessed soffit, vertical blade sign with frame and bracket. Built **in world coordinates** (span and pavement height baked in) with a JSON part report, so staging is a single spawn at the origin. |
|
||||||
|
| `ue_stage_theatre_trim.py` | Imports the trim to `/Game/Hollywood/Theatre` and stages one actor per piece in the DemoMap with site materials; clears a previous run first. |
|
||||||
|
| `ue_apply_facade_kit.py` | Lays the kit over a building's street face from its own bounding box: bay count = round(width / 4.96), storeys from the height, shopfronts on the ground floor, windows above, pilasters on bay boundaries, cornice and parapet at the top. Clears a previous application first; unique labels per building. |
|
||||||
|
| `generate_facade_textures.py` | Deterministic stdlib PNG generator for the facade maps (`T_StoneAshlar`, `T_Stucco`, `T_Asphalt`, `T_Concrete`, each `_BC` base colour + `_R` roughness). Pure-zlib writer, hash-based value noise, identical bytes on every run; `--size` and `--out` (`/tmp/gigline/textures` by default). `ue_build_demo_map.py` imports them into `/Game/Hollywood/Textures` and samples them world-aligned. |
|
||||||
|
| `ue_audit_scene.py` | Gate on the DemoMap: no unmaterialed actor, every verified frontage span within tolerance (`ue_placement`), no duplicate actor labels. Writes `/tmp/gigline/scene-audit.txt`; `render_screenshots.sh --verify` fails on a non-zero failure count. Catches the class of defect that still renders plausibly. |
|
||||||
|
| `ue_placement.py` | Single source of truth for where staged actors belong in the DemoMap (hero position, verified frontage spans) plus `span_error`/`describe_span_failure` validators. Exists because the hero position was written twice and drifted — the importer kept the pre-`0.1.0` value and moved the hero 13 m east onto the wrong side of the boulevard. No `unreal` import, so it is unit-tested. |
|
||||||
|
| `ue_common.py` | Shared helpers for the Unreal commandlet scripts: `make_writable()` clears the read-only bit an LFS `.umap` checks out with (otherwise `save_map()` returns False silently), and `require_saved()` makes that failure explicit. |
|
||||||
| `fbx_export_preset.py` | The **single** FBX export preset (`ue_metric_v1` / `ue_character_v1`): metres→centimetres via `FBX_SCALE_NONE`, axis mapping east +X / north −Y / up +Z, `mesh_smooth_type="FACE"` (smoothing groups survive the import). Every Blender exporter imports it instead of copying the argument block; `preset_kwargs()` is bpy-free and unit-tested. |
|
| `fbx_export_preset.py` | The **single** FBX export preset (`ue_metric_v1` / `ue_character_v1`): metres→centimetres via `FBX_SCALE_NONE`, axis mapping east +X / north −Y / up +Z, `mesh_smooth_type="FACE"` (smoothing groups survive the import). Every Blender exporter imports it instead of copying the argument block; `preset_kwargs()` is bpy-free and unit-tested. |
|
||||||
| `sibling_asset_audit.py` | Reads the sibling project's prop library from GLB headers into `Art/props/reuse-manifest.json`; copies nothing |
|
| `sibling_asset_audit.py` | Reads the sibling project's prop library from GLB headers into `Art/props/reuse-manifest.json`; copies nothing |
|
||||||
| `scale_test_export.py` / `ue_scale_test_import.py` | The 1 m = 100 uu scale test; the named presets `ue_metric_v1` live in these two files |
|
| `scale_test_export.py` / `ue_scale_test_import.py` | The 1 m = 100 uu scale test; the named presets `ue_metric_v1` live in these two files |
|
||||||
@@ -51,7 +60,9 @@ Geographic tools live in [`Geo/tools/`](../Geo/README.md) and the Blender-side b
|
|||||||
| Tool | Purpose |
|
| Tool | Purpose |
|
||||||
| --- | --- |
|
| --- | --- |
|
||||||
| `render_screenshots.py` / `render_screenshots.sh` | Rebuilds the review map, runs the game, captures the shot list into `Screenshots/` |
|
| `render_screenshots.py` / `render_screenshots.sh` | Rebuilds the review map, runs the game, captures the shot list into `Screenshots/` |
|
||||||
| `judge_screenshot.py` | Vision-model pre-check: judges a screenshot against an expected-scene description (`--image` + `--expect`, or `--shots` for every shot in the README). Reads `OPENROUTER_API_KEY` from `.env` (ignored). Optional helper — a human must still approve. |
|
| `judge_screenshot.py` | Vision-model pre-check: judges a screenshot against an expected-scene description (`--image` + `--expect`, or `--shots` for every shot in the README). Reads `OPENROUTER_API_KEY` from `.env` (ignored). Optional, advisory only — the model is unreliable on this content and never replaces a human verdict. |
|
||||||
|
| `visual_loop.py` | The reference-vs-render loop. `--verify` gates each capture against its expectation in `visual_targets.json` (blank/short/wrong-size/tone-less shots fail); `--compare` measures luminance, contrast, edge density, palette and 3×3 composition against the reference photograph and turns the deltas into modelling actions naming the skill to consult (`blender-modeler`, `archviz`, `materials`, `lookdev`, `camera-cinematography`); `--iteration N` writes the artefacts under `Art/reference/iterations/N/`, which is **gitignored until the owner confirms a shot**. |
|
||||||
|
| `visual_targets.json` | Per-shot expectation and reference photograph for `visual_loop.py`. Every entry must be a demo-map shot. |
|
||||||
|
|
||||||
## Gameplay QA
|
## Gameplay QA
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,315 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""Build the reusable façade module kit (0.1.4).
|
||||||
|
|
||||||
|
One module, measured: the **4.96 m bay** (the hero's column rhythm, which the
|
||||||
|
El Capitan Theatre next door shares per the reference notes). Every block building
|
||||||
|
should assemble from the same pieces instead of being a bare extrusion, so the
|
||||||
|
street wall gets a consistent window rhythm, shopfront line, cornice and parapet.
|
||||||
|
|
||||||
|
Pieces (`ue_metric_v1`, metres, origin at the module's bottom-front centre so a
|
||||||
|
building can place them by bay index and storey index):
|
||||||
|
|
||||||
|
Kit_Pilaster vertical pier on a bay boundary
|
||||||
|
Kit_Sill window sill band
|
||||||
|
Kit_Lintel window head / lintel band
|
||||||
|
Kit_WindowBay recessed window opening in one bay (glass + reveal)
|
||||||
|
Kit_ShopfrontBay ground-floor shopfront: piers, glass, fascia
|
||||||
|
Kit_DoorBay entrance bay: recessed door, jambs, threshold
|
||||||
|
Kit_Cornice projecting cornice band, one module long
|
||||||
|
Kit_Parapet parapet wall + coping, one module long
|
||||||
|
|
||||||
|
Dimensions come from the measured storey height and the reference photography
|
||||||
|
where it resolves them (window proportions, shopfront line); everything the
|
||||||
|
photographs do not resolve is an assumption and is recorded as one in the part
|
||||||
|
report, so model and evidence stay separable.
|
||||||
|
|
||||||
|
python3 Tools/facade_kit_build.py --out /tmp/gigline/kit \
|
||||||
|
--report /tmp/gigline/kit-report.json
|
||||||
|
blender -b --factory-startup --python Tools/facade_kit_build.py -- --out /tmp/gigline/kit
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import argparse
|
||||||
|
import json
|
||||||
|
import math
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
|
||||||
|
import bpy
|
||||||
|
|
||||||
|
sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
|
||||||
|
import fbx_export_preset as fbx_preset # noqa: E402
|
||||||
|
|
||||||
|
# --- module ------------------------------------------------------------------
|
||||||
|
|
||||||
|
BAY = 4.96 # m, measured: hero column rhythm, shared by the theatre
|
||||||
|
STOREY = 3.10 # m, the calibrated level height (see block_height_report)
|
||||||
|
PARAPET_H = 0.90 # m, coping above the top storey
|
||||||
|
CORNICE_H = 0.55 # m, projecting band
|
||||||
|
CORNICE_OUT = 0.35 # m, how far the cornice projects
|
||||||
|
|
||||||
|
PILASTER_W = 0.60
|
||||||
|
PILASTER_D = 0.28
|
||||||
|
|
||||||
|
WINDOW_W = 1.60
|
||||||
|
WINDOW_H = 1.85
|
||||||
|
WINDOW_REVEAL = 0.18 # how far the glass sits behind the wall face
|
||||||
|
SILL_H = 0.14
|
||||||
|
LINTEL_H = 0.34
|
||||||
|
|
||||||
|
SHOP_GLASS_H = 2.30 # shopfront glazing height from the pavement
|
||||||
|
SHOP_FASCIA_H = 0.55 # sign band above the glazing
|
||||||
|
DOOR_W = 1.90
|
||||||
|
DOOR_H = 2.45
|
||||||
|
|
||||||
|
STONE = None # materials are assigned in Unreal; Blender carries one grey
|
||||||
|
PARTS: list[dict] = []
|
||||||
|
|
||||||
|
|
||||||
|
def make_material(name, value):
|
||||||
|
mat = bpy.data.materials.new(name)
|
||||||
|
mat.use_nodes = True
|
||||||
|
bsdf = mat.node_tree.nodes.get("Principled BSDF")
|
||||||
|
bsdf.inputs["Base Color"].default_value = (value, value * 0.98, value * 0.94, 1.0)
|
||||||
|
bsdf.inputs["Roughness"].default_value = 0.75
|
||||||
|
return mat
|
||||||
|
|
||||||
|
|
||||||
|
def validate(obj, tri_budget=4000):
|
||||||
|
"""Topology check before handoff (blender-modeler cleanup checklist).
|
||||||
|
|
||||||
|
Reports non-manifold edges, loose geometry and the triangle count. Kit pieces
|
||||||
|
are primitive solids, so any non-manifold edge means a modelling slip rather
|
||||||
|
than an intentional hole.
|
||||||
|
"""
|
||||||
|
import bmesh
|
||||||
|
|
||||||
|
me = obj.data
|
||||||
|
bm = bmesh.new()
|
||||||
|
bm.from_mesh(me)
|
||||||
|
bm.edges.ensure_lookup_table()
|
||||||
|
non_manifold = [e for e in bm.edges if len(e.link_faces) != 2]
|
||||||
|
loose_verts = [v for v in bm.verts if not v.link_edges]
|
||||||
|
tri_count = sum(len(f.verts) - 2 for f in bm.faces)
|
||||||
|
bm.free()
|
||||||
|
if non_manifold or loose_verts:
|
||||||
|
print(f"[kit] WARNING {obj.name}: {len(non_manifold)} non-manifold edge(s), "
|
||||||
|
f"{len(loose_verts)} loose vertex/vertices")
|
||||||
|
if tri_count > tri_budget:
|
||||||
|
print(f"[kit] WARNING {obj.name}: {tri_count} tris exceeds the {tri_budget} budget")
|
||||||
|
return {"manifold": not non_manifold, "loose_verts": len(loose_verts),
|
||||||
|
"tris": tri_count}
|
||||||
|
|
||||||
|
|
||||||
|
def note(obj, note_text):
|
||||||
|
PARTS.append({
|
||||||
|
"object": obj.name,
|
||||||
|
"asset": "SM_" + obj.name,
|
||||||
|
"dims_m": [round(d, 3) for d in obj.dimensions],
|
||||||
|
"location_m": [round(c, 3) for c in obj.location],
|
||||||
|
"topology": validate(obj),
|
||||||
|
"evidence": note_text,
|
||||||
|
})
|
||||||
|
print(f"[kit] {obj.name:22} {obj.dimensions.x:.2f} x {obj.dimensions.y:.2f} "
|
||||||
|
f"x {obj.dimensions.z:.2f} m ({note_text})")
|
||||||
|
|
||||||
|
|
||||||
|
def cube(name, x, y, z, dx, dy, dz, material):
|
||||||
|
bpy.ops.mesh.primitive_cube_add(size=1.0, location=(x, y, z))
|
||||||
|
obj = bpy.context.active_object
|
||||||
|
obj.name = name
|
||||||
|
obj.scale = (dx, dy, dz)
|
||||||
|
bpy.ops.object.transform_apply(location=False, rotation=False, scale=True)
|
||||||
|
obj.data.materials.append(material)
|
||||||
|
return obj
|
||||||
|
|
||||||
|
|
||||||
|
def cylinder(name, x, y, z0, z1, radius, material, vertices=16):
|
||||||
|
bpy.ops.mesh.primitive_cylinder_add(vertices=vertices, radius=radius,
|
||||||
|
depth=(z1 - z0), location=(x, y, (z0 + z1) / 2.0))
|
||||||
|
obj = bpy.context.active_object
|
||||||
|
obj.name = name
|
||||||
|
obj.data.materials.append(material)
|
||||||
|
bpy.ops.object.shade_smooth_by_angle(angle=math.radians(35.0))
|
||||||
|
return obj
|
||||||
|
|
||||||
|
|
||||||
|
def clear_scene():
|
||||||
|
"""Remove objects and orphan meshes. Materials are left alone: deleting them
|
||||||
|
invalidates handles the previous piece still holds ("StructRNA of type
|
||||||
|
Material has been removed")."""
|
||||||
|
bpy.ops.object.select_all(action="SELECT")
|
||||||
|
bpy.ops.object.delete(use_global=False)
|
||||||
|
for mesh in list(bpy.data.meshes):
|
||||||
|
if mesh.users == 0:
|
||||||
|
bpy.data.meshes.remove(mesh)
|
||||||
|
|
||||||
|
|
||||||
|
# --- pieces ------------------------------------------------------------------
|
||||||
|
# Each builder returns [(object, note)] for one module. Origin: bottom-front
|
||||||
|
# centre of the module, +Y into the wall depth, +Z up, X across the bay.
|
||||||
|
|
||||||
|
|
||||||
|
def build_pilaster(mat):
|
||||||
|
o = cube("Kit_Pilaster", 0.0, PILASTER_D / 2.0, STOREY / 2.0,
|
||||||
|
PILASTER_W, PILASTER_D, STOREY, mat)
|
||||||
|
return [(o, "assumption: 0.60 x 0.28 m pier on the bay boundary")]
|
||||||
|
|
||||||
|
|
||||||
|
def build_sill(mat):
|
||||||
|
o = cube("Kit_Sill", 0.0, 0.10, SILL_H / 2.0, WINDOW_W + 0.30, 0.24, SILL_H, mat)
|
||||||
|
return [(o, "assumption: projecting stone sill under each opening")]
|
||||||
|
|
||||||
|
|
||||||
|
def build_lintel(mat):
|
||||||
|
o = cube("Kit_Lintel", 0.0, 0.10, LINTEL_H / 2.0, WINDOW_W + 0.30, 0.24, LINTEL_H, mat)
|
||||||
|
return [(o, "assumption: lintel band over each opening")]
|
||||||
|
|
||||||
|
|
||||||
|
def build_window_bay(mat):
|
||||||
|
"""One storey of window: a surround standing proud of the wall, glazing inset.
|
||||||
|
|
||||||
|
The blockout wall is a solid box, so a real recess cannot be cut. The piece is
|
||||||
|
therefore a shallow surround that projects 3 cm and a glazing plane set 6 cm
|
||||||
|
behind its front, which reads as a window from any street angle. The first
|
||||||
|
version modelled everything at +Y (into the wall), so the whole bay sat inside
|
||||||
|
the solid wall and was invisible in every render.
|
||||||
|
"""
|
||||||
|
parts = []
|
||||||
|
surround_d = 0.22
|
||||||
|
surround = cube("Kit_WindowBay", 0.0, -0.03 + surround_d / 2.0, WINDOW_H / 2.0,
|
||||||
|
WINDOW_W, surround_d, WINDOW_H, mat)
|
||||||
|
parts.append((surround, "photographs: window proportions 1.60 x 1.85 m; surround "
|
||||||
|
"projects 3 cm so it reads against a solid blockout wall"))
|
||||||
|
glass = cube("Kit_WindowGlass", 0.0, 0.09, WINDOW_H / 2.0,
|
||||||
|
WINDOW_W - 0.16, 0.04, WINDOW_H - 0.16, mat)
|
||||||
|
parts.append((glass, "glazing plane set 6 cm behind the surround face; "
|
||||||
|
"material assigned in Unreal"))
|
||||||
|
return parts
|
||||||
|
|
||||||
|
|
||||||
|
def build_shopfront_bay(mat):
|
||||||
|
"""Ground floor: two piers, glazing between, fascia above."""
|
||||||
|
parts = []
|
||||||
|
for side in (-1, 1):
|
||||||
|
x = side * (BAY / 2.0 - PILASTER_W / 2.0)
|
||||||
|
o = cube(f"Kit_ShopPier_{'L' if side < 0 else 'R'}", x, PILASTER_D / 2.0,
|
||||||
|
SHOP_GLASS_H / 2.0, PILASTER_W, PILASTER_D, SHOP_GLASS_H, mat)
|
||||||
|
parts.append((o, "shopfront pier, same 0.60 m module as the pilaster"))
|
||||||
|
glass = cube("Kit_ShopGlass", 0.0, 0.06, SHOP_GLASS_H / 2.0,
|
||||||
|
BAY - 2 * PILASTER_W, 0.05, SHOP_GLASS_H - 0.25, mat)
|
||||||
|
parts.append((glass, "photographs: continuous shopfront glazing to a "
|
||||||
|
"2.30 m head"))
|
||||||
|
fascia = cube("Kit_ShopFascia", 0.0, 0.14, SHOP_GLASS_H + SHOP_FASCIA_H / 2.0,
|
||||||
|
BAY, 0.26, SHOP_FASCIA_H, mat)
|
||||||
|
parts.append((fascia, "sign band over the shopfront; 0.55 m assumption"))
|
||||||
|
return parts
|
||||||
|
|
||||||
|
|
||||||
|
def build_door_bay(mat):
|
||||||
|
parts = []
|
||||||
|
for side in (-1, 1):
|
||||||
|
x = side * (DOOR_W / 2.0 + 0.35)
|
||||||
|
o = cube(f"Kit_DoorJamb_{'L' if side < 0 else 'R'}", x, 0.14, STOREY / 2.0,
|
||||||
|
0.35, 0.30, STOREY, mat)
|
||||||
|
parts.append((o, "entrance jamb standing 0.30 m proud of the wall"))
|
||||||
|
door = cube("Kit_Door", 0.0, 0.06, DOOR_H / 2.0, DOOR_W, 0.09, DOOR_H, mat)
|
||||||
|
parts.append((door, "assumption: 1.90 x 2.45 m recessed door"))
|
||||||
|
threshold = cube("Kit_Threshold", 0.0, 0.22, 0.05, DOOR_W + 0.70, 0.44, 0.10, mat)
|
||||||
|
parts.append((threshold, "threshold slab, flush with the pavement"))
|
||||||
|
return parts
|
||||||
|
|
||||||
|
|
||||||
|
def build_cornice(mat):
|
||||||
|
o = cube("Kit_Cornice", 0.0, -CORNICE_OUT / 2.0, CORNICE_H / 2.0,
|
||||||
|
BAY, CORNICE_OUT * 2.0, CORNICE_H, mat)
|
||||||
|
return [(o, "projecting cornice band, one bay long; 0.35 m projection assumption")]
|
||||||
|
|
||||||
|
|
||||||
|
def build_parapet(mat):
|
||||||
|
parts = []
|
||||||
|
wall = cube("Kit_Parapet", 0.0, -0.15, PARAPET_H / 2.0, BAY, 0.55, PARAPET_H, mat)
|
||||||
|
parts.append((wall, "assumption: 0.90 m parapet above the top storey"))
|
||||||
|
coping = cube("Kit_Coping", 0.0, -0.15, PARAPET_H - 0.06, BAY, 0.65, 0.12, mat)
|
||||||
|
parts.append((coping, "coping slab over the parapet"))
|
||||||
|
return parts
|
||||||
|
|
||||||
|
|
||||||
|
BUILDERS = {
|
||||||
|
"pilaster": build_pilaster,
|
||||||
|
"sill": build_sill,
|
||||||
|
"lintel": build_lintel,
|
||||||
|
"window": build_window_bay,
|
||||||
|
"shopfront": build_shopfront_bay,
|
||||||
|
"door": build_door_bay,
|
||||||
|
"cornice": build_cornice,
|
||||||
|
"parapet": build_parapet,
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
def build_piece(name):
|
||||||
|
"""Build one piece group in a clean scene. The material is created *after*
|
||||||
|
the clear so its handle stays valid."""
|
||||||
|
clear_scene()
|
||||||
|
mat = make_material(f"M_Kit_{name}", 0.6)
|
||||||
|
return BUILDERS[name](mat)
|
||||||
|
|
||||||
|
|
||||||
|
def export_group(objects, out_dir):
|
||||||
|
"""Export each object to its own FBX with the shared preset."""
|
||||||
|
exported = []
|
||||||
|
for obj in objects:
|
||||||
|
bpy.ops.object.select_all(action="DESELECT")
|
||||||
|
obj.select_set(True)
|
||||||
|
bpy.context.view_layer.objects.active = obj
|
||||||
|
# Unreal naming convention for static meshes is SM_<Name>; the Blender
|
||||||
|
# object keeps its readable name, the asset the importable one.
|
||||||
|
path = os.path.join(out_dir, "SM_" + obj.name + ".fbx")
|
||||||
|
fbx_preset.export_static(path, selection=True)
|
||||||
|
exported.append(path)
|
||||||
|
return exported
|
||||||
|
|
||||||
|
|
||||||
|
def main(argv=None) -> int:
|
||||||
|
argv = sys.argv[sys.argv.index("--") + 1:] if "--" in sys.argv else []
|
||||||
|
ap = argparse.ArgumentParser(description=__doc__,
|
||||||
|
formatter_class=argparse.RawDescriptionHelpFormatter)
|
||||||
|
ap.add_argument("--out", default="/tmp/gigline/kit")
|
||||||
|
ap.add_argument("--report", default=None)
|
||||||
|
ap.add_argument("--only", default=None, help="comma-separated piece names")
|
||||||
|
args = ap.parse_args(argv)
|
||||||
|
os.makedirs(args.out, exist_ok=True)
|
||||||
|
|
||||||
|
names = [n.strip() for n in args.only.split(",")] if args.only else list(BUILDERS)
|
||||||
|
unknown = [n for n in names if n not in BUILDERS]
|
||||||
|
if unknown:
|
||||||
|
raise SystemExit(f"unknown piece(s): {unknown}; known: {sorted(BUILDERS)}")
|
||||||
|
|
||||||
|
total = 0
|
||||||
|
for name in names:
|
||||||
|
made = build_piece(name)
|
||||||
|
if name == "window":
|
||||||
|
made = made # window bay exports two objects (reveal + glass)
|
||||||
|
objects = [o for o, _ in made]
|
||||||
|
for obj, note_text in made:
|
||||||
|
note(obj, note_text)
|
||||||
|
exported = export_group(objects, args.out)
|
||||||
|
total += len(exported)
|
||||||
|
print(f"[kit] {name}: {len(exported)} object(s) exported")
|
||||||
|
|
||||||
|
if args.report:
|
||||||
|
with open(args.report, "w", encoding="utf-8") as handle:
|
||||||
|
json.dump({
|
||||||
|
"module_m": BAY,
|
||||||
|
"storey_m": STOREY,
|
||||||
|
"pieces": sorted(BUILDERS),
|
||||||
|
"parts": PARTS,
|
||||||
|
}, handle, indent=2)
|
||||||
|
print(f"[kit] report -> {args.report}")
|
||||||
|
print(f"[kit] exported {total} object(s) to {args.out}")
|
||||||
|
return 0
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
raise SystemExit(main())
|
||||||
@@ -0,0 +1,226 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""Generate the facade texture maps the demo materials sample.
|
||||||
|
|
||||||
|
The materials started as procedural graphs only (noise + coursing), which reads
|
||||||
|
as tinted stone but has no grain. These maps add the grain: an ashlar stone
|
||||||
|
course pattern, a rendered stucco tooth, asphalt aggregate and concrete pores,
|
||||||
|
each as base colour plus a roughness map so the specular breaks up.
|
||||||
|
|
||||||
|
Standard library only — the PNG encoder is the same pure-zlib writer used for the
|
||||||
|
MPFB skin, so this runs on the host without numpy or Pillow.
|
||||||
|
|
||||||
|
python3 Tools/generate_facade_textures.py # write to /tmp/gigline/textures
|
||||||
|
python3 Tools/generate_facade_textures.py --size 1024
|
||||||
|
python3 Tools/generate_facade_textures.py --out Art/blender/textures
|
||||||
|
|
||||||
|
Output names match the Unreal import destination `/Game/Hollywood/Textures`:
|
||||||
|
|
||||||
|
T_StoneAshlar_BC.png T_StoneAshlar_R.png
|
||||||
|
T_Stucco_BC.png T_Stucco_R.png
|
||||||
|
T_Asphalt_BC.png T_Asphalt_R.png
|
||||||
|
T_Concrete_BC.png T_Concrete_R.png
|
||||||
|
|
||||||
|
Determinism: every pattern is a deterministic hash of the pixel coordinate, so a
|
||||||
|
re-run reproduces the same files byte for byte (no RNG state, no seam drift).
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import argparse
|
||||||
|
import math
|
||||||
|
import os
|
||||||
|
import struct
|
||||||
|
import zlib
|
||||||
|
|
||||||
|
# --- PNG -------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
||||||
|
def write_png(path, width, height, rgb_rows):
|
||||||
|
"""Write 8-bit RGB rows (bytes) to a PNG. `rgb_rows` is one bytes per row."""
|
||||||
|
raw = bytearray()
|
||||||
|
for y in range(height):
|
||||||
|
raw.append(0) # filter type 0
|
||||||
|
raw += rgb_rows[y]
|
||||||
|
|
||||||
|
def chunk(tag, data):
|
||||||
|
c = struct.pack(">I", len(data)) + tag + data
|
||||||
|
c += struct.pack(">I", zlib.crc32(tag + data) & 0xFFFFFFFF)
|
||||||
|
return c
|
||||||
|
|
||||||
|
ihdr = struct.pack(">IIBBBBB", width, height, 8, 2, 0, 0, 0) # colour type 2 = RGB
|
||||||
|
png = b"\x89PNG\r\n\x1a\n" + chunk(b"IHDR", ihdr)
|
||||||
|
png += chunk(b"IDAT", zlib.compress(bytes(raw), 6)) + chunk(b"IEND", b"")
|
||||||
|
os.makedirs(os.path.dirname(os.path.abspath(path)), exist_ok=True)
|
||||||
|
with open(path, "wb") as handle:
|
||||||
|
handle.write(png)
|
||||||
|
return path
|
||||||
|
|
||||||
|
|
||||||
|
# --- deterministic value noise --------------------------------------------
|
||||||
|
|
||||||
|
|
||||||
|
def _hash2(x, y, seed=0):
|
||||||
|
"""Deterministic 0..1 hash of an integer cell. No RNG, fully reproducible."""
|
||||||
|
n = (x * 374761393 + y * 668265263 + seed * 1442695040888963407) & 0xFFFFFFFF
|
||||||
|
n = (n ^ (n >> 13)) * 1274126177 & 0xFFFFFFFF
|
||||||
|
return ((n ^ (n >> 16)) & 0xFFFF) / 65535.0
|
||||||
|
|
||||||
|
|
||||||
|
def value_noise(x, y, seed=0):
|
||||||
|
"""Smooth bilinear value noise at continuous (x, y) in cell units."""
|
||||||
|
xi, yi = math.floor(x), math.floor(y)
|
||||||
|
fx, fy = x - xi, y - yi
|
||||||
|
# smoothstep weights so cells blend without visible lattice lines
|
||||||
|
sx = fx * fx * (3.0 - 2.0 * fx)
|
||||||
|
sy = fy * fy * (3.0 - 2.0 * fy)
|
||||||
|
n00 = _hash2(xi, yi, seed)
|
||||||
|
n10 = _hash2(xi + 1, yi, seed)
|
||||||
|
n01 = _hash2(xi, yi + 1, seed)
|
||||||
|
n11 = _hash2(xi + 1, yi + 1, seed)
|
||||||
|
return (n00 * (1 - sx) + n10 * sx) * (1 - sy) + (n01 * (1 - sx) + n11 * sx) * sy
|
||||||
|
|
||||||
|
|
||||||
|
def fractal(x, y, octaves=4, seed=0):
|
||||||
|
"""Fractal sum of value noise, normalised to 0..1."""
|
||||||
|
total, amp, freq, norm = 0.0, 1.0, 1.0, 0.0
|
||||||
|
for _ in range(octaves):
|
||||||
|
total += value_noise(x * freq, y * freq, seed) * amp
|
||||||
|
norm += amp
|
||||||
|
amp *= 0.5
|
||||||
|
freq *= 2.0
|
||||||
|
return total / norm
|
||||||
|
|
||||||
|
|
||||||
|
def clamp8(value):
|
||||||
|
return max(0, min(255, int(value * 255.0 + 0.5)))
|
||||||
|
|
||||||
|
|
||||||
|
# --- patterns --------------------------------------------------------------
|
||||||
|
# Each returns (base_rgb_function, roughness_function) over u, v in 0..1.
|
||||||
|
|
||||||
|
|
||||||
|
def stone_ashlar(courses=8, per_course=4):
|
||||||
|
"""Coursed ashlar: horizontal beds, staggered vertical joints, stone grain."""
|
||||||
|
course_h = 1.0 / courses
|
||||||
|
|
||||||
|
def block_uv(u, v):
|
||||||
|
row = int(v / course_h)
|
||||||
|
shift = 0.5 * (row % 2) / per_course # stagger alternate courses
|
||||||
|
col = int(((u + shift) % 1.0) * per_course)
|
||||||
|
return row, col, (v % course_h) / course_h, ((u + shift) % 1.0) * per_course - col
|
||||||
|
|
||||||
|
def base(u, v, px, py):
|
||||||
|
row, col, fj, fi = block_uv(u, v)
|
||||||
|
tone = 0.62 + 0.10 * _hash2(row, col, 11) # each block a little different
|
||||||
|
grain = 0.06 * (fractal(px * 0.15, py * 0.15, 5, 3) - 0.5) * 2.0
|
||||||
|
# joint: a dark band along the bed and the head of each block
|
||||||
|
joint = 0.0
|
||||||
|
if fj < 0.055 or fi < 0.055:
|
||||||
|
joint = -0.22
|
||||||
|
value = tone + grain + joint
|
||||||
|
return (value * 1.02, value, value * 0.94) # faintly warm stone
|
||||||
|
|
||||||
|
def rough(u, v, px, py):
|
||||||
|
row, col, fj, fi = block_uv(u, v)
|
||||||
|
base_r = 0.66 + 0.10 * _hash2(row, col, 29)
|
||||||
|
if fj < 0.055 or fi < 0.055:
|
||||||
|
base_r = 0.86 # joints are matte
|
||||||
|
return base_r + 0.06 * fractal(px * 0.2, py * 0.2, 3, 7)
|
||||||
|
|
||||||
|
return base, rough
|
||||||
|
|
||||||
|
|
||||||
|
def stucco():
|
||||||
|
def base(u, v, px, py):
|
||||||
|
blot = fractal(px * 0.03, py * 0.03, 4, 21)
|
||||||
|
tooth = fractal(px * 0.9, py * 0.9, 3, 22)
|
||||||
|
value = 0.58 + 0.12 * (blot - 0.5) + 0.05 * (tooth - 0.5)
|
||||||
|
return (value * 1.03, value, value * 0.96)
|
||||||
|
|
||||||
|
def rough(u, v, px, py):
|
||||||
|
return 0.74 + 0.18 * fractal(px * 0.06, py * 0.06, 3, 31)
|
||||||
|
|
||||||
|
return base, rough
|
||||||
|
|
||||||
|
|
||||||
|
def asphalt():
|
||||||
|
def base(u, v, px, py):
|
||||||
|
agg = fractal(px * 1.4, py * 1.4, 3, 41) # fine aggregate
|
||||||
|
patch = fractal(px * 0.02, py * 0.02, 3, 42) # repair patches
|
||||||
|
value = 0.055 + 0.05 * (patch - 0.5) + 0.06 * (agg - 0.5)
|
||||||
|
return (value, value, value * 1.05) # cool bitumen
|
||||||
|
|
||||||
|
def rough(u, v, px, py):
|
||||||
|
return 0.90 + 0.08 * fractal(px * 0.5, py * 0.5, 2, 43)
|
||||||
|
|
||||||
|
return base, rough
|
||||||
|
|
||||||
|
|
||||||
|
def concrete():
|
||||||
|
def base(u, v, px, py):
|
||||||
|
pores = fractal(px * 0.8, py * 0.8, 3, 51)
|
||||||
|
tone = fractal(px * 0.04, py * 0.04, 4, 52)
|
||||||
|
value = 0.50 + 0.10 * (tone - 0.5) + 0.05 * (pores - 0.5)
|
||||||
|
return (value * 1.01, value, value * 0.97)
|
||||||
|
|
||||||
|
def rough(u, v, px, py):
|
||||||
|
return 0.84 + 0.10 * fractal(px * 0.15, py * 0.15, 3, 53)
|
||||||
|
|
||||||
|
return base, rough
|
||||||
|
|
||||||
|
|
||||||
|
TEXTURES = {
|
||||||
|
"T_StoneAshlar": stone_ashlar(),
|
||||||
|
"T_Stucco": stucco(),
|
||||||
|
"T_Asphalt": asphalt(),
|
||||||
|
"T_Concrete": concrete(),
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
def render(name, pattern, size):
|
||||||
|
"""Render one texture; returns {suffix: path} for _BC and _R."""
|
||||||
|
base_fn, rough_fn = pattern
|
||||||
|
paths = {}
|
||||||
|
for suffix, fn in (("BC", base_fn), ("R", rough_fn)):
|
||||||
|
# Wrap in a function so the inner loop stays tight.
|
||||||
|
rows = []
|
||||||
|
for y in range(size):
|
||||||
|
v = y / size
|
||||||
|
row = bytearray()
|
||||||
|
for x in range(size):
|
||||||
|
u = x / size
|
||||||
|
value = fn(u, v, x / size * size / 64.0, y / size * size / 64.0)
|
||||||
|
if isinstance(value, tuple):
|
||||||
|
row += bytes((clamp8(value[0]), clamp8(value[1]), clamp8(value[2])))
|
||||||
|
else:
|
||||||
|
grey = clamp8(value)
|
||||||
|
row += bytes((grey, grey, grey))
|
||||||
|
rows.append(bytes(row))
|
||||||
|
path = os.path.join(OUT_DIR, f"{name}_{suffix}.png")
|
||||||
|
write_png(path, size, size, rows)
|
||||||
|
paths[suffix] = path
|
||||||
|
return paths
|
||||||
|
|
||||||
|
|
||||||
|
def main(argv=None) -> int:
|
||||||
|
global OUT_DIR
|
||||||
|
ap = argparse.ArgumentParser(description=__doc__,
|
||||||
|
formatter_class=argparse.RawDescriptionHelpFormatter)
|
||||||
|
ap.add_argument("--out", default="/tmp/gigline/textures",
|
||||||
|
help="output directory (default /tmp/gigline/textures)")
|
||||||
|
ap.add_argument("--size", type=int, default=512, help="edge length in pixels")
|
||||||
|
args = ap.parse_args(argv)
|
||||||
|
OUT_DIR = args.out
|
||||||
|
|
||||||
|
print(f"writing {len(TEXTURES) * 2} maps at {args.size}² to {OUT_DIR}")
|
||||||
|
for name, pattern in TEXTURES.items():
|
||||||
|
paths = render(name, pattern, args.size)
|
||||||
|
for suffix, path in paths.items():
|
||||||
|
print(f" {os.path.basename(path):28} {os.path.getsize(path):>8} bytes")
|
||||||
|
return 0
|
||||||
|
|
||||||
|
|
||||||
|
OUT_DIR = "/tmp/gigline/textures"
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
raise SystemExit(main())
|
||||||
@@ -174,6 +174,20 @@ def build():
|
|||||||
cap_z0 = HEIGHT - ENTABLATURE_H - PARAPET_H - CAPITAL_H
|
cap_z0 = HEIGHT - ENTABLATURE_H - PARAPET_H - CAPITAL_H
|
||||||
cylinder(f"Hero_Column_{i + 1}", x, -0.55, shaft_z0, cap_z0,
|
cylinder(f"Hero_Column_{i + 1}", x, -0.55, shaft_z0, cap_z0,
|
||||||
COLUMN_D / 2.0, stone)
|
COLUMN_D / 2.0, stone)
|
||||||
|
|
||||||
|
# Fluting: 20 shallow grooves around the shaft, as the photographs show on
|
||||||
|
# the colonnade. Each flute is a thin cylinder set into the shaft surface;
|
||||||
|
# modelling them as grooves would need booleans on 6 columns for detail the
|
||||||
|
# eye reads only in silhouette and grazing light, so they are added mass in
|
||||||
|
# the *shadow* direction instead — cheap, and it breaks the flat highlight.
|
||||||
|
flutes = 20
|
||||||
|
flute_r = 0.032
|
||||||
|
for f in range(flutes):
|
||||||
|
ang = (2.0 * math.pi * f) / flutes
|
||||||
|
fx = x + (COLUMN_D / 2.0 - flute_r * 0.4) * math.cos(ang)
|
||||||
|
fy = -0.55 + (COLUMN_D / 2.0 - flute_r * 0.4) * math.sin(ang)
|
||||||
|
cylinder(f"Hero_Flute_{i + 1}_{f + 1}", fx, fy, shaft_z0 + 0.05,
|
||||||
|
cap_z0 - 0.05, flute_r, stone_dark, vertices=6)
|
||||||
# Capital block, then the square abacus that carries the architrave.
|
# Capital block, then the square abacus that carries the architrave.
|
||||||
# The abacus takes the top 0.14 m of the capital zone so the colonnade
|
# The abacus takes the top 0.14 m of the capital zone so the colonnade
|
||||||
# still ends at HEIGHT - ENTABLATURE_H - PARAPET_H (proportion unchanged).
|
# still ends at HEIGHT - ENTABLATURE_H - PARAPET_H (proportion unchanged).
|
||||||
@@ -185,6 +199,9 @@ def build():
|
|||||||
1.30, 1.30, abacus_h, stone)
|
1.30, 1.30, abacus_h, stone)
|
||||||
note("Hero_Column_x6", (COLUMN_D, COLUMN_D, HEIGHT - ENTABLATURE_H - PARAPET_H - PEDESTAL_H),
|
note("Hero_Column_x6", (COLUMN_D, COLUMN_D, HEIGHT - ENTABLATURE_H - PARAPET_H - PEDESTAL_H),
|
||||||
f"6 columns at {COLUMN_BAY} m centres (sibling notes x measured frontage)")
|
f"6 columns at {COLUMN_BAY} m centres (sibling notes x measured frontage)")
|
||||||
|
note("Hero_Flute_x120", (0.064, 0.064, cap_z0 - shaft_z0),
|
||||||
|
"photographs: fluted shafts; 20 grooves per column modelled as shallow "
|
||||||
|
"cylinders in the shadow tone rather than boolean cut grooves")
|
||||||
note("Hero_ColumnBase_x6", (1.36, 1.36, 0.22),
|
note("Hero_ColumnBase_x6", (1.36, 1.36, 0.22),
|
||||||
"assumption: Ionic base moulding; photographs show a wider base at the shaft foot")
|
"assumption: Ionic base moulding; photographs show a wider base at the shaft foot")
|
||||||
note("Hero_Abacus_x6", (1.30, 1.30, 0.14),
|
note("Hero_Abacus_x6", (1.30, 1.30, 0.14),
|
||||||
@@ -223,10 +240,22 @@ def build():
|
|||||||
FRONTAGE_W, 1.2, PARAPET_H, stone)
|
FRONTAGE_W, 1.2, PARAPET_H, stone)
|
||||||
note("Hero_Parapet", (FRONTAGE_W, 1.2, PARAPET_H), "assumption: parapet carries the motto band")
|
note("Hero_Parapet", (FRONTAGE_W, 1.2, PARAPET_H), "assumption: parapet carries the motto band")
|
||||||
|
|
||||||
# Motto band: a recessed panel across the parapet, as the photographs show.
|
# Motto band: a recessed panel across the parapet with a moulded surround, as
|
||||||
cube("Hero_MottoBand", 0.0, -1.32, HEIGHT - PARAPET_H / 2.0,
|
# the photographs show. The panel was a 8 cm stripe of dark stone; it is now a
|
||||||
FRONTAGE_W - 3.0, 0.08, 0.85, stone_dark)
|
# real recess — surround frame standing 6 cm proud, panel set 12 cm behind it —
|
||||||
note("Hero_MottoBand", (FRONTAGE_W - 3.0, 0.08, 0.85), "photographs: motto panel above the colonnade")
|
# so it reads as an inscription panel from the street instead of a paint line.
|
||||||
|
motto_z = HEIGHT - PARAPET_H / 2.0
|
||||||
|
motto_w = FRONTAGE_W - 3.0
|
||||||
|
for x_off in (-(motto_w / 2.0 + 0.16), (motto_w / 2.0 + 0.16)):
|
||||||
|
cube(f"Hero_MottoSurround_v{int(x_off > 0)}", x_off, -1.13, motto_z,
|
||||||
|
0.32, 0.26, 1.15, stone)
|
||||||
|
for z_off in (-0.575, 0.575):
|
||||||
|
cube(f"Hero_MottoSurround_h{int(z_off > 0)}", 0.0, -1.13, motto_z + z_off,
|
||||||
|
motto_w + 0.64, 0.26, 0.32, stone)
|
||||||
|
cube("Hero_MottoBand", 0.0, -1.06, motto_z, motto_w, 0.10, 0.85, stone_dark)
|
||||||
|
note("Hero_MottoBand", (motto_w, 0.10, 0.85),
|
||||||
|
"photographs: motto panel above the colonnade, now an inset panel with a "
|
||||||
|
"moulded surround rather than a flat stripe")
|
||||||
|
|
||||||
# Medallions either side of the inscription: discs on the parapet face, not
|
# Medallions either side of the inscription: discs on the parapet face, not
|
||||||
# pancakes on the coping (see disc_on_wall).
|
# pancakes on the coping (see disc_on_wall).
|
||||||
|
|||||||
+52
-35
@@ -1,56 +1,73 @@
|
|||||||
#!/usr/bin/env bash
|
#!/usr/bin/env bash
|
||||||
# Render in-game screenshots for visual review into <repo>/Screenshots/ (gitignored).
|
# Render the demo-map screenshots used for visual review into <repo>/Screenshots/demo/.
|
||||||
#
|
#
|
||||||
# Usage: Tools/render_screenshots.sh
|
# Usage: Tools/render_screenshots.sh [--verify]
|
||||||
# Rebuilds /Game/Test/ReviewMap and /Game/Hollywood/DemoMap (shot lists), runs
|
#
|
||||||
# the game on each map (the GigLineShotActor walks the cameras, captures, quits),
|
# Only the **demo map** is captured: the review staging map exists to hold the
|
||||||
# and copies the PNGs into Screenshots/. PNGs are never committed.
|
# hero/camera list, not to produce review screenshots. Everything here is what a
|
||||||
|
# player actually sees when the demo runs.
|
||||||
|
#
|
||||||
|
# --verify run Tools/visual_loop.py --verify afterwards and fail if a shot
|
||||||
|
# does not show the content its expectation describes
|
||||||
|
#
|
||||||
|
# The PNGs are committed (see Screenshots/README.md): a reviewer must see exactly
|
||||||
|
# what the build rendered.
|
||||||
set -euo pipefail
|
set -euo pipefail
|
||||||
|
|
||||||
REPO="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
|
REPO="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
|
||||||
ENGINE="${UE_ENGINE_DIR:-/run/media/system/DATA/Engines/UE_5.8.4}"
|
ENGINE="${UE_ENGINE_DIR:-/run/media/system/DATA/Engines/UE_5.8.4}"
|
||||||
PROJ="$REPO/Game/GigLine/GigLine.uproject"
|
PROJ="$REPO/Game/GigLine/GigLine.uproject"
|
||||||
|
VERIFY="no"
|
||||||
|
[ "${1:-}" = "--verify" ] && VERIFY="yes"
|
||||||
|
|
||||||
echo "== 1/4 build review + demo shot maps =="
|
run_cmdlet() {
|
||||||
cd "$ENGINE"
|
local script="$1" label="$2"
|
||||||
./Engine/Binaries/Linux/UnrealEditor-Cmd "$PROJ" \
|
echo "== $label =="
|
||||||
-run=pythonscript -script="$REPO/Tools/ue_build_review_map.py" \
|
( cd "$ENGINE" && ./Engine/Binaries/Linux/UnrealEditor-Cmd "$PROJ" \
|
||||||
-nullrhi -unattended -nopause -stdout
|
-run=pythonscript -script="$REPO/Tools/$script" \
|
||||||
cat /tmp/ue-review-map-result.txt | tail -3
|
-nullrhi -unattended -nopause -stdout )
|
||||||
|
|
||||||
run_capture() {
|
|
||||||
local map="$1" label="$2"
|
|
||||||
echo "== capture: $label =="
|
|
||||||
./Engine/Binaries/Linux/UnrealEditor-Cmd "$PROJ" "$map" \
|
|
||||||
-game -windowed -ResX=1920 -ResY=1080 -nopause -nosplash -stdout
|
|
||||||
}
|
}
|
||||||
|
|
||||||
echo "== 2/4 run game on review map (hero/scene shots) =="
|
# Order matters. The demo builder rebuilds the site materials, so any actor staged
|
||||||
run_capture "/Game/Test/ReviewMap" "review"
|
# by another tool must be (re)staged after it or it keeps a reference to a material
|
||||||
|
# that no longer exists - that is how 604 actors ended up untextured once.
|
||||||
|
run_cmdlet ue_build_demo_map.py "1/5 rebuild the demo map (materials, site, frontage)"
|
||||||
|
run_cmdlet ue_apply_facade_kit.py "2/5 assemble the facade kit"
|
||||||
|
run_cmdlet ue_stage_theatre_trim.py "3/5 stage the theatre trim"
|
||||||
|
run_cmdlet ue_audit_scene.py "4/6 audit the scene (materials, placement, labels)"
|
||||||
|
run_cmdlet ue_build_review_map.py "5/6 stage the demo shot cameras"
|
||||||
|
tail -3 /tmp/ue-review-map-result.txt 2>/dev/null || true
|
||||||
|
|
||||||
echo "== 3/4 run game on demo map (walkable-demo shots) =="
|
echo "== 6/6 capture the demo map =="
|
||||||
run_capture "/Game/Hollywood/DemoMap" "demo"
|
./Engine/Binaries/Linux/UnrealEditor-Cmd "$PROJ" /Game/Hollywood/DemoMap \
|
||||||
|
-game -windowed -ResX=1920 -ResY=1080 -nopause -nosplash -stdout
|
||||||
|
|
||||||
echo "== 4/4 collect screenshots =="
|
echo "== collect the demo screenshots =="
|
||||||
mkdir -p "$REPO/Screenshots/demo"
|
mkdir -p "$REPO/Screenshots/demo"
|
||||||
rm -f "$REPO"/Screenshots/review-*.png "$REPO"/Screenshots/demo/review-*.png
|
rm -f "$REPO"/Screenshots/demo/review-*.png
|
||||||
found=0
|
found=0
|
||||||
for src in \
|
for src in \
|
||||||
"$REPO/Game/GigLine/Saved/Screenshots"/*/review-*.png \
|
"$REPO/Game/GigLine/Saved/Screenshots"/*/review-*.png \
|
||||||
"$REPO/Game/GigLine/Saved/Screenshots"/*/*/review-*.png \
|
"$REPO/Game/GigLine/Saved/Screenshots"/*/*/review-*.png \
|
||||||
"$HOME/.config/Epic/GigLine/Saved/Screenshots"/*/review-*.png; do
|
"$HOME/.config/Epic/GigLine/Saved/Screenshots"/*/review-*.png; do
|
||||||
[ -e "$src" ] || continue
|
[ -e "$src" ] || continue
|
||||||
name=$(basename "$src")
|
case "$(basename "$src")" in
|
||||||
if [[ "$name" == review-?("")-demo-* || "$name" == review-*-demo-* || "$name" == review-0?-demo-* ]] \
|
*demo-*) cp -f "$src" "$REPO/Screenshots/demo/"; found=1 ;;
|
||||||
|| [[ "$name" == *demo-* ]]; then
|
esac
|
||||||
cp -f "$src" "$REPO/Screenshots/demo/$name"
|
|
||||||
else
|
|
||||||
cp -f "$src" "$REPO/Screenshots/$name"
|
|
||||||
fi
|
|
||||||
found=1
|
|
||||||
done
|
done
|
||||||
[ "$found" = "1" ] || { echo "ERROR: no review-*.png found"; exit 1; }
|
[ "$found" = "1" ] || { echo "ERROR: no demo screenshots found"; exit 1; }
|
||||||
echo "--- Screenshots/ ---"
|
|
||||||
ls -1 "$REPO/Screenshots"
|
|
||||||
echo "--- Screenshots/demo/ ---"
|
|
||||||
ls -1 "$REPO/Screenshots/demo"
|
ls -1 "$REPO/Screenshots/demo"
|
||||||
|
|
||||||
|
# The scene audit is a gate: a wrong-material or misplaced actor is exactly the
|
||||||
|
# kind of failure that still renders plausibly.
|
||||||
|
if grep -q "failure(s) ==" /tmp/gigline/scene-audit.txt 2>/dev/null; then
|
||||||
|
grep -E "FAIL|failure\(s\)" /tmp/gigline/scene-audit.txt | tail -5
|
||||||
|
grep -q "== 0 failure(s) ==" /tmp/gigline/scene-audit.txt || {
|
||||||
|
echo "ERROR: scene audit failed — see /tmp/gigline/scene-audit.txt"; exit 1; }
|
||||||
|
fi
|
||||||
|
|
||||||
|
if [ "$VERIFY" = "yes" ]; then
|
||||||
|
echo "== verify: does each shot show what it should? =="
|
||||||
|
cd "$REPO"
|
||||||
|
python3 Tools/visual_loop.py --verify
|
||||||
|
fi
|
||||||
@@ -0,0 +1,70 @@
|
|||||||
|
"""Tests for Tools/ue_common.py — the shared Unreal commandlet helpers.
|
||||||
|
|
||||||
|
Pure stdlib: these helpers exist to turn two silent failures into loud ones, so
|
||||||
|
they are tested without starting Unreal.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import stat
|
||||||
|
import sys
|
||||||
|
import tempfile
|
||||||
|
import unittest
|
||||||
|
|
||||||
|
ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), "..", ".."))
|
||||||
|
sys.path.insert(0, os.path.join(ROOT, "Tools"))
|
||||||
|
|
||||||
|
import ue_common # noqa: E402
|
||||||
|
|
||||||
|
|
||||||
|
class MakeWritableTest(unittest.TestCase):
|
||||||
|
def test_read_only_file_is_made_writable(self):
|
||||||
|
# Exactly how an LFS-tracked .umap checks out: 0o444.
|
||||||
|
with tempfile.NamedTemporaryFile(delete=False) as handle:
|
||||||
|
path = handle.name
|
||||||
|
try:
|
||||||
|
os.chmod(path, 0o444)
|
||||||
|
self.assertFalse(os.access(path, os.W_OK))
|
||||||
|
note = ue_common.make_writable(path)
|
||||||
|
self.assertIsNotNone(note)
|
||||||
|
self.assertIn("chmod u+w", note)
|
||||||
|
self.assertTrue(os.access(path, os.W_OK))
|
||||||
|
finally:
|
||||||
|
os.chmod(path, 0o644)
|
||||||
|
os.unlink(path)
|
||||||
|
|
||||||
|
def test_writable_file_reports_nothing(self):
|
||||||
|
with tempfile.NamedTemporaryFile(delete=False) as handle:
|
||||||
|
path = handle.name
|
||||||
|
try:
|
||||||
|
os.chmod(path, 0o644)
|
||||||
|
self.assertIsNone(ue_common.make_writable(path))
|
||||||
|
finally:
|
||||||
|
os.unlink(path)
|
||||||
|
|
||||||
|
def test_missing_file_raises(self):
|
||||||
|
# A typo must not look like a permissions problem.
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
with self.assertRaises(FileNotFoundError):
|
||||||
|
ue_common.make_writable(os.path.join(tmp, "absent.umap"))
|
||||||
|
|
||||||
|
def test_read_bits_are_preserved(self):
|
||||||
|
with tempfile.NamedTemporaryFile(delete=False) as handle:
|
||||||
|
path = handle.name
|
||||||
|
try:
|
||||||
|
os.chmod(path, 0o400)
|
||||||
|
ue_common.make_writable(path)
|
||||||
|
mode = os.stat(path).st_mode
|
||||||
|
self.assertTrue(mode & stat.S_IRUSR)
|
||||||
|
finally:
|
||||||
|
os.chmod(path, 0o644)
|
||||||
|
os.unlink(path)
|
||||||
|
|
||||||
|
|
||||||
|
class RequireSavedTest(unittest.TestCase):
|
||||||
|
def test_passes_the_save_result_through(self):
|
||||||
|
self.assertTrue(ue_common.require_saved(True, "/Game/X"))
|
||||||
|
self.assertFalse(ue_common.require_saved(False, "/Game/X"))
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
@@ -0,0 +1,86 @@
|
|||||||
|
"""Tests for Tools/ue_placement.py — the single source of truth for placement.
|
||||||
|
|
||||||
|
A regression test for a real bug: `ue_hero_facade_import.py` carried its own hero
|
||||||
|
position (the pre-0.1.0 one) while `ue_build_demo_map.py` staged the verified one.
|
||||||
|
Running the importer moved the hero 13 m east and onto the north side of the
|
||||||
|
boulevard, and nothing complained because a displaced building still renders.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
import unittest
|
||||||
|
|
||||||
|
ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), "..", ".."))
|
||||||
|
sys.path.insert(0, os.path.join(ROOT, "Tools"))
|
||||||
|
|
||||||
|
import ue_placement # noqa: E402
|
||||||
|
|
||||||
|
|
||||||
|
class HeroPlacementTest(unittest.TestCase):
|
||||||
|
def test_hero_actor_name(self):
|
||||||
|
self.assertEqual(ue_placement.HERO_ACTOR, "Hero_Facade")
|
||||||
|
|
||||||
|
def test_hero_location_is_the_verified_building_line(self):
|
||||||
|
x, y, z = ue_placement.HERO_LOCATION_CM
|
||||||
|
self.assertEqual((x, y, z), (-13336.0, 1307.0, 0.0))
|
||||||
|
# North is -Y in Unreal, so the hero on the south side must have Y > 0.
|
||||||
|
self.assertGreater(y, 0.0, "hero must be on the south side (Y > 0)")
|
||||||
|
|
||||||
|
def test_hero_span_is_verified(self):
|
||||||
|
# The mesh spans -1308..+1308 cm in X about its origin.
|
||||||
|
lo = ue_placement.HERO_LOCATION_CM[0] - 1308.0
|
||||||
|
hi = ue_placement.HERO_LOCATION_CM[0] + 1308.0
|
||||||
|
exp_min, exp_max, ok = ue_placement.span_error("hero", lo, hi)
|
||||||
|
self.assertTrue(ok)
|
||||||
|
# The mesh spans exactly the verified footprint, so the two agree.
|
||||||
|
self.assertAlmostEqual(exp_min, lo, delta=ue_placement.SPAN_TOLERANCE_CM)
|
||||||
|
self.assertAlmostEqual(exp_max, hi, delta=ue_placement.SPAN_TOLERANCE_CM)
|
||||||
|
|
||||||
|
|
||||||
|
class SpanValidatorTest(unittest.TestCase):
|
||||||
|
def test_misplaced_hero_is_reported(self):
|
||||||
|
# The exact failure observed: the stale import position put the hero here.
|
||||||
|
lo, hi = -13332.0, -10716.0
|
||||||
|
exp_min, exp_max, ok = ue_placement.span_error("hero", lo, hi)
|
||||||
|
self.assertFalse(ok)
|
||||||
|
message = ue_placement.describe_span_failure("hero", lo, hi)
|
||||||
|
self.assertIn("does not match the verified", message)
|
||||||
|
self.assertIn(f"{exp_min:.0f}", message)
|
||||||
|
|
||||||
|
def test_correct_span_reports_nothing(self):
|
||||||
|
self.assertEqual(ue_placement.describe_span_failure("hero", -14644, -12028), "")
|
||||||
|
|
||||||
|
def test_within_tolerance_passes(self):
|
||||||
|
# 99 cm off is inside the ±100 cm tolerance.
|
||||||
|
self.assertTrue(ue_placement.span_error("6922", -24276 + 99, -17700 - 99)[2])
|
||||||
|
|
||||||
|
def test_outside_tolerance_fails(self):
|
||||||
|
self.assertFalse(ue_placement.span_error("6922", -24276 + 101, -17700)[2])
|
||||||
|
|
||||||
|
def test_unknown_building_raises(self):
|
||||||
|
with self.assertRaises(KeyError):
|
||||||
|
ue_placement.span_error("no-such-building", 0, 0)
|
||||||
|
|
||||||
|
def test_every_frontage_span_is_ordered(self):
|
||||||
|
for name, (lo, hi) in ue_placement.FRONTAGE_SPANS_CM.items():
|
||||||
|
self.assertLess(lo, hi, f"{name} span is inverted")
|
||||||
|
|
||||||
|
|
||||||
|
class SingleSourceOfTruthTest(unittest.TestCase):
|
||||||
|
"""The import script and the demo builder must not carry their own literals."""
|
||||||
|
|
||||||
|
def test_import_script_uses_the_shared_constant(self):
|
||||||
|
src = open(os.path.join(ROOT, "Tools", "ue_hero_facade_import.py"),
|
||||||
|
encoding="utf-8").read()
|
||||||
|
self.assertIn("ue_placement.HERO_LOCATION_CM", src)
|
||||||
|
self.assertNotIn("(-12024.5, -1450.0, 0.0)", src,
|
||||||
|
"the stale pre-0.1.0 hero position must not come back")
|
||||||
|
|
||||||
|
def test_demo_builder_uses_the_shared_constant(self):
|
||||||
|
src = open(os.path.join(ROOT, "Tools", "ue_build_demo_map.py"),
|
||||||
|
encoding="utf-8").read()
|
||||||
|
self.assertIn("ue_placement.HERO_LOCATION_CM", src)
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
@@ -0,0 +1,139 @@
|
|||||||
|
"""Tests for Tools/visual_loop.py — metrics, comparison and feedback rules.
|
||||||
|
|
||||||
|
Synthetic 8-bit images are written to a temp directory and measured, so the
|
||||||
|
numbers can be reasoned about directly (a mid-grey flat frame has near-zero edge
|
||||||
|
density and contrast; a checkerboard has a lot of both).
|
||||||
|
"""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
import tempfile
|
||||||
|
import unittest
|
||||||
|
|
||||||
|
from PIL import Image
|
||||||
|
|
||||||
|
ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), "..", ".."))
|
||||||
|
sys.path.insert(0, os.path.join(ROOT, "Tools"))
|
||||||
|
|
||||||
|
import visual_loop as vl # noqa: E402
|
||||||
|
|
||||||
|
|
||||||
|
def write_image(path, kind, size=(64, 64)):
|
||||||
|
img = Image.new("RGB", size)
|
||||||
|
if kind == "flat":
|
||||||
|
img.paste((128, 128, 128), (0, 0, size[0], size[1]))
|
||||||
|
elif kind == "dark":
|
||||||
|
img.paste((10, 10, 10), (0, 0, size[0], size[1]))
|
||||||
|
elif kind == "checker":
|
||||||
|
px = img.load()
|
||||||
|
for y in range(size[1]):
|
||||||
|
for x in range(size[0]):
|
||||||
|
px[x, y] = (240, 240, 240) if (x // 4 + y // 4) % 2 else (20, 20, 20)
|
||||||
|
elif kind == "sky":
|
||||||
|
# bright top third, mid stone bottom: composition like an elevation
|
||||||
|
px = img.load()
|
||||||
|
for y in range(size[1]):
|
||||||
|
for x in range(size[0]):
|
||||||
|
px[x, y] = (150, 190, 230) if y < size[1] // 3 else (170, 165, 155)
|
||||||
|
img.save(path)
|
||||||
|
return path
|
||||||
|
|
||||||
|
|
||||||
|
class MetricsTest(unittest.TestCase):
|
||||||
|
def test_flat_frame_has_no_edges_and_no_contrast(self):
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
m = vl.metrics(write_image(os.path.join(tmp, "a.png"), "flat"))
|
||||||
|
self.assertEqual(m["size"], [64, 64])
|
||||||
|
self.assertAlmostEqual(m["mean_luma"], 128.0, delta=1.0)
|
||||||
|
self.assertLess(m["rms_contrast"], 1.0)
|
||||||
|
self.assertLess(m["edge_density"], vl.MIN_EDGE_DENSITY)
|
||||||
|
|
||||||
|
def test_checker_frame_has_edges_and_contrast(self):
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
m = vl.metrics(write_image(os.path.join(tmp, "b.png"), "checker"))
|
||||||
|
self.assertGreater(m["edge_density"], 0.01)
|
||||||
|
self.assertGreater(m["rms_contrast"], 50.0)
|
||||||
|
|
||||||
|
def test_composition_grid_is_nine_cells(self):
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
m = vl.metrics(write_image(os.path.join(tmp, "c.png"), "sky"))
|
||||||
|
self.assertEqual(len(m["grid_luma"]), 9)
|
||||||
|
# sky row should be brighter than the stone rows
|
||||||
|
self.assertGreater(m["grid_luma"][0], m["grid_luma"][8])
|
||||||
|
|
||||||
|
def test_dominant_palette_is_reported(self):
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
m = vl.metrics(write_image(os.path.join(tmp, "d.png"), "flat"))
|
||||||
|
self.assertTrue(m["dominant"])
|
||||||
|
self.assertAlmostEqual(m["dominant"][0]["share"], 1.0, delta=0.05)
|
||||||
|
|
||||||
|
|
||||||
|
class CompareTest(unittest.TestCase):
|
||||||
|
def _metrics(self, kind):
|
||||||
|
with tempfile.TemporaryDirectory() as tmp:
|
||||||
|
return vl.metrics(write_image(os.path.join(tmp, "x.png"), kind))
|
||||||
|
|
||||||
|
def test_identical_images_score_high(self):
|
||||||
|
m = self._metrics("checker")
|
||||||
|
delta = vl.compare(m, m)
|
||||||
|
self.assertGreater(delta["score"], 0.9)
|
||||||
|
self.assertEqual(delta["delta_luma"], 0.0)
|
||||||
|
|
||||||
|
def test_flat_against_checker_scores_low_on_edges(self):
|
||||||
|
flat, checker = self._metrics("flat"), self._metrics("checker")
|
||||||
|
delta = vl.compare(flat, checker)
|
||||||
|
self.assertLess(delta["edge_ratio"], 0.5)
|
||||||
|
self.assertLess(delta["score"], 0.9)
|
||||||
|
|
||||||
|
def test_dark_against_bright_reports_a_luma_delta(self):
|
||||||
|
dark, bright = self._metrics("dark"), self._metrics("checker")
|
||||||
|
delta = vl.compare(dark, bright)
|
||||||
|
self.assertLess(delta["delta_luma"], -50)
|
||||||
|
|
||||||
|
|
||||||
|
class FeedbackTest(unittest.TestCase):
|
||||||
|
def test_low_edge_ratio_names_the_modelling_skills(self):
|
||||||
|
notes = vl.feedback("x", {}, {}, {
|
||||||
|
"delta_luma": 0, "contrast_ratio": 1.0, "edge_ratio": 0.3,
|
||||||
|
"palette_distance": 10, "composition_correlation": 0.8, "score": 0.5})
|
||||||
|
joined = " ".join(notes)
|
||||||
|
self.assertIn("Relief", joined)
|
||||||
|
self.assertIn("blender-modeler", joined)
|
||||||
|
|
||||||
|
def test_bright_and_compressed_flags_lighting_and_lookdev(self):
|
||||||
|
notes = vl.feedback("x", {}, {}, {
|
||||||
|
"delta_luma": 40, "contrast_ratio": 0.5, "edge_ratio": 1.0,
|
||||||
|
"palette_distance": 10, "composition_correlation": 0.8, "score": 0.5})
|
||||||
|
joined = " ".join(notes)
|
||||||
|
self.assertIn("Too bright", joined)
|
||||||
|
self.assertIn("lookdev", joined)
|
||||||
|
|
||||||
|
def test_framing_mismatch_names_the_camera_skill(self):
|
||||||
|
notes = vl.feedback("x", {}, {}, {
|
||||||
|
"delta_luma": 0, "contrast_ratio": 1.0, "edge_ratio": 1.0,
|
||||||
|
"palette_distance": 10, "composition_correlation": 0.1, "score": 0.5})
|
||||||
|
self.assertIn("camera-cinematography", " ".join(notes))
|
||||||
|
|
||||||
|
def test_no_delta_reports_no_action(self):
|
||||||
|
notes = vl.feedback("x", {}, {}, {
|
||||||
|
"delta_luma": 0, "contrast_ratio": 1.0, "edge_ratio": 1.0,
|
||||||
|
"palette_distance": 5, "composition_correlation": 0.9, "score": 0.95})
|
||||||
|
self.assertIn("No metric is outside its tolerance", notes[0])
|
||||||
|
|
||||||
|
|
||||||
|
class TargetSpecTest(unittest.TestCase):
|
||||||
|
def test_every_target_has_expectation_and_reference(self):
|
||||||
|
for target in vl.load_targets():
|
||||||
|
self.assertTrue(target["expect"].strip(), target["name"])
|
||||||
|
self.assertTrue(target["references"], target["name"])
|
||||||
|
self.assertTrue(target["shot"].startswith("Screenshots/demo/"),
|
||||||
|
f"{target['name']} must be a demo-map shot")
|
||||||
|
|
||||||
|
def test_iteration_output_is_gitignored(self):
|
||||||
|
# Iterations are deliberately uncommitted until the owner confirms a shot.
|
||||||
|
ignore = open(os.path.join(ROOT, ".gitignore"), encoding="utf-8").read()
|
||||||
|
self.assertIn("/Art/reference/iterations/", ignore)
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
unittest.main()
|
||||||
@@ -0,0 +1,208 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""El Capitan Theatre (6838) trim: marquee canopy and blade sign.
|
||||||
|
|
||||||
|
The theatre blockout is a plain volume; what makes it read as *the* El Capitan
|
||||||
|
Theatre is the projecting marquee over the pavement and the vertical blade sign on
|
||||||
|
the corner. Both are built **in world coordinates** here, so the export carries its
|
||||||
|
position and the Unreal staging step is a single spawn at the origin — the same
|
||||||
|
convention as the block frontage and the north buildings.
|
||||||
|
|
||||||
|
The building's verified span (from `Geo/verification/2026-11-06-el-capitan-block.md`
|
||||||
|
and the staged mesh) is X −12082..−8210, street face at Y ≈ +1001, height 20.55 m.
|
||||||
|
|
||||||
|
blender -b --factory-startup --python Tools/theatre_trim_build.py -- \
|
||||||
|
--out /tmp/gigline/theatre --report /tmp/gigline/theatre-report.json
|
||||||
|
|
||||||
|
Everything the photographs do not resolve at this scale (exact marquee depth,
|
||||||
|
soffit height, sign lettering) is an assumption recorded in the part report.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import argparse
|
||||||
|
import json
|
||||||
|
import math
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
|
||||||
|
import bpy
|
||||||
|
|
||||||
|
sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
|
||||||
|
import fbx_export_preset as fbx_preset # noqa: E402
|
||||||
|
|
||||||
|
# Verified building span, metres (Unreal X, Y, Z but Blender-frame: X east,
|
||||||
|
# Y north, Z up, metres).
|
||||||
|
THEATRE_X0 = -120.82
|
||||||
|
THEATRE_X1 = -82.10
|
||||||
|
# Street face of the blockout volume, in BLENDER metres. Blender's +Y is north and
|
||||||
|
# Unreal's is -Y, so the theatre's Unreal face at Y +1001 cm is Blender Y -10.01 m.
|
||||||
|
# Getting this sign wrong put the whole marquee on the far side of the boulevard.
|
||||||
|
FACE_Y = -10.01
|
||||||
|
HEIGHT = 20.55
|
||||||
|
# Ground level at the theatre frontage, in the shared frame (Z up, metres). The
|
||||||
|
# pavement here is 4.56 m above the project datum, so every trim height below is
|
||||||
|
# measured from the pavement and then offset: the first build used absolute Z and
|
||||||
|
# put the marquee canopy 5 cm above the pavement, where the wall probe found it at
|
||||||
|
# knee height.
|
||||||
|
GROUND_Z = 4.56
|
||||||
|
|
||||||
|
# Marquee: centred on the frontage, projecting over the pavement.
|
||||||
|
MARQUEE_W = 12.0
|
||||||
|
MARQUEE_DEPTH = 2.60
|
||||||
|
MARQUEE_Z = GROUND_Z + 4.60 # underside of the canopy above the pavement
|
||||||
|
CANOPY_T = 0.30
|
||||||
|
FASCIA_H = 1.10
|
||||||
|
|
||||||
|
BLADE_W = 1.30
|
||||||
|
BLADE_H = 5.20
|
||||||
|
BLADE_Z0 = GROUND_Z + 5.20
|
||||||
|
BLADE_PROJECT = 2.00
|
||||||
|
|
||||||
|
PARTS: list[dict] = []
|
||||||
|
|
||||||
|
|
||||||
|
def make_material(name, value, rough=0.6):
|
||||||
|
mat = bpy.data.materials.new(name)
|
||||||
|
mat.use_nodes = True
|
||||||
|
bsdf = mat.node_tree.nodes.get("Principled BSDF")
|
||||||
|
bsdf.inputs["Base Color"].default_value = (value, value * 0.97, value * 0.92, 1.0)
|
||||||
|
bsdf.inputs["Roughness"].default_value = rough
|
||||||
|
return mat
|
||||||
|
|
||||||
|
|
||||||
|
def validate(obj, tri_budget=6000):
|
||||||
|
import bmesh
|
||||||
|
|
||||||
|
bm = bmesh.new()
|
||||||
|
bm.from_mesh(obj.data)
|
||||||
|
bm.edges.ensure_lookup_table()
|
||||||
|
bad = [e for e in bm.edges if len(e.link_faces) != 2]
|
||||||
|
loose = [v for v in bm.verts if not v.link_edges]
|
||||||
|
tris = sum(len(f.verts) - 2 for f in bm.faces)
|
||||||
|
bm.free()
|
||||||
|
if bad or loose:
|
||||||
|
print(f"[trim] WARNING {obj.name}: {len(bad)} non-manifold, {len(loose)} loose")
|
||||||
|
if tris > tri_budget:
|
||||||
|
print(f"[trim] WARNING {obj.name}: {tris} tris over budget")
|
||||||
|
return {"manifold": not bad, "loose_verts": len(loose), "tris": tris}
|
||||||
|
|
||||||
|
|
||||||
|
def note(obj, text):
|
||||||
|
PARTS.append({
|
||||||
|
"object": obj.name,
|
||||||
|
"asset": "SM_" + obj.name,
|
||||||
|
"dims_m": [round(d, 3) for d in obj.dimensions],
|
||||||
|
"location_m": [round(c, 3) for c in obj.location],
|
||||||
|
"topology": validate(obj),
|
||||||
|
"evidence": text,
|
||||||
|
})
|
||||||
|
print(f"[trim] {obj.name:22} {obj.dimensions.x:6.2f} x {obj.dimensions.y:5.2f} "
|
||||||
|
f"x {obj.dimensions.z:5.2f} m ({text})")
|
||||||
|
|
||||||
|
|
||||||
|
def cube(name, x, y, z, dx, dy, dz, material):
|
||||||
|
bpy.ops.mesh.primitive_cube_add(size=1.0, location=(x, y, z))
|
||||||
|
obj = bpy.context.active_object
|
||||||
|
obj.name = name
|
||||||
|
obj.scale = (dx, dy, dz)
|
||||||
|
bpy.ops.object.transform_apply(location=False, rotation=False, scale=True)
|
||||||
|
obj.data.materials.append(material)
|
||||||
|
return obj
|
||||||
|
|
||||||
|
|
||||||
|
def clear_scene():
|
||||||
|
bpy.ops.object.select_all(action="SELECT")
|
||||||
|
bpy.ops.object.delete(use_global=False)
|
||||||
|
for mesh in list(bpy.data.meshes):
|
||||||
|
if mesh.users == 0:
|
||||||
|
bpy.data.meshes.remove(mesh)
|
||||||
|
|
||||||
|
|
||||||
|
def build():
|
||||||
|
"""Marquee canopy, fascia, soffit and the blade sign, in world metres."""
|
||||||
|
clear_scene()
|
||||||
|
stone = make_material("M_Trim_Stone", 0.62)
|
||||||
|
dark = make_material("M_Trim_Shadow", 0.30, rough=0.75)
|
||||||
|
lightbox = make_material("M_Trim_Lightbox", 0.86, rough=0.35)
|
||||||
|
|
||||||
|
cx = (THEATRE_X0 + THEATRE_X1) / 2.0
|
||||||
|
# The street is at smaller Unreal Y than the face, which is *larger* Blender Y,
|
||||||
|
# so the marquee projects in +Y here.
|
||||||
|
y_outer = FACE_Y + MARQUEE_DEPTH
|
||||||
|
|
||||||
|
parts = []
|
||||||
|
canopy = cube("Trim_MarqueeCanopy", cx, FACE_Y + MARQUEE_DEPTH / 2.0,
|
||||||
|
MARQUEE_Z + CANOPY_T / 2.0, MARQUEE_W, MARQUEE_DEPTH, CANOPY_T, stone)
|
||||||
|
parts.append((canopy, "photographs: projecting marquee over the pavement; "
|
||||||
|
"12.0 x 2.60 m at 4.60 m soffit — depth and height assumed"))
|
||||||
|
|
||||||
|
fascia = cube("Trim_MarqueeFascia", cx, y_outer - 0.12,
|
||||||
|
MARQUEE_Z + CANOPY_T + FASCIA_H / 2.0,
|
||||||
|
MARQUEE_W, 0.24, FASCIA_H, lightbox)
|
||||||
|
parts.append((fascia, "photographs: illuminated fascia band around the marquee "
|
||||||
|
"edge; 1.10 m depth assumed"))
|
||||||
|
|
||||||
|
soffit = cube("Trim_MarqueeSoffit", cx, FACE_Y + MARQUEE_DEPTH / 2.0,
|
||||||
|
MARQUEE_Z - 0.05, MARQUEE_W - 0.4, MARQUEE_DEPTH - 0.4, 0.10, dark)
|
||||||
|
parts.append((soffit, "assumption: recessed soffit panel under the canopy; "
|
||||||
|
"reads as shadow from the pavement"))
|
||||||
|
|
||||||
|
# Blade sign on the west corner, projecting perpendicular to the facade.
|
||||||
|
bx = THEATRE_X0 + 3.6
|
||||||
|
blade = cube("Trim_BladeSign", bx, FACE_Y + BLADE_PROJECT / 2.0,
|
||||||
|
BLADE_Z0 + BLADE_H / 2.0, BLADE_W, 0.42, BLADE_H, lightbox)
|
||||||
|
parts.append((blade, "photographs: vertical blade sign on the corner; "
|
||||||
|
"1.30 x 5.20 m, 2.00 m projection assumed"))
|
||||||
|
|
||||||
|
frame = cube("Trim_BladeSignFrame", bx, FACE_Y + BLADE_PROJECT / 2.0,
|
||||||
|
BLADE_Z0 + BLADE_H / 2.0, BLADE_W + 0.28, 0.60, 0.35, stone)
|
||||||
|
parts.append((frame, "assumption: stone frame band at the base of the blade"))
|
||||||
|
|
||||||
|
# Bracket back to the wall so the sign does not float.
|
||||||
|
bracket = cube("Trim_BladeSignBracket", bx, FACE_Y + BLADE_PROJECT - 0.6,
|
||||||
|
BLADE_Z0 + BLADE_H - 0.5, 0.60, 1.40, 0.30, dark)
|
||||||
|
parts.append((bracket, "assumption: bracket tying the blade back to the facade"))
|
||||||
|
|
||||||
|
return parts
|
||||||
|
|
||||||
|
|
||||||
|
def export_objects(parts, out_dir):
|
||||||
|
count = 0
|
||||||
|
for obj, _text in parts:
|
||||||
|
bpy.ops.object.select_all(action="DESELECT")
|
||||||
|
obj.select_set(True)
|
||||||
|
bpy.context.view_layer.objects.active = obj
|
||||||
|
path = os.path.join(out_dir, "SM_" + obj.name + ".fbx")
|
||||||
|
fbx_preset.export_static(path, selection=True)
|
||||||
|
count += 1
|
||||||
|
return count
|
||||||
|
|
||||||
|
|
||||||
|
def main(argv=None) -> int:
|
||||||
|
argv = sys.argv[sys.argv.index("--") + 1:] if "--" in sys.argv else []
|
||||||
|
ap = argparse.ArgumentParser(description=__doc__,
|
||||||
|
formatter_class=argparse.RawDescriptionHelpFormatter)
|
||||||
|
ap.add_argument("--out", default="/tmp/gigline/theatre")
|
||||||
|
ap.add_argument("--report", default=None)
|
||||||
|
args = ap.parse_args(argv)
|
||||||
|
os.makedirs(args.out, exist_ok=True)
|
||||||
|
|
||||||
|
parts = build()
|
||||||
|
for obj, text in parts:
|
||||||
|
note(obj, text)
|
||||||
|
exported = export_objects(parts, args.out)
|
||||||
|
print(f"[trim] exported {exported} object(s) to {args.out}")
|
||||||
|
|
||||||
|
if args.report:
|
||||||
|
with open(args.report, "w", encoding="utf-8") as handle:
|
||||||
|
json.dump({
|
||||||
|
"building": "El Capitan Theatre (6838)",
|
||||||
|
"span_m": [THEATRE_X0, THEATRE_X1, FACE_Y, HEIGHT],
|
||||||
|
"parts": PARTS,
|
||||||
|
}, handle, indent=2)
|
||||||
|
print(f"[trim] report -> {args.report}")
|
||||||
|
return 0
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
raise SystemExit(main())
|
||||||
Loaded 100 of 113 files, more files were not shown because too many files have changed in this diff.
Show more
Reference in new issue
Block a user