Fix legacy code

This MR fix the name of several tau collection and the way we get the mu value, now we do it from eventInfo rather than using readhandles as this is legacy code, also it improves a bit warning/debug printouts.

Tagging @malconad @guindon

Jenkins please retry a build

WARNING: big files (>100K) are found in the changeset

184K in file Trigger/TrigMonitoring/TrigTauMonitoring/src/HLTTauMonTool.cxx

This merge request affects 1 package:

• Trigger/TrigMonitoring/TrigTauMonitoring

added DQ Tau Trigger master labels

CI Result SUCCESS (hash c408fb44)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15849]

added 1 commit

• 338d2dab - Fix online mu value and histogram range

Compare with previous version

changed the description

unmarked as a Work In Progress

Jenkins please retry a build

WARNING: big files (>100K) are found in the changeset

184K in file Trigger/TrigMonitoring/TrigTauMonitoring/src/HLTTauMonTool.cxx

This merge request affects 1 package:

• Trigger/TrigMonitoring/TrigTauMonitoring

Adding @carquin as watcher

added review-pending-level-1 label

CI Result FAILURE (hash 338d2dab)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 16350]

Error in trigger test is unrelated: 2020-06-30T232842 CEST TrigValSteering ERROR RootComp killed on timeout of 300 seconds

The changes look fine and the failing test is a known issue. Approving.

Jason (L1)

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

@carquin please update the MR description. "Fix legacy code" says nothing about what you changed. No need to rerun the CI, but I will not merge this before a proper description has been added.

changed the description

resolved all threads

merged

mentioned in commit 8da0dde0

added sweep:ignore label