Skip to content

Cleaning hit track decorator in IDPVM

Noemi Calace requested to merge ncalace/athena:feat-clean-IDPVM into main

Cleaning unused code. In particular I have removed two variables, `m_ptThreshold` and `m_doUpgrade`, that seem not to be configurable and reset anywhere else. Especially for `m_ptThreshold`, this could lead to unwanted and unexpected track selections.

Removing the if clause, made me re-indent the code... I am sorry for the inconvenience this brings to shifters reviewing it.

It seems this code doesn't run when producing IDTRKVALID, but in more specific workflows where additional information on PRDs is required.

FYI: @tstreble @goetz @sroe

and some others that might be using this decorator: @oducu @xai @mvessell @gfrattar @cgrefe

Merge request reports