Skip to content

Conversation

@Butakus
Copy link
Member

@Butakus Butakus commented Dec 16, 2025

Follow-up to EasyNavigation/EasyNavigation#82.

I also removed some C++20 stuff like std::numbers::pi or std::map::contains().

Hope the CI is green, local tests did pass.

Best.

@fmrico
Copy link
Contributor

fmrico commented Dec 16, 2025

LGTM with these changes

@fmrico
Copy link
Contributor

fmrico commented Dec 16, 2025

Tested with

Param file Robot Sim/Real World Sensors Maps Managers Localizer Planner Controller Checked
Test3 Kobuki Sim House Laser Simple AMCL/Simple A*/Simple Simple
Test4 Kobuki Sim House Laser Costmap (obstacles + inflation) AMCL/Costmap A*/Costmap Serest
Test5 Kobuki Sim House Laser Costmap (obstacles + inflation) + Routes AMCL/Costmap A*/Costmap MPPI

@Butakus Maybe you should do the same with https://github.com/EasyNavigation/easynav_lidarslam_ros2 and https://github.com/EasyNavigation/easynav_gridmap_stack until we decide to deprecate them in anytime

Merging!!! 🚀

@fmrico fmrico merged commit 63aa3e5 into EasyNavigation:rolling Dec 16, 2025
1 check passed
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.

3 participants