- From: Jeremy Carroll <jjc@hplb.hpl.hp.com>
- Date: Mon, 30 Sep 2002 19:45:44 +0100
- To: "Smith, Michael K" <michael.smith@eds.com>, "webont" <www-webont-wg@w3.org>
Summary: Mike's spellchequer needs to have an RDF/XML mode.
I split my review into two parts, the boring bits here, and comments that
are a little more interesting; the interesting comments probably get a
message each, at least one of them touches on Pat's doc as well.
The boring comments are split into the moderately boring, and deadly boring.
I have only really reviewed the guide with an RDF/XML hat - I haven't
attempted to comment on its pedagogical value.
Moderately Boring
=================
History section - move to appendix?
[[
New information cannot it cannot negate any previous information. Facts and
entailments can only be added, never negated.
]] - typo and how about adding
"OWL <em>does</em> allow negative information to be explicitly stated."
or something like that.
[[
MinCardinality defaults to 0. There is no default value for cardinality or
maxCardinality.
]]
misleading, unnecessary, delete (there are no defaults).
Deadly Boring
=============
(global)
whitespace is significant in RDF literal values, and is not trimmed.
Hence probably not what was intended:
[[[
<owl:Class rdf:ID="WINE">
<rdfs:subClassOf rdf:resource="#POTABLE-LIQUID"/>
<rdfs:label xml:lang="en"> wine </rdfs:label>
<rdfs:label xml:lang="fr"> vin </rdfs:label>
...
</owl:Class>
]]]
I suggest
<owl:Class rdf:ID="WINE">
<rdfs:subClassOf rdf:resource="#POTABLE-LIQUID"/>
<rdfs:label xml:lang="en">wine</rdfs:label>
<rdfs:label xml:lang="fr">vin</rdfs:label>
...
</owl:Class>
And simply incorrect (many times)
[[[
<owl:Restriction>
<owl:onProperty rdf:resource="#REGION"/>
<owl:minCardinality> 1 </owl:minCardinality>
</owl:Restriction>
]]]
" 1 " is not an integer, you need "1".
i.e.
[[[
<owl:Restriction>
<owl:onProperty rdf:resource="#REGION"/>
<owl:minCardinality>1</owl:minCardinality>
</owl:Restriction>
]]]
Many times:
[[
rdf:parseType="collection"
]]
sp: Collection
(approx doc order)
</owl:ontology> sp: big O
<rdf:rdf sp: <rdf:RDF
</rdf:rdf> sp: </rdf:RDF>
[[
The syntax ID="WINE-REGION" is
]]
I prefer ... rdf:ID=...
(ID by itself is illegal)
[[
This is the familiar ID attribute defined by XML.
]]
no it's not.
How about
[[
This is the
<a
href="http://www.w3.org/TR/rdf-syntax-grammar/#idAttr">
rdf:ID</a> attribute
which is like the familiar ID attribute defined by XML.
]]
[[
Given this namespace definition, VIN:WINERY can be used to
]]
misleading, the URI http://www.w3.org/2003/??/owl-guide#WINERY refers to the
resource. the QName VIN:WINERY can be used only on typed node, property
element and property attribute constructions. (See on, you used this
incorrectly later).
[[
SubclassOf is the fundamental
]]
sp: <code>subClassOf</code> even at the beginning of sentence?
[[
<owl:Thing rdf:about="VIN:CENTRAL-COAST-LOCATION">
<type rdf:resource="VIN:CALIFORNIA-REGION"/>
</owl:Thing>
]]
illegal VIN is not a URI scheme name. "<type" must be namespace qualified.
What you probably mean is:
<owl:Thing
rdf:about="http://www.w3.org/2003/??/owl-guide#CENTRAL-COAST-LOCATION">
<rdf:type
rdf:resource="http://www.w3.org/2003/??/owl-guide#CALIFORNIA-REGION"/>
</owl:Thing>
If you want to shorten those URIs you need to use XML entities.
[[[
<owl:ObjectProperty id="MADE-FROM-GRAPE">
<rdfs:domain rdf:resource="#WINE"/>
<rdfs:range rdf:resource="#WINE-GRAPE"/>
</owl:ObjectProperty>
]]]
sp: rdf:ID
[[[
<owl:Restriction>
<owl:onProperty rdf:resource="#MADE-FROM-GRAPE"/>
<minCardinality> 1 </minCardinality>
</owl:Restriction>
]]]
sp: owl:minCardinality x 2
[[[
TransitiveProperty
...
P(x,y) and P(y,z) iff P(x,z)
]]]
iff should read ->
[[[
<owl:minCardinality> 2000 </owl:minCardinality>
]]]
whitespace error
[[[
<owl:sameIndividualAs about="#EVENING-STAR" />
]]]
sp: rdf:about
[[[
<owl:differentIndividualFrom about="#PARIS-FRANCE" />
]]]
sp: rdf:about
[[[
<WINE-COLOR rdf:about="#WHITE">
<rdf:label> White </rdf:label>
</WINE-COLOR>
]]]
I prefer "White" wine to " White " wine.
Jeremy
Received on Monday, 30 September 2002 14:42:34 UTC