Skip to content

improve reading of DICOM coordinate information#1869

Merged
robertoostenveld merged 1 commit intofieldtrip:masterfrom
robertoostenveld:dicom
Sep 2, 2021
Merged

improve reading of DICOM coordinate information#1869
robertoostenveld merged 1 commit intofieldtrip:masterfrom
robertoostenveld:dicom

Conversation

@robertoostenveld
Copy link
Copy Markdown
Member

This improves the dicom_old implementation, which now has a full implementation of the mri.transform matrix (including translations and rotations) according to https://doi.org/10.1016/j.jneumeth.2016.03.001.

Furthermore, the 1st and 2nd dimension of the output 3D volume is flipped to get the representation of DICOM rows and columns consistent with MATLAB.

Fhis improves the dicom_old implementation, which now has  a full implementation of the mri.transform matrix (including translations and rotations) according to https://doi.org/10.1016/j.jneumeth.2016.03.001. Furthermore, the  1st and 2nd dimension of the output 3D volume is flipped to get the representation of DICOM rows and columns consistent with MATLAB.
@robertoostenveld
Copy link
Copy Markdown
Member Author

this PR also includes some updates to the help of ft_read_mri, most notably that some of the undocumented features (which are format-specific) are now documented.

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.

1 participant