Small-R truth jets in DAOD_PHYS

This MR implements the following

(1) Changes in MCTruthCommon allow to add small-R jets in DAOD_PHYS. This was implemented by switching back to the old way of setting up things. The current implementation causes problems when scheduling jet reconstruction in different places. In the future we will switch to the new configuration hopefully everywhere

(2) Fix the MET_track container name in fJVT calculation

(3) Add calculation of event level cleaning variables

Item (1) makes @zmarshal happy hopefully :) Item (2) and (3) were reported in ATLPHYSVAL-671

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMCTruth
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys
• Reconstruction/Jet/JetMomentTools

Affected files list will not be printed in this case

Adding @zmarshal ,@goetz ,@jmellent ,@jveatch ,@nelsonc ,@dshope ,@ispokhar as watchers

added Derivation JetEtmiss Reconstruction master review-pending-level-1 labels

CI Result SUCCESS (hash 80536a06)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23877]

Thanks @cdelitzs , this is great!

Just to check, does this also restore the flavor tagging variables?

added 1 commit

• a849104f - Avoid duplication of code

Compare with previous version

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkJetEtMiss
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkMCTruth
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys
• Reconstruction/Jet/JetMomentTools

Affected files list will not be printed in this case

Adding @zmarshal ,@goetz ,@jmellent ,@jveatch ,@nelsonc ,@dshope ,@ispokhar as watchers

Hi @zmarshal,

yes, this should include also variables like GhostBHadrons, PartonTruthLabelID.

Cheers, Chris

resolved all threads

CI Result SUCCESS (hash a849104f)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23887]

resolved all threads

No outstanding issue to L1, escalating to L2 for another look into MCTruthCommon.py. Thanks, Pienpen (L1)

added review-pending-level-2 label and removed review-pending-level-1 label

added review-approved label and removed review-pending-level-2 label

Looks good to me. Karolos (L2)

merged

mentioned in commit 43120263

added sweep:ignore label