Skip to content

implement GEC based on VELO clusters

Christina Agapopoulou requested to merge velo_gec into 2024-patches

This MR:

  • implements a GEC based on the number of velo clusters
  • turns it on by default for all sequence with min_value = 0 and max_value = 35k
  • increase the SciFi GEC to 20k

The number of VELO clusters is the real value from the Retina decoding. In case the SP sequence is used, an estimate cluster size based on the bank size is used instead.

There's also a small refactoring of the python modules that was needed to avoid having a circular import between velo_reconstruction.py and utils.py

To be tested with MooreOnline!476 (merged)

Related to #559 (closed)

@dovombru @decianm @cmarinbe @rmatev

Edited by Miroslav Saur

Merge request reports