Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

@ddobrigk ddobrigk commented Sep 8, 2025

This PR:

  • Saves approximately 10% disk space in derived data by:
    • detaching TOF information from daughter track information to save disk space and links the new table with an index
    • compresses TPC Nsigma into tiny format (from -6.35 to +6.35 in 0.05-sized increments) while keeping the interface unchanged for a smooth transition
  • Provides converters to keep existing data usable
  • Cleans up the strangenesstofpid further and also alters the second processing algorithm to use adjusted primary TOF information as opposed to a standalone calculation, effectively leading to e-loss-level performance as reported in the PAG but with negligible CPU overhead. The adjustment versus primary TOF Nsigmas has been tested to lead to an Nsigma distribution centered around zero, which would not be the case if the primary-particle Nsigmas were used. More testing to take place in continuation.

Tagging @romainschotter @ercolessi @gianniliveraro @njacazio

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

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

romainschotter
romainschotter previously approved these changes Sep 8, 2025
@romainschotter romainschotter enabled auto-merge (squash) September 8, 2025 15:33
romainschotter
romainschotter previously approved these changes Sep 8, 2025
@ddobrigk ddobrigk disabled auto-merge September 9, 2025 10:48
@romainschotter romainschotter enabled auto-merge (squash) September 9, 2025 19:21
@romainschotter romainschotter merged commit 0ae609c into AliceO2Group:master Sep 11, 2025
11 of 14 checks passed
jmunozme pushed a commit to jmunozme/O2Physics that referenced this pull request Oct 3, 2025
…id cleanup (AliceO2Group#12912)

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
…id cleanup (AliceO2Group#12912)

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
…id cleanup (AliceO2Group#12912)

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
…id cleanup (AliceO2Group#12912)

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
…id cleanup (AliceO2Group#12912)

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

Development

Successfully merging this pull request may close these issues.

3 participants