Skip to content

cmsDriver crash with customisation to remove MC matching #48170

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

Open
thomreis opened this issue May 25, 2025 · 5 comments
Open

cmsDriver crash with customisation to remove MC matching #48170

thomreis opened this issue May 25, 2025 · 5 comments

Comments

@thomreis
Copy link
Contributor

I observe a crash of cmsDriver.py in a reco step in CMSSW_15_0_5 when adding a customisation to remove MC matching.
This was possibly introduced with #46196.

Customisation:

def customise(process):
    from PhysicsTools.PatAlgos.tools.coreTools import removeMCMatching
    removeMCMatching(process, ['All'], outputModules=[], postfix="")
    return process

Output:

$ cmsDriver.py reco -s RAW2DIGI,L1Reco,RECO,PAT --eventcontent '' --conditions 150X_dataRun3_Prompt_v1 --era Run3 --data --python_filename=align_rereco_cfg.py --no_exec --processName=EcalAlignment --customise customisetest.py -n -1 --filein=root://cms-xrd-global.cern.ch//store/data/Run2025C/EGamma1/RAW-RECO/ZElectron-PromptReco-v1/000/392/363/00000/cc8df3a1-bd7a-41b0-9f91-a937d49c099e.root
RAW2DIGI,L1Reco,RECO,PAT,ENDJOB
entry root://cms-xrd-global.cern.ch//store/data/Run2025C/EGamma1/RAW-RECO/ZElectron-PromptReco-v1/000/392/363/00000/cc8df3a1-bd7a-41b0-9f91-a937d49c099e.root
Step: RAW2DIGI Spec: 
Step: L1Reco Spec: 
Step: RECO Spec: 
Step: PAT Spec: 
Step: ENDJOB Spec: 
customising the process with customise from customisetest
************** MC dependence removal ************
removing MC dependencies for photons
removing MC dependencies for out of time photons
removing MC dependencies for electrons
removing MC dependencies for LowPtElectrons
removing MC dependencies for muons
removing MC dependencies for taus
removing MC dependencies for taus boosted 
...skipped since taus boosted  are not part of process.
removing MC dependencies for jets
removing MC dependencies for AK8 jets
customising the process with miniAOD_customizeAllData from PhysicsTools/PatAlgos/slimming/miniAOD_tools
Traceback (most recent call last):
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/bin/el8_amd64_gcc12/cmsDriver.py", line 40, in <module>
    run()
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/bin/el8_amd64_gcc12/cmsDriver.py", line 16, in run
    configBuilder.prepare()
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/src/Configuration/Applications/python/ConfigBuilder.py", line 2472, in prepare
    self.pythonCfgCode += self.addCustomise(1)
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/src/Configuration/Applications/python/ConfigBuilder.py", line 958, in addCustomise
    self.process=getattr(package,fcn)(self.process)
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/src/PhysicsTools/PatAlgos/python/slimming/miniAOD_tools.py", line 699, in miniAOD_customizeAllData
    miniAOD_customizeCommon(process)
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/src/PhysicsTools/PatAlgos/python/slimming/miniAOD_tools.py", line 584, in miniAOD_customizeCommon
    _addUTagToTaus(process, task,
  File "/cvmfs/cms.cern.ch/el8_amd64_gcc12/cms/cmssw/CMSSW_15_0_5/src/PhysicsTools/PatAlgos/python/slimming/miniAOD_tools.py", line 537, in _addUTagToTaus
    process.tauGenJetMatch.clone(src = jetCollection),
AttributeError: 'Process' object has no attribute 'tauGenJetMatch'
@cmsbuild
Copy link
Contributor

cmsbuild commented May 25, 2025

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @thomreis.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor

assign Configuration/Applications

@cmsbuild
Copy link
Contributor

New categories assigned: operations

@antoniovilela,@davidlange6,@fabiocos,@mandrenguyen,@rappoccio you have been requested to review this Pull request/Issue and eventually sign? Thanks

@makortel
Copy link
Contributor

FYI @cms-sw/xpog-l2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants