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
Revive the green status on the lumi-based plots in GEM onlineDQM, a backport to 12_4_X #39326
Revive the green status on the lumi-based plots in GEM onlineDQM, a backport to 12_4_X #39326
Conversation
A new Pull Request was created by @quark2 for CMSSW_12_4_X. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
backport of #39310 |
please test |
testing at P5, too |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-448896/27417/summary.html Comparison SummarySummary:
|
@quark2 is this intended to be deployed at P5 for data-taking as soon as possible? |
@quark2 we test this PR at playback using run 357900. Attached is the figure. Is one of the lumi-based figures OK? 05-GE21-P-L2? All others look green now. It's also true that before this PR, 05-GE21-P-L2 is completely red. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) 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) |
urgent
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) 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) |
hold
|
Pull request has been put on hold by @francescobrivio |
Hi @syuvivida, If you are asking about the appearance of green stuffs on the lumi based plots, that's exactly what this PR makes. Meanwhile, the red bins should be already there before (and after) this PR. Best regards, |
Hi Byeonghak @quark2 |
Yep, that's right. It's because currently GE21 station has only one chamber in P5. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) 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 |
Hi @emanueleusai, Can I ask when this PR will be deployed to P5? (or the charging person on it if you are not) |
@quark2, |
Ahh, that's okay then. Thanks! |
@quark2, |
@syuvivida And sorry again, it would be good to check the results now. Could you deploy it now? |
@quark2, |
Ahh, thanks a lot! I'll have a look. |
PR description:
The green status on the lumi-block-based status plot was discarded before, but end-users pointed out that the green status is more useful than the expectation. So, it is decided to revive the function.
PR validation:
Test are done with
cmsRun $CMSSW_RELEASE_BASE/src/DQM/Integration/python/clients/gem_dqm_sourceclient-live_cfg.py unitTest=True
@jshlee @watson-ij @seungjin-yang
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
This is a backport of #39310 to CMSSW_12_4_X