ID AlignMon GenericTracks

new variable for d0bs binning can be controlled from jobOptions and some histo range changes

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

120K in file InnerDetector/InDetMonitoring/InDetAlignmentMonitoring/src/IDAlignMonGenericTracks.cxx

This merge request affects 2 packages:

• InnerDetector/InDetMonitoring/InDetAlignmentMonitoring
• InnerDetector/InDetMonitoring/InDetPerformanceMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@jajimene ,@martis ,@sroe as watchers

added DQ InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash 29d5d88f)

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

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

Hi @martis

I see this code uses thread-hostile LW-Histograms. Looks like a deprecated, legacy monitoring tool. Why do you want to update it?

• Walter

Dear @wlampl

thanks for your mail. We want to update it because we had a fixed d0 binning that for large stats is ok, but for short tests it is not

I didn't know the LW-Histograms were not recommended.

One thing I can do in the near future (1-2 week) is to replace them and use TH1 instead.

Cheers Salva

mentioned in commit 661c83ac

merged

added sweep:ignore label

mentioned in merge request !43507 (merged)