Offense fsm skel #1753
Annotations
10 errors and 22 warnings
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L107
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L131
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L158
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L182
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L206
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L230
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L275
Expected: (failure_count) < (50), actual: 500 vs 50
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L299
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L325
Value of: !path.empty()
|
|
Test:
soccer/src/soccer/planning/tests/planner_test.cpp#L371
Expected: (failure_count) < (100), actual: 500 vs 100
|
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Build
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
Build
unused variable 'n'
|
|
Build
field '_kdtree' will be initialized after field '_dimensions'
|
|
Build
field '_dimensions' will be initialized after field '_nodemap'
|
|
Build
field '_parent' will be initialized after field '_state'
|
|
Build
field '_state' will be initialized after field '_vec'
|
|
Build
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
Build
unused variable 'n'
|
|
Build
'mousePressEvent' overrides a member function but is not marked 'override'
|
|
Build
'mouseMoveEvent' overrides a member function but is not marked 'override'
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/agent_action_client.cpp#L46
the parameter 'msg' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.cpp#L27
function 'next_state' has cognitive complexity of 40 (threshold 25)
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.cpp#L121
do not use 'else' after 'return'
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.cpp#L229
narrowing conversion from 'double' to 'float'
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.cpp#L362
statement should be inside braces
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.cpp#L445
use range-based for loop instead
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.hpp#L92
switch has 5 consecutive identical branches
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.hpp#L108
switch has 2 consecutive identical branches
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.hpp#L160
function 'timed_out' should be marked [[nodiscard]]
|
|
Run clang-tidy:
soccer/src/soccer/strategy/agent/position/offense.hpp#L202
function 'has_open_shot' should be marked [[nodiscard]]
|
The logs for this run have expired and are no longer available.
Loading