Rewrite Offense Position #1715
Triggered via pull request
February 12, 2024 20:44
sid-parikh
Status
Failure
Total duration
12m 20s
Artifacts
–
ci.yml
on: pull_request
build-and-test
12m 13s
fix-style
1m 58s
Annotations
11 errors and 29 warnings
|
fix-style
Process completed with exit code 1.
|
|
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, peter-evans/create-pull-request@v3. 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, peter-evans/create-pull-request@v3. 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/
|
|
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/
|
|
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/
|
|
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/
|
|
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.hpp#L86
switch has 5 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L96
switch has 4 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L104
switch has 2 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L132
switch has 2 consecutive identical branches
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L150
function 'timed_out' should be marked [[nodiscard]]
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L190
function 'has_open_shot' should be marked [[nodiscard]]
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L195
function 'distance_from_their_robots' should be marked [[nodiscard]]
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L200
function 'can_steal_ball' should be marked [[nodiscard]]
|
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.hpp#L205
function 'distance_to_ball' should be marked [[nodiscard]]
|