Skip to content

Commit 5796126

Browse files
authored
Merge pull request #49330 from smuzaffar/ecal-dqm-bin-order
DQMEcalMonitor: Root now requires bins in increasing order
2 parents 1cf33b6 + a80305d commit 5796126

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

DQM/EcalCommon/src/MESetEcal.cc

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,10 @@ namespace ecaldqm {
177177
specs[iSpec]->edges = std::vector<float>(specs[iSpec]->nbins + 1);
178178
int nbins(specs[iSpec]->nbins);
179179
double low(specs[iSpec]->low), high(specs[iSpec]->high);
180-
for (int i(0); i < nbins + 1; i++)
181-
specs[iSpec]->edges[i] = low + (high - low) / nbins * i;
180+
double binSize = (high - low) / nbins;
181+
specs[iSpec]->edges[0] = low;
182+
for (int i(1); i < nbins + 1; i++)
183+
specs[iSpec]->edges[i] = specs[iSpec]->edges[i - 1] + binSize;
182184
}
183185
}
184186
me = _ibooker.book2D(name, name, xaxis.nbins, &(xaxis.edges[0]), yaxis.nbins, &(yaxis.edges[0]));

DQM/EcalMonitorTasks/python/SelectiveReadoutTask_cfi.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
import FWCore.ParameterSet.Config as cms
22

3-
dccSizeBinEdges = []
4-
for i in range(11) :
5-
dccSizeBinEdges.append(0.608 / 10. * i)
3+
binSize = 0.0608
4+
dccSizeBinEdges = [0]
5+
for i in range(1, 11) :
6+
dccSizeBinEdges.append(dccSizeBinEdges[-1]+binSize)
7+
binSize = binSize * 10.
68
for i in range(11, 79) :
7-
dccSizeBinEdges.append(0.608 * (i - 10.))
9+
dccSizeBinEdges.append(dccSizeBinEdges[-1]+binSize)
810

911
ecalSelectiveReadoutTask = cms.untracked.PSet(
1012
params = cms.untracked.PSet(

0 commit comments

Comments
 (0)