-
Notifications
You must be signed in to change notification settings - Fork 179
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
Online DQM update of Pixel folder #1333
Conversation
'PixelPhase1/Layouts/04 - PixelPhase1_Cluster_Number', | ||
'PixelPhase1/Layouts/05 - PixelPhase1_Cluster_Charge', | ||
'PixelPhase1/Layouts/06 - PixelPhase1_Cluster_Size', | ||
#'PixelPhase1/Layouts/00b - PixelPhase1_Error_Summary', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Payal7890 Did you mean to remove those layouts from the PixelPhase1 workspace completely? It doesn't seem they were removed in your test: https://tinyurl.com/yc4njw2h
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I can see, you will also need to modify the pixelPhase1-layouts.py
as well to completely remove them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Payal7890 kind reminder for this.
@Payal7890 this will require some more modifications from your side, as another, already merged PR, has created conflicts. |
Hello,
I will fix it as soon as possible and get back to you. Thanks for your
understanding.
Best
Payal
…On Tue, Jul 16, 2024 at 9:48 AM Dimitris Papagiannis < ***@***.***> wrote:
@Payal7890 <https://github.com/Payal7890> this will require some more
modifications from your side, as another, already merged PR, has created
conflicts.
—
Reply to this email directly, view it on GitHub
<#1333 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVS26YCO2TXBJ3NBPHLLWJ3ZMUXD5AVCNFSM6AAAAABKDHJWPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZRGEYTCNZWGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hello @nothingface0, we only intend to delete those plots you mentioned above from the "Quick Collection" folder. We wanted to keep 000,00a,19, 20, 24, 27, 28, 30,36, 37,43a,b as you can see here: https://tinyurl.com/4zvb5mx9. We don't want to delete anything from the PixelPhase1 workspace completely. Please let me know if that makes sense. |
@Payal7890 Sure! It's up to you to decide what you want to show. In the meantime, we have deployed this PR on our playback machines, here: Unfortunately this machine has some network issues lately, so it might take some time to load. Please take a look and let us know if it looks OK. |
Hi @nothingface0 , can you update the playback machine? We just updated the code (to reintroduce some plots). Thanks |
@alaperto It's updated, feel free to take a look |
Ok, looks good! |
Hi @nothingface0 , when this will be deployed? |
Reminder for @nothingface0 |
@alaperto Sorry for the delay, I was on vacation. We will update the online DQMGUI this week, along with a CMSSW release which has been planned. Is this also needed for Offline? |
@alaperto This is now deployed on the Online DQMGUI |
We have removed few plots of Online DQM GUI from PixelPhase1 workspace (Quick Collection) and Shift workspaces (PixelPhase1 folder) in agreement with Pixel Operation team, in order to simplify the work of the Online DQM shifter.
We tested the changes in the private GUI prova1234, using runs 380066 and 380074.
To connect to the private GUI outside of CERN network: tunneling (and proxy in browser, with same port number) are needed:
ssh -D CERNUSERNAME@lxplus.cern.ch
We hereby attach the tinyURL links before (Official, GUI) and after the changes (New, prova1234 GUI) for the run 380066.
PixelPhase 1 (Quick Collection):
Original: https://tinyurl.com/2jh6wy8e
New: : https://tinyurl.com/4zvb5mx9
Shift (PixelPhase1):
Original: https://tinyurl.com/dydxe649
New: https://tinyurl.com/3mcdvuu2
For both Online and Offline Pixel Summary maps, the bin size of the z-axis was changed in order to have the green color in the range from 0.9 to 1.0.