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
re-improve check on size of L1-uGT digis in HLTL1TSeed
plugin
#41469
re-improve check on size of L1-uGT digis in HLTL1TSeed
plugin
#41469
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41469/35331
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4ceb94/32261/summary.html Comparison SummarySummary:
|
+hlt @cms-sw/orp-l2 , please consider this PR for the upcoming IB (it is not as urgent as #41466, but I would prefer to have this PR integrated and backported soon). |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR amends a mistake introduced in #40655. The check on
size == 0
should not have been removed in #40655, mainly because, ifBXVector::size() == 0
, the call toBXVector::isEmpty(0)
might lead to a crash (which is arguably a limit of the implementation ofBXVector::isEmpty(int bx)
).Thanks to @swagata87 for indirectly reminding me of this.
PR validation:
None.
If this PR is a backport, please specify the original PR and why you need to backport that PR. If this PR will be backported, please specify to which release cycle the backport is meant for:
CMSSW_13_0_X