editorial review of TriG

Here are the editorial issues I found in the current TriG editor's draft
https://dvcs.w3.org/hg/rdf/raw-file/3fcaf6781dc8/trig/index.html

None seems serious, and hopefully all can be fixed before publication.

I have a few larger questions which I'll send separately, and I'm going 
to try to do an implementation to check the grammar and test suite.

     -- Sandro


    Status of This Document

We should include a statement here to the effect that we might skip CR, 
and pointing folks at the test suite, implementation report, etc.  For 
example see the red box in the Status section of 
http://www.w3.org/TR/2012/WD-sparql11-query-20120724/

We can do this by hand during publication if respec somehow makes it hard.


    2. An Introduction to TriG

    The IRI or blank node label of the graph statement may be used in
    another graph statement which implies taking the union of the tripes
    generated by each graph statement.


That's a pretty confusing sentence for me, and I know what it's trying 
to say.   Plus, if we're being pedantic, the grammar also allows 
blankNodePropertyLists and Collections to be used as graph names.    
Maybe go up a level and just say a trig file is the same as turtle, 
except there are additional graph statements, each of which pairs a term 
(which is evaluated to form a graph name) and a graph in { }.    Or 
something like that.


      2.1 Graph Statements

    The graph name of a graph statement may be omitted.

That sounds confusingly like one could say "GRAPH { ... }". Maybe we can 
change things to not call default graph triples "graph statements"?

    A RDF Dataset may contain only a single graph.

Could be read as saying a dataset is only allowed to contain one 
graph.   Perhaps change to, "A simple RDF Dataset might contain only a 
single graph statement".


      4.4 Escape Sequences

In the table, the heading "Strings" should probably be a link like the 
heading local_names


    5. Parsing

    This section maps a string conforming to the grammar in section Not
    found 'sec-grammar-grammar'
    <https://dvcs.w3.org/hg/rdf/raw-file/3fcaf6781dc8/trig/index.html#sec-grammar-grammar>
    to a set of triples by

typo in section name, I assume


      5.1 Parser State

I love the idea of linking to tests here, but alas, none of those links 
are working.  I guess the links were copied from turtle as relative 
links.  (Which is ironic, since the first tests are for @base.)

    When undefines, triples are detined for the default graph.

typo in "undefines" and "detined"

I'm not seeing any text about what this is supposed to generate:

    PREFIX : <http://a.example.org/>
    PREFIX : <http://b.example.org/>
    :c :c :c.

Actually, looking in the Turtle spec, I don't see anything like that 
there, either.


      5.3 RDF Triples Construction

has some @@'s!


        5.3.1 Output Graph


I find the use of "unset" and "undefined" a little unsettling.   I think 
it would be clearer to have a special value for the default graph.  But 
whatever.

    The rule labelOrSubject
    <https://dvcs.w3.org/hg/rdf/raw-file/3fcaf6781dc8/trig/index.html#grammar-production-labelOrSubject>
    sets both |curGraph|. and |curSubject|

extra period


      5.4 Differences from Previous TriG

    This section describes the main differences between TriG, as defined
    in this ddcument, and the

Typo 'ddcument' and the sentence just ends in the middle.


      5.5 Internet Media Type, File Extension and Macintosh File Type

I'd suggest just calling this Media Type Registration

    This information that follows has been submitted to the IESG
    <http://lists.w3.org/Archives/Public/www-archive/2007Dec/0065.html>
    for review, approval, and registration with IANA.

that's for Turtle.   Instead for TriG say it will be submitted.

In Security considerations, the formatting is weird -- there should be 
blank lines between paragraphs, I guess.  And there are still a few 
references to turtle that should be trig.

In Base URI, I'd say "The TriG base directive can change...." so that it 
glosses over BASE vs @base.

That's it!

Received on Monday, 16 September 2013 03:21:59 UTC