Use JVT selection in Jets.FlavourTaggingEventSF block

Self explanatory

added CPAlgorithms label

mentioned in merge request !69784 (merged)

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/FullCPAlgorithmsTest.py

Adding @tadej ,@krumnack as watchers

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

Shouldn't we also update the YAML config that is used in comparison CI tests ?

added 1 commit

• 892349e0 - Update example_config.yaml

Compare with previous version

resolved all threads

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig

This merge request affects 2 files:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/data/example_config.yaml
• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/FullCPAlgorithmsTest.py

Adding @krumnack ,@tadej as watchers

CI Result FAILURE (hash 88c3ea97)

	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 6675] (remote access info)

@ravinab looks like it might not be as simple as updating the containerName to AnaJets.baselineJvt

CI Result FAILURE (hash 892349e0)

	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 6677] (remote access info)

MR given back to developer due to unresolved thread and many CI test failures. Cheers, L1

removed review-pending-level-1 label

added review-user-action-required label

added 1 commit

• 1fc5492b - Update to readNameAndSelection

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig
• PhysicsAnalysis/Algorithms/FTagAnalysisAlgorithms

Affected files list will not be printed in this case

Adding @krumnack ,@bdong ,@tadej as watchers

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

CI Result SUCCESS (hash 1fc5492b)

	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 6738] (remote access info)

The changes are straightforward enough and add the JVT preselection for jets used to calculate the flag efficiency SF. So approving for analysis release purposes.

Giovanni (AR)

removed analysis-review-required label