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

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

Open
fedorov opened this issue Aug 8, 2018 · 4 comments

Comments

@fedorov
Copy link

fedorov commented Aug 8, 2018

As discussed in QIICR/dcmqi#348 (comment), the output produced does not pass the validation by dciodvfy. See the referenced issue for details and the dataset illustrating the problem.

image

@drdanz
Copy link
Member

drdanz commented Sep 6, 2018

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
Copy link
Author

fedorov commented Sep 6, 2018

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

@mmromero
Copy link

mmromero commented Sep 6, 2018

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
Copy link
Author

fedorov commented Sep 6, 2018

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

input_data.zip

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

3 participants