IsolationSelection: Move setting of isLRT decoration to merge algorithms.

Instead, rely on it being set by the LRT merge algorithms. This avoids issues with multiple algorithms trying to set the same decoration. Also, when this algorithm was being run on the output of the LRT merge algorithms, the decoration was being set incorrectly.

For the LRT merge algorithms, use decoration handles so that the decorations get locked.

added full-integration-tests label

cc @sagar @schaffer

changed title from IsolationSelection: Remove setting of isLRT decoration. to IsolationSelection: Move setting of isLRT decoration to merge algorithms.

Also to be requested for r24 once it seems ok. (Conflicts, so can't sweep.)

This merge request affects 3 packages:

• PhysicsAnalysis/Algorithms/EgammaAnalysisAlgorithms
• PhysicsAnalysis/Algorithms/MuonAnalysisAlgorithms
• PhysicsAnalysis/AnalysisCommon/IsolationSelection

Affected files list will not be printed in this case

Adding @sfrances ,@tadej ,@falonso ,@oducu ,@sangelid ,@lucam ,@nicolaid ,@apsallid ,@krumnack as watchers

added Analysis CPAlgorithms Egamma analysis-review-required main review-pending-level-1 labels

CI Result SUCCESS (hash f188790a)

	Athena	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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

Hi, since it touches the algorithm, ask expert for help. @krumnack would you please take a look? Many thanks. Best regards

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

added 1 commit

• 2624623f - Make decorator objects static const

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/Algorithms/EgammaAnalysisAlgorithms
• PhysicsAnalysis/Algorithms/MuonAnalysisAlgorithms
• PhysicsAnalysis/AnalysisCommon/IsolationSelection

Affected files list will not be printed in this case

Adding @falonso ,@sangelid ,@krumnack ,@sfrances ,@tadej ,@lucam ,@nicolaid ,@oducu ,@apsallid as watchers

CI Result SUCCESS (hash 2624623f)

	Athena	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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

resolved all threads

Changes are fine and CI succeeds, approving for L1. -L1

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

Purely technical update, approving for AR.

Nils [Analysis RC]

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

merged

mentioned in commit 71de1769