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
implement multiple working points in ID for e/g triggers based on HGCAL clusters #37758
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37758/29633
|
A new Pull Request was created by @jbsauvan (Jean-Baptiste Sauvan) for master. It involves the following packages:
@rekovic, @epalencia, @cmsbuild, @AdrianoDee, @srimanob, @cecilecaillol can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c1cf59/24369/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+l1 |
+upgrade
|
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) |
throw cms::Exception("HGCalTriggerClusterIdentificationBDT|BadInitialization") | ||
<< "Inconsistent numbers of categories, BDT weight files and working points"; | ||
} | ||
categories_.reserve(working_points_.size()); | ||
bdts_.reserve(working_points_.size()); | ||
|
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.
To improve readability, what about definining here an, e..g.
size_t categories_size = categories_etamin.size();
and use if in the following instead of categories_etamin.size()
repeatedly?
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37758/29733
|
Pull request #37758 was updated. @rekovic, @epalencia, @cmsbuild, @AdrianoDee, @srimanob, @cecilecaillol can you please check and sign again. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c1cf59/24471/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+upgrade |
+l1 |
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:
Add the possibility to define several working points in the identification BDTs for the L1 e/g triggers based on HGCAL clusters.
FYI @cerminar
PR validation:
Private tests in
L1Trigger/L1THGCalUtilities/test
:Tested D49, D60, D68, D77, D86 workflows