Fix Expert Monitoring Histograms in InDetTrigMT_xAODParticleCreatorAlg_minBias_IDTrig

Fixed binning for SCT, TRT, and Pixel hits histograms. Edited unit and binning for track Pt. Added Pt times charge histogram with range -/+1 TeV. Tagging: @tbold @sutt

This merge request affects 1 package:

• Trigger/TrigTools/TrigInDetMonitoringTools

This merge request affects 2 files:

• Trigger/TrigTools/TrigInDetMonitoringTools/python/TrigInDetTrackingMonitoring.py
• Trigger/TrigTools/TrigInDetMonitoringTools/src/TrigInDetTrackMonitoringTool.cxx

Adding @sutt ,@amorley ,@nagano as watchers

added Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash 4fdc55bc)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 26573]

Would it be possible to add histogram with the number of tracks per event. You'll need:

trkCount  = Monitored::Scalar("trkCount", tracks.size());

And the histogram should likely be binned like the counterpart in FTF.

Can we change the title of this MR to something more descriptive?

In the mean time, seems like there are still discussions ongoing, so removing the L1 label. Since it seems like there may continue to be discussion, please consider marking this as a draft until everyone is happy.

Thanks!

-Mike (L1)

added review-user-action-required label and removed review-pending-level-1 label

changed title from Fix Histograms to Fix Expert Monitoring Histograms in InDetTrigMT_xAODParticleCreatorAlg_minBias_IDTrig

marked this merge request as draft

added 1 commit

• 6a09dac0 - 1st revision

Compare with previous version

added 1 commit

• dad0a611 - comments..

Compare with previous version

resolved all threads

added 1 commit

• 1bb74b5e - 3rd revision

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 1 package:

• Trigger/TrigTools/TrigInDetMonitoringTools

This merge request affects 2 files:

• Trigger/TrigTools/TrigInDetMonitoringTools/python/TrigInDetTrackingMonitoring.py
• Trigger/TrigTools/TrigInDetMonitoringTools/src/TrigInDetTrackMonitoringTool.cxx

Adding @sutt ,@amorley ,@nagano as watchers

added review-pending-level-1 label and removed review-user-action-required label