-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
trackingOnly
relval matrix workflows looks messed up
#36347
Comments
A new Issue was created by @mmusich Marco Musich. @Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign pdmv |
New categories assigned: pdmv @bbilin,@wajidalikhan,@jordan-martins,@kskovpen you have been requested to review this Pull request/Issue and eventually sign? Thanks |
thanks. Indeed the 2021-like workflows look OK in $ runTheMatrix.py --what upgrade -nel 11624.1,11634.1,12024.1,12034.1,12424.1,12434.1,12824.1,12834.1
ignoring non-requested file relval_standard
ignoring non-requested file relval_highstats
ignoring non-requested file relval_pileup
ignoring non-requested file relval_generator
ignoring non-requested file relval_extendedgen
ignoring non-requested file relval_production
ignoring non-requested file relval_ged
processing relval_upgrade
ignoring non-requested file relval_cleanedupgrade
ignoring non-requested file relval_gpu
ignoring non-requested file relval_2017
ignoring non-requested file relval_2026
ignoring non-requested file relval_identity
ignoring non-requested file relval_machine
ignoring non-requested file relval_premix
found a total of 21000 workflows:
of which the following 8 were selected:
11624.1 2021_trackingOnly+TTbar_13TeV_TuneCUETP8M1_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2021_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,50
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2021_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2021_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2021_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
11634.1 2021_trackingOnly+TTbar_14TeV_TuneCP5_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_14TeV_TuneCP5_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2021_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,100
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2021_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2021_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2021_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12024.1 2021Design_trackingOnly+TTbar_13TeV_TuneCUETP8M1_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2021_design --beamspot GaussSigmaZ4cm --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,50
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2021_design --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2021_design --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2021_design --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12034.1 2021Design_trackingOnly+TTbar_14TeV_TuneCP5_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_14TeV_TuneCP5_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2021_design --beamspot GaussSigmaZ4cm --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,100
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2021_design --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2021_design --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2021_design --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12424.1 2023_trackingOnly+TTbar_13TeV_TuneCUETP8M1_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2023_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,50
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2023_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2023_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2023_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12434.1 2023_trackingOnly+TTbar_14TeV_TuneCP5_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_14TeV_TuneCP5_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2023_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,100
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2023_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2023_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2023_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12824.1 2024_trackingOnly+TTbar_13TeV_TuneCUETP8M1_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2024_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,50
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2024_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2024_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2024_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100
12834.1 2024_trackingOnly+TTbar_14TeV_TuneCP5_GenSim+Digi+Reco+HARVEST [1]: cmsDriver.py TTbar_14TeV_TuneCP5_cfi -s GEN,SIM -n 10 --conditions auto:phase1_2024_realistic --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --datatier GEN-SIM --eventcontent FEVTDEBUG --geometry DB:Extended --era Run3 --relval 9000,100
[2]: cmsDriver.py step2 -s DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2021 --conditions auto:phase1_2024_realistic --datatier GEN-SIM-DIGI-RAW -n 10 --eventcontent FEVTDEBUGHLT --geometry DB:Extended --era Run3
[3]: cmsDriver.py step3 -s RAW2DIGI,RECO:reconstruction_trackingOnly,VALIDATION:@trackingOnlyValidation,DQM:@trackingOnlyDQM --conditions auto:phase1_2024_realistic --datatier GEN-SIM-RECO,DQMIO -n 10 --eventcontent RECOSIM,DQM --geometry DB:Extended --era Run3
[4]: cmsDriver.py step4 -s HARVESTING:@trackingOnlyValidation+@trackingOnlyDQM --conditions auto:phase1_2024_realistic --mc --geometry DB:Extended --scenario pp --filetype DQM --era Run3 -n 100 The Run2 ones still use |
Looking in the IBs, the issue appears between CMSSW_12_2_X_2021-11-29-1100 and CMSSW_12_2_X_2021-11-29-2300 The following PRs were merged in between: A suspect being #36167, as Slava already suggested independently... |
... and by the way, it is not yet fixed in latest CMSSW_12_2_X_2021-12-03-1100 |
On the practical side: we were supposed to start building 12_2_0_pre3 by this evening. Should we wait to solve this issue before that? Or none of those workflows is needed to produce the validation samples in pre3 (in which case we can even wait to fix it in time for the final 12_2_o in January)? |
even though the workflows are not used for relval production, it's very inconvenent for development. |
This is indeed another casualty of the Nano step changes for Run3; I will submit a fix today. |
see #36350 |
@perrotta here are my observations based on
So even if the immediate concern regarding the development workflows for Run-3 and Phase-2 is settled, I prefer to let the issue open until also the last point is addressed. |
Sure, thank you @mmusich ! |
see #36378 |
@kpedro88 $ runTheMatrix.py --what upgrade -nel 10024.1
ignoring non-requested file relval_standard
ignoring non-requested file relval_highstats
ignoring non-requested file relval_pileup
ignoring non-requested file relval_generator
ignoring non-requested file relval_extendedgen
ignoring non-requested file relval_production
ignoring non-requested file relval_ged
processing relval_upgrade
ERROR reading file: relval_upgrade 'HARVESTFakeHLT_trackingMkFit_2017'
Traceback (most recent call last):
File "/cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_3_X_2021-12-12-0000/bin/slc7_amd64_gcc900/runTheMatrix.py", line 598, in <module>
ret = runSelected(opt)
File "/cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_3_X_2021-12-12-0000/bin/slc7_amd64_gcc900/runTheMatrix.py", line 23, in runSelected
mrd.prepare(opt.useInput, opt.refRel, opt.fromScratch)
File "/cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_3_X_2021-12-12-0000/python/Configuration/PyReleaseValidation/MatrixReader.py", line 506, in prepare
self.readMatrix(matrixFile, useInput, refRel, fromScratch)
File "/cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_3_X_2021-12-12-0000/python/Configuration/PyReleaseValidation/MatrixReader.py", line 251, in readMatrix
if self.relvalModule.steps[stepName] is None:
KeyError: 'HARVESTFakeHLT_trackingMkFit_2017' EDIT :if the |
Sorry, this was just a mistake in the previous PR: see #36477 |
Based on a recent test in
so everything looks OK. @cms-sw/pdmv-l2 you might want to sign, so that this can be closed. |
+1 |
This issue is fully signed and ready to be closed. |
In
CMSSW_12_2_X_2021-11-29-2300
out of the 9 trackingOnly wfs available in the standard matrix:RECO
and notRECO:reconstruction_trackingOnly
for what concerns the upgrade matrix:
RECO
and notRECO:reconstruction_trackingOnly
PLEASE FIX
The text was updated successfully, but these errors were encountered: