Skip to content

Commit

Permalink
[EMCAL-795]: Add protection for TRUs when cheking HW address (AliceO2…
Browse files Browse the repository at this point in the history
…Group#12839)

- Minor altro errors are observed with the error "CHANNEL_ORDER"
- The altro index for TRUs does not exist and therefore, the altro index is uninitialized
- Hence, no check for the altro index has to be done for TRUs
- TRUs correpond to fec==0 in each branch. The Altro Index check is therefore only performed for fec > 0

Co-authored-by: jokonig <jokonig@cern.ch>
  • Loading branch information
2 people authored and mwinn2 committed Apr 25, 2024
1 parent fdb7ff5 commit cbb1d00
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Detectors/EMCAL/reconstruction/src/AltroDecoder.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -178,8 +178,10 @@ bool AltroDecoder::checkChannelHWAddress(int hwaddress)
if (fec > 9) {
return false;
}
if (!(altro == 0 || altro == 2 || altro == 3 || altro == 4)) {
return false;
if (fec > 0) { // fec == 0 corresponds to trigger, which only has a fake altro information. Hence it's not checked
if (!(altro == 0 || altro == 2 || altro == 3 || altro == 4)) {
return false;
}
}
return true;
}
Expand Down

0 comments on commit cbb1d00

Please sign in to comment.