[DPG] adding new QA task to monitor outliers and pileup in OO and NeNe #14845
altsybeeAnnotations
10 errors and 2 notices
|
[magic-number]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L968
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L739
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L601
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L403
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[std-prefix]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L1212
Use std:: prefix for names from the std namespace.
|
|
[std-prefix]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L1188
Use std:: prefix for names from the std namespace.
|
|
[std-prefix]:
DPG/Tasks/AOTEvent/eventSelectionQa.cxx#L1176
Use std:: prefix for names from the std namespace.
|
|
[name/o2-workflow]:
DPG/Tasks/AOTEvent/CMakeLists.txt#L1
Use kebab-case for names of workflows and match the name of the workflow file.
|
|
[name/o2-workflow]:
DPG/Tasks/AOTEvent/CMakeLists.txt#L52
Workflow name occupancy-vs-dedx-qa does not match its file name dEdxVsOccupancyWithTrackQAinfo.cxx. (Matches occupancyVsDedxQa.cxx.)
|
|
[name/o2-workflow]:
DPG/Tasks/AOTEvent/CMakeLists.txt#L47
Workflow name light-ions-evsel-qa does not match its file name lightIonsEvSelQa.cxx. (Matches lightIonsEvselQa.cxx.)
|
|
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.
|