Skip to content

dwipreproc: Permit rigid-body between DWI and SE-EPI#1736

Merged
jdtournier merged 3 commits intomasterfrom
dwipreproc_seepi_rigid_transform
Dec 4, 2019
Merged

dwipreproc: Permit rigid-body between DWI and SE-EPI#1736
jdtournier merged 3 commits intomasterfrom
dwipreproc_seepi_rigid_transform

Conversation

@Lestropie
Copy link
Copy Markdown
Member

Better support in dwipreproc for a very specific type of image acquisition protocol. The only change is to prevent an unnecessary image re-gridding of the SE-EPI data onto the DWI voxel grid.

While an API function is modified to add an optional input argument, the default value of this parameter means that behaviour and interface are unchanged elsewhere. I will resolve the change manually with dev, since I would prefer to more consistently use key-value function arguments in the updated API.

If the -align_seepi option is used, and the only difference between the DWI series and the SE-EPI series is a rigid-body transformation, then there is no need for the SE-EPI volumes to be explicitly re-gridded to match the DWI voxel grid; it can instead be safely concatenated with the first DWI b=0 image despite the difference in header transform. The header transform of the SE-EPI series is therefore replaced with that of the DWIs explcitly, but only in this specific instance (i.e. -align_seepi is used, and the only difference between the images is within the header transform).
@Lestropie Lestropie requested a review from a team September 25, 2019 02:42
@Lestropie Lestropie self-assigned this Sep 25, 2019
@Lestropie
Copy link
Copy Markdown
Member Author

Also: See relevant discussion in bids-apps/MRtrix3_connectome#61.

Lestropie added a commit that referenced this pull request Sep 25, 2019
Only two remaining functions within the Python API where it was considered more appropriate to use keyword arguments rather than ordered arguments. Was raised in #1736, but is ultimately downstream from #1449.
@jdtournier jdtournier merged commit 067c775 into master Dec 4, 2019
@jdtournier jdtournier deleted the dwipreproc_seepi_rigid_transform branch December 4, 2019 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants