Skip to content

Commit 1a07de6

Browse files
coderwyvernNils-ChristianIsekeTacha-Swentasahsuchetanrs
authored
Merge main into humble_main (#5108)
* Pre-Commit (#4915) * Add pre-commit Signed-off-by: Nils-ChristianIseke <[email protected]> * Add codespell workflow Signed-off-by: Nils-ChristianIseke <[email protected]> * Codespell write_changes=false. As otherwise CI does not fail. Signed-off-by: Nils-ChristianIseke <[email protected]> * Configure isort Signed-off-by: Nils-ChristianIseke <[email protected]> * add precommit Signed-off-by: Nils-ChristianIseke <[email protected]> * Introducing some issues. Signed-off-by: Nils-ChristianIseke <[email protected]> * Revert "Introducing some issues." This reverts commit 5377b65. Signed-off-by: Nils-ChristianIseke <[email protected]> * Removing pre-commit workflow. Signed-off-by: Nils-ChristianIseke <[email protected]> * Fix formatting error Signed-off-by: Nils-ChristianIseke <[email protected]> * Merge remote-tracking branch 'origin/main' into precommit Signed-off-by: Nils-ChristianIseke <[email protected]> * Change v31 to v32 Signed-off-by: Nils-ChristianIseke <[email protected]> * Revert "Merge remote-tracking branch 'origin/main' into precommit" This reverts commit 8a7ca39. Signed-off-by: Nils-ChristianIseke <[email protected]> * Rm submodule Signed-off-by: Nils-ChristianIseke <[email protected]> * pre-commit run --all after merge Signed-off-by: Nils-ChristianIseke <[email protected]> --------- Signed-off-by: Nils-ChristianIseke <[email protected]> * Fix path comparison to avoid unnecessary updates (#5009) Signed-off-by: Tatsuro Sakaguchi <[email protected]> * nav2_rviz_plugins: Don't use non-existent slot (#5016) The definition of the slot was removed in commit e6f500e ("nav2_rviz_plugins: Remove slots without implementation (#4974)", 2025-03-10), because it had no implementation. But we forgot to remove the reference to this slot, because the compiler cannot detect it. We remove the reference now. Without this, rviz shows warnings like: QObject::connect: No such slot nav2_rviz_plugins::CostmapCostTool::updateAutoDeactivate() QObject::connect: (sender name: 'Single click') Signed-off-by: Michal Sojka <[email protected]> * * Parametrize collision checking in nav2_graceful_controller (#5006) * * Parametrize collision checking in nav2_graceful_controller Signed-off-by: suchetanrs <[email protected]> * * Fix linting errors Signed-off-by: suchetanrs <[email protected]> * * Address PR comments * Add parameter to dynamic reconfigure Signed-off-by: suchetanrs <[email protected]> * * Add test for the use_collision_detection parameter Signed-off-by: suchetanrs <[email protected]> --------- Signed-off-by: suchetanrs <[email protected]> * Update smac planner types (#4927) * Update smac planner types Signed-off-by: Michael Carlstrom <[email protected]> * Test ament_mypy Signed-off-by: Michael Carlstrom <[email protected]> * Add packages Signed-off-by: Michael Carlstrom <[email protected]> * Fix arg name Signed-off-by: Michael Carlstrom <[email protected]> * Add ** Signed-off-by: Michael Carlstrom <[email protected]> * Specific package Signed-off-by: Michael Carlstrom <[email protected]> * re-run ci Signed-off-by: Michael Carlstrom <[email protected]> * re-run ci Signed-off-by: Michael Carlstrom <[email protected]> --------- Signed-off-by: Michael Carlstrom <[email protected]> * Declare_parameter_if_not_declared in docking navigator (#5023) Signed-off-by: Alberto Tudela <[email protected]> * Fix naming of launch file Signed-off-by: Steve Macenski <[email protected]> * Fixed naming conventions of launch file Signed-off-by: Steve Macenski <[email protected]> * navfn : fix performance issue (#4945) Signed-off-by: Yancey <[email protected]> * Configured mypy strict for nav2_smac_planner (#5022) * Configured mypy strict on nav2_smac_planner. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Modify workflow to contain mypy configuration. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added dependency for type support Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Mypy nav2 common (#5031) * Configured mypy for nav2_common Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_common to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Removed all instances of Any. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Fixed unsafe fixes using ruff. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Clear costmap if reset distance exceeds costmap bounds. (#5010) * Migrate costmap bound check to clearArea for STVL override Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added unbounded map to world function. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added test for mapToWorldNoBounds Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Add near collision cost and warnings for misaligned parameter settings in MPPI critics (#4996) * Add warning when settings not aligned; Add tunable option for near collision Signed-off-by: mini-1235 <[email protected]> * Linting Signed-off-by: mini-1235 <[email protected]> * Add more information for warning Signed-off-by: mini-1235 <[email protected]> * Add test for coverage Signed-off-by: mini-1235 <[email protected]> * Throw controller exception instead of invalid argument Signed-off-by: mini-1235 <[email protected]> --------- Signed-off-by: mini-1235 <[email protected]> * Added pre-commit with ament ignores to the workflow. (#5029) * Added pre-commit with ament ignores to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Removed codespell from workflow and enabled write changes in pre-commit. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Update pull request template with a testing description section. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * feat(nav2_rotation_shim_controller): add use_path_orientations (#5034) * feat(nav2_rotation_shim_controller): add use_path_orientations Signed-off-by: Kemal Bektas <[email protected]> * Update nav2_rotation_shim_controller/src/nav2_rotation_shim_controller.cpp Signed-off-by: Steve Macenski <[email protected]> --------- Signed-off-by: Kemal Bektas <[email protected]> Signed-off-by: Steve Macenski <[email protected]> Co-authored-by: Kemal Bektas <[email protected]> Co-authored-by: Steve Macenski <[email protected]> * Mypy nav2 bringup (#5040) * Configured nav2_bringup to be compatible with mypy. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_bringup to the linting workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Add service introspection (#4955) * Add service introspection for client and server side, more tests to go Signed-off-by: mini-1235 <[email protected]> * Add test for service server, fix repeat declaration Signed-off-by: mini-1235 <[email protected]> * Fix cpplint Signed-off-by: mini-1235 <[email protected]> * Add test for coverage Signed-off-by: mini-1235 <[email protected]> * Declare and set the parameter in service client and server class Signed-off-by: mini-1235 <[email protected]> * Add typedef; Fix costmap cost tool and bt service Signed-off-by: mini-1235 <[email protected]> * Rebase; Add tests for coverage Signed-off-by: mini-1235 <[email protected]> * Fix costmap cost tool; Typedef Signed-off-by: mini-1235 <[email protected]> * Cleanup Signed-off-by: mini-1235 <[email protected]> * Add spin thread Signed-off-by: mini-1235 <[email protected]> * Add spin Signed-off-by: mini-1235 <[email protected]> * Change spin thread to internal executor Signed-off-by: mini-1235 <[email protected]> * Use rclcpp spin when no internal executor Signed-off-by: mini-1235 <[email protected]> * Update nav2_util/include/nav2_util/service_client.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_util/include/nav2_util/service_client.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_util/include/nav2_util/service_client.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_util/include/nav2_util/service_client.hpp Signed-off-by: Steve Macenski <[email protected]> * Refactor Signed-off-by: mini-1235 <[email protected]> * Update nav2_util/include/nav2_util/service_client.hpp Signed-off-by: Steve Macenski <[email protected]> --------- Signed-off-by: mini-1235 <[email protected]> Signed-off-by: Steve Macenski <[email protected]> Co-authored-by: Steve Macenski <[email protected]> * Mypy nav2_collision_monitor (#5043) * Configured nav2_collision_monitor to use mypy strict. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_collision_monitor to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Mypy nav2 costmap 2d (#5044) * Configured mypy to run on the costmap_2d package. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added mypy_nav2_costmap_2d to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Halt recovery action when main action succeeds to reset status (#5027) * Halt recovery action when main action succeeds to reset status Signed-off-by: Steve Macenski <[email protected]> * Update nav2_behavior_tree/plugins/control/recovery_node.cpp Signed-off-by: Steve Macenski <[email protected]> --------- Signed-off-by: Steve Macenski <[email protected]> * Mypy opennav_docking (#5047) * Define attributes for nav2_msgs. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Configured opennav_docking to use strict mypy. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added opennav_docking to the mypy linting workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Remove unused code in BT utils (#5048) * Clean set port Signed-off-by: Alberto Tudela <[email protected]> * Remove set Signed-off-by: Alberto Tudela <[email protected]> --------- Signed-off-by: Alberto Tudela <[email protected]> * Mypy nav2 lifecycle manager (#5051) * Configured nav2_lifecycle_manager to be compatible with mypy Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_lifecycle_manager to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * fix initializing in drive on heading action node (#5053) Signed-off-by: Johannes Plapp <[email protected]> * Mypy nav2 loopback sim (#5052) * Configured nav2_loopback_sim to be compatible with mypy. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_loopback_sim to the workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Separated packages from list for mypy workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Fix mppi bidirectional settings (#4954) * Add constraint; fix settings Signed-off-by: mini-1235 <[email protected]> * Add lower and upper bound Signed-off-by: mini-1235 <[email protected]> * Update tests Signed-off-by: mini-1235 <[email protected]> * Update nav2_mppi_controller/include/nav2_mppi_controller/motion_models.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_mppi_controller/include/nav2_mppi_controller/motion_models.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_mppi_controller/include/nav2_mppi_controller/motion_models.hpp Signed-off-by: Steve Macenski <[email protected]> * Update nav2_mppi_controller/include/nav2_mppi_controller/motion_models.hpp Signed-off-by: Steve Macenski <[email protected]> --------- Signed-off-by: mini-1235 <[email protected]> Signed-off-by: Steve Macenski <[email protected]> Co-authored-by: Steve Macenski <[email protected]> * Mypy nav2 map server (#5058) * Configured nav2_map_server to be mypy compliant. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_map_server to the mypy workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Publishing a full time-series MPPI trajectory if desirable (#5057) * Publishing a full MPPI trajectory if desirable Signed-off-by: Steve Macenski <[email protected]> * only create publisher when needed Signed-off-by: Steve Macenski <[email protected]> * adjust topics Signed-off-by: Steve Macenski <[email protected]> * Moving for lint Signed-off-by: Steve Macenski <[email protected]> * updating topics in tests Signed-off-by: Steve Macenski <[email protected]> --------- Signed-off-by: Steve Macenski <[email protected]> * Add double spin_some in some BT nodes (#5055) * Add a double spin Signed-off-by: Alberto Tudela <[email protected]> * Add comment Signed-off-by: Alberto Tudela <[email protected]> * Move spin to constructor Signed-off-by: Alberto Tudela <[email protected]> --------- Signed-off-by: Alberto Tudela <[email protected]> * Remove duplicate test (#5062) There remained a duplicate test of test_system_launch after Groot monitoring was removed Signed-off-by: Rasmus Larsson <[email protected]> * Remove docking absolute topic namespaces (#5068) Signed-off-by: Steve Macenski <[email protected]> * Mypy nav2 simple commander (#5059) * Added most nav2_simple_commander files that can be trivially converted to be mypy compatible. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Modified edge cases and types for robot_navigator to pass mypy checks. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_simple_commander to the linting workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added logs and ignored type errors due to spins w/o timeout. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Make NavigateThroughPoses navigator report waypoint statuses information. (fix #4846) (#4994) * Add WaypointStatus message type & Remove MissedWaypoint message type Signed-off-by: zz990099 <[email protected]> * Add waypoint_statuses records in RemoveInCollisionGoals/RemovePassedGoals actions Signed-off-by: zz990099 <[email protected]> * Add validation tests Signed-off-by: zz990099 <[email protected]> * Make NavigateThroughPoses navigator report waypoint statuses information Signed-off-by: zz990099 <[email protected]> * Make WaypointFollower use WaypointStatus message type in action result Signed-off-by: zz990099 <[email protected]> * Split MISSED status into SKIPPED & FAILED Signed-off-by: zz990099 <[email protected]> * fix edge case handle in RemoveInCollision/RemovePassed BT actions Signed-off-by: zz990099 <[email protected]> * Add validation test cases for RemoveInCollision&RemovePassed BT actions Signed-off-by: zz990099 <[email protected]> * Update RemovePassedGoals/RemoveInCollisionGoals node in Groot XML Signed-off-by: zz990099 <[email protected]> --------- Signed-off-by: zz990099 <[email protected]> * Enable Groot2 monitoring (#5065) * Revert removing live groot monitoring from Nav2 (#2696) Signed-off-by: Alberto Tudela <[email protected]> * Update to Groot2 Signed-off-by: Alberto Tudela <[email protected]> * Added JSON conversions Signed-off-by: Alberto Tudela <[email protected]> * Fix rebase Signed-off-by: Alberto Tudela <[email protected]> * Update to nav_msgs::Goals Signed-off-by: Alberto Tudela <[email protected]> * Added nav_msgs to json utils Signed-off-by: Alberto Tudela <[email protected]> * Add register to types Signed-off-by: Alberto Tudela <[email protected]> * Fix null-dereference Signed-off-by: Alberto Tudela <[email protected]> * Added Json test Signed-off-by: Alberto Tudela <[email protected]> * Fix some tests Signed-off-by: Alberto Tudela <[email protected]> * Fix flake Signed-off-by: Alberto Tudela <[email protected]> * Update package dependency Signed-off-by: Alberto Tudela <[email protected]> * Minor fixes Signed-off-by: Alberto Tudela <[email protected]> * Fix test Signed-off-by: Alberto Tudela <[email protected]> * Rename groot_publisher_port parameter to groot_server_port Signed-off-by: Alberto Tudela <[email protected]> * Minor fix in tst Signed-off-by: Alberto Tudela <[email protected]> * Added JSON for waypoint_status Signed-off-by: Alberto Tudela <[email protected]> --------- Signed-off-by: Alberto Tudela <[email protected]> * Fix incorrect world <-> map coordinates conversions (#5049) * Fix incorrect world <-> map coordinates conversions The conversion between world and map continuous (!) coordinates do not require a +/-0.5. This offset is only required when converting discrete map cell indexes to the coordinates of its center. Signed-off-by: Dylan De Coeyer <[email protected]> * nav2_smac_planner: fix smoother test when path is in collision The added pose was indeed invalid, but the it was ignored by the smoother, since considered as a cusp. Instead, let's make the end of the plan invalid as it won't be ignored. Also, let's duplicate the last pose to make the orientation estimation fail, rather than adding a new arbitrary pose. Signed-off-by: Dylan De Coeyer <[email protected]> --------- Signed-off-by: Dylan De Coeyer <[email protected]> * [AMCL] 50% initialisation (and reset) speed improvement (#4941) * index lookup and comparator optimisation Signed-off-by: Guillaume Doisy <[email protected]> * lint Signed-off-by: Guillaume Doisy <[email protected]> --------- Signed-off-by: Guillaume Doisy <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> * Add note to Asynchronous nodes (#5035) * Add warning to some of the nodes Signed-off-by: Adi Vardi <[email protected]> * remove unused includes Signed-off-by: Adi Vardi <[email protected]> * Add warning to remaining nodes using isStatusActive to initialize Signed-off-by: Adi Vardi <[email protected]> * Change to note & update phrsing Signed-off-by: Adi Vardi <[email protected]> * replace reset by re-initialize Signed-off-by: Adi Vardi <[email protected]> --------- Signed-off-by: Adi Vardi <[email protected]> * [AMCL] Rebuild cspace map only when needed (#5076) * for LikelihoodFieldModel Signed-off-by: Guillaume Doisy <[email protected]> * for LikelihoodFieldModelProb Signed-off-by: Guillaume Doisy <[email protected]> * typo Signed-off-by: Guillaume Doisy <[email protected]> * init max_occ_dist in map.c and remove unnecessary condition Signed-off-by: Guillaume Doisy <[email protected]> --------- Signed-off-by: Guillaume Doisy <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> * Consistency in Groot 2 (#5078) Signed-off-by: Alberto Tudela <[email protected]> * Remove duplicate code (#5080) * Remove duplicate code Signed-off-by: mini-1235 <[email protected]> * Add twist validation for controller server Signed-off-by: mini-1235 <[email protected]> --------- Signed-off-by: mini-1235 <[email protected]> * Added waypointstatus to bt_utils (#5077) Signed-off-by: Alberto Tudela <[email protected]> * Lock costmap in getCostsCallback (#5075) * Add IsStoppedBTNode Signed-off-by: Tony Najjar <[email protected]> * add topic name + reformat Signed-off-by: Tony Najjar <[email protected]> * fix comment Signed-off-by: Tony Najjar <[email protected]> * fix abs Signed-off-by: Tony Najjar <[email protected]> * remove log Signed-off-by: Tony Najjar <[email protected]> * add getter functions for raw twist Signed-off-by: Tony Najjar <[email protected]> * remove unused code Signed-off-by: Tony Najjar <[email protected]> * use odomsmoother Signed-off-by: Tony Najjar <[email protected]> * fix formatting Signed-off-by: Tony Najjar <[email protected]> * update groot Signed-off-by: Tony Najjar <[email protected]> * Add test Signed-off-by: Tony Najjar <[email protected]> * reset at success Signed-off-by: Tony Najjar <[email protected]> * FIX velocity_threshold_ Signed-off-by: Tony Najjar <[email protected]> * Fix stopped Node Signed-off-by: Tony Najjar <[email protected]> * Add tests to odometry_utils Signed-off-by: Tony Najjar <[email protected]> * fix linting Signed-off-by: Tony Najjar <[email protected]> * lock costmap Signed-off-by: Tony Najjar <[email protected]> * improvement Signed-off-by: Tony Najjar <[email protected]> * remove spacing Signed-off-by: Tony Najjar <[email protected]> * remove unlock Signed-off-by: Tony Najjar <[email protected]> --------- Signed-off-by: Tony Najjar <[email protected]> * Mypy nav2 system tests (#5083) * Configured nodes of nav2_system_tests to be mypy compliant. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Configured launch files of nav2_system_tests to be mypy compliant. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added nav2_system_tests to the linting workflow. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Configure the entire nav2 stack with mypy (#5084) * Configured tools to be mypy compliant. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Added mypy as a pre-commit hook. Signed-off-by: Leander Stephen D'Souza <[email protected]> * Simplified workflow by including all packages with mypy. Signed-off-by: Leander Stephen D'Souza <[email protected]> --------- Signed-off-by: Leander Stephen D'Souza <[email protected]> * Dynamic param patterns (#4971) * redesign dynamic param patterns Signed-off-by: Nils-ChristianIseke <[email protected]> * change cache version Signed-off-by: Nils-ChristianIseke <[email protected]> * check that parameter of type double are >=0.0 Signed-off-by: Nils-ChristianIseke <[email protected]> --------- Signed-off-by: Nils-ChristianIseke <[email protected]> * [nav2_behavior_tree] Add force_use_current_pose to ComputePathToPoseAction (#4925) * Add force_use_current_pose Signed-off-by: Guillaume Doisy <[email protected]> * xml update Signed-off-by: Guillaume Doisy <[email protected]> * rename to use_start Signed-off-by: Guillaume Doisy <[email protected]> * lint Signed-off-by: Guillaume Doisy <[email protected]> * descriptions Signed-off-by: Guillaume Doisy <[email protected]> * simplify logic Signed-off-by: Guillaume Doisy <[email protected]> --------- Signed-off-by: Guillaume Doisy <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> * [CostmapTopicCollisionChecker] Alternative constructor with footprint string (#4926) * [CostmapTopicCollisionChecker] Alternative constructor with footprint Signed-off-by: Guillaume Doisy <[email protected]> * raw pointer Signed-off-by: Guillaume Doisy <[email protected]> * suggestions from review Signed-off-by: Guillaume Doisy <[email protected]> --------- Signed-off-by: Guillaume Doisy <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> * Merged Fix navfn_planner from humble PR #5087 (#5092) * merged changes from humble for goal.header fix * reverted back, error in merge * ported goal.header fix in navfn_planner.cpp from humble * reverted to navfn_planner.cpp to origin/main * merged navfn_planner.cpp from humble * fixed the merge * Docking backwards as plugin param (#5079) * Replace dock_backwards_ param with dock plugin function Signed-off-by: Alberto Tudela <[email protected]> * Deprecated dock_backwards warning Signed-off-by: Alberto Tudela <[email protected]> * Convert dockDirection from bool to enum Signed-off-by: Alberto Tudela <[email protected]> * Minor fixes Signed-off-by: Alberto Tudela <[email protected]> * Improve deprecated param handling Signed-off-by: Alberto Tudela <[email protected]> * Set default to forward Signed-off-by: Alberto Tudela <[email protected]> * Added tests Signed-off-by: Alberto Tudela <[email protected]> * Update nav2_docking/README.md Co-authored-by: Steve Macenski <[email protected]> Signed-off-by: Alberto Tudela <[email protected]> * Upper and others Signed-off-by: Alberto Tudela <[email protected]> * Revert declare_parameters_from_overrides Signed-off-by: Alberto Tudela <[email protected]> * Added throw on configure plugin Signed-off-by: Alberto Tudela <[email protected]> * Remove node_utils Signed-off-by: Alberto Tudela <[email protected]> --------- Signed-off-by: Alberto Tudela <[email protected]> Co-authored-by: Steve Macenski <[email protected]> * Update map_io library to use Eigen method for faster map loading (#5071) * Update map_io library to use opencv method for faster map loading Signed-off-by: Vignesh T <[email protected]> * Update pre-commit config changes Signed-off-by: Vignesh T <[email protected]> * Use Eigen approach instead of OpenCV Signed-off-by: Vignesh T <[email protected]> * Update pre-commit changes Signed-off-by: Vignesh T <[email protected]> * Update include header include order Signed-off-by: Vignesh T <[email protected]> * Remove intermediary alpha matrix Signed-off-by: Vignesh T <[email protected]> * Add comments for the code understanding Signed-off-by: Vignesh T <[email protected]> * Fix else braces rule issue Signed-off-by: Vignesh T <[email protected]> * Create and use alpha_matrix when applying mask Signed-off-by: Vignesh T <[email protected]> * Update pre-commit changes Signed-off-by: Vignesh T <[email protected]> * Take flip part out of if-else Signed-off-by: Vignesh T <[email protected]> * Update pre-commit changes Signed-off-by: Vignesh T <[email protected]> --------- Signed-off-by: Vignesh T <[email protected]> * Fix CI builds (#5104) Signed-off-by: Steve Macenski <[email protected]> * Increase docking coverage && fix backward docking (#5097) * Add dock_database tests Signed-off-by: Alberto Tudela <[email protected]> * Add utils test Signed-off-by: Alberto Tudela <[email protected]> * Improve filter tests Signed-off-by: Alberto Tudela <[email protected]> * Added simple charging tests Signed-off-by: Alberto Tudela <[email protected]> * Improve comments Signed-off-by: Alberto Tudela <[email protected]> * Fix backward and redo main test Signed-off-by: Alberto Tudela <[email protected]> * Change test period to reduce test time Signed-off-by: Alberto Tudela <[email protected]> * Revert "Change test period to reduce test time" This reverts commit ef1555e. Signed-off-by: Alberto Tudela <[email protected]> * Delete print pose Signed-off-by: Alberto Tudela <[email protected]> --------- Signed-off-by: Alberto Tudela <[email protected]> * Run ament_uncrustify to remove linting issues Signed-off-by: coderwyvern <[email protected]> --------- Signed-off-by: Nils-ChristianIseke <[email protected]> Signed-off-by: Tatsuro Sakaguchi <[email protected]> Signed-off-by: Michal Sojka <[email protected]> Signed-off-by: suchetanrs <[email protected]> Signed-off-by: Michael Carlstrom <[email protected]> Signed-off-by: Alberto Tudela <[email protected]> Signed-off-by: Steve Macenski <[email protected]> Signed-off-by: Yancey <[email protected]> Signed-off-by: Leander Stephen D'Souza <[email protected]> Signed-off-by: mini-1235 <[email protected]> Signed-off-by: Kemal Bektas <[email protected]> Signed-off-by: Johannes Plapp <[email protected]> Signed-off-by: Rasmus Larsson <[email protected]> Signed-off-by: zz990099 <[email protected]> Signed-off-by: Dylan De Coeyer <[email protected]> Signed-off-by: Guillaume Doisy <[email protected]> Signed-off-by: Adi Vardi <[email protected]> Signed-off-by: Tony Najjar <[email protected]> Signed-off-by: Vignesh T <[email protected]> Signed-off-by: coderwyvern <[email protected]> Co-authored-by: Nils-Christian Iseke <[email protected]> Co-authored-by: Tatsuro Sakaguchi <[email protected]> Co-authored-by: Michal Sojka <[email protected]> Co-authored-by: suchetanrs <[email protected]> Co-authored-by: Michael Carlstrom <[email protected]> Co-authored-by: Alberto Tudela <[email protected]> Co-authored-by: Steve Macenski <[email protected]> Co-authored-by: Yancey <[email protected]> Co-authored-by: Leander Stephen D'Souza <[email protected]> Co-authored-by: mini-1235 <[email protected]> Co-authored-by: Kemal Bektas <[email protected]> Co-authored-by: Kemal Bektas <[email protected]> Co-authored-by: Johannes Plapp <[email protected]> Co-authored-by: RasmusLar <[email protected]> Co-authored-by: zz99 <[email protected]> Co-authored-by: DylanDeCoeyer-Quimesis <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> Co-authored-by: Guillaume Doisy <[email protected]> Co-authored-by: Adi Vardi <[email protected]> Co-authored-by: Tony Najjar <[email protected]> Co-authored-by: Sandeep Dutta <[email protected]> Co-authored-by: Vignesh T <[email protected]>
1 parent 018448f commit 1a07de6

File tree

477 files changed

+6822
-2459
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

477 files changed

+6822
-2459
lines changed

.circleci/config.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ _commands:
3333
- restore_cache:
3434
name: Restore Cache << parameters.key >>
3535
keys:
36-
- "<< parameters.key >>-v32\
36+
- "<< parameters.key >>-v33\
3737
-{{ arch }}\
3838
-{{ .Branch }}\
3939
-{{ .Environment.CIRCLE_PR_NUMBER }}\
4040
-{{ checksum \"<< parameters.workspace >>/lockfile.txt\" }}"
41-
- "<< parameters.key >>-v32\
41+
- "<< parameters.key >>-v33\
4242
-{{ arch }}\
4343
-main\
4444
-<no value>\
@@ -58,7 +58,7 @@ _commands:
5858
steps:
5959
- save_cache:
6060
name: Save Cache << parameters.key >>
61-
key: "<< parameters.key >>-v32\
61+
key: "<< parameters.key >>-v33\
6262
-{{ arch }}\
6363
-{{ .Branch }}\
6464
-{{ .Environment.CIRCLE_PR_NUMBER }}\

.devcontainer/caddy/srv/assets/foxglove/nav2_layout.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -460,4 +460,4 @@
460460
"direction": "row",
461461
"splitPercentage": 74.87855655794587
462462
}
463-
}
463+
}

.devcontainer/caddy/srv/nav2/github-markdown.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1099,4 +1099,4 @@
10991099

11001100
.markdown-body ::-webkit-calendar-picker-indicator {
11011101
filter: invert(50%);
1102-
}
1102+
}

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,13 @@
1313

