-
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
Refactoring PI plugin for PPSTiming -- added input parameters #43543
Refactoring PI plugin for PPSTiming -- added input parameters #43543
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43543/38122
|
A new Pull Request was created by @Kszemka (Aleksandra Krzemińska) for master. It involves the following packages:
@consuegs, @saumyaphor4252, @cmsbuild, @perrotta, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@perrotta would it be possible to run a battery of tests via cmsbot here ? |
please test |
-1 Failed Tests: UnitTests Unit TestsI found 1 errors in the following unit tests: ---> test testPPSTimingCalibration had ERRORS Comparison SummarySummary:
|
@Kszemka the unit test failure is due to the modifications implemented in this PR: please check and fix |
… with input params
90b4bae
to
9329689
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43543/38342
|
Pull request #43543 was updated. @perrotta, @cmsbuild, @consuegs, @saumyaphor4252, @francescobrivio can you please check and sign again. |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c5c586/36729/summary.html Comparison SummarySummary:
|
+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, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
PR contains changes that allows user to provide input parameters when using Payload Inspector - PPSTimingCalibration module on the cms-conddb website.
PR validation:
Testing stays the same at it was in the previous version for PPSTimingCalibration - Payload Inspector, test is defined in testPPSTimingCalibration.sh.