Re: [Gen-art] Gen-ART review of draft-ietf-geopriv-pdif-lo-profile-13.txt

"Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com> Tue, 25 November 2008 07:52 UTC

Return-Path: <gen-art-bounces@ietf.org>
X-Original-To: gen-art-archive@optimus.ietf.org
Delivered-To: ietfarch-gen-art-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id C14063A6A90; Mon, 24 Nov 2008 23:52:31 -0800 (PST)
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 953883A6A95 for <gen-art@core3.amsl.com>; Mon, 24 Nov 2008 23:52:31 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.949
X-Spam-Level:
X-Spam-Status: No, score=-5.949 tagged_above=-999 required=5 tests=[AWL=-0.300, BAYES_00=-2.599, HELO_EQ_SE=0.35, J_CHICKENPOX_26=0.6, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xSNkOSQzdW0w for <gen-art@core3.amsl.com>; Mon, 24 Nov 2008 23:52:30 -0800 (PST)
Received: from mailgw4.ericsson.se (mailgw4.ericsson.se [193.180.251.62]) by core3.amsl.com (Postfix) with ESMTP id 626D43A6A90 for <gen-art@ietf.org>; Mon, 24 Nov 2008 23:52:30 -0800 (PST)
Received: from mailgw4.ericsson.se (unknown [127.0.0.1]) by mailgw4.ericsson.se (Symantec Mail Security) with ESMTP id 2D80E214AB; Tue, 25 Nov 2008 08:52:27 +0100 (CET)
X-AuditID: c1b4fb3e-af787bb00000537b-fe-492baeba6048
Received: from esealmw129.eemea.ericsson.se (unknown [153.88.254.124]) by mailgw4.ericsson.se (Symantec Mail Security) with ESMTP id F3A9D200C8; Tue, 25 Nov 2008 08:52:26 +0100 (CET)
Received: from esealmw127.eemea.ericsson.se ([153.88.254.171]) by esealmw129.eemea.ericsson.se with Microsoft SMTPSVC(6.0.3790.1830); Tue, 25 Nov 2008 08:52:26 +0100
Received: from [159.107.25.31] ([159.107.25.31]) by esealmw127.eemea.ericsson.se with Microsoft SMTPSVC(6.0.3790.1830); Tue, 25 Nov 2008 08:52:26 +0100
Message-ID: <492BAEB9.2020004@ericsson.com>
Date: Tue, 25 Nov 2008 08:52:25 +0100
From: "Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com>
User-Agent: Thunderbird 2.0.0.18 (Windows/20081105)
MIME-Version: 1.0
To: "Thomson, Martin" <Martin.Thomson@andrew.com>
References: <492ABA55.3030509@ericsson.com> <E51D5B15BFDEFD448F90BDD17D41CFF10520FF51@AHQEX1.andrew.com>
In-Reply-To: <E51D5B15BFDEFD448F90BDD17D41CFF10520FF51@AHQEX1.andrew.com>
X-OriginalArrivalTime: 25 Nov 2008 07:52:26.0328 (UTC) FILETIME=[C218E580:01C94ED2]
X-Brightmail-Tracker: AAAAAA==
Cc: geopriv-ads@tools.ietf.org, geopriv-chairs@tools.ietf.org, General Area Review Team <gen-art@ietf.org>, "Winterbottom, James" <James.Winterbottom@andrew.com>, draft-ietf-geopriv-pdif-lo-profile@tools.ietf.org
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-geopriv-pdif-lo-profile-13.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/pipermail/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>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Sender: gen-art-bounces@ietf.org
Errors-To: gen-art-bounces@ietf.org

Martin:

I noticed that the validator gave a bunch of errors, but I checked them 
and they were related to the XML Schema (mostly in OpenGML schemas), not 
to the examples you provided. I am not sure if this is the same issue you 
are referring to.

/Miguel

Thomson, Martin wrote:
> Hi Miguel,
> 
> Thanks for the review.  I will rev the document today or tomorrow and address all your comments.
> 
> Regarding the examples, they are all being changed in the revision.  Unfortunately, all of the parseContents="lax" directives cause the validator to stop checking.  We missed the same errors [1].  I now use schema documents with all instances of "lax" replaced with "strict" for validation.  That method is better able to find errors than using the straightforward method.
> 
> Maybe there is a lesson to be learned there...
> 
> WGS84 is an acronym that is so well known in the location business that I guess we (wrongly) assumed that it would make sense.  This is the coordinate system used by the GPS system, and many other mapping and location systems.  It defines what latitude and longitude really mean.
> 
> Regards,
> Martin
> 
> [1] The errors: the method element should be qualified with "gp:" and the timestamp element needed to be qualified with "dm:" and moved after the dm:device element.
> 
>> -----Original Message-----
>> From: Miguel A. Garcia [mailto:Miguel.A.Garcia@ericsson.com]
>> Sent: Monday, 24 November 2008 8:30 AM
>> To: draft-ietf-geopriv-pdif-lo-profile@tools.ietf.org; geopriv-
>> chairs@tools.ietf.org; geopriv-ads@tools.ietf.org
>> Cc: General Area Review Team
>> Subject: Gen-ART review of draft-ietf-geopriv-pdif-lo-profile-13.txt
>>
>> I have been selected as the General Area Review Team (Gen-ART)
>> reviewer for this draft (for background on Gen-ART, please see
>> http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).
>>
>> Please resolve these comments along with any other Last Call comments
>> you may receive.
>>
>> Document: draft-ietf-geopriv-pdif-lo-profile-13.txt
>> Reviewer: Miguel Garcia <miguel.a.garcia@ericsson.com>
>> Review Date: 2008-11-24
>> IETF LC End Date: 2008-11-30
>>
>> Summary: The document is ready for publication as a standards track
>> RFC.
>>
>> Comments:
>> - I have taken the effort of validating all the XML examples included
>> in
>> the draft. I used the Xerces validator available at
>> http://validate.openlaboratory.net/. I find awkward to include 39 XML
>> Schema files (thanks to the OpenGML explosion of schema files), but
>> that
>> doesn't seem to be a problem (other than for humans). Other than that,
>> the examples all validate.
>>
>> - The short name of the draft (which is printed as a heading in every
>> page) is set to "GEOPRIV PIDF-LO Profile". I think it should be set to
>> "GEOPRIV PIDF-LO Usage", since neither the title nor the text refer to
>> the document as a profile.
>>
>> - Section 1, first paragraph, add an informative reference to GMLv3
>> when
>> it is mentioned in the text.
>>
>> - Section 2, 5th paragraph, expand "LIS" and "LCP".
>>
>> - Section 5, the first paragraph on page 15. What is a "WGS84 CRS"? Is
>> this something where an informative reference could be added?
>>
>>
>> /Miguel
>>
>>
>> --
>> Miguel A. Garcia
>> +34-91-339-3608
>> Ericsson Spain
> 
> ------------------------------------------------------------------------------------------------
> This message is for the designated recipient only and may
> contain privileged, proprietary, or otherwise private information.  
> If you have received it in error, please notify the sender
> immediately and delete the original.  Any unauthorized use of
> this email is prohibited.
> ------------------------------------------------------------------------------------------------
> [mf2]

-- 
Miguel A. Garcia
+34-91-339-3608
Ericsson Spain
_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art