-
Notifications
You must be signed in to change notification settings - Fork 207
[ROOT636] Root update v6 36 00 patches 20250922 #10087
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
[ROOT636] Root update v6 36 00 patches 20250922 #10087
Conversation
|
A new Pull Request was created by @akritkbehera for branch IB/CMSSW_16_0_X/root636. @akritkbehera, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
Please test |
|
-1 Failed Tests: UnitTests Unit TestsI found 1 errors in the following unit tests: ---> test testFWCoreFWLite had ERRORS Comparison SummarySummary:
|
|
test parameters:
|
|
Please test |
The crash happens on line 231: |
|
-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 1 errors in the following unit tests: ---> test testFWCoreFWLite had ERRORS Comparison SummarySummary:
|
|
assign core |
|
New categories assigned: core @Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
|
Looking at the diff cms-sw/root@cms/v6-36-00-patches/55016315be...cms/v6-36-00-patches/fc959539ca root-project/root#19915 / root-project/root#19873 looks like possibly related. Although then I'd wonder why the ROOT master update #10086 didn't show similar failure. FYI @pcanal |
|
@cmsbuild, please test Let's see if the failure reproduces |
|
-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 1 errors in the following unit tests: ---> test testFWCoreFWLite had ERRORS Comparison SummarySummary:
|
|
@makortel What is the actual error/failure? @vepadulano Do you see a link to the |
|
@pcanal segmentation violation caused by TChain eventChain(edm::poolNames::eventTreeName().c_str());
eventChain.Add("good.root");
eventChain.Add("good_delta5.root");
edm::Wrapper<edmtest::OtherThingCollection>* pOthers = nullptr;
TBranch* othersBranch = nullptr;
eventChain.SetBranchAddress("edmtestOtherThings_OtherThing_testUserTag_TEST.", &pOthers, &othersBranch);
edm::Wrapper<edmtest::ThingCollection>* pThings = nullptr;
TBranch* thingsBranch = nullptr;
eventChain.SetBranchAddress("edmtestThings_Thing__TEST.", &pThings, &thingsBranch);
int nev = eventChain.GetEntries();
for (int ev = 0; ev < nev; ++ev) {
std::cout << "event #" << ev << std::endl;
othersBranch->SetAddress(&pOthers); // othersBranch is nullptr here |
No description provided.