- From: Sandro Hawke <sandro@w3.org>
- Date: Sun, 15 Sep 2013 23:21:52 -0400
- To: RDF WG <public-rdf-wg@w3.org>
- Message-ID: <52367950.8090906@w3.org>
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