Private tool fix/muon segment combiners

See merge request !36134

Update Likelihood for new Ambiguity type from atlas/athena!35293

See merge request atlas/athena!36269

Rework ownership of Trk::Track::m_trackSummary

See merge request !36277

Move HLT_noalg_L1MBTS_1_EMPTY to PhysicsP1 menu (ATR-21740)

See merge request !36290

RpcRawDataMonitoring: Fix identifier validation test.

See merge request !36291

MetaDataSvc: Migrate to IOptionsSvc

See merge request !36272

METUtilities: Migrate METRebuilder to IOptionsSvc

See merge request atlas/athena!36257

drop unused tau variable, and code cleanup

See merge request !36205

Bertrand Martin Dit Latour authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

Hello,

This MR is dropping a variable that is not used anywhere (anymore) in the tau reconstruction.
We now use a Et-resolution weighted average between calo and substructure pt as input to the MVA pt, no longer an interpolation.
I have checked it does not change the reconstruction output, besides this variable that is now gone.
Minor code cleanup was done.

Cheers,
Bertrand

Pixel ladder shape change

See merge request atlas/athena!34961

TrigP1Test: Fix for magFieldToggle ART test

Closes ATR-21978

See merge request !36248

Fix data overlay to use IDCInDetBSErrContainer

Closes ATLASSIM-4746

See merge request !36265

Updated git-package-pseudomerge.py scrip to python3

See merge request !36266

TileMonitoring: Fix Tile digitizer time monitoring histogram name for Run 3

See merge request !36280

Magnetic Field Elements : Move more inline methods to the respective .icc files

See merge request !36267

IOVDbSvc: Remove usage of IJobOptionsSvc

See merge request !36278

HLT: Fixes for IOptionsSvc migration

See merge request !36271

Calling Identifier::is_valid() is not the same as the isValid flag
returned by RpcIdHelper.

Fixes (not entirely reproducible) crash seen with q222.

Tile digitizer time monitoring histogram name
have been changed, since digitizer starts from 1

`IJobOptionsSvc` is deprecated. Instead use the `IProperty` interface
directly to read the properties of the `EventSelector`.

In the existing code base, this member is a raw pointer with assumed
ownership, and there is also a data hiding problem since the
TrackSummaryTool is marked as a friend class of Track. This commit
fixes several minor issues with this data member:

1. The member is converted to a std::unique_ptr, instead of a raw
pointer. This makes the ownership mechanics clearer, and saves us a few
deletes here and there.

2. All instances where the data member was being accessed outside of
the class have been replaced with a setter method.

3. The existing setter method was changed to accept a std::unique_ptr,
in accordance with the data member itself.

4. Trk::TrackSummaryTool was removed as a friend class of Trk::Track.

Merge branch 'master' of https://gitlab.cern.ch/atlas/athena into PrivateToolFix/MuonSegmentCombiners

!36223 introduced a slight change in behavior when reading
`DF_Enabled_ROB_IDs`, i.e. it was always overwriting the list of enabled
ROBs even if a value was set explicitly via job options.

Fix bug when parsing job properties of the type `MyAlg.MyTool.MyProp`.
The client name is in this case `MyAlg.MyTool` and not only `MyAlg`.

AthenaServices: Delete AthenaJobOptionsSvc

See merge request !36254

RungeKuttaUtils avoid repeat of  const Amg::Transform3D&  T = su->transform()...

See merge request !36262

Do not write EventTag in new-style simulation jobs

See merge request atlas/athena!36261

Migrate various unit tests to IOptionsSvc

See merge request !36256

AthenaPython: Delete PyJobOptionsSvc

See merge request !36253

TrigP1Test: Update BeamSpot test reference

See merge request !36252

Preparation of Athena  to HepMC3

See merge request !36245