1414
---
1515

16+
## Description of testing performed
17+
<!--
18+
For example: Linting validation using -> pre-commit run --all,
19+
Package testing using -> colcon test --packages-select <modified package>,
20+
or functional testing of changes on the robot or in simulation
21+
-->
22+
1623
## Description of contribution in a few bullet points
1724

1825
<!--

.github/mergify.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ pull_request_rules:
77
backport:
88
branches:
99
- jazzy
10-
10+
1111
- name: backport to iron at reviewers discretion
1212
conditions:
1313
- base=main
@@ -16,7 +16,7 @@ pull_request_rules:
1616
backport:
1717
branches:
1818
- iron
19-
19+
2020
- name: backport to humble at reviewers discretion
2121
conditions:
2222
- base=main

.github/workflows/lint.yml

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,34 @@ jobs:
1111
strategy:
1212
fail-fast: false
1313
matrix:
14-
linter: [xmllint, cpplint, uncrustify, pep257, flake8]
14+
linter: [xmllint, cpplint, uncrustify, pep257, flake8, mypy]
1515
steps:
1616
- uses: actions/checkout@v4
17+
18+
- name: Install typeshed for mypy
19+
if: matrix.linter == 'mypy'
20+
run: sudo apt update && sudo apt install -y python3-typeshed
21+
1722
- uses: ros-tooling/[email protected]
1823
with:
1924
linter: ${{ matrix.linter }}
2025
distribution: rolling
2126
package-name: "*"
27+
arguments: ${{ matrix.linter == 'mypy' && '--config tools/pyproject.toml' || '' }}
28+
29+
pre-commit:
30+
name: pre-commit
31+
runs-on: ubuntu-latest
32+
steps:
33+
- uses: actions/checkout@v4
34+
- uses: actions/setup-python@v5
35+
- uses: pre-commit/[email protected]
36+
env:
37+
SKIP: >-
38+
ament_lint_cmake,
39+
ament_cpplint,
40+
ament_uncrustify,
41+
ament_xmllint,
42+
ament_flake8,
43+
ament_pep257,
44+
ament_mypy

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,4 +69,3 @@ Session.vim
6969

