-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Added RooAbsPdf.h and RooAbsArg.h in PhysicsTools/TagAndProbe/src/TagProbeFitter.cc #49074
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
Added RooAbsPdf.h and RooAbsArg.h in PhysicsTools/TagAndProbe/src/TagProbeFitter.cc #49074
Conversation
+#include RooAbsArg.h
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49074/46310 |
|
A new Pull Request was created by @akritkbehera for master. It involves the following packages:
@cmsbuild, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
please test for CMSSW_16_0_ROOT6_X |
|
@akritkbehera do you have the PR that added this in the master branch? We should denote what this is the backport of |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-590ada/48502/summary.html Comparison SummarySummary:
|
|
@tvami this is not a backport. This change is required to update root in the ROOT6_X IBs: cms-sw/cmsdist#10110 (comment) |
|
thanks @iarspider but I'm still confused ROOT6_X branch is meant for 10_6_X release cycle? |
|
@tvami CMSSW_16_0_ROOT6_X uses master branch, if that's what you are asking. And this PR is for master branch. |
|
+1
|
|
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. @sextonkennedy, @ftenchini, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
No description provided.