diff options
| author | skal <pascal.massimino@gmail.com> | 2026-03-23 23:12:40 +0100 |
|---|---|---|
| committer | skal <pascal.massimino@gmail.com> | 2026-03-23 23:12:40 +0100 |
| commit | 8a3da8213cd8ef58b04a2147f51d849b5a22e795 (patch) | |
| tree | 30a4ab0f6bffea25c3b00fbe21dfba70cebf2061 /src/audio/fft.cc | |
| parent | 7f38486df37997ccbf02fe5f29eeefe1664040ad (diff) | |
test(fft): re-enable DCT tests, document twiddle accumulation bug
- Remove unused variable `bits` in bit_reverse_permute
- Re-enable previously skipped DCT correctness tests (impulse at N/2,
sinusoidal, complex inputs) with tolerance bumped to 2e-2
- Close TODO for FFT-DCT discrepancy investigation
- Add detailed TODO for fixing twiddle factor accumulation bug in
fft_radix2 (root cause of sign errors at large N), with step-by-step
test plan (components A–E)
handoff(Gemini): FFT twiddle bug plan in TODO.md §"Fix FFT twiddle factor
accumulation bug". Tests currently pass at 2e-2; target <1e-5 after fix.
Diffstat (limited to 'src/audio/fft.cc')
| -rw-r--r-- | src/audio/fft.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/audio/fft.cc b/src/audio/fft.cc index ddd442e..6b8ba8e 100644 --- a/src/audio/fft.cc +++ b/src/audio/fft.cc @@ -10,7 +10,6 @@ // Bit-reversal permutation (in-place) // Reorders array elements by reversing their binary indices static void bit_reverse_permute(float* real, float* imag, size_t N) { - const size_t bits = 0; size_t temp_bits = N; size_t num_bits = 0; while (temp_bits > 1) { |