Update CalibTracker/SiStripHitEfficiency/plugins/SiStripHitEfficiency…

…Harvester.cc as per code review.
cms-sw · Jul 5, 2023 · e68a53c · e68a53c
1 parent c41694d
commit e68a53c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/CalibTracker/SiStripHitEfficiency/plugins/SiStripHitEfficiencyHarvester.cc b/CalibTracker/SiStripHitEfficiency/plugins/SiStripHitEfficiencyHarvester.cc
@@ -403,7 +403,7 @@ void SiStripHitEfficiencyHarvester::dqmEndJob(DQMStore::IBooker& booker, DQMStor
             if (denom) {
               assert(num <= denom);  // can't have this happen
               const auto eff = num / denom;
-              const auto eff_up = (num <= denom) ? TEfficiency::Bayesian(denom, num, .99, 1, 1, true) : 1.f;
+              const auto eff_up = TEfficiency::Bayesian(denom, num, .99, 1, 1, true);
 
               if ((denom >= nModsMin_) && (eff_up < layer_min_eff)) {
                 //We have a bad module, put it in the list!