-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[14.0.X] Add customize to include Alpaka HCal PF Clustering at HLT #43979
[14.0.X] Add customize to include Alpaka HCal PF Clustering at HLT #43979
Conversation
A new Pull Request was created by @waredjeb (Wahid Redjeb) for CMSSW_14_0_X. It involves the following packages:
@cmsbuild, @mmusich, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
cms-bot internal usage |
backport of #43971 |
type pf |
is there a backport of this too? |
please amend the PR description then. |
enable gpu |
test parameters:
|
will trigger test here only once #43971 (comment) is clarified. |
2b472df
to
28e9220
Compare
Pull request #43979 was updated. @mmusich, @Martin-Grunewald, @cmsbuild can you please check and sign again. |
@cmsbuild, please test |
@cmsbuild, please abort |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-26f5d9/37514/summary.html Comparison SummarySummary:
GPU Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_14_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_14_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
+1
|
This is a backport of #43971
This PR add
customizeHLTforAlpakaParticleFlowClustering()
tocustomizeHLTForAlpaka()
to integrate the Alpaka HCAL PF Clustering at HLT.The function defines all the needed modules and replace them in the necessary Sequences. Currently it also run the Alpaka CPU-serial version up to the PFClusterSoA collection, before the Legacy conversion.
To compare the GPU version against the CPU version a follow-up PR will be needed to perform the CPU legacy conversion and the comparison at DQM.
Validation
Validated running the following HLT configuration
And also by merging #43958 and running workflow 12434.423
FYI @jsamudio @hatakeyamak