Minor fix in DetDescrDBEnvelopeSvc to support building GeoModel description
from SQLite database

Walter Lampl authored 2 years ago and Julien Maurer committed 2 years ago

reduce the number of conf2toConfigurable WARNINGs

Scott Snyder authored 2 years ago and Tadej Novak committed 2 years ago

BeamPipeGeoModel: Fix memory leaks.

Fix a few (one-time) memory leaks.

John Derek Chapman authored 2 years ago and Peter Berta committed 2 years ago

Rename CA-based configuration modules for Simulation/Digitization package + minor syntax for for LArDigitizationConfig

John Derek Chapman authored 2 years ago and Julien Maurer committed 2 years ago

Rename legacy Digitization, SubdetectorEnvelopes and TrackRecordGenerator configuration modules

Frank Winklmeier authored 2 years ago and Julien Maurer committed 2 years ago

RecExCommon: remove workaround in B-tagging configuration

Remove `configurableRun3Behavior` workaround in RecExCommon.

Relates to ATLASRECTS-6635.

John Derek Chapman authored 2 years ago and Peter Berta committed 2 years ago

Clean-up of AtlasTrackingGeometrySvcConfig

Update AtlasTrackingGeometrySvcConfig module to use more standard
CA-based configuration syntax.

Adapt to interface changes of `StoredMaterialManager`.

Pass strings by const reference, not by value.
Use boost::starts_with.

Enable thread-safety checking.

Be explicit if not reference file comparison is used by unit test.

DataQualityInterfaces, LArGeoEndcap, DumpEventDataToJSON, BeamPipeGeoModel cppcheck / clang-tidy fixes

(don't create configuration classes at module level)

Updated AtlasGeometryCommon packages to use FindGeoModel.cmake. Which was introduced
to serve the new GeoModel uni-project, which was introduced into our build with !39900.

Bonus:
 * Turned CavernInfraGeoModel into component-only package
 * Removed several commented-out include statements from VP1DetInfo.cxx

If a dtor is final, the class itself should also be final.

Enable thread-safety checking.

Dropped the redundant AbsMaterialManager interface, and removed all references to this
interface from client packages.

Dropped redundant methods from RDBMaterialManager, and in this way got rid of lots of
duplicated code.

- remove `atlas_depends_on_subdirs`
- update link dependencies
- replace `atlas_install_headers` with interface library or remove it
  entirely if headers not used outside package
- enable flake8 where applicable

There is already a library that provides the public headers and hence
the `atlas_install_headers` is redundant.

If a dtor is declared final, the class itself should also be declared final.

If a dtor is declared final, the class itself should also be declared final.

Library dependency fixes.