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

Support GE non-HFS DTI bvec #736

Closed
wants to merge 1 commit into from

Conversation

mr-jaemin
Copy link
Collaborator

@mr-jaemin mr-jaemin commented Aug 7, 2023

While current dcm2niix generates the following message for GE non-HFS (i.e. feet-first) DTI,
"GE DTI directions require head first supine acquisition"
Diffusion gradient vector (*.bvec) will be generated without any required conversion. So, bvec will be inaccurate as shown below

This pull request enables the geCorrectBvecs() routine for GE non-HFS DTI with the following warning message:
"Limited validation for non-HFS (Head first supine) GE DTI: confirm gradient vector transformation"

The bvec conversion for non-HFS would be same as HFS on GE based on this validation dataset:
Both Head-first and feet-first Knee DTI with both freq. encoding directions: RL and AP collected on the same volunteer within the same exam.

Screenshot 2023-08-07 at 3 49 32 PM

@neurolabusc
Copy link
Collaborator

@mr-jaemin thanks. I had submitted a much larger set of conflicting commits to the development branch. I have now applied your fix directly, so this PR has been effectively merged.

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