the simple fix possible. sorry for all this to ended up in released code

[ktf]

No description provided.

[ktf]

This is apparently very urgent @davidlange6 objections in bypassing so that we get it in an IB?

[cmsbuild]

A new Pull Request was created by @ktf (Giulio Eulisse) for CMSSW_7_0_X.

the simple fix possible. sorry for all this to ended up in released code

It involves the following packages:

RecoEgamma/EgammaIsolationAlgos

@nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please review it and eventually sign? Thanks.
@lgray 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.

[cmsbuild]

Pull request #3302 was updated. @nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please check and sign again.

[slava77]

the code change looks OK
I haven't tested it yet to see if there are no downstream effects (anyone seeing id=0 killing the jobs just as well).
Considering that Vincenzo doesn't see a crash anymore and if you nee to build 70X patch now, feel free to go ahead and bypass.

[davidlange6]

no rush - the agr is done
On Apr 11, 2014, at 5:39 PM, Slava Krutelyov notifications@github.com wrote:

the code change looks OK
I haven't tested it yet to see if there are no downstream effects (anyone seeing id=0 killing the jobs just as well).
Considering that Vincenzo doesn't see a crash anymore and if you nee to build 70X patch now, feel free to go ahead and bypass.

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

[slava77]

+1

#3302 eaf38d0

as expected, fixes the problem in AGR reported in https://hypernews.cern.ch/HyperNews/CMS/get/recoDevelopment/1255.html

confirmed the crash is going away for wf 1000.0 and file /store/data/Commissioning2014/MinimumBias/RAW/v1/000/220/820/00000/A8DDCB7F-70BF-E311-99FC-02163E00EA35.root

[cmsbuild]

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

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3302/127/summary.html

[cmsbuild]

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

[cmsbuild]

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

[cmsbuild]

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