Skip to main content

IETF Last Call Review of draft-ietf-avtcore-rtp-v3c-12
review-ietf-avtcore-rtp-v3c-12-genart-lc-eggert-2025-10-28-00

Request Review of draft-ietf-avtcore-rtp-v3c
Requested revision No specific revision (document currently at 16)
Type IETF Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2025-10-29
Requested 2025-10-15
Authors Lauri Ilola , Lukasz Kondrad
I-D last updated 2026-01-28 (Latest revision 2026-01-28)
Completed reviews Genart IETF Last Call review of -12 by Lars Eggert (diff)
Secdir IETF Last Call review of -12 by Carl Wallace (diff)
Tsvart IETF Last Call review of -12 by Magnus Westerlund (diff)
Tsvart Telechat review of -14 by Magnus Westerlund (diff)
Assignment Reviewer Lars Eggert
State Completed
Request IETF Last Call review on draft-ietf-avtcore-rtp-v3c by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/2NadNzI56iRhggUxHEBbWvhZXiQ
Reviewed revision 12 (document currently at 16)
Result Ready w/nits
Completed 2025-10-28
review-ietf-avtcore-rtp-v3c-12-genart-lc-eggert-2025-10-28-00
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://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-avtcore-rtp-v3c-??
Reviewer: Lars Eggert
Review Date: 2025-10-28
IETF LC End Date: 2025-10-29
IESG Telechat date: Not scheduled for a telechat

# genart review of draft-ietf-avtcore-rtp-v3c-12

CC @larseggert

## Comments

This is in good shape, No specific concerns.

### Section 4.3.1, paragraph 3
```
      unsigned int(5) vuh_unit_type;
```
What does the parenthesized integer indicate? Am not familiar with this style of pseudocode.

### Boilerplate

This document uses the RFC2119 keywords "SHALL", "SHOULD", "MUST NOT", "SHALL
NOT", "MAY", "OPTIONAL", "MUST", and "MUST NOT", but does not contain the
recommended RFC8174 boilerplate. (It contains a variant of the RFC2119
boilerplate.)

### Inclusive language

Found terminology that should be reviewed for inclusivity; see
https://www.rfc-editor.org/part2/#inclusive_language for background and more
guidance:

 * Term `traditional`; alternatives might be `classic`, `classical`, `common`,
   `conventional`, `customary`, `fixed`, `habitual`, `historic`,
   `long-established`, `popular`, `prescribed`, `regular`, `rooted`,
   `time-honored`, `universal`, `widely used`, `widespread`

## Nits

All comments below are about very minor potential issues that you may choose to
address in some way - or ignore - as you see fit. Some were flagged by
automated tools (via https://github.com/larseggert/ietf-reviewtool), so there
will likely be some false positives. There is no need to let me know what you
did with these suggestions.

### Typos

#### Section 3.1.2, paragraph 17
```
-    volumetric frame: set of 3D points specified by their cartesian
-                                                          ^
+    volumetric frame: set of 3D points specified by their Cartesian
+                                                          ^
```

### Grammar/style

#### Section 5.4.3, paragraph 8
```
the conditional DOND/DONL field. Otherwise v3c-tile-id field MUST NOT be pre
                                 ^^^^^^^^^
```
A comma may be missing after the conjunctive/linking adverb "Otherwise".

#### Section 7.2, paragraph 26
```
ion level. The attribute allows to associate V3C specific media format parame
                                ^^^^^^^^^^^^
```
Did you mean "associating"? Or maybe you should add a pronoun? In active voice,
"allow" + "to" takes an object, usually a pronoun.

#### Section 7.2, paragraph 36
```
e to understand them. It allows to associate V3C specific parameters with the
                                ^^^^^^^^^^^^
```
Did you mean "associating"? Or maybe you should add a pronoun? In active voice,
"allow" + "to" takes an object, usually a pronoun.

#### Section 9.3, paragraph 7
```
ll be registered with IANA; see Section Section 7.1. 10.2. V3C format paramet
                                ^^^^^^^^^^^^^^^
```
Possible typo: you repeated a word.

#### Section 9.4, paragraph 1
```
line. Its format is defined in Section Section 9.1. Further semantics are pro
                               ^^^^^^^^^^^^^^^
```
Possible typo: you repeated a word.

#### Section 9.4, paragraph 5
```
TP in general. This responsibility lays on anyone using RTP in an application
                                   ^^^^^^^
```
Did you mean "lies on"?

## Notes

This review is in the ["IETF Comments" Markdown format][ICMF]. You can use the
[`ietf-comments` tool][ICT] to automatically convert this review into
individual GitHub issues. Review generated by the [`ietf-reviewtool`][IRT].

[ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md
[ICT]: https://github.com/mnot/ietf-comments
[IRT]: https://github.com/larseggert/ietf-reviewtool