Skip to content

Commit

Permalink
Add comment on Birks' Law usage in SDAction
Browse files Browse the repository at this point in the history
Add comment about using our custom Birks' Law implementation in the
SDAction. In particular to avoid potential double application of Birks'
correction if the Geant4StepHandler is used on this subdetector.
  • Loading branch information
lopezzot committed Nov 27, 2024
1 parent f066fdb commit adf67d0
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions plugins/DRTubesSDAction.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,13 @@ bool Geant4SensitiveAction<DRTubesSDData>::process(const G4Step* aStep,
// (see https://github.com/AIDASoft/DD4hep/issues/1319).
// Therefore we use copynumbers instead of volIDs.

// NOTE: in this SDAction we apply our custom Birks' Law correction
// on the G4Step via the method DRTubesSglHpr::ApplyBirks().
// However it should be known that the dd4hep Geant4StepHandler can apply Birks' Law
// correction too (using internally the Geant4 class G4EmSaturation).
// Therefore, if the Geant4StepHandler is used on this subdetector it might
// lead to a double application of Birks' correction.

#ifdef DRTubesSDDebug
// Print out some info step-by-step in sensitive volumes
//
Expand Down

0 comments on commit adf67d0

Please sign in to comment.