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
bugfix XML for PPS strips mapping #37498
Conversation
@vavati FYI |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37498/29200
|
A new Pull Request was created by @grzanka (Leszek Grzanka) for master. It involves the following packages:
@cmsbuild, @malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
type bug-fix |
Hi @grzanka does 136.793 really test this change? Based on what you wrote it doesnt really, otherwise it would have been spotted. Seems to me nothing tests this code, no? |
I ran 136.793 to check if it doesn't break anything else. |
@cmsbuild , please test |
urgent
|
OK, it would be nice to have a unit test for this |
For the time being we have only Minidaq runs...like this |
+alca
|
This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. 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 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3de855/23740/summary.html Comparison SummarySummary:
|
There are indeed quite large differences in the totemRPRawToDigi_TrackingStrip distributions (0 entries vs large number of entries) in wf 138.4 and 139.001 (2021 MinBias). Could anybody have a look and explain the differences? |
The bug in xml caused the daq mapping not be uploaded and then all corresponding digi were not created. |
+1 |
@vavati before merging the backport PR for 12_3, could you please verify in the root outputs of these tests whether those Totem distributions are really empty, as you are assuming? (Maybe they are just filled with zero's, as the entries in the root outputs are there) |
PR description:
PR fixes a wrong comment tag in the XML mapping for the PPS silicon strips
Only now we could run the DAQ on the PPS detectors with silicon strips, so we couldn't spot it before,
PR validation:
relvals 136.793
if this PR is a backport please specify the original PR and why you need to backport that PR:
no