Skip to content

Conversation

@wuctlby
Copy link
Contributor

@wuctlby wuctlby commented Oct 10, 2025

Hi @fgrosa ! This PR adds the corresponding config and test files for OO collisions.

  • GeneratorHF_D2H_ccbar_and_bbbar_gap2_SCCR_OO.ini points to pythia8_charmhadronic_with_decays_SCCR_OO.cfg with a gap trigger of 2
  • pythia8_charmhadronic_with_decays_SCCR_OO.cfg used the SC-CR tune from Christiansen.
  • In GeneratorHF_D2H_ccbar_and_bbbar_gap2_SCCR_OO.C, I moved the path, which points to "o2sim_Kine.root", to be the parameter of the function External. This makes it more convenient for local testing. But let's see if it breaks the build test.
    Please let me know if you have any suggestions or comments!

Also tagging @mfaggin and @stefanopolitano

@wuctlby wuctlby requested a review from a team as a code owner October 10, 2025 11:07
@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

@wuctlby wuctlby enabled auto-merge (squash) October 10, 2025 14:07
@wuctlby
Copy link
Contributor Author

wuctlby commented Oct 10, 2025

Hi @fgrosa, the build test was successful. If there are no further comments, could you please approve this PR? Because I cannot approve my own PR.

@fgrosa
Copy link
Contributor

fgrosa commented Oct 10, 2025

Thanks @wuctlby!

@wuctlby wuctlby merged commit 58270d6 into AliceO2Group:master Oct 10, 2025
8 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