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-hcx239 Attempt to make HcalSimulationParameters to be read in for dd4hep #28381
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28381/12712
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-28381/12713
|
A new Pull Request was created by @bsunanda for master. It involves the following packages: Geometry/HcalAlgo @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @cmsbuild 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. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@bsunanda - please, rebase |
Ianna has taken the code from here, modified and saved it in #28391. There is nothing more to be added from this. |
PR description:
Attempt to make HcalSimulationParameters created from dd4hep as well as from ddd
PR validation:
There are 2 cfg's in Geometry/HGCalCommonData/test/data/runHcalSimParameterAnalyzer_cfg.py (DDD) and Geometry/HGCalCommonData/test/data/runHcalSimParameterTest_cfg.py (dd4hep) to test these
if this PR is a backport please specify the original PR:
Nothing special