Re: [codec] AD review: draft-ietf-codec-opus-11

"Timothy B. Terriberry" <tterribe@xiph.org> Wed, 18 April 2012 20:45 UTC

Return-Path: <tterribe@xiph.org>
X-Original-To: codec@ietfa.amsl.com
Delivered-To: codec@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9211011E8091 for <codec@ietfa.amsl.com>; Wed, 18 Apr 2012 13:45:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id NdFeahMKcvpg for <codec@ietfa.amsl.com>; Wed, 18 Apr 2012 13:44:59 -0700 (PDT)
Received: from dm-mail03.mozilla.org (dm-mail03.mozilla.org [63.245.208.213]) by ietfa.amsl.com (Postfix) with ESMTP id 8414811E80A2 for <codec@ietf.org>; Wed, 18 Apr 2012 13:44:59 -0700 (PDT)
Received: from [172.17.0.5] (c-69-181-137-38.hsd1.ca.comcast.net [69.181.137.38]) (Authenticated sender: tterriberry@mozilla.com) by dm-mail03.mozilla.org (Postfix) with ESMTP id 2A6BC4AED8E; Wed, 18 Apr 2012 13:44:59 -0700 (PDT)
Message-ID: <4F8F27CA.4010800@xiph.org>
Date: Wed, 18 Apr 2012 13:44:58 -0700
From: "Timothy B. Terriberry" <tterribe@xiph.org>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20120113 SeaMonkey/2.4.1
MIME-Version: 1.0
To: Robert Sparks <rjsparks@nostrum.com>
References: <4F8F209F.90505@nostrum.com>
In-Reply-To: <4F8F209F.90505@nostrum.com>
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: codec@ietf.org, codec-chairs@tools.ietf.org, draft-ietf-codec-opus@tools.ietf.org
Subject: Re: [codec] AD review: draft-ietf-codec-opus-11
X-BeenThere: codec@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Codec WG <codec.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/codec>, <mailto:codec-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/codec>
List-Post: <mailto:codec@ietf.org>
List-Help: <mailto:codec-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/codec>, <mailto:codec-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Apr 2012 20:45:03 -0000

Thank you for the detailed review. While we will of course address all 
the comments, I wanted to make one brief point now.

Robert Sparks wrote:
> 22) celt/cwrs.c contains nearly two pages of exposition in a comment,
> complete with references. Why isn't this in the main draft text instead?

The exposition here primarily concerns a specific set of optimizations, 
designed to minimize the required ROM usage at the expense of more 
complex code. These are not the only way to implement these functions. 
For example, the code I posted at 
http://www.ietf.org/mail-archive/web/codec/current/msg02140.html simply 
uses a moderately-sized lookup table, and requires none of these 
explanations. It reimplements the entire decoding half of cwrs.c, 
roughly 300 lines of code, in one function that is less than 20 lines, 
including comments.

The text in the spec tries to limit itself to the simplest mathematical 
definition of the process, for ease of understanding, leaving specific 
optimizations to the implementor. It also references the same paper the 
code does.