Message: Bug in DICOM example / "ReadPatientDataFile" ? Not Logged In (login)
 Next-in-Thread Next-in-Thread
 Next-in-Forum Next-in-Forum

Question Bug in DICOM example / "ReadPatientDataFile" ? 

Forum: Medical Applications
Date: 25 Jul, 2011
From: Holger Gottschlag <Holger Gottschlag>

Hi geanteers,

i think there might be a little "bug" in the dicom example (geant4.9.3.p01):

in "Data.dat" a "10" should specify ten different kinds of tissue (air to dense bone).

After debugging it looks like that in DicomDetectorConstruction::ReadPatientDataFile at approx. Line 352 starting "G4string newMateName = ..." ,this leads to an memory access violation during runtime, since mateID is defined from 0 to 9, so that "mateOrig->GetName()" does not work for a "10"

Easy solution: I changed "10" in "Data.dat" to "9" ..

This seems to work fine for me, but i am not sure, what the authors would think of it?

Are there complications in possible other parts in the code doing it this way?

I noticed, that the "DicomPatientZSliceHeader reading material ..." - message does not print material number "10" this way.. seems that there are some more inconstencies.. ?! (what about Colourmap and CT-2-density ?)

All the best, Holger

Inline Depth:
 1 1
 All All
Outline Depth:
 1 1
 2 2
 All All
Add message: (add)

1 None: Re: Bug in DICOM example /   (Asja Pfaffenberger - 03 Aug, 2011)
 Add Message Add Message
to: "Bug in DICOM example / "ReadPatientDataFile" ?"

 Subscribe Subscribe

This site runs SLAC HyperNews version 1.11-slac-98, derived from the original HyperNews

[ Geant 4 Home | Geant 4 HyperNews | Search | Request New Forum | Feedback ]