Fix sigmaIetaIphi for photon regression (75X)

[lgray]

Forward port of #11403.

[lgray]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

A new Pull Request was created by @lgray (Lindsey Gray) for CMSSW_7_5_X.

Fix sigmaIetaIphi for photon regression (75X)

It involves the following packages:

RecoEgamma/EgammaTools

@cmsbuild, @cvuosalo, @slava77 can you please review it and eventually sign? Thanks.
@Sam-Harper 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.

[cmsbuild]

+1
Tested at: 3b2a460
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11406/8255/summary.html

[cmsbuild]

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

[slava77]

I suppose this PR goes in only if we kill the RECO-based regression PR #11402, right?

[lgray]

Yes.

(Sent from my Nexus 6)
On Sep 24, 2015 2:42 PM, "Slava Krutelyov" notifications@github.com wrote:

I suppose this PR goes in only if we kill the RECO-based regression PR
#11402 #11402, right?

—
Reply to this email directly or view it on GitHub
#11406 (comment).

[slava77]

+1

for #11406 3b2a460

• small fix in the definition of the regression input variable for the miniAOD photon energy correction
• the same changes as in 74X Fix sigmaIetaIphi for photon regression (74X not patches) #11405 93f9992; see details of performance changes there
  • miniAOD photon energies are changed slightly after the fix
• jenkins tests pass and comparisons with baseline showed no differences (the variables are not monitored in current miniAOD monitoring)

This PR is not needed if #11402 can be merged (requires a relval cycle).
If 75X DR campaign/reprocessing release should be cut before 760pre6 relval or another 75X relval cycle is in order, this PR should be merged in 75X.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1