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
Switching 50 ns Run 2 configuration of HCAL noise filters to Run 1 mode #8833
Switching 50 ns Run 2 configuration of HCAL noise filters to Run 1 mode #8833
Conversation
A new Pull Request was created by @igv4321 for CMSSW_7_5_X. Switching 50 ns Run 2 configuration of HCAL noise filters to Run 1 mode It involves the following packages: CommonTools/RecoAlgos @civanch, @nclopezo, @cvuosalo, @mdhildreth, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar |
+1 |
Switching 50 ns Run 2 configuration of HCAL noise filters to Run 1 mode
|
||
def customise_DQM_25ns(process): | ||
# Switch the default decision of the HCAL noise filter | ||
if hasattr(process,'HBHENoiseFilterResultProducer'): | ||
process.HBHENoiseFilterResultProducer.defaultDecision = cms.string("HBHENoiseFilterResultRun2Loose") | ||
return process |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This gives me an error, presumably because the function customise_DQM_25ns is
used/called before it is defined.
Creating RECO+VALIDATION+DQM GRun_MC
RAW2DIGI,L1Reco,RECO,VALIDATION,DQM,ENDJOB
entry file:RelVal_DigiL1RawHLT_GRun_MC.root
Step: RAW2DIGI Spec:
Step: L1Reco Spec:
Step: RECO Spec:
Step: VALIDATION Spec:
in preparing validation
Step: DQM Spec:
Step: ENDJOB Spec:
# Conditions read from CMS_CONDITIONS via FrontierProd
customising the process with Base from HLTrigger/Configuration/CustomConfigs
customising the process with customisePostLS1 from SLHCUpgradeSimulations/Configuration/postLS1Customs
Traceback (most recent call last):
File "/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw/CMSSW_7_5_X_2015-04-23-2300/bin/slc6_amd64_gcc491/cmsDriver.py", line 43, in <module>
run()
File "/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw/CMSSW_7_5_X_2015-04-23-2300/bin/slc6_amd64_gcc491/cmsDriver.py", line 15, in run
configBuilder.prepare()
File "/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw/CMSSW_7_5_X_2015-04-23-2300/python/Configuration/Applications/ConfigBuilder.py", line 2188, in prepare
self.pythonCfgCode += self.addCustomise()
File "/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw/CMSSW_7_5_X_2015-04-23-2300/python/Configuration/Applications/ConfigBuilder.py", line 869, in addCustomise
self.process=getattr(package,fcn)(self.process)
File "/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw/CMSSW_7_5_X_2015-04-23-2300/python/SLHCUpgradeSimulations/Configuration/postLS1Customs.py", line 53, in customisePostLS1
process = customise_DQM_25ns(process)
NameError: global name 'customise_DQM_25ns' is not defined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, David mentioned a race condition...
As the title mentions. Consistent with PR #8805.