diff options
| author | skal <pascal.massimino@gmail.com> | 2026-02-08 17:43:44 +0100 |
|---|---|---|
| committer | skal <pascal.massimino@gmail.com> | 2026-02-08 17:43:44 +0100 |
| commit | 4b23fdc63d422b31b6ad86d34218e7b66b462514 (patch) | |
| tree | 028e4fec0d49a7a3a997bd446c9f1ce480ce787b /src/gpu | |
| parent | 8296fe5180b979b9d1f32f6375b41f0e0a8a399d (diff) | |
feat(gpu): Add parameter-driven GaussianBlurEffect
Extends shader parametrization system to GaussianBlurEffect with
strength parameter (Task #73 continued).
Changes:
- Added GaussianBlurParams struct (strength, default: 2.0f)
- Added GaussianBlurUniforms with proper WGSL alignment (32 bytes)
- Updated shader to use parameterized strength instead of hardcoded 2.0
- Extended seq_compiler to parse strength parameter
- Updated demo.seq with 2 parameterized instances:
* Line 38: strength=3.0 (stronger blur for particles)
* Line 48: strength=1.5 (subtle blur)
Technical details:
- Backward-compatible default constructor maintained
- Migrated from raw buffer to UniformBuffer<GaussianBlurUniforms>
- Shader replaces 'let base_size = 2.0' with 'uniforms.strength'
- Generated code creates GaussianBlurParams initialization
Testing:
- All 32/32 tests pass
- Demo runs without errors
- Generated code verified correct
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Diffstat (limited to 'src/gpu')
| -rw-r--r-- | src/gpu/demo_effects.h | 26 | ||||
| -rw-r--r-- | src/gpu/effects/gaussian_blur_effect.cc | 48 |
2 files changed, 60 insertions, 14 deletions
diff --git a/src/gpu/demo_effects.h b/src/gpu/demo_effects.h index 36be118..6c8729d 100644 --- a/src/gpu/demo_effects.h +++ b/src/gpu/demo_effects.h @@ -77,12 +77,38 @@ class ParticleSprayEffect : public Effect { GpuBuffer particles_buffer_; }; +// Parameters for GaussianBlurEffect (set at construction time) +struct GaussianBlurParams { + float strength = 2.0f; // Default: 2.0 pixel blur radius +}; + +// Uniform data sent to GPU shader +struct GaussianBlurUniforms { + float time; // offset 0 + float beat; // offset 4 + float intensity; // offset 8 + float aspect_ratio; // offset 12 + float width; // offset 16 + float height; // offset 20 + float strength; // offset 24 + float _pad; // offset 28 +}; +static_assert(sizeof(GaussianBlurUniforms) == 32, + "GaussianBlurUniforms must be 32 bytes for WGSL alignment"); + class GaussianBlurEffect : public PostProcessEffect { public: + // Backward compatibility constructor (uses default params) GaussianBlurEffect(const GpuContext& ctx); + // New parameterized constructor + GaussianBlurEffect(const GpuContext& ctx, const GaussianBlurParams& params); void render(WGPURenderPassEncoder pass, float time, float beat, float intensity, float aspect_ratio) override; void update_bind_group(WGPUTextureView input_view) override; + + private: + GaussianBlurParams params_; + UniformBuffer<GaussianBlurUniforms> uniforms_; }; class SolarizeEffect : public PostProcessEffect { diff --git a/src/gpu/effects/gaussian_blur_effect.cc b/src/gpu/effects/gaussian_blur_effect.cc index f1736bf..3975aff 100644 --- a/src/gpu/effects/gaussian_blur_effect.cc +++ b/src/gpu/effects/gaussian_blur_effect.cc @@ -1,26 +1,46 @@ // This file is part of the 64k demo project. -// It implements the GaussianBlurEffect. +// It implements the GaussianBlurEffect with parameterization. #include "gpu/demo_effects.h" +#include "gpu/effects/post_process_helper.h" #include "gpu/gpu.h" // --- GaussianBlurEffect --- + +// Backward compatibility constructor (delegates to parameterized constructor) GaussianBlurEffect::GaussianBlurEffect(const GpuContext& ctx) - : PostProcessEffect(ctx) { - uniforms_ = - gpu_create_buffer(ctx_.device, sizeof(float) * 6, - WGPUBufferUsage_Uniform | WGPUBufferUsage_CopyDst); + : GaussianBlurEffect(ctx, GaussianBlurParams{}) { +} + +// Parameterized constructor +GaussianBlurEffect::GaussianBlurEffect(const GpuContext& ctx, + const GaussianBlurParams& params) + : PostProcessEffect(ctx), params_(params) { pipeline_ = create_post_process_pipeline(ctx_.device, ctx_.format, gaussian_blur_shader_wgsl); + uniforms_.init(ctx_.device); } -void GaussianBlurEffect::render(WGPURenderPassEncoder pass, float t, float b, - float i, float a) { - struct { - float t, b, i, a, w, h; - } u = {t, b, i, a, (float)width_, (float)height_}; - wgpuQueueWriteBuffer(ctx_.queue, uniforms_.buffer, 0, &u, sizeof(u)); - PostProcessEffect::render(pass, t, b, i, a); + +void GaussianBlurEffect::render(WGPURenderPassEncoder pass, float time, + float beat, float intensity, + float aspect_ratio) { + // Update uniforms with current state and parameters + const GaussianBlurUniforms u = {.time = time, + .beat = beat, + .intensity = intensity, + .aspect_ratio = aspect_ratio, + .width = (float)width_, + .height = (float)height_, + .strength = params_.strength, + ._pad = 0.0f}; + uniforms_.update(ctx_.queue, u); + + wgpuRenderPassEncoderSetPipeline(pass, pipeline_); + wgpuRenderPassEncoderSetBindGroup(pass, 0, bind_group_, 0, nullptr); + wgpuRenderPassEncoderDraw(pass, 3, 1, 0, 0); } -void GaussianBlurEffect::update_bind_group(WGPUTextureView v) { - pp_update_bind_group(ctx_.device, pipeline_, &bind_group_, v, uniforms_); + +void GaussianBlurEffect::update_bind_group(WGPUTextureView input_view) { + pp_update_bind_group(ctx_.device, pipeline_, &bind_group_, input_view, + uniforms_.get()); } |
