-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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-Sim47 Allow saving tracks with momentum above a certain threshold #28088
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28088/12068
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-28088/12069
|
A new Pull Request was created by @bsunanda for master. It involves the following packages: Geometry/HGCalSimData @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @kpedro88 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. |
@cmsbuild Please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
@bsunanda , sorry, I do not raise this question before. Is it optimal introduce MCtruth into sensitive detector code? I mean that until now MCtruth was in TrackingAction/StackingAction - it is more natural place, because there is sorting particles at production - we create a new MCtruth object depending on energy of a particle at production. Inside the SD code it is more easy identify the moment when a particle hit HGCal but to handle particles produced inside HGCal user actions are better. |
Comparison is ready Comparison Summary:
|
+1 |
+1 |
@kpedro88 I think your remarks have been addressed, and you had already signed this code. Latest changes look straightforward to me, I merge the PR, please check it and comment in case. |
+1 |
merge |
+upgrade |
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 be automatically merged. |
PR description:
These changes allow saving some of the Sim tracks entering in the HGCal region and have kinetic energies above a threshold. By default the thresholds are set high - one need to configure it to allow tracks at lower energies
PR validation:
Tested with cfg files in SimG4CMS/Calo/test/python area and also with the standard run matrix
if this PR is a backport please specify the original PR:
Nothing special