Skip to content

Commit

Permalink
Merge pull request #45030 from mbluj/CMSSW_14_1_X_omtfTrkAddrFix
Browse files Browse the repository at this point in the history
Set 3 instead of 4 OMTF track sub-address as expected by uGMT
  • Loading branch information
cmsbuild committed May 28, 2024
2 parents f1b0254 + b215850 commit 74eac92
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion L1Trigger/L1TMuonOverlapPhase1/src/Omtf/OMTFProcessor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,8 @@ std::vector<l1t::RegionalMuonCand> OMTFProcessor<GoldenPatternType>::getFinalcan
//check if it matters if it needs to be here as well
trackAddr[1] = myCand->getRefLayer();
trackAddr[2] = myCand->getDisc();
trackAddr[3] = myCand->getGpResultUnconstr().getPdfSumUnconstr();
//TODO: uGMT expects only 3 sub-addresses, so not set 4th. This is anyway currently not used.
//trackAddr[3] = myCand->getGpResultUnconstr().getPdfSumUnconstr();
if (candidate.hwPt() > 0 || candidate.hwPtUnconstrained() > 0) {
candidate.setTrackAddress(trackAddr);
candidate.setTFIdentifiers(iProcessor, mtfType);
Expand Down

0 comments on commit 74eac92

Please sign in to comment.