-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Revive of reverted #37852 with a fix on the issue #38044 #38100
Revive of reverted #37852 with a fix on the issue #38044 #38100
Conversation
A new Pull Request was created by @quark2 for CMSSW_12_3_X. It involves the following packages:
@civanch, @emanueleusai, @mdhildreth, @cmsbuild, @jfernan2, @ahmad3213, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please @quark2 submit the fix to the master first. |
Hi @perrotta, Okay, I had no idea about the correct order to make PRs. I'll make the PR soon. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-369f5b/25167/summary.html Comparison SummarySummary:
|
+1 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. 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) |
Pull request #38100 was updated. @civanch, @emanueleusai, @mdhildreth, @cmsbuild, @jfernan2, @ahmad3213, @pmandrik, @micsucmed, @rvenditti can you please check and sign again. |
Hi @qliphy, Yep, sure. And also I added the idea of Andrea, which wasn't equipped yet. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-369f5b/25345/summary.html Comparison SummarySummary:
|
+1 |
testing at P5 |
@emanueleusai Any news on the P5 test? Thanks. |
tests completed successfully. Processing time is similar to what we observed before. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. 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 |
PR description:
This PR revives PR #37852 which is reverted because of issue #38044. The issue has been fixed, replacing part of calculations with a massive loop by lighter codes.
Also, this PR contains a backport of #37995 which fixes issue #37906.
Since the issue happened on CMSSW_12_3_X, a PR to this branch is provided first.
PR validation:
Test are done and one can check again by
runTheMatrix
workflows@jshlee @watson-ij