-
Notifications
You must be signed in to change notification settings - Fork 183
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
DD4Hep - get latest on master #6826
Conversation
Resolves: #6824
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_12_0_X/master. @cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks. |
please test |
-1 Failed Tests: UnitTests The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Unit TestsI found errors in the following unit tests: ---> test GeometryMTDGeometryBuilderTestDriver had ERRORS Comparison SummarySummary:
|
@fabiocos , we are trying to add updated DD4Hep to the master. In 11_3_X branch all tests are passed but here the unit test GeometryMTDGeometryBuilderTestDriver has errors. Is this connected with the recent PRs for MTD? |
@civanch the update for this unit test was not included in the 1100 IB yesterday, it will work with the 2300 |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-03036e/14379/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
Resolves: #6824
this is the same version as in dd4hep cmsdist branch