Activating the bias rails inefficiency for PS-p modules by default as…

… it is the intrinsic to the sensor itself. SimHits are discarded when the trajectory is entirely inside the bias rail region. At the moment we consider the SimHits for which the trajectory is partly outside the bias rail region
cms-sw · Nov 15, 2022 · 5dde5a0 · 5dde5a0
1 parent 2d0da5e
commit 5dde5a0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/SimTracker/SiPhase2Digitizer/plugins/PSPDigitizerAlgorithm.cc b/SimTracker/SiPhase2Digitizer/plugins/PSPDigitizerAlgorithm.cc
@@ -68,7 +68,7 @@ bool PSPDigitizerAlgorithm::isInBiasRailRegion(const PSimHit& hit) const {
   constexpr float block_unit = implant + bRail;
   float yin = hit.entryPoint().y() + block_len;
   float yout = hit.exitPoint().y() + block_len;
-  if (std::fmod(yin, block_unit) > implant || std::fmod(yout, block_unit) > implant)
+  if (std::fmod(yin, block_unit) > implant && std::fmod(yout, block_unit) > implant)
     return true;
   else
     return false;

diff --git a/SimTracker/SiPhase2Digitizer/python/phase2TrackerDigitizer_cfi.py b/SimTracker/SiPhase2Digitizer/python/phase2TrackerDigitizer_cfi.py
@@ -127,7 +127,7 @@
       EfficiencyFactors_Endcap = cms.vdouble(0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 0.999, 
       0.999, 0.999 ),#Efficiencies kept as Side2Disk1,Side1Disk1 and so on
       CellsToKill = cms.VPSet(),
-      AddBiasRailInefficiency= cms.bool(False)
+      AddBiasRailInefficiency= cms.bool(True)
     ),
 #Strip in PS module
     PSSDigitizerAlgorithm = cms.PSet(