-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Hf shower library update #16049
Conversation
A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_8_1_X. It involves the following packages: SimG4CMS/Calo @cmsbuild, @civanch, @mdhildreth, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
please test with cms-sw/cmsdist#2543 |
The tests are being triggered in jenkins. |
@davidlange6 , should be keep v3 also in the external? Looks like cmssw code still has ref to it
|
No - we should clean up the v3 usages (wondering why I didn’t noticed the first one of these..)
|
This appears to have caused many of the RelVals to fail with
|
#16066 should fix it |
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