Skip to content

Update to the PPS layouts #1350

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

AndreaBellora
Copy link

In this PR we made a minor cosmetic update to the presentation of TProfiles, and removed some confusing and duplicated histograms from the shifter layout

@nothingface0
Copy link
Contributor

Thank you @AndreaBellora , let me know when it's ready for testing!

@AndreaBellora AndreaBellora marked this pull request as ready for review April 9, 2025 14:29
@AndreaBellora
Copy link
Author

Thank you @AndreaBellora , let me know when it's ready for testing!

I just confirmed with my colleagues that we are good to go 👍

@nothingface0
Copy link
Contributor

nothingface0 commented Apr 10, 2025

@AndreaBellora we have updated the Online playback DQMGUI (here), where you can check the Online Shift layout changes.

We've also updated the offline-test DQMGUI to preview the T0 shift layouts. Unfortunately I don't think there's any data uploaded to this instance with CTTPPS histograms filled. Feel free to upload root files there (https://cmsweb.cern.ch/dqm/offline-test, using visDQMUpload) to test the histograms, if needed.

@AndreaBellora
Copy link
Author

Thank you very much @nothingface0, I checked the online version and everything looks fine to me. Since our offline is identical to the online, there's nothing to worry about that. I also checked it offline with a deployment on OpenStack and it seemed right.
Everything's good on our side 👍

@nothingface0
Copy link
Contributor

nothingface0 commented Apr 10, 2025

Thank you! We will schedule an update of the online and offline DQMGUIs in the following days.

From your side, is it urgent?

@AndreaBellora
Copy link
Author

Not very urgent. We'll need this for the first time at the alignment run, which is scheduled for May 2nd at the moment. Anything before that works for us

@nothingface0
Copy link
Contributor

nothingface0 commented Apr 10, 2025

In the end, we decided to schedule it for tonight, since there's no online DQM shifter.

Thanks Andrea!

@AndreaBellora
Copy link
Author

Thanks to you!

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.

2 participants