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

Update to DQM onlinebeammonitor #34717

Closed
francescobrivio opened this issue Aug 2, 2021 · 5 comments · Fixed by #34719
Closed

Update to DQM onlinebeammonitor #34717

francescobrivio opened this issue Aug 2, 2021 · 5 comments · Fixed by #34719

Comments

@francescobrivio
Copy link
Contributor

In order to properly monitor the updates of the BeamSpotOnline tags at lumisection level the configuration of the client
onlinebeammonitor_dqm_sourceclient-live_cfg.py needs to be updated to include the refreshTime parameter.
This change only affects the monitoring of the BeamSpotOnline updates on the DQM side.

The fix is rather simple (just adding a toGet to customize the process.GlobalTag is enough) and it has been tested already in the HLT menu live at P5 (see http://cmsonline.cern.ch/cms-elog/1121707).
I will prepare PRs for 11_3, 12_0 and 12_1 by today.

@perrotta @qliphy @silviodonato This is just to let you know that it would be nice to include this in the new build for CRUZET (not sure whether is this already planned or if the issues with HCAL GPU code still do not have a solution).

FYI
@gennai @ggovi
@amassiro @boudoul
@cms-sw/dqm-l2

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2021

A new Issue was created by @francescobrivio .

@Dr15Jones, @perrotta, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@francescobrivio
Copy link
Contributor Author

assign dqm

(as I said I will provide the PRs)

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2021

New categories assigned: dqm

@jfernan2,@kmaeshima,@rvenditti,@andrius-k,@ErnestaP,@ahmad3213 you have been requested to review this Pull request/Issue and eventually sign? Thanks

@jfernan2
Copy link
Contributor

jfernan2 commented Aug 3, 2021

+1
Fixed in the PRs above

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2021

This issue is fully signed and ready to be closed.

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

Successfully merging a pull request may close this issue.

3 participants