Disabled ROCs for Online DQM Pixel Summary #45026
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45026/40319
|
|
A new Pull Request was created by @alaperto for master. It involves the following packages:
@rvenditti, @antoniovagnerini, @cmsbuild, @tjavaid, @nothingface0, @syuvivida can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
@cmsbuild please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-33094a/39492/summary.html Comparison SummarySummary:
|
| @@ -272,6 +272,7 @@ void SiPixelPhase1Summary::fillSummaries(DQMStore::IBooker& iBooker, DQMStore::I | |||
| //Fill the dead ROC summary | |||
| std::vector<trendPlots> trendOrder = {layer1, layer2, layer3, layer4, ring1, ring2}; | |||
| std::vector<int> nRocsPerTrend = {1536, 3584, 5632, 8192, 4224, 6528}; | |||
| std::vector<int> nDisabledRocs = {12, 128, 240, 320, 96, 120}; | |||
There was a problem hiding this comment.
Thank you for the clarification. It's good to know that the list is stable and that the numbers have been consistent since June 2023. Let's hope that the detector remains stable and we don't encounter further losses in 2024 and 2025.
|
+1 |
cmsbuild
added
dqm-approved
fully-signed
and removed
dqm-pending
pending-signatures
labels
|
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 |
Exclude disabled ROCs when calculating the fraction of "good" ROCs for the Online DQM Pixel Summary map.
PR validation:
Data produced with cmsRun on recent 2024 pp run 380400:
cmsRun DQM/Integration/python/clients/pixel_dqm_sourceclient-live_cfg.py dataset=/ZeroBias/Run2024D-v1/RAW runNumber=380400
Results available in private GUI
About backport
This PR is intended for 2024 data taking. There will be a backport to 14_0_X in a separate PR.