Re: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt

John Fletcher <John.Fletcher@bbc.co.uk> Sun, 07 January 2018 15:02 UTC

Return-Path: <John.Fletcher@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 6CA04124B18 for <payload@ietfa.amsl.com>; Sun, 7 Jan 2018 07:02:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.211
X-Spam-Level:
X-Spam-Status: No, score=-4.211 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-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 plD3lk16q2BA for <payload@ietfa.amsl.com>; Sun, 7 Jan 2018 07:02:05 -0800 (PST)
Received: from mailout1.cwwtf.bbc.co.uk (mailout1.cwwtf.bbc.co.uk [132.185.160.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 F2D9C12422F for <payload@ietf.org>; Sun, 7 Jan 2018 07:02:04 -0800 (PST)
Received: from BGB01XI1003.national.core.bbc.co.uk ([10.184.50.53]) by mailout1.cwwtf.bbc.co.uk (8.15.2/8.15.2) with ESMTP id w07F20Cu009971; Sun, 7 Jan 2018 15:02:00 GMT
Received: from BGB01XUD1011.national.core.bbc.co.uk ([10.161.14.9]) by BGB01XI1003.national.core.bbc.co.uk ([10.184.50.53]) with mapi id 14.03.0361.001; Sun, 7 Jan 2018 15:01:57 +0000
From: John Fletcher <John.Fletcher@bbc.co.uk>
To: Thomas Edwards <Thomas.Edwards@fox.com>, "payload@ietf.org" <payload@ietf.org>
Thread-Topic: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt
Thread-Index: AQHTddyoPsNc+Sr2Y02pCni/2NRTZ6NGBrmAgAZ7MAWAFoX/gIAAruahgAJwWoCAAn3qKg==
Date: Sun, 07 Jan 2018 15:01:56 +0000
Message-ID: <B1D49063AD5FBD4688F3EEDEC68B2017C39D5706@bgb01xud1011>
References: <151336685958.30439.12106998885650898556@ietfa.amsl.com> <4E1ED364-EACD-48FA-A80E-C029EFCD96C8@networked.media> <B1D49063AD5FBD4688F3EEDEC68B2017C39CF5D2@bgb01xud1011> <BBD61F65-23E2-4E26-B4B8-F86CEF456897@foxeg.com> <B1D49063AD5FBD4688F3EEDEC68B2017C39D1F54@bgb01xud1011>, <4E73ADBA-259B-49F1-ADA8-20BBB87F7390@foxeg.com>
In-Reply-To: <4E73ADBA-259B-49F1-ADA8-20BBB87F7390@foxeg.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [172.19.161.212]
x-exclaimer-md-config: 1cd3ac1c-62e5-43f2-8404-6b688271c769
x-tm-as-product-ver: SMEX-11.0.0.4255-8.200.1013-23578.007
x-tm-as-result: No--15.973800-0.000000-31
x-tm-as-user-approved-sender: Yes
x-tm-as-user-blocked-sender: No
Content-Type: text/plain; charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/x_GvigQXWn9ncPySupwsE6fg38A>
Subject: Re: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt
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: Sun, 07 Jan 2018 15:02:07 -0000

Looks good to me.
John
________________________________________
From: Thomas Edwards [Thomas.Edwards@fox.com]
Sent: 06 January 2018 00:58
To: John Fletcher; payload@ietf.org
Cc: Ali C. Begen
Subject: Re: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt

OK, I am fine with this:

Horizontal_Offset: 12 bits
           This field defines the location of the ANC data packet in an
           SDI raster relative to the start of active video (SAV, a
           digital synchronizing signal present in SDI interfaces) as an
           unsigned integer in network byte order.  A value of 0 means
           that the Ancillary Data Flag (ADF) of the ANC data packet
           begins immediately following SAV.  The horizontal offset
           from SAV is measured in terms of 10-bit words of the
           indicated data stream and data channel.

For line number, I see your point.  Really no point in referencing anything even for SD, how about just:

Line_Number: 11 bits
           This field contains the digital interface line number
           that corresponds to the location of the ANC data
           packet as an unsigned integer in network
           byte order.

-Thomas


On 1/4/18, 4:21 AM, "John Fletcher" <John.Fletcher@bbc.co.uk> wrote:

    That is better but there is still the issue that no definition is given for UHD.  I don't think there is any need for make a distinction for SD/HD/UHD, so the last sentence could be as below (deleting the "For SD..." sentence):

    "The horizontal offset from SAV is measured in terms of 10-bit words of the indicated data stream/data channel."

    I also have a slight issue with the Line_Number definition because it says line number for HD is defined in ITU-R BT.1120 but that does not cover 3G Level A or HFR.  So I would suggest changing the first paragraph to:

    "This field contains the line number (as defined in ITU-R BT.1700 [BT1700] for SD video or the appropriate interface mapping specification for HD or UHD video) that corresponds to the location of the ANC data packet in an SDI raster as an unsigned integer in network byte order."

    I would also move the paragraph beginning "In multi-stream interfaces ..." so that it follows the first paragraph.

    And finally, in the "Note ..." paragraph, I would change "sample structure specification" to "interface mapping specification".

    John
    ________________________________________
    From: Thomas Edwards [Thomas.Edwards@fox.com]
    Sent: 04 January 2018 01:17
    To: John Fletcher; payload@ietf.org
    Cc: Ali C. Begen
    Subject: Re: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt

    draft-ietf-payload-rtp-ancillary-13 currently says for Horizontal_Offset:

    “This field defines the location of the ANC data packet in an SDI raster relative to the start of active video (SAV, a digital synchronizing signal present in SDI interfaces) as an unsigned integer in network byte order.  A value of 0 means that the Ancillary Data Flag (ADF) of the ANC data packet begins immediately following SAV.  For HD, this is in units of Y samples as defined in ITU-R BT.1120 [BT1120]…”

    How about we change the last sentence to:

    “For HD, the horizontal offset from SAV is measured in terms of 10-bit words of the particular channel indicated by the ‘C’ bit.”

    -Thomas


    On 12/20/17, 9:30 AM, "payload on behalf of John Fletcher" <payload-bounces@ietf.org on behalf of John.Fletcher@bbc.co.uk> wrote:

        I think there may still be an issue with the definition of Horizontal_Offset.  No definition is given for UHD and I'm not sure that the HD definition in terms of Y samples is always applicable.

        The ANC data consists of 10-bit words in a 10-bit data stream (or a data channel of a data stream).  The position should be measured in terms of  10-bit words relative to the SAV sequence in the data stream.  A definition along those lines works regardless of the image format (YCbCr, RGB, 10-bit, 12-bit) or the mapping from source image to interface.

        John Fletcher
        ________________________________________
        From: payload [payload-bounces@ietf.org] on behalf of Ali C. Begen [ali.begen@networked.media]
        Sent: 16 December 2017 14:21
        To: payload@ietf.org
        Subject: Re: [payload] I-D Action: draft-ietf-payload-rtp-ancillary-13.txt

        Payload WG,

        Is there any objection to the latest revision that addresses the issues Thomas pointed out earlier? The draft was in the RFC editor queue and if there are no objections, we will soon ship it back there. But to give people a chance to read and officially comment on the revision, I think we should start a new IETF LC for a week.

        @Ben, could we do that?

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