Ecal pedestal pcl fixes 93x

[argiro]

fix for Nans, added histos, added check, added exclusion of non-stable beam (off in config)
Same as #20366 (based on master) , but for 93x

[cmsbuild]

A new Pull Request was created by @argiro for CMSSW_9_3_X.

It involves the following packages:

Calibration/EcalCalibAlgos

@ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @lpernie can you please review it and eventually sign? Thanks.
@ghellwig, @mmusich, @tocheng this is something you requested to watch as well.
@davidlange6, @slava77 you are the release manager for this.

cms-bot commands are listed here

[lpernie]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/22748/console Started: 2017/09/06 15:08

[cmsbuild]

+1
Tested at: e5c726f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22748/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
b9e4992
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22748/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22748/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

Pull request #20399 was updated. @ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @lpernie can you please check and sign again.

[cmsbuild]

Pull request #20399 was updated. @ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @lpernie can you please check and sign again.

[lpernie]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/22755/console Started: 2017/09/06 17:11

[cmsbuild]

Pull request #20399 was updated. @ghellwig, @kkousour, @arunhep, @cerminar, @fabozzi, @cmsbuild, @franzoni, @kpedro88, @GurpreetSinghChahal, @lpernie can you please check and sign again.

[arunhep]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/22829/console Started: 2017/09/09 00:20

[cmsbuild]

+1
Tested at: 5e88e68
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22829/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
3043808
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22829/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20399/22829/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 26
• DQMHistoTests: Total histograms compared: 2641982
• DQMHistoTests: Total failures: 203
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2641590
• DQMHistoTests: Total skipped: 189
• DQMHistoTests: Total Missing objects: 0
• Checked 107 log files, 14 edm output root files, 26 DQM output files

[fabozzi]

+1

[arunhep]

+1

[davidlange6]

merge