Changing input collection for tauGenJets in miniAOD

[jpavel]

In the default PAT, the tauGenJets are made from genParticles. The miniAOD does not contain genParticles, so the links stored in pat::Tau are broken. The prunedGenParticles contain all particles necessary for tauGenJet building and are contained in miniAOD.

Physics output impact: none expected (however the links genJet constituents will work in miniAOD)

Timing impact: might be slightly faster (using smaller collection for tauGenJet production)
Output size impact: none expected (the references were filled also before, only they were pointing to non-existing objects)

@mbluj @gpetruc @arizzi

Can be forward-ported to 71x and 72x

[cmsbuild]

A new Pull Request was created by @jpavel (Pavel Jez) for CMSSW_7_0_X.

Changing input collection for tauGenJets in miniAOD

It involves the following packages:

PhysicsTools/PatAlgos

@nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please review it and eventually sign? Thanks.
@nhanvtran, @TaiSakuma, @mmarionncern, @schoef 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.

[monttj]

+1
For curiosity, maybe to other experts, I think the link between genJet and genParticle is also broken so the genGenConstituents() can not be called. Would it be also possible to keep the link easily?

[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?

[arizzi]

Right now we just drop/zero genjets constituents but they could be
rekeyd to the packed Gen as we do for regular Jets... But it takes more
space...
Il 30/lug/2014 23:47 "Tae Jeong Kim" notifications@github.com ha scritto:

+1
For curiosity, maybe to other experts, I think the link between genJet and
genParticle is also broken so the genGenConstituents() can not be called.
Would it be also possible to keep the link easily?

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

[cmsbuild]

+1
Tested at: 89b5a4e
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4833/241/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?