Skip to content
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

Remove .bval and .bvec files from anat (mp2rage) #41

Merged
merged 1 commit into from
Mar 1, 2024

Conversation

kaitj
Copy link
Contributor

@kaitj kaitj commented Mar 1, 2024

Some recent MP2RAGE acquisitions now contain (0019, 100c) [B_value] in the DICOM header, causing dcm2niix to interpret these images as diffusion and generating a .bval and .bvec file. As it no longer follows the BIDS spec since these are not anatomical-associated files, the bids validation is failing.

New CFMM MP2RAGE sequence contains (0019, 100c) [B_value] in the DICOM
header, causing dcm2niix to interpret these images as diffusion and
generating a .bval and .bvec file.
@kaitj kaitj self-assigned this Mar 1, 2024
@kaitj kaitj changed the title Remove .bval and .bvec files from anat Remove .bval and .bvec files from anat (mp2rage) Mar 1, 2024
@kaitj kaitj requested a review from akhanf March 1, 2024 17:35
@akhanf
Copy link
Member

akhanf commented Mar 1, 2024

does this happen with the latest version of dcm2niix too?

Copy link
Member

@akhanf akhanf left a comment

Choose a reason for hiding this comment

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

looks fine to me as a quick fix - strange that the dicom tag is there!

@kaitj
Copy link
Contributor Author

kaitj commented Mar 1, 2024

does this happen with the latest version of dcm2niix too?

Hmm I'm not sure. I see they just released a new version 2 weeks ago. My guess from quickly skimming through the issues closed is that it wouldn't fix this issue, but it is worth testing it out.

@kaitj kaitj merged commit 8a8d342 into khanlab:master Mar 1, 2024
1 check passed
@kaitj kaitj deleted the maint/mp2rage branch March 1, 2024 17:55
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