Skip to content
Snippets Groups Projects
Commit 01f96bf5 authored by Duc Ta's avatar Duc Ta
Browse files

Merge branch 'egammaTopoClusterCopier_typo' into '24.0'

egammaTopoClusterCopier : Small fix on the logic for the counters

See merge request !72145
parents cd8a4585 0cb1f23d
No related branches found
No related tags found
2 merge requests!721722024-06-12: merge of 24.0 into main,!72145egammaTopoClusterCopier : Small fix on the logic for the counters
......@@ -181,8 +181,6 @@ StatusCode egammaTopoClusterCopier::execute(const EventContext& ctx) const {
const bool valid_for_central = checkIfValidForCentral(aeta, m_etaCut, clusterE, m_ECut);
const bool valid_for_fwd = checkIfValidForFwd(aeta, m_fwdEtaCut, m_fwdETCut, clus, m_doForwardClusters, m_hasITk);
const bool valid_for_both = valid_for_central && valid_for_fwd;
if (valid_for_central) {
++buff_CentralPassPreSelection;
} else if (valid_for_fwd) {
......@@ -191,7 +189,9 @@ StatusCode egammaTopoClusterCopier::execute(const EventContext& ctx) const {
continue;
}
const bool valid_for_both = valid_for_central && valid_for_fwd;
if (valid_for_both) {
++buff_FwdPassPreSelection; // since we count them only in the else above
++buff_SharedPassPreSelection;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment