CTPPS Pixel unpacker patch (backport) #20121
Conversation
|
A new Pull Request was created by @fabferro (Fabrizio Ferro) for CMSSW_9_2_X. It involves the following packages: EventFilter/CTPPSRawToDigi @perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
|
@cmsbuild please test |
|
The tests are being triggered in jenkins. |
|
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
|
Comparison job queued. |
|
Comparison is ready Comparison Summary:
|
|
@cmsbuild please test |
|
The tests are being triggered in jenkins. |
|
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
|
Comparison job queued. |
|
+1
|
cmsbuild
added
fully-signed
reconstruction-approved
and removed
reconstruction-pending
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_3_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @smuzaffar (and backports should be raised in the release meeting by the corresponding L2) |
|
urgent a rough guess is that we may start having o(1%) failure rate per job in promptreco without this fix. |
|
backport of #20120 |
|
Comparison is ready Comparison Summary:
|
|
+1 |
adding small patch in the unpacker to prevent corrupted row/col from making the jobs crash (as in PR #20120 )