-
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
[SiStrip-DQM] Reduce the number of bins in the Eloss histograms #22856
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22856/4243 |
A new Pull Request was created by @ptrstn (Peter Stein) for master. It involves the following packages: Validation/TrackerHits @vazzolini, @kmaeshima, @dmitrijus, @cmsbuild, @jfernan2, @vanbesien can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hello @dmitrijus , @jfernan2 , I confirm that @ptrstn is working for me (Tracker DPG) - thanks |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
Hi @ptrstn, and welcome to the DQM world! The right place to make yourself a proper DQM developer is the contacts page [0] where you should be able to add yourself to the right e-groups. Also, feel free to pass by the core-DQM meeting on tuesdays [1]. We'd like subsystem developers to come there to know what you are working on, before we get confronted with the PRs. There is not much to talk about for this PR, but since Tracker DQM tends to have quite a lot of requests, we want you there rather sooner than later. [0] https://twiki.cern.ch/twiki/bin/viewauth/CMS/DQMContacts#Tracker_SiStrips_and_Pixels |
Thanks @schneiml , maybe you could then ask your colleagues to review and sign this PR ? |
@boudoul the usual procedure for small PRs (like this) is that they will be processed once per week, around the ORP. For things that might require discussion, please e-mail cms-dqm-coreTeam@cern.ch (github notifications won't work reliably) and probably come to the core-DQM meeting (this is sth. we want to require this year). |
+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 welcome optimization |
Hello,
@dmitrijus , @jfernan2 , I am a new SiStrip-DQM Developer (@boudoul FYI)
I reduced the number of bin in the SiStrip Eloss histograms, since the number was previously way too high.