Add replacement config to generator_pythia8 to replace D*/Sigmac+ by Xic0/Omegac0#2186
Add replacement config to generator_pythia8 to replace D*/Sigmac+ by Xic0/Omegac0#2186jackal1-66 merged 5 commits intoAliceO2Group:masterfrom
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
fgrosa
left a comment
There was a problem hiding this comment.
Hi @FDUEnrich thanks a lot for the development!
I have a few comments (see below) and in addition you need to add the test macro corresponding to the new ini file in https://github.com/AliceO2Group/O2DPG/tree/master/MC/config/PWGHF/ini/tests (it is required for every ini file).
MC/config/PWGHF/ini/GeneratorHF_D2H_ccbar_and_bbbar_PbPb_replaced.ini
Outdated
Show resolved
Hide resolved
MC/config/PWGHF/ini/GeneratorHF_D2H_ccbar_and_bbbar_PbPb_replaced.ini
Outdated
Show resolved
Hide resolved
MC/config/PWGHF/pythia8/generator/pythia8_charmhadronic_decays_Mode2_hardQCD_5TeV_XicOmegaC.cfg
Outdated
Show resolved
Hide resolved
|
@klsmith15k apparently the test macro for GeneratorHF_D2H_ccbar_and_bbbar_gap5_Mode2_CharmBaryons_pp_ref.ini that you committed is missing. Can you please implement it and open a PR? Thanks! |
|
Hi @jackal1-66 @sawenzel would it be possible to restart the tests for this PR? The error should have been solved in #2199. Many thanks! |
|
Hello, there are few tests still failing: and Could you please check them out? |
|
Also I checked a bit further and noticed that these tests: Generate more than 2000 Pythia events to complete. Could you please add the line |
Hi @jackal1-66 thanks a lot for the suggestion! I reduced the number of test events in this PR #2216. I also fixed the ones crashing, and reduced even more than 10 the number for few |
|
The CI is still red on GitHub, however it's a timeout issue. In particular the list of generators taking a long time is: ordered timewise. |
Hi @jackal1-66 thanks a lot! Indeed we should find a way to perform the tests, without being limited by the timeout in case many configurations need to be tested. Let's follow this up for the future. |
FYI #2250 |
No description provided.