71 x merge 70 x pre mixing 4 29

[mdhildreth]

Merge of PreMixing from 70X changes onto 71X. Recovering from failure of auto-forward-port from last week. Merged onto latest IB.

[cmsbuild]

A new Pull Request was created by @mdhildreth for CMSSW_7_1_X.

71 x merge 70 x pre mixing 4 29

It involves the following packages:

Configuration/Applications
Configuration/EventContent
Configuration/StandardSequences
SimCalorimetry/CaloSimAlgos
SimCalorimetry/EcalSimAlgos
SimCalorimetry/EcalSimProducers
SimCalorimetry/HcalSimAlgos
SimCalorimetry/HcalSimProducers
SimCalorimetry/HcalTestBeam
SimGeneral/DataMixingModule
SimGeneral/MixingModule

@civanch, @nclopezo, @vlimant, @mdhildreth, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @wmtan, @GiacomoSguazzoni, @rovere, @argiro, @cerati 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.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[mdhildreth]

+1

[davidlange6]

+1

[cmsbuild]

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

[cmsbuild]

-1
There is a std::cout in the diff for this pull request. Could you please clean it up?

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (but tests are reportedly failing). @nclopezo, @ktf can you please take care of it?

[ktf]

At least in this case, this is legitimate complain. Mike can you clean up the cout and and the commented code? Thanks.

[cmsbuild]

Pull request #3576 was updated. @civanch, @nclopezo, @vlimant, @mdhildreth, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please check and sign again.

[cmsbuild]

-1
I ran the usual tests and I found errors in the following unit tests:

---> test TestConfigDP had ERRORS

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

[cmsbuild]

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

[davidlange6]

+1
for operations - I don't think the errors had anything to do with this PR did they?
@mdhildreth - can you sign for sim?

[civanch]

+1
I hope Mike agree

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (but tests are reportedly failing). @nclopezo, @ktf can you please take care of it?