Skip to content

Updated version of dynamicMRIAcquisition for v5#34

Merged
lzehl merged 22 commits into
openMetadataInitiative:devfrom
Alixbonard:dev-functionalMRIAcquisition
Feb 20, 2026
Merged

Updated version of dynamicMRIAcquisition for v5#34
lzehl merged 22 commits into
openMetadataInitiative:devfrom
Alixbonard:dev-functionalMRIAcquisition

Conversation

@Alixbonard

@Alixbonard Alixbonard commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Major updates:

  • Add acquisitionDuration and behavioralProtocol as required
  • delayAfterTrigger is removed
  • delayTime is optional
  • fieldMap, sbref and structureMRI are becoming distortion correction, motion correction, registration data respectively

19/02/2026:

  • rename functionalMRIAcquisition to dynamicMRIAcquisition (fMRI includes few dynamicMRIAcquisition mostly Bold signal related)
  • distortion correction, motion correction and registration data are moved to staticMRIAcquisition
  • rename acquisitionDuration to volumeAcquisitionTime

@lauramble lauramble added major update most likely causes a new version v5.0 candidate labels Jan 27, 2026
@Alixbonard Alixbonard marked this pull request as ready for review February 2, 2026 12:05
@lzehl

lzehl commented Feb 13, 2026

Copy link
Copy Markdown
Member

Move distortion correction, motion correction and registration data to MRIAcquisition (see comment in #43)

Rename acquisitionDuration to volumeAcquisitionTime because this is should not be confused with the length of the time series but refers to the time required to acquire all slices that make up one 3D brain volume. In a lot of fMRI settings this time is equal to TR (where one TR = one full volume), only in advanced fMRI cases the TR might be larger or smaller (depending on the case).

we should rename the activity dynamicMRIAcquisition which would include besides functional MRI also physiological MRI techniques since all of them seem to use the same parameter types and only different parameter values and constraints. So they can be handled by one activity schema.

following this you could also rename the activity MRIAcquisition to staticMRIAcquisition (covering then standard structural MRI and diffusion MRIs)

@Alixbonard Alixbonard changed the title Updated version of functionalMRIAcquisition for v5 Updated version of dynamicMRIAcquisition for v5 Feb 20, 2026
Comment thread schemas/activity/dynamicMRIAcquisition.schema.tpl.json Outdated
Comment thread schemas/activity/dynamicMRIAcquisition.schema.tpl.json Outdated
Comment thread schemas/activity/dynamicMRIAcquisition.schema.tpl.json Outdated
Comment thread schemas/activity/dynamicMRIAcquisition.schema.tpl.json Outdated
Comment thread schemas/activity/dynamicMRIAcquisition.schema.tpl.json Outdated
Alixbonard and others added 7 commits February 20, 2026 16:14
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>
Co-authored-by: Lyuba Zehl <lyuba.zehl@ebrains.eu>
@@ -0,0 +1,41 @@
{
"_extends": "activity/staticMRIAcquisition.schema.tpl.json",

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.

Suggested change
"_extends": "activity/staticMRIAcquisition.schema.tpl.json",
"_extends": "activity/MRIAcquisition.schema.tpl.json",

@lzehl lzehl merged commit 1a9e34d into openMetadataInitiative:dev Feb 20, 2026
1 check failed
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.

4 participants