Skip to content
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

Prototype decouple event gen from transport #1539

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

sawenzel
Copy link
Contributor

@sawenzel sawenzel commented Mar 18, 2024

(Optional) decoupling of event gen from transport
Development allowing event generation to run in a separate task
from detector simulation. In this mode, event generation will write
kinematics to disc, which is later picked up by o2-sim.

Development done in order to increase CPU efficiency in cases
where event generation was slow and not being able to serve the workers
of o2-sim with enought work. The development fixes such cases, because
multiple event generators (for different timeframes) can now run in parallel.

The new mode is the default, but we can return to the intregrated mode
by using option --event-gen-mode integrated.

@sawenzel sawenzel marked this pull request as draft March 18, 2024 14:58
Copy link

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2022-pp-apass4
async-2023-pbpb-apass
async-2023-pp-apass1
async-data
async-mc
async-2022-pp-apass6

@sawenzel sawenzel force-pushed the swenzel/eventgendecoupling branch from 5de2b3e to 40a9186 Compare March 21, 2024 08:10
@sawenzel sawenzel marked this pull request as ready for review March 21, 2024 08:10
@sawenzel
Copy link
Contributor Author

Needs AliceO2Group/AliceO2#12899 and AliceO2Group/AliceO2#12891 to work properly.

@sawenzel
Copy link
Contributor Author

+async-label async-2022-pp-apass6

Development allowing event generation to run in a separate task
from detector simulation. In this mode, event generation will write
kinematics to disc, which is later picked up by o2-sim.

Development done in order to increase CPU efficiency in cases
where event generation was slow and not being able to serve the workers
of o2-sim with enought work. The development fixes such cases, because
multiple event generators (for different timeframes) can now run in parallel.

The new mode is the default, but we can return to the intregrated mode
by using option `--event-gen-mode integrated`.
@sawenzel sawenzel force-pushed the swenzel/eventgendecoupling branch from 2ff54f6 to 2139f26 Compare March 22, 2024 16:47
@sawenzel
Copy link
Contributor Author

Also needs AliceO2Group/AliceO2#12920

@sawenzel sawenzel merged commit 3b4451e into AliceO2Group:master Mar 26, 2024
6 checks passed
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.

2 participants