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
Bugfix in L1 GCT ET sums hardware emulator #9437
Conversation
A new Pull Request was created by @nsmith- (Nicholas Smith) for CMSSW_7_5_X. Bugfix in L1 GCT ET sums hardware emulator It involves the following packages: L1Trigger/L1TCalorimeter @cmsbuild, @mulhearn can you please review it and eventually sign? Thanks. |
please test |
+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_7_5_X IBs or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
+1 |
Bugfix in L1 GCT ET sums hardware emulator
Algorithm bug triggered an assertion.
Fix has been discussed with @mulhearn
This is identical to #9430, but applied to 75X
First observed in Run 245684, Event 1063307554, LumiSection 476.
To reproduce, need to change the region sums source in
L1Trigger/L1TCalorimeter/test/SimL1Emulator_Stage1.py
to the hardware values rather than reproducing from RAW. This is due to the particular run setup for that run.The event can be found at
/afs/cern.ch/user/z/zhangj/public/Splashes_Run245684_1_2_IKY.root
The output was