Skip to content

Conversation

@sawankumawat
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

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

@dmallick2
Copy link
Collaborator

@sawankumawat please fix o2linter issue

@sawankumawat
Copy link
Contributor Author

sawankumawat commented Jan 7, 2026

@sawankumawat please fix o2linter issue
@dmallick2
In the code, I apply pT-dependent PID, which requires using different nσ cut values at different pT intervals for both kaons and pions.
The O2 linter gives the error: Assign the value to a clearly named variable or constant.
However, assigning a name to every numeric cut value will create confusion and reduce the readability of the code, especially when many pT and PID ranges are used.

@dmallick2 dmallick2 merged commit 5422ab6 into AliceO2Group:master Jan 7, 2026
11 of 12 checks passed
sigurdnese pushed a commit to sigurdnese/O2Physics that referenced this pull request Jan 9, 2026
Co-authored-by: Sawan Sawan <sawan.sawan@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.

2 participants