biolab-unige / nifti2dicom

Convert 3D NIfTI images (and other formats supported by ITK) to DICOM 2D series
http://www.bio.dist.unige.it/
GNU General Public License v3.0
87 stars 23 forks source link

Trailing characters make the output dataset non-compliant with the standard #33

Open fedorov opened 6 years ago

fedorov commented 6 years ago

As discussed in https://github.com/QIICR/dcmqi/issues/348#issuecomment-411106796, the output produced does not pass the validation by dciodvfy. See the referenced issue for details and the dataset illustrating the problem.

image

drdanz commented 5 years ago

Hello @fedorov and thanks for reporting the issue, sorry for the delay but I was on holiday.

From the linked issue, it looks like that a very old version for macOS, using a very outdated ITK version was used, I just uploaded an updated version here: https://github.com/biolab-unige/nifti2dicom/releases/tag/v0.4.11 (still a bit old, but this is the most recent build that I have available at the moment). It would be really useful if you could try with this version that uses more recent ITK and GDCM.

Also does this happen with a specific dataset or with any dataset? Can you provide me the dataset linked in the issue? The link is no longer valid.

Thanks.

fedorov commented 5 years ago

@mmromero reported this issue, so hopefully he can respond. I myself have not used your tool, just tried to help him.

mmromero commented 5 years ago

Hi @drdanz I used the version available in http://www.bio.dist.unige.it/projects/national/n2d to transform the nifti files to dicoms, so I could later use them with dcmqi. The problem happened for several datasets, so I finally used Slicer to export the nifti files to dicoms, and that worked. I don't have the dicoms created with nifti2dicom anymore. Let me know if I can be of any help.

fedorov commented 5 years ago

I found the dataset @mmromero shared earlier, see attached.

input_data.zip