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

New pileup Jet Id weights for 76X #7

Merged
2 commits merged into from Feb 25, 2016
Merged

Conversation

jbrands
Copy link
Contributor

@jbrands jbrands commented Feb 25, 2016

New pileup Jet Id weights trained on DY samples by @kkousour and @ahinzmann. Validation plots can be found here: http://www.hephy.at/user/jbrandstetter/PUId_validation76X/

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @jbrands for branch master.

@cmsbuild, @smuzaffar, @Degano, @iahmad-khan, @davidlange6 can you please review it and eventually sign? Thanks.
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.

external issue cms-sw/cmsdist#2165

@ghost
Copy link

ghost commented Feb 25, 2016

@jbrands I will now merge this and prepare the corresponding PR for CMSDIST for the branch 8_1_X.

ghost pushed a commit that referenced this pull request Feb 25, 2016
New pileup Jet Id weights for 76X
@ghost ghost merged commit 90ae65a into cms-data:master Feb 25, 2016
@ahinzmann
Copy link
Contributor

@Degano : Could you make a version of this for all 76X, 80X and 81X.
The reason is that this will be used for an analysis recipes in 76X and 80X as well.

@ghost
Copy link

ghost commented Feb 25, 2016

@ahinzmann The tests are now running for 81X in cms-sw/cmsdist#2166, it can be backported when it'll be integrated.

@ahinzmann
Copy link
Contributor

Thanks you!

@slava77
Copy link

slava77 commented Mar 2, 2016

We need 80X and 76X versions of this.
I mentioned 80X in the ORP yesterday, although only verbally.
@Degano please clarify if you can make these.

@slava77
Copy link

slava77 commented Mar 2, 2016

well, the comment should have gone to the cmsdist PR, sorry

@ghost
Copy link

ghost commented Mar 2, 2016

@slava77 Sure I'll make the PRs now.

This pull request was closed.
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