Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

  • adds a pidTPCModule class similar to other classes that deal with core services
  • this class is used in a pidTPCService test task that provides Nsigma values to users in the exact same manner (with identical output at least in the default mode). This task is able to switch between Tracks and TracksIU for testing differences due to use of Tracks vs TracksIU with existing calibrations in an easier way.
  • While previously the regular TPC PID required 3 DPL devices (one multiplicity counter, one corrected dE/dx device and one actual PID calculation task), this new refactored version has only one DPL device.

The main practical consequence of this refactoring is that the memory footprint ('dirty private') decreased from 213 MB to 139 MB (-35%) and the PSS in a typical Pb-Pb test scenario went from approximately 500MB to 320MB (-35%). Further optimizations are still pending.

@github-actions
Copy link

github-actions bot commented Jul 27, 2025

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

@ddobrigk ddobrigk marked this pull request as ready for review July 30, 2025 06:55
@ddobrigk ddobrigk requested review from a team, alibuild, iarsene, jgrosseo and ktf as code owners July 30, 2025 06:55
@ddobrigk ddobrigk enabled auto-merge (squash) July 30, 2025 10:40
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk merged commit b148970 into AliceO2Group:master Jul 30, 2025
10 of 12 checks passed
jpxrk pushed a commit to jpxrk/O2Physics that referenced this pull request Aug 12, 2025
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
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
alibuild added a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
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.

2 participants