Skip to content

[DPG] New method to select primaries - #17248

Merged
njacazio merged 2 commits into
AliceO2Group:masterfrom
rliotino99:method_Rcut_for_primary
Jul 30, 2026
Merged

[DPG] New method to select primaries#17248
njacazio merged 2 commits into
AliceO2Group:masterfrom
rliotino99:method_Rcut_for_primary

Conversation

@rliotino99

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the dpg label Jul 29, 2026
@github-actions github-actions Bot changed the title New method to select primaries [DPG] New method to select primaries Jul 29, 2026
@github-actions

Copy link
Copy Markdown

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

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 6e9b0e7 at 2026-07-29 13:19:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 15 warnings.'
Found 3 errors and 15 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/eab763d8117e435fcf79478a77d957502a908f90/O2Physics-code-check/errors.txt
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:106:16: error: member 'mAbsCosT' of type 'const double' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:106:57: error: no header providing "TMath::DegToRad" is directly included [misc-include-cleaner,-warnings-as-errors]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:107:16: error: member 'mAbsSinT' of type 'const double' is const qualified [cppcoreguidelines-avoid-const-or-ref-data-members,-warnings-as-errors]
++ [[ 15 -gt 0 ]]
++ cat /sw/BUILD/eab763d8117e435fcf79478a77d957502a908f90/O2Physics-code-check/warnings.txt
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:268:48: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:312:76: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:318:80: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:328:53: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:406:40: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:415:49: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:418:54: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:424:21: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:473:40: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:475:44: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:477:44: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:479:11: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:485:25: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:530:61: warning: statement should be inside braces [readability-braces-around-statements]
DPG/Tasks/AOTTrack/PID/HMPID/hmpidTableProducer.cxx:559:61: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

@njacazio
njacazio enabled auto-merge (squash) July 30, 2026 13:16
@njacazio
njacazio merged commit 3204913 into AliceO2Group:master Jul 30, 2026
14 checks passed
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