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

Philips Multiframe with ADC isn't truncated appropriately #122

Open
ma-tripleT opened this issue Aug 5, 2022 · 0 comments
Open

Philips Multiframe with ADC isn't truncated appropriately #122

ma-tripleT opened this issue Aug 5, 2022 · 0 comments

Comments

@ma-tripleT
Copy link

Hi Team,

For Philips Multiframe diffusion images that have ADC, in convert_philips.py the function _create_bvals_bvecs() makes a call out to _fix_diffusion_images()for the purposes of removing the ADC, and a truncated nifti is returned.

However _create_bvals_bvecs() doesn't return this truncated file back up to_multiframe_to_nifti() which means the NIfTI that is saved will be one frame longer than the BVAL and BVEC files.

This could easily be fixed with the addition of nifti being returned at line 535 and an additional variable output_file to line 271.

Thanks,
Matthew

ma-tripleT added a commit to ma-tripleT/dicom2nifti that referenced this issue Jun 25, 2024
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

1 participant