-
Notifications
You must be signed in to change notification settings - Fork 207
[RNTUPLE] Updates root to tip of v6-36-00-patches #10196
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
Conversation
|
please test |
|
A new Pull Request was created by @smuzaffar for branch IB/CMSSW_16_0_X/rntuple. @akritkbehera, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
-1 Failed Tests: UnitTests RelVals Failed Unit TestsI found 9 errors in the following unit tests: ---> test test-btvNano-run had ERRORS ---> test TestIOPoolInputNoParentDictionary had ERRORS ---> test test-btvNano-check had ERRORS and more ... Failed RelValsExpand to see more relval errors ...
|
|
please test |
|
-1 Failed Tests: RelVals Failed RelValsExpand to see more relval errors ...
|
|
please test with cms-sw/cmssw#49387 for CMSSW_16_0_RNTUPLE_X |
|
-1 Failed Tests: RelVals 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: Failed RelValsExpand to see more relval errors ...
|
|
please test with cms-sw/cmssw#49387 for CMSSW_16_0_RNTUPLE_X using full cmssw |
|
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: Failed Unit TestsI found 1 errors in the following unit tests: ---> test TestRNTupleTempEventHistory had ERRORS Comparison SummarySummary:
|
The failure is @Dr15Jones any thoughts? |
|
please test |
|
-1 Failed Tests: UnitTests Failed Unit TestsI found 1 errors in the following unit tests: ---> test TestRNTupleTempEventHistory had ERRORS Comparison SummarySummary:
|
|
@makortel , any objections on getting this in IBs and then debugging the unit test issue in the IB env? |
|
IBs are probably better way to debug, so no objections |
|
+externals integrating it for next RNTUPLE IBs |
|
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_16_0_X/rntuple IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
No description provided.