Re: [MMUSIC] SDP Directorate: Review of draft-ietf-xrblock-rtcp-xr-decodability-07

Dear Ali,

Thank you for the review. Please see inline.

Best Regards!
Rachel

-----Original Message-----
From: mmusic-bounces@ietf.org [mailto:mmusic-bounces@ietf.org] On Behalf Of Ali C. Begen (abegen)
Sent: Monday, February 25, 2013 5:55 AM
To: draft-ietf-xrblock-rtcp-xr-decodability@tools.ietf.org
Cc: mmusic@ietf.org; gonzalo.camarillo@ericsson.com
Subject: [MMUSIC] SDP Directorate: Review of draft-ietf-xrblock-rtcp-xr-decodability-07

I am the assigned SDP directorate reviewer for this draft. For background
on the SDP directorate, please see the FAQ at
http://www.ietf.org/iesg/directorate/sdp.html

Please wait for direction from your document shepherd or AD before posting
a new version of the draft.

Summary:
--------
- The RTCP XR extension syntax looks fine
- There is one (small) technical issue
- There are quite a few editorial issues

Technical Issues:
-----------------
1) I am puzzled why you named the new block as "non-psi-ts-decodability".
It does not follow what you have in the title. I also did not get the
"non" part.

[Rachel]: To get consistent with the title we used, how about we propose to change the block name in SDP as "ts-psi-indep-decodability"?

Editorial Issues:
-----------------
1) I don't have the ISO document handy but I think the container is called
MPEG2 Transport Stream not MPEG Transport Stream. Please double check.

[Rachel]: Okay. Thanks.

2) The abstract mentions the use of broadcast of transport streams in IPTV
systems. IPTV uses only unicast and multicast. Cable and satellite systems
use the broadcast of transport streams.

[Rachel]: Okay. Will fix this.

3) Document's short name is "RTCP XR Non PSI Decodability". Why?

[Rachel]: Will be changed to "RTCP XR PSI Independent Decodability".

4) The title has "metric" but the abstract has "metrics." Please be
consistent.

[Rachel]:Okay.

5) Section 1.1. "application-dependant" -> "application-dependent"

[Rachel]:Okay.

6) Use either "decodability" or "de-codability".

[Rachel]:Okay.

7) Section 1.4. AFAIK, there is no such thing called "MPEG4 TS content."
The content could be encoded with MPEG4 but the transport stream is still
MPEG2 Transport Stream. So, you could say "MPEG4 over MPEG2-TS over RTP."

[Rachel]:Right.

8) The text seems to be using both "MPEG TS PSI Independent decodability
statistics metrics" and "MPEG-TS PSI Independent Decodability Metrics". I
think you need the former.

[Rachel]:Okay.

9) There are several typos (mostly due to copy/paste errors). Please fix
them. Acknowledgements section also has misspellings and typos.

[Rachel]: Will do.

_______________________________________________
mmusic mailing list
mmusic@ietf.org
https://www.ietf.org/mailman/listinfo/mmusic