]> Creatis software - gdcm.git/blobdiff - TODO
ENH: If possible move the stl include after the gdcm ones, to avoid vc warnings
[gdcm.git] / TODO
diff --git a/TODO b/TODO
index 0ad762c9b76a1ef581e6566ee39403d25066ddbb..f410e287caa1226334df139b34f67bb91e7fb0bc 100644 (file)
--- a/TODO
+++ b/TODO
@@ -13,6 +13,27 @@ Comments:
 
 -----------------------------------------------------------------------------
 -----------------------------------------------------------------------------
+Description: vtk/vtkGdcmReader doesn't positionate the normal to the image
+Date: 2004 Oct 1
+Attributed:
+Details: vtkGdcmReader::CheckFileCoherence() sets the DataOrigin[i]
+         but doesn't set the plane (image seen in 3D) normal (is it
+         possible any how). This plane normal could be extracted from 
+         the "orientation" info of the gdcmHeader ( refer to
+         grep "Orientation" Dicts/dicomV3.dic).
+         Problem exhibiting this defect: cine loop on a pile of images
+               whose Origin is correct, but whose normal is not set will
+               plainly suck !
+Comments:
+        * vtkGdcmReader inherits from vtkImageReader which aggregates
+          a vtkTranform. vtkGdcmReader could store (when the user requires
+          it, see below) the origin/normal taken from the Dicom Header
+          within this vtkTransform (looks like a natural place to store
+          this spacial information).
+        * Both settings of the origin and/OR the normal of the plane (image)
+          should be an option defined with a flag (On/Off) in the
+          vtkGdcmReader...
+-----------------------------------------------------------------------------
 Description: Fix the Python wrappers
 Date: 2004 Sep 24
 Attributed: no
@@ -122,12 +143,26 @@ Comments:
      What should the wrapper do in such a case !?
  * Frog: what does VM stand for ?
 -----------------------------------------------------------------------------
+Description: Add information on supported imagers (constructor/model)
+Date: 2004 9 7
+Attributed:
+Details: in order to promote gdcm make a list (on the web pages)
+         of images successfully parsed based on a constructor/model ordering
+Comments: * frog: gdcmData only lists pathological images. How to collect
+    the ones gdcm works smoothly with (hopefully gdcmData is a small
+    subset of what we would like).        
+-----------------------------------------------------------------------------
+Description: Add a GetVersion() global function.
+Date: 2003 july 7
+Attributed:
+Details: This is to be used for version assertion with gdcmPython
+Comments:
+-----------------------------------------------------------------------------
+-----------------------------------------------------------------------------
 * vtk/vtkGdcmHeader.cxx: if speed becomes a concern some changes can
   be made at the cost of memory consumption (refer to header of 
   vtk/vtkGdcmHeader.cxx)
 -----------------------------------------------------------------------------
-* Add a GetVersion() global function.
------------------------------------------------------------------------------
 * gdcmElValSet::SetElValueLengthByNumber IMNSHO should be trashed.
   It's only purpose is a onliner substitute to calling GetElValueByNumber
   and then SetLength. This only obfuscates the caller code more than
@@ -147,7 +182,7 @@ Comments:
   would allow those method to avoid artificial calls to ElValue::GetElement(),
   ElValue::GetVR()...
 -----------------------------------------------------------------------------
-* Eat leanding_trailing_whitespace (found in python/gdcmPython/gdcmi) should
+* Eat leading_trailing_whitespace (found in python/gdcmPython/gdcmi) should
   be used when parsing the dictionary in C++ !
 -----------------------------------------------------------------------------
 * Group length is not a unique tag in a file. Hence avoid putting it