Skip to content

[PWGJE] V0 QA#12811

Merged
nzardosh merged 2 commits intoAliceO2Group:masterfrom
GijsvWeelden:fragmentation
Sep 1, 2025
Merged

[PWGJE] V0 QA#12811
nzardosh merged 2 commits intoAliceO2Group:masterfrom
GijsvWeelden:fragmentation

Conversation

@GijsvWeelden
Copy link
Contributor

@GijsvWeelden GijsvWeelden commented Aug 31, 2025

Reconstruction efficiency:

  • Fill pT of generated V0s in hists for reconstructed V0s
  • Check rapidity, isPrimary, has_daughters for all generated V0s

Weighted/Subtracted jet finder:

  • Add inclusive V0 hists for debugging purposes
  • Moved hist filling into functions

V0 Track QA:

  • Added event selection
  • Added event histogram

@GijsvWeelden
[PWGJE] V0 QA
9e787cb 
Reconstruction efficiency:
* Fill pT of generated V0s in hists for reconstructed V0s
* Check rapidity, isPrimary, has_daughters for all generated V0s

Weighted/Subtracted jet finder:
* Add inclusive V0 hists for debugging purposes
* Moved hist filling into functions

V0 Track QA:
* Added event selection
* Added event histogram
@github-actions github-actions bot added the pwgje label Aug 31, 2025
@github-actions
Copy link

github-actions bot commented Aug 31, 2025 •
edited
Loading

O2 linter results: ❌ 1 errors, ⚠️ 0 warnings, 🔕 0 disabled

@alibuild
Copy link
Collaborator

alibuild commented Aug 31, 2025

Error while checking build/O2Physics/o2 for 9e787cb at 2025-09-01 00:35:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/12811-slc9_x86-64/0/PWGJE/Tasks/v0QA.cxx:512:10: error: 'v0' was not declared in this scope; did you mean 'o2::aod::v0'?
/sw/SOURCES/O2Physics/12811-slc9_x86-64/0/PWGJE/Tasks/v0QA.cxx:510:43: error: unused parameter 'pv0' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/12811-slc9_x86-64/0/PWGJE/Tasks/v0QA.cxx:565:50: error: unused parameter 'v0' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/12811-slc9_x86-64/0/PWGJE/Tasks/v0QA.cxx:598:74: error: unused parameter 'v0' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@nzardosh nzardosh enabled auto-merge (squash) September 1, 2025 09:08
@nzardosh nzardosh merged commit 4eee3ed into AliceO2Group:master Sep 1, 2025
12 of 14 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
@GijsvWeelden @jinhyunni
[PWGJE] V0 QA (AliceO2Group#12811)
d993def
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
@GijsvWeelden @ThePhDane
[PWGJE] V0 QA (AliceO2Group#12811)
04fc8fd
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@GijsvWeelden @jloemker
[PWGJE] V0 QA (AliceO2Group#12811)
cb56429
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
@GijsvWeelden
[PWGJE] V0 QA (AliceO2Group#12811)
3940a98
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nzardosh nzardosh nzardosh approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@maoyx maoyx Awaiting requested review from maoyx maoyx is a code owner

@fjonasALICE fjonasALICE Awaiting requested review from fjonasALICE fjonasALICE is a code owner

@mfasDa mfasDa Awaiting requested review from mfasDa mfasDa is a code owner

@mhemmer-cern mhemmer-cern Awaiting requested review from mhemmer-cern mhemmer-cern is a code owner

Assignees

No one assigned

Labels

pwgje

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@GijsvWeelden @alibuild @nzardosh

Comments