[Insight-developers] [GDCM] Writting DICOM from scratch

[Mathieu Malaterre]

Hello,

	I have checked in all necesseray changes for writting DICOM from 
scratch. It is commented out for now in the Image Factory, so only if 
people need it they can turn it on.

	To turn it on, edit: itkImageIOFactory.cxx and uncomment the GDCM line:

//    ObjectFactoryBase::RegisterFactory( GDCMImageIOFactory::New() );

	Since it has to be put before DICOMImageIO2 (same .dcm extension or no 
extension at all) it could have change the default behavior. So this is 
disable by default until further notice.

Mathieu


$ cvs ci -m"ENH: Adding functionality for writting DICOM from scratch, 
but it is commented out right now" 
      /cvsroot/Insight/Insight/Code/IO/CMakeLists.txt,v  <--  CMakeLists.txt
new revision: 1.60; previous revision: 1.59
/cvsroot/Insight/Insight/Code/IO/itkGDCMImageIO.cxx,v  <-- 
itkGDCMImageIO.cxx
new revision: 1.60; previous revision: 1.59
/cvsroot/Insight/Insight/Code/IO/itkGDCMImageIOFactory.cxx,v  <-- 
itkGDCMImageIOFactory.cxx
initial revision: 1.1
/cvsroot/Insight/Insight/Code/IO/itkGDCMImageIOFactory.h,v  <-- 
itkGDCMImageIOFactory.h
initial revision: 1.1
/cvsroot/Insight/Insight/Code/IO/itkImageIOFactory.cxx,v  <-- 
itkImageIOFactory.cxx
new revision: 1.24; previous revision: 1.23

Mathieu Malaterre wrote:
> Hello,
> 
>     Since gdcm 1.0 allows one to writea DICOM image from scratch I would 
> like to pull the functionality in the itkGDCM class. therefore user will 
> have two options:
> 
> 1. Carry the DICOM Dictionary around through the MetaData Dictionary. 
> And Set it properly in the Writer.
> 
> 2. Or use the ImageIOFactory and it will instanciate (based on .dcm 
> extension) an itkGDCMImageIO, with a DICOM dictionary created from scratch.
> 
> 
> Option 2. Is great if you don't have access to a 'template' DICOM image 
> that you would reuse to write your DICOM image. But for now it is far 
> from complete. For example DICOM requires that an image has an 
> orientation. And right now I cannot get this information from nowhere.
> 
> Anyway I would still like to push the functionality in CVS HEAD.
> 
> Mathieu
> Ps: patch is attached