Mantis Bugtracker

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0003023 [Slicer4] Module Diffusion major always 2013-03-20 10:43 2013-04-05 12:17
Reporter pieper View Status public  
Assigned To hjmjohnson
Priority normal Resolution fixed  
Status closed   Product Version
Summary 0003023: DWI convert doesn't work as a shared library
Description Running the converter requests a file name, but since the CLI is built as a shared library the output volume is being passed as an encoded pointer -- but the data is written to a file anyway.

One possible fix is to specify EXECUTABLE_ONLY in cmake.

Another is to fix the module to write through the pointer, which should be doable since the older converter did this.
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0008182)
inorton (developer)
2013-03-20 18:52

Steve, does it write a usable file anywhere? When I run with 2013-02-24 or 2013-03-19 nightlies, DWIConverter produces a 0-byte file in the selected output directory; and there is no data written to the DWINode in Slicer either.
(0008183)
pieper (administrator)
2013-03-21 07:35

Yes, I was able to get a dwi nrrd file in the selected output directory but the filename was the embedded pointer name, something like slicer:0x015215... but when I changed the name to dwi.nrrd I was able to load it and estimate the tensors (the gradients were not exactly right, but that's a different issue).

This is the first time I'd tried DWI convert so I don't know how it works on other data.
(0008184)
inorton (developer)
2013-03-21 09:53

Thanks, I was using DWI-DTI data and didn't get that far, just got the 0-byte file called "slicer". I'm on Windows 7 x64. For the gradients, it might be worth trying the "use B-matrix" option if it is Siemens DWI data.
(0008191)
pieper (administrator)
2013-03-21 11:16

Ah - makes sense - the : breaks the path on windows (I was testing on mac).

I'll pass on the B-matrix info to Maarten and maybe that will fix his issue.
(0008248)
pieper (administrator)
2013-03-27 08:30

Ron reports that importing DWI through the DICOM module is also not working. This would be a side-effect of the fact that DWIConvert is not working.
(0008249)
hjmjohnson (developer)
2013-03-27 08:36

Steve,

We know what the problem is, we just need to figure out how to integrate with Slicer. DWIConvert is being built as a shared library, and that requires that the ITK image IO be used for writing images. This is not possible because we need to write out all the specialized gradient information, so we write a compliant NRRD file out to disk manually.

We are investigating what is necessary to pass all the necessary information to ITK in order to write out all the necessary meta data for compliant and complete DWI NRRD files.

I think the problem was that DICOMToNRRD was only built as a stand alone application, and DWIConvert is being built as a shared library.

==========
Would you be able to run a test for us?

1) Remove the DWIconvert shared library, and attempt to use the slower command line interface to load a DWI data set?

Thanks,
Hans
(0008259)
hjmjohnson (developer)
2013-03-28 09:09

BRAINSTools/DWIConvert/CMakeLists.txt was modified so that EXECUTABLE_ONLY flag is used when building the SEM packages. This disables the failing shared library that was superceeding the execution of the command line version.
(0008320)
pieper (administrator)
2013-04-05 12:17

The executable only flag does the trick. I've tested this and it works when invoked from the dicom diffusion importer.

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21862 [^]

- Issue History
Date Modified Username Field Change
2013-03-20 10:43 pieper New Issue
2013-03-20 10:43 pieper Status new => assigned
2013-03-20 10:43 pieper Assigned To => hjmjohnson
2013-03-20 18:52 inorton Note Added: 0008182
2013-03-21 07:35 pieper Note Added: 0008183
2013-03-21 09:53 inorton Note Added: 0008184
2013-03-21 11:16 pieper Note Added: 0008191
2013-03-27 08:30 pieper Note Added: 0008248
2013-03-27 08:36 hjmjohnson Note Added: 0008249
2013-03-28 09:09 hjmjohnson Note Added: 0008259
2013-03-28 09:09 hjmjohnson Status assigned => resolved
2013-03-28 09:09 hjmjohnson Resolution open => fixed
2013-04-05 12:17 pieper Note Added: 0008320
2013-04-05 12:17 pieper Status resolved => closed
2013-04-05 12:17 pieper Fixed in Version => Slicer 4.3.0


Mantis 1.1.4[^]
Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker