Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add protection against 0-element uGtAlgoBlocks in HLTL1TSeed module #13510

Merged
merged 2 commits into from Mar 1, 2016
Merged

Add protection against 0-element uGtAlgoBlocks in HLTL1TSeed module #13510

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
cc32556
Add protection against 0-element uGtAlgoBlocks
Martin-Grunewald Feb 27, 2016
7753ae9
More formatting
Martin-Grunewald Feb 27, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
55 changes: 33 additions & 22 deletions HLTrigger/HLTfilters/src/HLTL1TSeed.cc
View file
Open in desktop
Expand Up @@ -161,10 +161,10 @@ bool HLTL1TSeed::seedsAll(edm::Event & iEvent, trigger::TriggerFilterObjectWithR
iEvent.getByToken(m_l1MuonToken, muons);
if (!muons.isValid()){
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1MuonBxCollection with input tag "
<< " Warning: L1MuonBxCollection with input tag "
<< m_l1MuonTag
<< "\nrequested in configuration, but not found in the event."
<< "\nNo muons added to filterproduct."
<< " requested in configuration, but not found in the event."
<< " No muons added to filterproduct."
<< endl;
} else {

Expand All @@ -184,10 +184,10 @@ bool HLTL1TSeed::seedsAll(edm::Event & iEvent, trigger::TriggerFilterObjectWithR
iEvent.getByToken(m_l1EGammaToken, egammas);
if (!egammas.isValid()){
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1EGammaBxCollection with input tag "
<< " Warning: L1EGammaBxCollection with input tag "
<< m_l1EGammaTag
<< "\nrequested in configuration, but not found in the event."
<< "\nNo egammas added to filterproduct."
<< " requested in configuration, but not found in the event."
<< " No egammas added to filterproduct."
<< endl;
} else {

Expand All @@ -207,10 +207,10 @@ bool HLTL1TSeed::seedsAll(edm::Event & iEvent, trigger::TriggerFilterObjectWithR
iEvent.getByToken(m_l1JetToken, jets);
if (!jets.isValid()){
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1JetBxCollection with input tag "
<< " Warning: L1JetBxCollection with input tag "
<< m_l1JetTag
<< "\nrequested in configuration, but not found in the event."
<< "\nNo jets added to filterproduct."
<< " requested in configuration, but not found in the event."
<< " No jets added to filterproduct."
<< endl;
} else {

Expand All @@ -230,10 +230,10 @@ bool HLTL1TSeed::seedsAll(edm::Event & iEvent, trigger::TriggerFilterObjectWithR
iEvent.getByToken(m_l1TauToken, taus);
if (!taus.isValid()){
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1TauBxCollection with input tag "
<< " Warning: L1TauBxCollection with input tag "
<< m_l1TauTag
<< "\nrequested in configuration, but not found in the event."
<< "\nNo taus added to filterproduct."
<< " requested in configuration, but not found in the event."
<< " No taus added to filterproduct."
<< endl;
} else {

Expand All @@ -253,10 +253,10 @@ bool HLTL1TSeed::seedsAll(edm::Event & iEvent, trigger::TriggerFilterObjectWithR
iEvent.getByToken(m_l1EtSumToken, etsums);
if (!etsums.isValid()){
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1EtSumBxCollection with input tag "
<< " Warning: L1EtSumBxCollection with input tag "
<< m_l1EtSumTag
<< "\nrequested in configuration, but not found in the event."
<< "\nNo etsums added to filterproduct."
<< " requested in configuration, but not found in the event."
<< " No etsums added to filterproduct."
<< endl;
} else {

Expand Down Expand Up @@ -481,11 +481,22 @@ bool HLTL1TSeed::seedsL1TriggerObjectMaps(edm::Event& iEvent,
if (!uGtAlgoBlocks.isValid()) {

edm::LogWarning("HLTL1TSeed")
<< "\nWarning: GlobalAlgBlkBxCollection with input tag "
<< " Warning: GlobalAlgBlkBxCollection with input tag "
<< m_l1GlobalTag
<< "\nrequested in configuration, but not found in the event." << std::endl;
<< " requested in configuration, but not found in the event." << std::endl;

return false;
return false;
}

// check size
if(uGtAlgoBlocks->size() == 0) {

edm::LogWarning("HLTL1TSeed")
<< " Warning: GlobalAlgBlkBxCollection with input tag "
<< m_l1GlobalTag
<< " is empty." << std::endl;

return false;
}

// get handle to object maps (one object map per algorithm)
Expand All @@ -495,9 +506,9 @@ bool HLTL1TSeed::seedsL1TriggerObjectMaps(edm::Event& iEvent,
if (!gtObjectMapRecord.isValid()) {

edm::LogWarning("HLTL1TSeed")
<< "\nWarning: L1GlobalTriggerObjectMapRecord with input tag "
<< " Warning: L1GlobalTriggerObjectMapRecord with input tag "
<< m_l1GtObjectMapTag
<< "\nrequested in configuration, but not found in the event." << std::endl;
<< " requested in configuration, but not found in the event." << std::endl;

return false;
}
Expand Down Expand Up @@ -566,7 +577,7 @@ bool HLTL1TSeed::seedsL1TriggerObjectMaps(edm::Event& iEvent,
if(objMap == 0) {

edm::LogWarning("HLTL1TSeed")
<< "\nWarning: seed with name " << algoName << " cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap" << std::endl;
<< " Warning: seed with name " << algoName << " cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap" << std::endl;
return false;

}
Expand Down Expand Up @@ -612,7 +623,7 @@ bool HLTL1TSeed::seedsL1TriggerObjectMaps(edm::Event& iEvent,
// Should not get here
//
edm::LogWarning("HLTL1TSeed")
<< "\nWarning: seed with name " << algoSeedName << " cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap" << std::endl;
<< " Warning: seed with name " << algoSeedName << " cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap" << std::endl;
return false;

}
Expand Down