extended_DIGI_window

[amartelli]

extended ECAL digitization window

[cmsbuild]

A new Pull Request was created by @amartelli for CMSSW_7_4_X.

extended_DIGI_window

It involves the following packages:

SimCalorimetry/EcalSimAlgos

@cmsbuild, @civanch, @nclopezo, @mdhildreth can you please review it and eventually sign? Thanks.
@argiro 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.

[cmsbuild]

+1
Tested at: 4b0940f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6777/1425/summary.html

[davidlange6]

73x and 74x are nominally both closed to digi changes now. Is there a discussion of this request somewhere?

On Dec 4, 2014, at 10:01 AM, amartelli notifications@github.com
wrote:

extended ECAL digitization window

You can merge this Pull Request by running

git pull https://github.com/amartelli/cmssw extended_DIGI_window
Or view, comment on, or merge it at:

#6777

Commit Summary

• extended_DIGI_window
File Changes

• M SimCalorimetry/EcalSimAlgos/src/EcalHitResponse.cc (2)
Patch Links:

• https://github.com/cms-sw/cmssw/pull/6777.patch
• https://github.com/cms-sw/cmssw/pull/6777.diff
—
Reply to this email directly or view it on GitHub.

[amartelli]

These requests are to fix the ECAL digi window to account for the one used in the mixing. It is important to include it in the coming 73X production.

[amartelli]

To proceed further and have these included, should a dedicated discussion be opened somewhere else?

[mdhildreth]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged.