Skip to content

Fix redundant access to FullPhysVol and AlignableTransform in the SCT readout

Marilena Bandieramonte requested to merge mbandier/athena:opt-sctDD-read into 23.0

Similarly to !62531 (merged) and following !57843 (merged) (merged) this MR optimize the time spent by GeoModel in reading the SCT geometry from an SQLite file and building the corresponding readout geometry in the new Detector Description (DD) workflow (not the default workflow). This is done by adding 2 private members to the SCT_ComponentFactory class and 2 extra-arguments to the constructor of this class so that the maps are not retrieved each time an object derived from the SCT_ComponentFactory class is created. Instead they are passed as arguments in the constructor.

With this MR the total time spent to build the SCT geometry in the SQLite workflow drops from 4.3 s to 0.08 s.

Merge request reports