-
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
Update HCAL aging parameters and thresholds #23638
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23638/5285 Code check has found code style and quality issues which could be resolved by applying a patch in https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23638/5285/git-diff.patch You can run |
The code-checks are being triggered in jenkins. |
Comparison is ready Comparison Summary:
|
+1 |
+1 |
+1 |
# slopes taken from https://twiki.cern.ch/twiki/pub/CMSPublic/HcalDPGResultsCMSDPS2017042/dark_current_vs_lumi.png | ||
# crosstalk divided out | ||
# heUpgrade intlumiToNeutrons factor (below) used to convert from /fb-1 to /neutrons | ||
depVsNeutrons = cms.vdouble(5.543e-10,8.012e-10), |
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.
This is a factor 10 larger than the previous values: please confirm that it is correct
(How can I get these numbers from what in https://twiki.cern.ch/twiki/pub/CMSPublic/HcalDPGResultsCMSDPS2017042/dark_current_vs_lumi.png)
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.
Indeed, the prediction has increased by a factor of 10. (This necessitated the re-evaluation of PF thresholds. The previous prediction came from a much older measurement.)
You can get the numbers from the plot by taking the slopes, multiplying by (1-lambda)
(lambda
is given as crosstalk
in the SiPMCharacteristics
section further down in this file; 0.17 or 0.196 for the different SiPM types), and dividing by the HE dose value intlumiToNeutrons = cms.double(2.92e7)
(which is in fb-1/neutrons). This is indicated in the comment.
+1 |
+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 |
The new thresholds were proposed, tested, and validated by @hatakeyamak (thanks!): https://baylor.app.box.com/s/w32ja75krcbxcycyifexu28dwlgrj7wg
We observed that PF gammas get a lot of extra energy introduced by ECAL aging; this will be followed up with the appropriate people.