-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
TagProbeFitter: replace call to deprecated overload of RooPdf::paramOn #41322
Conversation
It was marked as deprecated for a long time, and was recently completely removed
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41322/35135
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41322/35137
|
please test |
A new Pull Request was created by @iarspider for master. It involves the following packages:
can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@iarspider , I would suggest to test this PR with out cms-sw/cmsdist#8433 as this fix should work for root 6.26 and above. |
please test this should work for root 6.26 too, so restarting the test to use default IBs |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc4afe/31941/summary.html Comparison SummarySummary:
|
@perrotta @rappoccio can we get this in. This is needed for ROOT6 update (it removes the use of deprecated |
+1 |
merge |
PR description:
It was marked as deprecated for a long time, and was completely removed in root-project/root#12608