-
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
HCAL: move data-specific SeverityLevel customization(s) to Reconstruction_Data_cff.py #34282
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34282/23590
|
A new Pull Request was created by @abdoulline (Salavat Abdullin) for master. It involves the following packages: Configuration/DataProcessing @cmsbuild, @silviodonato, @qliphy, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
cc @cms-sw/hcal-dpg-l2 |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f9daf7/16332/summary.html Comparison SummarySummary:
|
assign reconstruction |
+reconstruction
|
+operations |
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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Suggested by @slava77 in (issue) #27767
re-arrangement of HCAL data-specific SeverityLevel settings (of RecHits flags) :
move the relevant stuff from RecoTLR_cff.py customization snippets
to Reconstruction_Data_cff.py appropriate era/modifiers.
NB: data settings are now in sync with the MC ones for >=Run3
(First they were synchronized yet in RecoTLR_cff.py in #33520 by @hatakeyamak )
PR validation:
runTheMatrix.py -l limited --ibeos --useInput all
Checked explicitly that SeverityLevel groups stayed unchanged w/wo this PR in
(1) 136.731_RunSinglePh2016B
(2) 136.793_RunDoubleEG2017C
(3) 136.874_RunEGamma2018C