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

Exception processing latest go-plus.owl with owltools #17263

Closed
kimrutherford opened this issue May 9, 2019 · 10 comments
Closed

Exception processing latest go-plus.owl with owltools #17263

kimrutherford opened this issue May 9, 2019 · 10 comments
Assignees

Comments

@kimrutherford
Copy link

No description provided.

@kimrutherford
Copy link
Author

Hi.

We've noticed that the latest go-plus.owl can't be processed by owltools. The version from before Saturday or so were fine. We're getting the file from http://purl.obolibrary.org/obo/go/snapshot/extensions/go-plus.owl

If this is an owltools problem I'll open an issue there.

Thanks!

owltools go-plus.owl -o -f obo out-file.obo                                                                                            
Exception in thread "main" org.semanticweb.owlapi.model.OWLOntologyStorageException: org.obolibrary.oboformat.model.FrameStructureException: multiple name tags not allowed. in frame:Frame(CHEBI:67208 id( CHEBI:67208)def( A double stranded ribonucleic acid.)synonym( double stranded ribonucleic acid RELATED)name( double-stranded RNA)name( dsRNA)is_a( CHEBI:33697)is_a( CHEBI:23367))
        at org.semanticweb.owlapi.oboformat.OBOFormatRenderer.render(OBOFormatRenderer.java:90)
        at org.semanticweb.owlapi.oboformat.OBOFormatStorer.storeOntology(OBOFormatStorer.java:42)
        at org.semanticweb.owlapi.util.AbstractOWLStorer.storeOntology(AbstractOWLStorer.java:155)
        at org.semanticweb.owlapi.util.AbstractOWLStorer.storeOntology(AbstractOWLStorer.java:119)
        at uk.ac.manchester.cs.owl.owlapi.OWLOntologyManagerImpl.saveOntology(OWLOntologyManagerImpl.java:1525)
        at uk.ac.manchester.cs.owl.owlapi.OWLOntologyManagerImpl.saveOntology(OWLOntologyManagerImpl.java:1502)
        at owltools.io.ParserWrapper.saveOWL(ParserWrapper.java:289)
        at owltools.io.ParserWrapper.saveOWL(ParserWrapper.java:209)
        at owltools.cli.CommandRunner.runSingleIteration(CommandRunner.java:3712)
        at owltools.cli.CommandRunnerBase.run(CommandRunnerBase.java:76)
        at owltools.cli.CommandRunnerBase.run(CommandRunnerBase.java:68)
        at owltools.cli.CommandLineInterface.main(CommandLineInterface.java:12)
Caused by: org.obolibrary.oboformat.model.FrameStructureException: multiple name tags not allowed. in frame:Frame(CHEBI:67208 id( CHEBI:67208)def( A double stranded ribonucleic acid.)synonym( double stranded ribonucleic acid RELATED)name( double-stranded RNA)name( dsRNA)is_a( CHEBI:33697)is_a( CHEBI:23367))
        at org.obolibrary.oboformat.model.Frame.checkMaxOneCardinality(Frame.java:424)
        at org.obolibrary.oboformat.model.Frame.check(Frame.java:405)
        at org.obolibrary.oboformat.model.OBODoc.check(OBODoc.java:390)
        at org.obolibrary.oboformat.writer.OBOFormatWriter.write(OBOFormatWriter.java:183)
        at org.semanticweb.owlapi.oboformat.OBOFormatRenderer.render(OBOFormatRenderer.java:88)
        ... 11 more
[Exit 1 ]

@balhoff
Copy link
Member

balhoff commented May 9, 2019

There was a change that is probably the cause of this: geneontology/go-announcements#56

I assume it's because there is content in the file now that was previously not saved to OBO, since it was in an external import.

@cmungall is there an owltools option that will let this through?

@balhoff
Copy link
Member

balhoff commented May 9, 2019

In the meantime I will try to see why we're getting two labels.

@balhoff balhoff self-assigned this May 9, 2019
@balhoff
Copy link
Member

balhoff commented May 10, 2019

The extra label is coming from the PRO import.

@balhoff
Copy link
Member

balhoff commented May 14, 2019

@kimrutherford out of curiosity do you require OBO format for the next step in your workflow? If you save to OWL instead you shouldn't have an issue.

@kimrutherford
Copy link
Author

Unfortunately our workflow depends on OBO.

@balhoff
Copy link
Member

balhoff commented May 15, 2019

I've got a fix for the double CHEBI labels coming from PRO. However then this reveals that the OBA import contains double labels for OBA terms, which is actually that way in OBA. Will try to sort that out next.

@balhoff
Copy link
Member

balhoff commented May 15, 2019

@kimrutherford I just realized there is a --no-check option for owltools:

owltools go-plus.owl -o -f obo --no-check out-file.obo

should succeed. The question then is whether multiple labels causes you any other problems.

@kimrutherford
Copy link
Author

I just realized there is a --no-check option for owltools

That works. Thanks!

The question then is whether multiple labels causes you any other problems.

It looks like it's all fine.

@ukemi
Copy link
Contributor

ukemi commented Jun 11, 2019

Looks like this is solved. If the issue persists, please reopen.

@ukemi ukemi closed this as completed Jun 11, 2019
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

3 participants