Here's all my feedback on the current Syntax document. Every nit-pick is attached as an HTML file with my comments highlighted in yellow. It took 5 hours to get through the document this time, take out time for edits and it could be a 3 hour read. Overall, it looks very solid - much has improved since the last time I went through it - great work Mark and Shane! Here are the high-level comments, see the attached HTML file for the rest... ----------------------------------------------------------------------- I spent a good deal of time looking at the processing model and found a couple of minor issues with it. Issue #1: The biggest one was in how the "new subject"/"current subject" is set and used for completing incomplete triples (hanging rels). I think there are some nasty side-effects in the way "new subject" is set and used to initialize "current subject". Most notably, it looks like this: <div resource="#betty" rel="foaf:knows"> <span resource="#fred"></span> </div> generates the following triple: <#betty> <foaf:knows> <#fred> . Issue #2: The "recurse" flag is disabled after a triple is generated. This makes the parser stop entirely when the first triple is generated - which isn't what we want... Issue #3: @instanceof generates a new bnode, even if @about is present? That's how I interpreted the processing rules (see HTML file). ----------------------------------------------------------------------- There were several detail-oriented things that confused me in Section 7, the CURIE specification (see HTML file). ----------------------------------------------------------------------- The dbpedia namespace should have a more verbose namespace abbreviation, some might confuse p: with a property on the <p> HTML tag in the examples. p: http://dbpedia.org/property/ Perhaps the following should be used: dbp: http://dbpedia.org/property/ Or: ped: http://dbpedia.org/property/ dbpedia: http://dbpedia.org/property/ ------------------------------------------------------------------------ -- manu -- Manu Sporny President/CEO - Digital Bazaar, Inc. blog: Intro to the Semantic Web in 6 minutes (video) http://blog.digitalbazaar.com/2007/12/26/semantic-web-intro
This archive was generated by hypermail 2.2.0+W3C-0.50 : Tuesday, 8 January 2008 14:15:20 GMT