Skip to content

Commit 0c5d342

Browse files
authored
fix occupancy flag comparison
1 parent 2c46716 commit 0c5d342

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎DPG/Tasks/AOTEvent/bcOccupancyQa.cxx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ struct BcOccupancyQaTask {
164164
// so bcInTF = (globalBC - globalBC of the first row) % nBCsPerTF; check that nOrbitsPerTF is consistent with the table
165165
const uint64_t firstGlobalBC = bcOccs.iteratorAt(0).globalBC();
166166
const uint64_t lastGlobalBC = bcOccs.iteratorAt(bcOccs.size() - 1).globalBC();
167-
const bool isBcInTFValid = (static_cast<int64_t>(bcOccs.size()) % nBCsPerTF == 0) && (static_cast<int64_t>((lastGlobalBC - firstGlobalBC) % nBCsPerTF) == nBCsPerTF - 1);
167+
const bool isBcInTFValid = (bcOccs.size() % nBCsPerTF == 0) && (static_cast<int64_t>((lastGlobalBC - firstGlobalBC) % nBCsPerTF) == nBCsPerTF - 1);
168168
if (!isBcInTFValid) {
169169
LOGP(warning, "BC occupancy QA: table size {} is not consistent with {} orbits per TF, no histograms vs BC in TF for this DF", bcOccs.size(), nOrbitsPerTF.value);
170170
}
@@ -267,7 +267,7 @@ struct BcOccupancyQaTask {
267267
if (bc.trackOccupancyInTimeRange() >= 0 && occAll.trackOccupancyInTimeRange() >= 0) {
268268
histos.fill(HIST("BcOccsVsBcOccsAll/hCheck"), 4);
269269
if (bc.trackOccupancyInTimeRange() == occAll.trackOccupancyInTimeRange() && bc.ft0cOccupancyInTimeRange() == occAll.ft0cOccupancyInTimeRange() &&
270-
bc.occupancyMedianTime() == occAll.occupancyMedianTime() && bc.occupancyFlags() == occAll.occupancyFlags()) {
270+
bc.occupancyMedianTime() == occAll.occupancyMedianTime() && bc.occupancyFlags_raw() == occAll.occupancyFlags_raw()) {
271271
histos.fill(HIST("BcOccsVsBcOccsAll/hCheck"), 5);
272272
}
273273
}

0 commit comments

Comments
 (0)