]> Creatis software - gdcm.git/blobdiff - TODO
ENH: * Finished lossless transition, not only do we now read all lossless jpeg
[gdcm.git] / TODO
diff --git a/TODO b/TODO
index 6481daf15a4db42cec6bfea6195e163f54e9f213..51890575e48d2a14aa0a59b1765eb3623a70dd14 100644 (file)
--- a/TODO
+++ b/TODO
@@ -12,6 +12,81 @@ Comments:
 
 
 -----------------------------------------------------------------------------
+Description:  gdcmJpeg8 is strictly a copy/paste of gdcmJpeg12.cxx.
+Date: 2004 Oct 13
+Attributed: 
+Details:
+We should write the code in a common place, then include this 'cxx' file so the the define from gdcm_mangle redefine to the proper one.
+Comments:
+This will be usefull since I may need in the future a 16bits version of this reading
+-----------------------------------------------------------------------------
+Description: Change jpeg 'exit' call to standard c++ exception using the jpeg error
+handler
+Date: 2004 Oct 8
+Attributed: 
+Details:
+Comments:
+-----------------------------------------------------------------------------
+Description: Add kwsys as a subdir somewhere in gdcm
+Date: 2004 Oct 8
+Attributed: Mathieu
+Details:
+Comments:
+-----------------------------------------------------------------------------
+Description: ljpeg
+Date: 2004 Oct 8
+Attributed: Mathieu
+Details:
+Comments: ljpeg was rip from medcon and not the official one. medcon tried to
+optimised function using MACRO (doh!), so it make its very unreadable and very
+hard to fix warnings. Should go back to official source, copy proper copyright
+and fix warnings on dashboard
+13/10: update apparently no dicom toolkit use this lib as it is too buggy. We should use the ls-patch for ijg instead. Thus we can safely get rid of that lib.
+14/10: PHILIPS_Gyroscan-12-MONO2-Jpeg_Lossless.dcm prove that I was right the old Cornwell lib is buggy and does not read anything.
+-----------------------------------------------------------------------------
+Description: Extent reading support
+Date: 2004 Oct 8
+Attributed: Mathieu
+Details:
+Comments: All ITK/VTK readers support selecting extent. gdcm should support selecting
+extent before being inserted into ITK
+-----------------------------------------------------------------------------
+Description: Generate new UID each time we write DICOM
+Date: 2004 Oct 8
+Attributed: Mathieu
+Details:
+Comments: According to DICOM ref a new UID should be created each we write a
+DICOM images. I guess it should be an option so that we can still use md5sum to
+check dicom file. The proposed way was:
+http://www.creatis.insa-lyon.fr/pipermail/dcmlib/2004-September/000611.html
+
+Bah, comme Win32 pose encore pb:
+  echo "gdcm" | od -b
+  0000000 147 144 143 155 012
+et si on prenait:
+ radical + 147.144.143.155 + IP + time()
+
+-----------------------------------------------------------------------------
+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
@@ -62,13 +137,15 @@ Description: fix definitively the memory leaks problems.
 Date: 2004 Sep 24
 Attributed:
 Details:
-Comments:
+Comments: There is a nightly dashboard that run valgrind every night
+(zorglub | GDCM-Linux-g++)
 -----------------------------------------------------------------------------
 Description: test gdcm on a big endian OS.
 Date: 2004 Sep 24
 Attributed:
 Details:
-Comments:
+Comments: There is a nightly dashboard that run on MacOSX each nite
+(midworld.kitwarein | GDCM-DarwinG5-g++ )
 -----------------------------------------------------------------------------
 Description: More tests !
 Date: 2004 Sep 24
@@ -82,10 +159,14 @@ Comments:
 Description: revoir la gestion des resources, win32 permet d'inclure des
    fichiers texte (=dicomV3.dic) dans une dll ou quelquechose du genre.
 Date: 2004 Sep 24
-Attributed:
+Attributed: Mathieu
 Details:
 Comments:
  * Frog: no comprendo !? De plus est-ce portable ?
+ * To improve load time it could be usefull to have the dictionary directly in
+ 'c++' code or in a more binary format.
+ * This will also solve some issues where /dummy/ user did nor set
+ GDCM_DICT_PATH properly neither 'make install'
 -----------------------------------------------------------------------------
 Description: Convert the C-like IO to C++ IO:
 Date: 2004 Jul 30
@@ -100,6 +181,9 @@ Comments:
    build or pass the proper FILE* to jpeg libs ?
  * Binary IO references are available at
    http://www.angelfire.com/country/aldev0/cpphowto/cpp_BinaryFileIO.html
+ * The internal API should be rewritten so that gdcm speaking to jpeg lib is
+   done with stream/string and not directly opened FILE*
+ * No stdio.h anymore anywhere tolerated !
 -----------------------------------------------------------------------------
 Description: generate methods based on VM.
 Date: 2004 Jul 30
@@ -121,6 +205,7 @@ Comments:
      two tags=(group, element) can share the same name].
      What should the wrapper do in such a case !?
  * Frog: what does VM stand for ?
+ * VM = Value Multiplicity
 -----------------------------------------------------------------------------
 Description: Add information on supported imagers (constructor/model)
 Date: 2004 9 7