Making PatPV3D a Transformer instead of MulitTransformerFilter (!1325) · Merge requests · LHCb / Rec

Christoph Hasse requested to merge PatPV3DFutureNoFilter into master Nov 30, 2018

Removing the filter functionality from PatPV3D.

This could mean a degradation of the throughput since an event with zero PVs which still has some VELO tracks will not be thrown away and thus cost time in following algorithms. I guess therefore we shouldn't quite yet merge this.

However, given @olupton's work in !1291 (merged) we could wait for that to be merged and add a similar filter? Or the other option would be a filter like the PrGECFilter given that the filter on PV's would also kill the whole Event and not provide a selection as the filters discussed in !1291 (merged) do. Hoping for some input on this.

fyi: @sstahl @cattanem

Edited Dec 05, 2018 by Marco Cattaneo

Making PatPV3D a Transformer instead of MulitTransformerFilter

Merge request reports