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

Add support for --debug switch.

Hello,

This MR is renaming a few track variables in TauTrackRNNClassifier, to make it a bit easier to read.
It does not change the reconstruction output.

Cheers,
Bertrand

Christos Anastopoulos authored 4 years ago and Adam Edward Barton committed 4 years ago

ATLASRECTS-6099: Change unneded decorators to Accessors. Should avoid future confusion. Also use virtual, override, final. Additionally, use nullptr, use empty,  qualify auto

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

Remove references to obsolete CaloNoiseTool and friends.
Either migrate to CaloNoise or remove entirely.

Mark Hodgkinson authored 4 years ago and Adam Edward Barton committed 4 years ago

Change number of threads from 5 to 8 to reflect what we will use in production system.

Trigger D3PDs are no longer produced/needed. Delete the
`WriteTriggerD3PD` flag and associated production job options. Note that
trigger D3PD information may still be used in other DPDs.

Bertrand Martin Dit Latour authored 4 years ago and Adam Edward Barton committed 4 years ago

Hello,

As discussed in ATLASRECTS-6106 and ATLASRECTS-6104, we make an unnecessary and inappropriate use of Decorator on non-const objects, and we should have used Accessor instead.
Now fixed in TauTrackFinder.
Note for later: this could be a bit more streamlined by using the TauTrack::setDetail functionality of the EDM, extending the TrackDetail enum in TauDefs.h.

This MR does not change the reconstruction output.

Cheers,
Bertrand

Christos Anastopoulos authored 4 years ago and Adam Edward Barton committed 4 years ago

ATLASRECTS-6104 . Replace un-needed Decorators with Accessors. As input is const. Also qualify auto, use empty, use nullptr

Simplified the logic in the CMakeLists.txt file, and split the
tool dictionaries into a new library (tauRecToolsDict) instead
of having them in tauRecToolsLib.

Fixed a small issue in tauRecTools/TrackSort.h, which would've
made it unusable in AthAnalysis. (Currently that header is not
included in an AthAnalysis build.)

The main motivation was to make sure that the public headers of the
package would be properly installed in "Athena mode", in order for
the dictionary to work correctly.

But the cleanup grew into something more. I replaced the CINT
dictionary of the package into a Reflex one, introduced the
PanTauAlgsLib library that would be built in all circumstances,
cleaned up the dependencies of the package, and made sure that
the dictionary would refer to types with their correct names.

This is primarily meant to fix the issue described in ATEAM-721.

Loss of precision in int->float conversion.

Christos Anastopoulos authored 4 years ago and Edward Moyse committed 4 years ago

ATLASRECTS-6103 :CaloIsolation, remove unused (option seems to be always false), non-thread safe code

Kerstin Tackmann authored 4 years ago and Edward Moyse committed 4 years ago

Add histograms for more detailed checks of conversion reconstruction: vs pileup, fake conversions, track precision hit fraction and TRT PID.

Make sure all members of LayerConfig are initialized.

Protect against out-of-bounds vector access.

Mark Hodgkinson authored 4 years ago and Edward Moyse committed 4 years ago

Also remove hard coding of conditons tag, which is no longer needed.

This is to fix issues when new configuration did not match old.
Also fix logger of MuonConfig so it can turn CA debugging on

Hello,

This MR is changing the seed jet pt cut from 10 GeV to 0 GeV in the tau reconstruction.
It does not change the reconstruction output, because this cut is set upstream to 15 GeV in the jet reconstruction: https://gitlab.cern.ch/atlas/athena/-/blob/master/Reconstruction/Jet/JetRec/share/JetRec_jobOptions.py#L33.
However, for the R22 reprocessing, we will have to drop this 15 GeV cut from the AntiKt4LCTopo jet reconstruction (by setting jetFlags.useCalibJetThreshold to False), and instead apply a tau pt cut at the end of the tau reconstruction, in the tau thinning algorithm.

Cheers,
Bertrand

Try to fix ATLASRECTS-6027 , Also try to format MuonCreatorTool, run clang-tidy for nullptr, qualified auto etc

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

Avoid spurious copies in range-for loops.

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

Avoid spurious copy in range-for.