Adding Seeking #1679
ci.yml
on: pull_request
build-and-test
11m 57s
fix-style
1m 2s
Annotations
23 warnings
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
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/position/seeking.hpp#L26
class 'Seeking' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.hpp#L47
invalid case style for private member 'target_pt'
|
build-and-test:
soccer/src/soccer/strategy/agent/position/offense.cpp#L17
function 'update_state' has cognitive complexity of 37 (threshold 25)
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.cpp#L25
parameter 'field_dimensions' is passed by value and only copied once; consider moving it to avoid unnecessary copies
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.cpp#L31
the parameter 'field_dimensions' 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/seeking.cpp#L70
method 'correct_point' can be made const
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.cpp#L70
the parameter 'field_dimensions' 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/seeking.cpp#L71
invalid case style for variable 'BORDER_BUFFER'
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.cpp#L131
method 'eval_point' can be made static
|
build-and-test:
soccer/src/soccer/strategy/agent/position/seeking.cpp#L144
variable 'curr_dp' is not initialized
|