Skip to content
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

Hf shower library update #16049

Merged
merged 2 commits into from
Sep 30, 2016
Merged

Conversation

davidlange6
Copy link
Contributor

primary change is to reformat the HF shower library itself. Now each "event" is formatted as a vector of floats and the basket size is 1 - this means events can be read one at a time so much less time is spent reading the data (which is essentially randomly read).

needs cms-sw/cmsdist#2543

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_8_1_X.

It involves the following packages:

SimG4CMS/Calo
SimG4Core/Application

@cmsbuild, @civanch, @mdhildreth, @davidlange6 can you please review it and eventually sign? Thanks.
@makortel this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@davidlange6
Copy link
Contributor Author

please test with cms-sw/cmsdist#2543

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 30, 2016

The tests are being triggered in jenkins.
Using externals from cms-sw/cmsdist#2543
https://cmssdt.cern.ch/jenkins/job/cmsdist-cmssw-test-pr/77/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@davidlange6 davidlange6 merged commit f644896 into cms-sw:CMSSW_8_1_X Sep 30, 2016
@smuzaffar
Copy link
Contributor

@davidlange6 , should be keep v3 also in the external? Looks like cmssw code still has ref to it

FastSimulation/Calorimetry/python/Calorimetry_cff.py:run2_common.toModify(FamosCalorimetryBlock.Calorimetry.HFShowerLibrary, FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root' )
SLHCUpgradeSimulations/Configuration/python/fastSimCustoms.py:    process.famosSimHits.Calorimetry.HFShowerLibrary.FileName = cms.FileInPath('SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root
SLHCUpgradeSimulations/Configuration/python/postLS1Customs.py:    process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py_since760pre6:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py_since760pre6_forGEN_760pre6:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'

@davidlange6
Copy link
Contributor Author

No - we should clean up the v3 usages (wondering why I didn’t noticed the first one of these..)

On Oct 1, 2016, at 1:36 PM, Malik Shahzad Muzaffar notifications@github.com wrote:

@davidlange6 , should be keep v3 also in the external? Looks like cmssw code still has ref to it

FastSimulation/Calorimetry/python/Calorimetry_cff.py:run2_common.toModify(FamosCalorimetryBlock.Calorimetry.HFShowerLibrary, FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root' )
SLHCUpgradeSimulations/Configuration/python/fastSimCustoms.py: process.famosSimHits.Calorimetry.HFShowerLibrary.FileName = cms.FileInPath('SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root
SLHCUpgradeSimulations/Configuration/python/postLS1Customs.py: process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py_since760pre6:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'
Validation/CaloTowers/test/CaloScan/template.py_since760pre6_forGEN_760pre6:process.g4SimHits.HFShowerLibrary.FileName = 'SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root'


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Dr15Jones
Copy link
Contributor

This appears to have caused many of the RelVals to fail with

----- Begin Fatal Exception 01-Oct-2016 10:35:37 CEST-----------------------
An exception of category 'ConfigFileReadError' occurred while
   [0] Processing the python configuration file named TTbar_13TeV_TuneCUETP8M1_cfi_GEN_SIM_RECOBEFMIX_DIGI_L1_DIGI2RAW_L1Reco_RECO_EI_HLT_VALIDATION_DQM.py
Exception Message:
python encountered the error: <type 'exceptions.RuntimeError'>
edm::FileInPath unable to find file SimG4CMS/Calo/data/HFShowerLibrary_npmt_noatt_eta4_16en_v3.root anywhere in the search path.
The search path is defined by: CMSSW_SEARCH_PATH
${CMSSW_SEARCH_PATH} is: /build/cmsbld/jenkins-workarea/workspace/ib-run-relvals/CMSSW_8_1_X_2016-09-30-2300/src:/build/cmsbld/jenkins-workarea/workspace/ib-run-relvals/CMSSW_8_1_X_2016-09-30-2300/external/slc6_amd64_gcc530/data:/cvmfs/cms-ib.cern.ch/week0/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-09-30-2300/src:/cvmfs/cms-ib.cern.ch/week0/slc6_amd64_gcc530/cms/cmssw/CMSSW_8_1_X_2016-09-30-2300/external/slc6_amd64_gcc530/data
Current directory is: /build/cmsbld/jenkins-workarea/workspace/ib-run-relvals/CMSSW_8_1_X_2016-09-30-2300/pyRelval/135.1/135.1_TTbar_13+TTbarFS_13+HARVESTUP15FS+MINIAODMCUP15FS

----- End Fatal Exception -------------------------------------------------

@smuzaffar
Copy link
Contributor

#16066 should fix it

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

Successfully merging this pull request may close these issues.

4 participants