Skip to content

Replace ptr removeCell method with ElementLink in egammaForwardBuilder

Lucy Lewitt requested to merge llewitt/athena:egammaFwdBuilderRemoveByElementLink into main

The previous implementation of cookie cutting in egamma fwd removed cells using a pointer to the cell to be removed. This method is slow as it requires a loop over all cells to match the pointer to the cell in the container to be removed.

This MR replaces the call to this method with the alternative provided by CaloClusterCellLinks which allows for deletion by an element link which requires no additional loop.

This methodology also allows for the elimination of an intermediary vector, and an associated loop, of cells to remove which is nice 😄

Edited by Lucy Lewitt

Merge request reports