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
Concurrently construct Source while making Modules #38603
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38603/30878
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages:
@cmsbuild, @smuzaffar, @Dr15Jones, @makortel can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: HeaderConsistency ClangBuild Clang BuildI found compilation warning while trying to compile with clang. Command used:
See details on the summary page. |
931b8a4
to
f014987
Compare
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38603/30879
|
Pull request #38603 was updated. @makortel, @smuzaffar, @Dr15Jones can you please check and sign again. |
-1 Failed Tests: HeaderConsistency Comparison SummarySummary:
|
@cmsbuild, please test The difference persisted, so let's try again in a new IB |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14cf39/26495/summary.html Comparison SummarySummary:
|
The issue persists with 4.53. I have not seen that kind of comparison differences in other recent PRs, which makes me suspect the changes in this PR would somehow be causing that. |
The differences in histograms has been found and fixed in #38900. It was a bug in the algorithm used to decide what data to read which had a hysteresis problem. |
@cmsbuild, please test Refresh to see if comparison differences are now gone |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14cf39/26623/summary.html Comparison SummarySummary:
|
+1 Comparisons are clean now |
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, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Use two different TBB tasks, one to create the Source and a second to make all the ED modules.
PR validation:
Did
runTheMatrix.py -l limited -t 4
and the jobs succeeded.