Skip to content

Conversation

@vbarbaso
Copy link
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 <vernika.barbasova@cern.ch>
@github-actions
Copy link

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
…2Group#12901)

Signed-off-by: Veronika Barbasova <vernika.barbasova@cern.ch>
Co-authored-by: Veronika Barbasova <vernika.barbasova@cern.ch>
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
…2Group#12901)

Signed-off-by: Veronika Barbasova <vernika.barbasova@cern.ch>
Co-authored-by: Veronika Barbasova <vernika.barbasova@cern.ch>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
…2Group#12901)

Signed-off-by: Veronika Barbasova <vernika.barbasova@cern.ch>
Co-authored-by: Veronika Barbasova <vernika.barbasova@cern.ch>
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
…2Group#12901)

Signed-off-by: Veronika Barbasova <vernika.barbasova@cern.ch>
Co-authored-by: Veronika Barbasova <vernika.barbasova@cern.ch>
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