[EMCAL-524] Fix filling energy spectrum for DCAL supermodules (#812)

mfasDa · web-flow · commit 11bfdabcc3a8 · 2021-08-20T14:59:42.000+02:00
Bug introduced in #792: Supemodule ID was not checked against range, therefore all supermodules treated as EMCAL supermodules.
diff --git a/Modules/EMCAL/src/DigitsQcTask.cxx b/Modules/EMCAL/src/DigitsQcTask.cxx
@@ -465,7 +465,7 @@ void DigitsQcTask::DigitsHistograms::fillHistograms(const o2::emcal::Cell& digit
       fillOptional2D(mDigitAmpSupermoduleCalib, digit.getEnergy(), supermoduleID);
       fillOptional2D(mDigitTimeSupermoduleCalib, digit.getTimeStamp() - timecalib, supermoduleID);
     }
-    if (supermoduleID) {
+    if (supermoduleID < 12) {
       fillOptional1D(mDigitAmplitudeEMCAL, digit.getEnergy());
     } else {
       fillOptional1D(mDigitAmplitudeDCAL, digit.getEnergy());

Original file line number	Diff line number	Diff line change
`@@ -465,7 +465,7 @@ void DigitsQcTask::DigitsHistograms::fillHistograms(const o2::emcal::Cell& digit`
`465`	`465`	`fillOptional2D(mDigitAmpSupermoduleCalib, digit.getEnergy(), supermoduleID);`
`466`	`466`	`fillOptional2D(mDigitTimeSupermoduleCalib, digit.getTimeStamp() - timecalib, supermoduleID);`
`467`	`467`	`}`
`468`		`- if (supermoduleID) {`
	`468`	`+ if (supermoduleID < 12) {`
`469`	`469`	`fillOptional1D(mDigitAmplitudeEMCAL, digit.getEnergy());`
`470`	`470`	`} else {`
`471`	`471`	`fillOptional1D(mDigitAmplitudeDCAL, digit.getEnergy());`