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
Phase 1 HBHE fixes related to SiPM nonlinearity and cosmic run mode #16197
Conversation
A new Pull Request was created by @igv4321 (Igor Volobouev) for CMSSW_8_1_X. It involves the following packages: RecoLocalCalo/HcalRecAlgos @cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
|
@slava77 @cvuosalo @davidlange6 please review/approve, other PRs waiting on this one |
@cmsbuild please test |
The tests are being triggered in jenkins. |
... I guess, incrementally, it's better for this PR to be in first, and then after #16070 is in, its nontrivial features can be picked up here |
-1 Tested at: f49af13 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: DAS Error140.53 step1 DAS Error1000.0 step1 DAS Error1003.0 step1 DAS Error |
@cmsbuild please test @smuzaffar please check what was happening with ... I'll run the short matrix locally and signoff based on that |
The tests are being triggered in jenkins. |
Comparison job queued. |
+1 for #16197 f49af13
the energy of the added hits is seen from the reducedhit energy distribution |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar |
@davidlange6 ping |
+1 |
Added the capability to disable "pulse containment correction". Will do this for Method 0 in the cosmic run mode.
Finer control over RecHit flags -- enable some legacy flags for QIE8 (but not HBHETriangleNoise).
Ready for the new SiPM nonlinearity correction (will need another minimal change after PR #16070 is integrated).
Bug fix: always retrieve HcalRecoParams from the database because it is needed unconditionally for determining "pulseShapeID".
This PR is not supposed to change any results apart from the HBHERecHit noise flags (for MC, may be not even that).