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
Disabled duplicate GEM TP warnings in EMTF GEM unpacker #35042
Disabled duplicate GEM TP warnings in EMTF GEM unpacker #35042
Conversation
…a format is fixed
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35042/24905
|
A new Pull Request was created by @eyigitba for master. It involves the following packages:
@cmsbuild, @rekovic, @cecilecaillol can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b3c91d/18084/summary.html Comparison SummarySummary:
|
@eyigitba can you please also mention in the PR description that a backport to 11_3_X is expected? |
Yes, please. Although usually it's better for the master PR to merge first. However the L1 signature happens very slowly in this summer period, so maybe we can ask @perrotta @qliphy to comment on the PR / merge it without the L1 signiture?
Depends, if the GEM TP data format is expected to be fixed by CRAFT then no. If it's not expected to be fixed, then yes submitting the backport there is one possibility. Of course we also have some time until CRAFT, so a solution with the MessageLogger printing this warning just few a times and collect the total at the end of the job could be the way to go as well |
OK, then I'll wait for now to hear from the GEM team and in the meanwhile start implementing the other solution we discussed. This should be resolved until CRAFT, so there is no need for a backport to 12_0_X. |
+1 |
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, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
urgent |
+1 |
@tvami I just submitted the backport. I also started working on the more permanent solution as we discussed. I'll keep you updated. |
Commented out the duplicate GEM TP warning in EMTF GEM block unpacker until the GEM-EMTF data format is fixed.
This was discussed in #34309
This needs to be backported to 11_3_X once this PR is merged.