Enable thread-safety checking and fix warnings.

Unused variable.

Missing override keywords.

TBB is not a ROOT component.

Pass class instances by const reference, not by value.

Redundant / misplaced null pointer tests.

Add TRT and Pixel PID information directly to track particles rather
than first adding the information to the track summary, For statistically
combined muon track particles compute pixel PID, but only copy TRT PID
information over from the input track particles (original behaviour).

For the code path not starting from tracks only the default Pixel and
TRT PID values are set (tracks are needed to run the PID).

Patrick Scholer authored 2 years ago and Vakhtang Tsulaia committed 2 years ago

Temporarily remove single strip charge cut in MM reco

Patrick Scholer authored 2 years ago and Vakhtang Tsulaia committed 2 years ago

Update MM Overlay Time WIndow

The `Station` class was printing a lot of unprotected VERBOSE messages.
Also there are a lot of local `MsgStream` instances created. Instead
derive from `AthMessaging` and use the `ATH_MSG` macros for logging.

Relates to ATR-21692.

Constantin Heidegger authored 2 years ago and Frank Winklmeier committed 2 years ago

passivation test algo

Conflicts:
	MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondTest/src/MuonCondTest_entries.cxx

sTGC strips with small charge are kept due to the near-neighbour logic.
With pileup, hits near the lower or upper time limit would created a
cluster of strips. However, not all these strips pass the selection
due to time smearing. Sometimes, only the strips with small charge
pass the selection, leading to a cluster without charge that breaks
the clusterizing method.

Patrick Scholer authored 2 years ago and Frank Winklmeier committed 2 years ago

fix  peak in MM overlay time spectrum

Alexandre Laurier authored 2 years ago and Frank Winklmeier committed 2 years ago

Fix MM digi identifier warnings (ATLASSIM-5822)

Johannes Junggeburth authored 2 years ago and Frank Winklmeier committed 2 years ago

MuonMDT cabling - Do not kill the inner stations if the asymmetric layout is loaded

Alexandre Laurier authored 2 years ago and Frank Winklmeier committed 2 years ago

NSW RDO data time window for pileup

Scott Snyder authored 2 years ago and Frank Winklmeier committed 2 years ago

RPC_CondCabling: Fix clang warning.

RDOindex::operator= cannot be defaulted due to the presence of a 
reference member.

Scott Snyder authored 2 years ago and Frank Winklmeier committed 2 years ago

MuonMDT_Cabling: Increase test timeout.

Test has been timing out in dbg builds.

Use AthMessaging instead of private MsgStream member.

Johannes Junggeburth authored 2 years ago and Frank Winklmeier committed 2 years ago

MuonPRTTest - Add mechanism to save Trees in subdirectories & minor bug-fix