X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;ds=sidebyside;f=src%2FgdcmFile.cxx;h=1c5bebc48f7a4571e12a4ecb82b8e37e0aa575d5;hb=a18aa6ed35e7e7c47e8ae5079cad94965dadf4bd;hp=917ef615ab0c75f04fc0a750964554ac78928868;hpb=e9a711266d34e33b482edbe241b2e40db7d9915a;p=gdcm.git diff --git a/src/gdcmFile.cxx b/src/gdcmFile.cxx index 917ef615..1c5bebc4 100644 --- a/src/gdcmFile.cxx +++ b/src/gdcmFile.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmFile.cxx,v $ Language: C++ - Date: $Date: 2005/11/09 10:18:44 $ - Version: $Revision: 1.303 $ + Date: $Date: 2005/12/21 14:52:12 $ + Version: $Revision: 1.313 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -155,7 +155,6 @@ bool File::Load( ) */ bool File::DoTheLoadingJob( ) { - // for some ACR-NEMA images GrPixel, NumPixel is *not* 7fe0,0010 // We may encounter the 'RETired' (0x0028, 0x0200) tag // (Image Location") . This entry contains the number of @@ -562,18 +561,18 @@ float File::GetYSpacing() * \brief gets the info from 0018,0088 : Space Between Slices * else from 0018,0050 : Slice Thickness * else 1.0 + * + * When an element is missing, we suppose slices join together + * (no overlapping, no interslice gap) but we have no way to check it ! + * For *Dicom* images, ZSpacing *should be* calculated using + * XOrigin, YOrigin, ZOrigin (of the top left image corner) + * of 2 consecutive images, and the Orientation + * Computing ZSpacing on a single image is not really meaningfull ! + * @return Z dimension of a voxel-to be */ float File::GetZSpacing() { - // ---> - // ---> Warning : - // ---> - // For *Dicom* images, ZSpacing should be calculated using - // XOrigin, YOrigin, ZOrigin (of the top left image corner) - // of 2 consecutive images, and the Orientation - // - // Computing ZSpacing on a single image is not really meaningfull ! float zspacing = 1.0f; @@ -601,7 +600,6 @@ float File::GetZSpacing() // if no 'Spacing Between Slices' is found, // we assume slices join together // (no overlapping, no interslice gap) - // if they don't, we're fucked up entry = GetDataEntry(0x0018,0x0050); if( entry ) { @@ -614,6 +612,21 @@ float File::GetZSpacing() else gdcmWarningMacro("Unfound Slice Thickness (0018,0050)"); + // if no 'Spacing Between Slices' is found, + // we assume slices join together + // (no overlapping, no interslice gap) + entry = GetDataEntry(0x3004,0x000c); + if( entry ) + { + float z1 = (float)entry->GetValue(0); + float z2 = (float)entry->GetValue(1); + zspacing = z2 - z1; // can be negative... + + if ( zspacing == 0.0 ) + zspacing = 1.0; + return zspacing; + } + return zspacing; } @@ -755,16 +768,19 @@ float File::GetZOrigin() /** * \brief gets the info from 0020,0037 : Image Orientation Patient * or from 0020 0035 : Image Orientation (RET) + * * (needed to organize DICOM files based on their x,y,z position) - * @param iop adress of the (6)float array to receive values - * @return true when one of the tag is found - * false when nothing is found + * + * @param iop adress of the (6)float array to receive values. + * (defaulted as 1.,0.,0.,0.,1.,0. if nothing -or inconsistent stuff- + * is found. + * @return true when one of the tag -with consistent values- is found + * false when nothing or inconsistent stuff - is found */ bool File::GetImageOrientationPatient( float iop[6] ) { std::string strImOriPat; //iop is supposed to be float[6] - iop[0] = iop[1] = iop[2] = iop[3] = iop[4] = iop[5] = 0.; // 0020 0037 DS REL Image Orientation (Patient) if ( (strImOriPat = GetEntryString(0x0020,0x0037)) != GDCM_UNFOUND ) @@ -772,10 +788,14 @@ bool File::GetImageOrientationPatient( float iop[6] ) if ( sscanf( strImOriPat.c_str(), "%f \\ %f \\%f \\%f \\%f \\%f ", &iop[0], &iop[1], &iop[2], &iop[3], &iop[4], &iop[5]) != 6 ) { + iop[0] = iop[4] = 1.; + iop[1] = iop[2] = iop[3] = iop[5] = 0.; gdcmWarningMacro( "Wrong Image Orientation Patient (0020,0037)." << " Less than 6 values were found." ); return false; } + else + return true; } //For ACR-NEMA // 0020 0035 DS REL Image Orientation (RET) @@ -784,12 +804,16 @@ bool File::GetImageOrientationPatient( float iop[6] ) if ( sscanf( strImOriPat.c_str(), "%f \\ %f \\%f \\%f \\%f \\%f ", &iop[0], &iop[1], &iop[2], &iop[3], &iop[4], &iop[5]) != 6 ) { + iop[0] = iop[4] = 1.; + iop[1] = iop[2] = iop[3] = iop[5] = 0.; gdcmWarningMacro( "wrong Image Orientation Patient (0020,0035). " << "Less than 6 values were found." ); return false; } + else + return true; } - return true; + return false; } /** @@ -1139,7 +1163,7 @@ int File::GetLUTNbits() /** *\brief gets the info from 0028,1052 : Rescale Intercept - * @return Rescale Intercept + * @return Rescale Intercept. defaulted to 0.0 is not found or empty */ float File::GetRescaleIntercept() { @@ -1156,7 +1180,7 @@ float File::GetRescaleIntercept() /** *\brief gets the info from 0028,1053 : Rescale Slope - * @return Rescale Slope + * @return Rescale Slope. defaulted to 0.0 is not found or empty */ float File::GetRescaleSlope() { @@ -1482,7 +1506,7 @@ bool File::Write(std::string fileName, FileType writetype) e0000->SetString(sLen.str()); } - // FIXME : Derma?.dcm does not have it...let's remove it ?!? JPRx + /// \todo FIXME : Derma?.dcm does not have it...let's remove it ?!? JPRx if( writetype != JPEG ) { int i_lgPix = GetEntryLength(GrPixel, NumPixel); @@ -1502,6 +1526,7 @@ bool File::Write(std::string fileName, FileType writetype) return true; } + //----------------------------------------------------------------------------- // Protected @@ -1724,7 +1749,7 @@ bool File::ReadTag(uint16_t testGroup, uint16_t testElem) { // in order not to pollute output we don't warn on 'delimitors' if (itemTagGroup != 0xfffe || testGroup != 0xfffe ) - gdcmErrorMacro( "Wrong Item Tag found:" + gdcmWarningMacro( "Wrong Item Tag found:" << " We should have found tag (" << DictEntry::TranslateToKey(testGroup,testElem) << ")" << std::endl << " but instead we encountered tag (" @@ -1816,7 +1841,7 @@ void File::ReadEncapsulatedBasicOffsetTable() // These are the deprecated method that one day should be removed (after the next release) -#ifndef GDCM_LEGACY_REMOVE +//#ifndef GDCM_LEGACY_REMOVE /* * \ brief Loader. (DEPRECATED : temporaryly kept not to break the API) * @ param fileName file to be open for parsing @@ -1824,6 +1849,7 @@ void File::ReadEncapsulatedBasicOffsetTable() * or no tag was found. * @ deprecated Use the Load() [ + SetLoadMode() ] + SetFileName() functions instead */ + /* bool File::Load( std::string const &fileName ) { GDCM_LEGACY_REPLACED_BODY(File::Load(std::string), "1.2", @@ -1834,7 +1860,8 @@ bool File::Load( std::string const &fileName ) return DoTheLoadingJob( ); } -#endif +*/ +//#endif //----------------------------------------------------------------------------- // Print