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
L1 Phase2 for HLT TDR - Multy Primary Vtx, TkMuons Fix for low-pt #32571
L1 Phase2 for HLT TDR - Multy Primary Vtx, TkMuons Fix for low-pt #32571
Conversation
please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32571/20552
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
interesting ... and
don't give any changes. |
c2d230b
to
1eafbf7
Compare
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32571/20560
|
A new Pull Request was created by @rekovic for master. It involves the following packages: DataFormats/L1TCorrelator @rekovic, @kpedro88, @jmduarte can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
-1 Failed Tests: RelVals AddOn RelVals
AddOn Tests
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32571/20806
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c1be85/12379/summary.html Comparison SummarySummary:
|
+1 |
@rekovic can I suggest renaming the PR to "L1T Phase 2 update for HLT TDR: multiple primary vtx, fix for low pT tk muons" ? |
@@ -468,11 +490,26 @@ void L1TkMuonProducer::runOnMTFCollection_v1(const edm::Handle<RegionalMuonCandB | |||
|
|||
TkMuon l1tkmu(l1tkp4, l1muRef, l1tkPtr, trkisol); | |||
|
|||
if (useRegionEtaMatching_) { |
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.
this exact pattern appears twice in the PR: it could be made into a function like
bool inRegionEta(const TkMuon& l1tkmu, int detector) {
return ((detector == barrel_MTF_region and std::abs(l1tkmu.eta()) <= etaBO_) or
(detector == overlap_MTF_region and (std::abs(l1tkmu.eta()) >= etaBO_ and std::abs(l1tkmu.eta()) <= etaOE_)) or
(detector == endcap_MTF_region and std::abs(l1tkmu.eta()) >= etaOE_));
}
then here and below would just be:
if (useRegionEtaMatching_ and !inRegionEta(l1tkmu, detector))
continue;
@@ -635,6 +677,12 @@ std::vector<L1TkMuMantraDF::muon_df> L1TkMuonProducer::product_to_muvec(const EM | |||
std::vector<L1TkMuMantraDF::muon_df> result(l1mus.size()); | |||
for (uint imu = 0; imu < l1mus.size(); ++imu) { | |||
auto& mu = l1mus[imu]; | |||
|
|||
// dropping the emtf tracks with certain quality... | |||
int emtfQual = (mu.Mode() == 11 || mu.Mode() == 13 || mu.Mode() == 14 || mu.Mode() == 15); |
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.
do these quality values have particular meanings? (useful to explain such meanings in a comment, or use named constants if appropriate)
float trkisol = -999; | ||
TkMuon l1tkmu(l1tkp4, l1emtfTrk, l1tkPtr, trkisol); | ||
l1tkmu.setTrackCurvature(matchTk.rInv()); | ||
l1tkmu.setTrkzVtx((float)tkv3.z()); |
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.
is the extra float cast necessary? (c-style casts should be avoided in general, code rule 4.15 http://cms-sw.github.io/cms_coding_rules.html)
+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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
LT Phase2 for HLT TDR, featuring:
PR validation:
Done with the integration branch phase2-l1t-integration-CMSSW_11_1_3.
if this PR is a backport please specify the original PR and why you need to backport that PR:
Before submitting your pull requests, make sure you followed this checklist: