Commit cae6da25 authored by vaillancour's avatar vaillancour Committed by Sandesh Venkatesh
Browse files

more correction on constant error, plus one potential saturation was missing

parent 6237fb9c

lib_enc/ivas_stereo_td_analysis.c

+20 −4
Original line number Diff line number Diff line
@@ -262,7 +262,11 @@ Word16 stereo_tdm_ener_analysis_fx( 
        rms_thd_fx = L_shr( rms_thd_fx, 2 ); /*Q16*/ /*rms_thd_fx *= 0.25f*/

        test();

        test();

#ifdef FIX_1301_CORRECT_TD_CNST

        IF( LE_32( hStereoTD->tdm_lt_rms_L_fx, 4915200 /* 75 in Q16*/ ) || LE_32( hStereoTD->tdm_lt_rms_R_fx, 4915200 /* 75 in Q16*/ ) /*|| sts[0]->last_coder_type == TRANSITION */ )

#else

        IF( LE_32( hStereoTD->tdm_lt_rms_L_fx, 4915200 /* 75 in Q16*/ ) || LE_32( hStereoTD->tdm_lt_rms_R_fx, 75 /* 75 in Q16*/ ) /*|| sts[0]->last_coder_type == TRANSITION */ )

#endif

        {

            rms_thd_fx = L_shr( rms_thd_fx, 5 ); /* Q16*/ /*rms_thd_fx *= 0.03125f*/

        }
@@ -336,6 +340,10 @@ Word16 stereo_tdm_ener_analysis_fx( 
        move16();

    }



#ifdef FIX_1301_CORRECT_TD_CNST

    rms_L_fx = L_shl( rms_L_fx, sub( Q16, q_rms_L ) );  /* All the following energy comparison are done in Q16 */

    rms_R_fx = L_shl( rms_R_fx, sub( Q16, q_rms_R ) );

#endif

    test();

    IF( EQ_16( hStereoTD->prev_fr_LRTD_TD_dec, 1 ) && side_can_change == 0 )

    {
@@ -434,7 +442,11 @@ Word16 stereo_tdm_ener_analysis_fx( 
        ELSE

        {

            /*ratio_L = ( 1.0f - cosf( EVS_PI * ratio_L / 2.0f ) ) / 2.0f;*/

#ifdef FIX_1301_CORRECT_TD_CNST

            ratio_L_fx = L_deposit_h( sub_sat( ONE_IN_Q14, getCosWord16( extract_l( Mpy_32_32( 1647099 /* EVS_PI/2 in Q20 */, ratio_L_fx ) ) ) ) ); // Q31 (Q14 + Q1(division by 2.0f) + Q16)

#else

            ratio_L_fx = L_deposit_h( sub( ONE_IN_Q14, getCosWord16( extract_l( Mpy_32_32( 1647099 /* EVS_PI/2 in Q20 */, ratio_L_fx ) ) ) ) ); // Q31 (Q14 + Q1(division by 2.0f) + Q16)

#endif

        }



        test();
@@ -482,7 +494,7 @@ Word16 stereo_tdm_ener_analysis_fx( 
        {

            test();

            test();

            IF( GE_32( hCPE->element_brate, IVAS_48k ) && sts[0]->hVAD->hangover_cnt != 0 && LT_32( L_max( hStereoTD->tdm_lt_rms_L_fx, hStereoTD->tdm_lt_rms_R_fx ), 33554432 /* 512.0f */ ) )

            IF( GE_32( hCPE->element_brate, IVAS_48k ) && sts[0]->hVAD->hangover_cnt != 0 && LT_32( L_max( hStereoTD->tdm_lt_rms_L_fx, hStereoTD->tdm_lt_rms_R_fx ), 33554432 /* 512.0f in Q16*/ ) )

            {

                ratio_L_fx = check_bounds_l( ratio_L_fx, 644245094 /*0.3f in Q31*/, 1503238554 /*0.7f in Q31*/ ); /* Q31 */

            }
@@ -491,7 +503,7 @@ Word16 stereo_tdm_ener_analysis_fx( 
            test();

            test();

            test();

            IF( ( GT_32( hCPE->hStereoTCA->instTargetGain_fx, 644245094 /*1.2f in Q29*/ ) || GT_32( hCPE->hStereoTCA->targetGain_fx, ONE_IN_Q29 ) ) && LT_32( ratio_L_fx, 858993459 /*0.4f*/ ) )

            IF( ( GT_32( hCPE->hStereoTCA->instTargetGain_fx, 644245094 /*1.2f in Q29*/ ) || GT_32( hCPE->hStereoTCA->targetGain_fx, ONE_IN_Q29 ) ) && LT_32( ratio_L_fx, 858993459 /*0.4f  in Q31*/ ) )

            {

                ratio_L_fx = 858993459; /*0.4f in Q31*/

                move32();
@@ -569,7 +581,7 @@ Word16 stereo_tdm_ener_analysis_fx( 
        }

    }



    IF( LT_16( sub( sts[1]->lp_speech_fx, sts[1]->lp_noise_fx ), 12800 /*50.0f*/ ) ) /* likely presence of noisy content */

    IF( LT_16( sub( sts[1]->lp_speech_fx, sts[1]->lp_noise_fx ), 12800 /*50.0f in Q8*/ ) ) /* likely presence of noisy content */

    {

        /* pointing in the right direction, inverse it else do nothing */

        test();
@@ -1063,7 +1075,11 @@ static void NOOP_decision_fx( 
    }

    ELSE

    {

#ifdef FIX_1301_CORRECT_TD_CNST

        if ( LT_32( sts[0]->ee_old_fx, 320000 /* 5000.f in Q6 */ ) && LT_32( sts[1]->ee_old_fx, 320000  /* 5000.f in Q6 */ ) )

#else

        if ( LT_32( sts[0]->ee_old_fx, 160000 /* 5000.f in Q6 */ ) && LT_32( sts[1]->ee_old_fx, 160000 /* 5000.f in Q6 */ ) )  

#endif

        {

            tdm_NOOP_switch_flag = 1;

            move16();