Prefer an initializer list to setting members in a ctor body.

Reading the HLT trigger configuration is no longer supported in Run-3.
Delete the `readHLTconfigFromXML` flag and references to it.

Suppress a cppcheck warning.
Fix redundant map lookups.

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

Migrate uses of LArADC2MeVTool to use conditions data.
Remove unused component LArSuperCellBuilderDriver.

Peter Loch authored 3 years ago and Adam Edward Barton committed 3 years ago

A number of cells/sampling moment is added to the cluster moments. This moment is not
a scalar (float) but a vector of counters represented by xAOD::CaloCluster::counter_t
for compact storage.

Edward Moyse authored 3 years ago and Adam Edward Barton committed 3 years ago

For tracks, prefer parameters over measurements

Scott Snyder authored 4 years ago and Edward Moyse committed 4 years ago

Remove uses of LArOFCTool.
LArRawChannelBuilderToolOFC is completely unused --- remove it.
Update LArRawChannelBuilder to use LArOFCCondAlg instead.

Scott Snyder authored 4 years ago and Edward Moyse committed 4 years ago

Split off ConstAuxElement from AuxElement, making it a base.
Refactoring to allow iterating using AuxElement instances directly
(without going though the pointers).

Frank Winklmeier authored 4 years ago and Frank Winklmeier committed 4 years ago

Remove printing of obsolete `PACKAGE_VERSION` and some minor code
cleanups. Update link dependencies.

Linking against `SGTests` is only needed if the `StoreGate/SGtests.h` is
used.

Relates to the duplicate CLID errors discussed in ATEAM-731.

John Derek Chapman authored 4 years ago and Edward Moyse committed 4 years ago

Part of the changes for ATLASSIM-3837.

    
Don't use references in a range-for if the object must be copied.

Remove printout of obsolete `PACKAGE_VERSION`. For tools, remove the
entire `initialize` method in case it didn't do anything else. Also add
`override` keyword where needed.

That's what the new class is meant for, so I went ahead and swapped it
in in all places that seem to be using `AnaToolHandle` in such a manner.

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

Avoid/remove covariance matrix allocations from clients of TrackParameters. use std::optional intrernally to express that is "nullable" but allow for value semantics

As it turns out AthAnalysis also provides AtlasSTLAddReflex, which comes
with that dictionary itself.

Scott Snyder authored 4 years ago and Johannes Elmsheuser committed 4 years ago

Remove remaining pieces of CaloTowerContainer dealing with CaloNoiseTool.

Missing close tag.

Judith Hofer authored 4 years ago and Walter Lampl committed 4 years ago

- Add a new method in the PhysValUtils that can read the json file.
- Add a member to the PhysvalBTag class holding the information.
- Missing:
. registering the json file as data
. propagating the information to the initializeHistograms function of the BTaggingValidationPlots class

Handle sTgcClusterOnTrack and MMClusterOnTrack.