adding back few Module level maps

[fioriNTU]

Adding back few module level plots for cluster cherge, size, and digi ADC

[cmsbuild]

A new Pull Request was created by @fioriNTU for master.

It involves the following packages:

DQM/SiPixelPhase1Clusters
DQM/SiPixelPhase1Digis
DQM/SiPixelPhase1TrackClusters

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@idebruyn, @threus, @hdelanno this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[dmitrijus]

+1

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19686/console Started: 2017/05/09 18:07

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[cmsbuild]

+1
Tested at: 46bd601
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18627/19686/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18627/19686/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 3297 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1831050
• DQMHistoTests: Total failures: 47004
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 1783865
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[davidlange6]

hi @fioriNTU @dmitrijus - how many more histogram bins does this imply?

[fioriNTU]

There are 30 more histograms, the total bins are 1856 (# of pixel modules) * 5 = 9280
If this is too much I can disable other less important histograms, but the ones I am adding here would be extremely useful to have in production.

[davidlange6]

+1 *30 I guess. or 600kB. Seems ok

…

On May 10, 2017, at 9:28 AM, fioriNTU ***@***.***> wrote: There are 30 more histograms, the total bins are 1856 (# of pixel modules) * 5 = 9280 If this is too much I can disable other less important histograms, but the ones I am adding here would be extremely useful to have in production. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[fioriNTU]

@davidlange6 the number of bins is correct, you have not to multiply by 30, 1856 is the total number of bins in a set of 6 plots. So, it should be more than ok :-)