Skip to content

Added alien path compatibility, Phi rotation and event pool randomisation to GenO2Kine#13571

Merged
sawenzel merged 22 commits intoAliceO2Group:devfrom
jackal1-66:eventpool
Oct 31, 2024
Merged

Added alien path compatibility, Phi rotation and event pool randomisation to GenO2Kine#13571
sawenzel merged 22 commits intoAliceO2Group:devfrom
jackal1-66:eventpool

Conversation

@jackal1-66
Copy link
Copy Markdown
Collaborator

This PR is related to the ticket https://its.cern.ch/jira/browse/O2-5216
It has been tested locally and the feature works as expected.
Check the ticket for more information.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 7, 2024

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-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0

@jackal1-66
Copy link
Copy Markdown
Collaborator Author

This PR is strictly connected to AliceO2Group/O2DPG#1760

Comment thread run/o2sim_parallel.cxx Outdated
Comment thread Generators/src/GeneratorFromFile.cxx Outdated
@sawenzel
Copy link
Copy Markdown
Collaborator

Can you also change the title of the PR and remove the WIP?

@jackal1-66 jackal1-66 changed the title [WIP] Implemented flag for .root kinematics input Added alien path compatibility, Phi rotation and event pool randomisation to GenO2Kine Oct 29, 2024
This reverts commit 2d3b692, reversing
changes made to c9a5dad.
@jackal1-66 jackal1-66 requested a review from shahor02 as a code owner October 30, 2024 08:50
@jackal1-66
Copy link
Copy Markdown
Collaborator Author

There's a bit of a mess with the commits. Please DO NOT MERGE until I've sorted them out.

@jackal1-66
Copy link
Copy Markdown
Collaborator Author

Problem fixed. The merging can go forward.

Comment thread Generators/src/GeneratorFromFile.cxx
@sawenzel sawenzel merged commit 8ec256c into AliceO2Group:dev Oct 31, 2024
@alcaliva alcaliva added the async-2024-pp-apass1 Request porting to async-2024-pp-apass1 label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async-2024-pp-apass1 Request porting to async-2024-pp-apass1

Development

Successfully merging this pull request may close these issues.

3 participants