7070
# Vim Temporary
7171
.netrwhist
72-

.pre-commit-config.yaml

Lines changed: 104 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,104 @@
1+
2+
# To use:
3+
#
4+
# pre-commit run -a
5+
#
6+
# Or:
7+
#
8+
# pre-commit install # (runs every time you commit in git)
9+
#
10+
# To update this file:
11+
#
12+
# pre-commit autoupdate
13+
#
14+
# See https://github.com/pre-commit/pre-commit
15+
exclude: ".pgm$|.svg$"
16+
repos:
17+
- repo: https://github.com/pre-commit/pre-commit-hooks
18+
rev: v5.0.0
19+
hooks:
20+
- id: check-added-large-files
21+
- id: check-ast
22+
- id: check-case-conflict
23+
- id: check-merge-conflict
24+
- id: check-symlinks
25+
- id: check-xml
26+
- id: check-yaml
27+
args: ["--allow-multiple-documents"]
28+
- id: debug-statements
29+
- id: end-of-file-fixer
30+
- id: forbid-submodules
31+
- id: mixed-line-ending
32+
- id: trailing-whitespace
33+
exclude_types: [rst]
34+
- id: fix-byte-order-marker
35+
- repo: https://github.com/pycqa/isort
36+
rev: 6.0.1
37+
hooks:
38+
- id: isort
39+
args: ["tools/pyproject.toml"]
40+
name: isort (python)
41+
42+
- repo: https://github.com/codespell-project/codespell
43+
rev: v2.4.1
44+
hooks:
45+
- id: codespell
46+
additional_dependencies:
47+
- tomli
48+
args:
49+
[--toml=./tools/pyproject.toml]
50+
- repo: https://github.com/python-jsonschema/check-jsonschema
51+
rev: 0.31.1
52+
hooks:
53+
- id: check-github-workflows
54+
args: ["--verbose"]
55+
- id: check-github-actions
56+
args: ["--verbose"]
57+
- id: check-dependabot
58+
args: ["--verbose"]
59+
- repo: local
60+
hooks:
61+
- id: ament_lint_cmake
62+
name: ament_lint_cmake
63+
description: Check CMake code style using cmakelint.
64+
language: system
65+
types: [cmake]
66+
entry: ament_lint_cmake
67+
- id: ament_cpplint
68+
name: ament_cpplint
69+
description: Code style checking using cpplint.
70+
language: system
71+
types: [c++]
72+
entry: ament_cpplint
73+
- id: ament_uncrustify
74+
name: ament_uncrustify
75+
description: Code style checking using uncrustify.
76+
language: system
77+
types: [c++]
78+
args: ["--reformat"]
79+
entry: ament_uncrustify
80+
- id: ament_xmllint
81+
name: ament_xmllint
82+
description: Check XML markup using xmllint.
83+
language: system
84+
types: [xml]
85+
entry: ament_xmllint
86+
- id: ament_flake8
87+
name: ament_flake8
88+
description: Check Python code style using flake8.
89+
language: system
90+
types: [python]
91+
entry: ament_flake8
92+
- id: ament_pep257
93+
name: ament_pep257
94+
description: Check Python code style using pep257.
95+
language: system
96+
types: [python]
97+
entry: ament_pep257
98+
- id: ament_mypy
99+
name: ament_mypy
100+
description: Check Python code style using mypy.
101+
language: system
102+
types: [python]
103+
args: ["--config", "tools/pyproject.toml"]
104+
entry: ament_mypy

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ RUN mkdir -p $ROOT_SRV
168168

169169
# install demo dependencies
170170
RUN apt-get update && apt-get install -y \
171-
ros-$ROS_DISTRO-rviz2
171+
ros-$ROS_DISTRO-rviz2
172172

173173
# install gzweb dependacies
174174
RUN apt-get install -y --no-install-recommends \

0 commit comments

Comments
 (0)