-
Notifications
You must be signed in to change notification settings - Fork 2
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
Changing mapping that is causing unsat on Uberon #36
Comments
cc @dosumis |
We are moving the location of these mappings to the BICAN repos - splitting to separate repos so we can provide a PURL for each. Here's DMBA: https://github.com/brain-bican/developing_mouse_brain_atlas_ontology/tree/main @hkir-dev is responsible for the move. @gouttegd - what is the build process that is finding this? Looks like we need to review and move to the new repos. DO we need to fix in more than one place in this case given the current assumptions about builds? |
The bridge files in this repo are referenced from the Uberon Makefile, lines 1254–1272. |
@hkir-dev - can we switch to equivalent files on the new DMBA repo? We can then update mapping there. |
Equivalent files are:
I can create a pull request to change locations. But we changed namespace of the entities as well. Such as Will this namespace change cause any problem? |
|
I don’t foresee any issue with that at least within Uberon. DMBA entities are not referenced from anywhere else than the bridge itself, so the namespace change should not require to update anything. |
I’ve updated the Uberon Makefile to pull the MBA/DMBA bridges from the new indicated location (in a private branch, for now). Can the issue about DMBA:16271 be fixed in https://github.com/brain-bican/developing_mouse_brain_atlas_ontology? |
@dosumis can we please prioritise this issue here soon? Its a bit disruptive to have this in the Uberon pipeline like this.. :P Thanks! |
The bridge file updated at https://github.com/brain-bican/developing_mouse_brain_atlas_ontology/blob/main/src/ontology/new-bridges/new-uberon-bridge-to-dmba.owl |
@hkir-dev Not sure what you mean by “do you still have that branch”. Uberon is configured to fetch the bridge from that branch, if that’s what you mean. So if the issue has been fixed there, we will automatically get the fix the next time we update our external resources. :) |
This is a simple mapping error. In Uberon lateral tuberal nucleus is explicitly intended to refer to a structure in human and higher primates, there is a distinct term intended to refer to a similar structure in rodents: tuberal nucleus (sensu Rodentia), where a comment explains:
Of note, this erroneous mapping comes from one of the custom bridges we have with the Allen Mouse Brain Atlas, which are maintained elsewhere and simply used as they come in Uberon. So the fix needs to happen in the remote source.
Originally posted by @gouttegd in obophenotype/uberon#2928 (comment)
The text was updated successfully, but these errors were encountered: