-
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
add HLT-EGM customisation for 2018 Data #37606
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37606/29380
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
urgent "urgent" here means that, ideally, this PR would go in This PR poses no risk for IBs or release-building (this function is not used anywhere centrally). This PR simply allows TSG recipes to remain correct as other parts of the source code change. |
please test |
Thanks Marino for taking care of it.
yes we would like to check the rates with |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d6987d/23988/summary.html Comparison SummarySummary:
|
+hlt
|
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR updates the function
customiseFor2018Input
used by TSG to characterise the Run-3 HLT menu on 2018 collisions data.It adds a function named
customiseEGammaRecoFor2018Input
, which currently sets the flagregTrainedWithPS
toFalse
to handle correctly the HLT-EGM energy regression tags of 2018 [*].#37588 changed the value of
regTrainedWithPS
totrue
for Run 3 via the source code. This PR updates the Run-2 customisation such that TSG studies usingcustomiseFor2018Input
are not affected by #37588.If #37588 is backported to
12_3_X
, this PR will have to be backported as well.No changes expected. Note that PR tests are completely blind to the modifications in this PR.
[*] If it turns out that the preferred approach is to instead use the Run-3 tags on 2018 data (keeping
regTrainedWithPS=true
),customiseEGammaRecoFor2018Input
could be updated in the future to overwrite the relevant GT tags instead of changing the value of theregTrainedWithPS
flag.FYI: @cms-sw/egamma-pog-l2
PR validation:
Validated with manual tests based on TSG recipes.
If this PR is a backport, please specify the original PR and why you need to backport that PR:
N/A