Is somebody looking into that issue ? <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-819199-208fc263772333bb5a07781cdaa80466812e9900@reply.github.com">reply+i-819199-208fc263772333bb5a07781cdaa80466812e9900@reply.github.com</a>></span><br>

Date: Wed, Apr 27, 2011 at 7:29 AM<br>Subject: [GitHub] modalitiesInStudy not read correctly and more Modality related issues [commontk/CTK GH-15]<br>To: <a href="mailto:jchris.fillionr@kitware.com">jchris.fillionr@kitware.com</a><br>

<br><br>1.<br>
in CTK/Libs/DICOM/Core/ctkDICOMDatabase.cpp<br>
the tag DCM_ModalitiesInStudy is not read correctly<br>
Instead of:<br>
dataset->findAndGetOFString(DCM_ModalitiesInStudy, modalitiesInStudy);<br>
it should be:<br>
dataset->findAndGetOFStringArray(DCM_ModalitiesInStudy, modalitiesInStudy);<br>
to allow reading multiple modalities (XX\YY\ZZ)<br>
<br>
2.<br>
Also, you might consider more "standard" way of performing the query with the modalities filter. Although the fact that many SCP implement search by the ModalitiesInStudy tag, it is not according to the DICOM standard and is rather a IHE extension with undefined behavior when used as a search criterion (well, it works well with dcm4chee). So it might now work with all DICOM compliant SCPs. The "proper" way (according to DICOM specifications) would be to perform a query for all needed modalities and combine the results.<br>


<br>
3. Rather more important issue is that the current database design lacks a VERY important thing. In DICOM, modalities are part of Series, not the Study, and the ModalitiesInStudy is "informative" and new feature that you should use on Study level for information on the Study, but in your current implementation, you don't have the information on the actual modality of every series. This is a very important information and it is missing although it exists in the dataset. So at present with (1) not fixed you get for study with 2 modalities (XA/OT)<br>


STUDY (XA)<br>
 - SERIES 1<br>
 - SERIES 2<br>
instead of<br>
STUDY (XA/OT)  -----> *with (1) fixed<br>
 - SERIES 1 (XA)<br>
 - SERIES 2 (OT)<br>
<br>
Regards<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/15" target="_blank">https://github.com/commontk/CTK/issues/15</a><br>
</font></div><br><br clear="all"><br>-- <br>+1 919 869 8849<br><br>