-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
DQM: reportSummaryMap: change the sqrt(s) energy from 13 to 13.6 TeV #44757
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44757/39977
|
A new Pull Request was created by @syuvivida for master. It involves the following packages:
@antoniovagnerini, @rvenditti, @syuvivida, @nothingface0, @cmsbuild, @tjavaid can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e6952f/38937/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. @antoniovilela, @sextonkennedy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR changes the display of sqrt(s) energy in the reportSummaryMap from 13 TeV to 13.6 TeV. Before this PR request, an update of DQMGUI layout was already committed and merged: github link. The PR in DQMGUI layout ensures there is no problem when rendering two different version of root files (one with 13 TeV and the other with 13.6 TeV).
PR validation:
We have tested this PR at the playback machines using DQM streamers of cosmic run 378711, 900 GeV pp run 378239, 13.6 TeV pp run 379315. All displayed as expected in the reportSummaryMap.