Skip to content

Remove or demote excessive printout from TileMuonReceiverDecision and TGCTriggerDBAlg

Mark Sutton requested to merge sutt/athena:nonsense into master

Excessive WARNINGS from TileMuonReceiverDecision and TGCTriggerDBAlg have been demoted.

In the case of TileMuonReceiverDecision, it demotes the output after the first 20 WARNINGS in the loop, such that the first 20 WARNINGS are shown as usual, and the subsequent WARNINGS are instead printed as DEBUG.

For the TCG output, since the information in the channel was not being printed, then all the INFO lines were identical, which is a bit excessive so the first is printed as INFO, and all the subsequent, identical entries are printed as DEBUG. It they were left as INFO intentionally, and the intention is simply to spot any which are not identical, then it would be better to simply do a comparison of them all, and then print out the number that are different. As this is an INFO presumably it was not important.

What we really need is for all this extra code / logic to itself be protected by some macro so that it can be preprocessed out, ie, we really want a MSG_LEVEL_INFO macro that we can put around the whole code fragment to remove if for production code.

Partly ddresses ATR-23214 and ATR-23215, and can be treated only as a temporary solution to reduce the size of the log files until a proper solution is providedm or, if the WARNINGS are an indication of a genuine problem, until that issue can be resolved.

Edited by Mark Sutton

Merge request reports