View Issue Details

IDProjectCategoryView StatusLast Update
0003736Slicer4Core: Base Codepublic2018-03-02 11:06
Reporterspujol Assigned Tokentwilliams  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product VersionSlicer 4.3.1-2 
Target VersionSlicer 4.4.0Fixed in VersionSlicer 4.4.0 
Summary0003736: Does DWIConverter module write Nrrd file on disk ?
Description

I used the DWIConverter module to convert a DICOM dwi file to Nrrd.

The output dwi file didn't get written on disk once the conversion was finished.

To save the file, I used the standard File--> Save from the main menu.

What is the function of the 'Output Directory' entry in the 'DICOM to Nrrd parameters' ?

TagsNo tags attached.

Activities

jcfr

jcfr

2014-06-13 08:25

administrator   ~0012051

@Sonia: I associated the issue to Kent. Can you work with him on that ?

@kent: The issue is targeted for 4.4 .. is it something you will have time to address. If not we all add it to the backlog.

spujol

spujol

2014-06-13 09:24

developer   ~0012052

Last edited: 2014-06-13 09:26

Sure, I am happy to. Farukh can help with the debugging of the module. I have sent her a link to the bug report. Thanks!

kentwilliams

kentwilliams

2014-06-13 09:37

developer   ~0012053

I have never actually used DWIConvert from the Slicer GUI. Some observations:

The "Output Directory" parameter specifies the directory in which the NRRD file will be written. In general, this is a useless parameter; it makes more sense to put the full path in the output NRRD filename. "Output Directory" is there because it was there when I started work on DWIConvert and never changed or removed it.

If you do specify the full path for a NRRD filename, then specify OutputDirectory, then the write will probably fail, e.g. on our network if I saved on our network, if "Output Filename" is

/raid0/homes/kent/test.nrrd

If I mistakenly specified "Output Directory" as /raid0/homes/kent, then DWIConvert would try to write /raid0/homes/kent/raid0/homes/kent/test.nrrd which will fail.

I don't know if anyone else has mentioned this yet, but "File->Save As..." in Slicer has nothing to do with the DWIConvert program. I don't even know what it can save though I imagine it's the current MRML scene, not the NRRD file.

I can fix DWIConvert if someone can tell me what's actually needed. Take away the "Output Directory" parameter?

jcfr

jcfr

2014-06-13 09:48

administrator   ~0012054

@Kent: Thanks for the detailed report. To clarify, using "Save as" in Slicer you can save either the whole scene or any file independently.

@Sonia, Farukh: Can you work with Kent and the Slice community to define which parameter makes sense and if anybody is using this "mysterious" parameters.

Thanks and have a good weekend,
Jc

jcfr

jcfr

2014-07-20 12:18

administrator   ~0012209

Fixed in r23468
See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=2346

Related Changesets

Import 2017-06-07 23:51:09: master b1359c77

2014-07-20 16:07:26

jcfr

Details Diff
COMP: Provide Updates to BRAINSFit and DWIConvert

After the NAMIC meeting, several changes were made to address issues
with BRIANSFit and DWIConvert (Fixes 0003736).

Reference atlas is a new module and is not needed by Slicer.

From: Hans Johnson <hans-johnson@uiowa.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@23468 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild.cmake Diff File

Issue History

Date Modified Username Field Change
2014-06-13 07:35 spujol New Issue
2014-06-13 07:35 spujol Status new => assigned
2014-06-13 07:35 spujol Assigned To => jcfr
2014-06-13 08:24 jcfr Assigned To jcfr => kentwilliams
2014-06-13 08:24 jcfr Target Version => Slicer 4.4.0
2014-06-13 08:25 jcfr Note Added: 0012051
2014-06-13 09:24 spujol Note Added: 0012052
2014-06-13 09:26 spujol Note Edited: 0012052
2014-06-13 09:37 kentwilliams Note Added: 0012053
2014-06-13 09:48 jcfr Note Added: 0012054
2014-07-20 12:18 jcfr Note Added: 0012209
2014-07-20 12:18 jcfr Status assigned => resolved
2014-07-20 12:18 jcfr Fixed in Version => Slicer 4.4.0
2014-07-20 12:18 jcfr Resolution open => fixed
2017-06-10 08:51 jcfr Changeset attached => Slicer master b1359c77
2018-03-02 11:06 jcfr Status resolved => closed