search for: chandrakala

Hi, Can anyone let me know if this is a bug? Thank you, Chandrakala ----- Original Message ----- From: "Logan Stromberg" <loganstromberg at gmail.com> To: "Chandrakala Madhira" <chandrakala.madhira at soctronics.com> Cc: opus at xiph.org Sent: Wednesday, November 22, 2017 12:12:39 PM Subject: Re: [opus] Reg an issue with smoot...

Hi Chandrakala, Logan, Can you confirm that the attached patch fixes the overflow problem? Koen, can you confirm the fix makes sense? Cheers, Jean-Marc On 11/27/2017 12:10 PM, Logan Stromberg wrote: > Sorry, long holiday weekend in America. > I can say with pretty high certainty that there is an overf...

...c and then encoded 2 hours of mixed speech/audio with these parameters: Sample rate = 48000 Channels = 1 Application = OPUS_APPLICATION_AUDIO Bitrate = 24 KB/s Force Mode = MODE_SILK_ONLY Signal Type = OPUS_SIGNAL_AUTO Complexity = 10 Frame size = 480 samples (10ms) No errors came up in encoding. Chandrakala, are these the encoding parameters that you believe should trigger the error? - Logan Hi, > > 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 a...

...MULWB(32768 + speech_nrg, SA_Q15); I can't speak for the logic where the speech energy gets doubled. It obviously seems intentional but I don't know why. Maybe so that smoothing is performed at a constant rate regardless of whether framesize is 10 or 20ms? On Sun, Nov 26, 2017 at 8:07 PM, Chandrakala Madhira < chandrakala.madhira at soctronics.com> wrote: > Hi, > > Can anyone let me know if this is a bug? > > Thank you, > Chandrakala > > ------------------------------ > *From: *"Logan Stromberg" <loganstromberg at gmail.com> > *To: *"Cha...

Yes, yes, I can reproduce it now, but only on platforms that define a 16-bit int by default (SA_Q15 is an opus_int rather than opus_int32). What system are you compiling this for? On Tue, Nov 21, 2017 at 8:34 PM, Chandrakala Madhira < chandrakala.madhira at soctronics.com> wrote: > Hi Logan, > > Please find attached the input stream we are using testing. > > Thank you, > Chandrakala > > ------------------------------ > *From: *"Chandrakala Madhira" <chandrakala.madhira at...

...s 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. Thank you, Chandrakala -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xiph.org/pipermail/opus/attachments/20171120/8fcd19c3/attachment.html>