View Issue Details

IDProjectCategoryView StatusLast Update
0002812Slicer4Module DICOMpublic2017-06-07 23:27
Reporterfedorov Assigned Topieper  
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Product VersionSlicer 4.2.0 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.3.0 
Summary0002812: Initialize default location for DICOM DB
Description

As discussed at RSNA2012, it would be probably more user-friendly if the database was initialized to some default location first time Slicer is launched.

TagsRSNA2012

Relationships

duplicate of 0002870 closedpieper DICOM Load Panel - Cancel button doesn't quit the panel 

Activities

pieper

pieper

2012-12-11 23:32

administrator   ~0007485

By default, should probably use the Qt standard method for storing application data in a cross platform manner.

See QDesktopServices - should DICOM data be in DataLocation or in DocumentsLocation?

lassoan

lassoan

2012-12-18 10:04

developer   ~0007538

We recently gave a Slicer training to about 10 students and this was a major inconvenience. Users didn't understand why the popup is displayed, some people just wanted to get rid of it and were very pissed because hitting cancel didn't do anything (the popup came back, couldn't close the popup in any way, couldn't even close Slicer), while other thought that they should give the location of their DICOM files (also very bad, imported DICOM files were mixed with input DICOM files).

So, it's important to initialize the DICOM DB location by default. By default the the database could be in the same folder as the Slicer.ini file (maybe in a DicomDatabase subfolder).

fedorov

fedorov

2012-12-18 10:19

developer   ~0007540

Andras: this describes our experience at RSNA pretty well ...

pieper

pieper

2012-12-18 10:33

administrator   ~0007541

I think this belongs in the Documents directory. I don't like the idea of putting it into a hidden directory like .config or .local since it can be a fairly large, and actually important directory that shouldn't be hidden. Particularly since Ron requested that Copy On Import be the new default setting.

I agree about the 'can't cancel' issue - I'll see about fixing that.

fedorov

fedorov

2012-12-18 12:23

developer   ~0007550

I agree with Steve that Documents is a better location. I think this is the location where OxiriX puts its database directory by the way.

pieper

pieper

2013-01-16 09:40

administrator   ~0007685

With this a default location is selected to start up.

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21524

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 7fa89eb0

2012-12-20 17:02:21

pieper

Details Diff
BUG: 0002812 start with default database directory

Use Qt to specify default location in a platform-appropriate
manner.

Also some minor style cleanups or trailing spaces.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21524 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/Scripts/DICOM.py Diff File

Issue History

Date Modified Username Field Change
2012-12-03 06:19 fedorov New Issue
2012-12-03 06:19 fedorov Status new => assigned
2012-12-03 06:19 fedorov Assigned To => pieper
2012-12-03 06:19 fedorov Tag Attached: RSNA2012
2012-12-08 09:34 jcfr Target Version => Slicer 4.2.3
2012-12-11 23:32 pieper Note Added: 0007485
2012-12-18 10:04 lassoan Note Added: 0007538
2012-12-18 10:19 fedorov Note Added: 0007540
2012-12-18 10:33 pieper Note Added: 0007541
2012-12-18 12:23 fedorov Note Added: 0007550
2013-01-16 09:40 pieper Note Added: 0007685
2013-01-16 09:40 pieper Status assigned => resolved
2013-01-16 09:40 pieper Fixed in Version => Slicer 4.2.3
2013-01-16 09:40 pieper Resolution open => fixed
2013-01-16 09:40 pieper Relationship added duplicate of 0002870
2013-02-10 03:31 fedorov Status resolved => closed
2013-02-12 09:39 jcfr Target Version Slicer 4.2.3 => Slicer 4.3.0
2013-02-12 09:40 jcfr Fixed in Version Slicer 4.2.3 => Slicer 4.3.0
2017-06-07 23:27 pieper Changeset attached => Slicer 2145-support-for-installing-extension-from-file 7fa89eb0