Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

@ddobrigk ddobrigk commented Sep 18, 2025

Optimization changes:

  • When including photon processing, the V0 building done to evaluate duplicates was propagating prongs with XIU = 83.1f to the PV to calculate the DCA to PV, incurring in a large CPU cost for obtaining a variable that isn't checked in the de-duplication process. This PR makes it such that this propagation is skipped when constructing V0s in the de-duplication procedure, but kept for all other use cases, keeping default operation unchanged. Will lead to a sizable improvement in performance when processing photons. Tagging @gianniliveraro
  • Switches the propagate-to-PV call for V0 prongs to be done only on TrackPar (instead of TrackParCov) to avoid that multiple scattering contributions are calculated needlessly as only DCAxy is used, further saving some CPU.

@github-actions github-actions bot added the pwglf label Sep 18, 2025
@github-actions
Copy link

github-actions bot commented Sep 18, 2025

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

romainschotter
romainschotter previously approved these changes Sep 26, 2025
@romainschotter romainschotter enabled auto-merge (squash) September 26, 2025 21:32
@romainschotter romainschotter merged commit e66b9a4 into AliceO2Group:master Sep 28, 2025
12 of 14 checks passed
jmunozme pushed a commit to jmunozme/O2Physics that referenced this pull request Oct 3, 2025
…#13031)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
…#13031)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
…#13031)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
…#13031)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
…#13031)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
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