Seppo Ingalsuo

Results 105 comments of Seppo Ingalsuo

Some results for MFCC component fixed point accuracy with 1s chirp signal test: - MFCC Matlab concept vs. Pytorch reference, ceps RMS error 1e-4 - MFCC component vs. Pytorch reference,...

> @singalsu is the small accuracy delta a result of using 32bit numbers for FW ? The largest difference contribution is from 16 bit FFT and 16 bit Mel band...

> > @singalsu is the small accuracy delta a result of using 32bit numbers for FW ? > > The largest difference contribution is from 16 bit FFT and 16...

To fix this I tried to change to src/audio/Kconfig `select COMP_MODULE_ADAPTER` instead of depends. ``` [ 96%] Linking C executable sof /home/sof/work/xtensa-imx-elf/lib/gcc/xtensa-imx-elf/10.2.0/../../../../xtensa-imx-elf/bin/ld: CMakeFiles/sof.dir/src/audio/mfcc/mfcc.c.o: in function `sys_comp_module_mfcc_interface_init': /home/sof/work/sof.git/src/audio/mfcc/mfcc.c:282: undefined reference to...

Changed vs. previous push, in Kconfig ``` depends on COMP_MODULE_ADAPTER depends on !COMP_LEGACY_INTERFACE ``` that seems to avoid the imx build issue in CI.

Some new library functions need: - linspace - window functions, tbd. (simplest need only trig. functions) - DCT, which type We can use existing FFT

The MFCC output from Matlab/Octave version now fairly well matches pytorch except in since sweep higher frequency part. I'm still finding out where it happens: ![Screenshot from 2022-05-17 16-56-26](https://user-images.githubusercontent.com/28754793/168838304-d37e3a5a-859a-429a-bc44-a4f809423ae9.png)

> > The purpose of this Matlab or Octave script is to draft C implementation of a SOF MFCC generator component. > > Can you elaborate what you mean by...

I just pushed new version that achieves good compatibility with Kaldi and Matlab, and fair compatibility with librosa. The delta-MFCC plots are below for chirp test signal. Librosa needs a...

> no indent. Yes, I've written with both Emacs and Matlab and they use different indent styles, though I've already changed to use tab instead of default small 2 character...