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

Ensure that the relation map used by ontobio in sync with the current state of the ontology #398

Open
kltm opened this issue Oct 9, 2024 · 2 comments

Comments

@kltm
Copy link
Member

kltm commented Oct 9, 2024

This stems from #397

Basically, the mapping of relation labels to IDs is hard-coded in ontobio and we want to fix that.

https://github.com/biolink/ontobio/blob/4abdd9fb83c0789f3f7400170969431f528c82a1/ontobio/rdfgen/relations.py#L128
https://github.com/biolink/ontobio/blob/4abdd9fb83c0789f3f7400170969431f528c82a1/ontobio/io/gafparser.py#L522
https://github.com/biolink/ontobio/blob/4abdd9fb83c0789f3f7400170969431f528c82a1/ontobio/model/association.py#L445

The most recent conversation is:


@pgaudet

The thing is, this relation is valid in RO, and the inference chain is correct. However we have decided that we dont want to use this in GO. I had added subsets in RO to express which relations were valid for GO (annotations, extensions, ontology...) but @cmungall remarked that we shouldn't have information specific to a project in a 'general'/broader application ontology, which makes sense.

My question is, where can we put this information, since RO is not sufficient? I see a few options:

  • We trust RO and deepen relations as appropriate (but that would be reverting many changes that we asked curators to fix)
  • Use GOREL (which seems to be unavoidable?)
  • Use ShEX.

There must be a documented source of truth for what's hard-coded in the code.

Thanks, Pascale


@sierra-moxon

For my understanding, is GOREL curated or otherwise automatically maintained as the source of truth for relations applicable in GO? From this file: https://github.com/geneontology/go-ontology/blob/master/src/ontology/extensions/gorel.obo I see that all terms in GOREL have an xref to RO. Can I use that with confidence to do this same work in the code (but use the GOREL ontology instead of the map in the code)? If that is the case, I don't think it would be terribly hard to convert.

@kltm
Copy link
Member Author

kltm commented Oct 9, 2024

I would note that while we should have a fix in, this is also a "GORULE" and is something that Alex may be taking care of wrt end user files.
As well, I was under the impression that we are getting rid of GOREL long-term, so wouldn't that just be deferring the issue (e.g. : geneontology/go-ontology#19497)?

@pgaudet
Copy link
Contributor

pgaudet commented Oct 10, 2024

this is also a "GORULE"

Is this right ? AFAIK we dont 'fix' extensions. But we could say we delete non-compliant annotations (which is to obad, because the curator never made that statement)

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

No branches or pull requests

2 participants