BCIDs from all Tile DMUs in one module are compared with one
from the 2nd DMU in the same module and if they are different
then channels from the corresponding DMUs are masked on the fly.
But if the 2nd DMU has bad BCID itself then whole module is masked on the fly.
But in the case when it is known that particular Tile DMU has wrong BCID
it is not very good.

To avoid masking channels (in the worst case whole module) when it is known
that the corresponding Tile DMU has wrong BCID possiblity to ignore
this known wrong BCID by putting special flag into DB has been added.

New Tile channel status OnlineWrongBCID and WrongBCID have been added
to mark channels from Tile DMU which is known to have wrong BCID in the DB.
They are differnt for possiblity to change known wrong BCID in offline
if it is needed.

Initialization of Tile DQ status has been modified accordingly
to ignore known wrong Tile BCID.

Reference files for unit tests have been updated.

Disable reentrancy for TrigMultiTrkHypoMT

See merge request atlas/athena!23616

Protection against division by zero (obo Igor Gavrilenko)

Closes ATLASRECTS-4205

See merge request atlas/athena!23621

fixes CA unit tests that have forgotten to swich configurable behaviour to run3

See merge request atlas/athena!23617

Squash MdtCalibrationDbSvc MT migration v3

See merge request atlas/athena!23626

Remove public tools from the GSF

See merge request atlas/athena!23620

Fix ATLASRECTS-4966 by downgrading ERROR messages to WARNING messages and...

Closes ATLASRECTS-4966

See merge request atlas/athena!23625

Susumu Oda authored 5 years ago and Adam Edward Barton committed 5 years ago

Fix ATLASRECTS-4966 by downgrading ERROR messages to WARNING messages and limiting the number of such WARNINGs.

Sweeping !23596 from 21.2 to master.
DirectIOART: Adding TFile::Open tests probing rucio redirector

See merge request atlas/athena!23642

Sweeping !23214 from 21.3 to master.
Fix get_hash function of MuonIdHelper for non-continuos Identifiers and unchecked identifier in MdtCalibDbCoolStrTool

See merge request atlas/athena!23644

SCT_Cabling: Fix clang warnings.

See merge request atlas/athena!23640

SCT_Digitization: Fix compilation with clang.

See merge request atlas/athena!23639

DerivationFrameworkInDet: ToolHandle const fix.

See merge request atlas/athena!23637

John Derek Chapman authored 5 years ago and Atlas Nightlybuild committed 5 years ago

Fix get_hash function of MuonIdHelper for non-continuos Identifiers and unchecked identifier in MdtCalibDbCoolStrTool

See merge request atlas/athena!23214

(cherry picked from commit af5e486eacf52a86837b8a8787c73cf23ae15360)

24c46f12 make calc_hash method of MuonIdHelpers work also for non-continues vectors of identifiers
6fe16e59 add != end() check
4ca6b07a Merge branch '21.3' of ssh://gitlab.cern.ch:7999/atlas/athena into muHelperHashes
5e6de5a9 check the detector element hash for the validity of the chamber identifier
467e2e26 Merge branch '21.3' of ssh://gitlab.cern.ch:7999/nkoehler/athena into muHelperHashes
366b732a always check the StatusCode
437cbf09 Merge branch '21.3' of ssh://gitlab.cern.ch:7999/atlas/athena into muHelperHashes
a6ae72d5 check for invalid identifiers in MdtCalibDbCoolStrTool

Nils Erik Krumnack authored 5 years ago and Atlas Nightlybuild committed 5 years ago

DirectIOART: Adding TFile::Open tests probing rucio redirector

See merge request atlas/athena!23596

(cherry picked from commit a191c959)

1b28a81e Adding TFile::Open tests probing rucio redirector

Clang warnings: missing override keywords.

Ambiguity in applying operator+ to Gaudi::Property.

Use the new const interfaces of IPRD_AssociationTool to avoid
calling non-const functions through a const ToolHandle.

Fix warnings in TrigMultiTrkHypoMT.cxx

See merge request atlas/athena!23615

Add mu_mc grid test to TrigUpgradeTest

See merge request atlas/athena!23610

Apply sTGCSimHit and MMSimHit to "DetectorDescription" (ATLASSIM-3711, ATLASSIM-3755)

See merge request atlas/athena!23605

xAODTrig: Enable thread-safety checker and related fixes

See merge request atlas/athena!23592

ISF Syntax Tidying

See merge request atlas/athena!23594

John Derek Chapman authored 5 years ago and Adam Edward Barton committed 5 years ago

Now `setupEventST` just calls `setupEvent` and `releaseEventST` just
calls `releaseEvent` in `BaseSimulatorTool`. This allows the syntax
of some derived classes to be simplified (`FastrasSimTool` and `TransportTool`)
and fixes a warning message about `ParticleKillerTool`.

CxxUtils+AthenaKernel+StoreGate: Allow CondCont::insert to extend the last range.

See merge request atlas/athena!23534

Scott Snyder authored 5 years ago and Adam Edward Barton committed 5 years ago

Add an option to ConcurrentRangeMap::emplace to allow extending the
range of the last entry in the case of a duplicate.  We used to do this
from WriteCondHandle, but that was racy; do it here so we can do both
steps without releasing the lock.

Cleanup of LArCafJobs/LArNoiseBursts

See merge request atlas/athena!23599

StoreGate: Optimize ReadHandle SG lookup.

See merge request atlas/athena!23603

LArTools: Enable thread-safety checking for this package.

See merge request atlas/athena!23604

IOVSvc+AthenaKernel: Fix ubsan warning; general cleanup.

See merge request atlas/athena!23602

Scott Snyder authored 5 years ago and Adam Edward Barton committed 5 years ago

Remove releaseTool calls from finalize().
They're not really needed, and it's possible for the tools to be
deleted before this is called, which results in warnings
from the ub sanitizer.

Also do some general cleanup.

Remove std::move (SiTrackMaker_xk.cxx)

See merge request atlas/athena!23593