View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002605 | Slicer4 | Core: Scripting (Wrapping, Python) | public | 2012-10-03 08:46 | 2014-03-06 06:09 |
Reporter | Assigned To | pieper | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.1.1 | ||||
Target Version | Slicer 4.2.0 | Fixed in Version | Slicer 4.2.0 | ||
Summary | 0002605: Windows: DICOMS + SaveSceneToSlicerDataBundleDirectory() yields 0kb .nrrd file | ||||
Description | When a DICOM set is loaded and SaveSceneToSlicerDataBundleDirectory() is called, the .nrrd file that gets written in the <scene>/Data/ directory is 0kb in size. Per the thread below, it has to do with how Windows interprets certain characters in the filename. | ||||
Additional Information | Conversation with Steve Pieper: Ah - interesting - I think you found a windows-specific bug in the data bundle. Since the DICOM module prefixes the loaded volumes with the series number followed by a colon, the name of the volume node isn't always a valid windows path - in this case it is mistaken for a drive letter but that only happens for certain dicom series names. I'll have to look into this and see if there's good generic fix for this. If you have a chance, could you file a bug report on this and send me the link? I've got to run now but should be able to clean it up pretty quick once I have a minute. -Steve On Tue, Oct 2, 2012 at 2:59 PM, Kumar, Sunil <kumars@mir.wustl.edu> wrote:
| ||||
Tags | No tags attached. | ||||
The issue is that the MRB save function needs to make filenames for the new storage nodes it creates in the tmp directory. It uses the volume name as the basis filename. After looking into this, I think the best solution is to url encode the filename, so that any non-letter/number characters are replaced with %HH where HH is the hex code for the character. This won't change the name of the volume in the slicer application, just the autogenerated filename, so users won't see this unless they decide to save the volume out through another mechanism. |
|
This commit should take care of it: http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21184 Sunil - can you test it out with your module and if it looks good, close this issue. If it doesn't work, just let me know and we'll dig deeper. -Steve |
|
Works well! Thanks so much. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2012-10-03 08:46 |
|
New Issue | |
2012-10-03 08:46 |
|
Status | new => assigned |
2012-10-03 08:46 |
|
Assigned To | => jcfr |
2012-10-03 10:04 | jcfr | Assigned To | jcfr => pieper |
2012-10-03 10:04 | jcfr | Target Version | => Slicer 4.2.0 - coming release |
2012-10-06 05:44 | pieper | Relationship added | related to 0002366 |
2012-10-06 05:49 | pieper | Note Added: 0006420 | |
2012-10-15 06:29 | pieper | Note Added: 0006528 | |
2012-10-15 06:29 | pieper | Status | assigned => resolved |
2012-10-15 06:29 | pieper | Resolution | open => fixed |
2012-10-15 12:04 |
|
Note Added: 0006545 | |
2012-10-15 12:04 |
|
Status | resolved => closed |
2014-03-06 06:09 | jcfr | Fixed in Version | => Slicer 4.2.0 |