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

Fix switch to ak8GenJetsNoNu in miniaod #9121

Merged
merged 1 commit into from Jun 2, 2015

Conversation

ahinzmann
Copy link
Contributor

In the previous PR #8241 where MiniAOD GenJets were switched to NoNu, AK8GenJetsNoNu were forgotten.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @ahinzmann for CMSSW_7_5_X.

Fix switch to ak8GenJetsNoNu in miniaod

It involves the following packages:

PhysicsTools/PatAlgos

@cmsbuild, @vadler, @nclopezo, @monttj can you please review it and eventually sign? Thanks.
@rappoccio, @imarches, @acaudron, @mmarionncern, @jdolen, @nhanvtran, @schoef, @ferencek, @gpetruc, @mariadalfonso, @pvmulder, @TaiSakuma 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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@ahinzmann
Copy link
Contributor Author

@gpetruc
It would be nice to have this fixed for the next round of MiniAODs. Is a backport of this PR to 7_4_X needed for that?

@gpetruc
Copy link
Contributor

gpetruc commented May 28, 2015

We still have to see whether the next round will be made with 74X or 75X.
Anyway, for the moment it should not be backported to 74X: the PR can be
made, but it should not be merged, since the 74X production is still
ongoing.

On Thu, May 28, 2015 at 9:43 AM, ahinzmann notifications@github.com wrote:

@gpetruc https://github.com/gpetruc
It would be nice to have this fixed for the next round of MiniAODs. Is a
backport of this PR to 7_4_X needed for that?


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

@davidlange6
Copy link
Contributor

+1

davidlange6 added a commit that referenced this pull request Jun 2, 2015
Fix switch to ak8GenJetsNoNu in miniaod
@davidlange6 davidlange6 merged commit c73a0af into cms-sw:CMSSW_7_5_X Jun 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants