IDTPM: Adding configurable unlinkedAsFakes option

Adding unlinkedAsFakes trackAnalysis flag to not produce unlinked rate plots, and instead consider truth-unlinked tracks as fakes. The default is true. By setting it to false unlinked and fake rates plots are produced for continuity with the current implementation. This setup is used in the ART test that performs the comparison with IDPVM.

CC @tstreble @ncalace @mvessell @rgarg @piazza

changed the description

added 1 commit

• c545b736 - unlinkedAsFakes = True by default

Compare with previous version

changed the description

added 63 commits

• c545b736...18676746 - 61 commits from branch atlas:main
• 41284277 - Adding configurable unlinkedAsFakes option
• 855d07b7 - unlinkedAsFakes = True by default

Compare with previous version

added 1 commit

• eebdade4 - Adding comment offlTrkAnaConfig.json

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 1 package:

• InnerDetector/InDetValidation/InDetTrackPerfMon

This merge request affects 8 files:

• InnerDetector/InDetValidation/InDetTrackPerfMon/InDetTrackPerfMon/ITrackAnalysisDefinitionSvc.h
• InnerDetector/InDetValidation/InDetTrackPerfMon/data/offlTrkAnaConfig.json
• InnerDetector/InDetValidation/InDetTrackPerfMon/python/InDetTrackPerfMonConfig.py
• InnerDetector/InDetValidation/InDetTrackPerfMon/python/InDetTrackPerfMonFlags.py
• InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.cxx
• InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.h
• InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisDefinitionSvc.h
• InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx

Adding @sroe ,@maparo ,@goetz ,@cvarni as watchers

added EFTracking InnerDetector Tracking TriggerID main review-pending-level-1 labels

CI Result SUCCESS (hash eebdade4)

	Athena
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
For experts only: Jenkins output (remote access info)

Looks fine to me. However, a lot of if-statments and new returns have been added. L2 should have a quick look to ensure I didn't miss anything.

Malte (L1)

removed review-pending-level-1 label

added review-pending-level-2 label

lgtm

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

merged

mentioned in commit 0f62f625