<div dir="ltr"><div><div>Just for fun, I tried to reproduce such an overflow. I turned on all debug macros, assertions, and checked arithmetic and then encoded 2 hours of mixed speech/audio with these parameters:<br></div><div><br></div><div>Sample rate = 48000</div><div>Channels = 1</div><div>Application = OPUS_APPLICATION_AUDIO</div><div>Bitrate = 24 KB/s<br></div><div>Force Mode = MODE_SILK_ONLY</div><div>Signal Type = OPUS_SIGNAL_AUTO</div><div>Complexity = 10</div>Frame size = 480 samples (10ms)<br></div><div><br></div><div>No errors came up in encoding. Chandrakala, are these the encoding parameters that you believe should trigger the error?</div><div><br></div><div>- Logan<br></div><div><div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br><br>
We are looking at the VAD implementation used in opus. We are looking at
 the code where speech probability is calculated based on which SNR is 
estimated. Below is the part of the code I am talking about.<br><br>
/*****************************<wbr>****/<br>
/* Speech Probability Estimation */<br>
/*****************************<wbr>****/<br>
SA_Q15 = silk_sigm_Q15( silk_SMULWB( VAD_SNR_FACTOR_Q16, pSNR_dB_Q7 ) - 
VAD_NEGATIVE_OFFSET_Q5 ); // step1: Calculate speech probability : 
comment by me<br><br>
/* Power scaling */<br>
if( speech_nrg <= 0 ) { // step2: update speech probability based on speech energy : comment by me<br>
SA_Q15 = silk_RSHIFT( SA_Q15, 1 );<br>
} else if( speech_nrg < 32768 ) {<br>
if( psEncC->frame_length == 10 * psEncC->fs_kHz ) {<br>
speech_nrg = silk_LSHIFT_SAT32( speech_nrg, 16 ); // Energy is doubled here : comment by me<br>
} else {<br>
speech_nrg = silk_LSHIFT_SAT32( speech_nrg, 15 );<br>
}<br><br>
/* square-root */<br>
speech_nrg = silk_SQRT_APPROX( speech_nrg );<br>
SA_Q15 = silk_SMULWB( 32768 + speech_nrg, SA_Q15 );<br>
}<br><br>
/* Smoothing coefficient */<br>
smooth_coef_Q16 = silk_SMULWB( VAD_SNR_SMOOTH_COEF_Q18, silk_SMULWB( 
(opus_int32)SA_Q15, SA_Q15 ) ); // step3: Update the smoothing factor 
based on speech probability : comment by me<br><br>
if( psEncC->frame_length == 10 * psEncC->fs_kHz ) {<br>
smooth_coef_Q16 >>= 1;<br>
}<br><br>
Here, in step1, Speech probability is calculated whose value is expected
 to be within [0, 1) in Q15 format. Then based on the speech energy 
levels, in Step2, the probability is updated whose value shall also lie 
between [0, 1). Later in Step3, the smooth coeff is calculated. This 
code do not have any issue when the frame size is more than or equal to 
20msec. But, if the frame size is 10ms, then in step2, the energy is 
doubled (this may be done because the original Silk code is for 20ms. To
 convert the energy for 20ms, it could have been doubled). When this is 
done the probability which is updated in step2 becomes more than 1. When
 this is used in multiplication in Step3, the value is treated as a 
negative number because its a 32x16 multiplication. This is will result 
in a negative smooth coefficient. Please let me know if this is a bug.<br><br><br>
Thank you,<br>
Chandrakala</blockquote>








</div></div></div>