Skip to content
  • Jean-Marc Valin's avatar
    Addressing multiple LSF-related issues · f6e781ab
    Jean-Marc Valin authored
    - Merged the LPC stabilization from NLSF2A_stable.c into NLSF2A.c
    - The bandwidth expansion in NLSF2A() now operates on int32 LPC coefficients in
    Q17 domain (instead of int16 Q12 coefficients)
    - The function bwexpander_32() has a more precise way of updating the chirp
    variable (round to nearest, instead of round down)
    - Changed a few variables in NLSF_stabilize() from int16 to int32 to avoid signed
    wrap-around (no difference in results as the wrap-around would always be reversed
    later)
    - The LSF codebook for WB speech has a quantization stepsize of 0.15 (was 0.16).
    This doesn't break the bitstream, although it slightly limits quality of signals
    encoded with the old version and decoded with the new one (I can't really hear it
    and PESQ gives high scores as well).  I does improve handling of tonal signals.
    - As discussed: the Q-domain of the poly function is now in Q16 (was Q20)
    - As discussed: limiting the LSFs in NLSF_decode() to 0...32767
    - The silk_NLSF_DELTA_MIN values were lowered to deal with a possible future situation with less or no input HP filtering.
    f6e781ab