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
Hcal TP fix ngHE saturation + switch from bare to effective pedestals #22262
Hcal TP fix ngHE saturation + switch from bare to effective pedestals #22262
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22262/3440 |
A new Pull Request was created by @christopheralanwest for master. It involves the following packages: CalibCalorimetry/HcalTPGAlgos @ghellwig, @cmsbuild, @arunhep, @cerminar, @nsmith-, @rekovic, @franzoni, @thomreis, @ggovi, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
+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 |
Two changes to the HCAL trigger LUTs are implemented in this PR:
To avoid increased quantization errors as a consequence of the increased longitudinal segmentation after the HE upgrade, the energies that enter each trigger tower sum are represented with an additional bit of precision (11 bits) relative to the equivalent quantities in the legacy HE (10 bits). See the bullet "Linearization LUT" on page 23 of the uHTR spec for details. For the 2017 run, in view of the very compressed schedule for preparing Plan 1, it was requested that the HEP17 appear technically as similar to the legacy system as possible.
The change of the linearization LUT LSB to 1/16 GeV needed in order to implement the originally envisioned scheme has already been integrated (d95fcf4#diff-5e541a4bbf886fa4e796702c32e8fe71R10). But the maximum number of bits representing the (transverse) energy was not changed, effectively truncating the linearized transverse energy entering the trigger tower sum sum at 64 GeV rather than 128 GeV. This gives rise to spurious outliers in the RecHit/TP ratio. The plots below compare the trigger tower sum with the equivalent method 0 RECO quantity before and after the changes, using the RelValQCD_FlatPt_15_3000HS_13 samples [1] to emphasize the behavior at high pT. These plots were generated including the recently merged PR #22121, and impose a requirement of pT > 30 GeV, with saturated towers excluded.
The treatment of 2017 era is unchanged, aside from a change in the numbering of the bits used for generating ngHE depth feature bits; this functionality was not used in 2017.
The second change, a switch from bare to effective pedestals in LUT generation to take into account the effect of radiation damage on SiPMs, is negligible except for a modest effect in trigger tower 28. Though this change is logically unrelated to the first, since it modifies some of the same files as the first change, the effects are negligible, and has already been presented to L1 experts at a recent L1 trigger primitives meeting, it has been included here for simplicity.
This PR should only change quantities involving ngHE TP, and all changes should be small.
[1]
/RelValQCD_FlatPt_15_3000HS_13/CMSSW_10_0_0-100X_upgrade2018_realistic_v6_mahiOFF-v1/GEN-SIM-DIGI-RAW
/RelValQCD_FlatPt_15_3000HS_13/CMSSW_10_0_0-100X_upgrade2018_realistic_v6_mahiOFF-v1/GEN-SIM-RECO