[Ctk-developers] ctkDICOMImage in ctkDICOMDatabase

[Julien Finet]

It seems that if ctkDICOMDatabase is in CTKCore, then it doesn't need
thumbnails.
Maybe ctkDICOMDatabase could be overloaded in CTKWidget and handles
thumbnails there.
Julien.

On Thu, Jun 9, 2011 at 1:44 PM, Steve Pieper <pieper at ibility.net> wrote:

> Hi Julien -
>
> The idea here is that the ctkDICOMDatabase should be able to
> automatically make the thumbnail images (png) on the fly as dicom
> files are added.  ctkDICOMImage uses the QImage class to do this,
> which is in QtGui even though it seems to me to be a purely data
> formatting class, not a display class.  Even though it's still being
> debugged, I'm planning to make use of the thumbnail feature in
> slicer4's use of ctkDICOM.
>
> Can you think of a way we can encapsulate the thumbnail calculation in
> ctkDICOMCore?  Perhaps we should make an abstract thumbnail generator
> class in ctkDICOMCore and then have a QImage-based implementation in
> ctkDICOMWidgets that is registered with ctkDICOMDatabase at run time.
> Alternatively perhaps ctkDICOMDatabase could link directly to the png
> library and make the thumbnails with that.
>
> -Steve
>
>
> On Thu, Jun 9, 2011 at 1:16 PM, Julien Finet <julien.finet at kitware.com>
> wrote:
> > Hi Marco/Steve,
> > ctkDICOMDatabase is in Libs/DICOM/Core which means it shouldn't depend on
> > QtGUI, however, it internally uses ctkDICOMImage which is QtGUI depend
> (and
> > needs to be moved in Libs/DICOM/Widgets).
> > Is ctkDICOMImage necessary in ctkDICOMDatabase? Right now, it seems to be
> > just a test. Are you ok if I remove it ?
> > Thanks,
> > Julien.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20110609/16fd7ee6/attachment.html>