From a88cc48be61ab2e468ee71c984e84ee1abe1ed9b Mon Sep 17 00:00:00 2001 From: Zwitschi Date: Sun, 8 Nov 2026 15:00:25 +0100 Subject: [PATCH] docs: close the diagnostics TODO item; memory note --- MEMORY.md | 12 ++++++++++++ TODO.md | 2 +- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/MEMORY.md b/MEMORY.md index 5cbea1d..a72ba08 100644 --- a/MEMORY.md +++ b/MEMORY.md @@ -319,3 +319,15 @@ Scope: facts and traps that cost time to establish. Planned work lives in [TODO. 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) + +- 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`. +- 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). diff --git a/TODO.md b/TODO.md index ef6d26f..a98a1c8 100644 --- a/TODO.md +++ b/TODO.md @@ -111,7 +111,7 @@ The MPFB character asset is planned as the NPC template (see `ROADMAP.md`). Keep - [x] Player can traverse both sides of the demo without falling through geometry. **Verified 2026-11-08:** Blender pre-check plus an in-engine swept transect (`-qaselftest`) over the north sidewalk, carriageway and south sidewalk from X -14900 to -9420 — 84 samples, 0 holes, ground Z 317..395 (sidewalk 395, road 366, south walk 317). The player spawns at Z 456 and stays there for the whole run (`fall=0`, single Z). - [x] Building placement, street scale, and site identity checked against references. **Done 2026-11-08:** rendered current DemoMap content, probed world-space geometry (road 24.3 m vs ref 20.9–25.9 m; sidewalk centreline offsets 12.5/13.2 vs ref 12.6/13.6 m; hero frontage 26.0 m, setback ~13 m vs ref 25.75/14.5 m; frontage + north buildings in verified OSM positions), recorded PASS + deliberate deviations (flat blockout surfaces → `0.1.4`, kerb upstand → `0.1.1`, kerb-flush ramps → `0.1.1`) in `Docs/release-checks.md`, and fixed the mis-aimed north-buildings camera (was 9.5 km west → 85% sky; now frames the buildings). Final human verdict pending on `Screenshots/demo/*`. - [x] No blocking collision or packaging defects. **Done 2026-11-08:** extended `-qaselftest` — transect now 1456 samples / 16 rows including the kerb/road/sidewalk seam Ys (0 holes), plus a ground-referenced 30 m wall probe (0/35 blocking hits; no invisible walls). Walk/reset/pause all PASS on the packaged build; `GigLineShotActor` now skips under `-qaselftest`. Evidence in `Docs/release-checks.md` §`0.1.0` audit note. -- [A] Remove TEMP QA diagnostics (`GigLine SIM` / `GigLine GROUND` log lines and the settle-timer log) from `GigLineCharacter.cpp` before the release tag, or gate them behind a cvar. (SIM/GROUND probes already removed in the input fix, PR #8. Remaining: the settle-timer "start captured" log -> gate behind a cvar. The `-qaselftest` and setup logs are deliberate features, keep.) +- [x] Remove TEMP QA diagnostics (`GigLine SIM` / `GigLine GROUND` log lines and the settle-timer log) from `GigLineCharacter.cpp` before the release tag, or gate them behind a cvar. **Done 2026-11-08:** SIM/GROUND probes were already removed in the input fix (PR #8); the remaining settle-timer + `GigLine IN:` input-setup logs are now gated behind the `gigline.SetupLogs` cvar (default 0 in `DefaultEngine.ini [ConsoleVariables]`, re-enable at runtime). Verified on the packaged build: 0 setup-log lines, `-qaselftest` still all PASS. - [ ] 60 FPS at 1440p on the dev workstation (Bazzite, Ryzen 7 7700X, 64 GB RAM, RTX 5070 Ti 16 GB) using a documented quality preset; record upscaling state; report performance without frame generation. - [ ] Fresh project checkout builds with the documented external engine installation. - [ ] Package Linux build with documented controls.