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
Make the CUDA/SoA pixel clusterizer use the same thresholds as the legacy module [12.0.x] #35571
Make the CUDA/SoA pixel clusterizer use the same thresholds as the legacy module [12.0.x] #35571
Conversation
enable gpu |
please test |
A new Pull Request was created by @fwyzard (Andrea Bocci) for CMSSW_12_0_X. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
This PR may or may not be needed, depending if and how #35518 is backported to 12.0.x . |
please abort I don't see the tests running, trying to restart them. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5d1eb3/19475/summary.html GPU Comparison SummarySummary:
Comparison SummarySummary:
|
Yes, exactly:
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Make the CUDA/SoA pixel clusterizer use the same thresholds as the legacy module.
This allows for changes like those in #35518 to be picked up transparently by the CUDA/SoA modules.
In addition, make the legacy pixel channel threshold coherent with the value used in the gpu module.
PR validation:
Checked that modifications to the parameters in the legacy modules propagate to the CUDA/SoA modules.
if this PR is a backport please specify the original PR and why you need to backport that PR:
Backport #35570 with additional changes.