IETF Logo Mail Archive

Re: [Gen-art] Genart last call review of draft-ietf-codec-opus-update-08

Paul Kyzivat <pkyzivat@alum.mit.edu> Tue, 01 August 2017 19:40 UTC

Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6D00C1322D3 for <gen-art@ietfa.amsl.com>; Tue, 1 Aug 2017 12:40:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.202
X-Spam-Level:
X-Spam-Status: No, score=-4.202 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001] 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 xH2N-wLqLH8K for <gen-art@ietfa.amsl.com>; Tue, 1 Aug 2017 12:40:57 -0700 (PDT)
Received: from alum-mailsec-scanner-1.mit.edu (alum-mailsec-scanner-1.mit.edu [18.7.68.12]) by ietfa.amsl.com (Postfix) with ESMTP id 66D6F12EB5D for <gen-art@ietf.org>; Tue, 1 Aug 2017 12:40:54 -0700 (PDT)
X-AuditID: 1207440c-c4bff70000000b4f-81-5980d94515a4
Received: from outgoing-alum.mit.edu (OUTGOING-ALUM.MIT.EDU [18.7.68.33]) (using TLS with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by alum-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP id 84.5E.02895.549D0895; Tue, 1 Aug 2017 15:40:53 -0400 (EDT)
Received: from PaulKyzivatsMBP.localdomain (c-24-62-227-142.hsd1.ma.comcast.net [24.62.227.142]) (authenticated bits=0) (User authenticated as pkyzivat@ALUM.MIT.EDU) by outgoing-alum.mit.edu (8.13.8/8.12.4) with ESMTP id v71Jep8Z020681 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT) for <gen-art@ietf.org>; Tue, 1 Aug 2017 15:40:53 -0400
To: gen-art@ietf.org
References: <150161082225.9505.16606445945798880852@ietfa.amsl.com> <7a13a147-5be7-f15b-2d93-88b0922ef543@jmvalin.ca>
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Message-ID: <a0df0672-77ea-cd80-dd3e-3c0e18d09187@alum.mit.edu>
Date: Tue, 01 Aug 2017 15:40:51 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <7a13a147-5be7-f15b-2d93-88b0922ef543@jmvalin.ca>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrPIsWRmVeSWpSXmKPExsUixO6iqOt6syHSoKVL3uLqq88sDoweS5b8 ZApgjOKySUnNySxLLdK3S+DKmNt/mamgU6Ti/cE1LA2MUwS6GDk5JARMJJ53vGHvYuTiEBLY wSSx4sMfKOc7k8Tb6ZeZQKqEBYIlbk57yAhiiwiISLzf18sOYgsJlEu0TugCq2ET0JKYc+g/ C4jNK2AvcaNtOVicRUBFYv6vFcwgtqhAmsSM79eZIWoEJU7OfAJWzylgJzFl70FWEJtZwExi 3uaHzBC2uMStJ/OZIGx5ie1v5zBPYOSfhaR9FpKWWUhaZiFpWcDIsopRLjGnNFc3NzEzpzg1 Wbc4OTEvL7VI11AvN7NELzWldBMjJCx5djB+WydziFGAg1GJh5ejvSFSiDWxrLgy9xCjJAeT kiivYk99pBBfUn5KZUZicUZ8UWlOavEhRgkOZiUR3jfXgcp5UxIrq1KL8mFS0hwsSuK8qkvU /YQE0hNLUrNTUwtSi2CyMhwcShK8e0AaBYtS01Mr0jJzShDSTBycIMN5gIbfuAoyvLggMbc4 Mx0if4pRl+PXzK1fmIRY8vLzUqXEeU1uABUJgBRllObBzYGlk1eM4kBvCfOaglTxAFMR3KRX QEuYgJZIltaCLClJREhJNTB2rSywsWfRuPg+61Kb85yVcjdCluT5mBg3T+q7J/Dzwoc5f7bx vhNIPqkiG7lzKct5+5U+06Slbbb1b7geGBotMLHn4pIvlRtvfBZzCRHfufHY6h2LtI5M+PZ4 eWz1+YZZJtVC05fInON/8DnUbIo0z/an8d/Xr/6Wa53Lf7/3aez86zf81p2VVGIpzkg01GIu Kk4EALZjtmgCAwAA
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/6BNBoc6p-drBH9XYESBT6aOjEGs>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-codec-opus-update-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 01 Aug 2017 19:40:58 -0000

On 8/1/17 3:08 PM, Jean-Marc Valin wrote:
> Hi Robert,
> 
> Thanks for the review. The main issue with including uuencoded
> testvectors in the document is that it would cause the RFC to be around
> 32000 pages long. That's why RFC6716 included the source code, but not
> the test vectors. Back then, we were told that the meeting material was
> the only place we could put the test vectors. I don't really like doing
> that, but it still beats the other options I'm aware of (unless I missed
> something).

Sounds like a requirement for storage of supporting "documents" that are 
normatively referenced by an RFC.

	Thanks,
	Paul

> Cheers,
> 
> 	Jean-Marc
> 
> On 01/08/17 02:07 PM, Robert Sparks wrote:
>> Reviewer: Robert Sparks
>> Review result: Ready
>>
>> I am the assigned Gen-ART reviewer for this draft. The General Area
>> Review Team (Gen-ART) reviews all IETF documents being processed
>> by the IESG for the IETF Chair.  Please treat these comments just
>> like any other last call comments.
>>
>> For more information, please see the FAQ at
>>
>> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>>
>> Document: draft-ietf-codec-opus-update-08
>> Reviewer: Robert Sparks
>> Review Date: 2017-08-01
>> IETF LC End Date: 2017-08-09
>> IESG Telechat date: Not scheduled for a telechat
>>
>> Summary: Ready for publication as a Standards Track RFC
>>
>> This document is straightforward in the changes it is making to OPUS.
>>
>> My only note of sadness is that it continues to use a documentation mechanism
>> started by RFC6716 of effectively making a normative reference to the
>> _proceedings_ of previous IETF meetings. (Note that this document does this
>> twice: once for the patch file, which is a convenience - the information is in
>> the draft, and once for the updated test vectors. This is _not_ a convenience,
>> the information is not in the draft. If, for whatever reason, the proceedings
>> URL could not be retrieved, someone could not verify their implementation with
>> the updated test vectors).
>>
>> On the one hand, we've set the precedent, and we could agree to just let this
>> go (I'm recommending that to the IESG with this review). On the other hand, we
>> could make things _slightly_ better (or perhaps just different) by putting the
>> test vectors in the doc as an appendix as a uuencoded compressed tarball.
>>
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art
>

v2.23.0 | Report a Bug | By Email