Skip to content

[PWGLF] Fix strangenesstofpid converter#16560

Merged
romainschotter merged 3 commits into
masterfrom
romainschotter-patch-1
Jun 8, 2026
Merged

[PWGLF] Fix strangenesstofpid converter#16560
romainschotter merged 3 commits into
masterfrom
romainschotter-patch-1

Conversation

@romainschotter

Copy link
Copy Markdown
Collaborator
  • fill DauTrackTOFPIDs_002 table with available columns in DauTrackTOFPIDs_001
  • add safety in the strangenesstofpid to prevent heap corruption and warn the user (even though it could intentional in case using specific converters)

@github-actions github-actions Bot added the pwglf label Jun 8, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fix strangenesstofpid converter [PWGLF] Fix strangenesstofpid converter Jun 8, 2026
romainschotter and others added 2 commits June 8, 2026 07:59
Added a check for valid dauTrackExtraId before indexing and logged a warning if invalid.

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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 @romainschotter.

@romainschotter romainschotter merged commit d2e5b3d into master Jun 8, 2026
15 of 17 checks passed
@romainschotter romainschotter deleted the romainschotter-patch-1 branch June 8, 2026 06:52
fjonasALICE pushed a commit to fjonasALICE/O2Physics that referenced this pull request Jun 9, 2026
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