-
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
[12_4_X] Add check of Tracker HV in BeamSpot DQM clients #39353
[12_4_X] Add check of Tracker HV in BeamSpot DQM clients #39353
Conversation
backport of #39347 |
A new Pull Request was created by @francescobrivio for CMSSW_12_4_X. It involves the following packages:
@malbouis, @yuanchao, @pmandrik, @micsucmed, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @ChrisMisan, @jfernan2, @clacaputo, @syuvivida, @saumyaphor4252, @francescobrivio, @mandrenguyen, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
type performance-improvement |
urgent
|
@cmsbuild please test |
+1 |
+1 |
+alca |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_12_6_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-eea644/27440/summary.html Comparison SummarySummary:
|
+1 |
PR description:
Backport of #39347
This also includes the backport of #38925 (update the unitTest input file) which is a 1 commit PR that was never backport to 12_4_X.
A check on the Tracker HV (both pixels and strips) is added in order not to run on events which are by construction not useful for the BeamSpot determination.
Plus some minor improvements to the DQM BeamSpot clients.
PR validation:
See master PR
Backport:
Backport of #39347