-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
backport of LHEmttFilter for 10_6 #44745
Conversation
A new Pull Request was created by @agrohsje for CMSSW_10_6_X. It involves the following packages:
@GurpreetSinghChahal, @SiewYan, @menglu21, @cmsbuild, @bbilin, @alberto-sanchez, @mkirsano can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
cms-bot internal usage |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-741983/39031/summary.html Comparison SummarySummary:
|
Hi @rappoccio . Is there already the new 10_6 release? Could still go in? @menglu21 @bbilin please sign the back-port. Thanks! |
+1 Sorry to have missed it for 10_6_41... |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_14_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description/backport info:
This PR is a backport of the LHE mttbar filter to 10_6. It is needed for UL EFT MC samples.
PR validation:
I verified that the code works properly in 10_6. Generating a few events and plotting mttbar all looked good.
This is a link to the original PR by Giulia:
#33541