Monitoring: fixing copying of data to shifted live histograms including TProfile

• refilling of new kLive histograms with offset (on Xaxis = Lumiblocks ) w.r.t. old ones
• special treatment of the TProfile histograms for this

added DQ InnerDetector master labels

marked this merge request as ready

changed title from WIP: Monitoring: fixing copy of data to shifted live histograms including tprofile to Monitoring: fixing copying of data to shifted live histograms including TProfile

This merge request affects 1 package:

• Control/AthenaMonitoringKernel

This merge request affects 1 file:

• Control/AthenaMonitoringKernel/src/HistogramFiller/LiveHistogramProvider.h

Adding @ssnyder as watcher

added Core review-pending-level-1 labels

CI Result SUCCESS (hash e4370f92)

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

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

merged

mentioned in commit 96befad8

Looking at the code-diff I was wondering about the special treatment of TProfile. Can't that be implement as template specialization of the copyDataToNewHistogram method? The if-check inside the loop over bins looks awkward and potentially slow.

OK @wlampl, the point taken, I'll look into it, thank you for the suggestion!

added sweep:ignore label

mentioned in merge request !47292 (merged)