StauTool has a private data member m_runNumber which is set in a couple of places
inside the code, but is never been used. Removal of this data member also made
it possible to remove the reference to the old EventInfo

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

Read test input files from cvmfs rather than afs.
Seeing if this will help with the timeouts that have been observed
in recent nightlies.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

Create bunch of config files for classes used by IP2D, IP3D and RNNIP taggers
Cleaning in TrackSelector.cxx (ToolSvc not used)

Note the objects produced by tauRec as transient in objKeyStore.
Needed for the output streams to get the correct dependencies in MT.

Note the objects produced by MuonCreatorAlg as transient in objKeyStore.
Needed for the output streams to get the correct dependencies in MT.

John Kenneth Anders authored 6 years ago and Atlas Nightlybuild committed 6 years ago

Adding variable HadronConeExclTruthLabelID to heavy-ion jet collections

See merge request atlas/athena!20652

(cherry picked from commit d0330f7a)

1fed1e45 Adding variable HadronConeExclTruthLabelID to heavy-ion jet collections.

HandleKeys must be declared as properties in order for dependencies
to propagate correctly.

Needed to get dependencies propagated from tools.

Digits for TMDB from bytestream are unpacked in the same time
when Tile digits are requested and upacked and put directly
into StoreGate ignoring standard conversion mechanism.
The same is done in the case of unpacking raw channels for TMDB.
This is not very good for AthenaMT (especially for scheduler).

Tile bytestream converters have been refactored
to unpack digits and raw channels for TMDB
only if they are requested (needed)
using standard conversion mechanism.

Unit tests for reading Tile digits and raw channels from bytestream
and the corresponding reference files have been updated accordingly.

Use Read/Write handles instead of direct SG access.
Also general cleanup.  Make tool methods const and the algorithm reentrant.

Update METMakerAlg to work with accept() returning AcceptData.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

This is in preparation for making ICaloCellMakerTool::process const,
for use in MT.  This particular tool hasn't been converted to work
reentrantly, so give an error if someone attempts to do that.

Private data members should start with m_.

In C++11, implicit declarations of copy and assignment are deprecated if the
class has a user defined destructor or copy or assignment.  gcc9 now warns
about this by default.
Adjust to avoid the warning.

Remove debugging code left in by mistake.

In cases where MuGirl muons are also reconstructed by STACO, they can have an MS track particle, the one used by STACO, associated to them.
Given that this MS track was not used in reconstructing the muon, this can be misleading: this fix ensures that MuGirl muons never have an associated MS track.
Since the MCP tools use the ME track, this change should be transparent for the vast majority of analyzers.
The MS-only extrapolated track, only used by experts for debugging, is retained.

clang warnings: unused private class members.

Update `TrackParticleTruthAthenaPool` to write out the new
`TrackParticleTruthCollection_p2` and `TrackParticleTruthCollectionContainer_p2`
classes which use `HepMcParticleLink_p2` by default.
(cherry picked from commit aa3286c2a93ac47d342d30131581dc15970e1f27)

Add unit tests the for new TP Converters
(cherry picked from commit 4d3cc18027874d378522730b6e941ced8e8eba5a)

Update the new unit tests to use valid `HepMcParticleLinks`.

Non-members should not start with m_.