rordenlab / dcm2niix

dcm2nii DICOM to NIfTI converter: compiled versions available from NITRC
https://www.nitrc.org/plugins/mwiki/index.php/dcm2nii:MainPage
Other
854 stars 226 forks source link

Error in bvec extraction when UIH diffusion data is converted to nifti #836

Closed XYWCQM closed 1 month ago

XYWCQM commented 1 month ago

Hello: When I converted the UIH diffusion data using the latest Releases version (version 2-February-2024) of dcm2niix, I found that the generated bvec file was reading from the (0018,9089) tag and not from (0065,1037). The correct bvec for UIH should be read from (0065,1037), because the bvec in there takes into account the true orientation of the rotated orientation frame, whereas the (0018,9089) tag holds the original, unrotated orientation. PS: I tested it with a previous version of dcm2niix and its extracted bvec is read from (0065,1037).

neurolabusc commented 1 month ago

Please work with UIH to resolve this. 0018,9089 is a public tag with a clearly defined interpretation. dcm2niix assumes that DICOM images are truthful. If this is not the case, your images are in violation of the DICOM standard. The manufacturer has an obligation to meet their DICOM conformance statement. While manufacturers are allow to define private tags as they wish, they are required to obey the standard if they employ public tags.

While we could add a kludge to dcm2niix to handle errors, I am reluctant to do so. The resulting images would break other tools, and dcm2niix would fail when the vendor updates their software to match the DICOM specification.

XYWCQM commented 1 month ago

Thanks, I'll try to talk to the manufacturer.

neurolabusc commented 1 month ago

I have added a kludge that gives precedence to the private tag if the manufacturer is UIH. However, it would be great to see public tags providing correct values to ensure compatibility with other tools.