diff options
Diffstat (limited to 'doc/CNN_DEBUG.md')
| -rw-r--r-- | doc/CNN_DEBUG.md | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/doc/CNN_DEBUG.md b/doc/CNN_DEBUG.md index dba0b60..ba220a0 100644 --- a/doc/CNN_DEBUG.md +++ b/doc/CNN_DEBUG.md @@ -11,7 +11,7 @@ CNN post-processing effect showed black screen when activated at 11.50s, despite **Fix**: Moved capture before loop starts (lines 308-346). Capture now copies `framebuffer_a` to `captured_frame` auxiliary texture ONCE before any post-effects run. ### Bug 2: Missing Uniforms Update ⚠️ CRITICAL -**Location**: `src/gpu/effects/cnn_effect.cc` +**Location**: `src/effects/cnn_effect.cc` **Issue**: `CNNEffect::update_bind_group()` never updated `uniforms_` buffer. `uniforms.resolution` uninitialized (0,0 or garbage) → UV calculation `p.xy / uniforms.resolution` produced NaN → all texture samples black. **Fix**: Added uniforms update before bind group creation (lines 132-142): ```cpp @@ -34,7 +34,7 @@ uniforms_.update(ctx_.queue, u); ## Files Modified - `src/gpu/effect.cc`: Lines 308-346 (capture timing) -- `src/gpu/effects/cnn_effect.cc`: Lines 132-142 (uniforms update) +- `src/effects/cnn_effect.cc`: Lines 132-142 (uniforms update) ## Verification Test: `demo64k --seek 11.5` |