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
Updated list of magnetic field steppers and EM physlists for Geant4 10.4 #22073
Conversation
The code-checks are being triggered in jenkins. |
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22073/3201 |
The tests are being triggered in jenkins. |
A new Pull Request was created by @civanch (Vladimir Ivantchenko) for master. It involves the following packages: SimG4Core/Application @cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
+1 |
The code-checks are being triggered in jenkins. |
+1 |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22073/3210 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
@civanch as far as I can see the default stepper is unchanged. What about the changes in the physics lists? What should be the effect? |
@fabiocos , I have checked, that these modifications do not change results - random number sequence not changed. These minor changes means do not instantiate objects, which are not used and skip some calls to these objects. |
+1 |
These modifications are recommended by Geant4 experts for the version 10.4. None of those should affect current results, because the default configuration is not changed. This PR includes: