-
Notifications
You must be signed in to change notification settings - Fork 99
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
DDSim: correct the number of events when running over all events (-1) #1258
Conversation
Test Results 14 files 14 suites 7h 4m 55s ⏱️ Results for commit 991228d. ♻️ This comment has been updated with latest results. |
01360b2
to
872ac4b
Compare
@@ -30,6 +30,8 @@ Geant4GeneratorActionInit::Geant4GeneratorActionInit(Geant4Context* ctxt, const | |||
InstanceCount::increment(this); | |||
context()->kernel().runAction().callAtEnd(this,&Geant4GeneratorActionInit::end); | |||
context()->kernel().runAction().callAtBegin(this,&Geant4GeneratorActionInit::begin); | |||
declareProperty("numberOfEvents", m_evtTotal); |
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.
@MarkusFrankATcernch
Is this the best way to let me read the number of events from the plugin, or is there something else that would be cleaner?
@andresailer In principle there is a property |
But here the number of events to be processed is going to be |
GenerationActionInit: declare properties to access number of processed runs and events
872ac4b
to
991228d
Compare
@andresailer Correct. |
Thanks @MarkusFrankATcernch |
BEGINRELEASENOTES
ENDRELEASENOTES