-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
HI Plots for the CaloLayer2 DQM #24835
Conversation
@vukasinmilosevic, CMSSW_10_3_X branch is closed for direct updates. cms-bot is going to move this PR to master branch. |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24835/6784 |
A new Pull Request was created by @vukasinmilosevic (Vukasin Milosevic) for master. It involves the following packages: DQM/L1TMonitor @kmaeshima, @cmsbuild, @andrius-k, @jfernan2, @schneiml can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Hi @vukasinmilosevic |
Hi @thomreis, |
I see. OK in that case this should be fine. Thanks. |
-1 Tested at: d586ad8 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: AddOn
I found errors in the following addon tests: cmsDriver.py RelVal -s L1REPACK:Full --data --scenario=pp -n 10 --conditions auto:run2_hlt_Fake2 --relval 9000,50 --datatier "RAW" --eventcontent RAW --customise=HLTrigger/Configuration/CustomConfigs.L1T --era Run2_2016 --fileout file:RelVal_Raw_Fake2_DATA.root --filein /store/data/Run2016B/JetHT/RAW/v1/000/272/762/00000/C666CDE2-E013-E611-B15A-02163E011DBE.root : FAILED - time: date Tue Oct 9 16:39:17 2018-date Tue Oct 9 16:34:23 2018 s - exit: 23552 |
Comparison job queued. |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@@ -293,9 +296,17 @@ void L1TStage2CaloLayer2::analyze(const edm::Event & e, const edm::EventSetup & | |||
stage2CaloLayer2ETTEMRank_->Fill(itEtSum->hwPt()); | |||
} else if (l1t::EtSum::EtSumType::kTowerCount == itEtSum->getType()) { | |||
stage2CaloLayer2TowCount_->Fill(itEtSum->hwPt()); | |||
} else{ | |||
} else if (l1t::EtSum::EtSumType::kTotalHt == itEtSum->getType()){ |
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.
@vukasinmilosevic the differences observed in the DQM plots for non HI workflows come from here I guess, am I correct? Is this desired?
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.
Hi, that is the only difference in filling the plots (besides the additional plots). I was told to use that EtSumType (kTotalHT) connected to HTTRank
@jfernan2 @andrius-k is this PR ok for you? |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Hi @vukasinmilosevic |
Centrality and Asymmetry plots as well as the event-by-event comparison requested in:
https://its.cern.ch/jira/browse/CMSLITDPG-693