Fix method names in CombinedP4FromRecoTausClass

@tbold

mentioned in merge request !31103 (merged)

This merge request affects 1 package:

• Reconstruction/tauRecTools

Adding @goetz ,@martindl ,@adbailey ,@xiaozhon as watchers

added Reconstruction Tau master review-pending-level-1 labels

marked as a Work In Progress

CI Result FAILURE (hash de5e124e)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 2
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10872]

There are compilation errors. Please have a look:

/var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/PhysicsAnalysis/TauID/TauAnalysisTools/Root/CommonSmearingTool.cxx:215:90: error: 'class CombinedP4FromRecoTaus' has no member named 'GetUseCaloPtFlag'; did you mean 'getUseCaloPtFlag'? bool bVeto = dynamic_cast<CombinedP4FromRecoTaus*>(m_tCombinedP4FromRecoTaus.get())->GetUseCaloPtFlag(&xTau);

Removing review label as this MR is a WIP.

Xiaozhong (L1)

removed review-pending-level-1 label

added 1 commit

• 520d5fda - Make method getCaloResolution public in CombinedP4FromRecoTaus

Compare with previous version

unmarked as a Work In Progress

This merge request affects 1 package:

• Reconstruction/tauRecTools

Adding @goetz ,@martindl ,@adbailey ,@xiaozhon as watchers

added review-pending-level-1 label

CI Result FAILURE (hash 520d5fda)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 5
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11026]

added 1 commit

• 7e1e18a1 - Fix method name

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/TauID/TauAnalysisTools
• Reconstruction/tauRecTools

Adding @goetz ,@martindl ,@adbailey ,@xiaozhon as watchers

added Analysis label

CI Result SUCCESS (hash 7e1e18a1)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 3
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11047]

The changes are clear and these warnings are known (and are since fixed), so this can go ahead.

MLB (L1)

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

added review-user-action-required label

Developed merge conflicts.