[Gen-art] Gen-ART IETF LC review of draft-ietf-xrblock-rtcp-xr-psi-decodability-04
Tom Taylor <tom.taylor.stds@gmail.com> Mon, 07 July 2014 02:36 UTC
Return-Path: <tom.taylor.stds@gmail.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E5D551A00D5; Sun, 6 Jul 2014 19:36:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2
X-Spam-Level:
X-Spam-Status: No, score=-2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, SPF_PASS=-0.001] autolearn=ham
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 u-_Va99LXzmG; Sun, 6 Jul 2014 19:36:50 -0700 (PDT)
Received: from mail-ie0-x22a.google.com (mail-ie0-x22a.google.com [IPv6:2607:f8b0:4001:c03::22a]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B05A81A00C9; Sun, 6 Jul 2014 19:36:50 -0700 (PDT)
Received: by mail-ie0-f170.google.com with SMTP id lx4so2208648iec.29 for <multiple recipients>; Sun, 06 Jul 2014 19:36:50 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:subject :content-type:content-transfer-encoding; bh=Ept7JHqUkhdYIdZYcykAZOZk3TLwVs2OmCjHudEwwaA=; b=VugoiSi/Kcd8ZYsSyEeaaRk50YHzGchnDqQhPOY2b+2v0cUCMcE6eWzAYImrARWCrh devACfw1s9hzpGg6VMyjqC7PT1Iw26jmTXzLOyWHknZIeL0YZbCaEFH2lwlGyeFDZheI VMNE9bgdVWtQPxOYIgSJM+etGlGk5Jw2ZeF5KKOCrZhloo75eWc6S2TZcE6Wpy4nmeeq W1FLzEHwkYe/h7qliFx6WKO+QZ6Dt97Mpvn82pIq6VP3EbzNsAchStYQfL+F6b8R+MuQ B+EzwQYSbrVjXjTZrGiEmLXq/Gi9NSUPFwvqVMKHmdpehBQCwEUoI9oqI5bJDyQbZbSv 0g8g==
X-Received: by 10.50.30.228 with SMTP id v4mr113686igh.26.1404700609056; Sun, 06 Jul 2014 19:36:49 -0700 (PDT)
Received: from [192.168.0.102] (dsl-173-206-11-121.tor.primus.ca. [173.206.11.121]) by mx.google.com with ESMTPSA id 8sm71911339igr.16.2014.07.06.19.36.48 for <multiple recipients> (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 06 Jul 2014 19:36:48 -0700 (PDT)
Message-ID: <53BA07BF.9090709@gmail.com>
Date: Sun, 06 Jul 2014 22:36:47 -0400
From: Tom Taylor <tom.taylor.stds@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: xrblock-chairs@tools.ietf.org, draft-ietf-xrblock-rtcp-xr-psi-decodability@tools.ietf.org, Alissa Cooper <alissa@cooperw.in>, Gen Art <gen-art@ietf.org>, The IETF <ietf@ietf.org>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/g-l1E00narzPStfgz6rGX-GhKhM
Subject: [Gen-art] Gen-ART IETF LC review of draft-ietf-xrblock-rtcp-xr-psi-decodability-04
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 07 Jul 2014 02:36:53 -0000
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-psi-decodability-04 Reviewer: Tom Taylor Review Date: 6 July 2014 IETF LC End Date: 7 July 2014 IESG Telechat date: (not known) Summary: basically ready with very minor issues and a number of editorial suggestions. Major issues: none. Minor issues: (1) It might be helpful to add text in Section 3 explaining that PAT_error_2_count and PMT_error_2_count are actually replacements for and improvements on PAT_error_count and PMT_error_count respectively and are therefore preferred in future implementations. (2) Condition (2) of PAT_error_2_count: "one table with table_id other than 0x00" is more precise than intended by [ETSI]. s/one/a/. This comment also applies to PMT_error_2_count (third from last line of first paragraph) and CAT_error_count (both conditions). Nits/editorial comments: General: blanks are missing in a number of places, typically following a comma or preceding a parenthesis. Abstract -------- "statistics metrics" seems a bit redundant, but I wonder if "metric" has a special meaning to people working in this area. To me, "metric" is another word for "measurement result". So its use to describe the contents of the XR block makes sense. However, when we get to Section 3, "metric" is used in place of "indicator". Is that really correct usage? s/Program specific information/Program Specific Information/ Section 1.1 ----------- Some redundancy with the opening paragraph of 1.1, some cramming together of different ideas. Suggested alternative: OLD This memo is based on information consistency tests and resulting indicators defined by ETSI [ETSI] and defines a new block type to augment those defined in [RFC3611] for use with MPEG2 Transport Stream (TS) [ISO-IEC.13818-1.2007]. The new block type supports reporting of the number of occurrences of each Program Specific Information (PSI) indicator in the first and second priorities that supplements information from PSI independent Decodability Statistics Metrics Block [RFC6990]; third priority indicators are not supported. NEW This memo defines a new block type for use with MPEG2 Transport Stream (TS) [ISO-IEC.13818-1.2007], to augment those defined in [RFC3611]. The new block type supports reporting of the number of occurrences of each Program Specific Information (PSI) indicator in the first and second priorities listed by [ETSI] sections 5.2.1 and 5.2.2 respectively. Third priority indicators are not supported. The metrics defined here supplement information from the PSI-independent Decodability Statistics Metrics Block [RFC6990]. Section 1.2 ----------- s/defined/defines/ on second line for consistency with the other sentences. Section 1.3 ----------- s/Architectures [RFC6792]/Architecture [RFC6792]/ s/guideline/guidelines/ s/for reporting block format using RTCP XR/for RTCP XR reporting block formats/ Section 1.4 ----------- s/;/,/ on second line. s/;/./ on third-last line. Section 3 --------- See remark on use of "metric" above (Section 1.1). Could the first sentence be rewritten: OLD ETSI TR 101290 [ETSI] generally defines metrics related to error events while this document contains counts of those metrics defined in [ETSI]. NEW ETSI TR 101290 [ETSI] generally defines indicators related to error events, while the XR block defined in this document contains counts of occurrences of the [ETSI] indicators. Fifth line: s/PSI independent/PSI-independent/ (add hyphen) Paragraph below the CRC and CAT bullets: (1) What do you mean by: "scrambling may be considered"? Do you mean that the presence or absence of scrambling is part of the error checking, or something else? (2) I'd suggest expanding "The other parameters ..." to "The other parameters defined in [ETSI] Section 5 [or whatever scope you intended] but not listed above ...". Section 3, PID_Error_Count -------------------------- Second sentence is not quite accurate. It should read: OLD A PID_error occurs when MPEG TS streams are remultiplexed and any PID doesn't refer to an actual data stream, as defined in the section 5.2.1 of [ETSI] NEW A PID error occurs [is indicated?] when no data stream is present corresponding to a given PID. This may be caused by multiplexing or demultiplexing, then remultiplexing. See section 5.2.1 of [ETSI].
- [Gen-art] Gen-ART IETF LC review of draft-ietf-xr… Tom Taylor
- Re: [Gen-art] Gen-ART IETF LC review of draft-iet… Qin Wu
- Re: [Gen-art] Gen-ART IETF LC review of draft-iet… Tom Taylor
- Re: [Gen-art] Gen-ART IETF LC review of draft-iet… Qin Wu
- Re: [Gen-art] Gen-ART IETF LC review of draft-iet… Jari Arkko
- Re: [Gen-art] Gen-ART IETF LC review of draft-iet… Qin Wu