-
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
Simulation: update the test for the definition of back-scattering from CALO #44945
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44945/40216
|
A new Pull Request was created by @fabiocos for master. It involves the following packages:
@cmsbuild, @mdhildreth, @civanch can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7a901f/39328/summary.html Comparison SummarySummary:
|
@cms-sw/simulation-l2 comments? |
6803e7b
to
cfae1d0
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44945/40253
|
Pull request #44945 was updated. @mdhildreth, @civanch, @cmsbuild can you please check and sign again. |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Comparison SummarySummary:
|
please test |
-1 Failed Tests: UnitTests RelVals-INPUT Unit TestsI found 1 errors in the following unit tests: ---> test test-das-selected-lumis had ERRORS RelVals-INPUT
Comparison SummarySummary:
|
Seems to be a DAS problem. Should we ignore this or restart tests? |
please test let's try again... |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7a901f/39398/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. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
The transition from CALO to Tracker is monitored in the
Phase2SteppingAction
ofSimG4Core/Application
to define the back-scattering condition of a particle, used to define the corresponding category for BTLPSimHit
. This condition was updated in #43142 to cope with the fact that the the transition changed in Phase2 geometries, removing at some point (Tracker version T34) the intermediate step into CSME volume. At that time a check on the distance from the origin was introduced, requiring that the post-step has a smaller distance from the origin than the pre-step. This condition is anyway not always fulfilled, driving to an incorrect classification of hits from back-scattering as secondary.This PR updates the test so as to fix this issue. Either a direct CALO -> Tracker transition is found, or a CSME -> Tracker transition is tested, with the understanding based on navigation checks that it is possible for particles coming from CALO only. As soon as older geometries will be abandoned, this test will not be any more necessary.
PR validation:
Tested with the debugging verbosity of
Phase2SteppingAction
and of theCMSSteppingVerbose