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
Run2-sim29 Avoid crashes in SteppingVerboseAction #25950
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25950/8445
|
A new Pull Request was created by @bsunanda for master. It involves the following packages: SimG4Core/TrackingVerbose @cmsbuild, @civanch, @mdhildreth 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. |
-1 Tested at: b3d4d5d You can see the results of the tests here: I found follow errors while testing this PR Failed tests: AddOn
I found errors in the following addon tests: cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi --conditions auto:run2_mc --fast -n 100 --eventcontent AODSIM,DQM --relval 100000,1000 -s GEN,SIM,RECOBEFMIX,DIGI:pdigi_valid,L1,DIGI2RAW,L1Reco,RECO,EI,VALIDATION --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --datatier GEN-SIM-DIGI-RECO,DQMIO --beamspot NominalCollision2015 --era Run2_2016 : FAILED - time: date Fri Feb 15 19:15:28 2019-date Fri Feb 15 19:06:48 2019 s - exit: 35584 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@bsunanda , inside g4SimHits main configuration file there are options allowing different level of stepping verbosity. I do not think, that this sub-library can provide anything more detailed. In the main script there are possibilities to select event but number, track by id, use different level of printout. Please, try out these features. |
@cmsbuild Please test |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+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) |
hold pending a clarification |
Pull request has been put on hold by @fabiocos |
unhold I see the effect of this PR, this anyway means that the SteppingVerbose is no more an option. I have verified that if this setting is activated it runs smoothly without crashing: TV: Get the Tracking Manager: 0 and the SteppingVerbose: 0 |
+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 be automatically merged. |
No description provided.