frog [Mon, 12 May 2003 14:32:42 +0000 (14:32 +0000)]
* src/gdcmHeader>[h/cxx] added gdcmHeader::GetPixelSize()
* vtk/vtkGdcmReader.cxx now properly inports the image in the
vtk data structure (an image Flip was required).
* vtk/testvtkGdcmReader.cxx refers to gdcmData subdir instead of Data.
* cosmetic changes in documentation.
frog [Wed, 7 May 2003 13:21:49 +0000 (13:21 +0000)]
* The subdirectory Data (containing all the images used for the
test suite) is not part of this repository anymore. A new module
containing those images is now available at
:pserver:xxx@cvs.creatis.insa-lyon.fr:2402/cvs/public
with the name gdcmData.
All the python scripts (including the package initialisation file
gdcmPython/__init__.py) were adapated to take this change into
account (basically GDCM_DATA_PATH is now GDCM_TEST_DATA_PATH).
frog [Mon, 5 May 2003 14:13:58 +0000 (14:13 +0000)]
* vtk subdir added. Contains vtkGdcmReader.[cxx|h] a vtk class
inherinting from vtkImageReader and testvtkGdcmReader.cxx a small
demo of the usage of this class.
Compilation of this vtk part is only done when using the --enable-vtk
at configure (or autogen.sh) stage.
UML design file added. This file contains a class diagram that shows the
actual state of the project. Some work has to be done in order to:
- Eliminate relationship cycles, if possible.
- Separate classes in packages.
- Include all existing documentation.
- Do a software engineering process to discover the package's real behaviour
Vs its desired behaviour. Ideally, this difference should be 0.
* More memmory link related corrections and documentation fixes.
Notes on valgrind:
- maximum info is obtained with a command of the form:
valgrind --leak-check=yes --leak-resolution=high --num-callers=40
--show-reachable=yes PrintHeader
- the remaining reachable blocks seem to come from the STL
allocation scheme through the usage of map and list. It looks
like this memory cannot be freed but it is not a memory leak
(in fact further invocation to the STL would recollect the
unused memory allthough it cannot explicitely be freed).
* gdcmPython/demo/vtkGdcmDemo.py added: this is a small demo
of displaying an image parsed with gdcm and displayed with VTK.
Note: some images don't seem to work e.g.
python vtkGdcmDemo.py ../../Data/US-RGB-8-esopecho.dcm
* src/gdcmHeader.x: dicom_vr and Dicts are not class members anymore.
Allthough this weakens the semantics, it is a ditch attempt to
make gdcm more thread friendly. --- Frog
* Memory link hunt (by using valgrind --leak-check=yes PrintHeader).
- added src/gdcmVR.cxx gdcmVR.h that objectify the previous
gdcmHeader::_dicom_vr.
- gdcmHeader::InitVRDict transfered as gdcmVR::gdcmVR().
- gdcmHeader::dicom_vr is now of type gdcmVR* as opposed to
VRHT*.
- gdcmGlobal global object contained class added (see src/gdcmUtil.x)
Correction calcul 'gdcmZSize' dans AddAndDefaultElements.
Question : Utilité d'une telle fonction, qui ajoute des Elements non ACR/DICOM dans le PubElValSet
(problèmes lors du DCM Write à prévoir)
--> A remplacer par des accesseurs propres ?
frog [Mon, 31 Mar 2003 12:02:26 +0000 (12:02 +0000)]
* src/gdcmHeader.h: LoadElements() is not a public method anymore
(to avoid double call by end user).
* Test/*.cxx LoadElements() doesn't need to be called explicitely
by end user, since gdcmHeader::gdcmHeader now handles it. --- Frog
frog [Sat, 29 Mar 2003 17:21:48 +0000 (17:21 +0000)]
* gdcm/Doc many doxygen changes:
- Doxyfile now has two different versions, DoxyfileUsers that generates
the html.users end users oriented documentation and
DoxyfileDeveloppers that generates the html.developper developper
oriented documentation.
- README changed accordingly
- requirement.txt removed (old design info moved to TODO).
- Additional files DoxyInstallation.txt, DoxyIntroduction.txt,
DoxyMainPage.txt, DoxyPython.txt provide more information.
- Note: the website http://www.creatis.insa-lyon.fr/Public/Gdcm/ can now
be updated by simply regenerating the docs with doxygen and copying
html.developper and html.users to tux:/home/httpd/html/Public/Gdcm
* src/gdcmElValSet.cxx: stl <map>.count() can only return 0 or 1.
Hence all the tests in the form "if (<map>.count() >1)" were removed.
* src/gdcmFile.cxx: cosmetic changes to avoid messages of doxygen about
ill-formed documentation sections.
* src/gdcmHeader.cxx: ditto --- Frog
frog [Thu, 27 Mar 2003 14:57:07 +0000 (14:57 +0000)]
* setup.py fixed for un*x (needed macro for stdint.h)
* Test/test.cxx renamed to PrintHeader.cxx to avoid collision on
un*ces with sh-utils test utility.
* gdcmPython/demo/test.py renamed to PrintHeader.py
* gdcmPython/demo/ReorganiseFiles.py new demo script added.
frog [Tue, 25 Mar 2003 11:41:16 +0000 (11:41 +0000)]
* gdcmPython/Makefile.am fixed linking against libstdc++
* gdcmPython/_gdcm.so symbolic link to gdcmPython/.libs/pygdcm.so added.
* Swig subdir (historical tests of swig version 1.3.17++) removed
* Test/testChangeEntete.cxx some lines commented out in order to make
compilation work.
* src/gdcmHeader.cxx cleaned up wild JPR's dirty kludge.
* src/gdcmElValSet.cxx cleaned up frog's forgotten debug message
frog [Mon, 24 Mar 2003 13:11:23 +0000 (13:11 +0000)]
* src/gdcmHeader.cxx gdcmHeader::gdcmHeader now calls LoadElements(),
hence the user doesn't need this explicit call.
- gdcmPython/testSuite.py, gdcmPython/demo/*.py changed accordingly
i.e. LoadElements() call removed.
* src/gdcmCommon.h now defines UINT32_MAX (see stdint.h).
- src/Makefile.am declares the proper flag for g++.
* src/gdcmElValSet.h added gdcmElValSet::GenerateFreeTagKeyInGroup
for adding user defined new tags.
- src/gdcmHeader.cxx gdcmHeader::NewManualElValToPubDict added.
- src/gdcmHeader.cxx gdcmHeader::AddAndDefaultElements now
cleanly adds some gdcm added tags (like gdcmXSize, gdcmYSize...)
bellet [Mon, 17 Mar 2003 14:44:55 +0000 (14:44 +0000)]
2003-03-17 Fabrice Bellet <Fabrice.Bellet@creatis.insa-lyon.fr>
* Makefile.am, acinclude.m4, configure.in, python.m4, Dicts/Makefile.am,
Doc/Makefile.am, Test/Makefile.am, gdcmPython/Makefile.am,
src/Makefile.am : the project should properly compile and install
with the autotools, included the python wrappers part.
* gdcm.spec.in : added a spec file for RPM packaging.
jpr [Fri, 14 Mar 2003 14:26:01 +0000 (14:26 +0000)]
Pricipe de precaution : j'inhibe le recalcul des longueur des groupes
dans WriteDcm et WriteAcr
Ca ne sera pas génant pour les *vrais* DICOM Readers.
Ca *pourra* l'etre pour des ACR Readers un peu teigneux ...
frog [Wed, 12 Mar 2003 23:38:58 +0000 (23:38 +0000)]
* gdcmPython/__init__.py fix of GDCM_DICT_PATH which was not
to exported to the environement (see gdcmPython/testSuite.py
for usage).
* src/gdcmElValSet.cxx, src/gdcmUtil.[cxx-h] Tokenize utility function
moved away from gdcmElValSet.cxx to gdcmUtil.cxx.
frog [Wed, 12 Mar 2003 21:33:19 +0000 (21:33 +0000)]
* src/gdcmElValSet.[cxx/h], gdcmElValue.[cxx/h], gdcmFile.[cxx/h],
gdcmHeader.[cxx/h]: ElValSet class renamed to gdcmElValSet, and
ElValue class renamed to gdcmElValue for obvious consistency reasons.
jpr [Wed, 12 Mar 2003 13:09:00 +0000 (13:09 +0000)]
POur traiter certaines (?) images Philips pour lesquelles le File Preamble
est suivi de l'elem (0002,0001) et non pas de l'elem (0002,0000)
-pas d'autre possibilité envisageable ...-
Peut-etre
frog [Mon, 10 Mar 2003 10:16:40 +0000 (10:16 +0000)]
* gdcmPython/__init__.py doesn't crash anymore when running in
in InstallMode or PreInstallMode but with an environement given
value of GDCM_DICT_PATH.
* src/gdcmDictSet.[cxx/h] coding style. --- Frog
frog [Thu, 6 Mar 2003 14:04:34 +0000 (14:04 +0000)]
* src/gdcmHeader.h and gdcmHeader.cxx Coding style + doxigenation.
* src/gdcm.h general comments moved to TODO, and README
* src/gdcm.h should now be seen as a user commodity (including it
should suffice to fully use gdcm). It is nowhere including within
any of the kernel files src/*.cxx and src/*.h.
frog [Wed, 5 Mar 2003 14:55:49 +0000 (14:55 +0000)]
* src/gdcm.h splitted in gdcmCommon.h, gdcmDict.h, gdcmDictEntry.h,
gdcmDictSet.h, gdcmElValSet.h, gdcmElValue.h, gdcmFile.h,
gdcmHeader.h
* src/gdcm*.cxx only include their corresponding include file
(as opposed to gdcm.h)
* gdcmPython/gdcm.i changed accordingly
* Test/Makefile.am corrected and added NEWS, AUTHORS in order for
the autogen.sh generated "make snapshot" command to work.
* autogen.sh removed history related references to crea package.
frog [Mon, 3 Mar 2003 17:56:21 +0000 (17:56 +0000)]
* Eventually, python/gdcmPython was renamed to gdcmPython. This
was done in a last ditch attempt to get setup.py _really_
operationnal. The python/gdcmPython layaout add the advantage
of hidding away the python related distutils specific files
(MANIFEST.in, setup.py, distutils*.py...) from the main directory.
Alas, the src directory was out of scope (i.e. it's relative
position to setup.py was ../src) which made things impossible to
handle (see previous version of python/setup.py). Crossing fingers...
frog [Fri, 21 Feb 2003 16:00:13 +0000 (16:00 +0000)]
* python/setup.py now works on Un*x. (harder than I thougth)
* python/distutilsSwigCPlusPlus.py can now have include files
among the list of sources. --- Frog
frog [Wed, 19 Feb 2003 23:37:47 +0000 (23:37 +0000)]
* Doc/Doxyfile configuration file for Doxygen and corresponding
README file for usage added.
* Test/testWrite.cxx added. This new test comes from a split of
test.cxx that is now reverted to its original purpous i.e.
testing the proper parsing of a Dicom related file.
testWrite.cxx concentrates on testing the IO part of gdcm.
* Test/bug1.cxx (containing a bug on WIn32) added.
* src/gdcm.h, gdcmHeader.cxx, gdcmDictSet.cxx:
- gdcmHeader::GetPubTagNames and gdcmHeader::GetPubTagNamesByCategory
whose purpose is to publish the content of the TagNames of the
Dicom public dictionnary were not accessible without an
instance of class gdcmHeader.
- those methods are now static methods of gdcmDictSet and hence
require no instances at all to be invocated.
- within gdcmDictSet this change required to change some method
to class methods (i.e. are now static) among which SetDictPath
(renamed to BuildDictPath) and LoadDefaultPubDict.
* python/gdcmPython/demo/printGroupedPublicDict.py changed to illustrate
the above changes by calling the new method classes of gdcmDictSet.
* python/gdcmPython/__init__.py now exposes gdcm.gdcmDictSet for
the above to be effective.
* python/gdcmPython: in order to wrap properly the above changes
for Python, swig version now needs to be > 1.3.17.
* python/gdcmPython/Makefile fixed to adapt itself to new
configure/make shema introduced by Johan Montagnat (thanks for the
contribution). Alas this Makefile cannot be turned into a proper
Makefile.am without some heavy changes in the configure.in
(for python dectection).
* python/gdcmPython/gdcm.i: the out typemap map<string, list<string>>*
now avoids publishing the empty entries. --- Frog
jpr [Tue, 28 Jan 2003 09:26:26 +0000 (09:26 +0000)]
Dernier commit avant supsension provisoire des travaux DCMLib.
Je retourne à Cardio.
(pb des nouvelles images Philips à mettre dans un format traitable par FindTaggs)
See You
frog [Wed, 22 Jan 2003 20:15:52 +0000 (20:15 +0000)]
* python/distutilsSwigCPlusPlus.py now properly collects the
shadow classes generated by "swig -c++" (gdcm.py in our case)
when using "python setup.py install".
* python/gdcmPython/__init__.py imports gdcm.py and only manually
reexports the working classes.
* src/gdcmHeader.cxx all the try/catch/throw calls were replaced
by the classical C errno scheme. This is to prevent an obscure
behavior of the python wrappers when importing wxPython.wx prior
to gdcmPython (which ended up in an abort call). An illustration
of this oddity can be found in the Test/ExceptionAndPython
subdir (see the README file). This problem probably due to
an combination of g++ and dynamic loading.
* added Test/ExceptionAndPython (see above) --- Frog
frog [Fri, 17 Jan 2003 10:48:13 +0000 (10:48 +0000)]
* Changed the layout of the python part to avoid bloating main
directory with setup.py, distutilsSwigCPlusPlus.py, MANIFEST.in
i.e. the distutils arsenal. All the python related stuff is
again in a python subdir, but the package itself is now in
python/gdcmPython.
* setup.py was cleaned up:
- pythonIncludePath removed
- python setup.py bdist target is now functional.
* gdcmPython/__init__.py doesn't export FileName any more (to avoid
collision with other packages). gdcmPython/demo/*.py changed
accordingly. --- Frog
frog [Wed, 15 Jan 2003 21:48:33 +0000 (21:48 +0000)]
* gdcmPython/setup.py and distutilsSwigCPlusPlus.py added. The
distutils installer is operational.
* - gdcmPython/__init__.py now properly loads the swig generated
shadow classes (from gdcm.py).
- gdcmPython/demo/*.py changed to import the package gdcmPython
instead of gdcmPython/demo/load.py.
- gdcmPython/testSuite.py changed to import the package gdcmPython.
jpr [Tue, 14 Jan 2003 16:31:41 +0000 (16:31 +0000)]
Ajout fonction
int gdcmFile::WriteDcm (string nomFichier)
ATTENTION :
-->Elle ecrit *forcement* en IMPLICIT little Endian (si on a un processeur intel ...)
-->Elle part d'un gdcmFile *forcement* correct (aucun test de vraisemblance n'est fait)
-->Rien n'est encore prévu pour 'fabriquer' un gdcmHeader ex-nihilo
jpr [Tue, 14 Jan 2003 13:23:48 +0000 (13:23 +0000)]
Ajout fonctions
int AddNewEntry (gdcmDictEntry* NewEntry);
int ReplaceEntry(gdcmDictEntry* NewEntry);
int RemoveEntry (TagKey k);
int RemoveEntry (guint16 group, guint16 element);
jpr [Tue, 14 Jan 2003 11:56:04 +0000 (11:56 +0000)]
modifie (0028, 0005) -number of dimensions- et (0028,0200) -image location-
de VR RET pour les mettre en US
(eviter les tests épouvantables pour rien ...)
jpr [Thu, 19 Dec 2002 17:51:42 +0000 (17:51 +0000)]
Ajout de quelques Accesseurs pour la Transfert Syntax.
IsExplicitBigEndianTransfertSyntax IsJPEGSpectralSelectionTransfertSyntax, etc...
(Je ne suis pas sur que ce soit la bonne maniere dee proceder.
Il y en a VINGT-CING en tout,
J'ai craque au bout de 8 ...
Y a-t-il la possibilite de faire chose generalisable ?
frog [Mon, 16 Dec 2002 14:37:07 +0000 (14:37 +0000)]
* src/gdcm.h, src/gdcmHeader.cxx: added GetPubTagNames() resp.
GetPubTagNamesByCategory() to gdcmHeader that return a list of the
entries within the associated public Dicom dictionary resp. the same
information but sorted by the fourth field (PAT, IMG, DIR) of the
dictionary.
- Dicts/dicomV3.dic Entries which had and unspecified fourth field
are now in the "???" group.
- python/gdcm.i changed accordingly,
- python/demo/printGroupedPublicDict.py added, that gives an example
of the above new functionalities in Python. --- Frog
frog [Mon, 9 Dec 2002 08:51:35 +0000 (08:51 +0000)]
* Test/Makefile building now depends on the one of libgdcm.so
* src/gdcmHeader.cxx and gdcm.h are now OB (undefined length encoded
pixel data) aware which enables finding the address (offset) of
the pixel data of JPEG encoded DICOM files. This leaves only a single
file in the testSuite whose pixel data address (offset) is unknown.
* python/testSuite.py changed accordingly. --- Frog
yougz [Fri, 6 Dec 2002 16:03:28 +0000 (16:03 +0000)]
2002-12-6 Eric Boix <Eric.Boix@creatis.insa-lyon.fr>
* wrapping python correct with standalone wrapped dll (don't use separate
dll under windows !!!!)
* removed glib references
* typedef's inserted in gdcm.i for correct swig type management
* python21_d MODE DEBUG enabled (ask Frog how to use it)
* NO problem with having an STL member of class for example string in C++
* PYTHON TEST OF DCMLIB OK UNDER WINDOWS ON A LARGE SET (one) of IMAGE(s).