Christos Anastopoulos authored 6 years ago and Christos Anastopoulos committed 6 years ago

Former-commit-id: 9ee46528d8f8b6a4af33343605c92a804681af98

MuRcvRawChCnt was moved to being produced via its own coverter, rather than
as a side effect of unpacking the main container.  Adjust the
ByteStreamAddressProviderSvc configuration accordingly.

As part of the migration away from the old-style event info, the reference to EventInfo
has been removed from the EventIncident. The clients of EventIncident have been updated
such that they now pass event information via EventContext, which is a data member of
the Gaudi Incident class.

Several standalone algorithms (either test or conditions loading jobs) have been switched
to direct usage of event contexts (via getContext()) instead of taking them from the
incidents.

Also dropped the redundant CheckIOV incident

- MetaDataSvc does not call prepareOutput at stop when it fires the
  incident
- EventFormatSvc runs into problems in MP. Was missing check for
  pre-existing object

Change defining Parameter interface for NeutralParticle to be the same as TrackParticle, cleaner and MT compatible

Change EventCounterAlg to use ReadHandle.

CaloRinger is on by default in RecExCommon, and doesn't obey the doWrite
switches.  Futher, it loads Athena libraries during job configuration,
which can lead to crashes in gcc8 builds.  Explicitly disable it.

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.

Fix issues found by the thread-safety static checker.

The static accessor objects declared in the xAODCore macros
should be const.  Avoids warnings from the thread-safety checker.

Remove references to TileBeamInfoProvider from TileMcSignal_jobOptions.py.
This JO fragment is not a complete JO but neither is it currently
used from anywhere, so this is a best guess but untested.

Enable thread-safety checker for this package; fix problems that
it identified.

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.

Needed for current gcc9.

a moment we then check if the PFO has it.

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.

Add option to selectively exclude items from the dump.

Update CaloOverlay_joboptions.py for switch from (MT-unsafe)
TileBeamInfoProvider to using TileDQstatusAlg.

Add option to sg-dump to skip events before starting dumping.