From: jpr Date: Tue, 5 Jul 2005 18:42:15 +0000 (+0000) Subject: Update Changelog X-Git-Tag: Version1.2.bp~375 X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=commitdiff_plain;h=c36a3a368b73cd8755907d9b94706cad7176dfa2;p=gdcm.git Update Changelog --- diff --git a/ChangeLog b/ChangeLog index a8d920d0..b6c8f41d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,11 +1,35 @@ +Sun Jul 3 14:42:04 2005 by jpr + gdcmDocument.cxx + Should avoid some troubles with 'no length' SQItems within 'true length' + Sequences. + Thx to Loic Boussel for providing data to fix the bug. + +Sun Jul 3 14:47:23 2005 by jpr + gdcmSQItem.cxx + Enh warning message + +Fri Jul 1 13:25:51 2005 by jpr + gdcmPixelReadConvert.cxx + Initialize RLEInfo and JPEGInfo to 0 a avoid further problems + (e.g. PrintFile on a DICOMDIR document) + +Thu Jun 30 16:49:02 2005 by jpr + gdcmFile.cxx + Enhance gdcm::File::IsReable() heuristics to be able to process some Loic + Boussel's Hall of Shame Dicom images + +Wed Jun 29 17:58:33 2005 by jpr + gdcmDocument.cxx + Add the NO_SHADOWSEQ load mode + 2005-06-26 Mathieu Malaterre - * Quite a few new dictionaries were added in gdcm. The file: - gdcm/Dict/README should always explain were the info was found - (in case of double check). Also some dict are still only - express as 'template' (they contain 'xx'), therefore - a python script should be run first to generate a full dict. - In the longer term this python script should be replaced by - a cmake script. + * Quite a few new dictionaries were added in gdcm. The file: + gdcm/Dict/README should always explain were the info was found + (in case of double check). Also some dict are still only + express as 'template' (they contain 'xx'), therefore + a python script should be run first to generate a full dict. + In the longer term this python script should be replaced by + a cmake script. 2005-06-25 Mathieu Malaterre * dicomV3.dic is deprecated in favor of gdcm.dic @@ -14,47 +38,264 @@ - Papyrus.dic -> Papyrus - NIH.dic -> tas that were never allowed in the main standart +Fri Jun 24 12:50:20 2005 CEST by jpr + gdcmFile.cxx + Pixel Spacing related problems : update comments. + Thx to D. Clunie for providing data to check the code + +Fri Jun 24 12:55:58 2005 by jpr + Summer nights are really too hot to sleep. + Coding style : kosherization of all the 'if's (300 amongst 900 were haram) + +Thu Jun 23 11:20:23 2005 by jpr + gdcmDocument.cxx + Extend warning message, for debugging purpose + +Wed Jun 22 10:11:23 2005 by jpr + gdcmFile.cxx + Coding Style + +Wed Jun 22 10:00:29 2005 by jpr + gdcmPixelReadConvert.cxx + Fix misstyping, while removing spurious \t characters + +Wed Jun 22 09:52:32 2005 by jpr + gdcmPixelReadConvert.cxx + Some constructors (at least one ...) use BitsAllocated =! BitsStored for + negative Pixels. + That break at lot of viewers ... + We now *propagate the sign bit* upon the unused ones, better than masking them + out. + Thx to Luca Picello for profiding data to fix the bug + +Wed Jun 22 09:53:52 2005 by jpr + gdcmFileHelper.cxx + gdcm images are now "GDCM 1.1" built. + +Mon Jun 20 19:10:17 2005 by jpr + gdcmFileHelper.cxx + Avoid re writing Palette Color Lookup Table UID when Palette Color is removed + +Mon Jun 20 17:58:53 2005 by jpr + gdcmDocument.cxx + Preventive bug fix for LUT Data. + I said : LUT Date, *not* Palette Color + +Sun Jun 19 17:57:29 2005 by jpr + gdcmDocumentcxx + To avoid warnings at compile time + +Fri Jun 17 14:27:52 2005 by jpr + gdcmRLEFrames.cxx + Try to solve RLE pb + +Fri Jun 17 14:36:07 2005 by jpr + gdcmDocument.cxx + Add comments + +Fri Jun 17 14:35:00 2005 by jpr + gdcmFileHelper.cxx + Add some accessors for debugging purpose + +Tue Jun 14 15:56:41 2005 by jpr + gdcmPixelReadConvert.cxx + Now gdcm deals with 16 bits per pixel images with 16 bits Palette colors + (3 * 65536 short int ...) + Nice fat header. + "Segmented Palettes" pb still pending, but I never saw any viewer dealing with it. + +Mon Jun 13 17:43:48 2005 by jpr + gdcmPixelReadConvert.cxx + Some more oddities dealt with. + Doesn't work yet with 65536 entries 16 Bits LUT (?!?), with 16 bits pixels. + Still looking for accurate info on the web about it + + +Fri Jun 10 16:05:38 2005 by jpr + gdcmDocument.cxx + Bug fix in Load method. + Thx to Benoit for providing fix. 2005-06-08 Benoit Regrain * src/gdcmUtil.cxx : change call to Util::Format to sprintf because of lantency with the Util::Format function +Tue Jun 7 15:48:35 2005 by jpr + gdcmUtil.cxx + The highy time consuming method GetMacAdress is now called just once + +Tue Jun 7 11:58:29 2005 by jpr + gdcmDicomDirElement.cxx + Use TranslateToKey better than Format + +Tue Jun 7 12:07:20 by jpr + gdcmDictEntry.cxx + Remove some spurious calls to Format + // according to 'Purify', TranslateToKey is one of the most + // time consuming methods. + // Let's try to shorten it ! + +Mon Jun 6 14:37:58 2005 + class ArgMgr (for Arguments Manager) is designed for + command line arguments management + (to make programmer's live easier) + (could be used for any kind of 'command line program') + 2005-06-07 Mathieu Malaterre * Adding users file in CVSROOT for mail blaming 2005-06-04 Mathieu Malaterre * Adding potential support for JPEG-LS DICOM files. +Fri Jun 3 18:05:23 2005 by jpr + gdcmDocument.cxx + Fix VR processing, to be CP-288 conform. + Now we can parse any kind of DICOM SR. + +Fri Jun 3 11:05:15 2005 by jpr + gdcmDocument.h + Doxygenation + +Thu Jun 2 15:42:18 2005 by jpr + gdcmDocument.cxx + Add warning about Load Mode + +Thu Jun 2 16:20:14 2005 by jpr + gdcmFile.cxx + Harden gdcm::File::IsReadable(), in order not to be confused any longer + by lost of Pixel Element + +Fri May 27 12:51:00 2005 by jpr + gdcmFileHelper.cxx + To avoid pushing out twice the meta elements, wile trying to write 'ACR-NEMA- + style. + And avoid some memry leaks -hope so- + +Wed May 25 14:54:17 2005 by jpr + gdcmDocument.cxx + Should avoid some memory leaks + 2005-05-22 Mathieu Malaterre * Adding potential support for mpeg and jpeg2000 through two new lin - - JasPer for JPEG2000 - - libmpeg2 from the MPEG consortium - Also adding intial work to use this lib. - + - JasPer for JPEG2000 + - libmpeg2 from the MPEG consortium + Also adding intial work to use this lib. + +Fri May 20 10:27:55 2005 by jpr + gdcmFileHelper.cxx + To avoid warning. + (this time, it's wasn't dangerous at run time) + +Thu May 19 17:31:21 2005 by jpr + gdcmFileHelper.cxx + FIX possible troubles when checking + 'Hight Bit Position' vs 'Bits Allocated' and 'Bits Stored' + Thx to Mathieu for reporting bug + +Wed May 18 12:12:07 2005 by jpr + gdcmDocument.cxx + Fix : now dcm deals with 'No preamble', Big Endian Explicit VR group 0002. + Thx to Dennis Hu for providing a sample image to fix the bug. + +Tue May 17 14:44:09 2005 by jpr + gdcmFile.cxx + ENH : According to David Clunie's advice, we check now 'Imager Pixel Spacing' + before 'Pixel Spacing', when we want to get X or Y Spacing. + +Tue May 17 19:10:49 2005 by jpr + gdcmFileHelper.cxx + ENH : Now, FileHelper::CheckMandatoryElements() makes some consistency checking, + at LTTG (Least Time To Go) + around Bits Allocated, Bits Stored, High Bit Position, + to avoid unaware user to mess up the stuff. + +Tue May 3 11:43:04 2005 by jpr + gdcmDicomDir.h, .cxx + Load returns now a bool (allow user not to call IsReadable() ) + +Tue May 3 11:51:06 2005by jpr + gdcmFileHelper.cxx + Now the gdcm::File remains unimpaired after writting, in any case + +Fri Apr 29 17:10:55 2005by jpr + gdcmDocument.cxx + Don't abort the prog when File not open + +Wed Apr 27 12:00:35 2005 by jpr + gdcmFile.cxx + Removal of now useless lut-related stuff + +Wed Apr 27 11:52:27 2005 by jpr + gdcmFile.cxx + Pb of 'Bits Allocated = 12' is now dealt with in + FileHelper::CheckMandatoryElements, no longer in File::Write. + Thx to Mathieu for reporting bug. 2005-04-26 Benoit Regrain * Testing/TestAllReadCompareDicom.cxx : update the test printed results and description. +Tue Apr 26 18:18:23 2005 by jpr + gdcmDocument.cxx + Should leave gdcm::Document constructor in a much quicker + and much cleaner way when anything wrong occurs. + +Tue Apr 26 18:25:49 2005 by jpr + gdcmPixelReadConvert.cxx + Prepare MPEG Dicom encapsulated processing (within a loooong future) + +Fri Apr 22 14:59:58 2005 by jpr + gdcmDocument.cxx + Modify Print() output + +Fri Apr 22 15:37:57 2005 by jpr + gdcmTS.cxx + Prepare (future) MPEG integration + +Thu Apr 21 09:40:00 2005 by jpr + gdcmTS.cxx + Taking into account Dicom MPEG + 2005-04-20 Mathieu Malaterre * FIX : Numerical exception on borland (cannot divide by 0, doh !) - But this is a case where we should really signify the user there is - few chance the serie is really a serie... + But this is a case where we should really signify the user there is + few chance the serie is really a serie... 2005-04-20 Benoit Regrain * FIX : on X, the python part is correctly installed. All files are installed in gdcm sub-directory +Tue Apr 19 11:58:19 2005 by jpr + gdcmDocEntry + Should suppress some warnings from Borland compiler + 2005-04-19 Benoit Regrain * Builder/ : add something to facilitate the RPM creation of gdcm 2005-04-15 Mathieu Malaterre * Fix gdcmDirList::IsDirectory. Apparently borland did not like the ending - '/' + '/' * Also fix compilation on gcc 3.3.5 does not have ios::failure either ... +Thu Apr 14 16:26:19 2005 by jpr + gdcmDicomDir.h, .cxx + Now, user can ask gdcm::DicomDir to skip SeqEntries or shadow groups, when + possible : + // Old style (still available) : + dcmdir = new gdcm::DicomDir(dirName, true); + + // new style (user is allowed no to load Sequences an/or Shadow Groups) + dcmdir = new gdcm::DicomDir( ); + dcmdir->SetParseDir(true); + dcmdir->SetLoadMode(NO_SEQ | NO_SHADOW); + dcmdir->Load(dirName); + 2005-04-11 Mathieu Malaterre * vtk/vtkGdcmWriter fix bug when writing spacing. The Y spacing comes first - then the X spacing. - * Fix compilation on gcc 2.96 (which does not have ios::failure) + then the X spacing. + * Fix compilation on gcc 2.96 (which does not have ios::failure) + +Mon Apr 4 17:15:24 2005 by jpr + gdcmFileHelper.cxx + Fix stupid bug when writting ACR-NEMA image from 'ACR-LibIDO' one. 2005-04-05 Benoit Regrain * src/gdcmDictGroupName.[h|cxx] : add a correlation between a group (number) @@ -70,17 +311,129 @@ format) to replace 'at-run' dicom files generated by the TestAllReadCompareDicom test +Tue Mar 22 12:29:37 2005 by jpr + gdcmDocument.cxx + * FIX : element 0x0000 of shadow groups is now force to a ValEntry, + and it's VR is forced to UL + * ENH : user id now allowed to tell he doesn't want to deal with Sequences + and/or with Shadow groups + use SetLoadMode (NO_SEQ | NO-SHADOW); + +Tue Mar 22 12:40:13 2005 by jpr + gdcmFileHelper.cxx + *FIX Restore pushed out elements + +Fri Mar 11 12:08:31 2005 by jpr + gdcmDocEntry.cxx + Remove spurious #define MAX_SIZE_PRINT_ELEMENT_VALUE + +Fri Mar 11 12:12:13 2005 by jpr + gdcmFile.cxx + Final removal of gdcmFileHelper::InitializeDefaultFile(); + +Wed Mar 9 20:29:38 2005 by jpr + gdcmDicomDir.h, .cxx + Add Anonymize method (not yet fully checked) + 2005-03-07 Benoit Regrain * src/gdcmFileHelper.cxx : remove memory leaks and bugs when writing files +Wed Mar 2 18:18:32 2005by jpr + gdcmFile.cxx + Fix gdcm::FileHelper::CheckMandatoryElements() to take into account David + Clunie's remarks. + 2005-02-22 Benoit Regrain * src/gdcmFileHelper.cxx : JPR bad personal choices fix +Mon Feb 21 18:45:41 2005 by jpr + gdcmFileHelper.cxx + * Replace method CheckMetaElements by CheckManadatoryElements + To deal with the pb of UID's, and with Type 1 an Type 2 Entries + (probabely some stuff still missing) + * We shall have to decide the way we proceed to allow user 'freeze' Serie UID + between a given call and the next one (to add images to a Serie he just + created) + * We shall have to decide the way we proceed to allow user 'freeze' Study UID + between a given call and the next one, or to reuse an already existing + Study UID (i he wants to add his own Series to an already existing Study) + +Thu Feb 17 17:18:33 2005 by jpr + gdcmFile.cxx + To avoid further pb, Initialize defaults now Photochromatic Interpretation to + MONOCHROME2 (low values = dark), better than MONOCHRONE1. + Thx to Benoit for reporting bug + +Thu Feb 17 11:56:20 2005 by jpr + gdcmUtil.cxx + Prepare FileMetaInformationVersion (group 0002) management + +Thu Feb 17 12:02:47 2005 by jpr + gdcmFile.cxx + Group 0002 management + +Thu Feb 17 12:03:32 2005 by jpr + gdcmDocEntryArchive.cxx + Fix comments + +Wed Feb 16 11:19:19 2005 CET by jpr + gdcmPixelReadConvert.cxx + Taking into account MONOCHROME1 + Signed Pixels (?!) + +Tue Feb 15 19:12:34 2005 by jpr + gdcmFile.cxx + Taking into account the difference between MONOCHROME1 (low values = bright) + and MONOCHROME2 (low values = dark) + +Mon Feb 14 11:45:04 2005by jpr + gdcmDocument.cxx + Remove some useless mask operations, to save CPU time + +Mon Feb 14 15:51:10 2005 by jpr + gdcmUtil.cxx + *never* forget to re-compile just *before* committing! + +Mon Feb 14 15:34:27 2005 by jpr + gdcmUtil.cxx + First commit on the post 1.0 (Version? / Tag? / Branch?) : + Fix Doxygen warnings + +Mon Feb 14 11:45:47 2005 by jpr + gdcmValEntry.cxx + remove useless comments + 2005-02-11 Benoit Regrain * Testing/TestMAkeIcon.cxx : complete the test 2005-02-10 Benoit Regrain - * Testing/TestWriteSimple.cxx, TestSequence.cxx : fix memroy leaks + * Testing/TestWriteSimple.cxx, TestSequence.cxx : fix memory leaks + +Fri Feb 11 16:22:18 2005 by jpr + gdcmDocEntrySet.cxx + Typo, comments, doxygenation + +Thu Feb 10 11:55:26 2005 by jpr + gdcmFile.cxx + AnonymizeNoLoad improvement + +Thu Feb 10 10:07:19 2005 by jpr + gdcmFile.cxx + Duplicate Matthieu's hack to prevent bombing in Anonymize + +Thu Feb 10 12:20:34 2005by jpr + gdcmDocEntrySrt.cxx + The following gdcm::DocEntrySet methods, no re-definable is child classes are no + longer virtual + virtual ValEntry *GetValEntry(uint16_t group, uint16_t elem); + virtual BinEntry *GetBinEntry(uint16_t group, uint16_t elem); + virtual SeqEntry *GetSeqEntry(uint16_t group, uint16_t elem); + virtual bool SetValEntry(std::string const &content, + virtual bool SetBinEntry(uint8_t *content, int lgth, + virtual bool SetValEntry(std::string const &content, ValEntry *entry); + virtual bool SetBinEntry(uint8_t *content, int lgth, BinEntry *entry); + virtual ValEntry *InsertValEntry(std::string const &value, + virtual BinEntry *InsertBinEntry(uint8_t *binArea, int lgth, + virtual SeqEntry *InsertSeqEntry(uint16_t group, uint16_t elem); 2005-02-09 Benoit Regrain * src/gdcmFileHelper.[h|cxx] : now correctly set the VR of the datas when @@ -93,6 +446,11 @@ * Test/ : accelerate tests (it's not very significant accelerations). Remove TestCopyDicom that is redundant with TestCopyRescaleDicom +Mon Feb 7 15:48:34 2005 by jpr + gdcmDicomDir.h, .cxx + Add some commented out lines to prepare future version of 'Full DICOMDIR' stuff + + 2005-02-07 Benoit Regrain * Testing/TestSequence.cxx : add test for sequences of images... test that we have the same serie or the same study UID @@ -105,6 +463,18 @@ * src/gdcmFile.cxx : bug fix when reading values * gdcmPython/demo/ : change file names. Remove useless files +Sun Feb 6 15:31:09 2005 by jpr + gdcmDicomDirStudy.h + Update comments for DicomDir stuff + +Sun Feb 6 15:39:35 2005 by jpr + gdcmDocument.cxx + * Add method File::AnonymizeNoLoad, to allow anomymizing files + even when the pixels are not gdcm Readable + * Add methods File::AddAnonymizeElement and File::ClearAnonymizeList + to allow user to choose the elements whose value + he wants to white out / replace + 2005-02-04 Benoit Regrain * src/gdcmFile.cxx : remove the default array of initialization. The static array contained pointers to non-static values (comming from @@ -112,10 +482,19 @@ static. So, the array can't be static, too ! To simplify, the array have been replaced by repeted lines to insert values in the File +Fri Feb 4 15:49:01 2005 by jpr + gdcmDocEntry.cxx + * Add GetFirstValEntry + GetNextValEntry, for Python users + * Some more Doxygenation + typo fixing + 2005-02-03 Benoit Regrain * src/gdcmBinEntry.cxx : bug fix for BIG_ENDIAN part when writing the content * gdcmPython/gdcm.i : rename gdcmHeader to gdcmHelper +Thu Feb 3 18:12:46 2005 by jpr + gdcmPixelReadConvert.cxx + Fix Doxygen warning + 2005-02-02 Jean-Pierre Roux * FIX SQItem::AddEntry inserts now *in the right place* any Entry (Dicom Element) into the Sequence Item @@ -130,24 +509,24 @@ 2005-01-31 Mathieu Malaterre * Finish cleaning up my JPEG mess. Should be ok now. * Getting toward a RLE very similar to JPEG. Every RLE stuff is now self - contained within the RLE* class(doh!) + contained within the RLE* class(doh!) 2005-01-30 Jean-Pierre Roux * SerieHeader::SetDirectory() allow recursive exploration of the Directory - * SerieHeader::AddFileName() creates as many Coherent Files lists - (same Serie UID) as it founds different Serie UID - amongst the files instead of discarting any file - that doesn't match with the first one he found. - * SerieHeader::OrderGdcmFileList has now one parameter : - the Coherent Files list the user wants to sort + * SerieHeader::AddFileName() creates as many Coherent Files lists + (same Serie UID) as it founds different Serie UID + amongst the files instead of discarting any file + that doesn't match with the first one he found. + * SerieHeader::OrderGdcmFileList has now one parameter : + the Coherent Files list the user wants to sort * SerieHeader::GetFirstCoherentFileList() | * SerieHeader::GetLastCoherentFileList() | allow to parse - the set of Coherent Files lists + the set of Coherent Files lists * const SerieHeader::GdcmFileList &GetGdcmFileList() is kept, for backwards - compatibility (it returns the *first* Coherent Files list, - since there was only one before) - * Test/TestSerieHeader gets gdcmData as a default root directory - + compatibility (it returns the *first* Coherent Files list, + since there was only one before) + * Test/TestSerieHeader gets gdcmData as a default root directory + 2005-01-28 Benoit Regrain * Remove friend between DocEntry and File classes * Remove friend between PixelReadConverter and RLEFramesInfo, @@ -173,8 +552,8 @@ 2005-01-27 Mathieu Malaterre * Rewrite the base 256 to base 10 conversion, using string operation only. - This allows us to get rid of the problematic 64bits integer type (does not - always exist on target platform). + This allows us to get rid of the problematic 64bits integer type (does not + always exist on target platform). 2005-01-27 Jean-Pierre Roux * Add construct and destructor to class gdcm::Debug to close the debug file