Last Call Review of draft-ietf-xrblock-rtcp-xr-jb-11
review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06-00
Request | Review of | draft-ietf-xrblock-rtcp-xr-jb |
---|---|---|
Requested revision | No specific revision (document currently at 14) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2013-05-31 | |
Requested | 2013-05-23 | |
Authors | Alan Clark , Varun Singh , Qin Wu | |
I-D last updated | 2013-06-06 | |
Completed reviews |
Genart Last Call review of -11
by Meral Shirazipour
(diff)
Genart Telechat review of -12 by Meral Shirazipour (diff) Secdir Last Call review of -11 by Shawn M Emery (diff) |
|
Assignment | Reviewer | Meral Shirazipour |
State | Completed | |
Request | Last Call review on draft-ietf-xrblock-rtcp-xr-jb by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 11 (document currently at 14) | |
Result | Ready w/nits | |
Completed | 2013-06-06 |
review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq . Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-xrblock-rtcp-xr-jb-11 Reviewer: Meral Shirazipour Review Date: 2013-05-31 IETF LC End Date: 2013-05-31 IESG Telechat date: NA Summary: This draft is almost ready to be published as Standard RFC but I do have some comments. Nits/editorial comments: -[Page 3], Section 1.4, Suggestion for clarity: Old:"These metrics are used to report how the jitter buffer at the receiving end of RTP stream behaves as a result of jitter in the network and are applicable to a range of RTP applications." New:"These metrics are used to report how the jitter buffer at the receiving end of RTP stream behaves as a result of jitter in the network; and they are applicable to a range of RTP applications." -[Page 5], Section 3, paragraph 5, Suggestion for consistency: "The Jitter Buffer"---->"The jitter buffer" -[Page 5], last sentence, Suggestion for clarity: "Assume that both are normalized in ticks."-----> "Assume that both are normalized in ticks (there are 10 000 ticks in a millisecond). -[Page 6], point (iii), Suggestion for clarity: "If r = t" --suggestion--> "If r == t" -[Page 6], point (iv), Possible inconsistance: "Delay playout of packet by D + (r-t)" Comment: D is not normalized in ticks. r and t are. -[Page 7], Section 4.1, Suggestion for consistency: "JB Metrics Block"---->"Jitter Block (JB) Metrics Block" -[Page 8], 3 lines before last, "the time spend"--typo-->"the time spent" -[Page 9], line 9, missing "." before "In simple": "It is calculated based on the time spent in the jitter buffer for the earliest arriving packet In simple..." -[Page 16], Section 10.2 Reference [QMB] does not point to latest version. Best Regards, Meral --- Meral Shirazipour Ericsson Research www.ericsson.com