Re: [payload] Review request for JPEG XS RTP payload format I-D

John Fletcher <John.Fletcher@bbc.co.uk> Thu, 15 November 2018 17:44 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 E0BDB130DE0 for <payload@ietfa.amsl.com>; Thu, 15 Nov 2018 09:44:46 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, 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 9ld2g0qZ_8GO for <payload@ietfa.amsl.com>; Thu, 15 Nov 2018 09:44:44 -0800 (PST)
Received: from mailout0.cwwtf.bbc.co.uk (mailout0.cwwtf.bbc.co.uk [132.185.160.179]) (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 34F4B130E6F for <payload@ietf.org>; Thu, 15 Nov 2018 09:44:44 -0800 (PST)
Received: from BGB01XI1010.national.core.bbc.co.uk (bgb01xi1010.national.core.bbc.co.uk [10.161.14.14]) by mailout0.cwwtf.bbc.co.uk (8.15.2/8.15.2) with ESMTP id wAFHigwP028762 for <payload@ietf.org>; Thu, 15 Nov 2018 17:44:42 GMT
Received: from BGB01XUD1011.national.core.bbc.co.uk ([10.161.14.9]) by BGB01XI1010.national.core.bbc.co.uk ([10.161.14.14]) with mapi id 14.03.0408.000; Thu, 15 Nov 2018 17:44:41 +0000
From: John Fletcher <John.Fletcher@bbc.co.uk>
To: "payload@ietf.org" <payload@ietf.org>
Thread-Topic: [payload] Review request for JPEG XS RTP payload format I-D
Thread-Index: AQHUAQtXUOJI6veZdke/6ryfR8+bE6VQhMyAgAF/oMA=
Date: Thu, 15 Nov 2018 17:44:41 +0000
Message-ID: <B1D49063AD5FBD4688F3EEDEC68B2017C3BC61E8@bgb01xud1011>
References: <00f6ebfa-0c81-8ba4-0094-4c97a2ae4e2b@dynmail.crt1.net> <bbb78cf6-48b1-2cea-ba65-38884809082d@dynmail.crt1.net>
In-Reply-To: <bbb78cf6-48b1-2cea-ba65-38884809082d@dynmail.crt1.net>
Accept-Language: en-GB, en-US
Content-Language: en-US
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-12.5.0.1300-8.2.1013-24054.007
x-tm-as-result: No-19.099700-8.000000-10
x-tmase-matchedrid: EMyCvCfVN1HW/bDrA6VrLeRw69tAYXNGwx0jRRxcQfMXPp87POq9xnZ5 WvuLBD7qYOQwyVWOqecoN1YOpZycEm7tzvY/Vhuuhv1+2J3yQFw0AKed0u9fB+ElDsiPTY4oQVl SEo8CA+J6t8+Hp3peGMMR+p3i+7r+fAF8SKkVTnrr/EBmiNuXt3rMPEZwURsKMMn1rcqKQajFSS EUw76he/caQCK4stDEk5yDhWKAyrSjZne6WKfprfYiLRVJ915Dt07/cudGAnsiFs20Vxq/wmrrA GyO72KPMpdM7GeJ31f81SKN5ASt732fQBux6RJb04Rmz/agfdz4qCLIu0mtIJ0j8OKRnVQ8Xkhf j9Xlxy3u92rByc2rbpwkqBm3xAQVIrDBDJBPVs6rw1RrGVJZQe2NkXQNpksJNOnYXKcDRxAtV2J 5bWWmZyvQOJv953kKZ7mbu0iyHsJWMHawB1jWZbU+IyHhkXf1BXJ6j2Ez1Iebkr+En8YSt60ZsW Hm21na+zDEhAx94CLxlOH5e7M5XHjxgHkp9duCcX5PeMxy2v7P60q/FX5hsFG8vLCRjD2po8WMk QWv6iXBcIE78YqRWsR4CqSjYlsD
x-tm-as-user-approved-sender: Yes
x-tm-as-user-blocked-sender: No
x-tmase-result: 10--19.099700-8.000000
x-tmase-version: SMEX-12.5.0.1300-8.2.1013-24054.007
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/CU655alVEouPURAMee43RwiXX8I>
Subject: Re: [payload] Review request for JPEG XS RTP payload format I-D
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.29
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, 15 Nov 2018 17:44:53 -0000

Some comments:

In section 2, the definitions of SOC and EOC seem the wrong way round to me.  I would have expected SOC to correspond to the start and that seems to be what is shown in Figure 3.

In section 4, "Each of those RTP streams is divided into Application Data Units", is it not the JPEG XS stream that is divided into ADUs?

In section 4, "Each ADU is split into packets" is confusing because these are not the RTP packets but the payloads of RTP packets?

In general, I don't find it very clear how the JPEG XS stream is split into payloads for RTP packets.

In section 4.1, the timestamp definition seems to imply that the timestamp is derived from a globally synchronised clock rather than the media clock.  It's not clear what to do if the media clock is not synchronous with a global reference clock.  RFC 3550 states: If RTP packets are generated periodically, the nominal sampling instant as determined from the sampling clock is to be used, not a reading of the system clock.  I think it would be better to omit the text about time since epoch and say instead that if the media clock is derived directly from a globally synchronised reference with origin at the SMPTE Epoch, the offset should be zero (as in RFC 7273 a=mediaclk:direct=0).

Section 6.2, Mapping to SDP, does not say where to put the required and optional parameters, i.e. in the a=fmtp line.  You should probably say that the mapping to SDP shall be in accordance with RFC 4855.

In 6.2.3, it is suggested to use the SMPTE ST 2110-21 TP parameter, i.e. with values such as 2110TPNL.  That is a reasonable thing to do but the draft SMPTE ST 2110-22 uses values such as 2110-22TPNL, the reason being that ST 2110-21 is for uncompressed video and ST 2110-22 is for compressed video and it was thought useful to make a distinction because in -22 (and in this ID) only the Network Compatibility model applies.  Maybe it would have been better for ST 2110-22 just to use 2110TPNL but I think we should aim for consistency between draft ST 2110-22 and this ID.


Regards,
John Fletcher

-----Original Message-----
From: payload <payload-bounces@ietf.org> On Behalf Of Sébastien Lugan
Sent: 14 November 2018 17:57
To: payload@ietf.org
Cc: Alexandre Willème <alexandre.willeme@uclouvain.be>; Gaël Rouvroy <g.rouvroy@intopix.com>
Subject: Re: [payload] Review request for JPEG XS RTP payload format I-D

Dear IETF Payload WG,


We recently uploaded a new version of our RTP payload format I-D for transporting JPEG XS encoded video:

   RTP Payload Format for ISO/IEC 21122 (JPEG XS)
   https://datatracker.ietf.org/doc/draft-lugan-payload-rtp-jpegxs/

Authors:
   S. Lugan, G. Rouvroy, A. Descampe (intoPIX)
   T. Richter (Fraunhofer IIS)
   A. Willeme (Université catholique de Louvain)

Filename: draft-lugan-payload-rtp-jpegxs-01.txt

Abstract:
    This document specifies a Real-Time Transport Protocol (RTP) payload
    format to be used for transporting JPEG XS (ISO/IEC 21122) encoded
    video.  JPEG XS is a low-latency, lightweight image coding system
    allowing for an increased resolution and frame rate, while offering
    visually lossless quality with reduced amount of resources such as
    power and bandwidth.


This new version includes corrections suggested on this list and directly by e-mail.

Might we kindly ask you to review this I-D and send your comments regarding its possible formal adoption by the WG?


Many thanks in advance,

Best regards,

Sébastien Lugan

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