Simplify LST input & output and some clean up of the HLT tracking-related sequence.#50819
Simplify LST input & output and some clean up of the HLT tracking-related sequence.#50819VourMa wants to merge 4 commits intocms-sw:masterfrom
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50819/49145
|
|
A new Pull Request was created by @VourMa for master. It involves the following packages:
@Martin-Grunewald, @Moanwar, @cmsbuild, @jfernan2, @mandrenguyen, @mmusich, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
test parameters:
|
|
please test |
|
-1 Failed Tests: HLTP2Timing Comparison SummarySummary:
|
Looking at the log it's unclear what actually failed. but the text around looks OK |
- Add flags for tracking/seeding production - Conditional creation of products - Per-iteration heap allocations moved out of the loop - Remove unnecessary OT hit loop + sort for pLS candidates
0c81f3f to
64bf303
Compare
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50819/49223
|
|
Pull request #50819 was updated. @Martin-Grunewald, @Moanwar, @cmsbuild, @jfernan2, @mandrenguyen, @mmusich, @srimanob can you please check and sign again. |
|
please test with #50867 |
|
+1 Size: This PR adds an extra 48KB to repository HLT P2 Timing: chart Comparison SummarySummary:
|
This is a technical PR, aiming to simplify and speed up the input and output of the LST algorithm. The physics performance has been verified to remain unchanged while the following small timing improvements were observed for the HLT:
As collateral developments of this PR: