Central bunch spacing producer for 76x

[davidlange6]

#11544 but for 76x.

[cmsbuild]

A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_7_6_X.

Central bunch spacing producer for 76x

It involves the following packages:

Configuration/StandardSequences
RecoEgamma/EgammaTools
RecoLocalCalo/EcalRecProducers
RecoLuminosity/LumiProducer
RecoParticleFlow/PFClusterProducer
Validation/Configuration

@civanch, @cvuosalo, @mdhildreth, @cmsbuild, @franzoni, @deguio, @slava77, @danduggan, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @mmarionncern, @Sam-Harper, @argiro, @makortel, @GiacomoSguazzoni, @rovere, @lgray, @Martin-Grunewald, @cerati, @VinInn, @dgulhan, @bachtis this is something you requested to watch as well.
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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[davidlange6]

cmsbuild, please test

[deguio]

please test

[smuzaffar]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 4aff40c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11576/8539/summary.html

[slava77]

I checked this locally including run2 relval 2015B and 2015C data workflows and confirm no differences.
The only problem is the cout.

[cmsbuild]

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

[civanch]

@davidlange6 , in BunchSpacingProducer uncontrolled std::cout is left...

[davidlange6]

yes, I’m aware

On Oct 2, 2015, at 10:40 AM, Vladimir Ivantchenko [email protected] wrote:

@davidlange6 , in BunchSpacingProducer uncontrolled std::cout is left...

—
Reply to this email directly or view it on GitHub.

[smuzaffar]

Could this be the PR due to which we have unit test failing IBs?
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_7_6_X_2015-10-15-2300/unitTestLogs/RecoMET/METProducers