Skip to content
Snippets Groups Projects

Add DecReportsMonitor to HLT2

Merged Sascha Stahl requested to merge decreports_monitor into master

The MR adds the monitoring of decisions to HLT2. The monitor cannot run in the output sequence as it would not see all events and only count the ones which were accepted. Therefore, the monitor is added after the trigger lines in a new main control flow node with the type NONLAZY_OR. To not use too much resources, the monitoring is only run on 10% of the events.

The current solution adds another layer of control flow nodes, we could make it simpler by not requiring to run on every event. In this case we would need an external normalization to monitor accept fractions.

Edited by Sascha Stahl

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sascha Stahl added 2 commits

    added 2 commits

    Compare with previous version

  • Sascha Stahl added 1 commit

    added 1 commit

    Compare with previous version

  • Sascha Stahl
  • Sascha Stahl added 1 commit

    added 1 commit

    Compare with previous version

  • assigned to @acasaisv

  • Author Maintainer

    The MR is ready to test. I ran some HLT2 tests myself and expect one failing test which should only need a trivial reference update.

  • Sascha Stahl marked this merge request as ready

    marked this merge request as ready

  • Edited by Software for LHCb
  • Adrian Casais Vidal assigned to @rmatev and unassigned @acasaisv

    assigned to @rmatev and unassigned @acasaisv

  • Sascha Stahl added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading