Skip to content
Snippets Groups Projects

Adding monitoring to particle creator in IDTrig precision tracking

Merged Matous Vozak requested to merge mvozak/athena:idtrig-onlinemon into master
All threads resolved!

This MR contains following:

1] Revival of TrackMonitoring tool in the offline version of particle cnv alg 2] Configuration of this alg for IDTrig (in builder form) 3] Calling this configuration in the IDTrig Precision tracking configuration 4] Some minor fixes in the plots and monitored variables

Outlook: In the next MR this will also be added to the TrigFastTrackFinder alg such that comparison can be made between fast and precision stage. However, atm there are few signatures relying on the current particle creator alg in the FTF. Therefore, this will be addressed in additional MR.

Mat

@sutt , @nagano , @smh , @jmasik

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
  • :white_check_mark: CI Result SUCCESS (hash edad9656)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :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:
    make :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:
    optional tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21867]

  • The MR looks good and also passes the CI tests. Marking for L2 to crosscheck before approving.

  • Johannes Junggeburth resolved all threads

    resolved all threads

  • merged

  • Edward Moyse mentioned in commit 9dfac524

    mentioned in commit 9dfac524

  • Please register or sign in to reply
    Loading