MuonSelectorToolsTester updates

This MR contains a few updates to the MuonSelectorToolsTester:

• Add Low-pT working point with MVA option as an additional working point
• Add author information both in the text output for each muon and in a new summary table
• Use the truthType decoration on the muon instead of running MCTruthClassifier, since MCTruthClassifier cannot work with the reduced truth content in DAOD_PHYS
• Add a unit test to process the ASG_TEST_FILE_MC (we ran only data before)

added Test alsoTargeting:21.2 master labels

This merge request affects 1 package:

• PhysicsAnalysis/MuonID/MuonSelectorTools

This merge request affects 3 files:

• PhysicsAnalysis/MuonID/MuonSelectorTools/CMakeLists.txt
• PhysicsAnalysis/MuonID/MuonSelectorTools/test/ut_MuonSelectorToolsTester_MC.cxx
• PhysicsAnalysis/MuonID/MuonSelectorTools/util/MuonSelectorToolsTester.cxx

Adding @jojungge as watcher

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

CI Result SUCCESS (hash 77e62216)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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 37619]

CI passed. Looks good. Approved

Sergei L1

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

Hi @mkbugge,

It's your choice, but wouldn't it be better to do this as a simple shell script instead of a compiled executable? You can add shell scripts directly as tests in cmake.

Cheers, Nils

added analysis-review-approved review-user-action-required labels and removed analysis-review-required review-approved labels

added 1 commit

• 136a952d - Move to shell scripts for unit tests

Compare with previous version

This merge request affects 1 package:

• PhysicsAnalysis/MuonID/MuonSelectorTools

This merge request affects 5 files:

• PhysicsAnalysis/MuonID/MuonSelectorTools/CMakeLists.txt
• PhysicsAnalysis/MuonID/MuonSelectorTools/test/ut_MuonSelectorToolsTester_MC.sh
• PhysicsAnalysis/MuonID/MuonSelectorTools/test/ut_MuonSelectorToolsTester_data.cxx
• PhysicsAnalysis/MuonID/MuonSelectorTools/test/ut_MuonSelectorToolsTester_data.sh
• PhysicsAnalysis/MuonID/MuonSelectorTools/util/MuonSelectorToolsTester.cxx

Adding @jojungge as watcher

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

resolved all threads

CI Result SUCCESS (hash 136a952d)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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 37673]

This had already been approved. Changing back the labels. Cheers Lino (L1)

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

removed analysis-review-required label

mentioned in commit fc446298

merged

added sweep:done label

added sweep:failed label