Re: Last-minute PROV-XML review/proofread

I committed my changes to the editor's draft based on Tom's feedback.

https://dvcs.w3.org/hg/prov/rev/7636745d349a

--Stephan

On Apr 18, 2013, at 8:53 AM, Stephan Zednik <zednis2@rpi.edu> wrote:

> Thanks Tom, comments below.
> 
> On Apr 18, 2013, at 4:03 AM, Tom De Nies <tom.denies@ugent.be> wrote:
> 
>> Hello PROV-XML editors,
>> 
>> First, my apologies that I did not get around to reading the document sooner, I was a bit short on time.
>> 
>> I've made a final pass over the latest editor's draft of the document, and it looks very good. I could hardly find anything wrong with it, apart from the minor things below.
>> So as far as I'm concerned: +1 for release as a Working Group Note. Thanks for producing such a comprehensive and understandable document.
>> 
>> Minor things/remarks:
>> 
>> - In the boilerplate, it still says "Dublic Core" instead of "Dublin Core"
> 
> Fixed (locally)
> 
>> 
>> - In the section "Schema Modularization", it might be desirable to make the mentions of "prov.xsd" etc. link to these schema's.
> 
> Added (locally)
> 
>> 
>> - I especially liked the "Elements vs. Attributes" section, since this essentially gives future implementers of new extensions a manual to convert their constructs to PROV-XML. Excellent!
>> 
>> - "Identifier Conventions": perhaps you could mention the workaround specified in RDF/XMl to convert invalid QName URIs to valid ones? (see last example PROV-Dictionary). Or do we want to discourage this?
> 
> Let's add this to the discussion of PROV-ISSUE-665
> 
>> 
>> - "Component 2: Derivations. The second component of PROV-DM is concerned with: derivations of entities from other entities and derivation subtypes WasRevisionOf (Revision), WasQuotedFrom (Quotation), and HasPrimarySource (Primary Source)."
>> Why are WasRevisionOf and the other constructs capitalized? If I'm not mistaken, they aren't mentioned like that anywhere else in the document. Couldn't this be revised to the simpler: "derivation subtypes Revision, Quotation, and Primary Source." ?
>> 
>> - same as above for the first paragraph of Component 3. 
>> 
>> - same for Component 5
> 
> These names are from a very early version of the document.  I like the suggested change and will implement it.
> 
>> 
>> - I almost missed the specification that all PROV-XML document should have a .provx extension. I'll change the extension of the example in PROV-Dictionary
>> 
>> - The last line in the document says "ignore". Shall I ignore this and assume it is to be removed? :)
> 
> Interesting.  That is an attribute I am using with some javascript which should subsequently be removed by the javascript.  I do not see this issue in my local preview of the HTML. What browser are you using? 
> 
> --Stephan
> 
>> 
>> Regards,
>> Tom
>> Tom De Nies
>> Researcher Semantic Web
>> Ghent University - iMinds
>> Faculty of Engineering and Architecture
>> Department of Electronics and Information Systems - Multimedia Lab
>> Gaston Crommenlaan 8 bus 201, B-9050 Ledeberg-Ghent, Belgium
>> 
>> t: +32 9 331 49 59
>> e: tom.denies@ugent.be
>> 
>> URL:  http://multimedialab.elis.ugent.be 
>> 
> 

Received on Thursday, 18 April 2013 15:03:46 UTC