sdatkinson / AudioDSPTools

A library of basic audio DSP tools
MIT License
27 stars 5 forks source link

LanczosResampler's hard-coded buffer sizes can get into trouble. #8

Closed sdatkinson closed 5 months ago

sdatkinson commented 5 months ago

A DAW buffer size of 4096 will get into trouble trying to use these when resampling. Here's the offender: https://github.com/sdatkinson/AudioDSPTools/blob/b62f49cb03cdb9d20f47dd166a8cf8ecb6f0dd70/dsp/ResamplingContainer/Dependencies/LanczosResampler.h#L135

Short-term fix: bump it up for the most extreme case I can foresee (44.1 -> 192k with buffer size 8192). Longer-term, fix the code.