Re: [payload] AD Evaluation: draft-ietf-avt-rtp-isac-04
Harald Alvestrand <harald@alvestrand.no> Thu, 13 June 2013 12:46 UTC
Return-Path: <harald@alvestrand.no>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0EE8421F9A7C for <payload@ietfa.amsl.com>; Thu, 13 Jun 2013 05:46:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -110.598
X-Spam-Level:
X-Spam-Status: No, score=-110.598 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8, USER_IN_WHITELIST=-100]
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 o1Q4v16Owp9F for <payload@ietfa.amsl.com>; Thu, 13 Jun 2013 05:46:30 -0700 (PDT)
Received: from eikenes.alvestrand.no (eikenes.alvestrand.no [158.38.152.233]) by ietfa.amsl.com (Postfix) with ESMTP id 86F6321F9A6A for <payload@ietf.org>; Thu, 13 Jun 2013 05:46:02 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by eikenes.alvestrand.no (Postfix) with ESMTP id C723D39E172; Thu, 13 Jun 2013 14:46:00 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at eikenes.alvestrand.no
Received: from eikenes.alvestrand.no ([127.0.0.1]) by localhost (eikenes.alvestrand.no [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id XCR8Vr2ADzbn; Thu, 13 Jun 2013 14:45:59 +0200 (CEST)
Received: from hta-dell.lul.corp.google.com (unknown [74.125.57.89]) by eikenes.alvestrand.no (Postfix) with ESMTPSA id B309639E058; Thu, 13 Jun 2013 14:45:58 +0200 (CEST)
Message-ID: <51B9BF06.80609@alvestrand.no>
Date: Thu, 13 Jun 2013 14:45:58 +0200
From: Harald Alvestrand <harald@alvestrand.no>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6
MIME-Version: 1.0
To: Richard Barnes <rlb@ipv.sx>
References: <CAL02cgS_0GYw0is8rW5tMhDmsb9A8Mng6oEVwxGks-OdZYi3Gg@mail.gmail.com> <062e01ce2f03$33956840$9ac038c0$@gmail.com> <CAL02cgTdFbOb=A4HbEd=rJAL8wae_fD7yUsAgfHJZ1AuwNFvkg@mail.gmail.com> <065f01ce2f22$26c76b80$74564280$@gmail.com> <CAL02cgQhAevRKTYyzSUcwNvDbgh=zpCy8vAiiD0yXPLip1mofg@mail.gmail.com>
In-Reply-To: <CAL02cgQhAevRKTYyzSUcwNvDbgh=zpCy8vAiiD0yXPLip1mofg@mail.gmail.com>
Content-Type: multipart/alternative; boundary="------------080508010602080103020704"
Cc: draft-ietf-avt-rtp-isac@tools.ietf.org, payload@ietf.org
Subject: Re: [payload] AD Evaluation: draft-ietf-avt-rtp-isac-04
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/payload>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 13 Jun 2013 12:46:35 -0000
On 05/28/2013 08:34 PM, Richard Barnes wrote: > Dear Authors, > > Any update on these issues? Working on it, but as you know from other discussions, there isn't exactly a lack of things to work on at the moment - it dropped off my queue, and I'm picking it up again now. We'll get back to you Real Soon Now. > > Thanks, > --Richard > > > On Mon, Apr 1, 2013 at 5:44 PM, Roni Even <ron.even.tlv@gmail.com > <mailto:ron.even.tlv@gmail.com>> wrote: > > Hi Richard, > > These fields are not part of the RTP header but part of the iSAC > payload header and I agree that it will be better to clarify them. > > Roni > > *From:*Richard Barnes [mailto:rlb@ipv.sx <mailto:rlb@ipv.sx>] > *Sent:* 01 April, 2013 9:47 PM > *To:* Roni Even > *Cc:* payload@ietf.org <mailto:payload@ietf.org>; > draft-ietf-avt-rtp-isac@tools.ietf.org > <mailto:draft-ietf-avt-rtp-isac@tools.ietf.org> > *Subject:* Re: [payload] AD Evaluation: draft-ietf-avt-rtp-isac-04 > > Sure. I didn't mean to imply that we need a reference to the > codec, in fact the opposite. I was looking for this document to > tell me how to parse out the fields I would feed into the black > box of the codec. They describe the fields, but it seems to me > that they need more detail in order for people to know how to > parse them out of the payload. > > --Richard > > On Mon, Apr 1, 2013 at 2:03 PM, Roni Even <ron.even.tlv@gmail.com > <mailto:ron.even.tlv@gmail.com>> wrote: > > Hi Richard, > > As a general comment, for RTP payload specifications we do not > require a normative reference to the codec. We only require enough > information that will allow an RTP receiver to parse the > information and move it to the decoder that can be a black box. > This allows us to publish RTP payload specification also for > codecs whose specifications are not publically available. > > Still in this case more information can be supplied. > > As for the comments the authors should address them > > Roni Even > > *From:*payload-bounces@ietf.org <mailto:payload-bounces@ietf.org> > [mailto:payload-bounces@ietf.org > <mailto:payload-bounces@ietf.org>] *On Behalf Of *Richard Barnes > *Sent:* 01 April, 2013 8:06 PM > *To:* payload@ietf.org <mailto:payload@ietf.org> > *Cc:* draft-ietf-avt-rtp-isac@tools.ietf.org > <mailto:draft-ietf-avt-rtp-isac@tools.ietf.org> > *Subject:* [payload] AD Evaluation: draft-ietf-avt-rtp-isac-04 > > Overall, this document looks in pretty good shape. A couple of > comments I would like to get resolved before IETF LC: > > Section 3.1., "Consult source code for details" > > If the details are only specified in the source code, then the > source needs to be a normative reference. And I would prefer to > avoid that :) Better to specify here how the BEI and FL fields > are encoded. Alternatively, if the codec really does expect a > combined BEI/FL field, you could specify such a field here, opaque > to the RTP layer. However, you would at least need to say how the > recipient knows where this field starts and stops. > > Section 3.2., "The length of the encoded data is variable and > depends on the signal characteristics and the target bit rate." > > However, there's nothing in this section that tells a recipient > how to determine what this length is. > > Section 3.3., "... verifying the CRC checksum ..." > > Please specify which CRC function is to be applied, and how the > CRC value field is formatted. > > Section 3.3., "If this value would exceed 255 at encoding..." > > It sounds like the LEN field is a single octet unsigned integer. > Please state that explicitly. > > Section 9.2. Informative References. > > Better path to source code would be > "webrtc/modules/audio_coding/codecs/isac" > > Thanks, > > --Richard > > > > > _______________________________________________ > payload mailing list > payload@ietf.org > https://www.ietf.org/mailman/listinfo/payload
- [payload] AD Evaluation: draft-ietf-avt-rtp-isac-… Richard Barnes
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Roni Even
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Richard Barnes
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Roni Even
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Richard Barnes
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Harald Alvestrand
- Re: [payload] AD Evaluation: draft-ietf-avt-rtp-i… Richard Barnes