From 8296fe5180b979b9d1f32f6375b41f0e0a8a399d Mon Sep 17 00:00:00 2001 From: skal Date: Sun, 8 Feb 2026 17:39:33 +0100 Subject: feat(gpu): Add parameter-driven ChromaAberrationEffect MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Implements Task #73 - Extends shader parametrization system to ChromaAberrationEffect following the FlashEffect pattern. Changes: - Added ChromaAberrationParams struct (offset_scale, angle) - Added ChromaUniforms with proper WGSL alignment (32 bytes) - Updated shader to compute offset direction from angle parameter - Extended seq_compiler to parse offset/angle parameters - Updated demo.seq with 2 parameterized instances: * Line 50: offset=0.03 angle=0.785 (45° diagonal, stronger) * Line 76: offset=0.01 angle=1.57 (90° vertical, subtle) Technical details: - Backward-compatible default constructor maintained - Migrated from raw buffer to UniformBuffer - Shader computes direction: vec2(cos(angle), sin(angle)) - Generated code creates ChromaAberrationParams initialization Testing: - All 32/32 tests pass - Demo runs without errors - Binary size: 5.6M stripped (~200-300 bytes impact) Co-Authored-By: Claude Sonnet 4.5 --- src/tests/test_jittered_audio.cc | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'src/tests/test_jittered_audio.cc') diff --git a/src/tests/test_jittered_audio.cc b/src/tests/test_jittered_audio.cc index c1376db..d8260ec 100644 --- a/src/tests/test_jittered_audio.cc +++ b/src/tests/test_jittered_audio.cc @@ -64,8 +64,8 @@ void test_jittered_audio_basic() { // Should have consumed some audio (exact amount depends on timing/jitter) // With minimal sleeps and 0.1s sim time, expect 50-1000 frames - assert(frames_consumed > 50); // At least some audio consumed - assert(frames_consumed < 2000); // Not excessive + assert(frames_consumed > 50); // At least some audio consumed + assert(frames_consumed < 2000); // Not excessive // Underruns are acceptable in this test, but shouldn't be excessive assert(underruns < 5); // Less than 5 underruns in 0.1 seconds @@ -133,10 +133,11 @@ void test_jittered_audio_with_acceleration() { // Should have consumed some audio (exact amount depends on timing/jitter) // With minimal sleeps and 0.6s sim time, expect more than basic test - assert(frames_consumed > 200); // At least some audio consumed - assert(frames_consumed < 5000); // Not excessive + assert(frames_consumed > 200); // At least some audio consumed + assert(frames_consumed < 5000); // Not excessive - // During acceleration with jitter, some underruns are expected but not excessive + // During acceleration with jitter, some underruns are expected but not + // excessive assert(underruns < 10); // Less than 10 underruns in 0.6 seconds printf(" ✓ Jittered audio with acceleration PASSED\n"); -- cgit v1.2.3