From 655d0a627c17b598562616a269e53bf06124d43f Mon Sep 17 00:00:00 2001 From: skal Date: Mon, 9 Feb 2026 12:37:57 +0100 Subject: docs: Archive Feb 9 completed tasks and clarify build configs - Move completed tasks (Uniform alignment, WGSL validation, test_demo fix) to COMPLETED.md - Clean up TODO.md and PROJECT_CONTEXT.md "Recently Completed" sections - Update HOWTO.md to clarify DEMO_ALL_OPTIONS enables STRIP_ALL - Note: test_demo PeakMeterEffect requires non-STRIP build Net: -26 lines (better context hygiene) Co-Authored-By: Claude Sonnet 4.5 --- TODO.md | 30 +----------------------------- 1 file changed, 1 insertion(+), 29 deletions(-) (limited to 'TODO.md') diff --git a/TODO.md b/TODO.md index a61ba59..46baa06 100644 --- a/TODO.md +++ b/TODO.md @@ -2,35 +2,7 @@ This file tracks prioritized tasks with detailed attack plans. -**Note:** For a history of recently completed tasks, see `COMPLETED.md`. - -## Recently Completed (February 9, 2026) - -- [x] **Fix test_demo Black Screen**: - - **Issue**: `test_demo` showed black screen because it failed to load its timeline sequence (`assets/test_demo.seq`). - - **Fix**: Added missing `LoadTimeline` call in `src/test_demo.cc`. - - **Result**: `FlashEffect` and `PeakMeterEffect` should now render correctly. - -- [x] **WGSL Uniform Buffer Validation & Consolidation (Task #75)**: - - **Standardization**: Refactored `DistortEffect` and others to use `CommonPostProcessUniforms` (binding 2) + `EffectParams` (binding 3). - - **Validation Tool**: Created `tools/validate_uniforms.py` to parse C++ and WGSL (including embedded strings) and verify size/alignment. - - **Integration**: Added validation step to CMake build system. - - **Cleanup**: Renamed generic `EffectParams` to specific names (`FadeParams`, `CircleMaskParams`, etc.) in WGSL and C++. - - **Documentation**: Added `doc/UNIFORM_BUFFER_GUIDELINES.md` and updated `CONTRIBUTING.md`. - -- [x] **Uniform Buffer Alignment (Task #74)**: Fixed WGSL struct alignment issues across multiple shaders: - - `circle_mask_compute.wgsl`: Changed `_pad: vec3` to three `f32` fields - - `fade_effect.cc`: Changed EffectParams padding from `vec3` to `_pad0/1/2: f32` - - `theme_modulation_effect.cc`: Same padding fix for EffectParams - - Fixed ODR violation in `demo_effects.h` (incomplete FadeEffect forward declaration) - - Renamed shadowing `uniforms_` members to `common_uniforms_`/`flash_uniforms_` - - Result: demo64k runs without crashes, 33/33 tests passing (100%) - -## Previously Completed (February 8, 2026) - -- [x] **Shader Parametrization System**: Full uniform parameter system with .seq syntax support. FlashEffect now supports color/decay parameters with per-frame animation. See `COMPLETED.md` for details. -- [x] **ChromaAberrationEffect Parametrization**: Added offset_scale and angle parameters. Supports diagonal and vertical aberration modes via .seq syntax. -- [x] **GaussianBlurEffect Parametrization**: Added strength parameter. Replaces hardcoded blur radius with configurable value. +**Note:** For completed tasks, see `doc/COMPLETED.md`. --- -- cgit v1.2.3