Skip to content

Commit

Permalink
Change FEStatus quality threshold
Browse files Browse the repository at this point in the history
  • Loading branch information
abhih1 committed Jul 11, 2024
1 parent 0882bff commit a3bef8c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
2 changes: 1 addition & 1 deletion DQM/EcalMonitorClient/interface/RawDataClient.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ namespace ecaldqm {

private:
void setParams(edm::ParameterSet const&) override;

int minEvents_;
float synchErrThresholdFactor_;
};

Expand Down
4 changes: 3 additions & 1 deletion DQM/EcalMonitorClient/python/RawDataClient_cfi.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,11 @@
from DQM.EcalMonitorTasks.RawDataTask_cfi import ecalRawDataTask

synchErrThresholdFactor = 1.
minEvents = 400

ecalRawDataClient = cms.untracked.PSet(
params = cms.untracked.PSet(
minEvents = cms.untracked.int32(minEvents),
synchErrThresholdFactor = cms.untracked.double(synchErrThresholdFactor)
),
sources = cms.untracked.PSet(
Expand All @@ -20,7 +22,7 @@
kind = cms.untracked.string('TH2F'),
otype = cms.untracked.string('Ecal3P'),
btype = cms.untracked.string('SuperCrystal'),
description = cms.untracked.string('Summary of the raw data (DCC and front-end) quality. A channel is red if it has nonzero events with FE status that is different from any of ENABLED, DISABLED, SUPPRESSED, FIFOFULL, FIFOFULL_L1ADESYNC, and FORCEDZS. A FED can also go red if its number of L1A desynchronization errors is greater than ' + str(synchErrThresholdFactor) + ' * log10(total entries).')
description = cms.untracked.string('Summary of the raw data (DCC and front-end) quality. A channel is red if it has '+ str(minEvents)+' or more events with FE status that is different from any of ENABLED, DISABLED, SUPPRESSED, FIFOFULL, FIFOFULL_L1ADESYNC, and FORCEDZS. A FED can also go red if its number of L1A desynchronization errors is greater than ' + str(synchErrThresholdFactor) + ' * log10(total entries).')
),
ErrorsSummary = cms.untracked.PSet(
path = cms.untracked.string('%(subdet)s/%(prefix)sSummaryClient/%(prefix)sSFT front-end status errors summary'),
Expand Down
5 changes: 3 additions & 2 deletions DQM/EcalMonitorClient/src/RawDataClient.cc
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,12 @@

namespace ecaldqm {

RawDataClient::RawDataClient() : DQWorkerClient(), synchErrThresholdFactor_(0.) {
RawDataClient::RawDataClient() : DQWorkerClient(), minEvents_(0), synchErrThresholdFactor_(0.) {
qualitySummaries_.insert("QualitySummary");
}

void RawDataClient::setParams(edm::ParameterSet const& _params) {
minEvents_ = _params.getUntrackedParameter<int>("minEvents");
synchErrThresholdFactor_ = _params.getUntrackedParameter<double>("synchErrThresholdFactor");
}

Expand Down Expand Up @@ -57,7 +58,7 @@ namespace ecaldqm {
for (unsigned iS(0); iS < nFEFlags; iS++) {
float entries(sFEStatus.getBinContent(getEcalDQMSetupObjects(), id, iS + 1));
towerEntries += entries;
if (entries > 0. && iS != Enabled && iS != Suppressed && iS != ForcedFullSupp && iS != FIFOFull &&
if (entries > minEvents_ && iS != Enabled && iS != Suppressed && iS != ForcedFullSupp && iS != FIFOFull &&
iS != ForcedZS)
towerStatus = doMask ? kMBad : kBad;
}
Expand Down

0 comments on commit a3bef8c

Please sign in to comment.