Adding two by-lumisection plots to ECAL DQM, for CMSSW master

[tanmaymudholkar]

(1) Added new by-lumisection plots to monitor the behavior of TTF4 flags
(2) Fixed the TTF4 trend plots to display number of masked towers only from the current lumisection.

PR for backport to CMSSW 10_1_X: 23921
PR for backport to CMSSW 10_2_X: 23952
PR for DQM deployment: 652

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23920/5617

[cmsbuild]

A new Pull Request was created by @tanmaymudholkar (Tanmay Mudholkar) for master.

It involves the following packages:

DQM/EcalMonitorClient
DQM/EcalMonitorTasks

@andrius-k, @kmaeshima, @schneiml, @dmitrijus, @cmsbuild, @jfernan2 can you please review it and eventually sign? Thanks.
@argiro this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[andrius-k]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/29384/console Started: 2018/07/24 13:47

[cmsbuild]

-1

Tested at: 97caa53

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23920/29384/summary.html

I found follow errors while testing this PR

Failed tests: RelVals

• RelVals:

The relvals timed out after 2 hours.

[cmsbuild]

Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped)

[tanmaymudholkar]

Hi @andrius-k , I'm not quite sure what caused the crash, but it appears to be unrelated to ECAL DQM.

[boudoul]

Hi @tanmaymudholkar , the reason is mentioned above : the relvals timed out after 2 hours .

The tests should be retriggered

[jfernan2]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/29432/console Started: 2018/07/25 09:41

[cmsbuild]

+1
Tested at: 97caa53
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23920/29432/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 2889585
• DQMHistoTests: Total failures: 2
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2889393
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 3583.245 KiB( 30 files compared)
• DQMHistoSizes: changed ( 150.0,... ): 162.824 KiB EcalEndcap/EETriggerTowerTask
• DQMHistoSizes: changed ( 150.0,... ): 20.934 KiB EcalBarrel/EBTriggerTowerTask
• DQMHistoSizes: changed ( 4.22,... ): 81.404 KiB EcalEndcap/EETriggerTowerTask
• DQMHistoSizes: changed ( 4.22,... ): 10.463 KiB EcalBarrel/EBTriggerTowerTask
• Checked 128 log files, 14 edm output root files, 31 DQM output files

[andrius-k]

+1

[cmsbuild]

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 will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[kpedro88]

+1