New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move max iterations message to LogDebug #19242
Conversation
A new Pull Request was created by @emanueledimarco (Emanuele Di Marco) for master. It involves the following packages: RecoLocalCalo/EcalRecAlgos @perrotta, @cmsbuild, @slava77, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
The tests are being triggered in jenkins. |
Comparison job queued. |
remind me - are LogDebugs effectively no-ops?
…
|
On 6/15/17 8:34 AM, David Lange wrote:
remind me - are LogDebugs effectively no-ops?
https://github.com/cms-sw/cmssw/blob/master/FWCore/MessageLogger/interface/MessageLogger.h#L502-L509
it is a bit more than no-op, but it's pretty close.
…
> On Jun 15, 2017, at 3:18 PM, Slava Krutelyov ***@***.***> wrote:
>
> @cmsbuild please test
>
> @fwyzard
> I suppose that master is enough for this issue. Please confirm.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#19242 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AEdcbtC43qKG1dECwDJHXuJZyNQx-9hKks5sEU7xgaJpZM4N7Jko>.
|
Comparison is ready Comparison Summary:
|
Assuming the 100s per event are soon to go away that is probably ok
Cheers,
David
On 15 Jun 2017, at 18:56, Slava Krutelyov ***@***.******@***.***>> wrote:
On 6/15/17 8:34 AM, David Lange wrote:
remind me - are LogDebugs effectively no-ops?
https://github.com/cms-sw/cmssw/blob/master/FWCore/MessageLogger/interface/MessageLogger.h#L502-L509
it is a bit more than no-op, but it's pretty close.
> On Jun 15, 2017, at 3:18 PM, Slava Krutelyov ***@***.******@***.***>> wrote:
>
> @cmsbuild please test
>
> @fwyzard
> I suppose that master is enough for this issue. Please confirm.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#19242 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AEdcbtC43qKG1dECwDJHXuJZyNQx-9hKks5sEU7xgaJpZM4N7Jko>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#19242 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEzywzY0IoTekFziqpZVGKLWNgtjAWAHks5sEWIxgaJpZM4N7Jko>.
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
Sure, master is fine, thank you.
.A
|
+1 |
move max iterations message to LogDebug
Moving max iterations message to LogDebug, since in the first runs of 2017 (full MF enabled for 100% of hits) is flooding logfiles, see this HN.
Privately will check if this is by change related to some conditions (pedestal, pulse shape) for any channel slightly off.
@bendavid @amassiro @fwyzard also follow this