Good morning folks, <br><br>Could somebody consider the remark of Dragan ? <br><br>May be multiple issue should be entered in the issue tracker, prioritized and assigned to the appropriate people. <br><br>I let the ctkDICOM guru chime in ... :) <br>

<br>Thanks<br>Jc<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">toromand</b> <span dir="ltr"><<a href="mailto:reply%2Bi-842542-9e237d982bb977f5158e4336e117d72ea9fe04af@reply.github.com">reply+i-842542-9e237d982bb977f5158e4336e117d72ea9fe04af@reply.github.com</a>></span><br>

Date: Tue, May 3, 2011 at 6:12 AM<br>Subject: [GitHub] Retrieve (and Database) bugs [commontk/CTK GH-16]<br>To: <a href="mailto:jchris.fillionr@kitware.com">jchris.fillionr@kitware.com</a><br><br><br>- The network connection in ctkDICOMRetrieve is never "dropped". Consequent calls to the retrieve function ends up with the message that "The network address is already in use" as the association is never closed.<br>


Should include call to closeAssociation on several places. It would be even better to use the member SCU (narked as to-do) in the source, to make it even better. Special care should be taken on exception when called from "outside".<br>


- Retrieved object are written to the disk two times. The portion of code "borrowed" from the dcmdjpeg example should not exist in the retrieve code, because the data (according to the design) should be and is written in the database.insert().<br>


<br>
- Also, in ctkDICOMDatabase, there is a problem with retrieved object paths/filenames. The present system of using UIDs is not compliant with the DICOM standard, especially if you would like to create (proper) DICOMDIR and archive media. Paths and filenames should be generated to have max 8 characters according to the DICOM standard (check with Clunie if you don't beleive me :) )<br>


- Different behavior of in-memory and file database should not be implemented (at least not like that) in the Core library code. One should expect that the call to a function (insert()) should depend on parameters only.<br>


<br>
- You should also correct the calls to database.insert() to reflect the needed use.<br>
For instance, in ctkDICOMQuery, the call to database.insert() is missing "false, false" (causing some side-efects, like trying to generate thumbnails for non-existent images)<br>
<br>
Enough for this post :)<br>
Keep the good work,<br>
Dragan Toroman<br>
<font color="#888888"><br>
--<br>
Reply to this email directly or view it on GitHub:<br>
<a href="https://github.com/commontk/CTK/issues/16" target="_blank">https://github.com/commontk/CTK/issues/16</a><br>
</font></div><br><br clear="all"><br>-- <br>+1 919 869 8849<br><br>