Skip to content
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

New plots on GEM online DQM #1054

Merged
merged 6 commits into from May 31, 2021
Merged

New plots on GEM online DQM #1054

merged 6 commits into from May 31, 2021

Conversation

quark2
Copy link
Contributor

@quark2 quark2 commented May 25, 2021

PR description:

This PR updates and extends the part of onlineDQM for GEM. The details are following:

  • Additional plots displaying summarized VFAT status
  • recHit wheel occupancy

This PR is related to PRs on cms-sw; see
cms-sw/cmssw#33767 (to master)
cms-sw/cmssw#33768 (backport)

For more detail, please see this to check more details

@jshlee @watson-ij @seungjin-yang

@muhammadimranfarooqi
Copy link
Contributor

@quark2

Is it ready to deploy in testbed?

@quark2
Copy link
Contributor Author

quark2 commented May 25, 2021

Hi @muhammadimranfarooqi,

yep, I'm ready for the testbed.
(FYI, you can check the result in http://cms.sscc.uos.ac.kr:50100/dqm/online-dev/, run 341169.)

@seungjin-yang
Copy link
Contributor

@muhammadimranfarooqi hi, can I add more commits to this PR for updating GEM offline DQM?

@muhammadimranfarooqi
Copy link
Contributor

@seungjin-yang

Yes, as long as this is not merged, you can add more commits.

@seungjin-yang
Copy link
Contributor

@muhammadimranfarooqi Hi, this PR is ready for deployment in both online and offline testbeds.

@muhammadimranfarooqi muhammadimranfarooqi merged commit 526d557 into dmwm:master May 31, 2021
@muhammadimranfarooqi
Copy link
Contributor

@quark2

This PR has been deployed in cmsweb-testbed cluster. Please test and confirm its readiness for production cluster.

@seungjin-yang
Copy link
Contributor

@muhammadimranfarooqi I confirmed that the updated offline DQM GUI works well.

@quark2
Copy link
Contributor Author

quark2 commented May 31, 2021

Hi @muhammadimranfarooqi,

All other changes on this PR seem deployed into cmsweb-testbed, except the change in the commit c41daea which is for the online DQM GUI. Could you check it?

Best regards,
Byeonghak Ko

@muhammadimranfarooqi
Copy link
Contributor

@quark2
Copy link
Contributor Author

quark2 commented May 31, 2021

@muhammadimranfarooqi

Yep, the commit is merged well into the master branch of this repository. But the latest codes seem not deployed into the onlineDQM in the cmsweb-testbed.

@muhammadimranfarooqi
Copy link
Contributor

@quark2

I don't deploy in onlineDQM. I deploy only in vocms0731 which is testbed node and vocms0738, vocms0739 which are production nodes.

You may ask relevant person who is responsible for onlineDQM as this VM is at P5 network.

@quark2
Copy link
Contributor Author

quark2 commented May 31, 2021

@muhammadimranfarooqi

I see. Sorry for bothering you... I'll notice when the test is done.

@muhammadimranfarooqi
Copy link
Contributor

Thanks @quark2 for informing me.

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.

None yet

3 participants