speexdsp-rs icon indicating copy to clipboard operation
speexdsp-rs copied to clipboard

Add enough test coverage for the resampler

Open lu-zero opened this issue 2 years ago • 3 comments

Currently the code is not properly tested.

lu-zero avatar May 07 '22 13:05 lu-zero

@lu-zero what is needed to ship 2.0? (I can help add test etc dependent what's blocking)

xnorpx avatar Jun 23 '22 17:06 xnorpx

We need good test to make sure multichannel resampling is working, the current code for the stand-alone reworked resampler is competely bogus in that regard, I planned to rewrite it, but that requires tests first :)

lu-zero avatar Jun 24 '22 07:06 lu-zero

ok, will take a look, written integration test when consuming the crate so maybe I can port some of those down. depending on how much using 2 one-channel resampler is iching :)

xnorpx avatar Jun 24 '22 17:06 xnorpx