It seems that if ctkDICOMDatabase is in CTKCore, then it doesn't need thumbnails.<div>Maybe ctkDICOMDatabase could be overloaded in CTKWidget and handles thumbnails there.</div><div>Julien.<br><br><div class="gmail_quote">
On Thu, Jun 9, 2011 at 1:44 PM, Steve Pieper <span dir="ltr"><<a href="mailto:pieper@ibility.net">pieper@ibility.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Julien -<br>
<br>
The idea here is that the ctkDICOMDatabase should be able to<br>
automatically make the thumbnail images (png) on the fly as dicom<br>
files are added. ctkDICOMImage uses the QImage class to do this,<br>
which is in QtGui even though it seems to me to be a purely data<br>
formatting class, not a display class. Even though it's still being<br>
debugged, I'm planning to make use of the thumbnail feature in<br>
slicer4's use of ctkDICOM.<br>
<br>
Can you think of a way we can encapsulate the thumbnail calculation in<br>
ctkDICOMCore? Perhaps we should make an abstract thumbnail generator<br>
class in ctkDICOMCore and then have a QImage-based implementation in<br>
ctkDICOMWidgets that is registered with ctkDICOMDatabase at run time.<br>
Alternatively perhaps ctkDICOMDatabase could link directly to the png<br>
library and make the thumbnails with that.<br>
<font color="#888888"><br>
-Steve<br>
</font><div><div></div><div class="h5"><br>
<br>
On Thu, Jun 9, 2011 at 1:16 PM, Julien Finet <<a href="mailto:julien.finet@kitware.com">julien.finet@kitware.com</a>> wrote:<br>
> Hi Marco/Steve,<br>
> ctkDICOMDatabase is in Libs/DICOM/Core which means it shouldn't depend on<br>
> QtGUI, however, it internally uses ctkDICOMImage which is QtGUI depend (and<br>
> needs to be moved in Libs/DICOM/Widgets).<br>
> Is ctkDICOMImage necessary in ctkDICOMDatabase? Right now, it seems to be<br>
> just a test. Are you ok if I remove it ?<br>
> Thanks,<br>
> Julien.<br>
</div></div></blockquote></div><br></div>