summaryrefslogtreecommitdiff
path: root/src/gpu/effects/cnn_effect.cc
diff options
context:
space:
mode:
authorskal <pascal.massimino@gmail.com>2026-02-11 11:58:24 +0100
committerskal <pascal.massimino@gmail.com>2026-02-11 11:58:24 +0100
commit5ceb0256792053ee80841f2b5ac0dfae1fd4c6f8 (patch)
treee24dafeca0d270437a39c8d91c7f5c037cf0d2f4 /src/gpu/effects/cnn_effect.cc
parent58c595810d0346f0c2b32dbc70b7385aee545e11 (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.cc16
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!)