Skip to content

[non-BE] Resolve "CNA too high in ISM DTX" JBM addon

eichenseer requested to merge 552-cna-too-high-in-ism-dtx into main

Reason why this change is needed

  • Add missing functionality in JBM path. This was simply an oversight/incompleteness in !870 (merged).

Description of the change

  • The CNG fix CR_FIX_ISM_DTX_INFINITE_CNG_ON_TRAILING_SILENCE is now also included for JBM by copying the missing code parts from ivas_dec() into ivas_jbm_dec_tc(). Original purpose of fix: limit noise energy to noise energy of dominant object. See !870 (merged) for details.

Affected operating points

  • ISM DTX JBM: CNG parts are possibly affected, noise levels are lowered. Depends on test file.
  • Changes only in decoder, encoder not affected.
    1. Produced bitstream is BE compared to previous state.
  • non-BE output if the limitation is triggered.
Edited by eichenseer

Merge request reports

Loading