Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Puppi isolation electrons and photons miniAOD rebased #15628

Conversation

ishvetso
Copy link
Contributor

@ishvetso ishvetso commented Aug 27, 2016

This is a rebase of #15528
most of things are fixed, expect that as far as I understand patPhoton -> associatedPackedPFCandidates() is giving an error

awhitbeck and others added 30 commits August 23, 2016 10:56
…rgy to tiny value to avoid nan-eta for HO-only towers (when HO is'deactivated', i.e. energy set to zero)
…ower energy to tiny value to avoid nan-eta for HO-only towers (when HO is'deactivated', i.e. energy set to zero)"

This reverts commit 026a4e1.
…ecay modes with 0pi0s the switch has no effect. It is made here nevertheless for the sake of it.
…ble; N.B.: I assume towerP4[3]==0 && E_outer>0 should only happen, if theHOIsUsed==false anyway

process.egmElectronPUPPINoLeptonsIsolation = _egmElectronIsolationMiniAODPUPPINoLeptons.clone()
process.egmElectronPUPPINoLeptonsIsolation.srcToIsolate = cms.InputTag("reducedEgamma","reducedGedGsfElectrons")
process.egmElectronPUPPINoLeptonsIsolation.srcForIsolationCone = cms.InputTag("packedPFCandidates")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

newline is missing

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2016

Pull request #15628 was updated. @cmsbuild, @cvuosalo, @slava77, @monttj, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2016

Pull request #15628 was updated. @cmsbuild, @cvuosalo, @slava77, @monttj, @davidlange6 can you please check and sign again.

@slava77
Copy link
Contributor

slava77 commented Sep 4, 2016

-1
needs a rebase

@ishvetso
Copy link
Contributor Author

ishvetso commented Sep 4, 2016

please refer to #15722, rebased

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment