Remove getDet in CaloFutureElectron
The usage of getDet
has to be removed from https://gitlab.cern.ch/lhcb/Rec/-/blob/master/CaloFuture/CaloFutureTools/src/CaloFutureElectron.cpp
It appears in 3 methods:
-
closestState
: used by https://gitlab.cern.ch/lhcb/Rec/-/blob/master/CaloFuture/CaloFutureMoniDst/src/CaloFutureAlignmentNtp.cpp which can actually be removed. So bothCaloFutureAlignmentNtp.cpp
andCaloFutureElectron::closestState
should be removed -
caloTrajectoryL
: not used, so it can also be removed too -
caloState
: used byCaloFutureAlignmentNtp
,CaloFutureElectronNtp
andCaloFututreECorrection
. The first 2 can be removed. InCaloFututreECorrection
,caloState
is called bycomputeDTheta
, that is called byprocess
. Sinceprocess
here already takesDeCalorimeter
as argument, one just needs to pass it as argument tocomputeDTheta
and from there tocaloState
.
Also related to #330 (closed) FYI @sponce @bcouturi
Edited by Carla Marin Benito