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
Safer syntax for modifying the configuration parameters in g4SimHits_cfi.py #36524
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36524/27413
|
A new Pull Request was created by @perrotta (Andrea Perrotta) for master. It involves the following packages:
@cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
Why do you change TimeSliceUnit = 0.00 shouldn't it be TimeSliceUnit = 0.001 |
Thank you Sunanda: this is exactly what I meant for "trivial mistake" in the PR description... |
please abort |
ed070e1
to
5991123
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36524/27415
|
please test |
Pull request #36524 was updated. @cmsbuild, @civanch, @mdhildreth can you please check and sign again. |
@perrotta , thanks, I was not sure that you are on-line, so make extra PR - it will be closed. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36524/27466
|
Pull request #36524 was updated. @cmsbuild, @civanch, @mdhildreth can you please check and sign again. |
please test |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1
|
merge |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-277c1c/21373/summary.html Comparison SummarySummary:
|
In #36506 some partial migration to a safer syntax for modifying the configuration parameters in g4SimHits_cfi.py was started, which avoids overwriting types of those parameters or mis-naming them.
In order not to let the work half done, here are the remaining updates for the other non-migrated parameter updates in the same config
PR validation:
The short matrix runs without error.
Unless I made some trivial mistake in it, no changes are expected anywhere in the test outputs.