Bug #410: wrong domains for links - OpenAtlas - Redmine

Actions

Copy link

Bug #410

closed

wrong domains for links

Added by Alexander Watzinger almost 9 years ago. Updated almost 9 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Stefan Eichert

Category:

Target version:

0.6.0

Start date:

2015-05-04

Estimated time:

Found in version:

Description

After implementing domain and range checks for links, I noticed:

1) Linking E33 (Linguistic Object) with P70 (documents) to E21 (Person) raises an error because E33 is not a valid domain for P70.

Solution: changing P70 to P67

2) With dates we tried to link E61 (Time Primitive) with P2 (has type) to E55 (Type - Date Value Type), in this case E61 is also an invalid domain for P2

Solution: we allow linking from E59 (Primitive Value) with P2 to E55

3) E52 (Time-Span) - P127 (has broader term) - E52 (Time-Span): wrong domain E52 for P127, and I guess also wrong range

Solution: changing P127 to P86

Actions

Copy link

Updated by Alexander Watzinger almost 9 years ago

Subject changed from E33 is wrong domain for P70 to wrong domain E33 for P70, wrong domain E61 for P2
Description updated (diff)

Actions

Copy link

Updated by Alexander Watzinger almost 9 years ago

Subject changed from wrong domain E33 for P70, wrong domain E61 for P2 to wrong domains for links
Description updated (diff)

Actions

Copy link

Updated by Stefan Eichert almost 9 years ago

Great job with the range checks. It shows us errors we would not have found.

ad 1) thats true - p70 works only from E31. Solution: We exchange every P70 link for a P67 - "refers to/is referred to by" which is a super-property of P70 and has a domain of E89 and E1 as range

ad 2) This cannot be true. P1 has a domain of E1 which means you can link literally everything (including time-primitives) with a E55 Type. The error must be in the system and is surely not in our model.

ad 3) To mark a timespan that falls within another timespan we use (and already used) P86 (falls within/contains) and not P127 (which is only used to define sub/supertypes E55)

I hope this helps.
best regards,
Stefan

Actions

Copy link

Updated by Alexander Watzinger almost 9 years ago

File cidoc_crm_version_5.0.4-1.pdf added
Description updated (diff)

Solution for 1 and 3 already implemented and testet (local). Works great,

add 2)

E61 (Time Primitive) with P2 (has type) with E55 (Type - Date Value Type)

In this case the CIDOC CRM is correctly imported. According to the CIDOC CRM documentation (attachment included):
- E1 and E59 are not connected, both have no "sublcass of"
- the hierarchy (page 12/13) shows that E1 and E59 are on the same level and not including each other
- at page 28 is a valid link example e.g. E61 is linked with P81 to E52

Anyway, it seems to be invalid in the CIDOC CRM and I'm just reporting what the automated test finds. If it's getting too messy we always have the option to disable checks and mention it somewhere (like our shortcuts).

Actions

Copy link

Updated by Stefan Eichert almost 9 years ago

Good you checked it. I did not know that E59 is not included into the E1 subclasses (Though I was very sure about it ;-). But it is absolutely true, as the CRM Documentation says. I would suggest a workaround and as you said, we could modify the checks and allow E55 + E59 in our system and mention it somewhere.

Actions

Copy link