Reviews:CatherineRoussey about SimpleOrAggregated
From Odp
(Difference between revisions)
(New page: {{Content OP Proposal Review Template |CreationDate=2010/9/10 |SubmittedBy=CatherineRoussey |ContentOPUnderReview=SimpleOrAggregated |RevisionID=10064 |Score=0 - needs major revision |Revi...) |
|||
| Line 1: | Line 1: | ||
{{Content OP Proposal Review Template | {{Content OP Proposal Review Template | ||
| - | |||
|SubmittedBy=CatherineRoussey | |SubmittedBy=CatherineRoussey | ||
|ContentOPUnderReview=SimpleOrAggregated | |ContentOPUnderReview=SimpleOrAggregated | ||
|RevisionID=10064 | |RevisionID=10064 | ||
| + | |CreationDate=2010/9/10 | ||
|Score=0 - needs major revision | |Score=0 - needs major revision | ||
|ReviewSummary=I do not understand exactly what will be the purpose of this pattern and Which behavior the author expect from the reasoner. So I need more information to give a review. | |ReviewSummary=I do not understand exactly what will be the purpose of this pattern and Which behavior the author expect from the reasoner. So I need more information to give a review. | ||
| Line 9: | Line 9: | ||
The main problem is the semantic of the aggregation property. There exist different partOf relationship and I need to know the relationship between the aggregation property and the partOf one... | The main problem is the semantic of the aggregation property. There exist different partOf relationship and I need to know the relationship between the aggregation property and the partOf one... | ||
|ReviewConfidence=I know the design pattern composite in software engineering...and the DUL ontology. The hasPart and isPartOf property has to be associated to this pattern. | |ReviewConfidence=I know the design pattern composite in software engineering...and the DUL ontology. The hasPart and isPartOf property has to be associated to this pattern. | ||
| - | |ReviewProblems= No information is given on the meaning of the aggregation property: | + | |ReviewProblems=No information is given on the meaning of the aggregation property: |
is it a hasMember, hasPart, hasComponent property? | is it a hasMember, hasPart, hasComponent property? | ||
maybe the aggregation can be replace by any of them... | maybe the aggregation can be replace by any of them... | ||
| Line 19: | Line 19: | ||
if an individual O has two aggregated Members O1 and O2 asserted in the ontology... Moreover if O1 is fixed as same as O2... what do you expect from the reasoner? What's happen if instead 02 is also an aggregatedMember of O1? | if an individual O has two aggregated Members O1 and O2 asserted in the ontology... Moreover if O1 is fixed as same as O2... what do you expect from the reasoner? What's happen if instead 02 is also an aggregatedMember of O1? | ||
| - | |||
|ReviewClearProblem=This pattern is only applicable in certains conditions which are not well described ... | |ReviewClearProblem=This pattern is only applicable in certains conditions which are not well described ... | ||
|ReviewClearRelevance=The consequence on individual are not well described. | |ReviewClearRelevance=The consequence on individual are not well described. | ||
| Line 25: | Line 24: | ||
Is it what do you expect from the reasoner? | Is it what do you expect from the reasoner? | ||
| - | Moreover if the whole is | + | Moreover if the whole is classified as a simple Object and is composed of two objects that are equivalent to each other there are an unconsistency... |
So I do no think that the reasoning about number of parts is possible with this pattern... | So I do no think that the reasoning about number of parts is possible with this pattern... | ||
|ReviewMissing=the semantic of the aggregation property: hasAggregatedMember... | |ReviewMissing=the semantic of the aggregation property: hasAggregatedMember... | ||
}} | }} | ||
Revision as of 15:38, 10 September 2010
CatherineRoussey about SimpleOrAggregated (Revision ID: 10064)
Overall suggestion (score): 0 - needs major revision
Review Summary: I do not understand exactly what will be the purpose of this pattern and Which behavior the author expect from the reasoner. So I need more information to give a review.
The main problem is the semantic of the aggregation property. There exist different partOf relationship and I need to know the relationship between the aggregation property and the partOf one...
Reviewer Confidence: I know the design pattern composite in software engineering...and the DUL ontology. The hasPart and isPartOf property has to be associated to this pattern.
Problems: No information is given on the meaning of the aggregation property:
is it a hasMember, hasPart, hasComponent property? maybe the aggregation can be replace by any of them... and does this property transitive or not?
Why do you thing that all the objects can be classified as a simple one or an aggregated one? First I would rather fixe a subClassOf relationship between ObjectByCardinality and Object... Depend of the point of view (the scale) an object can be classified as simple or not... An organ is a simple object or not? Organ is composed of cells and cell is composed of...
if an individual O has two aggregated Members O1 and O2 asserted in the ontology... Moreover if O1 is fixed as same as O2... what do you expect from the reasoner? What's happen if instead 02 is also an aggregatedMember of O1?Community Relevance:
Relation to Best Practices:
Reusability:
Relations to Other Patterns:
Overall Understandability:
Clear Problem Description: This pattern is only applicable in certains conditions which are not well described ...
Clear Relevance and Consequences: The consequence on individual are not well described.
If a whole is composed of two aggregatedMembers which are fixed as equivalent, the whole is still classified as a aggregatedObject. Is it what do you expect from the reasoner?
Moreover if the whole is classified as a simple Object and is composed of two objects that are equivalent to each other there are an unconsistency...
So I do no think that the reasoning about number of parts is possible with this pattern...Clear Figures and Illustrations:
Missing Information: the semantic of the aggregation property: hasAggregatedMember...
Posted: 2010/9/10 Last modified: 2010/9/10
