Skip to content

Updated version of MRICoilUsage for v5#48

Merged
lzehl merged 17 commits into
openMetadataInitiative:devfrom
Alixbonard:dev--MRICoilUsage
Feb 20, 2026
Merged

Updated version of MRICoilUsage for v5#48
lzehl merged 17 commits into
openMetadataInitiative:devfrom
Alixbonard:dev--MRICoilUsage

Conversation

@Alixbonard

@Alixbonard Alixbonard commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

Major changes:

  • coilRole property is required
  • radioFrequencyCoilRole is replaced by coilRole

19/02/2026

  • rename radiofrequencyCoilRole to signalDirectionality

@Alixbonard Alixbonard marked this pull request as ready for review February 12, 2026 15:09
@Alixbonard Alixbonard added major update most likely causes a new version v5.0 candidate labels Feb 12, 2026
@Alixbonard Alixbonard self-assigned this Feb 12, 2026
Comment thread schemas/device/MRICoilUsage.schema.tpl.json Outdated
"_instruction": "Add the anatomical region(s) the coil is designed to cover.",
"_linkedTypes": [
"controlledTerms:MRIradioFrequencyCoilRole"
"controlledTerms:AnatomicalRegion"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This controlled term has to be renamed. Need to find a better term for what is meant here (head, finger, hand, torso, foot, leg, etc); suggestion BodyRegion or OuterBodyPart or ExternalBodyPart please clarify with @tgbugs

TBD: collection across species then also include stuff from other species than mammels (tarsus, fins, wing ?, feather?, nail?, hair?)

might be rather a new category of multiple controlled terms.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

all terms should be UBERON based I think, but we need a good terminology class or classes here

@lzehl lzehl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need to further introduce a property to the MRICoil schema:

intendedMountingLocation
_instruction: Add the intended mounting location, indicating the external body region where the coil is designed to be positioned. This information reflects the manufacturer’s design intent and is independent of the coil’s actual placement during a specific acquisition.
_linkedTypes: controlledTerms:ExternalBodyStructure

Comment thread schemas/device/MRICoilUsage.schema.tpl.json Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this needs to be corrected to type string only (keep it simple here).

Note: _linkedTypes can never group object and direct values!

Comment thread schemas/device/MRICoilUsage.schema.tpl.json Outdated
Comment thread schemas/device/MRICoilUsage.schema.tpl.json Outdated
Comment thread schemas/device/MRICoilUsage.schema.tpl.json Outdated
},
"targetAnatomyCoverage": {
"_instruction": "Add the anatomical region(s) the coil is designed to cover.",
"_linkedTypes": [

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should be best a _linkedCategories for all generic anatomical terminologies (external and internal, incl organs).

Alixbonard and others added 7 commits February 20, 2026 15:51
Co-authored-by: Lyuba Zehl <lyuba.zehl@ebrains.eu>
Co-authored-by: Lyuba Zehl <lyuba.zehl@ebrains.eu>
Co-authored-by: Lyuba Zehl <lyuba.zehl@ebrains.eu>
Co-authored-by: Lyuba Zehl <lyuba.zehl@ebrains.eu>
"controlledTerms:SignalDirectionality"
]
},
"targetAnatomyCoverage": {

@lzehl lzehl Feb 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

delete this property. has been moved into the acquisition

@lzehl lzehl merged commit 7874380 into openMetadataInitiative:dev Feb 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major update most likely causes a new version v5.0 candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants