Re: [payload] WGLC for draft-ietf-payload-rtp-vc2hq-01

"Ali C. Begen" <ali.begen@networked.media> Thu, 06 April 2017 02:57 UTC

Return-Path: <ali.begen@networked.media>
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 336BA127286 for <payload@ietfa.amsl.com>; Wed, 5 Apr 2017 19:57:04 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.898
X-Spam-Level:
X-Spam-Status: No, score=-1.898 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=networked-media.20150623.gappssmtp.com
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 kFNfkNrhLy2x for <payload@ietfa.amsl.com>; Wed, 5 Apr 2017 19:57:00 -0700 (PDT)
Received: from mail-wr0-x22c.google.com (mail-wr0-x22c.google.com [IPv6:2a00:1450:400c:c0c::22c]) (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 9B0B2128C83 for <payload@ietf.org>; Wed, 5 Apr 2017 19:56:56 -0700 (PDT)
Received: by mail-wr0-x22c.google.com with SMTP id o21so12818024wrb.2 for <payload@ietf.org>; Wed, 05 Apr 2017 19:56:56 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networked-media.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Y5psIAquFuMNl/MyF7TMdKC42jiuUp/nU8FvNauFF8M=; b=viI5AeVfdEmretKKpjwlzwJr8j8qlQ/SXJjH+ezldFLQUhNRwBdwWk8W8lJt7j7Sb4 eOmiaiZN8SDzrNXfy6WzRrnDcWzMs1Wgpsdd7CdKiuOKgImttp437GUOG1YdtDu2pEif K/ws/xmTD+e+4SFwjG2SnWjeXs5lhXq6VZ30hlzIPPRnD3eW+7+MJ6BfgN44Aafyi4uo 2EglMPFpjXoWDqbsxyJFfl0ml+kokmaCMtPWrWVIGNIF19PuVmQ+Oai3Ild0+3QQssSw OJbgU6702uRi7pmmaZDEKeN4rzkhHxVIuB+fK1P6om+LkD2FhD+Ds5J2Xlst7sl+ZIe2 fqSA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=Y5psIAquFuMNl/MyF7TMdKC42jiuUp/nU8FvNauFF8M=; b=t7QR726kLPIXub7nTEglu1Cg7yVR1lPfmqrDhtLHdHl+uxXB47Fhw5fiknA78ohjGs 1HebuJkaRtb0fWsKKGdfMUY6TET8/IQpYWOWHSZrbdGba8CzbQig64tE8KOnzLH6gelO Gw1EEwWd4CIL7wvVzE8CSXQRLfGAypVyKojH820k4ZiHEfNK6lmoT1W6tkhIZY0pnf0x He8D7KO8cJvqv3BzeOvUNLJl00xgX+bhZ4vvFOO2lW9vxd1OA7G48ba9qJQDARhAvfNb INBmxjDmGmgfXUYzV2P1LpkEloCIGgDwf0+qZ+exEEkUtnQPA876J7S+/0H1ED5Uvo7K YRkg==
X-Gm-Message-State: AFeK/H2ws3edEbRsedq6cBJMu4Lw3pTchHBbBFxPcRtdmDPonPlAFFmvLZVzdSDZTOGz8QyLnwNSEQM5kUoJIA==
X-Received: by 10.223.171.84 with SMTP id r20mr25975975wrc.159.1491447415015; Wed, 05 Apr 2017 19:56:55 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.28.102.138 with HTTP; Wed, 5 Apr 2017 19:56:54 -0700 (PDT)
In-Reply-To: <B1D49063AD5FBD4688F3EEDEC68B2017C37DE099@bgb01xud1011>
References: <CAA4Mczssf9Bt+LoamPpJa0kXuaOfE+n6bJH15GvQdZ3j+ZHjsA@mail.gmail.com> <CAA4McztuCK3RF0YWZtNyMbuWL_SVpdcPCEpzMH6WthPO6jaoWw@mail.gmail.com> <B1D49063AD5FBD4688F3EEDEC68B2017C37DE099@bgb01xud1011>
From: "Ali C. Begen" <ali.begen@networked.media>
Date: Thu, 06 Apr 2017 12:56:54 +1000
Message-ID: <CAA4MczvO_6F70YJi-PLGVLnf9j5d3Gn3tJo4VHBQUn-m3_MPEQ@mail.gmail.com>
To: John Fletcher <John.Fletcher@bbc.co.uk>
Cc: "payload@ietf.org" <payload@ietf.org>
Content-Type: multipart/alternative; boundary="94eb2c1b40da5f9f30054c76a9be"
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/5TIq26gQgrKrauG-i5sIbjtNq4Q>
Subject: Re: [payload] WGLC for draft-ietf-payload-rtp-vc2hq-01
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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, 06 Apr 2017 02:57:04 -0000

Hi John,

On Wed, Apr 5, 2017 at 8:47 PM, John Fletcher <John.Fletcher@bbc.co.uk>
wrote:

> Here are some comments:
>
>
>
> A revised version of the VC-2 specification SMPTE ST 2042-1 has reached
> Draft Publication status and is likely to be published in the next 2 or 3
> months.  Given that section 4.3 of the memo discusses items that are
> changed in the new version of VC-2, I suggest that the memo is updated to
> reflect the new version.  I am not sure of the timescale for progression to
> RFC but I would suggest delaying this if necessary until the SMPTE document
> is published.
>

Depending on the external parties always creates concerns and also some
issues. So, if the final spec due date is that close, I think we better
wait for it and then finish the RTP work. Does anybody object to this?


>  In section 4.1, the description of the Timestamp field says it
> “corresponds to the sampling instant of the coded picture”.  This will not
> be the case if source of the RTP stream is replayed video.  It also may not
> be the case if the input to the RTP encoder is slightly delayed with
> respect to image capture (highly likely) and/or the interface does not
> provide the capture time.  I suggest re-wording this section.
>

How the RTP timestamps is used, it is a bit different story. I see that
Thomas replied to this as well. The payload drafts are flexible in terms of
how they can use the timestamps. So, if there is any requirement from the
SMPTE folks on this, we should use that one in this draft. Is there?


>  Also in section 4.1, description of Timestamp, I suggest adding: “If the
> required timestamp does not correspond to an integer value of the clock,
> e.g. when the frame or field rate is 60000/1001, the value SHALL be
> truncated to the next lowest integer.”
>
>
>
> Section 5, Congestion control, says it SHALL be in accordance with RFC3550
> but all that RFC3550 says is that congestion control SHOULD be defined in
> each RTP profile.  The section also makes reference to an Internet Draft
> and I believe this is not allowed.  Congestion control is not really a
> matter for Payload Format Specifications and I suggest this section is
> deleted.
>

See https://tools.ietf.org/html/draft-ietf-payload-rtp-howto-14#section-7.3

Essentially this section has to exist, what goes in there depends on the
payload format and how it is used (or how it should not be used).

-acbegen


>
> In section 6.1 under “security considerations”, I suggest changing the
> text to “Please see section 8 of this memo".
>
>
>
> In Normative References, RFC3551 is listed but it is not a normative
> reference, it is only referred to as an example.
>
>
>
> In Normative References, as noted earlier, reference to
> I-D.ietf-avtcore-rtp-circuit-breakers is inappropriate and will not be
> needed if the Congestion Control section is removed.
>
>
>
>
>
> Regards,
>
> John Fletcher
>
>
>
> *From:* payload [mailto:payload-bounces@ietf.org] *On Behalf Of *Ali C.
> Begen
> *Sent:* 04 April 2017 15:40
> *To:* payload@ietf.org
> *Subject:* Re: [payload] WGLC for draft-ietf-payload-rtp-vc2hq-01
>
>
>
> Any other comments on this draft? So far, we heard from Jean-Marie and
> Victor supporting the draft. If you have any comments, please send them by
> next Wed (April 12th).
>
>
>
> @Thomas
>
>
>
> I believe you volunteered to review this draft? Could you submit it by
> next week?
>
>
>
> Thanks.
>
> -acbegen (co-chair)
>
>
>
> On Fri, Feb 24, 2017 at 9:02 AM, Ali C. Begen <ali.begen@networked.media>
> wrote:
>
> WG,
>
>
>
> We are starting WGLC for the following draft. Please review the draft and
> send your comments to the list by March 13th. If you read the draft but you
> have no comments, say so as well.
>
>
>
> It is a short document, so should be a quick review.
>
> https://datatracker.ietf.org/doc/draft-ietf-payload-rtp-vc2hq/
>
>
>
> -acbegen
>
>
>
>
>
> ----------------------------
>
> http://www.bbc.co.uk
> This e-mail (and any attachments) is confidential and may contain personal
> views which are not the views of the BBC unless specifically stated.
> If you have received it in error, please delete it from your system.
> Do not use, copy or disclose the information in any way nor act in
> reliance on it and notify the sender immediately.
> Please note that the BBC monitors e-mails sent or received.
> Further communication will signify your consent to this.
>
> ---------------------
>