Offense fsm skel #1755
sid-parikhci.yml
on: pull_request
build-and-test
13m 21s
fix-style
1m 31s
Annotations
10 errors and 25 warnings
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L107
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L131
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L158
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L182
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L206
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L230
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L275
Expected: (failure_count) < (50), actual: 500 vs 50
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L299
Value of: check_trajectory_continuous(path, RobotConstraints{})
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L325
Value of: !path.empty()
|
|
build-and-test:
soccer/src/soccer/planning/tests/planner_test.cpp#L371
Expected: (failure_count) < (100), actual: 500 vs 100
|
|
fix-style
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/.
|
|
fix-style
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/
|
|
fix-style
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
build-and-test
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/.
|
|
build-and-test
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-and-test
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
build-and-test
unused variable 'n'
|
|
build-and-test
field '_kdtree' will be initialized after field '_dimensions'
|
|
build-and-test
field '_dimensions' will be initialized after field '_nodemap'
|
|
build-and-test
field '_parent' will be initialized after field '_state'
|
|
build-and-test
field '_state' will be initialized after field '_vec'
|
|
build-and-test
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
build-and-test
unused variable 'n'
|
|
build-and-test
'mousePressEvent' overrides a member function but is not marked 'override'
|
|
build-and-test
'mouseMoveEvent' overrides a member function but is not marked 'override'
|
|
build-and-test:
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
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L26
function 'next_state' has cognitive complexity of 40 (threshold 25)
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L120
do not use 'else' after 'return'
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L227
narrowing conversion from 'double' to 'float'
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L361
statement should be inside braces
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L444
use range-based for loop instead
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L92
switch has 5 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L108
switch has 2 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L160
function 'timed_out' should be marked [[nodiscard]]
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L202
function 'has_open_shot' should be marked [[nodiscard]]
|