-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GO versions are very out of date #45
Comments
This bug was due to a too small timeout (30s) for reloading the ontology; it's now fixed by setting timeout to 300s. Version IRIs are a good idea, will add them asap. |
@dosumis We have two versions of GO-PLUS in AberOWL, http://aber-owl.net/ontology/go-plus and http://aber-owl.net/ontology/GO-PLUS. Both seem to be manually uploaded. I changed GO-PLUS (in capitals) to sync from http://purl.obolibrary.org/obo/go/extensions/go-plus.owl, and leave go-plus (lowercase) as it is. |
@dosumis New version of GO-PLUS has still one unsatisfiable class (down from 17 in the version we had before), where to report? |
Closing the issue and open a new one for version IRIs. |
I can't see any inconsistent classes in the latest: http://purl.obolibrary.org/obo/go/releases/2016-05-05/extensions/go-plus.owl Do you have a list of the classes you've detected and which versions they effect? These shouldn't make it through the release pipeline. |
In AberOWL, querying GO-PLUS for owl:Nothing returns http://purl.obolibrary.org/obo/GO_0042628, 'mating plug formation'. I have no access to Protege today, but will check manually when I have the opportunity. |
Thanks. Seems it didn't show up in Protege for me because of a missing disjointness axiom (between single and multi-organism process). Adding it back makes 'mating plug formation' inconsistent. Not clear to me why you're getting this axiom and I'm not. I checked local vs URI based resolution of x-disjoint.owl, where this should live, but couldn't find it in the latest version from the URI. @cmungall - any ideas? Also, is it possible that inconsistent GO-plus versions sometimes get through the release process? I thought these should be blocked at the first Jenkins check of the editor's file + imports. Anyway I'll add back the disjoint and fix the issue with 'mating plug formation'. There are no other new inconsistent classes This fix will be in tonight's release. |
I'm offline right now, but from my local possibly stale copies of the DisjointUnionOf(BP, SOP, MOP) i.e. BP = SOP or MOP If you were only looking for explicit DisjointWith axioms David you Similarly, some reasoners (Elk?) may ignore DisjointUnionOf (even though On 6 May 2016, at 2:59, David Osumi-Sutherland wrote:
|
GO: date: 03:11:2015 11:44
GO-plus: date: 27:08:2015 08:12
Can they be kept up to date?
Also, please could you display the version IRI where this is recorded? This is a much more reliable way to track versions that the date field. (Version IRI from OBO (foundry) ontologies should be resolvable to the original file).
Cheers
The text was updated successfully, but these errors were encountered: