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

dicm2nii with physio dicom #58

Open
jamilaandoh opened this issue Mar 29, 2023 · 4 comments
Open

dicm2nii with physio dicom #58

jamilaandoh opened this issue Mar 29, 2023 · 4 comments

Comments

@jamilaandoh
Copy link

Dear all,
i used the dicom anonymiser for data collected using siemens [XA30A] and it worked great.
We also collect physio data which are also in dicom fomat (and grouped together with the volumes).
Do you know if the dicom2nii converter can deal with such physio files? i have some error when i run it, and i am not sure this is the issue.
"Error using reshape
Size arguments must be real integers.

Error in dicm_img (line 107)
img = reshape(img, [spp dim nFrame]);

Error in dicm2nii (line 832)
img = dicm_img(s, 0); % initialize img with dicm data type".

Moreover, the converter does not deal with multiple sessions as far as i can see. Is there a simple way to add sessions?
thanks
J.A

@xiangruili
Copy link
Owner

  1. Sessions: I am not aware of a reliable session parameter in dicom, so that is not implemented. If you know such a thing, let me know. Otherwise users have to deal with the session manually.

  2. Physio data: I don't think it makes sense to convert those into nifti. The converter should treat those as non-convertible files and ignore them. Apparently this is not the case for you. Would you mind sharing a file that gives the error?

@jamilaandoh
Copy link
Author

  1. ok
  2. agree that physio should not be converted to nifti. maybe saved as tsv? but my problem does not seem to be related to physio as i initially thought. the converter from rordenlab dcmniix works fine but the matlab one gives me this issue. i have no idea which file is problematic. the conversion starts fine then in the middle it just gives me the error i reported previously. i tried to send you a random file but the format (.dcm) is apparently not supported in this platform.

@jamilaandoh
Copy link
Author

jamilaandoh commented Mar 31, 2023 via email

@xiangruili
Copy link
Owner

@jamilaandoh Sorry I don't see the file.
You may use rename_dicm to rename the files, then it will be easier to tell which file is for physio.

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

No branches or pull requests

2 participants