Skip to content

modernise-Calo2Dview

Chitsanu Khurewathanakul requested to merge modernise-Calo2Dview into future

This request modernise Calo2Dview with the change such that there's no more unnecessary internally-cached variables. The previous implementation relies on getCaloParam to be called prior to filling the histograms, and stored the cached values in private member attributes. Those values are constants anyway, for 4 given subdetectors type. The new implementation groups them into struct and const references to it on demand instead of relying on calling getCaloParam to get things in sync. Effectively, this allow many class methods to be const for the future framework.

note: The original motivation was from the migration for Gaudi::Functional::Consumer in Rec/Calo/CaloMoniDst package. Those classes are derived from Calo2Dviewand they need the underlying class methods to be const.

Merge request reports