AnalysisTop: R22 Migration. Electron trigger efficiency SF config

AnalysisTop R22 migration. Electron trigger efficiency SF config ANALYSISTO-1228

• Removal of electronForceTrigger option
• Implementation of ElectronTriggerEfficiencyConfig option
• Empty default for trigger_string in EgammaCPTools
• User required to configure new option in config otherwise code throws an error

added AnalysisTop label

added 1 commit

• d546cf40 - Add new ElectronTriggerEfficiencyConfig to validation-cuts.txt for CI and to example config

Compare with previous version

marked this merge request as ready

WARNING: big files (>100K) are found in the changeset

124K in file PhysicsAnalysis/TopPhys/xAOD/TopConfiguration/TopConfiguration/TopConfig.h

176K in file PhysicsAnalysis/TopPhys/xAOD/TopConfiguration/Root/TopConfig.cxx

This merge request affects 3 packages:

• PhysicsAnalysis/TopPhys/xAOD/TopAnalysis
• PhysicsAnalysis/TopPhys/xAOD/TopCPTools
• PhysicsAnalysis/TopPhys/xAOD/TopConfiguration

Affected files list will not be printed in this case

Adding @omajersk ,@cabrera ,@ravinab ,@thsteven ,@tdado as watchers

added Analysis Egamma analysis-review-required master review-pending-level-1 labels

CI Result SUCCESS (hash d546cf40)

	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 54643]

lgtm approving -- L1

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

merged

mentioned in commit eca69381

added sweep:ignore label