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
DQM plots for the Phase 2 Track Trigger #17327
Conversation
A new Pull Request was created by @lmoreels for CMSSW_9_0_X. It involves the following packages: Configuration/EventContent The following packages do not have a category, yet: DQM/Phase2OuterTracker @hengne, @franzoni, @dmitrijus, @cmsbuild, @rekovic, @srimanob, @vanbesien, @fabozzi, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
@lmoreels there are merge conflicts, please rebase |
Pull request #17327 was updated. @ghellwig, @lveldere, @ianna, @cerminar, @fabozzi, @Martin-Grunewald, @rekovic, @vanbesien, @govoni, @perrotta, @civanch, @silviodonato, @franzoni, @cmsbuild, @fwyzard, @davidlange6, @smuzaffar, @Dr15Jones, @cvuosalo, @mdhildreth, @slava77, @ggovi, @mmusich, @mulhearn, @ssekmen, @arunhep, @perrozzi, @thuer, @dmitrijus, @alja, @srimanob, @hengne can you please check and sign again. |
merging is not rebasing. |
I'm sorry. I don't know how to rebase |
Also nice step by step recipe from Slava in github : #11426 |
ffe00f7
to
88a4c7c
Compare
5dc1699
to
bc8b154
Compare
Pull request #17327 was updated. @kpedro88, @dmitrijus, @cmsbuild, @rekovic, @franzoni, @vanbesien, @mulhearn, @davidlange6 can you please check and sign again. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
+1 |
@dmitrijus, @rekovic, @franzoni, @vanbesien, @mulhearn please sign |
+1 |
Hi @dmitrijus , could you please re-sign this PR ? thanks |
This PR includes new DQM plots for the phase 2 track trigger (e.g. position & occupancy plots, eta distributions of TTClusters and TTStubs). It is based on the code in L1Trigger/TrackTrigger/test/ and was previously included in the 62X_SLHC release. An overview of its use and purpose can be found in the linked presentation.
As the TTTracks are not yet available, the relevant code is commented.
This is a follow-up of PR #17314
https://indico.cern.ch/event/536863/contributions/2410741/attachments/1390920/2118789/161220_DQM_Phase2OuterTracker.pdf
@idebruyn @ivanpari