-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
adding Py8 Pt Gun with exponentially falling pT distribution #43354
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43354/37819
|
A new Pull Request was created by @bennettgberg for master. It involves the following packages:
@alberto-sanchez, @SiewYan, @menglu21, @GurpreetSinghChahal, @mkirsano, @cmsbuild, @bbilin can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hi all, we will greatly appreciate it if this can be taken care of in a timely manner. We of course understand that everyone is busy with many ongoing projects and deadlines, but we believe (and hope) that this PR will not require a significant effort from any of the involved parties because it's a straightforward modification of an existing particle gun. Since we will need to produce a relatively large number of events, which will carry its own delay, we are hoping that at least the approval for this simple PR can proceed smoothly. Right now we see that the "unfinished checks" need some "authorized user" to issue the test command, so it sounds like it should be a quick thing to do. Thank you for your help, |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-342989/36140/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
Dear @sextonkennedy, @rappoccio, @antoniovilela , GEN asks us to check this plugin on an IB release before we move on with the back-ports. Thanks, |
+1 |
PR description:
This PR adds a Pythia8 Pt Gun that uses an exponentially falling pT distribution between the minimum and maximum pT values instead of the uniform pT distribution.
This is needed for Central MC production of EtaTo2MuGamma, which is a background needed for the eta->2mu2e analysis. But the gun could also be used more generally with arbitrary minimum and maximum pT values.
The exponentially falling distribution is needed because the production cross section of the eta meson falls off rapidly with increasing pT, so much more statistics is needed at lower pT values (but only above the acceptance threshold).
It is just adding a new Pt gun, so output of existing code will not change.
JIRA ticket where this PR is requested: https://its.cern.ch/jira/browse/CMSBPHMC-52
PR validation:
code-checks, code-format, and runTheMatrix.py were all run successfully.
The new plugin was also tested thoroughly and successfully used to produce EtaTo2Mu2Gamma events in CMSSW_12_4_17 and CMSSW_13_0_14.
If this PR is accepted I will open a backport for CMSSW_12_4_X for Summer22 production.