mcwdsi / OMRSE

The Ontology for Modeling and Representation of Social Entities
Creative Commons Attribution 4.0 International
14 stars 7 forks source link

OMRSE_00000033 aggregate of organizations has two definitions #174

Closed zhengj2007 closed 2 years ago

zhengj2007 commented 2 years ago

OMRSE_00000033 aggregate of organizations has two definitions

Should remove one.

Besides, the term has 'imported from' annotation: http://purl.obolibrary.org/obo/omrse.owl. I think it should be deleted.

CDowland commented 2 years ago

Thanks for calling this to our attention. This appears to be a previously resolved problem that is now returning.

Source of the issue: D-acts has individually imported some OMRSE terms, including some that have been revised since the time they were imported. And since OMRSE now imports all of d-acts, OMRSE is getting back its old versions of those terms.

This was addressed and fixed before: https://github.com/d-acts/d-acts/issues/32.

But I see the latest version of d-acts somehow has the old versions of those terms again. So we may be once again getting our old version back from d-acts, and then it is combined with the current one as one term. But I don't think that's happening with the other terms that are also once again appearing as their old version in d-acts, so I'm a bit confused. I will ask about it over on the d-acts issue tracker: https://github.com/d-acts/d-acts/issues/32.

wdduncan commented 2 years ago

I seem to recall (and I may be completely wrong) that you can edit the appropriate *terms.txt file (e.g. d-acts_terms.txt) in order to specify which terms are used when building the import file. Here is a link to the "update import" instructions for pato: http://pato-ontology.github.io/pato/odk-workflows/UpdateImports/

I will try to read later.

Also, duplicate definitions exist elsewhere in the omrse-edit.owl file. E.g. see screenshot for continuant.

image

Note: In the latest omrse.owl release (i.e., http://purl.obolibrary.org/obo/omrse/releases/2022-04-06/omrse.owl), the duplicate definitions for continuant do not exist However, there are still two (distinct) definitions for aggregate of organizations.

image

This makes me wonder if duplicates are removed as part of the ODK process to create a release (I can't recall).