Add variables for LAr timing calibration to LLP1 derivation

Add variables for LAr timing calibration to LLP1 derivation. For LLP searches, in particular the Run 3 Displaced Electrons search, the calibrated timing from the LAr calorimeter is a sensitive analysis variable. To compute this calibrated time, a small list of variables from calorimeter cells are required to be added to the LLP DAOD. From testing on an r22 slepton MC sample, the size change in the output DAOD was as follows:

308 Events, 12M output DAOD before adding variables -> 13M output after adding variables

The change on the file size from running the derivation was checked more thoroughly on AODs in a few runs from 2018 and on TTbar MC

Data18: 23486 Events 36.14 kb/Evt Data18 + calo vars: 23486 Events 37.71 kb/Evt Ttbar: 314 Events 61.10 kb/Evt Ttbar + calo vars: 314 Events 62.22 kb/Evt

The size change is minimal, as was discussed and agreed with @jburzyns

marked this merge request as ready

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkLLP

This merge request affects 1 file:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkLLP/share/LLP1.py

added Derivation master review-pending-level-1 labels

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)AF3.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash b39cb199)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 54576]

Hi @ancsmith,

The code looks okay from L1 perspective. As you have discussed the change in DAOD have been approved.

Approving.

Cheers, Dong -L1

added review-approved label

removed review-pending-level-1 label

merged

mentioned in commit ab7d7f48

added sweep:ignore label