Skip to content

Fix redundant access to FullPhysVol and AlignableTransform in the Pixel readout

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

This MR contains 2 commits to reduce the execution time to load and build the Pixel geometry (raw+readout) in the SQLite workflow (not used in the default workflow):

Similarly to !57843 (merged), fix the redundant access to FullPhysVol and AlignableTransforms when reading the geometry from an SQLite file, by adding 2 private members to the PixelDetectorFactoryLite class and the GeoVPixelFactory class Adds 2 extra-arguments to the GeoVPixelFactory class so that the maps are not retrieved each time an object derived from the GeoVPixelFactory class is created. Instead they are passed as argument in the constructor.

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

Merge request reports

Loading