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

PWGHF: remove obsolete generator to avoid confusion #1742

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

fgrosa
Copy link
Contributor

@fgrosa fgrosa commented Sep 10, 2024

No description provided.

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-apass3
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0

@stefanopolitano
Copy link
Contributor

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 .ini file which points to it (https://github.com/AliceO2Group/O2DPG/blob/master/MC/config/PWGHF/ini/GeneratorHF.ini) and the .sh example for embedded simulations (https://github.com/AliceO2Group/O2DPG/blob/master/MC/run/PWGHF/run_embedding.sh) that points to the .ini file, as well.

@fgrosa
Copy link
Contributor Author

fgrosa commented Sep 10, 2024

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 .ini file which points to it (https://github.com/AliceO2Group/O2DPG/blob/master/MC/config/PWGHF/ini/GeneratorHF.ini) and the .sh example for embedded simulations (https://github.com/AliceO2Group/O2DPG/blob/master/MC/run/PWGHF/run_embedding.sh) that points to the .ini file, as well.

Good point @stefanopolitano! I removed another bunch of obsolete files on top of what you suggested.

@fgrosa fgrosa added async-2023-pbpb-apass4 Request porting to async-2023-pbpb-apass4 async-2023-pp-apass4 Request porting to async-2023-pp-apass4 async-2024-pp-apass1 Request porting to async-2024-pp-apass1 async-2022-pp-apass7 Request porting to async-2022-pp-apass7 labels Sep 10, 2024
@fgrosa
Copy link
Contributor Author

fgrosa commented Sep 10, 2024

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!

@lucamicheletti93
Copy link
Contributor

@fgrosa thanks a lot! I checked with @ffionda and we confirm that it is fine for us. We are not really using it, and in case we will upgrade to the new one.

@stefanopolitano
Copy link
Contributor

Hi @lucamicheletti93 ! Thanks for the update, I don't have any further comment. Approved!

@stefanopolitano stefanopolitano merged commit 9337da9 into AliceO2Group:master Sep 10, 2024
6 checks passed
alcaliva pushed a commit that referenced this pull request Sep 27, 2024
* PWGHF: remove obsolete generator to avoid confusion

* Remove other obsolete files

(cherry picked from commit 9337da9)
@alcaliva alcaliva removed the async-2023-pbpb-apass4 Request porting to async-2023-pbpb-apass4 label Sep 27, 2024
@alcaliva
Copy link
Collaborator

this PR is now included in the tag for the Pb-Pb apass4 that is being built. Label removed

alcaliva pushed a commit that referenced this pull request Sep 27, 2024
* PWGHF: remove obsolete generator to avoid confusion

* Remove other obsolete files

(cherry picked from commit 9337da9)
@alcaliva alcaliva removed the async-2024-pp-apass1 Request porting to async-2024-pp-apass1 label Oct 8, 2024
alcaliva pushed a commit that referenced this pull request Oct 17, 2024
* PWGHF: remove obsolete generator to avoid confusion

* Remove other obsolete files

(cherry picked from commit 9337da9)
@alcaliva alcaliva removed the async-2022-pp-apass7 Request porting to async-2022-pp-apass7 label Oct 17, 2024
alcaliva pushed a commit that referenced this pull request Oct 18, 2024
* PWGHF: remove obsolete generator to avoid confusion

* Remove other obsolete files

(cherry picked from commit 9337da9)
@alcaliva alcaliva removed the async-2023-pp-apass4 Request porting to async-2023-pp-apass4 label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants