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
Added a flag for enabling the application of the TED and TRE tables. #30847
Conversation
@aehart, CMSSW_11_2_X branch is closed for direct updates. cms-bot is going to move this PR to master branch. |
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30847/17196
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
please test |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30847/17199
|
The tests are being triggered in jenkins.
|
A new Pull Request was created by @aehart (Andrew Hart) for master. It involves the following packages: L1Trigger/TrackFindingTracklet @cmsbuild, @rekovic, @benkrikler can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
Comparison job queued. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
kind reminder @cms-sw/l1-l2 |
pending decision on #30818 |
I'm converting this PR as a draft just to remove this from the ORP spreadsheet. Of course feel free to open it back if needed. |
@rekovic @aehart @skinnari @tomalin I missed your comments in #30818 because I was on holiday. As soon as we managed to start the HLT TDR re-reco production, we decided to keep everything unchanged to avoid any possible risk. I think it would be good to have it in master. If you want to reactivate this PR, @aehart please push on the button "Ready for review" |
PR description:
This is a port to CMSSW_11_2_X of #30818, which disables the lookup tables in the TrackletEngineDisplaced and TripletEngine, which were found to cause excessive memory consumption.