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

Fourth parameter in --acqp for FSL eddy #1

Open
tashrifbillah opened this issue Jun 20, 2019 · 1 comment
Open

Fourth parameter in --acqp for FSL eddy #1

tashrifbillah opened this issue Jun 20, 2019 · 1 comment

Comments

@tashrifbillah
Copy link
Collaborator

tashrifbillah commented Jun 20, 2019

In a BIDS sidecar json file generated by dcm2niix, the fourth parameter is determined as one of the following with descending priority:

  1. TotalReadoutTime
  2. (BandwidthPerPixelPhaseEncode or just BandwidthPerPixel) * (PhaseEncodingSteps - 1)
  3. EffectiveEchoSpacing * (PhaseEncodingSteps - 1)
  4. Least preferred according to our observation of various json files: DwellTime * (PhaseEncodingSteps - 1)

The direction of scan can be determined as one of the following with descending priority:

  1. PhaseEncodingDirectionDisplayed i.e A->P
  2. PhaseEncodingDirection:
    right-anterior-superior are the positive directions. So if Master branch release #2 is j, that means P-->A, if it is j- or -j, then it is A-->P. In the former case, we want 0 1 0, in the latter case we want 0 -1 0 as the first three parameters on a line in --acqp file.

Reference:
https://fsl.fmrib.ox.ac.uk/fsl/fslwiki/eddy/UsersGuide

"This file is described here, here and in more detail here."

@sbouix
Copy link
Contributor

sbouix commented Jun 21, 2019

For the readout time, I agree and it doesn't really matter for us.
For the Phase encoding, it is almost correct except that the 0 -1 0 it has to be expressed in the "slice orientation". For axial acquisitions ((1,0,0),(0,1,0),(0,0,1)) in RAS space. A PA phase encoding direction corresponds to 0 -1 0, but if you have a sagittal acquisition ((0,1,0)(0,0,1)(1,0,0)) than your acq file shoudl be -1 0 0

tashrifbillah added a commit that referenced this issue May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants