online: Move step timers into global reco class.
Merge request reports
Activity
assigned to @fweig
Dear @v.friese, @a.bercuci, @p.kaehler, @d.smith, @fweig,
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.
added CodeOwners label
- Resolved by Volker Friese
What is the motivation for this change?
In my opinion, it would be beneficial to get an output of the processing time per step in the console log. Ideally, each step should deliver one line, with the proc. time, throughput and some basic numbers (like number of input and output data or whichever is considered meaningful). This is beneficial when running the application offline (for development, debugging and other purposes). That can hardly be achieved if the timer is moved outside of the step itself.
Another argument: Since for instance the trigger will be flexible (choose from a set of available trigger), the input data size might be different depending on the chosen trigger and its settings. So, the throughput cannot be calculated on the global reco level.
added 5 commits
-
e088d2d4...9dba6d42 - 3 commits from branch
computing:master
- 28dac60c - online: Move step timers into global reco class.
- ba747555 - format
-
e088d2d4...9dba6d42 - 3 commits from branch
added 8 commits
-
a079262a...ff96893e - 5 commits from branch
computing:master
- 561f963e - online: Move step timers into global reco class.
- 871ca366 - format
- c376c003 - Compile fixes.
Toggle commit list-
a079262a...ff96893e - 5 commits from branch