[DPG/AOTTrack/qaMatchEff] Add kNoTISROFrameBorder condition to study margin effects and linter corrections #19590
Triggered via pull request
October 29, 2025 15:31
mapalhares
synchronize
#13514
Status
Failure
Total duration
18s
Artifacts
–
Annotations
10 errors and 2 notices
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1779
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1762
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1745
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1728
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1711
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1694
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1624
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[pdg/explicit-code]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L3134
Avoid hard-coded PDG codes. Use named values from PDG_t or o2::constants::physics::Pdg instead.
|
|
[two-pi-add-subtract]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1614
Use RecoDecay::constrainAngle to restrict angle to a given range.
|
|
[two-pi-add-subtract]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1611
Use RecoDecay::constrainAngle to restrict angle to a given range.
|
|
O2 linter
To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".
|
|
O2 linter
Exceptionally, you can disable a test for a line by adding a comment with "o2-linter: disable=" followed by the name of the test and parentheses with a reason for the exception.
|