Skip to content

Conversation

@alcaliva
Copy link
Collaborator

No description provided.

@github-actions
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-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@alcaliva alcaliva enabled auto-merge (squash) January 29, 2026 15:25
@alcaliva
Copy link
Collaborator Author

alcaliva commented Jan 29, 2026

@ddobrigk ,
it looks like we don't have pO for EPOS4. Shall we add it?

@ddobrigk
Copy link
Collaborator

@ddobrigk , it looks like we don't have pO for EPOS4. Shall we add it?

Hi Alberto, super thanks! I think it would be useful, though likely the highest priority should probably be pp, OO and Pb-Pb IMHO.

@alcaliva
Copy link
Collaborator Author

Hi @jackal1-66,
the build fails and I get the following error message:

4-No particles marked to be transported. Make sure they are marked correctly (see https://aliceo2group.github.io/simulation/docs/generators/).

Do you have any clue of what I should modify? I simply adapted the existing code for PbPb to light ions.

@jackal1-66
Copy link
Collaborator

Ciao Alberto, there are two issues in this:
1 - There is a problem with NeNe, this is on me. Since it is not super urgent by David, I would suggest you to drop the ini for the moment (I need to get in touch with the EPOS4 developers).
2 - constexpr double kOxygenEnergy = kA * kEnucleon; // 85760 GeV this should be halved: each Oxygen ion will have around 85760/2 GeV. This should fix the remaining issue.

@jackal1-66
Copy link
Collaborator

alisw/alidist#6109 I included a fix in EPOS4 for the NeNe collision system after discussing with Klaus Werner on the correct values to set in one of the tables. After this is merged your test should succeed

@alcaliva
Copy link
Collaborator Author

Hi @jackal1-66,
thanks! I fixed the oxygen energy.

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