[non-BE] Resolve "LTV regression from #2507: missing signal"

Manuel Janderrequested to merge
basop-2540-ltv-regression-from-2507-missing-signal into main

Reason why this change is needed

  • Porting mistake in SQ_gain_estimate().

Description of the change

Fix condition

/* avoid SV with 1 bin of amp < 0.5f */
if ( tmp > 0.3f )
{
    ener += tmp;

    /* if SNR is above target -> break and increase offset */
    if ( ener > target )
    {
        offset += fac;
        break;
    }
}

which was wrongly split up in

/* avoid SV with 1 bin of amp < 0.5f */


if ( tmp > 0.3f )
{
    ener += tmp;

}

/* if SNR is above target -> break and increase offset */


if ( ener > target )
{
    offset += fac;
    break;
}

Affected operating points

  • All operating points using MDCT Stereo CPE
  • Observed improvements on STVs

Closes #2540 (closed)

Edited by multrus

Merge request reports