Re: [AVTCORE] Genart last call review of draft-ietf-payload-rtp-jpegxs-12

Tim Bruylants <TBR@intopix.com> Fri, 07 May 2021 12:59 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 68B983A2026; Fri, 7 May 2021 05:59:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.619
X-Spam-Level:
X-Spam-Status: No, score=-2.619 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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 WQPnmvd45dzg; Fri, 7 May 2021 05:59:43 -0700 (PDT)
Received: from dispatchb-eu1.ppe-hosted.com (dispatchb-eu1.ppe-hosted.com [185.132.181.11]) (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 0A0693A2039; Fri, 7 May 2021 05:59:39 -0700 (PDT)
X-Virus-Scanned: Proofpoint Essentials engine
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04lp2054.outbound.protection.outlook.com [104.47.13.54]) (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 95BAE9C007F; Fri, 7 May 2021 12:59:36 +0000 (UTC)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WAFhnimEl5EBa0DMD1TWqVWkqCnhPuYxXupbfKpcUx8bbAhxH9ZK0ijJI4F8Wgfun9Fa9s5CyE27iR+mN8Eep+R1VlkGv/YL8DlUDoL7iF8vZa4XVd8sNAhWj38GegZRs/VhP30Zm71yp9I9tyZ8WNS3IKzws860guEFo0Se+EtJUf9Sz0wZDqGFA/X82NZc738rqq7zmH342vCI5zw60afxTTzyvt6CZW3S0BW6Gi911eecgvP1i0AyNgqP666DTi4JFMRZ6ZlBj3ayKo9yh8A/nxy0sxQiYSp7AXXcYXmAREG9hifmpxzfw5MriPipd3JNxCv+p38Yb9GYCwoAKw==
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=l0ECw3uKfaZU7WWRRN5LcuUzZZe0QlmIZW+Gmvtd7cE=; b=WvWEWJdm6KNnceAnwpJf93X/IEK/SMjpIUUtrDVS0cE7Hx3cojAN9n0nA952BVmLM6SnKHKMBKuZHhZvaWbu5yLX16gGHdh5r0YocNgL3BwyVe2pokU7qa+4GHyiU7E3hx975Ze5YNBXRhfUnTbiq1cuzYfc8vQ8Wv7N+zl84b+t4ZGE5eRYmRs6+B4Fb0moxtnXCXpHYAKi4X5aaUp5x+5GVtYNF9awR7x8chyDeSDcTh3o/MAIXo3Myn/F4mYfqmPU3/BXwLs8e56YV6VRHeYKigz/5faH1tE0qR4pH+Ti/phKFLVYBRRAc79YUXE/KZmrQxJmp2IVBOjYxgSqWA==
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=l0ECw3uKfaZU7WWRRN5LcuUzZZe0QlmIZW+Gmvtd7cE=; b=VgB043czZy1wGq1x4jNjjsoc61hg+UAteQ2KZ4DpdeuhCv/j9B/4v7eBg6JAv2AGIxewFLmZiL/6682LC/Ip/NrgkDyqIYfn1huTYvaRMLXReXQxr0M+/7pA40ykM8CowAQ4n0In0IuzlRuHUHqbVpZjeNyyKLdl5ZaW+23njGk=
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:4f::24) by PAXP192MB1167.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:1a0::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.26; Fri, 7 May 2021 12:59:35 +0000
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::f598:e21e:5658:281f]) by PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::f598:e21e:5658:281f%5]) with mapi id 15.20.4108.029; Fri, 7 May 2021 12:59:35 +0000
From: Tim Bruylants <TBR@intopix.com>
To: Russ Housley <housley@vigilsec.com>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "last-call@ietf.org" <last-call@ietf.org>, "avt@ietf.org" <avt@ietf.org>, "draft-ietf-payload-rtp-jpegxs.all@ietf.org" <draft-ietf-payload-rtp-jpegxs.all@ietf.org>
Thread-Topic: [AVTCORE] Genart last call review of draft-ietf-payload-rtp-jpegxs-12
Thread-Index: AQHXQqAhhZJS9EYjKEeou2Vs8Z5+zqrX+3iQ
Date: Fri, 7 May 2021 12:59:35 +0000
Message-ID: <PR3P192MB074899DD70BC031D7D90FB15AC579@PR3P192MB0748.EURP192.PROD.OUTLOOK.COM>
References: <162032334274.10241.3586337784439040714@ietfa.amsl.com>
In-Reply-To: <162032334274.10241.3586337784439040714@ietfa.amsl.com>
Accept-Language: en-US, nl-NL
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: vigilsec.com; dkim=none (message not signed) header.d=none;vigilsec.com; dmarc=none action=none header.from=intopix.com;
x-originating-ip: [2a02:1810:1dbd:e901:989b:8cd0:4643:b603]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 46c6baf8-67a1-479c-1610-08d91157f6f9
x-ms-traffictypediagnostic: PAXP192MB1167:
x-microsoft-antispam-prvs: <PAXP192MB1167C38F35318145D8459B79AC579@PAXP192MB1167.EURP192.PROD.OUTLOOK.COM>
x-ms-oob-tlc-oobclassifiers: OLM:5236;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: sY46EoFKlWl30hi9xCr/iVcGi3IBB2lhgpX/W4IjuwXjoTAYnY4ODxpQsLamevveIUvh83miXHVvaS+u2i2eo2gQIX9VtgDDCYCTD0qyb5uSFFWt8HUVYusGrGCKDqpJeee9nlB149F/65I6WBg74plsmp42gfg0DDOfXaF2jUX43jWW6B/hmrYL4M9GB7xagyST/+URv2AiQZyCjfuL1iKFc6fMksaj1j/+lkjkWQ2WbxuXcDWuXaL/rFLSsjj098ibmflw1WlG95UbRv1pnFgJW0ExB0GmZYxGWioHYC87i2DMpPJuwd2IjUOjzq87rVhQNxwzqn40qChzv8T+gwss3POooqP4KIaXc2tJNUnuGhyDt+EEE38+At5owvgWkIATMu/okTKGIy2Pl/rMOoPZpG4wzwdqUEDYSM1zAPsYp09JQfzumjZi+xToPSfmaOBG7yRed1XxARiip8kjvFqcM/dpevaihPzKCnijQGoSZbxGsRTJnznpip2uBA1eo7ZIoe8jMk4gGoWydWkyE9n+yqbpbSKUEXTIJyaiAzFKdLXdqUtGjhhM4XRClpOfFumwWndciIyLiwJv0jAk/mqMtWPzYFZg+ejGHucOxlezePgvDWdZS2Un3FaoDSRE33C2k0RSX+gj0GyyeX4/ykB5/aIeCu7OlbZZSASHyV1jl7qgc9hWTinJPmd7w8/x
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:(39830400003)(376002)(396003)(346002)(366004)(136003)(66556008)(66446008)(64756008)(76116006)(66946007)(4326008)(122000001)(8676002)(71200400001)(66476007)(5660300002)(33656002)(38100700002)(6506007)(478600001)(55016002)(52536014)(7696005)(966005)(110136005)(316002)(54906003)(8936002)(53546011)(186003)(2906002)(86362001)(9686003)(83380400001); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?us-ascii?Q?pPBiptWvimq9yQxOcmKs6M1x93LAwYqIKubSVo4qmJBud6VZm+/whsM2RuG/?= =?us-ascii?Q?49QiiSxOf+joofLyViR12DeSkRufJdQFfTVOSxtNcKmbA0qbDfakRgWfCPzI?= =?us-ascii?Q?00gbjDDaddXvzY+2O7gVFS2gGHghYZIsx2OU3H35/gcQAWZHFhkTMk51yS4H?= =?us-ascii?Q?oZnI6strynEx8QKXdksnmqNUmnvjCaJ8V8Wk3y8e4Uo+oTo0xWewZQkSfMs0?= =?us-ascii?Q?M5+sqMv8ep/FQ9tJYt8+Gj3kDvXVlzIRpS130NjgDDDZk+U7YEbnwG0Fi+mk?= =?us-ascii?Q?nsQLCZR3AQwb9vXBuM3KYCuZPUZBa6xIFelD7h08rGSpNT6owmiPt7iQpcJx?= =?us-ascii?Q?SZjFJ8w5C0eoJB49/xkwgRERMo1JIhYJ41QbkQzIGdBRuGKdeVn4laSzXVLa?= =?us-ascii?Q?Pmx3qH6H/U0yROfBBZpBc36aoTs2dHwKJ6Hwi5HsZ3fbbcZuu7Awqjf2tgQY?= =?us-ascii?Q?9kFPg13/x1XEdXgtdGhD1x0LinGEaSw5Liyt3AZoJ5bAKhlsNFxVksAPoe71?= =?us-ascii?Q?nT85+aVPClfDcFu3SJZYgZevTHtwMb+9RR33UG08zlWq6rQ7iAbdrsR/Ugu4?= =?us-ascii?Q?Hsxbx10pmH2S2ctaHhfsXEgLJUhvLeVloX5JDBPUobqO7qeVyHhK+cVFHvbT?= =?us-ascii?Q?yIQRd0+cu3cicCKTfI5Wzl7R5WzGs8TitwEaoDiOvV9QHORUibVUjJLCHpxZ?= =?us-ascii?Q?7E4aEMqYkGLN+B94FRZmGckCtZe+fHuIp7xQ3re3f7Br4HfXy+8IeQakU26O?= =?us-ascii?Q?SK3lbvZhSVGB0elLOnKGEtXuY+JvWBFb6LEK4/USrHwsBYyHkeDZCOXBaGF2?= =?us-ascii?Q?94XOuKX/k181sSig/BDFATqaUSk0J6VTsFi4V5yCdtCzPAI97EVogq6p6pSn?= =?us-ascii?Q?AHmnm7dHQiBy8ICgDh84TpNzWpZXekHNVl5Kgpue3iIGFe4no1wFKq3hikBe?= =?us-ascii?Q?90I1OxMOeztcknz8DmbXqdj9p9chgp17w+8g6DS+lGJqmHG0FDVAClzznO0C?= =?us-ascii?Q?HJJI++whnwvExoSKp1y1FHQp8BQjaj/i5+DjTbYT+DMqZi7T5qbgWJT52YxD?= =?us-ascii?Q?WC530SRoJYKMgl90LZX8pnZMEjFsXBak9o5oAybJJLUzbM9pdZSj/8tGMC/W?= =?us-ascii?Q?gNAQjkAQw/I/kiSsuEYwKBlMqLgVS9qqoElVWC3T1uqV7lIOBQ5W4XQ6YaIJ?= =?us-ascii?Q?V7vQhub6DNIJdSdKhNIER/8zs42TKyCYZMmMbTZ670phjw7NegIEoAtcKsvw?= =?us-ascii?Q?MQEC1RTmd3r9kkREjCwl2TI9V2XVGtBgVy1SZcaSJC+zEnhJhmAMq92basE3?= =?us-ascii?Q?CjQc0waFUM6nJawHSsmS6KbZ47JqE4lYKv5Dqa55ZwsvUBm/UfoJEaVb0aRc?= =?us-ascii?Q?Fwae1wcQR9dpWzYXOoOVX38nES/3?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
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: 46c6baf8-67a1-479c-1610-08d91157f6f9
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 May 2021 12:59:35.1886 (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: C/qtPzhVIYgpTIBMO/KfysJO8iIvTr58e1k3wCrHPum49xYaXVsaYtqsz2l7jBoJ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXP192MB1167
X-MDID: 1620392377-N6mwSqKK9T1Q
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/iGIFGMJjRSXUMPDfEzt_trMlxQI>
Subject: Re: [AVTCORE] Genart last call review of draft-ietf-payload-rtp-jpegxs-12
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: Fri, 07 May 2021 12:59:48 -0000

Dear Russ,

Thank you for the review and the comments. Please find the following answers to your comments below. I have modified the draft and uploaded a -13 version.


> Major Concerns:
> Section 7.2.1:  Does the SHALL related to [SMPTE-ST2110-10] only apply when TP=2110TPNL or TP=2110TPW?  Please reword to be clear when this SHALL statement applies.

The short answer is yes, the SHALL only applies when using the RTP specification in combination with SMPTE ST 2110. But, we chose to remove the SHALL sentence about ST 2110. Originally it was trying to say that in the case of using this RTP Payload under a SMPTE ST 2110 system, that the SDP description must then also follow the rules of STMPTE ST 2110-10. But, it is evident that when implementing another standard, the one must follow that standard. It is a concern of a ST 2110 implementer. For this specification it changes nothing specifically (ST 2110-10 mandates some specific SDP fields and values, but this is allowed by the memo). We specifically made these changes to allow the RTP payload to be used also outside of a ST 2110 ecosystem.

> Minor Concerns:
> Section 3.3:  I do not have [ISO21122-2], and obviously an implementer will need that document.  Can a bit more be said about "the profile" and "the level and sublevel used" without making this document too big?  I am able to get a feel for the other things listed here from their names.

We reworked this section to clarify better the concepts of the profile and level/sublevel fields, without duplicating content from ISO21122-2.

> Section 3.4:  Can you please provide some explanation of the "frat field"?

We reformulated the text a bit and added an explicit reference for the frat field to ISO21122-3 where it is defined.

> Nits:
> Section 3.2:  I do not understand the the last sentence?  Is it the same as: It represents sample values of a single image, without any interpretation relative to a colour space.

Yes, that is correct. We changed the sentence to use your suggested wording.

> Section 5: s/of ST 2110-21 do not /of [SMPTE-ST2110-21] do not /

We have updated the reference.



I hope this addresses all concerns. If not, please let us know.

Best regards,
Tim Bruylants.


-----Original Message-----
From: avt <avt-bounces@ietf.org> On Behalf Of Russ Housley via Datatracker
Sent: Thursday 6 May 2021 19:49
To: gen-art@ietf.org
Cc: last-call@ietf.org; avt@ietf.org; draft-ietf-payload-rtp-jpegxs.all@ietf.org
Subject: [AVTCORE] Genart last call review of draft-ietf-payload-rtp-jpegxs-12

Reviewer: Russ Housley
Review result: Almost Ready

I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.

For more information, please see the FAQ at <https://urldefense.proofpoint.com/v2/url?u=http-3A__wiki.tools.ietf.org_area_gen_trac_wiki_GenArtfaq&d=DwICAg&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=LTxUGukLCEfEUdo_bq048Q&m=z2S49lWIieCrrBDNEiE0OzMQ61wadfshm9Ih9Hhkn2s&s=hEKUOxSpF8FPOPqAxvBrFJ9z2wBYMsQ774TGg8xqbJQ&e=>.

Document: draft-ietf-payload-rtp-jpegxs-12
Reviewer: Russ Housley
Review Date: 2021-05-06
IETF LC End Date: 2021-05-17
IESG Telechat date: Unknown


Summary: Almost Ready


Major Concerns:

Section 7.2.1:  Does the SHALL related to [SMPTE-ST2110-10] only apply when TP=2110TPNL or TP=2110TPW?  Please reword to be clear when this SHALL statement applies.


Minor Concerns:

Section 3.3:  I do not have [ISO21122-2], and obviously an implementer will need that document.  Can a bit more be said about "the profile"
and "the level and sublevel used" without making this document too big?  I am able to get a feel for the other things listed here from their names.

Section 3.4:  Can you please provide some explanation of the "frat field"?


Nits:

Section 3.2:  I do not understand the the last sentence?  Is it the same as: It represents sample values of a single image, without any interpretation relative to a colour space.

Section 5: s/of ST 2110-21 do not /of [SMPTE-ST2110-21] do not /



_______________________________________________
Audio/Video Transport Core Maintenance
avt@ietf.org
https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_avt&d=DwICAg&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=LTxUGukLCEfEUdo_bq048Q&m=z2S49lWIieCrrBDNEiE0OzMQ61wadfshm9Ih9Hhkn2s&s=o16hyf29LBiyL98baNxoA8_0Xsng0H8HPcS4gX7o9qA&e=