Skip to content
Snippets Groups Projects

Implement L1 jet matching to JetMonitoring

Merged Jannik Geisen requested to merge jgeisen/athena:jgeisen_l1jetmatching into master
All threads resolved!

This is addressing the remaining desired feature expressed in ATR-19048, namely adding the possibility to match L1 jets to HLT (or offline) jets.

Also included are changes to the L1 monitoring config and L1 monitoring Alg to implement the histograms of L1 jets matched to offline/online jets.

Tagging @jbossios

Edited by Jannik Geisen

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
  • Jonathan Bossio
  • Jonathan Bossio
  • Jannik Geisen added 1 commit

    added 1 commit

    • 08181c9b - Force crash for wrong matching settings, simplify TrigJetMonitorAlgorithm

    Compare with previous version

  • Jannik Geisen resolved all threads

    resolved all threads

  • Jannik Geisen marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 2 packages:

    • Reconstruction/Jet/JetMonitoring
    • Trigger/TrigMonitoring/TrigJetMonitoring

    Affected files list will not be printed in this case

    Adding @goetz ,@jgeisen as watchers

  • :white_check_mark: CI Result SUCCESS (hash 08181c9b)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :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
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30617]

  • The changes look fine and there are no CI issues. Approving.

    Jason (L1)

  • mentioned in commit da4e7c15

  • Please register or sign in to reply
    Loading