From d004f1760f500617c79455fcbe2d016f294c545c Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Thu, 3 Dec 2020 15:20:13 +0100 Subject: [PATCH 1/4] MuonCombinedTimingTools: cmake and flake8 fixes --- .../MuonCombinedTimingTools/CMakeLists.txt | 12 +++--------- .../python/MuonCombinedTimingConfig.py | 5 +---- 2 files changed, 4 insertions(+), 13 deletions(-) diff --git a/Reconstruction/MuonIdentification/MuonCombinedTimingTools/CMakeLists.txt b/Reconstruction/MuonIdentification/MuonCombinedTimingTools/CMakeLists.txt index a6231f3893b1..c185c0228d43 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedTimingTools/CMakeLists.txt +++ b/Reconstruction/MuonIdentification/MuonCombinedTimingTools/CMakeLists.txt @@ -1,20 +1,14 @@ -################################################################################ -# Package: MuonCombinedTimingTools -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( MuonCombinedTimingTools ) -# External dependencies: -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) - # Component(s) in the package: atlas_add_component( MuonCombinedTimingTools src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} AthenaBaseComps xAODMuon GaudiKernel MuonCombinedToolInterfaces TrkSegment TrkSpaceTimePoint TrkToolInterfaces ) + LINK_LIBRARIES AthenaBaseComps xAODMuon GaudiKernel MuonCombinedToolInterfaces TrkSegment TrkSpaceTimePoint TrkToolInterfaces ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedTimingTools/python/MuonCombinedTimingConfig.py b/Reconstruction/MuonIdentification/MuonCombinedTimingTools/python/MuonCombinedTimingConfig.py index aac39a182c80..abc96e804baf 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedTimingTools/python/MuonCombinedTimingConfig.py +++ b/Reconstruction/MuonIdentification/MuonCombinedTimingTools/python/MuonCombinedTimingConfig.py @@ -1,11 +1,8 @@ -# Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration ########################## start of MuonTiming Filter Fragment ################################## from AthenaCommon.AppMgr import ToolSvc -from AthenaCommon.AppMgr import ServiceMgr -from AthenaCommon.DetFlags import DetFlags import AthenaCommon.CfgMgr as CfgMgr -from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags from AthenaCommon.Include import include include( "TrackInCaloTools/TrackInCaloTools_jobOptions.py" ) #from TrackInCaloTools import TrackInCaloTools_jobOptions -- GitLab From e6237070bcd75498c5e0fb5f670f35e06bebc9cf Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Thu, 3 Dec 2020 15:20:49 +0100 Subject: [PATCH 2/4] MuonCombinedAlgs: cmake and flake8 fixes --- .../MuonIdentification/MuonCombinedAlgs/CMakeLists.txt | 8 +++----- .../MuonCombinedAlgs/python/MuonCombinedAlgsMonitoring.py | 1 + 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/Reconstruction/MuonIdentification/MuonCombinedAlgs/CMakeLists.txt b/Reconstruction/MuonIdentification/MuonCombinedAlgs/CMakeLists.txt index 4de1ad8ab9c5..38cdd9f98f66 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedAlgs/CMakeLists.txt +++ b/Reconstruction/MuonIdentification/MuonCombinedAlgs/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: MuonCombinedAlgs -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( MuonCombinedAlgs ) @@ -9,6 +7,6 @@ atlas_subdir( MuonCombinedAlgs ) atlas_add_component( MuonCombinedAlgs src/*.cxx src/components/*.cxx - LINK_LIBRARIES AthenaBaseComps AthenaMonitoringKernelLib xAODCaloEvent xAODMuon xAODTruth xAODTracking GaudiKernel MuonSegment MuonRecToolInterfaces MuonLayerEvent MuonCombinedEvent MuonCombinedToolInterfaces TrkSegment TrkTrack TrkToolInterfaces) + LINK_LIBRARIES AthenaBaseComps AthenaMonitoringKernelLib xAODCaloEvent xAODMuon xAODTruth xAODTracking GaudiKernel MuonSegment MuonRecToolInterfaces MuonLayerEvent MuonCombinedEvent MuonCombinedToolInterfaces MuonPrepRawData StoreGateLib TrkSegment TrkTrack TrkToolInterfaces) -atlas_install_python_modules( python/MuonCombinedAlgsMonitoring.py) +atlas_install_python_modules( python/MuonCombinedAlgsMonitoring.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedAlgs/python/MuonCombinedAlgsMonitoring.py b/Reconstruction/MuonIdentification/MuonCombinedAlgs/python/MuonCombinedAlgsMonitoring.py index 0bd1261b841d..44151e83ee65 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedAlgs/python/MuonCombinedAlgsMonitoring.py +++ b/Reconstruction/MuonIdentification/MuonCombinedAlgs/python/MuonCombinedAlgsMonitoring.py @@ -1,3 +1,4 @@ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Author: Laurynas Mince # Created on 26.07.2019 -- GitLab From f0874163a405477ca86ec55778932c766d89c990 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Fri, 4 Dec 2020 10:09:16 +0100 Subject: [PATCH 3/4] MuonCombinedRecExample: enable flake8 and fix code --- .../MuonCombinedRecExample/CMakeLists.txt | 6 ++-- .../python/CombinedMuonTrackSummary.py | 2 -- .../python/MuGirlTagTool.py | 9 ++--- .../python/MuonCaloTagTool.py | 4 +-- .../python/MuonCombinedAlgs.py | 7 ++-- .../python/MuonCombinedCalibConfig.py | 35 +++++++++---------- .../python/MuonCombinedFitTools.py | 22 +++++------- .../python/MuonCombinedRecExampleConfigDb.py | 7 +--- .../python/MuonCombinedRecFlags.py | 25 ++++--------- .../python/MuonCombinedRecOnlySetup.py | 6 ++-- .../python/MuonCombinedTools.py | 19 ++++------ .../python/MuonSegmentTaggerTools.py | 13 ++----- .../python/ScaleCalibrationInputSetup.py | 9 ++--- .../python/TagAndProbeInputSetup.py | 4 +-- 14 files changed, 59 insertions(+), 109 deletions(-) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/CMakeLists.txt b/Reconstruction/MuonIdentification/MuonCombinedRecExample/CMakeLists.txt index 821537cb7e16..28f5524d9b57 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/CMakeLists.txt +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/CMakeLists.txt @@ -1,11 +1,9 @@ -################################################################################ -# Package: MuonCombinedRecExample -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( MuonCombinedRecExample ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_joboptions( share/*.py ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/CombinedMuonTrackSummary.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/CombinedMuonTrackSummary.py index 89e0c5caf066..d52f506bae4f 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/CombinedMuonTrackSummary.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/CombinedMuonTrackSummary.py @@ -8,12 +8,10 @@ #========================================================================== from AthenaCommon.DetFlags import DetFlags -from AthenaCommon.AppMgr import ServiceMgr from AthenaCommon.AppMgr import ToolSvc from AthenaCommon.BeamFlags import jobproperties from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags from AthenaCommon.CfgGetter import getPublicTool -from AthenaCommon.GlobalFlags import globalflags beamFlags = jobproperties.Beam diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuGirlTagTool.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuGirlTagTool.py index ff9f2002549c..a5f9d3ab16c7 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuGirlTagTool.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuGirlTagTool.py @@ -3,7 +3,7 @@ ### JobOptions to run MuGirlTag in xAOD from AthenaCommon import CfgMgr -from AthenaCommon.CfgGetter import getPublicTool,getPublicToolClone,getService +from AthenaCommon.CfgGetter import getPublicTool from AthenaCommon.GlobalFlags import globalflags from RecExConfig.RecFlags import rec @@ -14,9 +14,6 @@ from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags from MuonRecExample.MooreTools import MuonSeededSegmentFinder, MuonChamberHoleRecoveryTool from MuonRecExample.MuonRecTools import DCMathSegmentMaker -###logfile -from AthenaCommon.Logging import log - from AtlasGeoModel.MuonGMJobProperties import MuonGeometryFlags from TriggerJobOpts.TriggerFlags import TriggerFlags from AthenaCommon.AthenaCommonFlags import athenaCommonFlags @@ -24,7 +21,7 @@ from AthenaCommon.AthenaCommonFlags import athenaCommonFlags def MuonInsideOutRecoTool( name="MuonInsideOutRecoTool", **kwargs ): if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("VertexContainer", "") - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) kwargs.setdefault("MuonLayerSegmentFinderTool", getPublicTool("MuonLayerSegmentFinderTool")) @@ -91,7 +88,7 @@ def MuonStauInsideOutRecoTool( name="MuonStauInsideOutRecoTool", **kwargs ): kwargs.setdefault("MuonCandidateTrackBuilderTool", getPublicTool("MuonStauCandidateTrackBuilderTool") ) if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("VertexContainer", "") - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) return CfgMgr.MuonCombined__MuonInsideOutRecoTool(name,**kwargs ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCaloTagTool.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCaloTagTool.py index 9300cd9534b7..af515db9baf9 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCaloTagTool.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCaloTagTool.py @@ -1,9 +1,9 @@ -# Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration ### JobOptions to run MuonCaloTag in xAOD from AthenaCommon import CfgMgr -from AthenaCommon.CfgGetter import getPublicTool,getService, getPrivateTool +from AthenaCommon.CfgGetter import getPublicTool, getPrivateTool ############################################################################### ## Helper function for defining THistSvc inputs diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedAlgs.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedAlgs.py index 49515f05765b..2035e3bf58a6 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedAlgs.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedAlgs.py @@ -1,14 +1,13 @@ # Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags -from AthenaCommon.CfgGetter import getPublicTool, getAlgorithm,getPublicToolClone +from AthenaCommon.CfgGetter import getPublicTool, getAlgorithm from MuonRecExample.ConfiguredMuonRec import ConfiguredMuonRec from MuonRecExample.MuonRecFlags import muonRecFlags muonRecFlags.setDefaults() from AthenaCommon.AlgSequence import AlgSequence from AthenaCommon import CfgMgr -from AthenaCommon.BeamFlags import jobproperties from AtlasGeoModel.MuonGMJobProperties import MuonGeometryFlags from MuonCombinedRecExample.MuonCombinedKeys import MuonCombinedKeys as MuonCbKeys @@ -19,7 +18,7 @@ from TriggerJobOpts.TriggerFlags import TriggerFlags def MuonCombinedInDetExtensionAlg(name="MuonCombinedInDetExtensionAlg",**kwargs): tools = [] if muonCombinedRecFlags.doCaloTrkMuId(): - tools.append(getTool("MuonCaloTagTool")) + tools.append(getPublicTool("MuonCaloTagTool")) kwargs.setdefault("TagMap", "caloTagMap" ) kwargs.setdefault("MuonCombinedInDetExtensionTools", tools ) kwargs.setdefault("HasCSC", MuonGeometryFlags.hasCSC() ) @@ -159,7 +158,7 @@ def recordMuonCreatorAlgObjs (kw): Alg = CfgMgr.MuonCreatorAlg def val (prop): d = kw.get (prop) - if d == None: + if d is None: d = Alg.__dict__[prop].default return str(d) objs = {'xAOD::MuonContainer': val('MuonContainerLocation'), diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedCalibConfig.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedCalibConfig.py index c89df613f4d2..3066b32d551e 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedCalibConfig.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedCalibConfig.py @@ -1,40 +1,37 @@ -# Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration -from AthenaCommon.JobProperties import JobProperty,JobPropertyContainer,jobproperties from AthenaCommon.AppMgr import ToolSvc from AthenaCommon.AlgSequence import AlgSequence from AthenaCommon.Logging import logging -from AthenaCommon import BeamFlags -beamFlags = jobproperties.Beam from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags from MuonRecExample.MuonRecFlags import muonRecFlags from MuonCnvExample.MuonCalibFlags import muonCalibFlags from RecExConfig.RecFlags import rec +log = logging.getLogger("MuonCombinedRec") + # configure calib algs for standalone if muonRecFlags.doCalibNtuple(): - from MuonRecExample import MuonAlignConfig + from MuonRecExample import MuonAlignConfig # noqa: F401 (import side-effects) from MuonCnvExample import setupMuonCalibNtuple setupMuonCalibNtuple() elif muonRecFlags.doCalib(): - from MuonRecExample import MuonAlignConfig + from MuonRecExample import MuonAlignConfig # noqa: F401 (import side-effects) from MuonCnvExample import setupMuonCalib setupMuonCalib() else: - logMuon.warning("Loading %s but not setting up any MuonCalibration or Ntuple" % __name__ ) + log.warning("Loading %s but not setting up any MuonCalibration or Ntuple", __name__ ) # NB. call setDefaults *after* import MuonCalibConfig muonCalibFlags.setDefaults() -log = logging.getLogger("MuonCombinedRec") - topSequence = AlgSequence() # # Move MuonCalibExtraTree to end of topSequence and configure for combined #s -from PyUtils.MetaReaderPeeker import metadata, convert_itemList +from PyUtils.MetaReaderPeeker import convert_itemList def checkContainerInInputFile(container_type, container_name): if container_type in convert_itemList(layout='dict'): if container_name in convert_itemList(layout='dict')[container_type]: @@ -43,19 +40,19 @@ def checkContainerInInputFile(container_type, container_name): def StacoMuonCollectionAvailable(): - return False + return False def MuidMuonCollectionAvailable(): - return False + return False def MuonsMuonCollectionAvailable(): - if rec.readESD() and checkContainerInInputFile("Analysis::MuonContainer", "Muons"): - return True - if muonCombinedRecFlags.doxAOD() and muonCombinedRecFlags.doCombinedFit(): - return True - if hasattr(topSequence, 'BuildMuonCollection'): - return True - return False + if rec.readESD() and checkContainerInInputFile("Analysis::MuonContainer", "Muons"): + return True + if muonCombinedRecFlags.doxAOD() and muonCombinedRecFlags.doCombinedFit(): + return True + if hasattr(topSequence, 'BuildMuonCollection'): + return True + return False try: diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedFitTools.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedFitTools.py index 2f628b59fadb..1cb5382fcf1a 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedFitTools.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedFitTools.py @@ -11,13 +11,11 @@ #============================================================== -from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags -from MuonRecExample.MuonRecFlags import muonRecFlags,muonStandaloneFlags +from MuonRecExample.MuonRecFlags import muonRecFlags from MuonRecExample.MuonStandaloneFlags import muonStandaloneFlags -from MuonRecExample.MuonRecUtils import ExtraFlags from AthenaCommon import CfgMgr -from AthenaCommon.CfgGetter import getPrivateTool,getPrivateToolClone,getPublicTool,getPublicToolClone,getService,getServiceClone +from AthenaCommon.CfgGetter import getPrivateTool, getPrivateToolClone, getPublicTool, getPublicToolClone, getService from AthenaCommon.BeamFlags import jobproperties beamFlags = jobproperties.Beam from AthenaCommon.DetFlags import DetFlags @@ -25,7 +23,6 @@ from AthenaCommon.SystemOfUnits import meter from IOVDbSvc.CondDB import conddb from AthenaCommon.GlobalFlags import globalflags -from RecExConfig.RecFlags import rec from AtlasGeoModel.MuonGMJobProperties import MuonGeometryFlags from TriggerJobOpts.TriggerFlags import TriggerFlags @@ -50,7 +47,7 @@ def iPatFitter( name='iPatFitter', **kwargs): if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("MaxIterations", 15) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) @@ -65,7 +62,7 @@ def iPatSLFitter( name='iPatSLFitter', **kwargs): if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("MaxIterations", 15) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) @@ -182,7 +179,7 @@ def MuonTrackQuery( name="MuonTrackQuery", **kwargs ): def MuidSegmentRegionRecoveryTool( name ='MuidSegmentRegionRecoveryTool', **kwargs ): kwargs.setdefault("Builder", getPublicTool("CombinedMuonTrackBuilderFit") ) - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) @@ -210,8 +207,7 @@ def MuidSegmentRegionRecoveryTool( name ='MuidSegmentRegionRecoveryTool', **kwar def MuonMaterialProviderTool( name = "MuonMaterialProviderTool"): - from TrkExTools.AtlasExtrapolator import AtlasExtrapolator - from TrackToCalo.TrackToCaloConf import Trk__ParticleCaloExtensionTool, Rec__MuonCaloEnergyTool, Rec__ParticleCaloCellAssociationTool + from TrackToCalo.TrackToCaloConf import Rec__MuonCaloEnergyTool, Rec__ParticleCaloCellAssociationTool from TrkMaterialProvider.TrkMaterialProviderConf import Trk__TrkMaterialProviderTool caloCellAssociationTool = Rec__ParticleCaloCellAssociationTool(ParticleCaloExtensionTool = getPublicTool("MuonParticleCaloExtensionTool")) from AthenaCommon.AppMgr import ToolSvc @@ -221,7 +217,7 @@ def MuonMaterialProviderTool( name = "MuonMaterialProviderTool"): ParticleCaloCellAssociationTool = caloCellAssociationTool) ToolSvc += muonCaloEnergyTool - materialProviderTool = Trk__TrkMaterialProviderTool(MuonCaloEnergyTool = muonCaloEnergyTool); + materialProviderTool = Trk__TrkMaterialProviderTool(MuonCaloEnergyTool = muonCaloEnergyTool) if TriggerFlags.MuonSlice.doTrigMuonConfig: materialProviderTool.UseCaloEnergyMeasurement = False return materialProviderTool @@ -285,7 +281,7 @@ def CombinedMuonTrackBuilderFit( name='CombinedMuonTrackBuilderFit', **kwargs ): kwargs.setdefault("Cleaner" , getPrivateToolClone("TrigMuidTrackCleaner_"+suffix,"MuidTrackCleaner", Fitter=kwargs["Fitter"]) ) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) kwargs.setdefault("TrackSummaryTool" , ToolSvc.CombinedMuonTrackSummary ) kwargs.setdefault("Propagator" , getPublicTool("MuonCombinedPropagator") ) kwargs.setdefault("SLPropagator" , getPublicTool("MuonCombinedPropagator") ) @@ -355,7 +351,7 @@ def CombinedMuonTrackBuilder( name='CombinedMuonTrackBuilder', **kwargs ): kwargs.setdefault("CscRotCreator" , "" ) kwargs.setdefault("Cleaner" , getPrivateToolClone("TrigMuidTrackCleaner_"+suffix, "MuidTrackCleaner", Fitter=kwargs["Fitter"]) ) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) kwargs.setdefault("MuonHoleRecovery" , getPublicTool("MuidSegmentRegionRecoveryTool") ) kwargs.setdefault("TrackSummaryTool" , ToolSvc.CombinedMuonTrackSummary ) kwargs.setdefault("Propagator" , getPublicTool("MuonCombinedPropagator") ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecExampleConfigDb.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecExampleConfigDb.py index 523dc66a2aa8..37df8e5880de 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecExampleConfigDb.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecExampleConfigDb.py @@ -1,12 +1,7 @@ # Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration -from AthenaCommon.CfgGetter import addTool, addToolClone, addService, addAlgorithm, \ - addTypesToExcludeIfDefaultValue, addNamesToExcludeIfDefaultValue, addFullNamesToExcludeIfDefaultValue, \ - addPropertiesToExcludeIfDefault, \ - addTypesToSkipIfNotAvailable, addNamesToSkipIfNotAvailable, addFullNamesToSkipIfNotAvailable, \ - addTypesOnlyToSkip +from AthenaCommon.CfgGetter import addTool, addAlgorithm -from AthenaCommon.Constants import * # FATAL,ERROR etc. # combined tools addTool("MuonCombinedRecExample.MuonCombinedTools.MuonCombinedTool","MuonCombinedTool") diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py index e7c94f4e2666..85e4a83d1b07 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py @@ -1,22 +1,12 @@ -# Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration __doc__ = """Flags to steer Muon Combined Reconstruction.""" -from AthenaCommon.JobProperties import JobProperty,JobPropertyContainer,jobproperties -from AthenaCommon.Logging import logging -from AthenaCommon import BeamFlags -from AthenaCommon import BFieldFlags -beamFlags = jobproperties.Beam - -from RecExConfig.RecFlags import rec -from RecExConfig.RecAlgsFlags import recAlgs - +from AthenaCommon.JobProperties import JobProperty,JobPropertyContainer +from AthenaCommon.BeamFlags import jobproperties +from MuonRecExample.MuonRecUtils import fillJobPropertyContainer,SummaryJobProperty from MuonRecExample.MuonRecFlags import muonRecFlags,muonStandaloneFlags -from MuonRecExample.MuonRecUtils import logMuon,fillJobPropertyContainer,override_lock_and_set_Value,SummaryJobProperty -from InDetRecExample.InDetJobProperties import InDetFlags - -logMuonComb = logging.getLogger("MuonCombinedRec") # # Flags for general use @@ -175,7 +165,6 @@ class MuonCombinedRec(JobPropertyContainer): """The flags to steer muon combined reconstruction""" def setDefaults(self): - global beamFlags,muonRecFlags,muonStandaloneFlags # set defaults of the flags that its using muonRecFlags.setDefaults() @@ -183,7 +172,7 @@ class MuonCombinedRec(JobPropertyContainer): from MuonRecExample.MuonRecUtils import setJobPropertyDefault as setDefault - haveMuonTracks = muonRecFlags.doStandalone() and self.doAOD +# haveMuonTracks = muonRecFlags.doStandalone() and self.doAOD # This logic doesn't work - doAOD seems to be off with Vakho's standard test # if not haveMuonTracks: # if not muonRecFlags.doStandalone(): @@ -192,7 +181,7 @@ class MuonCombinedRec(JobPropertyContainer): # print "turning off doxAOD because doAOD is off" # setDefault(self.doxAOD,False) - if beamFlags.beamType()=='cosmics' or beamFlags.beamType()=='singlebeam': + if jobproperties.Beam.beamType()=='cosmics' or jobproperties.Beam.beamType()=='singlebeam': # Algorithms for CaloMuonCollection setDefault(self.doCaloTrkMuId,self.doAOD()) # Algorithms for MuGirlLowBetaCollection @@ -205,7 +194,7 @@ class MuonCombinedRec(JobPropertyContainer): #setDefault(self.doMuGirlLowBeta,self.doAOD()) #setDefault(self.doMuGirl,self.doAOD()) # also for new MuonCollection - if beamFlags.beamType()=='cosmics': + if jobproperties.Beam.beamType()=='cosmics': setDefault(self.doLArMuId,True) else: setDefault(self.doLArMuId,False) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecOnlySetup.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecOnlySetup.py index d7b92c35741b..2dd40ee304ef 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecOnlySetup.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecOnlySetup.py @@ -1,17 +1,15 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration __doc__ = """Configure all flags (for RecExCommon) so that only Muon Standalone and Muon Combined reconstruction are run""" from AthenaCommon.DetFlags import DetFlags -from AthenaCommon.GlobalFlags import globalflags,GlobalFlags from RecExConfig.RecFlags import rec as recFlags from RecExConfig.RecAlgsFlags import recAlgs as recAlgsFlags -from RecExConfig.RecConfFlags import recConfFlags from InDetRecExample.InDetJobProperties import InDetFlags from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags # import standalone to avoid that it overwrites combined later on -import MuonRecExample.MuonRecStandaloneOnlySetup +import MuonRecExample.MuonRecStandaloneOnlySetup # noqa: F401 (import side-effects) # Output recFlags.doWriteESD = False diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedTools.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedTools.py index 3b5d2a405387..bcfed3b938bc 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedTools.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedTools.py @@ -1,19 +1,12 @@ # Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags -from MuonCombinedRecExample.MuonCombinedKeys import MuonCombinedKeys as MuonCbKeys -from MuonRecExample.MuonRecFlags import muonRecFlags from AthenaCommon.AppMgr import ToolSvc - -from AthenaCommon.Resilience import protectedInclude - from AthenaCommon import CfgMgr -from AthenaCommon.CfgGetter import getPrivateTool,getPrivateToolClone,getPublicTool,getPublicToolClone,getService,getServiceClone +from AthenaCommon.CfgGetter import getPublicTool, getPublicToolClone from AthenaCommon.BeamFlags import jobproperties beamFlags = jobproperties.Beam -from AthenaCommon.GlobalFlags import globalflags -from RecExConfig.RecFlags import rec from TriggerJobOpts.TriggerFlags import TriggerFlags def MuonCombinedInDetDetailedTrackSelectorTool( name='MuonCombinedInDetDetailedTrackSelectorTool', **kwargs): @@ -71,7 +64,7 @@ def MuonCombinedParticleCreator(name="MuonCombinedParticleCreator",**kwargs): if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("TrackSummaryTool" , getPublicTool("MuonTrackSummaryTool") ) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary ) #getPublicTool("CombinedMuonTrackSummary") ) kwargs.setdefault("Extrapolator", getPublicTool("AtlasExtrapolator") ) kwargs.setdefault("KeepAllPerigee",True ) @@ -81,7 +74,7 @@ def MuonCombinedParticleCreator(name="MuonCombinedParticleCreator",**kwargs): return CfgMgr.Trk__TrackParticleCreatorTool(name,**kwargs) def MuonCaloParticleCreator(name="MuonCaloParticleCreator",**kwargs): - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) kwargs.setdefault("Extrapolator", getPublicTool("AtlasExtrapolator") ) kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary ) #getPublicTool("CombinedMuonTrackSummary") ) kwargs.setdefault("KeepAllPerigee",True ) @@ -108,7 +101,7 @@ def MuonCreatorTool(name="MuonCreatorTool",**kwargs): getPublicTool("MuonMomentumBalanceSignificanceTool") getPublicTool("MuonScatteringAngleSignificanceTool") getPublicTool("MuonCaloParticleCreator") - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) @@ -130,7 +123,7 @@ def MuonCreatorTool_LRT(name="MuonCreatorTool_LRT",**kwargs): getPublicTool("MuonMomentumBalanceSignificanceTool") getPublicTool("MuonScatteringAngleSignificanceTool") getPublicTool("MuonCaloParticleCreator") - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) @@ -149,7 +142,7 @@ def ExtrapolateMuonToIPTool(name="ExtrapolateMuonToIPTool",**kwargs): if TriggerFlags.MuonSlice.doTrigMuonConfig: kwargs.setdefault("TrackSummaryTool", getPublicTool("MuonTrackSummaryTool")) else: - import MuonCombinedRecExample.CombinedMuonTrackSummary + import MuonCombinedRecExample.CombinedMuonTrackSummary # noqa: F401 (import side-effects) from AthenaCommon.AppMgr import ToolSvc kwargs.setdefault("TrackSummaryTool", ToolSvc.CombinedMuonTrackSummary) return CfgMgr.ExtrapolateMuonToIPTool(name,**kwargs) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonSegmentTaggerTools.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonSegmentTaggerTools.py index 347d51cb61a1..7be4911a6bea 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonSegmentTaggerTools.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonSegmentTaggerTools.py @@ -1,17 +1,8 @@ -# Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration - -from MuonCombinedRecExample.MuonCombinedRecFlags import muonCombinedRecFlags -from MuonCombinedRecExample.MuonCombinedKeys import MuonCombinedKeys as MuonCbKeys -from MuonRecExample.MuonRecFlags import muonRecFlags -from AthenaCommon.AppMgr import ToolSvc +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from AthenaCommon import CfgMgr -from AthenaCommon.CfgGetter import getPrivateTool,getPrivateToolClone,getPublicTool,getAlgorithm,getPublicToolClone,getService,getServiceClone -from AthenaCommon.BeamFlags import jobproperties -beamFlags = jobproperties.Beam +from AthenaCommon.CfgGetter import getPublicTool -from AthenaCommon.GlobalFlags import globalflags -from RecExConfig.RecFlags import rec def MuTagMatchingTool( name='MuTagMatchingTool', **kwargs ): kwargs.setdefault("AssumeLocalErrors", True ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py index 4894d6e13adf..aaa2cea36959 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py @@ -1,11 +1,12 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration - -from __future__ import print_function +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from AthenaCommon.AlgSequence import AlgSequence +from AthenaCommon.Constants import INFO from AthenaCommon import CfgMgr +import os def setupScaleCalibrationInput(): + from AthenaCommon.AppMgr import ServiceMgr, ToolSvc print("Setting up setupScaleCalibrationInput") topSequence = AlgSequence() @@ -17,7 +18,7 @@ def setupScaleCalibrationInput(): # Setup GRL GRL = os.path.expandvars( '$TestArea/PhysicsAnalysis/MuonID/MuonPerformanceAnalysis/MuonPtCalibNtupleMaker/grl/data12_8TeV.periodAllYear_HEAD_DQDefects-00-01-02_PHYS_CombinedPerf_Muon_Muon_calo.xml' ) ToolSvc += CfgMgr.GoodRunsListSelectionTool( 'GRLTool' , GoodRunsListVec = [ GRL ] ) - + # Setup MST ToolSvc += CfgMgr.CP__MuonSelectionTool( 'MuonSelectorTool', OutputLevel = INFO ) ToolSvc += CfgMgr.CalibMuonsSelectorTool( 'CalibMuonsSelectorTool', MuonSelectorTool = ToolSvc.MuonSelectorTool, OutputLevel = INFO ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py index 635cdf8cc4fb..e9d657bea796 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py @@ -1,6 +1,4 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration - -from __future__ import print_function +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from AthenaCommon.Include import include -- GitLab From a09cc4d633e9c5751fda32dbae331eb3908973ae Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Mon, 7 Dec 2020 13:36:50 +0100 Subject: [PATCH 4/4] MuonCombinedRecExample: delete obsolete calibration modules Delete obsolete `ScaleCalibrationInputSetup.py` and `TagAndProbeInputSetup.py` modules and their corresponding `MuonCombinedRecFlags`. --- .../python/MuonCombinedRecFlags.py | 10 --- .../python/ScaleCalibrationInputSetup.py | 27 -------- .../python/TagAndProbeInputSetup.py | 68 ------------------- .../share/MuonCombinedRec_postprocessing.py | 8 --- 4 files changed, 113 deletions(-) delete mode 100644 Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py delete mode 100644 Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py index 85e4a83d1b07..481b41b04e88 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/MuonCombinedRecFlags.py @@ -144,16 +144,6 @@ class printSummary(JobProperty): allowedTypes=['bool'] StoredValue=False -class createTagAndProbeInput(JobProperty): - statusOn=True - allowedTypes=['bool'] - StoredValue=False - -class createScaleCalibrationInput(JobProperty): - statusOn=True - allowedTypes=['bool'] - StoredValue=False - ## Decide whether to wrap the new configuration in the old. class useNewConfig(JobProperty): statusOn=True diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py deleted file mode 100644 index aaa2cea36959..000000000000 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/ScaleCalibrationInputSetup.py +++ /dev/null @@ -1,27 +0,0 @@ -# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration - -from AthenaCommon.AlgSequence import AlgSequence -from AthenaCommon.Constants import INFO -from AthenaCommon import CfgMgr -import os - -def setupScaleCalibrationInput(): - from AthenaCommon.AppMgr import ServiceMgr, ToolSvc - print("Setting up setupScaleCalibrationInput") - - topSequence = AlgSequence() - # THistSvc configuration - from GaudiSvc.GaudiSvcConf import THistSvc - ServiceMgr += THistSvc() - ServiceMgr.THistSvc.Output += [ "MCPCALIB DATAFILE='MCPCalibNtuple_xAOD.root' OPT='RECREATE'" ] - - # Setup GRL - GRL = os.path.expandvars( '$TestArea/PhysicsAnalysis/MuonID/MuonPerformanceAnalysis/MuonPtCalibNtupleMaker/grl/data12_8TeV.periodAllYear_HEAD_DQDefects-00-01-02_PHYS_CombinedPerf_Muon_Muon_calo.xml' ) - ToolSvc += CfgMgr.GoodRunsListSelectionTool( 'GRLTool' , GoodRunsListVec = [ GRL ] ) - - # Setup MST - ToolSvc += CfgMgr.CP__MuonSelectionTool( 'MuonSelectorTool', OutputLevel = INFO ) - ToolSvc += CfgMgr.CalibMuonsSelectorTool( 'CalibMuonsSelectorTool', MuonSelectorTool = ToolSvc.MuonSelectorTool, OutputLevel = INFO ) - - # Setup algorithm - topSequence += CfgMgr.CalibMuonsNtupleMaker( 'CalibMuonsNtupleMaker', OutputLevel = INFO, SelectorTool = SelectorTool, ExtendedVersion = False ) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py deleted file mode 100644 index e9d657bea796..000000000000 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/python/TagAndProbeInputSetup.py +++ /dev/null @@ -1,68 +0,0 @@ -# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration - -from AthenaCommon.Include import include - - -def setupTagAndProbeInput(): - print("Setting up setupTagAndProbeInput") - - #from MuonPerformanceAlgs.ZmumuTPAnalysis import * - #from MuonPerformanceAlgs.ZmumuTPIsolationAnalysis import * - #from MuonPerformanceAlgs.ZmumuTPTrigAnalysis import * - include('MuonPerformanceAlgs/ZmumuTPAnalysis.py') - include('MuonPerformanceAlgs/ZmumuTPIsolationAnalysis.py') - include('MuonPerformanceAlgs/ZmumuTPTrigAnalysis.py') - - # now we can add the TP analysis/es itself/themselves :) - - ############## Zmumu T&P Configuration ######################## - do_Zmumu_RecoEff_TagProbe = True # this is for Z->mumu, reco eff. You can add your own analysis in a similar way. - do_Zmumu_IsolationEff_TagProbe = True # this is for Z->mumu, isolation eff. You can add your own analysis in a similar way. - do_Zmumu_Trigger_TagProbe = True # this is for Z->mumu, Trigger eff. - - ##### General analysis options - writeNtuple = True # Write an ntuple on top of the histos - for detailed studies, but increases output file size - doEtaSlices = False - doClosure = False - doDRSys = False - doValid = False - - - # Add utilities (tool, upstream algorithms) we need - AddIsolationTools() - AddTrigDecisionTool() - AddTrigMatchingTool() - AddMuonSelectionTool() - - - ##### Reco eff analysis options - if do_Zmumu_RecoEff_TagProbe: - writeNtuple = True - doEtaSlices = False - doClosure = False - doDRSys = True - doValid = True - AddZmumuTPAnalysis(doEtaSlices,writeNtuple,doClosure,doDRSys,doValid) - ############################################################### - - - ##### Isolation eff analysis options - if do_Zmumu_IsolationEff_TagProbe: - doEtaSlices = False - doClosure = False - doDRSys = True - doValid = True - AddZmumuTPIsolationAnalysis(doEtaSlices,writeNtuple,doClosure,doDRSys) - pass - ############################################################### - - - ##### Trigger analysis options - if do_Zmumu_Trigger_TagProbe: - doTriggerL1 = True - doTriggerL2 = False - doTriggerEF = False - doTriggerHLT = True - doEtaSlices = True - doDRSys = False - AddTriggerTPAnalysis(doEtaSlices,writeNtuple,doClosure,doDRSys,doL1=doTriggerL1,doL2=doTriggerL2,doEF=doTriggerEF,doHLT=doTriggerHLT) diff --git a/Reconstruction/MuonIdentification/MuonCombinedRecExample/share/MuonCombinedRec_postprocessing.py b/Reconstruction/MuonIdentification/MuonCombinedRecExample/share/MuonCombinedRec_postprocessing.py index a662293fd218..4f735164d3aa 100644 --- a/Reconstruction/MuonIdentification/MuonCombinedRecExample/share/MuonCombinedRec_postprocessing.py +++ b/Reconstruction/MuonIdentification/MuonCombinedRecExample/share/MuonCombinedRec_postprocessing.py @@ -52,14 +52,6 @@ if rec.doTruth() and muonCombinedRecFlags.doxAOD() and rec.doMuonCombined(): if muonCombinedRecFlags.doTrackPerformance: include("MuonCombinedRecExample/MuonCombinedTrackPerformance_jobOptions.py") -if( muonCombinedRecFlags.createTagAndProbeInput() ): - from MuonCombinedRecExample.TagAndProbeInputSetup import setupTagAndProbeInput - setupTagAndProbeInput() - -if( muonCombinedRecFlags.createScaleCalibrationInput() ): - from MuonCombinedRecExample.ScaleCalibrationInputSetup import setupScaleCalibrationInput - setupScaleCalibrationInput() - if rec.doMuonCombined() and rec.doJetMissingETTag() and muonCombinedRecFlags.doxAOD(): muonTCLinkAlg = CfgMgr.ClusterMatching__CaloClusterMatchLinkAlg("MuonTCLinks",ClustersToDecorate="MuonClusterCollection") topSequence += muonTCLinkAlg -- GitLab