Skip to content

[PWGLF] Improvements and bug fix in phianalysisTHnSparse task#12901

Merged
dmallick2 merged 1 commit intoAliceO2Group:masterfrom
vbarbaso:master
Sep 8, 2025
Merged

[PWGLF] Improvements and bug fix in phianalysisTHnSparse task#12901
dmallick2 merged 1 commit intoAliceO2Group:masterfrom
vbarbaso:master

Conversation

@vbarbaso
Copy link
Copy Markdown
Contributor

@vbarbaso vbarbaso commented Sep 8, 2025

Improvements in phianalysisTHnSparse.cxx task

  • Removed broken compile-time filters to enable configurable cuts
  • Implemented selectedTrack() with runtime cut evaluation and charge-dependent logic
  • Added configurable GlobalTrack vs. PrimaryTrack selection option
  • Introduced TPC+TOF PID option (when TOF is available)
  • Fixed MC generator daughter validation logic
  • Enhanced QA histograms
  • Replaced TLorentzVector with ROOT-recommended alternative
  • Performed minor code cleanup and improved variable naming

Signed-off-by: Veronika Barbasova <[email protected]>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 8, 2025

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

@dmallick2 dmallick2 enabled auto-merge (squash) September 8, 2025 11:02
@dmallick2 dmallick2 merged commit 87063e3 into AliceO2Group:master Sep 8, 2025
14 of 15 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants