Skip to content

Commit 7ac2590

Browse files
author
AdrianoDee
committed
New GenOnly and SimOnGen wfs:
- Adding Sim step. - Introducing GenOnly and SimOnGen wfs for 2024 and D110. - Generalizing ALCA inputs.
1 parent f1b0254 commit 7ac2590

File tree

4 files changed

+64
-9
lines changed

4 files changed

+64
-9
lines changed

Configuration/PyReleaseValidation/python/WorkFlowRunner.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ def __init__(self, wf, noRun=False,dryRun=False,cafVeto=True,dasOptions="",jobRe
2525
self.nStreams=nStreams
2626
self.maxSteps=maxSteps
2727
self.nEvents=nEvents
28+
self.recoOutput=''
2829

2930
self.wfDir=str(self.wf.numId)+'_'+self.wf.nameId
3031
return
@@ -158,9 +159,16 @@ def closeCmd(i,ID):
158159
# Disable input for premix stage2 in FastSim to allow combined stage1+stage2 workflow (in FS, stage2 does also GEN)
159160
# Ugly hack but works
160161
if istep!=1 and not '--filein' in cmd and not 'premix_stage1' in cmd and not ("--fast" in cmd and "premix_stage2" in cmd):
161-
cmd+=' --filein file:step%s.root '%(istep-1,)
162+
if "ALCA" not in cmd:
163+
cmd+=' --filein file:step%s.root '%(istep-1,)
164+
elif "ALCA" in cmd and "RECO" in cmd:
165+
cmd+=' --filein file:step%s.root '%(istep-1,)
166+
else:
167+
cmd+=' --filein %s'%(self.recoOutput)
162168
if not '--fileout' in com:
163169
cmd+=' --fileout file:step%s.root '%(istep,)
170+
if "RECO" in cmd:
171+
self.recoOutput = "file:step%d.root"%(istep)
164172
if self.jobReport:
165173
cmd += ' --suffix "-j JobReport%s.xml " ' % istep
166174
if (self.nThreads > 1) and ('HARVESTING' not in cmd) and ('ALCAHARVEST' not in cmd):

Configuration/PyReleaseValidation/python/relval_steps.py

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4222,6 +4222,7 @@ def gen2023HiMix(fragment,howMuch):
42224222
defaultDataSets['2023FS']='CMSSW_13_0_11-130X_mcRun3_2023_realistic_withEarly2023BS_v1_FastSim-v'
42234223
defaultDataSets['2024']='CMSSW_14_0_0_pre3-140X_mcRun3_2024_realistic_v1_STD_2024_noPU-v'
42244224
defaultDataSets['2024HLTOnDigi']='CMSSW_14_0_0_pre3-140X_mcRun3_2024_realistic_v1_STD_2024_noPU-v'
4225+
defaultDataSets["2024SimOnGen"] = 'CMSSW_14_0_0_pre3-140X_mcRun3_2024_realistic_v1_STD_2024_noPU-v'
42254226
defaultDataSets['2026D49']='CMSSW_12_0_0_pre4-113X_mcRun4_realistic_v7_2026D49noPU-v'
42264227
defaultDataSets['2026D76']='CMSSW_12_0_0_pre4-113X_mcRun4_realistic_v7_2026D76noPU-v'
42274228
defaultDataSets['2026D77']='CMSSW_12_1_0_pre2-113X_mcRun4_realistic_v7_2026D77noPU-v'
@@ -4302,6 +4303,7 @@ def gen2023HiMix(fragment,howMuch):
43024303
'--eventcontent': 'FEVTDEBUG',
43034304
'--geometry' : geom
43044305
}
4306+
43054307
if beamspot is not None: upgradeStepDict['GenSim'][k]['--beamspot']=beamspot
43064308

43074309
upgradeStepDict['GenSimHLBeamSpot'][k]= {'-s' : 'GEN,SIM',
@@ -4331,6 +4333,18 @@ def gen2023HiMix(fragment,howMuch):
43314333
'--geometry' : geom
43324334
}
43334335

4336+
upgradeStepDict['Sim'][k]= {'-s' : 'SIM',
4337+
'-n' : 10,
4338+
'--conditions' : gt,
4339+
'--beamspot' : 'Realistic25ns13TeVEarly2017Collision',
4340+
'--datatier' : 'SIM',
4341+
'--eventcontent': 'FEVTDEBUG',
4342+
'--geometry' : geom
4343+
}
4344+
4345+
if beamspot is not None: upgradeStepDict['Sim'][k]['--beamspot']=beamspot
4346+
4347+
43344348
upgradeStepDict['Digi'][k] = {'-s':'DIGI:pdigi_valid,L1,DIGI2RAW,HLT:%s'%(hltversion),
43354349
'--conditions':gt,
43364350
'--datatier':'GEN-SIM-DIGI-RAW',
@@ -4461,7 +4475,6 @@ def gen2023HiMix(fragment,howMuch):
44614475
'-n':'10',
44624476
'--eventcontent':'ALCARECO',
44634477
'--geometry' : geom,
4464-
'--filein':'file:step3.root'
44654478
}
44664479

44674480
upgradeStepDict['ALCAPhase2'][k] = merge([{'-s':'ALCA:SiPixelCalSingleMuonLoose+SiPixelCalSingleMuonTight+TkAlMuonIsolated+TkAlMinBias+MuAlOverlaps+EcalESAlign+TkAlZMuMu+TkAlDiMuonAndVertex+HcalCalHBHEMuonProducerFilter+TkAlUpsilonMuMu+TkAlJpsiMuMu'},upgradeStepDict['ALCA'][k]])
@@ -4545,7 +4558,7 @@ def gen2023HiMix(fragment,howMuch):
45454558

