- From: Massimo Paolucci <paolucci@icarus.cimds.ri.cmu.edu>
- Date: Thu, 10 Oct 2002 10:04:30 -0400
- To: Mohamed Keshk <mkeshk1@yahoo.com>
- Cc: www-ws@w3.org
Thanks Mohamed for your comments. You are right on the first and the third point, the ___domain of UNSPSCcode and UNSPSCclassification should be Product not CompositeProcess and the Congo should refer to TemporalInstancnce instead of Time. I think you are right on the second point also, but I leave David Martin the final call. --- Massimo > Hi, > I went over DAML-S 0.7 draft version and found these > comments as feedback: > > 1 - In "ProfileHierarchy.daml" the following is > declared: > > <daml:Class rdf:ID="Product"> > <daml:comment> > Product is the top level of a Product ontology > for now it records the product classification in > UNSPSC > </daml:comment> > </daml:Class> > > > <daml:DataProperty rdf:ID="UNSPSCcode"> > <rdfs:___domain rdf:resource="#CompositeProcess"/> > <!-- SHOULD BE "Product" ??? --> > <rdfs:range rdf:resource="&xsd;#String"/> > </daml:DataProperty> > > <daml:DataProperty rdf:ID="UNSPSCclassification"> > <rdfs:___domain rdf:resource="#CompositeProcess"/> > <!-- SHOULD BE "Product" ??? --> > <rdfs:range rdf:resource="&xsd;#String"/> > </daml:DataProperty> > > > and I belive the ___domain in both cases should be > "#Product" not "#CompositeProcess", because it makes > more sense and at the same time "CompositeProcess" > class is not defined in the same ontology. > > > 2 - In "Grounding.daml" the following is declared: > > <daml:UniqueProperty rdf:ID="damlsProcess"> > <rdfs:comment> > The atomic process to which this grounding > applies. > </rdfs:comment> > <rdfs:___domain > rdf:resource="#WsdlAtomicProcessGrounding"/> > <rdfs:range > rdf:resource="&process;#AtomicProcessPowerSet"/> > </daml:UniqueProperty> > > > "AtomicProcessPowerSet" class is not defined in > process ontology > but "AtomicProcessClass" class is the one defined. > So, they should be switched. > > > 3 - In "CongoProfile.daml" the following is declared: > > <profile:input> > <profile:ParameterDescription > rdf:ID="CreditCardExpirationDate"> > <profile:parameterName> > creditCardExpirationDate > </profile:parameterName> > <profile:restrictedTo rdf:resource="&time;#Time"/> > <profile:refersTo rdf:resource = > "&congoProcess;#creditCardExpirationDate"/> > </profile:ParameterDescription> > </profile:input> > > > there is no such class "Time" in time ontology, should > be changed to > "TemporalEntity"?. I'm not sure what should it be. > > Please correct me if I'm missing something. > > Thank you. > > Mohamed Keshk > > > __________________________________________________ > Do you Yahoo!? > Faith Hill - Exclusive Performances, Videos & More > http://faith.yahoo.com >
Received on Thursday, 10 October 2002 10:04:38 UTC