Skip to content

Add swi and minIP suffixes for susceptibility weighted imaging#2343

Open
astewartau wants to merge 1 commit intobids-standard:masterfrom
astewartau:add-swi-minIP-suffixes
Open

Add swi and minIP suffixes for susceptibility weighted imaging#2343
astewartau wants to merge 1 commit intobids-standard:masterfrom
astewartau:add-swi-minIP-suffixes

Conversation

@astewartau
Copy link
Copy Markdown

Summary

Adds support for susceptibility weighted images (swi) and minimum intensity projections (minIP) as non-parametric anatomical MRI suffixes, addressing the remaining gaps identified in #1566.

  • Adds swi and minIP suffix definitions to src/schema/objects/suffixes.yaml
  • Adds a file rule group in src/schema/rules/files/raw/anat.yaml with standard MRI entities (no echo or part, since multi-echo combination occurs before SWI generation)
  • Adds swi and minIP to the non-parametric suffix table and filename template in the MRI documentation

These suffixes are classified as non-parametric (arbitrary intensity scale), consistent with how T2starw and other contrast-weighted images are handled. Like Chimap and other parametric maps, they can appear in raw datasets when pre-generated by the scanner, or in derivative datasets when computed offline.

Ref: #1566 (comment)

Test plan

  • Schema validation passes
  • Rendered docs show swi and minIP in the non-parametric suffix table and filename templates
  • Example dataset validates (see companion PR in bids-examples)

Add support for susceptibility weighted images (swi) and minimum
intensity projections (minIP) as non-parametric anatomical MRI
suffixes.

Closes bids-standard#1566
Comment on lines +58 to +59
- swi
- minIP
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.

Are these both raw images (i.e., you get them off a scanner) or should minIP be a derivative?

Also, is minIP ever sensibly used in a non-SWI context?

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.

I think the scanner will produce minIP images. Also, I got minIP images from an angio scan in ds002278 (see the JSON).

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.

How are minIP used? Are they for looking at or processing? If just looking at, minIP as a suffix seems fine, but if not, maybe it's more a stat-minIP_swi than a mod-swi_minIP?

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.

@astewartau would be better able to answer that, but I think they're for looking at.

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.

3 participants