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

James Barrett <James.Barrett@bbc.co.uk> Wed, 05 April 2017 09:42 UTC

Return-Path: <James.Barrett@bbc.co.uk>
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 8165E129418 for <payload@ietfa.amsl.com>; Wed, 5 Apr 2017 02:42:26 -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 z0mD3TMvBaXO for <payload@ietfa.amsl.com>; Wed, 5 Apr 2017 02:42:24 -0700 (PDT)
Received: from mailout1.telhc.bbc.co.uk (mailout1.telhc.bbc.co.uk [132.185.161.180]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5AA031201F8 for <payload@ietf.org>; Wed, 5 Apr 2017 02:42:24 -0700 (PDT)
Received: from BGB01XI1002.national.core.bbc.co.uk ([10.184.50.52]) by mailout1.telhc.bbc.co.uk (8.15.2/8.15.2) with ESMTP id v359gH9r027458; Wed, 5 Apr 2017 10:42:17 +0100 (BST)
Received: from BGB01XUD1007.national.core.bbc.co.uk ([10.161.14.5]) by BGB01XI1002.national.core.bbc.co.uk ([10.184.50.52]) with mapi id 14.03.0319.002; Wed, 5 Apr 2017 10:42:17 +0100
From: James Barrett <James.Barrett@bbc.co.uk>
To: Thomas Edwards <Thomas.Edwards@fox.com>
CC: "payload@ietf.org" <payload@ietf.org>
Thread-Topic: [payload] WGLC for draft-ietf-payload-rtp-vc2hq-01
Thread-Index: AQHSraNha8BUgcy1iEG0Jl3I77IB1KG2dcwA
Date: Wed, 05 Apr 2017 09:42:16 +0000
Message-ID: <AC254B17-6701-4F48-A391-7842161A2DAD@bbc.co.uk>
References: <CAA4Mczssf9Bt+LoamPpJa0kXuaOfE+n6bJH15GvQdZ3j+ZHjsA@mail.gmail.com> <CAA4McztuCK3RF0YWZtNyMbuWL_SVpdcPCEpzMH6WthPO6jaoWw@mail.gmail.com> <46E06A93-954A-46C2-8884-CDB08B63AAED@fox.com>
In-Reply-To: <46E06A93-954A-46C2-8884-CDB08B63AAED@fox.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.10.48.249]
x-exclaimer-md-config: 1cd3ac1c-62e5-43f2-8404-6b688271c769
x-tm-as-product-ver: SMEX-11.0.0.4179-8.100.1062-22986.005
x-tm-as-result: No--21.487200-0.000000-31
x-tm-as-user-approved-sender: Yes
x-tm-as-user-blocked-sender: No
Content-Type: text/plain; charset="utf-8"
Content-ID: <01F78BD3ED20094889180425F469E234@bbc.co.uk>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/yTfdc0sAEWPKMDjgXphu17P2IHk>
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: Wed, 05 Apr 2017 09:42:26 -0000

> SMPTE ST 2042-1 does define the structure of Auxiliary Data and Padding.  While this payload definition Auxiliary Data and Padding should be able to be carried in this payload.

I will work up a revised version with a mechanism for carrying Auxiliary Data which can also be used for padding, but I suggest that we recommend that padding need not be sent.

> Suggest expand “PC so it becomes  ‘the “PC” (Parse Code) field’ as this is first use of “PC”.
>
> Might want to be clear that the “F” bit doesn’t apply to progressive pictures.


Makes sense.

> I am concerned about the bit limitation on slice_prefix_bytes and slice_size_scalar.  While in practice 16 appropriate.

I understand your concern, but it is highly unlikely that either of these values will ever even approach the 16-bit limit, and a size/value pair could make parsing in hardware more complex.

> It scares me a bit that this draft is defining new parse codes, as opposed to using a fragmentation flag (or fragment number) in the payload header.  It may be confusing to implementers.

If it helps these parse codes match ones being added to the next iteration of the VC-2 specification currently being considered by SMPTE to allow fragmentation of pictures within a VC-2 stream.

--
James P. Weaver (né Barrett)
Research Engineer
BBC R&D North Lab,
Floor 5 Dock House, MediaCity,
M50 2LH
Tel: +44(0)30 3040-9521
e-mail: james.barrett@bbc.co.uk

> On 5 Apr 2017, at 01:26, Thomas Edwards <Thomas.Edwards@fox.com> wrote:
>
> I will preface this with the fact that I am NOT a VC-2 expert.  But I am very glad to see this I-D making progress.  So here we go:
>
> Comments on draft-ietf-payload-rtp-vc2hq-01 from Thomas Edwards:
>  “At time of writing there is currently no definition for the use of
>    Auxiliary Data in VC-2, and Padding Data is required to be ignored by
>    all receivers.”
>
> “  Since there is no definition for the use of Auxiliary Data Units and
>    Padding Data Units are defined by the VC-2 spec to be ignored by all
>    decoders this specification only covers the transport of Sequence
>    Headers, High Quality Pictures, and (optionally) End of Sequence
>    headers.”
>
> “These four packet-types can be distinguished by the fact that they
>    use different codes in the "PC" field,”
>
>
>
>
>
> -Thomas
>
> --
> Thomas Edwards
> VP Engineering & Development
> FOX Networks Engineering and Operations
> thomas.edwards@fox.com
> Phone: +1.310.369.6696
> 10201 West Pico Blvd.
> Los Angeles, CA 90035
>
> From: "Ali C. Begen" <ali.begen@networked.media>
> Date: Tuesday, April 4, 2017 at 7:40 AM
> To: "payload@ietf.org" <payload@ietf.org>
> Cc: Thomas Edwards <Thomas.Edwards@fox.com>
> Subject: Re: 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. 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 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
>
> _______________________________________________
> payload mailing list
> payload@ietf.org
> https://www.ietf.org/mailman/listinfo/payload



-----------------------------
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.
-----------------------------