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

PWGGA/PHOSTasks: added new task for neutral meson and photon analyses #23346

Merged
merged 4 commits into from
Apr 9, 2024

Conversation

vlad-kuskov
Copy link
Contributor

This task largely based on AliAnalysisTaskPHOSPi0EtaToGammaGamma and contains:

  • histograms for pi0, eta, photons analyses in Run 2;
  • QA histograms for PID study, TOF cut study, trigger performance study;
  • a major change compared to AliAnalysisTaskPHOSPi0EtaToGammaGamma: weighs (TOF/trigger) are applied to MC and not to data

@alibuild
Copy link
Collaborator

alibuild commented Apr 8, 2024

a5f182d: approval required: 1 of @FriederikeBock (Friederike Bock), @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @gconesab (Gustavo Conesa Balbastre), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @kharlov (Yuri Kharlov), @ktf (Giulio Eulisse), @loizides (Constantinos Loizides), @mikesas (Mike Henry Petrus Sas), @nschmidtALICE (Nicolas Vincent Schmidt), @peressounko (Dmitri Peresunko), @policheh (Boris Polishchuk), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@alibuild
Copy link
Collaborator

alibuild commented Apr 8, 2024

6784f45: approval required: 1 of @FriederikeBock (Friederike Bock), @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @gconesab (Gustavo Conesa Balbastre), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @kharlov (Yuri Kharlov), @ktf (Giulio Eulisse), @loizides (Constantinos Loizides), @mikesas (Mike Henry Petrus Sas), @nschmidtALICE (Nicolas Vincent Schmidt), @peressounko (Dmitri Peresunko), @policheh (Boris Polishchuk), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@peressounko
Copy link
Contributor

+1

@alibuild
Copy link
Collaborator

alibuild commented Apr 9, 2024

6784f45: approved: will be automatically merged on successful tests

@alibuild
Copy link
Collaborator

alibuild commented Apr 9, 2024

Error while checking build/AliPhysics/root6 for 6784f45 at 2024-04-09 12:48:

## sw/BUILD/AliPhysics-latest/log
/sw/SOURCES/AliPhysics/23346-slc7_x86-64/0/PWGGA/PHOSTasks/PHOS_Run2/AliAnalysisPHOSNeutralMesonsAndPhotons.h:11:10: fatal error: 'AliMCAnalysisUtils.h' file not found
Error: /sw/slc7_x86-64/ROOT/v6-30-01-alice4-local4/bin/rootcint: compilation failure (/sw/BUILD/875bc316733e78c9bc00f9dab7d7866aebac67a4/AliPhysics/PWGGA/PHOSTasks/G__PWGGAPHOSTasks4faabf84f9_dictUmbrella.h)
gmake[2]: *** [PWGGA/PHOSTasks/libPWGGAPHOSTasks.rootmap] Error 1
gmake[1]: *** [PWGGA/PHOSTasks/CMakeFiles/PWGGAPHOSTasks-object.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....
gmake: *** [all] Error 2

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Apr 9, 2024

6416260: approval required: 1 of @FriederikeBock (Friederike Bock), @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @gconesab (Gustavo Conesa Balbastre), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @kharlov (Yuri Kharlov), @ktf (Giulio Eulisse), @loizides (Constantinos Loizides), @mikesas (Mike Henry Petrus Sas), @nschmidtALICE (Nicolas Vincent Schmidt), @peressounko (Dmitri Peresunko), @policheh (Boris Polishchuk), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

@kharlov
Copy link
Contributor

kharlov commented Apr 9, 2024

+1

@alibuild
Copy link
Collaborator

alibuild commented Apr 9, 2024

6416260: approved: will be automatically merged on successful tests

@alibuild alibuild merged commit bc3a6af into alisw:master Apr 9, 2024
2 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
4 participants