[opus] Patches for adding 120 ms encoding

Jean-Marc Valin jmvalin at jmvalin.ca
Wed Jun 1 17:03:51 UTC 2016


Hi Felicia,

I still don't quite understand why you need to make 120 ms a special
case, rather than extend the code that already handles 40 ms and 60 ms.

Cheers,

	Jean-Marc

On 06/01/2016 12:58 PM, Felicia Lim wrote:
> Hi all, 
> 
> I've just realized that there's a better and simpler way of doing this
> which ensures that analysis and selection of the mode/bandwidth etc is
> done on the whole 120 ms frame. I believe that the logic and threshold
> values for making these decisions are still valid for 120 ms, but I
> might be missing something? Please find my updated patches attached. 
> 
> Thanks, 
> Felicia
> 
> 
> 
> On Tue, May 31, 2016 at 7:05 PM Felicia Lim <flim at google.com
> <mailto:flim at google.com>> wrote:
> 
>     Hi all,
> 
>     We (WebRTC/Google) would like to extend Opus to natively support 120
>     ms encoding instead of relying on repacketization as a post
>     processing step. This is to ensure that a valid 120 ms packet is
>     always available. I've attached a couple of patches to add this to
>     opus_encoder(), based on the internal repacketization process
>     carried out by 60 ms CELT. We intend to extend this later for the
>     multistream encoder as well. The first patch refactors out the
>     internal subframe encoding and repacketizing, and the second patch
>     actually adds the 120 ms support.
> 
>     Any thoughts would be appreciated.
> 
>     Thanks,
>     Felicia
> 
> 
> 
> _______________________________________________
> opus mailing list
> opus at xiph.org
> http://lists.xiph.org/mailman/listinfo/opus
> 


More information about the opus mailing list