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 weight files #6

Merged
2 commits merged into from Oct 23, 2015
Merged

new weight files #6

2 commits merged into from Oct 23, 2015

Conversation

jbrands
Copy link
Contributor

@jbrands jbrands commented Oct 22, 2015

This PR is once again just a renaming. This time hopefully the last. I will do a PR for the bug fix in RecoJets I have done and for which the renaming was necessary. Again the weight files with the ending _newNames are the ones which are now used in my code. The other ones are the old weight files.

@jbrands
Copy link
Contributor Author

jbrands commented Oct 22, 2015

This is the corresponding PR: cms-sw/cmssw#12053

@cmsbuild
Copy link
Contributor

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

new weight files

@cmsbuild, @Degano 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#1919

@slava77
Copy link

slava77 commented Oct 23, 2015

@Degano please check and merge this
Now there is PR in CMSSW that needs this data change

ghost pushed a commit that referenced this pull request Oct 23, 2015
@ghost ghost merged commit d5f7ea7 into cms-data:master Oct 23, 2015
@ghost
Copy link

ghost commented Oct 23, 2015

@jbrands @slava77 I've merged it for 76X here: cms-sw/cmsdist#1925

@slava77
Copy link

slava77 commented Oct 23, 2015

@Degano
Thank you.

@ahinzmann
Copy link
Contributor

I'd like to ask to merge this in 74X and 75X as well, such that it can be used in analyses.
It it possible with this PR, or do we need to create a new one?

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