[Dcmlib] [Gdcm2] Re: [Gdcm-cvs] gdcm TODO
Mathieu Malaterre
mathieu.malaterre at gmail.com
Mon Jan 9 17:05:28 CET 2006
J'ai mis la terminology histoire de rentre consistant gdcm. Je trouve que
c'est aussi important pour que l'on se mette d'accord sur le nom des
classes. Si on est d'accord sur la termino alors le nom des classes
devraient etre deduit facilement.
Sinon tu as raison gdcm2 fais un melange entre Read/Load sur les sequences.
Apres discussion avec Kitware, c'est assez mauvais comme design(*) de mettre
les methodes Read/Load dans l'objet. Il me faudra externaliser tout le code
pour faire une classe identique a gdcm::Parse pour faire un gdcm::Read et un
gdcm::Skip ...
Tu as aussi raison pour les info minimales. Un gdcm::ImageReader aura besoin
d'un gdcm::Parser (FileSeeker) pour acceder au groupe 0028. Cette remarque
devrait etre dans l'ImageReader, si c'est pas clair je peux mettre plus de
details. J'aime toujours pas le AddForceLoad, surtout qd dans notre cas
c'est linverse : par defaut on ne lit rien (Parsing). Uniquement une classe
specialiser pour les DICOMDIR pourra demander des infos precises
(intelligentes).
Mathieu
(*) Read est different selon que l'on lit du DICOM ou ACR NEMA. Faire une
classe laisse aussi l'option aux advanced users de faire une sous classes
plus intelligente...
On 1/6/06, Jean-Pierre Roux <jpr at creatis.insa-lyon.fr> wrote:
>
> Salut, Mathieu.
>
> --> Veux-tu dire que ces options existent, ou bien que c'est qq chose
> qui devra etre ajouté?
>
> >+
> >+ TERMINOLOGY:
> >+ - SKIP: DICOM has Group Length which allow us to skip elements
> >+ - Seek: Read a Group/Element+Length and then *seek* through an element
> >+ - Parse: Browse through a DICOM file without loading any information
> other than the minimum need for it's parsing (almost nothing for ACR, and
> group 0002 for DICOM)
> >+ - Load: Actually load each and every single element of a DICOM file.
> Very usefull for a PRINT operation
> >+ - DUMP/PRINT: Operation that consist of displaying all information (up
> to a certain level of detail) contained in a DICOM file
> >
> >
> >
> >
> -->J'ai modifie l'operateur << de Value pour qu'il affiche, lorsque
> c'est possible la valeur des DataElement.
> L'option par defaut est donc actuellement Load, c'est ca?
>
> --> Je n'ai pas trouve ce qu'il fallait faire, (ou si ca existe) pour
> charger (recursivement) une SequenceOfItems.
> Ca n'est pas fait par defaut, n'est ce pas?
> C'est une bonne option, car dans 99% des cas, on se tape de leur contenu.
> Si l'utilisateur en a besoin (par exemple 'Icon Image Sequence'), il
> devrait pouvoir faire qq chose comme 'ForceLoad(tag)'.
> Dans le cas d'un DICOMDIR, *toutes* les info pertinentes sont dans une
> seule SequenceOfItems. Il est donc imporatnt qu'on puisse la charger de
> maniere simple.
>
> --> En plus du Parse (qui ne charge rien), il faudrait une métode qui
> charge uniquement les 10 elements strictement necessaires pour la
> lecture des pixels d'une image,
> a savoir :
> imageElem 0028 0002 "1" // Samples per Pixel
> imageElem 0028 0004 " " // Photometric Interpretation
> imageElem 0028 0006 "0" // Planar Configuration
> imageElem 0028 0008 "1" // Number of Frames
> imageElem 0028 0010 "0" // Rows
> imageElem 0028 0011 "0" // Columns
> imageElem 0028 0030 "1.0\1.0 " // Pixel Spacing
> imageElem 0028 0100 " " // Bits Allocated
> imageElem 0028 0101 " " // Bits Stored
> imageElem 0028 0103 " " // Pixel Representation
>
> et une autre qui permette de demander a l'utilisateur de preciser
> *quels* DataElements il veut qu'on charge (Pour creer un DicomDir, par
> exemple, ou pour avoir les info necessaires a un browser d'images
> évolué, il y a 40 elements utiles (jamais vu personne qui a besoin
> d'autre chose, mais si l'utilisateur en veut 3 autres, il dit
> 'AddForceLoad(tag)', et son pb est resolu.
>
> 0010 0010 "" // Patient's Name
> 0010 0020 "" // Patient ID
> 0010 0030 "" // Patient's Birthdate
> 0010 0040 "" // Patient's Sex
> 0008 0020 // Study Date // Type 1C
> Conditional Element
> 0008 0030 // Study Time // Type 1C
> Conditional Element
> 0008 0050 // Accession Number // Type 1C
> Conditional Element
> 0008 1030 "" // Study Description
> 0020 000d "" // Study Instance UID :
> 0020 0010 "" // Study ID :
> 0008 0021 "" // Series Date
> 0008 0031 "" // Series Time
> 0008 0060 "" // Modality
> 0008 0080 "" // Institution Name :
> 0008 0081 "" // Institution Address :
> 0008 1010 "" // Station Name
> 0008 1050 "" // Performing Physician's Name
> 0008 103e "" // Series Description :
> 0018 1030 "" // Protocol Name
> 0020 000e "" // Series Instance UID :
> 0020 0011 "0" // Series Number :
> 000 1512 "" // Transfer Syntax
> 000 0008 "" // Image Type
> 0008 0016 "" // SOP Class UID :
> 0008 0018 "" // SOP Instance UID :
> 0008 0023 "" // Content Date
> 0008 0033 "" // Content Time
> 0018 1060 "0" // Trigger Time
> 0020 0013 "0" // Image Number
> 0020 0032 "1.0\1.0\1.0 " // Image Position Patient
> 0020 0037 "1\0\0\0\1\0 " // Image Orientation Patient
> 0020 0052 "0" // Frame of Reference UID
> 0020 1041 "0.0" // Slice Location
> 0020 4000 "" // Image Comments
> 0028 1052 "0 " // Rescale Intercept
> 0028 1053 "1 " // Rescale Slope
> 0050 0004 "0" // Calibration Image
>
> JP
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.creatis.insa-lyon.fr/pipermail/dcmlib/attachments/20060109/ab3e751c/attachment.html>
More information about the Dcmlib
mailing list