diff options
| author | skal <pascal.massimino@gmail.com> | 2026-02-11 11:58:24 +0100 |
|---|---|---|
| committer | skal <pascal.massimino@gmail.com> | 2026-02-11 11:58:24 +0100 |
| commit | 5ceb0256792053ee80841f2b5ac0dfae1fd4c6f8 (patch) | |
| tree | e24dafeca0d270437a39c8d91c7f5c037cf0d2f4 /src/gpu/effects/cnn_effect.cc | |
| parent | 58c595810d0346f0c2b32dbc70b7385aee545e11 (diff) | |
fix: Complete auxiliary texture initialization fix
Root cause: After swapping init/resize order, effects with Renderer3D crashed
because resize() called before init() tried to use uninitialized GPU resources.
Changes:
- Add guards in FlashCubeEffect::resize() and Hybrid3DEffect::resize() to
check ctx_.device before calling renderer_.resize()
- Remove lazy initialization remnants from CircleMaskEffect and CNNEffect
- Register auxiliary textures directly in init() (width_/height_ already set)
- Remove ensure_texture() methods and texture_initialized_ flags
All 36 tests passing. Demo runs without crashes.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Diffstat (limited to 'src/gpu/effects/cnn_effect.cc')
| -rw-r--r-- | src/gpu/effects/cnn_effect.cc | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/src/gpu/effects/cnn_effect.cc b/src/gpu/effects/cnn_effect.cc index 8673127..b2305b2 100644 --- a/src/gpu/effects/cnn_effect.cc +++ b/src/gpu/effects/cnn_effect.cc @@ -53,6 +53,11 @@ void CNNEffect::init(MainSequence* demo) { demo_ = demo; params_buffer_.init(ctx_.device); + // Register auxiliary texture for layer 0 (width_/height_ set by resize()) + if (layer_index_ == 0) { + demo_->register_auxiliary_texture("captured_frame", width_, height_); + } + // Initialize uniforms BEFORE any bind group creation uniforms_.update(ctx_.queue, get_common_uniforms()); @@ -60,13 +65,6 @@ void CNNEffect::init(MainSequence* demo) { params_buffer_.update(ctx_.queue, params); } -void CNNEffect::ensure_texture() { - if (!texture_initialized_ && layer_index_ == 0 && demo_) { - demo_->register_auxiliary_texture("captured_frame", width_, height_); - texture_initialized_ = true; - } -} - void CNNEffect::resize(int width, int height) { if (width == width_ && height == height_) return; @@ -74,7 +72,7 @@ void CNNEffect::resize(int width, int height) { PostProcessEffect::resize(width, height); // Only layer 0 owns the captured_frame texture - if (layer_index_ == 0 && demo_ && texture_initialized_) { + if (layer_index_ == 0 && demo_) { demo_->resize_auxiliary_texture("captured_frame", width, height); } } @@ -93,8 +91,6 @@ void CNNEffect::render(WGPURenderPassEncoder pass, } void CNNEffect::update_bind_group(WGPUTextureView input_view) { - ensure_texture(); - input_view_ = input_view; // Update common uniforms (CRITICAL for UV calculation!) |
