Skip to content

Add pedestal RMS and BC energy in ECAL DQM overview layout#1363

Merged
uhomsuwa merged 1 commit intodmwm:masterfrom
kyungminparkdrums:addSummaryPlots_ecalDQM
May 6, 2026
Merged

Add pedestal RMS and BC energy in ECAL DQM overview layout#1363
uhomsuwa merged 1 commit intodmwm:masterfrom
kyungminparkdrums:addSummaryPlots_ecalDQM

Conversation

@kyungminparkdrums
Copy link
Copy Markdown

This PR adds two plots (pedestal RMS and basic cluster energy plots) in ECAL DQM layout under 00 Overview. PR is validated by running the ECAL online DQM configuration and testing with private GUI.

@gabrielmscampos
Copy link
Copy Markdown

gabrielmscampos commented Apr 30, 2026

Hello,

This PR + #50826 is deployed in playback, could you check if it is what you expect?

Some extra questions:

  • Do you need both PRs patched before the next CMSSW release and dmwm release or just one of them?
  • I can see you also updated the T0_layouts, should the patch be applied in the Offline GUI too?

@kyungminparkdrums
Copy link
Copy Markdown
Author

Hi @gabrielmscampos,

Thank you for the playback link. I can confirm that it is what I would expect. For the answers to your questions:

  1. Both PRs need to be patched before the next release.
  2. Yes, the patch should ideally be applied in the Offline GUI as well.

Thank you!

@gabrielmscampos
Copy link
Copy Markdown

Hi @kyungminparkdrums,

The online, offline and dev instances of the DQMGUI were upgraded, likewise we patched our current CMSSW production release to include #50826.

@gabrielmscampos
Copy link
Copy Markdown

Hi @uhomsuwa,

Could you merge this PR? It is tested and validated, no need to release a new tag with it for the moment. Thanks!

Best,
Gabriel Moreira

@uhomsuwa uhomsuwa merged commit 6757a09 into dmwm:master May 6, 2026
@uhomsuwa
Copy link
Copy Markdown
Collaborator

uhomsuwa commented May 6, 2026

Hi @gabrielmscampos,
Done.

Best Regards,
Usakorn H.


JIRA ticket for the May release: CMSKUBERNETES-342

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants