Add tests for the flavor tagging EDM

This moves some tests for the flavor tagging EDM into FlavourTaggingTests to consolidate them.

Also adds a new test focused on checking the links to associated particles.

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/FlavourTaggingTests
• Projects/AnalysisBase
• Projects/AthAnalysis

Affected files list will not be printed in this case

Adding @nishu ,@rcreager ,@duperrin ,@krumnack ,@sanmay ,@akraszna ,@bdong ,@rbianchi as watchers

added Analysis BTagging Build Derivation JetEtmiss analysis-review-required master review-pending-level-1 labels

CI Result SUCCESS (hash db216163)

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

marking as urgent because people need this to understand some bugs in the HLT. I'd prefer to get it in before we branch 22.0.

added urgent label

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

Changes look fine from L1 point of view. Only was a little confused by the messaging. Passing on to L2 in the meantime to cross check that the CMakeLists are correctly set up.

Kira (L1)

Hi,

looks good apart from the unclear reason why std::cout is being used instead of the messaging service, and also why there is a char string rather than std::string. Once this is clear it will be good to go.

James (L2)

added 1 commit

• 65297e0d - use std::string rather than char, more consistent use of output streams

Compare with previous version

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/FlavourTaggingTests
• Projects/AnalysisBase
• Projects/AthAnalysis

Affected files list will not be printed in this case

Adding @nishu ,@rcreager ,@duperrin ,@krumnack ,@sanmay ,@akraszna ,@bdong ,@rbianchi as watchers

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

added 1 commit

• 92f73b63 - Add not thread safe thing

Compare with previous version

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

There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

This merge request affects 4 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/FlavourTaggingTests
• Projects/AnalysisBase
• Projects/AthAnalysis

Affected files list will not be printed in this case

Adding @nishu ,@rcreager ,@duperrin ,@krumnack ,@sanmay ,@akraszna ,@bdong ,@rbianchi as watchers

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

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

CI Result SUCCESS (hash 65297e0d)

	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 1
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 52246]