<div dir="ltr">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.<br><br>Also, do you guys prefer to do code review using pull requests on github? It seems that both are used for Opus. </div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 6, 2016 at 4:46 PM, Michael Graczyk <span dir="ltr"><<a href="mailto:mgraczyk@google.com" target="_blank">mgraczyk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">Here is the modified patch. I added a flag to <a href="http://configure.ac" target="_blank">configure.ac</a> 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.<br clear="all"><div><br></div><div><div dir="ltr"><br><div>Thanks,</div><div>Michael Graczyk</div></div></div>
</div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><br><div>Thanks,</div><div>Michael Graczyk</div></div></div>
</div>