Skip to content
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

Loading OBO Relations Ontology causes Protege 5.5 to hang #817

Closed
cmungall opened this issue Dec 14, 2018 · 7 comments
Closed

Loading OBO Relations Ontology causes Protege 5.5 to hang #817

cmungall opened this issue Dec 14, 2018 · 7 comments
Assignees

Comments

@cmungall
Copy link

This occurs when I try and load http://purl.obolibrary.org/obo/ro.owl in Protege 5.5.0 beta-5-SNAPSHOT

It doesn't matter if I download the file first and open directly, or open from a URL

The same file will load fine using the same version of the OWLAPI (tested in the absence of catalog files).

This is on OS X, but a user has reported this for a PC too.

I'm afraid I don't recall where to read log messages, but the hanging happens before the magic red triangle appears.

I suspect it must be something in an import, because if I include a catalog file and the imports from the top level of https://github.com/oborel/obo-relations/ (which is identical to the release imports) then loading is successful.

Note I did test using robot with a no catalog scenario and it parses fine, so my random guess is something in the Protege stack different from the owlapi stack about how redirects is handled in fetching from remote URLs...?

@matthewhorridge matthewhorridge self-assigned this Dec 17, 2018
@matthewhorridge
Copy link
Contributor

Thanks for this report @cmungall. Reproduced.

@matthewhorridge
Copy link
Contributor

Looking at a stack dump, possibly related to this:

owlcs/owlapi#806

@matthewhorridge
Copy link
Contributor

The problem is indeed due to
owlcs/owlapi#806. A deadlock is caused because ontologies are loaded in a separate manager and in a thread that is not the EDT.

Upgrading to OWL API 4.5.7 fixes the problem. This also breaks some things due to changes in the DLExpressivityChecker. I'll fix these problems and produce a new release.

@cmungall
Copy link
Author

AFAICR this happens if one of the imported ontologies is doing something naughty (or completely fails to parse), can you tell from the stack dump if there is an obvious offender?

@matthewhorridge
Copy link
Contributor

@cmungall I'll look into this, but one of the imports cannot be found (need to trace down the reason why) and this causes a prompt to be displayed. With the changes in the OWL API this caused concurrent access and a deadlock. OWL API 4.5.7 fixed this problem.

@matthewhorridge
Copy link
Contributor

Just a quick followup. All of the imports can be found on the Web. Protege does display message boxes for each import indicating that it can't be found locally, which is painful, but a different issue.

@matthewhorridge
Copy link
Contributor

Just a heads up @cmungall I've just produced a 5.5.0-beta-7 release that updates the OWL API and fixes the hanging problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants