Skip to content

[HLT] Move online error monitoring from event loop manager to a dedicated tool

Rafal Bielski requested to merge rbielski/athena:move-hlt-error-hist into master

Follow-up from a discussion in !36901 (comment 3830204)

Move the method algExecErrors from HltEventLoopMgr to a new dedicated tool in the TrigSteerMonitor. The new tool, TrigErrorMonTool will be expanded with a new histogram in !36901 (merged). The move is needed to minimise library dependencies in TrigServices.

Moving this to TrigSteerMonitor has the following advantages:

  • the tool creates framework monitoring histograms, so functionally this is the right package for it
  • there is already a header-only library TrigSteerMonitorLib which TrigServices already depend on (for SchedulerMonSvc1)
  • the TrigSteerMonitor component library already depends on everything that will be needed for the error histograms and contains other monitoring components having similar functions (e.g. the TrigSignatureMoniMT)
  • this is exactly where the same functionality for Run-2 was implemented - the old tool is still there, TrigErrorMon (similar name, the old code to be removed before Run 3)

cc @aporeba, @fwinkl, @tamartin, @wiedenma, @mark

1 SchedulerMonSvc may get moved somewhere else at some point, but we'll see about that. Still, this header-only library seems like a good solution to let HltEventLoopMgr use the external tool with minimal dependencies.

Merge request reports