[PWGLF] hybrid generator alternating INEL and jet+rope events

[alcaliva]

No description provided.

[github-actions]

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]

Hi @jackal1-66, hope I got it right. Please have a look.
Thanks

[jackal1-66]

Please do not auto-merge this. I want to run additional tests and possibly merge first another development I've been working on.

[alcaliva]

Hi Marco, thanks! Is this development supposed to fix the crush?

[jackal1-66]

Ciao @alcaliva , no my development will not fix the error you're experiencing in the CI. From the log, it seems that the Pythia8 configuration is not properly forwarded to the generator, making it crash

[jackal1-66]

#2223 this will allow O2DPG hybrid generators to be tested directly by the CI. When this is merged GeneratorLFStrangenessInJetsTriggered_gap4.C should be changed to integrate the Hybrid() function.

[jackal1-66]

Ciao @alcaliva , there are two issues with the PR:

1. The Pythia8 configuration when the external generator is used is not there anymore. You should keep

[GeneratorPythia8] # if triggered then this will be used as the background event
config=${O2DPG_MC_CONFIG_ROOT}/MC/config/PWGLF/pythia8/generator/pythia8_jet_ropes_136tev.cfg

before the Hybrid parameters setting
2. The hybrid test is not setup yet. The PR for the hybrid testing integration will be merged very soon, so I suggest you implement that part as well. It's fairly straightforward: you simply need to include in the MC/config/PWGLF/ini/tests/GeneratorLFStrangenessInJetsTriggered_gap4.C test file the function int Hybrid(). If the test is the same as the external one, you could simply add return External() inside of it. Let me know if you need any support for this.

[alcaliva]

Hi @jackal1-66 ,

to your first comment, shall I replace the current ini file with the following?

[GeneratorExternal]
fileName=${O2DPG_MC_CONFIG_ROOT}/MC/config/PWGLF/pythia8/generator_pythia8_LF.C
funcName=generateLFTriggered("${O2DPG_MC_CONFIG_ROOT}/MC/config/PWGLF/pythia8/generator/strangeparticlelist.gun", 4)

[GeneratorPythia8] # if triggered then this will be used as the background event
configFile=${O2DPG_MC_CONFIG_ROOT}/MC/config/PWGLF/pythia8/generator/pythia8_jet_ropes_136tev.cfg

[DecayerPythia8]
config[0]=${O2DPG_MC_CONFIG_ROOT}/MC/config/common/pythia8/decayer/base.cfg

[jackal1-66]

Ciao @alcaliva , no what I mean is that instead of removing the [GeneratorPythia8] part as you're currently doing, you should simply add

[GeneratorHybrid]
configFile=${O2DPG_MC_CONFIG_ROOT}/MC/config/common/external/generator/pythia8_inel_plus_jets_ropes_13p6TeV.json

to the ini file. From the commits I see you're instead replacing [GeneratorPythia8].

[alcaliva]

I decided to implement a bit different logic for now. Closing this PR