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

PR to merge back the private rwgt fork to official branch #1111

Merged
merged 11 commits into from
Mar 1, 2017

Conversation

yuanchao
Copy link
Contributor

No description provided.

@perrozzi
Copy link
Contributor

the PR seems to still have conflicts, can you please solve them?

@yuanchao
Copy link
Contributor Author

Conflicts resolved. (b_bbar_4l part removed for the moment)

@perrozzi
Copy link
Contributor

thanks. may I ask why b_bbar_4l is removed?

@yuanchao
Copy link
Contributor Author

For b_bbar_4l, the source was checked out separately in Aug. normal version. Some corresponding code is written accordingly in this script. Therefore for the Dec. rwgt version, I need to check again before adding it back.

@perrozzi
Copy link
Contributor

can you please coordinate with @intrepid42 on the inclusion on b_bbar_4l?
I assume that we can merge this PR, then re-include asap this process to let the TOP group perform tests and produce events, right?
comments? timescales?

@mseidel42
Copy link
Collaborator

@yuanchao @perrozzi POWHEG-BOX-RES/b_bbar_4l already supports the new weights syntax, so it most likely just works with your new version. Please add it back, thank you.

@perrozzi perrozzi merged commit bfa8b72 into cms-sw:powheg_gcc530 Mar 1, 2017
@perrozzi
Copy link
Contributor

perrozzi commented Mar 1, 2017

PR merged to start testing the new release, b_bbar_4l will be added back asap, @intrepid42 please ping if this doesn't happen "soon"

@mseidel42 mseidel42 mentioned this pull request Mar 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants