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
Temporarily reject 16-partition GE2/1 clusters in the EMTF #31409
Temporarily reject 16-partition GE2/1 clusters in the EMTF #31409
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-31409/18270
|
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
I did not want to interfere with EMTF development, so I suggested to EMTF folks what needed to be done on their end. Then Jia Fu proposed that I make the change myself, so I made a new pull request. At this point #31268 is almost signed off; I just need Vladimir's signature. |
+1 |
+1 |
+upgrade |
@silviodonato Could this PR be merged quickly? |
merge |
PR description:
Temporarily reject 16-partition GE2/1 clusters, because the EMTF is not yet adapted to handle these objects.
Follow-up of #31268 (to be merged first). The relevant commit for this PR is bef7957.
PR validation:
Code compiles.
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A
FYI @jiafulow @eyigitba @abrinke1