Improve plots in local processing (no change for DQM)

[namapane]

No description provided.

[cmsbuild]

A new Pull Request was created by @namapane (Nicola Amapane) for CMSSW_7_0_X.

Improve plots in local processing (no change for DQM)

It involves the following packages:

Validation/DTRecHits

@smuzaffar, @nclopezo, @danduggan, @rovere, @deguio, @eliasron can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.

[nclopezo]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/804/summary.html

[eliasron]

Dear Nicola,

there is currently a campaign to try to move the declaration of histograms away from the constructors to the beginRun process. This would avoid having large spikes in memory consumption at the end of the run. Do you think you could have a look and move them? Thanks.

[cmsbuild]

The following categories have been signed by eron (a.k.a. @eliasron on GitHub): DQM

@cms-git-dqm

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it?

[namapane]

Hi,

On 07-Oct-13 14:52, eliasron wrote:

Dear Nicola,

there is currently a campaign to try to move the declaration of
histograms away from the constructors to the beginRun process.

OK, but I supppose you mean to the beginJob, rather than beginRun?

Cheers,
Nicola

This
would avoid having large spikes in memory consumption at the end of the
run. Do you think you could have a look and move them? Thanks.

—
Reply to this email directly or view it on GitHub
#1004 (comment).

[deguio]

it is beginRun. elias is correct.
everything has been discussed back in June. see the details in:
https://indico.cern.ch/getFile.py/access?contribId=3&resId=0&materialId=slides&confId=256775