From 1f54a04be3e1e703e7b6e9088607db2190f0d24f Mon Sep 17 00:00:00 2001
From: Tomasz Bold <tomasz.bold@gmail.com>
Date: Wed, 23 Sep 2020 17:31:31 +0100
Subject: [PATCH] Minor fixes to code and tests

---
 Trigger/TrigTools/TrigInDetConfig/CMakeLists.txt              | 1 +
 Trigger/TrigTools/TrigInDetConfig/python/TrigInDetConfig.py   | 4 ++--
 .../TrigTools/TrigInDetConfig/python/TrigTrackingCutFlags.py  | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/Trigger/TrigTools/TrigInDetConfig/CMakeLists.txt b/Trigger/TrigTools/TrigInDetConfig/CMakeLists.txt
index eb0acbd46a1..cd14d84535d 100644
--- a/Trigger/TrigTools/TrigInDetConfig/CMakeLists.txt
+++ b/Trigger/TrigTools/TrigInDetConfig/CMakeLists.txt
@@ -9,3 +9,4 @@ atlas_subdir( TrigInDetConfig )
 atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} )
 
 atlas_add_test( TrigInDetCfg    SCRIPT python -m TrigInDetConfig.TrigInDetConfig    POST_EXEC_SCRIPT nopost.sh )
+atlas_add_test( TrigTrackingCutFlags    SCRIPT python -m TrigInDetConfig.TrigTrackingCutFlags    POST_EXEC_SCRIPT nopost.sh )
diff --git a/Trigger/TrigTools/TrigInDetConfig/python/TrigInDetConfig.py b/Trigger/TrigTools/TrigInDetConfig/python/TrigInDetConfig.py
index c2c7042dabf..640b9c2596f 100644
--- a/Trigger/TrigTools/TrigInDetConfig/python/TrigInDetConfig.py
+++ b/Trigger/TrigTools/TrigInDetConfig/python/TrigInDetConfig.py
@@ -729,7 +729,7 @@ def indetInViewRecoCfg( inflags, viewMakerName, signature='' ):
   There would certainly be additional algorithms
   """
   # redirect InDet.Tracking flags to point to a specific trigger setting
-  flags = inflags.cloneAndReplace("InDet.Tracking", "Trigger.InDetTracking.Electron")
+  flags = inflags.cloneAndReplace("InDet.Tracking", "Trigger.InDetTracking."+signature)
 
 
   from TriggerMenuMT.HLTMenuConfig.Menu.MenuComponents import InViewReco
@@ -766,7 +766,7 @@ if __name__ == "__main__":
     from ByteStreamCnvSvc.ByteStreamConfig import ByteStreamReadCfg
     acc.merge(ByteStreamReadCfg(ConfigFlags))
 
-    acc.merge( TrigInDetConfig( ConfigFlags ) )
+    acc.merge( indetInViewRecoCfg( ConfigFlags, viewMakerName="IMTest", signature="Electron" ) )
     from RegionSelector.RegSelConfig import regSelCfg
     rsc = regSelCfg( ConfigFlags )
     acc.merge( rsc )
diff --git a/Trigger/TrigTools/TrigInDetConfig/python/TrigTrackingCutFlags.py b/Trigger/TrigTools/TrigInDetConfig/python/TrigTrackingCutFlags.py
index 526d07c446e..6c9e24bbad5 100644
--- a/Trigger/TrigTools/TrigInDetConfig/python/TrigTrackingCutFlags.py
+++ b/Trigger/TrigTools/TrigInDetConfig/python/TrigTrackingCutFlags.py
@@ -6,7 +6,6 @@ from InDetConfig.TrackingCutsFlags import createTrackingFlags
 
 def __sliceFlags():
     flags = createTrackingFlags()
-    flags.minPT = Units.GeV * 0.5 # TODO revise values
     flags.addFlag("seedRadBinWidth", 2)
     flags.addFlag("d0SeedMax", 4.0)
     flags.addFlag("d0SeedPPSMax", 1.7)
@@ -17,6 +16,7 @@ def __sliceFlags():
 
 def __electronFlags():
     flags = __sliceFlags()
+    flags.minPT = Units.GeV * 0.5 # TODO revise values
     flags.checkRedundantSeeds=True
     return flags
 
-- 
GitLab