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

Update AliDecayerPythia8.cxx #1511

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update AliDecayerPythia8.cxx #1511

wants to merge 4 commits into from

Conversation

ddobrigk
Copy link
Contributor

@ddobrigk ddobrigk commented Feb 1, 2024

No description provided.

@alibuild
Copy link
Collaborator

alibuild commented Feb 1, 2024

f593ee4: approval required: 1 of @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@alibuild
Copy link
Collaborator

alibuild commented Feb 1, 2024

efa2ce4: approval required: 1 of @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@alibuild
Copy link
Collaborator

alibuild commented Feb 1, 2024

6ed31fe: approval required: 1 of @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@ddobrigk
Copy link
Contributor Author

ddobrigk commented Feb 1, 2024

This suggested change is such that the code now follows the revised ALICE primary particle definition regarding what to decay and what not to decay, avoiding the risk of forgetting to disable / enable any particles manually

@alibuild
Copy link
Collaborator

alibuild commented Feb 1, 2024

4981017: approval required: 1 of @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

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