<p> Attention is currently required from: George Joseph, Kevin Harwell. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/15953">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953/comment/66f77db2_1b53d255">Patch Set #6, Line 15:</a> <code style="font-family:monospace,monospace">ceteris paribus</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Can you _translate_ this 😊 […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953?tab=comments">Patch Set #6:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Couldn't we just create a macro that returns "quality" as a function of sample rate, sample size and […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Could there be codecs that might be similar in those properties but one is still inferior from a quality perspective? Then those should not have the same value.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/codec_builtin.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953/comment/9e1f7c14_b4f96f62">Patch Set #6, Line 623:</a> <code style="font-family:monospace,monospace">        .quality = 40,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">16KHz and only 40?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The qualities assigned are loosely based on these definitions: https://github.com/asterisk/asterisk/blob/master/channels/iax2/format_compatibility.c#L79</p><p style="white-space: pre-wrap; word-wrap: break-word;">This isn't ordering exactly the same property, it's not strictly quality but overall preference based on a few factors.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not familiar with the quality of many of the codecs, such as speex, we could not assign them a quality or somebody that knows the codecs better could suggestion assignments.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953/comment/b23e8cd6_73265b9d">Patch Set #6, Line 889:</a> <code style="font-family:monospace,monospace">   .samples_count = silk_samples,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why do the silks not have quality?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">They didn't appear in chan_iax2 so I left them alone for now, since translation ties are probably likely to occur only with more common codecs (e.g. ulaw/gsm).<br>So I felt it was better to leave them alone for now rather than try to come up with some arbitrary assignment.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15953">change 15953</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/15953"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4b7297e1baca7aac14fe4a3c7538e18e2dbe9fd6 </div>
<div style="display:none"> Gerrit-Change-Number: 15953 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <default.enum@gmail.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <default.enum@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 31 Oct 2022 13:42:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>