Skip to content
Snippets Groups Projects

Decrease log lines and log levels + log fixes in various places in Reco classes

Merged Pierre-Alain Loizeau requested to merge p.-a.loizeau/cbmroot:logs_reduction into master
  1. Decrease some log levels to debug in L1, CA, CbmRecoQaTask and in the calibration loading part of CbmTofEventClusterizer to reduce the size of the Initialization block in the logs (and the amount of details needed only by experts)
  2. Fix some spacing and line spreads in the Init part of Sts Reco (same reason)
  3. Use only FairLogger in L1 for consistency and to avoid options conflicts at compilation/runtime = reason for Bugfix label
  4. Use only FairLogger in CbmKFPartEfficiencies to allow control of the log level

Part of the stack of non-CI commits in !1874 (merged), separated to make it more visible/obvious and get it in without waiting for resolution of the online logger troubles

Set @f.uhlig as reviewer for the technical part but mainly expecting feedback from @s.zharko, @v.friese, @se.gorbunov and eventually @n.herrmann

Redmine: Refs #3399

Edited by Pierre-Alain Loizeau

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
  • @p.-a.loizeau,

    beside my small comment I am fine with the MR.

  • Florian Uhlig approved this merge request

    approved this merge request

  • added 2 commits

    • 3e0e70a6 - Revert back reco L1 to using only FairLogger
    • 4bce89e1 - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

  • Pierre-Alain Loizeau resolved all threads

    resolved all threads

  • added 8 commits

    • b9aa2250 - 1 commit from branch computing:master
    • 49da1fd3 - [L1/CA] decrease log level of a few printouts to debug
    • a58de820 - Switch some Init logs to debug in CbmTofTrackingInterface
    • 96f0e03e - Switch some calibration Init logs to debug in CbmTofEventClusterizer
    • cdfa4131 - Reduce lines-spread and fix spacings in some Init logs in Sts Reco
    • 45d2b621 - Switch some Init logs to debug in CbmRecoQaTask
    • 24050e77 - Revert back reco L1 to using only FairLogger
    • 79a4e093 - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

  • Pierre-Alain Loizeau requested review from @v.friese and removed review request for @f.uhlig

    requested review from @v.friese and removed review request for @f.uhlig

  • added 12 commits

    • 79a4e093...6a79da4e - 5 commits from branch computing:master
    • c5841556 - [L1/CA] decrease log level of a few printouts to debug
    • d9304738 - Switch some Init logs to debug in CbmTofTrackingInterface
    • 58750c04 - Switch some calibration Init logs to debug in CbmTofEventClusterizer
    • 37c267fe - Reduce lines-spread and fix spacings in some Init logs in Sts Reco
    • d0f8de38 - Switch some Init logs to debug in CbmRecoQaTask
    • 0c0a5094 - Revert back reco L1 to using only FairLogger
    • 6195aaee - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

  • mentioned in merge request !1888 (merged)

  • Pierre-Alain Loizeau changed the description

    changed the description

  • added 16 commits

    • 6195aaee...19971d11 - 9 commits from branch computing:master
    • 4ce57bb8 - [L1/CA] decrease log level of a few printouts to debug
    • 5e9b2dfb - Switch some Init logs to debug in CbmTofTrackingInterface
    • f26ec5e3 - Switch some calibration Init logs to debug in CbmTofEventClusterizer
    • adad8004 - Reduce lines-spread and fix spacings in some Init logs in Sts Reco
    • 8f46c22c - Switch some Init logs to debug in CbmRecoQaTask
    • 5895d121 - Revert back reco L1 to using only FairLogger
    • 384d996f - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

  • Sergey Gorbunov approved this merge request

    approved this merge request

  • Florian Uhlig added 8 commits

    added 8 commits

    • a90d0dcc - 1 commit from branch computing:master
    • 18746d87 - [L1/CA] decrease log level of a few printouts to debug
    • 1d1f3ef2 - Switch some Init logs to debug in CbmTofTrackingInterface
    • 551d8353 - Switch some calibration Init logs to debug in CbmTofEventClusterizer
    • 88b61823 - Reduce lines-spread and fix spacings in some Init logs in Sts Reco
    • 87409f75 - Switch some Init logs to debug in CbmRecoQaTask
    • a9eb965d - Revert back reco L1 to using only FairLogger
    • c4592691 - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

  • added 18 commits

    • c4592691...6390397e - 11 commits from branch computing:master
    • ee523910 - [L1/CA] decrease log level of a few printouts to debug
    • 1dcd0004 - Switch some Init logs to debug in CbmTofTrackingInterface
    • 26fc1501 - Switch some calibration Init logs to debug in CbmTofEventClusterizer
    • 39c87673 - Reduce lines-spread and fix spacings in some Init logs in Sts Reco
    • 6966348d - Switch some Init logs to debug in CbmRecoQaTask
    • 03dd7027 - Revert back reco L1 to using only FairLogger
    • aa9a2117 - Use FairLogger instead of cout in CbmKFPartEfficiencies

    Compare with previous version

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