Skip to content

TrackParameters updateParameters allow in place update of the covariance

Christos Anastopoulos requested to merge ATLAS-EGamma/athena:update_param_ref_overload into master

Have seen in places this pattern

AmgSymMatrix(5)* covariance = new AmgSymMatrix(5);
//Calculate 
foo.updateParameters(mean, covariance);

where foo is TrackParameters This leads to a new / delete cycle for the existing method, new matrix to pass, internally delete the one that is hold by the TrackParameters object.

Add another interface, which allows to pass by ref and if the m_covariance is there allows to be updated in place (no new/delete). If m_covariance is not there a new is created with the passed values.

Mentioning @ssnyder and @amorley

Edited by Christos Anastopoulos

Merge request reports