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
[Backport HGC trigger] V10/V11 e/g ID + calibration and concentrator module sums #30355
[Backport HGC trigger] V10/V11 e/g ID + calibration and concentrator module sums #30355
Conversation
first implementation of module sums use module ID and position remove deprecated comment add ntuple for TriggerSums, fill mipPt and pos remove wafer u/v and sim energy applied code-format and code-checks Implement JBS's comments second round of JBS comments
A new Pull Request was created by @jbsauvan (Jean-Baptiste Sauvan) for CMSSW_11_1_X. It involves the following packages: L1Trigger/L1THGCal @cmsbuild, @rekovic, @benkrikler, @kpedro88 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins.
|
-1 Tested at: d9a75ea CMSSW: CMSSW_11_1_X_2020-06-23-2300 I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following workflows: runTheMatrix-results/20434.0_TTbar_14TeV+TTbar_14TeV_TuneCP5_2026D41_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D41+RecoFullGlobal_2026D41+HARVESTFullGlobal_2026D41/step2_TTbar_14TeV+TTbar_14TeV_TuneCP5_2026D41_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D41+RecoFullGlobal_2026D41+HARVESTFullGlobal_2026D41.log23234.0 step2 runTheMatrix-results/23234.0_TTbar_14TeV+TTbar_14TeV_TuneCP5_2026D49_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D49+RecoFullGlobal_2026D49+HARVESTFullGlobal_2026D49/step2_TTbar_14TeV+TTbar_14TeV_TuneCP5_2026D49_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D49+RecoFullGlobal_2026D49+HARVESTFullGlobal_2026D49.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
Somehow it didn't find in the tests the data added in cms-data/L1Trigger-L1THGCal#18 |
@smuzaffar @mrodozov |
@kpedro88 , cms-sw/cmsdist#5991 backport the changes for 11.1.X |
please test with cms-sw/cmsdist#5991 |
The tests are being triggered in jenkins.
|
+1 |
Comparison job queued. |
backport of #30289 |
Comparison is ready Comparison Summary:
|
+upgrade |
merge |
PR description:
Backport of #30289