-
Notifications
You must be signed in to change notification settings - Fork 148
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
PWGHF: remove obsolete generator to avoid confusion #1742
Conversation
REQUEST FOR PRODUCTION RELEASES:
This will add The following labels are available |
Hi @fgrosa ! Thanks for the PR. For me, it is fine to remove this generator since it's quite old and I don't think it is used anymore. However, I would suggest removing the |
Good point @stefanopolitano! I removed another bunch of obsolete files on top of what you suggested. |
Hi @lucamicheletti93 @ffionda @crisjahnke we noticed that the generator https://github.com/AliceO2Group/O2DPG/blob/master/MC/config/PWGHF/external/generator/GeneratorHF.C that I am removing is used in DQ (https://github.com/AliceO2Group/O2DPG/blob/master/MC/config/PWGDQ/external/generator/GeneratorCharmToMu_EvtGen.C). Please note that this is obsolete, and, in case you need a generator for Pb-Pb from HF you should use this one https://github.com/AliceO2Group/O2DPG/blob/master/MC/config/PWGHF/external/generator/generator_pythia8_embed_hf.C. We will anyway don't merge this PR until you confirm us that it's good for you! |
Hi @lucamicheletti93 ! Thanks for the update, I don't have any further comment. Approved! |
* PWGHF: remove obsolete generator to avoid confusion * Remove other obsolete files (cherry picked from commit 9337da9)
this PR is now included in the tag for the Pb-Pb apass4 that is being built. Label removed |
* PWGHF: remove obsolete generator to avoid confusion * Remove other obsolete files (cherry picked from commit 9337da9)
* PWGHF: remove obsolete generator to avoid confusion * Remove other obsolete files (cherry picked from commit 9337da9)
* PWGHF: remove obsolete generator to avoid confusion * Remove other obsolete files (cherry picked from commit 9337da9)
No description provided.