-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Follow up on using cms::alpakatools deltaPhi functions #47405
Follow up on using cms::alpakatools deltaPhi functions #47405
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47405/43795 |
A new Pull Request was created by @VourMa for master. It involves the following packages:
@cmsbuild, @jfernan2, @mandrenguyen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
@cmsbuild please test |
+1 Size: This PR adds an extra 28KB to repository Comparison SummarySummary:
GPU Comparison SummarySummary:
|
assign heterogeneous |
+1 |
+heterogeneous |
enable gpu |
(sorry, I missed that the test had already been run!) |
+1 Size: This PR adds an extra 28KB to repository Comparison SummarySummary:
GPU Comparison SummarySummary:
|
+1 |
This is a follow up to PR #47208, where I missed moving some of the LST functions to the newly introduced
cms::alpakatools
deltaPhi functions. This PR takes care of those.The code has been tested locally: it compiles and gives the identical results (on CPU) for LST.