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
Run3-TB51 Correct the beam momentum generator for HGCal TB studies #29114
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29114/14018
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29114/14021
|
A new Pull Request was created by @bsunanda for master. It involves the following packages: IOMC/ParticleGuns @SiewYan, @efeyazgan, @mkirsano, @civanch, @mdhildreth, @cmsbuild, @agrohsje, @kpedro88, @alberto-sanchez, @qliphy can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Pull request #29114 was updated. @SiewYan, @efeyazgan, @mkirsano, @civanch, @mdhildreth, @cmsbuild, @agrohsje, @kpedro88, @alberto-sanchez, @qliphy can you please check and sign again. |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+upgrade |
+1 |
@kpedro88 is your proposal to use the existing functions strict or not, i.e. is it just a proposal? Waiting for this before signing. |
@agrohsje the PR was updated to use the deltaPhi functions where appropriate (some of the cases are slightly outside the scope of what is available centrally) |
+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. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Correct the beam generator used for HGCal TB studies and update the analyzer to take the beam dependent parameters
PR validation:
Use the cfg files in SimG4CMS/HGCalTestBeam/test
if this PR is a backport please specify the original PR and why you need to backport that PR:
Nothing special