[opus] [PATCH] Add Functions to Create Ambisonic Multistream Encoder

Jean-Marc Valin jmvalin at jmvalin.ca
Tue May 17 18:19:24 UTC 2016


Hi Michael,

Email is fine. I've not yet had time to review your patch. It looked
about right, but I need to have a closer look.

Cheers,

	Jean-Marc

On 05/16/2016 05:06 AM, Michael Graczyk wrote:
> Does anyone have more thoughts on this patch? I would like to get this
> in so I can send more involved patches related to ambisonics.
> 
> Also, do you guys prefer to do code review using pull requests on
> github? It seems that both are used for Opus. 
> 
> On Fri, May 6, 2016 at 4:46 PM, Michael Graczyk <mgraczyk at google.com
> <mailto:mgraczyk at google.com>> wrote:
> 
>     Here is the modified patch. I added a flag to configure.ac
>     <http://configure.ac> which is set to 0 to disable ambisonics, and 1
>     to enable it. Right now the implementation simply creates a surround
>     encoder with N uncoupled streams.
> 
> 
>     Thanks,
>     Michael Graczyk
> 
> 
> 
> 
> -- 
> 
> Thanks,
> Michael Graczyk


More information about the opus mailing list