Skip to content

Add multithreading support for ProcessDefinedStepInVolumeActor#927

Merged
tbaudier merged 3 commits intoOpenGATE:masterfrom
bhbrunt:processdefinedstepinvolume-multithreading
Apr 21, 2026
Merged

Add multithreading support for ProcessDefinedStepInVolumeActor#927
tbaudier merged 3 commits intoOpenGATE:masterfrom
bhbrunt:processdefinedstepinvolume-multithreading

Conversation

@bhbrunt
Copy link
Copy Markdown

@bhbrunt bhbrunt commented Mar 5, 2026

Add multithreading support for ProcessDefinedStepInVolumeActor

  • Define a G4Cache to hold the interaction counter.

@bhbrunt bhbrunt marked this pull request as ready for review March 8, 2026 14:13
@tbaudier
Copy link
Copy Markdown
Contributor

Hello @bhbrunt

Is it possible to rebase with the current master please? If you do not know how to do, I can help you
Thank you

@bhbrunt bhbrunt force-pushed the processdefinedstepinvolume-multithreading branch from fff4761 to e57af80 Compare March 16, 2026 13:14
@bhbrunt
Copy link
Copy Markdown
Author

bhbrunt commented Mar 16, 2026

Hello @bhbrunt

Is it possible to rebase with the current master please? If you do not know how to do, I can help you Thank you

Sure thing, done!

@dsarrut
Copy link
Copy Markdown
Contributor

dsarrut commented Mar 17, 2026

thanks a lot for this! A test (and doc) is needed to ensure everything works fine with MT. Can you please propose something inspired from test093 (https://github.com/OpenGATE/opengate/blob/master/opengate/tests/src/physics/test093_proc_in_vol_attribute.py) ? Thanks !

@tbaudier tbaudier force-pushed the processdefinedstepinvolume-multithreading branch from e57af80 to e56e274 Compare April 15, 2026 12:30
@tbaudier tbaudier merged commit fd5e2b3 into OpenGATE:master Apr 21, 2026
61 of 63 checks passed
@bhbrunt
Copy link
Copy Markdown
Author

bhbrunt commented Apr 21, 2026

Thanks @tbaudier for writing the test and getting this merged. Sorry I didn't get to it 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants