-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Adding a DQM module for dEdxHit info #9129
Conversation
A new Pull Request was created by @quertenmont (Loic Quertenmont) for CMSSW_7_5_X. Adding a DQM module for dEdxHit info It involves the following packages: Calibration/HcalAlCaRecoProducers @diguida, @cvuosalo, @cerminar, @cmsbuild, @nclopezo, @deguio, @slava77, @mmusich, @danduggan can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
By the way, I don't understand why Calibration/* and DQMOffline/* modules are also listed in the PR, |
Loic, why do you have the commits from Sunanda here? |
Yes, as I said I don't understand whyt Sunandra stuff is in this PR. Is there anyway to tell github to forget about the Sunandra commits ? Please help. The FAQ are really hard to understand regarding this simple (and very regular) problem. |
Something along e.g.
should do the job. |
d8dd78a
to
7d19f32
Compare
The "random differences" are all in the efficiency-like plots of HLT IsoTkMu triggers in run2 MC. They are reproducible. ZMM (wf 1330) These are very likely an effect of recompilation of DQM modules and some underlying issue with probably uninitialized variable. I'll keep this note as a somewhat out of place reference for the issue. |
+1
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @smuzaffar |
+1 |
Adding a DQM module for dEdxHit info
This PR add a DQM module to monitor dEdxHit info.
And also fix a small bug in the dEdxHitInfo producer in standard RECO.
Hope this can be included in the release that will be used for both MC and Data reconstruction
Backport of this changes for 74X are also there