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 1x1 TP production fixes #12916
HF 1x1 TP production fixes #12916
Conversation
With the 1x1 TP geometry, the ieta ring 29 gets added to ieta ring 30 for 1x1 TP. Properly sum up the values by using a vector of samples instead of a single sample in the intermediate data aggregation.
A new Pull Request was created by @matz-e (Matthias Wolf) for CMSSW_8_0_X. It involves the following packages: SimCalorimetry/HcalTrigPrimAlgos @cmsbuild, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks. Following commands in first line of a comment are recognized
|
+1 |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
Keep long and short fibers together and treat them at the same time. More in line with Alex Gudes original algo.
Pull request #12916 was updated. @cmsbuild, @mulhearn, @davidlange6 can you please check and sign again. |
Pull request #12916 was updated. @cmsbuild, @mulhearn, @davidlange6 can you please check and sign again. |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
May we have it in pre5, please? |
@mulhearn - ping |
@mulhearn - ping |
I declare that @mulhearn signed it once already... |
HF 1x1 TP production fixes
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
Sorry for missed ping... |
I have discovered two bugs in the code ported:
With the fixes in this PR, 1x1 TP matching to the 2x3 TP is improved a lot.