Skip to content

Manual sweep of !34258 from 21.3 to 21.9

Alexandre Laurier requested to merge alaurier/athena:21.9_CleanMMReadoutElements into 21.9

Manually sweeping !34258 (merged) from 21.3 to 21.9.

Changed the MM strip numbering scheme to start from 1 instead of 0 to protect the identifiers. Positions in the active MM volume which have no strips now return strip number 1 to protect against bad identifiers. Strip#1 are never readout in the MMs so strip#1 is a way to reduce errors and warnings.

MM channelPosition now returns the centre of the strip instead of the lower edge. This change is also mirrored in the stripNumber scheme.

Fixed a flaw in the sTGC strip position not returning exactly centre of the strips.

Cleaned a few glaring bad coding practices and indentations (always WIP).

Merge request reports