-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
[Geant4] Code adjustment for new Geant4 version #28637
[Geant4] Code adjustment for new Geant4 version #28637
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28637/13185
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. |
The tests are being triggered in jenkins.
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28637/13186
|
A new Pull Request was created by @mrodozov (Mircho Rodozov) for master. It involves the following packages: SimG4CMS/Calo @andrius-k, @kmaeshima, @schneiml, @civanch, @mdhildreth, @jfernan2, @fioriNTU can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
-1 Tested at: 06c80ed CMSSW: CMSSW_11_1_GEANT4_X_2019-12-12-2300 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: You can see the results of the tests here: I found follow errors while testing this PR Failed tests: UnitTests RelVals AddOn
I found errors in the following unit tests: ---> test materialBudgetTrackerPlots had ERRORS
When I ran the RelVals I found an error in the following workflows: runTheMatrix-results/101.0_SingleElectronE120EHCAL+SingleElectronE120EHCAL/step1_SingleElectronE120EHCAL+SingleElectronE120EHCAL.log11634.0 step1 runTheMatrix-results/11634.0_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_2021+RecoFull_2021+HARVESTFull_2021+ALCAFull_2021/step1_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_2021+RecoFull_2021+HARVESTFull_2021+ALCAFull_2021.log250202.181 step2 runTheMatrix-results/250202.181_TTbar_13UP18+TTbar_13UP18+PREMIXUP18_PU25+DIGIPRMXLOCALUP18_PU25+RECOPRMXUP18_PU25+HARVESTUP18_PU25/step2_TTbar_13UP18+TTbar_13UP18+PREMIXUP18_PU25+DIGIPRMXLOCALUP18_PU25+RECOPRMXUP18_PU25+HARVESTUP18_PU25.log
I found errors in the following addon tests: cmsDriver.py TTbar_Tauola_13TeV_TuneCUETP8M1_cfi -s GEN,SIM,DIGI,L1,DIGI2RAW --mc --scenario=pp -n 10 --conditions auto:run2_mc_Fake1 --relval 9000,50 --datatier "GEN-SIM-RAW" --eventcontent RAWSIM --customise=HLTrigger/Configuration/CustomConfigs.L1T --era Run2_25ns --fileout file:RelVal_Raw_Fake1_MC.root : FAILED - time: date Tue Dec 17 20:05:24 2019-date Tue Dec 17 19:58:27 2019 s - exit: 34304 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
this looks good to go in master IB. Once it is merged then we will merge cmsdist geant4 changes cms-sw/cmsdist#5424 |
+1 |
+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. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
if (idHC >= 0 && theHC != nullptr) { | ||
std::cout << "FiberhitSize " << theHC->entries() << std::endl; | ||
for (j = 0; j < theHC->entries(); j++) { | ||
std::cout << "FiberhitSize " << thehc_entries << std::endl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cout is not ideal, but this code is likely not part of official workflows meant to run in multithreaded mode
+1 technical update |
The tests are being triggered in jenkins.
|
PR description:
Changes to get new Geant4 10.6 external for the cmsdist Geant4 branch.
See cms-externals/geant4#42 , #28626 and #28629. This PR joins 28626 and 28629 + the requirements discussed in them.
PR validation:
Builds for both cmsdist/master (production "old" geant) and cmsdist/geant4 (10.6 geant)
The previous two cmssw PRs discussion suggests to use int counters to avoid the original build error and from other concerns. This PR builds with both master of cmdist and geant4 branch (works with both geant versions)