45464559
for step in upgradeStepDict.keys():
45474560
# we need to do this for each fragment
4548-
if ('Sim' in step and 'Fast' not in step) or ('Premix' in step) or ('Sim' not in step and 'Gen' in step):
4561+
if ('Sim' in step and ('Fast' not in step and step != 'Sim')) or ('Premix' in step) or ('Sim' not in step and 'Gen' in step):
45494562
for frag,info in upgradeFragments.items():
45504563
howMuch=info.howMuch
45514564
for key in [key for year in upgradeKeys for key in upgradeKeys[year]]:

Configuration/PyReleaseValidation/python/relval_upgrade.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ def makeStepName(key,frag,step,suffix):
3030
hasHarvest = False
3131
for step in upgradeProperties[year][key]['ScenToRun']:
3232
stepMaker = makeStepName
33-
if 'Sim' in step and 'Fast' not in step:
33+
if 'Sim' in step and 'Fast' not in step and step != "Sim":
3434
if 'HLBeamSpot' in step:
3535
if '14TeV' in frag:
3636
step = 'GenSimHLBeamSpot14'

Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py

Lines changed: 39 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,9 @@
3737
'2023HI',
3838
'2023HIRP', #RawPrime
3939
'2024HLTOnDigi',
40-
'2024HLTOnDigiPU'
41-
40+
'2024HLTOnDigiPU',
41+
'2024GenOnly',
42+
'2024SimOnGen',
4243
]
4344

4445
upgradeKeys[2026] = [
@@ -93,7 +94,9 @@
9394
'2026D113',
9495
'2026D113PU',
9596
'2026D114',
96-
'2026D114PU'
97+
'2026D114PU',
98+
'2026D110GenOnly',
99+
'2026D110SimOnGen',
97100
]
98101

99102
# pre-generation of WF numbers
@@ -178,7 +181,7 @@ def workflow_(self, workflows, num, fragment, stepList, key):
178181
def condition(self, fragment, stepList, key, hasHarvest):
179182
return False
180183
def preventReuse(self, stepName, stepDict, k):
181-
if "Sim" in stepName:
184+
if "Sim" in stepName and stepName != "Sim":
182185
stepDict[stepName][k] = None
183186
if "Gen" in stepName:
184187
stepDict[stepName][k] = None
@@ -198,6 +201,7 @@ def condition(self, fragment, stepList, key, hasHarvest):
198201
upgradeWFs['baseline'] = UpgradeWorkflow_baseline(
199202
steps = [
200203
'Gen',
204+
'Sim',
201205
'GenSim',
202206
'GenSimHLBeamSpot',
203207
'GenSimHLBeamSpot14',
@@ -2933,7 +2937,22 @@ def condition(self, fragment, stepList, key, hasHarvest):
29332937
'Era':'Run3_pp_on_PbPb_approxSiStripClusters',
29342938
'BeamSpot': 'DBrealistic',
29352939
'ScenToRun' : ['GenSim','Digi','RecoNano','HARVESTNano','ALCA'],
2936-
}
2940+
},
2941+
'2024GenOnly' : {
2942+
'Geom' : 'DB:Extended',
2943+
'GT' : 'auto:phase1_2024_realistic',
2944+
'Era' : 'Run3',
2945+
'BeamSpot': 'DBrealistic',
2946+
'ScenToRun' : ['Gen'],
2947+
},
2948+
'2024SimOnGen' : {
2949+
'Geom' : 'DB:Extended',
2950+
'GT' : 'auto:phase1_2024_realistic',
2951+
'HLTmenu': '@relval2024',
2952+
'Era' : 'Run3',
2953+
'BeamSpot': 'DBrealistic',
2954+
'ScenToRun' : ['Gen','Sim','Digi','RecoNano','HARVESTNano','ALCA'],
2955+
},
29372956
}
29382957

29392958
# standard PU sequences
@@ -3133,6 +3152,21 @@ def condition(self, fragment, stepList, key, hasHarvest):
31333152
'Era' : 'Phase2C17I13M9',
31343153
'ScenToRun' : ['GenSimHLBeamSpot','DigiTrigger','RecoGlobal', 'HARVESTGlobal', 'ALCAPhase2'],
31353154
},
3155+
'2026D110GenOnly' : {
3156+
'Geom' : 'Extended2026D110',
3157+
'Beamspot' : 'HLLHC',
3158+
'GT' : 'auto:phase2_realistic_T33',
3159+
'Era' : 'Phase2C17I13M9',
3160+
'ScenToRun' : ['Gen'],
3161+
},
3162+
'2026D110SimOnGen' : {
3163+
'Geom' : 'Extended2026D110',
3164+
'HLTmenu': '@relval2026',
3165+
'Beamspot' : 'HLLHC',
3166+
'GT' : 'auto:phase2_realistic_T33',
3167+
'Era' : 'Phase2C17I13M9',
3168+
'ScenToRun' : ['Gen','Sim','DigiTrigger','RecoGlobal', 'HARVESTGlobal', 'ALCAPhase2'],
3169+
},
31363170
}
31373171

31383172
# standard PU sequences

0 commit comments

Comments
 (0)