Skip to content
Snippets Groups Projects

algo: Add monitoring support in cbmreco.

Merged Felix Weiglhofer requested to merge fweig/cbmroot:monitor into master
All threads resolved!

Adds rudimentary monitoring support to cbmreco.

Note: I've only tested this with the file-sink so far.

Merge request reports

Merge request pipeline #22584 passed

Merge request pipeline passed for 7edc4eb9

Approved by

Merged by Felix WeiglhoferFelix Weiglhofer 1 year ago (Jun 12, 2023 2:15pm UTC)

Merge details

  • Changes merged into master with 7edc4eb9.
  • Deleted the source branch.

Pipeline #22613 passed

Pipeline passed for 7edc4eb9 on master

Activity

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

    added 2 commits

    • 1a37a2f6 - algo: Exclude monitor from root dictionary generation.
    • a315977c - external: Install fles-monitor ipp-files.

    Compare with previous version

  • Jan de Cuveland
  • Jan de Cuveland
  • Looks fine to me!

  • Felix Weiglhofer added 10 commits

    added 10 commits

    • a315977c...291e393f - 5 commits from branch computing:master
    • ba686896 - algo: Add monitoring support in cbmreco.
    • 6902cdc6 - external: Install library fles_monitoring.
    • 9d69b5e6 - algo: Exclude monitor from root dictionary generation.
    • 977836b5 - external: Install fles-monitor ipp-files.
    • 21802606 - algo: Forward declare cbm::Monitor in ChainContext.

    Compare with previous version

  • added 1 commit

    • 36db0433 - algo: Forward declare cbm::Monitor in ChainContext.

    Compare with previous version

  • added 1 commit

    • 7edc4eb9 - algo: Update monitor options description.

    Compare with previous version

  • Dear @f.uhlig, @v.friese, @p.-a.loizeau,

    you have been identified as code owner of at least one file which was changed with this merge request.

    Please check the changes and approve them or request changes.

  • Volker Friese approved this merge request

    approved this merge request

  • assigned to @fweig

  • Felix Weiglhofer resolved all threads

    resolved all threads

  • Pierre-Alain Loizeau mentioned in merge request !1172 (merged)

    mentioned in merge request !1172 (merged)

  • Please register or sign in to reply
    Loading