Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jazzy Sync 4: Dec 13, 2024 #4797

Merged
merged 5 commits into from
Dec 13, 2024
Merged

Jazzy Sync 4: Dec 13, 2024 #4797

merged 5 commits into from
Dec 13, 2024

Conversation

SteveMacenski
Copy link
Member

No description provided.

redvinaa and others added 5 commits December 13, 2024 13:32
* Pass IDLE to on_tick, use that for initialize condition

Signed-off-by: redvinaa <[email protected]>

* Fix battery sub creation bug

Signed-off-by: redvinaa <[email protected]>

---------

Signed-off-by: redvinaa <[email protected]>
…r= (#4753)

default_value_ is an attribute of the Costmap2D class and should be
copied along with the other attributes.

Signed-off-by: Dylan De Coeyer <[email protected]>
* updated CMakeLists.txt to include plugin_container_layer

Signed-off-by: alexander <[email protected]>

* added plugin container layer to costmap_plugins.xml

Signed-off-by: alexander <[email protected]>

* initial commit of plugin container layer

Signed-off-by: alexander <[email protected]>

* fixed plugin namespace

Signed-off-by: alexander <[email protected]>

* fixed message_filter

Signed-off-by: alexander <[email protected]>

* linting

Signed-off-by: alexander <[email protected]>

* modified addPlugin method to also take layer name

Signed-off-by: alexander <[email protected]>

* reverted default plugins to be empty, free costmap_ during layer destruction, changed addPlugin to take layer name as an argument

Signed-off-by: alexander <[email protected]>

* CMake changes to test plugin container layer

Signed-off-by: alexander <[email protected]>

* added helper method to add plugin container layer

Signed-off-by: alexander <[email protected]>

* added initial implementation of plugin container tests

Signed-off-by: alexander <[email protected]>

* added enable to dynamic params, removed unecessary comments, removed unecessary members

Signed-off-by: alexander <[email protected]>

* cleaned up and added additional tests

Signed-off-by: alexander <[email protected]>

* added Apache copyrights

Signed-off-by: alexander <[email protected]>

* linting for ament_cpplint

Signed-off-by: alexander <[email protected]>

* added example file for plugin_container_layer to nav2_bringup

Signed-off-by: alexander <[email protected]>

* removed unused rolling_window_ member variable

Signed-off-by: alexander <[email protected]>

* removed default plugins and plugin types

Signed-off-by: alexander <[email protected]>

* switched to using CombinationMethod enum, added updateWithMaxWithoutUnknownOverwrite case

Signed-off-by: alexander <[email protected]>

* removed combined_costmap_

Signed-off-by: alexander <[email protected]>

* fixed layer naming and accomodating tests

Signed-off-by: alexander <[email protected]>

* removed nav2_params_plugin_container_layer.yaml

Signed-off-by: alexander <[email protected]>

* added more comprehensive checks for checking if layers are clearable

Signed-off-by: alexander <[email protected]>

* added dynamics parameter handling, fixed current_ setting, increased test coverage

Signed-off-by: alexander <[email protected]>

* removed unnecessary locks, added default value

Signed-off-by: alexander <[email protected]>

* removed unecessary resetMap

Signed-off-by: alexander <[email protected]>

* added layer resetting when reset method is called

Signed-off-by: alexander <[email protected]>

* swapped logic for isClearable

Signed-off-by: alexander <[email protected]>

* fixed breaking tests, removed unnecessary combined_costmap_

Signed-off-by: alexander <[email protected]>

* consolidated initialization for loops

Signed-off-by: alexander <[email protected]>

* switched default_value_ to NO_INFORMATION

Signed-off-by: alexander <[email protected]>

* added clearArea function

Signed-off-by: alexander <[email protected]>

* added clearArea test

Signed-off-by: alexander <[email protected]>

* removed TODO message

Signed-off-by: alexander <[email protected]>

* removed constructor and destructors since they do nothing

Signed-off-by: alexander <[email protected]>

* added check on costmap layer to see if it is clearable first

Signed-off-by: alexander <[email protected]>

* fixed tests for clearing functionality

Signed-off-by: alexander <[email protected]>

* added try catch around initialization of plugins

Signed-off-by: alexander <[email protected]>

* fixed indents

Signed-off-by: alexander <[email protected]>

---------

Signed-off-by: alexander <[email protected]>
Signed-off-by: Steve Macenski <[email protected]>
@SteveMacenski SteveMacenski merged commit c8e8b23 into jazzy Dec 13, 2024
6 of 8 checks passed
@SteveMacenski SteveMacenski deleted the jazz_syn branch December 13, 2024 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants