re-working CaloClusterCorrectionCommon to retrieve CaloDetDescrManager from CondStore
Compare changes
An attempt to retrieve the CaloDetDesrcManager from the ConditionStore in CaloClusterCorrectionCommon. So far, this object was retreived in a somewhat hidden way and cached inside CaloClusterCorrectionCommon::DDHelper
.
The proposed way forward:
ReadCondHandleKey<CaloDetDescrManager>
to the base-class CaloClusterCorrection
CaloClusterCorrectionCommon::makeCorrection
. Caveat: The retrieve with the implict EventContext
-lookup happens more frequently than necessary and is potentially slow.CaloDetDescrManager
through the call-chain of CaloClusterCorrectionCommon::DDHelper
DDHelper
using whatever version of the CaloDetDescrManager
is valid at this point in time (and put in a CachedUniquePtr
) At least in theory, subsequent calls to the DDHelper may use a version of the CaloDetDescrManager
with updated alignment. I believe in practice this is irrelevant, though it's not nice.I'd like to hear @ssnyder's opinion ...
One unit-test (CaloScaleCluster_test.py) is still failing b/c its reference values come from an un-aligned geometry and I don't know how to obtain an unalinged CaloDetDescrManager instance with the new cond-algos. @tsulaia?