Carlo Varni authored 2 years ago and Vakhtang Tsulaia committed 2 years ago

Simplify Acts Configuration using enum type for reco sequence

Scott Snyder authored 2 years ago and Vakhtang Tsulaia committed 2 years ago

TestTools: Remove #undef NDEBUG.

Remove #undef NDEBUG from FLOATAssert.h.
Tests that want to force assertions should do the #undef explicitly
in the test source.
Fixes an odr-violation warning in JetTagTools in the lto build.

Add test for Read/WriteDecorHandleKey.

Christos Anastopoulos authored 2 years ago and Frank Winklmeier committed 2 years ago

CompetingRIO keep vector rather than ptr to vector

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

TestTools: Add TestAlloc.

Helper to use for testing handling of non-default allocators.

Remove --ignoreMuons --ignoreMET from comparexAODDigest.py, increase diff-root to 10 events, update --ignore-leaves parameter list of diff-root

Christos Anastopoulos authored 2 years ago and Tadej Novak committed 2 years ago

TSOS: keep FitQualityOnsurface as value. Re-organise constructors

EventInfoMgt: thread-checker cleanup

Remove redundant thread-checker annotations (the whole package is being
checked now). Also cleanup link dependencies.

Johannes Elmsheuser authored 2 years ago and Walter Lampl committed 2 years ago

Add RecoRun3Data_CAConfig and  RecoRun3Data_LegacyVsCA test, increase the CA reco tests to 25 events, add a diff-root test in the CA vs. Legacy configuration comparison

Remove (unused) dependency on the old EventInfo package.

enable FPE stack trace dump in one of the CI reco tests (ATR-26568)

Scott Snyder authored 2 years ago and Tadej Novak committed 2 years ago

AthenaPoolMultiTest: Fix cppcheck warning.

Redundant null check.

The addition of CutFlowSvc breaks `RecoRun4MC_DAODPHYS`. Temporarily
disable the test.

Marcin Nowak authored 2 years ago and Julien Maurer committed 2 years ago

Remove IOV Callback from TagInfoMgr

Removed the use of the IOVSvc callback as it is not thread-safe.
Now relying solely on incidents (begin run/file)
Also removed DataHandles use to not trigger gccchecker thread safety
errors (see ATEAM-855)

John Derek Chapman authored 2 years ago and Julien Maurer committed 2 years ago

TileByteStream: Fix test configuration.

Update for changes in tile configuration: need to disable Geant4 version check.
Update reference files.
Fixes test failure in master.

Julien Maurer authored 2 years ago and Edward Moyse committed 2 years ago

upgrade conditions to BLKPA-2022-07 for Run3 cosmics/calib CI tests

Julien Maurer authored 2 years ago and Stefan Richter committed 2 years ago

upgrade conditions to BLKPA-2022-07 for Run3 cosmics/calib CI tests

Fix another ordering issue in the unit tests. Make sure to use the Pool
catalog written by the dependee.

Closes ATEAM-859.

Adam Edward Barton authored 2 years ago and Atlas Nightlybuild committed 2 years ago

AthenaDBTestRec+AthenaPoolMultiTest: enable thread-checker

See merge request atlas/athena!56540

(cherry picked from commit ab86a0af)

3c9ab245 AthenaPoolMultiTest: enable thread-checker
1c65dccd AthenaDBTestRec: enable thread-checker
dace49e9 AthenaPoolMultiTest: use dedicated POOL catalogue for EventNoSplit test

Use a private working directory for the Overlay tests to avoid
collisions in `PoolCatalogue.xml` with the other tests. Also use private
catalogues for the `DoubleSelector` test.

Avoid chances of POOL catalogue collisions by using a dedicated
PoolCatalogue in the `EventNoSplit` test.

Mostly suppress checker warnings by marking testing classes as not
thread-safe.

Stefan Richter authored 2 years ago and Julien Maurer committed 2 years ago

tauRecTools: drop conditional decoration in TauIDVarCalculator

Hello,

This is resubmission of the broken !56154 .

We have a conditional decoration in tauRecTools, i.e. a decoration that is only added onto certain taus (those with at least 1 track), which causes trouble downstream when making derivations.
This decoration corresponds to the eProbabilityHT of the leading track of the tau.
It was created in R21 for convenience, but it is no longer used anywhere in R22 (and it can anyway be recomputed from the AOD).
As agreed at the last RIG meeting, this decoration is now dropped.

The ESD and AOD reference files for q442, q445 and q449 have been updated and deployed on EOS.
The AOD_content.ref files have been updated as well.

Cheers,
Bertrand

`atlas_add_citest` does not parse `DEPENDS_SUCCESS` correctly if another
argument follows it. All these arguments will be ignored.
Documentation on this has been updated as part of !56445.