allow using TEfficiency with book()/hist() mechanism

While it was seemingly supported so far, it didn't actually work. This should now be fixed, and a test for it added.

I also went ahead and made the basic function a template, so that users can use it with any histogram-like class deriving from TObject. Though that part will only work in AnalysisBase, as I only modified the AnalysisBase part.

added alsoTargeting:21.2 bugfix labels

This merge request affects 3 packages:

• PhysicsAnalysis/D3PDTools/AnaAlgorithm
• PhysicsAnalysis/D3PDTools/EventLoop
• PhysicsAnalysis/D3PDTools/EventLoopTest

Affected files list will not be printed in this case

Adding @krumnack as watcher

added Analysis analysis-review-required 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 5ca1a989)

	Athena	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 1
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 67953]

added 1 commit

• 402363af - allow using TEfficiency with book()/hist() mechanism

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/D3PDTools/AnaAlgorithm
• PhysicsAnalysis/D3PDTools/EventLoop
• PhysicsAnalysis/D3PDTools/EventLoopTest

Affected files list will not be printed in this case

Adding @krumnack as watcher

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 402363af)

	Athena	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 67965]

Looks good. Karolos (L2)

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

added review-approved-point1 label

added review-approved-tier0 label

added analysis-review-approved label and removed analysis-review-required label

mentioned in commit f35c3e41

merged

added sweep:done label

added sweep:failed label

Sweep summary
failed:

• 21.2

removed sweep:failed label

removing sweep:failed due to !65948 (merged).