Re: [AVTCORE] Benjamin Kaduk's No Objection on draft-ietf-payload-rtp-jpegxs-16: (with COMMENT)

Tim Bruylants <TBR@intopix.com> Thu, 17 June 2021 19:06 UTC

Return-Path: <TBR@intopix.com>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1572D3A2A3A; Thu, 17 Jun 2021 12:06:38 -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, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=intopix.onmicrosoft.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 n05MBpDQpPeP; Thu, 17 Jun 2021 12:06:32 -0700 (PDT)
Received: from dispatch1-eu1.ppe-hosted.com (dispatch1-eu1.ppe-hosted.com [185.183.29.33]) (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 04DDD3A2871; Thu, 17 Jun 2021 12:06:31 -0700 (PDT)
X-Virus-Scanned: Proofpoint Essentials engine
Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05lp2108.outbound.protection.outlook.com [104.47.18.108]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1-eu1.ppe-hosted.com (PPE Hosted ESMTP Server) with ESMTPS id D38A77C0066; Thu, 17 Jun 2021 19:06:29 +0000 (UTC)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fwtF3lpRprv31wTE4q7BHnQli5/WdtTrUww5OFZdyXQJqeHT4bJtWdnoflT/5c3ELeg11aSigjYjymK3nvpZP+TF9HrvnfSBIJH1e+9ci7dGZ6b0RU4Eu8kEu0Qpgv+koTLQtA7rsoGseWLddfbmF/G+d2t1juuVi5Ct0SoeeagZgR93WrHMlca99Q5nqDhzXdKhJxagat3ESyyNXJ0ae4w5D4TDWhL/nKFSgQviKGUiSKTlROsb5l78qy6Yf2XMdpIC7/1bBUtSQt9INVZbpIvugH+hQQPl+8vlJzrUpjESk77UfT7SHmJTUfKjfn6jLCAyxrb8gYE8dLp4rZj8iQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=8MyQyyIKz0uMlD+vlGh4U+In7wNBJ+cvo4TjCRVfo8Q=; b=fZljI/sFa9RlSwVDX56h+x07pqHVjm6wnNmc/y2WJeLXvcyUsP3fM7wz8NNz4j3pYLwyKssa60LACYUgsLYrLODR6WC9GQJvAbmX4diGB2W8eS68rB5ch07ro61yQYo8Em66qJY1ekkHHtVhpd34oUFBMrj2xNJlv/wfMtmmp5bea2VTD09Z4FtPUJ183ZU6TXIGibZAT0cJ/G6TxHug3zbrK1qiVa6nk38d1Sd64Dpq5T1xk3Bf9UI4/VwFY2381Ri7PWbOk84YfGEcnZ5uyBm9ZITy6qRSftMY83tL8a9GdjK/zyFTCarM8DYab5H/eJDm8XTcw2zSffIOoYz82w==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intopix.com; dmarc=pass action=none header.from=intopix.com; dkim=pass header.d=intopix.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intopix.onmicrosoft.com; s=selector1-intopix-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=8MyQyyIKz0uMlD+vlGh4U+In7wNBJ+cvo4TjCRVfo8Q=; b=Ah9YJaiOfDSw6bvwbiOICFXOkfNzJdl527IqTSgICBzoIYwGZFYlmPoKxey5Tp4lF+oYpgT0qOek1Qw6hOIH9W4yj9g8ayvTvx47ew4m52curNUA0aJDUxmnBB9b/dSg//Nv7jM+2aV7fTAwfseF9DPfsW+0yYu39o7gxxUKMJE=
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:4f::24) by PR3P192MB0826.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:49::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4242.19; Thu, 17 Jun 2021 19:06:29 +0000
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::841c:3124:d643:9f40]) by PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::841c:3124:d643:9f40%3]) with mapi id 15.20.4242.021; Thu, 17 Jun 2021 19:06:29 +0000
From: Tim Bruylants <TBR@intopix.com>
To: Benjamin Kaduk <kaduk@mit.edu>, The IESG <iesg@ietf.org>
CC: "draft-ietf-payload-rtp-jpegxs@ietf.org" <draft-ietf-payload-rtp-jpegxs@ietf.org>, "avtcore-chairs@ietf.org" <avtcore-chairs@ietf.org>, "avt@ietf.org" <avt@ietf.org>, Ali Begen <ali.begen@networked.media>, "bernard.aboba@gmail.com" <bernard.aboba@gmail.com>
Thread-Topic: Benjamin Kaduk's No Objection on draft-ietf-payload-rtp-jpegxs-16: (with COMMENT)
Thread-Index: AQHXY0zKH97XyqVT0ke0HUgM2uAtsasYkIcw
Date: Thu, 17 Jun 2021 19:06:29 +0000
Message-ID: <PR3P192MB07483D1BE2863F6E6AACE16BAC0E9@PR3P192MB0748.EURP192.PROD.OUTLOOK.COM>
References: <162391594041.10475.10269870855508874539@ietfa.amsl.com>
In-Reply-To: <162391594041.10475.10269870855508874539@ietfa.amsl.com>
Accept-Language: en-US, nl-NL
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: mit.edu; dkim=none (message not signed) header.d=none;mit.edu; dmarc=none action=none header.from=intopix.com;
x-originating-ip: [2a02:1810:1dbd:e901:c135:70ee:643f:e775]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 06999c33-808d-4382-1a72-08d931c30351
x-ms-traffictypediagnostic: PR3P192MB0826:
x-microsoft-antispam-prvs: <PR3P192MB0826C57D7F81796CBBE1A5C3AC0E9@PR3P192MB0826.EURP192.PROD.OUTLOOK.COM>
x-ms-oob-tlc-oobclassifiers: OLM:4303;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: jTmj6Rw1NvniBfJ0l5ZhVulcwJ3fM77wKU8DB6iKT2OEW7YQIvQfnrxRy5nj1wVTdvbKxcaO1qNnsxp0Y0q22toF6KP4f0SwOJzw61m0IhsasnRPmKIaRUDSZsaF7W9nj4ehFOoNiAVvo7YwpXXlDZh20HQjDzPHJ3bA+4hpjEfGCKTUu3TgYRh/grJbCNh2VYdZT9b535hQQAXRbvBrfmBVQ8RpgXnjJAM58LZTXOU5YGJXtu8UjfN1CXtk3JZ5rDSly7VFanj5sV7lIeainINIi4LRfm0WbdL+dxk2/Na9JlRWvyiPIKoLlT19qQd7VdCPAPLQ28Zpk7hcKO0DvuCpCkfn/xZ9zec8cqwnFP5xwL9ET6xcxxp6vz5lWcj2GYqqyEDgsp5ptDMo2U6BtytzyTW41v4xnubGIiXQwkI23lgS0MUX4dRk8i7pcIlF388Aj14pRaqDnsBOPpuA+JlrpAPRsjOKTFs04w01XNzXSDFp+I7lGrEcXQr5ajDP6bwhNa81Xm68DAvvyhF5j+AUxf9k7s4nKP0Y/cTWUAnEEi1yTjygNouMd7dn0oIUO5pKuJqynMSArKMlpfF4TN87FJiV495eraBjOL1Z5oA39pPuQsrZTmXdHt+FU1S972UvqkcrTWv4qPvzuaTxZ2AIRqXRFceQV6OswnBr9uHN5UCn/pkG//A0n1fwIDKvTaQEjDL8akRmxXz7ZcmnJw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PR3P192MB0748.EURP192.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(376002)(366004)(136003)(39830400003)(346002)(396003)(966005)(478600001)(4326008)(8936002)(83380400001)(8676002)(66574015)(5660300002)(86362001)(9686003)(55016002)(71200400001)(122000001)(2906002)(66446008)(6506007)(38100700002)(110136005)(76116006)(66946007)(316002)(186003)(33656002)(7696005)(66556008)(64756008)(66476007)(52536014)(54906003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: spfMF9/6BrOwWUfzRzqJoFBj/8NPwn4WG+PhkF68YMmgcPSPyhgjWcoD/yAV/u+2g53kzOJpFdy19mDqIKXYdTjfRKmAd9SpU2xWFZspyePCJRgqmml5bEgsUcAQvp1VJsDpvTj49z/W5HebHDaZQs1Co05SecuOeXqzJQiPaVdj/HuvVvD7cM0B8IgqrJPXaIUBWTzkZaEgKRF+oDRd/5ledu+Nib561pksjKBR9ofH3kMpevtVusxm79mhy8ezgLWA01l3zrQFuceAEcUY1aZvSC9nqxSJFoN/blP7BZez+bTH21ohAbzsYVBu58N9gF0P6aqXsljiFAWTkVq8tbRxh/v08V0OV5VnTUrWC7PLPTnyi/I1sNiRVT1XJjhvxaGJ5ywvH1kiPHfRS312D6w61QZSwfHsH4gLLY4Hn7tnl3BG/cHD5KLcCrb0IGwqJldBicBM3UV1bM5s8DvfCbMdyV+9qCZEgNW3c0Py0vC+wMNL3DnrtE/uEgdEIzz9yXqXePnGe/I3yBfn0shQ/i4T1Wdml5t9tEJMD3ZPPMbPs4fQe7VYba57Xh5MtHhzUtjkdu0vSmLPaW4sPsDshoft12cDtPOlbNZrNU4f9UEujECsv/qiVNwd+Exf4G+U/l1UnBjqyPb+C++0Bq831lnTW/tn6Rb8Kt5f9ypFsys2Kr6SQVjZR5bbxKfLaeSshzztAm2lB/87NujKQV3F6TDoOCiLWutakN4yW/AOJGPdy1xJ6+Ia+q8frYM4r2EzpVq/LWLabJ2Loca+A+XZDrERUIOHmcN8wa2nALK2wfx5d2RUKjYBt6ET9GwOl1AioPK+kWZEh9NdwZcLiHlaM4Y8kZ+lofCb9kULOezNpiYjv1/Q2Dzvq21TS+hKSjyeJsnDIRnYIKOW5FVRTxsoWgB2scr1SrEdFT3EPmE9w8S1fsVIZC/4ZJ0W7rFwPPDxCSI/0XKVmj3W9n+1txg0QcmUEPiSSCcGflfB2nFHk6sWN4HmKXVqgxG3y0m2ptrQowaUYGoLw06aWNwAacrP4P6zEINZN3l610blS7UnMyfHo0jg2LNgohNfBWREgcpmdbHlphNs1yHIavFX2vbLuPzclal7pQVlwJ9Mk5CovbifSgu4iGHznxbrLkhBhca7RiFw0sbX07LgM3wvb5PwxXK8y6GOlK4RWE0qW+9gJf5vm/QipARbhVG00GSWg4bZEnIlzKHlbtwqQuO8M4qYr4A1K+odqFW9UzdtUMtfTst7KYRwpK0u9Wg3YvvLkJ+RXR4Ree9qGuYn0oT8oh92aS5mK53KTiygeLgUp6Ik1/+axohpUjhzapT09vUkZv2k/PTtxW2Q9mXJVZUgUZMkRGTs6d6INzN0VeOt7t+kuTYZ61PuPvXckIsSLAhys5jA
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: intopix.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PR3P192MB0748.EURP192.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: 06999c33-808d-4382-1a72-08d931c30351
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Jun 2021 19:06:29.2506 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5f9168c7-cdac-4b23-9509-c278399e3c1f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: zssv2aLtORNbDcE9VEAZ/gJG2CxM6gb426xrlw2bUlRWkEV+VRwVg85ItTM2I1GK
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3P192MB0826
X-MDID: 1623956790-NGfhLA-TctJZ
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/2eYH_feZCV_7naev0HdVyMocDp0>
Subject: Re: [AVTCORE] Benjamin Kaduk's No Objection on draft-ietf-payload-rtp-jpegxs-16: (with COMMENT)
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/avt/>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Jun 2021 19:06:38 -0000

> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
>
> I'll echo the sentiment of other reviewers that the scope of review possible is limited witout access to the underlying ISO specification.
> I further note that in the recent case of https://urldefense.proofpoint.com/v2/url?u=https-3A__datatracker.ietf.org_doc_draft-2Dietf-2Dpayload-2Dvp9_&d=DwICaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lABbJkTi_k8nlRKSDgb6IQxDFAqulQ5BxeXA83yq3SI&m=EUpYAiLAgXylCuqQrCOWa_FavJHgm4h934uh2eTmjF8&s=Pdi60gBr_pxE6ALbEjNia0vvwAW1nUHIjda7-wGs8UE&e= (for which the underlying specification is freely available), there was an error in replicating the chroma subsampling details from the underlying reference to the internet-draft.  Any such errors are undetectable for this draft.

Indeed. See comment of "Francesca Palombini". This is a valid comment. We are unsure how to resolve this. ISO is very strict about distribution of documents outside of the working groups (i.e. it is not allowed at all).

> Section 4.3
>
> Does the value of the T and K bits need to be identical for all packets of a given RTP stream?

Indeed, the T and K bits are not allowed to change in the RTP stream session. We clarified this.

> Section 4.4
>
> It's perhaps needlessly confusing to have the human-readable slice labels in Figures 8 and 9 start at 1 but the SEP counter start at 0.

We chose to start the human counting with 1. The actual algorithm counter values for SEP and P however do indeed start at 0. It would be tricky to change the figures this late in the review process.

> nit: if SLH is an acronym it should be expanded somewhere (it only appears in the figures, at present).

SLH is an XS marker. We added this in Section 2 (without repeating its definition from ISO 21122-1).

> In the slice packetization modes, do we have reasonable guarantees that the JPEG XS header (including all markers and marker segments) will fit into a single RTP packet?

Yes. Slice packetization mode essentially states that each RTP packet contains data from only ONE slice. But slices can be split (at arbitrary boundaries) over multiple RTP packets (and this will almost always happen). Markers and marker segments can only occur in the first RTP packet of a slice (and these are very small -- 6 bytes for the SLH marker).

> Section 7.1
>
>    Applications that use this media type:
>       For example: SMPTE ST 2110, Video over IP, Video conferencing,
>       Broadcast applications.
>
> I think bland declarative statements like "applications that transmit video over RTP" tend to be more common than longer "for example"
> listings, in this type of registration.

Agreed. We changed this to: "Any application that transmits video over RTP (like SMPTE ST 2110)."

> Section 8
>
> nit: s/SPD/SDP in the section heading.

That's a good catch. Fixed it.