[Gdcm-hackers] [ gdcm-Bugs-3043115 ] DICOM file OK on Intel but not PPC (Missing byte swapping)

[SourceForge.net <no...@so...>]

Bugs item #3043115, was opened at 2010-08-11 07:51
Message generated for change (Settings changed) made by malat
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=739587&aid=3043115&group_id=137895

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: General (default)
>Group: 2.3
Status: Open
Resolution: Postponed
Priority: 9
Private: No
Submitted By: Sean McBride (sean_mcbride)
Assigned to: Mathieu Malaterre (malat)
Summary: DICOM file OK on Intel but not PPC (Missing byte swapping)

Initial Comment:
We have a DICOM file that does not work with our app (via GDCM) but does with other DICOM viewers.

On Intel, the image appears correctly; on PowerPC the voxel colours are all wrong.  I ran gdcmdump on both machines and compared the output.  Some values are swapped, ex:

(0019,1012) ?? (SL) 0\0\217841663 # 12,3 TablePositionOrigin
(0019,1013) ?? (SL) 0\0\217841663 # 12,3 ImaAbsTablePosition

(0019,1012) ?? (SL) 0\0\-1268  # 12,3 TablePositionOrigin
(0019,1013) ?? (SL) 0\0\-1268  # 12,3 ImaAbsTablePosition

217841663 is the byte swapped representation of -1268.

I cannot share the DICOM file in this public bug. Contact me privately.

----------------------------------------------------------------------

Comment By: Sean McBride (sean_mcbride)
Date: 2011-01-24 08:36

Message:
Mathieu, no problem if it waits for the next release. Is the code in some
branch that we could test?

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-01-21 07:17

Message:
Change it too big for this release. I'll have to delay for now.
GDCM 2.0.17 needs to be delivered this week.

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-01-21 06:18

Message:
[master b3fbfbb] Adding support for dual VR
 2 files changed, 27 insertions(+), 7 deletions(-)


----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-01-21 03:54

Message:
$ gdcmdump gdcmData/MR_SIEMENS_forceLoad29-1010_29-1020.dcm| grep Lar
(0028,0107) ?? (US or SS => US) 63239                             #
2,1 Largest Image Pixel Value

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-01-05 08:34

Message:
According to Sean the issue is still present:
...
We finally had a chance to test your fix for bug 3043115 and it's an
improvement but the issue is still there.  I could not figure out how to
reopen the bug at sourceforge.

Can you reopen the bug, or tell me how?

Again we used gdcmdump, and can see that the value for "2,1 Largest
Image Pixel Value" is byte swapped.  Other values may be swapped too,
but this technique won't reveal this if the value is 0 (as is the case
for "2,1 Smallest Image Pixel Value").
...

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2010-10-05 04:22

Message:
$ svn ci -m"BUG: 3043115 Implement VRToTag function require on PPC
architecture and implicit file."                            
~/Perso/gdcm/trunk/Source
Sending       
Source/DataStructureAndEncodingDefinition/gdcmImplicitDataElement.txx
Sending        Source/DataStructureAndEncodingDefinition/gdcmTagToVR.cxx
Sending        Source/DataStructureAndEncodingDefinition/gdcmTagToVR.h
Transmitting file data ...
Committed revision 7115.


----------------------------------------------------------------------

Comment By: Sean McBride (sean_mcbride)
Date: 2010-08-11 08:25

Message:
I have a 2nd DICOM file that is probably the same problem.  gdcmdump again
reveals byte swapped values.  But not for the same fields...

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=739587&aid=3043115&group_id=137895