Resolve ATLTAU-1769

ATLTAU-1769

Fixes issue accessing event containers used by tau truth matching.

Calling TauTruthMatchingTool::getTruth(xtau,truthTausEvents) will store pointers to containers in TruthTausEvent enum which is passed as parameter. Truth matching previously used member TruthTausEvent enum in truth matching which would have been empty.

added bugfix label

marked this merge request as ready

marked this merge request as draft

This merge request affects no known files AND packages. Consult an expert!

This merge request affects 0 files:

added master no-domain review-pending-level-1 labels

added 1 commit

• fe78b864 - Fix reading of tau truth matching containers (TruthTausEvent) for AOD->DAOD_PHYS

Compare with previous version

CI Result SUCCESS (hash 2e44a455)

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

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

This MR is still marked as Draft. Can it be merged?

added review-user-action-required label

marked this merge request as ready

@tsulaia removing Draft after completing some additional tests locally. Ready to be merged.

removed review-user-action-required label

This merge request affects 1 package:

• PhysicsAnalysis/TauID/TauAnalysisTools

This merge request affects 2 files:

• PhysicsAnalysis/TauID/TauAnalysisTools/Root/TauTruthMatchingTool.cxx
• PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/TauTruthMatchingTool.h

added Analysis Tau analysis-review-required review-pending-level-1 labels and removed review-approved label

CI Result FAILURE (hash fe78b864)

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

Jenkins please retry a build

This merge request affects 1 package:

• PhysicsAnalysis/TauID/TauAnalysisTools

This merge request affects 2 files:

• PhysicsAnalysis/TauID/TauAnalysisTools/Root/TauTruthMatchingTool.cxx
• PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/TauTruthMatchingTool.h

CI Result SUCCESS (hash fe78b864)

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

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

merged

mentioned in commit 0e00d514

added sweep:ignore label