Re: [codec] Mirja Kühlewind's No Objection on draft-ietf-codec-opus-update-08: (with COMMENT)

Ben Campbell <ben@nostrum.com> Mon, 14 August 2017 19:30 UTC

Return-Path: <ben@nostrum.com>
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 3593713240E; Mon, 14 Aug 2017 12:30:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.88
X-Spam-Level:
X-Spam-Status: No, score=-1.88 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8BNuUgTbuUWB; Mon, 14 Aug 2017 12:30:31 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 632A8132408; Mon, 14 Aug 2017 12:30:28 -0700 (PDT)
Received: from [10.0.1.63] (cpe-66-25-7-22.tx.res.rr.com [66.25.7.22]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id v7EJUQIo015149 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Mon, 14 Aug 2017 14:30:27 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-66-25-7-22.tx.res.rr.com [66.25.7.22] claimed to be [10.0.1.63]
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: Ben Campbell <ben@nostrum.com>
In-Reply-To: <150272618954.396.2989336786082869936.idtracker@ietfa.amsl.com>
Date: Mon, 14 Aug 2017 14:30:27 -0500
Cc: The IESG <iesg@ietf.org>, tterriberry@mozilla.com, codec-chairs@ietf.org, codec@ietf.org, draft-ietf-codec-opus-update@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <EE4F96D5-E98F-4539-8FA1-2993E893F3C1@nostrum.com>
References: <150272618954.396.2989336786082869936.idtracker@ietfa.amsl.com>
To: Mirja Kühlewind <ietf@kuehlewind.net>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/codec/Xn2xYKl5OZJTBQn0PQQwA8NCpCo>
Subject: Re: [codec] Mirja Kühlewind's No Objection on draft-ietf-codec-opus-update-08: (with COMMENT)
X-BeenThere: codec@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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: Mon, 14 Aug 2017 19:30:33 -0000

> On Aug 14, 2017, at 10:56 AM, Mirja Kühlewind <ietf@kuehlewind.net> wrote:
> 
> Mirja Kühlewind has entered the following ballot position for
> draft-ietf-codec-opus-update-08: No Objection
> 
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
> 
> 
> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> for more information about IESG DISCUSS and COMMENT positions.
> 
> 
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft-ietf-codec-opus-update/
> 
> 
> 
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> While I support the publication as this seems to be the right thing for me to
> do in this situation, I find it really weird that we have to publish an RFC to
> fix bugs in an example implementation in the appendix (that is even encoded).
> While it is a good thing to have code in an RFC as far as this helps
> implementation, maybe rfc6716 went to far with putting a whole reference
> implementation in there. I guess that has also led to a situation where
> everybody is just using the provided code while efforts to reimplement the spec
> could have detected these bugs earlier in the process. Maybe it's an item for
> the IESG to thinking about how to provide a reference implementation with an
> RFC (that maybe can be updated easier) other than just putting it in the
> appendix

The reference implementation in the appendix in RFC 6716 is not just an example implementation. It contains the normative specification of the bit stream decoder—which makes up the majority of the normative bits in that document. 

That has, so far, been a very uncommon approach, and I don’t know that we would do it again. But if we choose to do so, I absolutely agree we should consider whether it can be done in an easier to update fashion. 

Ben.

> 
> One more actionable comment: I think the abstract could say more about the
> updates, especially maybe noticing that this only updates the code in the
> appendix and not the normative language in the body.
> 
>