-
Notifications
You must be signed in to change notification settings - Fork 11
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
Rename phasediff image in dicom_to_nifti #191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add test of output JSON file for the fieldmap diff case
I may have misled you in our internal meeting. This PR only changes the name of the output phasediff from sub-XX_phase2.nii.gz and sub-XX_phase2.json to sub-XX_phasediff.nii.gz and sub-XX_phasediff.json. The relevant function which creates the json file for the fieldmap is I opened an issue to add the test to the relevant function #194 |
* Add phasediff instead of phase for phasediff images * Add potential issue where phasediff could be in the subject id * Add check on sequence name * account for .nii and .nii.gz * Remove .nii
Description
This PR fixes an issue in dicom_to_nifti where phasediff images were wrongly indentified in their filenames as phase images instead of phasediff images.
The function looks at the output of dcm2bids and if
It will rename the json and the nifti to phasediff.
A test was changed to reflect the new names.
Linked issues
Fixes #190