Skip to content

Commit

Permalink
Revert "Fixes mismatches between emulator and fw on CDC dimuon seed"
Browse files Browse the repository at this point in the history
This reverts commit c8aefec.
  • Loading branch information
missirol committed Dec 28, 2024
1 parent ab76956 commit 5db9511
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 59 deletions.
4 changes: 1 addition & 3 deletions L1Trigger/L1TGlobal/interface/GlobalBoard.h
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,7 @@ namespace l1t {
void receiveMuonObjectData(const edm::Event&,
const edm::EDGetTokenT<BXVector<l1t::Muon>>&,
const bool receiveMu,
const int nrL1Mu,
const std::vector<l1t::Muon>* muonVec_bxm2,
const std::vector<l1t::Muon>* muonVec_bxm1);
const int nrL1Mu);

void receiveMuonShowerObjectData(const edm::Event&,
const edm::EDGetTokenT<BXVector<l1t::MuonShower>>&,
Expand Down
13 changes: 1 addition & 12 deletions L1Trigger/L1TGlobal/plugins/L1TGlobalProducer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -266,10 +266,6 @@ L1TGlobalProducer::L1TGlobalProducer(const edm::ParameterSet& parSet)

m_currentLumi = 0;

//
std::vector<l1t::Muon> muonVec_bxm2;
std::vector<l1t::Muon> muonVec_bxm1;

// Set default, initial, dummy prescale factor table
std::vector<std::vector<double>> temp_prescaleTable;

Expand Down Expand Up @@ -645,7 +641,7 @@ void L1TGlobalProducer::produce(edm::Event& iEvent, const edm::EventSetup& evSet
receiveEtSumsZdc,
receiveCICADA);

m_uGtBrd->receiveMuonObjectData(iEvent, m_muInputToken, receiveMu, m_nrL1Mu, &muonVec_bxm2, &muonVec_bxm1);
m_uGtBrd->receiveMuonObjectData(iEvent, m_muInputToken, receiveMu, m_nrL1Mu);

if (m_useMuonShowers)
m_uGtBrd->receiveMuonShowerObjectData(iEvent, m_muShowerInputToken, receiveMuShower, m_nrL1MuShower);
Expand Down Expand Up @@ -704,13 +700,6 @@ void L1TGlobalProducer::produce(edm::Event& iEvent, const edm::EventSetup& evSet

} //End Loop over Bx

muonVec_bxm2 = muonVec_bxm1;
muonVec_bxm1.clear();
for (std::vector<const l1t::Muon*>::const_iterator iMu = (*(m_uGtBrd->getCandL1Mu())).begin(0);
iMu != (*(m_uGtBrd->getCandL1Mu())).end(0);
++iMu) {
muonVec_bxm1.push_back(**iMu);
}
// Add explicit reset of Board
m_uGtBrd->reset();

Expand Down
4 changes: 0 additions & 4 deletions L1Trigger/L1TGlobal/plugins/L1TGlobalProducer.h
Original file line number Diff line number Diff line change
Expand Up @@ -202,10 +202,6 @@ class L1TGlobalProducer : public edm::stream::EDProducer<> {

//switch to save axo scores in global board
bool m_produceAXOL1TLScore;

//vectors to store muon data for previous relative bx crossings
std::vector<l1t::Muon> muonVec_bxm2;
std::vector<l1t::Muon> muonVec_bxm1;
};

#endif // L1TGlobalProducer_h
49 changes: 9 additions & 40 deletions L1Trigger/L1TGlobal/src/GlobalBoard.cc
Original file line number Diff line number Diff line change
Expand Up @@ -376,9 +376,7 @@ void l1t::GlobalBoard::receiveCaloObjectData(const edm::Event& iEvent,
void l1t::GlobalBoard::receiveMuonObjectData(const edm::Event& iEvent,
const edm::EDGetTokenT<BXVector<l1t::Muon>>& muInputToken,
const bool receiveMu,
const int nrL1Mu,
const std::vector<l1t::Muon>* muonVec_bxm2,
const std::vector<l1t::Muon>* muonVec_bxm1) {
const int nrL1Mu) {
if (m_verbosity) {
LogDebug("L1TGlobal") << "\n**** GlobalBoard receiving muon data = ";
//<< "\n from input tag " << muInputTag << "\n"
Expand All @@ -405,45 +403,16 @@ void l1t::GlobalBoard::receiveMuonObjectData(const edm::Event& iEvent,

//Loop over Muons in this bx
int nObj = 0;
if (i == -2) {
for (std::vector<l1t::Muon>::const_iterator mu = muonVec_bxm2->begin(); mu != muonVec_bxm2->end(); ++mu) {
if (nObj < nrL1Mu) {
(*m_candL1Mu).push_back(i, &(*mu));
} else {
edm::LogWarning("L1TGlobal")
<< " Too many Muons (" << nObj << ") for uGT Configuration maxMu =" << nrL1Mu;
}

LogDebug("L1TGlobal") << "Muon Pt " << mu->hwPt() << " EtaAtVtx " << mu->hwEtaAtVtx() << " PhiAtVtx "
<< mu->hwPhiAtVtx() << " Qual " << mu->hwQual() << " Iso " << mu->hwIso();
nObj++;
}
} else if (i == -1) {
for (std::vector<l1t::Muon>::const_iterator mu = muonVec_bxm1->begin(); mu != muonVec_bxm1->end(); ++mu) {
if (nObj < nrL1Mu) {
(*m_candL1Mu).push_back(i, &(*mu));
} else {
edm::LogWarning("L1TGlobal")
<< " Too many Muons (" << nObj << ") for uGT Configuration maxMu =" << nrL1Mu;
}

LogDebug("L1TGlobal") << "Muon Pt " << mu->hwPt() << " EtaAtVtx " << mu->hwEtaAtVtx() << " PhiAtVtx "
<< mu->hwPhiAtVtx() << " Qual " << mu->hwQual() << " Iso " << mu->hwIso();
nObj++;
for (std::vector<l1t::Muon>::const_iterator mu = muonData->begin(i); mu != muonData->end(i); ++mu) {
if (nObj < nrL1Mu) {
(*m_candL1Mu).push_back(i, &(*mu));
} else {
edm::LogWarning("L1TGlobal") << " Too many Muons (" << nObj << ") for uGT Configuration maxMu =" << nrL1Mu;
}
} else {
for (std::vector<l1t::Muon>::const_iterator mu = muonData->begin(i); mu != muonData->end(i); ++mu) {
if (nObj < nrL1Mu) {
(*m_candL1Mu).push_back(i, &(*mu));
} else {
edm::LogWarning("L1TGlobal")
<< " Too many Muons (" << nObj << ") for uGT Configuration maxMu =" << nrL1Mu;
}

LogDebug("L1TGlobal") << "Muon Pt " << mu->hwPt() << " EtaAtVtx " << mu->hwEtaAtVtx() << " PhiAtVtx "
<< mu->hwPhiAtVtx() << " Qual " << mu->hwQual() << " Iso " << mu->hwIso();
nObj++;
}
LogDebug("L1TGlobal") << "Muon Pt " << mu->hwPt() << " EtaAtVtx " << mu->hwEtaAtVtx() << " PhiAtVtx "
<< mu->hwPhiAtVtx() << " Qual " << mu->hwQual() << " Iso " << mu->hwIso();
nObj++;
} //end loop over muons in bx
} //end loop over bx
} //end if over valid muon data
Expand Down

0 comments on commit 5db9511

Please sign in to comment.