qa(diag): gate setup logs behind gigline.SetupLogs cvar #14

Merged
zwitschi merged 2 commits from qa-diagnostics-0.1.0 into main 2026-11-08 15:01:34 +01:00
Owner

Summary

Closes the TODO item "Remove TEMP QA diagnostics … or gate them behind a cvar" for 0.1.0.

The GigLine SIM / GigLine GROUND probes were already removed in the input fix (PR #8). What remained were the settle-timer log and the GigLine IN: input-setup logs — now gated behind a cvar.

Changes:

  • GigLineCharacter.cpp — added TAutoConsoleVariable<int32> gigline.SetupLogs; gated the settle-timer "start captured" log, mapping-context push, subsystem/PC errors, and "bound N actions" log.
  • DefaultEngine.ini — [ConsoleVariables] gigline.SetupLogs=0 (default off; re-enable at runtime with gigline.SetupLogs 1).
  • -qaselftest QA logs stay ungated (deliberate release-check feature per Docs/release-checks.md).
  • TODO.md — item closed; MEMORY.md — note.

Verified on the packaged build: 0 setup-log lines; -qaselftest still all PASS (walk 761 cm, reset 0 cm, pause/unpause, transect 0 holes, wall probe 0/35).

## Summary Closes the TODO item **"Remove TEMP QA diagnostics … or gate them behind a cvar"** for `0.1.0`. The `GigLine SIM` / `GigLine GROUND` probes were already removed in the input fix (PR #8). What remained were the settle-timer log and the `GigLine IN:` input-setup logs — now gated behind a cvar. **Changes:** - `GigLineCharacter.cpp` — added `TAutoConsoleVariable<int32> gigline.SetupLogs`; gated the settle-timer "start captured" log, mapping-context push, subsystem/PC errors, and "bound N actions" log. - `DefaultEngine.ini` — `[ConsoleVariables] gigline.SetupLogs=0` (default off; re-enable at runtime with `gigline.SetupLogs 1`). - `-qaselftest` QA logs stay ungated (deliberate release-check feature per `Docs/release-checks.md`). - `TODO.md` — item closed; `MEMORY.md` — note. **Verified on the packaged build:** 0 setup-log lines; `-qaselftest` still all PASS (walk 761 cm, reset 0 cm, pause/unpause, transect 0 holes, wall probe 0/35).
zwitschi added 2 commits 2026-11-08 15:01:30 +01:00
TODO item: remove/gate the TEMP QA diagnostics. The GigLine SIM/GROUND probes
were already removed (PR #8); what remained was the settle-timer log and the
GigLine IN: input-setup logs. Now gated behind a TAutoConsoleVariable
(gigline.SetupLogs, default 0 in DefaultEngine.ini [ConsoleVariables]), so a
shipped build's log has no setup spam; re-enable at runtime with
'gigline.SetupLogs 1'. The -qaselftest QA logs are a deliberate release-check
feature (Docs/release-checks.md) and stay ungated. Verified on the packaged
build: 0 setup-log lines, QA suite still all PASS.
zwitschi merged commit 31f073d608 into main 2026-11-08 15:01:34 +01:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: zwitschi/gig-line#14