Skip to content

added term for MRI under tomography - #66

Merged
gouttegd merged 2 commits into
masterfrom
issue64addMRI
Jul 27, 2026
Merged

added term for MRI under tomography#66
gouttegd merged 2 commits into
masterfrom
issue64addMRI

Conversation

@sherwoodf

@sherwoodf sherwoodf commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Related issue: #64 (Add term for MRI)

Added term for magnetic resonance imaging, with a number of synonyms, under tomography.

I did my best to follow the structure of other terms - please let me know if there are any other standard annotation properties.

I am not sure how best to annotate the mappings (or if here is the useful place to do so?)

From local testing (sh run.sh make IMP=false test):

Violations: 466
-----------------
ERROR:      0
WARN:       444
INFO:       22

Before and after changes.

@sherwoodf
sherwoodf requested a review from gouttegd July 23, 2026 14:50
Comment thread src/ontology/components/orcidio.owl Outdated

@gouttegd gouttegd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, apart from the use of not-recommended-anymore properties oboInOwl:created_by and oboInOwl:creation_date.

Comment thread src/ontology/components/orcidio.owl Outdated
Comment thread src/ontology/fbbi-edit.owl Outdated
Comment thread src/ontology/fbbi-edit.owl Outdated
@sherwoodf
sherwoodf requested a review from gouttegd July 27, 2026 08:19
@gouttegd
gouttegd merged commit 4f7fd99 into master Jul 27, 2026
1 check passed
@gouttegd
gouttegd deleted the issue64addMRI branch July 27, 2026 09:10
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

Successfully merging this pull request may close these issues.

2 participants