Planner State Simplification #1582
kfu02
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
ci.yml
on: pull_request
build-and-test
16m 7s
fix-style
1m 33s
Annotations
12 errors and 22 warnings
|
fix-style
Process completed with exit code 141.
|
|
build-and-test
Process completed with exit code 2.
|
|
build-and-test
'this' argument to member function 'lock' has type 'std::lock_guard<const std::mutex>::mutex_type' (aka 'const std::mutex'), but function is not marked const
|
|
build-and-test
'this' argument to member function 'unlock' has type 'std::lock_guard<const std::mutex>::mutex_type' (aka 'const std::mutex'), but function is not marked const
|
|
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L25
member initializer 'PathPlanner' does not name a non-static data member or base class
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L26
only virtual member functions can be marked 'override'
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L35
only virtual member functions can be marked 'override'
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L47
only virtual member functions can be marked 'override'
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L48
only virtual member functions can be marked 'override'
|
|
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.hpp#L25
member initializer 'PathPlanner' does not name a non-static data member or base class
|
|
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/
|
|
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
definition of implicit copy assignment operator for 'Circle' is deprecated because it has a user-declared copy constructor
|
|
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:
soccer/src/soccer/planning/global_state.cpp#L8
the parameter 'state' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L15
the parameter 'settings' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L21
the parameter 'goalie' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L28
the parameter 'global_obstacles' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L35
the parameter 'def_area_obstacles' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L42
the parameter 'world_state' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L49
the parameter 'coach_state' is copied for each invocation but only used as a const reference; consider making it a const reference
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L80
return type 'const WorldState' is 'const'-qualified at the top level, which may reduce code readability without improving const correctness
|
|
build-and-test:
soccer/src/soccer/planning/global_state.cpp#L85
return type 'const rj_msgs::msg::CoachState' (aka 'const CoachState_<std::allocator<void>>') is 'const'-qualified at the top level, which may reduce code readability without improving const correctness
|
|
build-and-test:
soccer/src/soccer/planning/planner/escape_obstacles_path_planner.cpp#L2
inclusion of deprecated C++ header 'stddef.h'; consider using 'cstddef' instead
|