Skip to content

Commit

Permalink
Merge pull request #16501 from lgray/factorize_timing_customization
Browse files Browse the repository at this point in the history
Factorize fast timing layer geometry dependent things from RECO algorithms.
  • Loading branch information
davidlange6 authored Nov 11, 2016
2 parents efa9ca8 + e2bfbcb commit 350469b
Show file tree
Hide file tree
Showing 7 changed files with 20 additions and 7 deletions.
7 changes: 7 additions & 0 deletions Configuration/Eras/python/Era_Phase2C2_timing_layer_cff.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
import FWCore.ParameterSet.Config as cms

from Configuration.Eras.Era_Phase2C2_timing_cff import Phase2C2_timing
from Configuration.Eras.Modifier_phase2_timing_layer_cff import phase2_timing_layer

Phase2C2_timing_layer = cms.ModifierChain(Phase2C2_timing, phase2_timing_layer)

4 changes: 4 additions & 0 deletions Configuration/Eras/python/Modifier_phase2_timing_layer_cff.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import FWCore.ParameterSet.Config as cms

phase2_timing_layer = cms.Modifier()

2 changes: 1 addition & 1 deletion Configuration/Geometry/scripts/dict2023Geometry.py
Original file line number Diff line number Diff line change
Expand Up @@ -496,7 +496,7 @@
'from Geometry.HGCalCommonData.fastTimeParametersInitialization_cfi import *',
'from Geometry.HGCalCommonData.fastTimeNumberingInitialization_cfi import *',
],
"era" : "phase2_timing",
"era" : "phase2_timing, phase2_timing_layer",
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@
'Geom' : 'Extended2023D5',
'HLTmenu': '@fake',
'GT' : 'auto:phase2_realistic',
'Era' : 'Phase2C2_timing',
'Era' : 'Phase2C2_timing_layer',
'ScenToRun' : ['GenSimHLBeamSpotFull','DigiFull','RecoFullGlobal', 'HARVESTFullGlobal'],
},
'2023D6' : {
Expand Down
5 changes: 3 additions & 2 deletions Configuration/StandardSequences/python/Eras.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ def __init__(self):
'Phase2C1',
'Phase2C2',
'Phase2C1_timing',
'Phase2C2_timing']
'Phase2C2_timing',
'Phase2C2_timing_layer']

internalUseMods = ['run2_common', 'run2_25ns_specific',
'run2_50ns_specific', 'run2_HI_specific',
Expand All @@ -38,7 +39,7 @@ def __init__(self):
'phase1Pixel', 'run3_GEM',
'phase2_common', 'phase2_tracker',
'phase2_hgcal', 'phase2_muon', 'phase2_timing',
'phase2_hcal',
'phase2_timing_layer','phase2_hcal',
'trackingLowPU', 'trackingPhase1', 'trackingPhase1PU70', 'ctpps_2016', 'trackingPhase2PU140',
'tracker_apv_vfp30_2016']
internalUseModChains = ['run2_2017_core']
Expand Down
3 changes: 2 additions & 1 deletion SimGeneral/MixingModule/python/digitizers_cfi.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,11 +57,12 @@

from SimGeneral.MixingModule.ecalTimeDigitizer_cfi import ecalTimeDigitizer
from Configuration.Eras.Modifier_phase2_timing_cff import phase2_timing
from Configuration.Eras.Modifier_phase2_timing_layer_cff import phase2_timing_layer
phase2_timing.toModify( theDigitizers,
ecalTime = ecalTimeDigitizer.clone() )

from SimFastTiming.Configuration.SimFastTiming_cff import fastTimeDigitizer
phase2_timing.toModify( theDigitizers,
phase2_timing_layer.toModify( theDigitizers,
fastTimingLayer = fastTimeDigitizer.clone() )

theDigitizersValid = cms.PSet(
Expand Down
4 changes: 2 additions & 2 deletions SimGeneral/MixingModule/python/mixObjects_cfi.py
Original file line number Diff line number Diff line change
Expand Up @@ -247,8 +247,8 @@
)
)

from Configuration.Eras.Modifier_phase2_timing_cff import phase2_timing
phase2_timing.toModify( theMixObjects,
from Configuration.Eras.Modifier_phase2_timing_layer_cff import phase2_timing_layer
phase2_timing_layer.toModify( theMixObjects,
mixSH = dict(
input = theMixObjects.mixSH.input + [ cms.InputTag("g4SimHits","FastTimerHitsBarrel"), cms.InputTag("g4SimHits","FastTimerHitsEndcap") ],
subdets = theMixObjects.mixSH.subdets + [ 'FastTimerHitsBarrel','FastTimerHitsEndcap' ],
Expand Down

0 comments on commit 350469b

Please sign in to comment.