Skip to content
Snippets Groups Projects

[ATR-20835] HLT truncation monitoring for R3

Merged Tim Martin requested to merge tamartin/athena:tdtTruncation into master

Follows on from !30003 (merged) (cc @sutt)

Moves the logic behind where to look for truncation from the HLTMonTool back into the TDT.

Additionally exposes the ConfSvc in this tool to python manipulation, as for R3 we should be pointing it to the xAODConfSvc rather than its default, the TrigConfSvc.

Prefers to check the flag in xAOD::TrigDecision (for R3 will aim to make this the canonical way offline). But for R2 input in Athena, it can also fall back on HLTResult_HLT

Uses new enum location from !30292 (merged)

Edited by Tim Martin

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
  • Rafal Bielski
  • Tim Martin added 1 commit

    added 1 commit

    • d64e2591 - Update truncation monitoring for R3

    Compare with previous version

  • Tim Martin resolved all threads

    resolved all threads

  • Tim Martin
  • Rafal Bielski mentioned in merge request !30292 (merged)

    mentioned in merge request !30292 (merged)

  • Tim Martin unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Tim Martin changed the description

    changed the description

  • This merge request affects 3 packages:

    • Trigger/TrigAnalysis/TrigDecisionMaker
    • Trigger/TrigAnalysis/TrigDecisionTool
    • Trigger/TrigMonitoring/TrigHLTMonitoring

    Adding @tamartin ,@ebergeas as watchers

  • Tim Martin resolved all threads

    resolved all threads

  • Tim Martin added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 3 packages:

    • Trigger/TrigAnalysis/TrigDecisionMaker
    • Trigger/TrigAnalysis/TrigDecisionTool
    • Trigger/TrigMonitoring/TrigHLTMonitoring

    Adding @tamartin ,@ebergeas as watchers

  • :negative_squared_cross_mark: CI Result FAILURE (hash b1685fdc)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :o: :white_check_mark:
    required tests :o: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :o: Athena: number of compilation errors 1, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9557]

  • Tim Martin added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 3 packages:

    • Trigger/TrigAnalysis/TrigDecisionMaker
    • Trigger/TrigAnalysis/TrigDecisionTool
    • Trigger/TrigMonitoring/TrigHLTMonitoring

    Adding @tamartin ,@ebergeas as watchers

  • :white_check_mark: CI Result SUCCESS (hash 7134315d)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9579]

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