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
ECAL GPU unpacker: adapt the buffer size to the ECAL FEDs size [12.3.x] #38205
ECAL GPU unpacker: adapt the buffer size to the ECAL FEDs size [12.3.x] #38205
Conversation
type bugfix |
A new Pull Request was created by @fwyzard (Andrea Bocci) for CMSSW_12_3_X. It involves the following packages:
@Martin-Grunewald, @clacaputo, @cmsbuild, @missirol, @slava77, @jpata can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
backport #38202 |
enable gpu |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc0203/25210/summary.html GPU Comparison SummarySummary:
Comparison SummarySummary:
|
Automatically determine the cpu and gpu buffers' size used for the unpacking from the total size of the non-empty ECAL FEDs.
97e7f2b
to
7ffb562
Compare
Rebased to fix the merge conflict. |
enable gpu |
Pull request #38205 was updated. @Martin-Grunewald, @clacaputo, @cmsbuild, @missirol, @slava77, @jpata can you please check and sign again. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc0203/25326/summary.html GPU Comparison SummarySummary:
Comparison SummarySummary:
|
@@ -189,12 +189,23 @@ def customiseForOffline(process): | |||
return process | |||
|
|||
|
|||
# ECAL GPU unpacker: adapt the buffer size to the ECAL FEDs size (#38202) | |||
# remove the EcalRawToDigi.maxFedSize parameter from the menu | |||
def customizeHLTfor38202(process): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be 38205
, but I don't object, it's clear enough.
+hlt |
There are some reco infinitesimal differences in GPU comparison that weren't present in #38204 and #38202 . |
Mhm, given that this PR touches only the ECAL unpacker and nothing Pixel-related, I would assume so. |
+reconstruction
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_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:
Automatically determine the cpu and gpu buffers' size used for the unpacking from the total size of the non-empty ECAL FEDs.
Remove the
maxFedSize
parameter from theEcalRawToDigi
modules in the HLT menus.PR validation:
None.
if this PR is a backport please specify the original PR and why you need to backport that PR:
Backport of #38202 .