Specified a default stream name (ANALYSIS) for all histogram handling components in master (cherry-pick)

See merge request atlas/athena!37916

LArHV: Thread-safety cleanups.

See merge request atlas/athena!37845

RootUtils: cmake fix for clang compilation

See merge request atlas/athena!38110

TrigT1CaloByteStream: Make PpmByteStreamV1Tool const.

See merge request atlas/athena!38111

PixelRawDataByteStreamCnv: Use AthConstConverter.

See merge request atlas/athena!38112

L1TopoAlgorithms: Fix clang warning.

See merge request atlas/athena!38113

CaloTriggerTool: Use AthMessaging in LArTTCellMap.

See merge request atlas/athena!38115

ImplicitCollection: Fix cppcheck warnings.

See merge request atlas/athena!38116

IOVSvc: Remove locking from CondSvc::isValidID.

See merge request atlas/athena!38117

ATLASRECTS-5750: README for LwtnnUtils

See merge request atlas/athena!38033

move to DB with IOV over full Run 2

See merge request atlas/athena!38106

Delete Physics "tag" packages

See merge request atlas/athena!38086

RecExAnaTest: delete obsolete RTT test package

See merge request atlas/athena!38090

AnalysisTest: delete obsolete package

See merge request atlas/athena!38092

TopPhys/xAOD: cmake cleanup

See merge request atlas/athena!38120

PhysicsD3PDMaker: delete package

See merge request atlas/athena!38123

D3PDMakerConfig: package cleanup

See merge request atlas/athena!38124

RecExRecoTest: package cleanup

See merge request atlas/athena!38135

ATLASRECTS-5750: Lwtnn maps

See merge request atlas/athena!38018

Gabriel Facini authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

Tracks passing a selection designed to identify photon conversions were
given a shortcut through the ambi for a small efficiency gain. This had
the unwanted side-effect of allowing electron brem tracks (mostly in the
forward direction) to have incorrect hits and overshare IBL hits
lowering the EM LH identification rate.

Simple fix - remove short cut
Impact on digest - 3 tracks over 3 events

Delete obsolete RTT-based job options and other obsolete scripts.

Remove `atlas_depends_on_subdirs` and tidy link dependencies.

Delete obsolete physics D3PDMaker package and references to it.

Delete obsolete job options and python modules. Essentially everything
that is related to common or physics D3PDs and what is not used
elsewhere. Also enable flake8.

RecExCommon: Define TriggerFlags.

See merge request atlas/athena!38109

MM_Digitization: Fix clang warning.

See merge request atlas/athena!38108

The locking within CondSvc::isValidID is heavily contended in jobs with
many threads (32-64).  Rework so that we don't need to use locking.

Rather than doing a SG lookup each time isValidID is called, we instead
build a map from SG key to CondCont instances at the end of initialization.
(This is called from CondInputLoader::start).  We can then look up the
key directly in this map rather then querying SG.  Since this map
is modified only during START, we don't need any locking.

 - Pass class instances by const reference, not by value.

Use AthMessaging in LArTTCellMap.
In passing, fix cppcheck warnings.

 - fabs(int).  Use std::abs instead.
 

Change PixelRawContByteStreamCnv to derive from AthConstConverter.
Make methods const as needed.
Some general cleanup.

    
Make interfaces of PpmByteStreamV1Tool const, for MT compatibility.

Only the RootUtilsPyROOT library should depend on the pyroot libraries
(ROOTTPython and cppyy).  In particular, the RootUtils library should not
depend on these libraries.

Things were failing in the clang build because RootUtils was linking
against cppyy but _not_ against python, so things downstream were failing
to find python symbols.

(In principle this is wrong for the gcc build as well; not entirely
sure why it was working there.)

Missing import of TriggerFlags.

One-billionth is spelled 1e-9, not 10^-9.  The latter is -3.