CPAlgorithms: run FTag after JVT and OverlapRemoval are applied

Schedule the FTag sub-block to run after JVT and OR, so we get the correct per-event SFs.

This could be an alternative to !69639 (merged) @tstreble ? Or we could adopt the same setup in the new block proposed in that linked MR.

This merge request affects 4 packages:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig
• PhysicsAnalysis/Algorithms/AsgAnalysisAlgorithms
• PhysicsAnalysis/Algorithms/FTagAnalysisAlgorithms
• PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms

Affected files list will not be printed in this case

Adding @bdong ,@krumnack ,@tadej as watchers

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

CI Result SUCCESS (hash 760bdf6b)

	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 [CI-MERGE-REQUEST-EL9 6368] (remote access info)

We need to be able to run the FTAG jet selection before overlap removal, as some overlap removal schemes depend on it. We should still decouple then the FTAG jet selection and the SF computation, so !69639 (merged) should proceed still. The dependency should only be set for the SF computation.

Also it' unclear to me how this interacts with !69002 (merged)

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

Switching to review-user-action-required due to open thread and dependency on other MR

L1 shifter

added review-user-action-required label and removed review-pending-level-1 label

added 375 commits

• 760bdf6b...e4de25cb - 373 commits from branch atlas:main
• 88e1e660 - fix typos
• 7faf01f0 - support for multiple dependencies

Compare with previous version

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig

This merge request affects 3 files:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/ConfigBlock.py
• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/ConfigFactory.py
• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/ConfigSequence.py

Adding @krumnack ,@tadej as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash 7faf01f0)

	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 [CI-MERGE-REQUEST-EL9 6663] (remote access info)

I think in its essence this is fine. It needs to wait for the other MR to go in (if it hasn't already), and the thread above be resolved. Still let me approve this for AT.

Nils [Analysis RC]

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

Please resolve the last open thread

~L1

removed review-pending-level-1 label

added review-user-action-required label

resolved all threads

Changes are fine and CI succeeds, approoving for L1.

removed review-user-action-required label

added review-approved label