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

Galactic sync 6 #2940

Merged
merged 22 commits into from
May 6, 2022
Merged

Galactic sync 6 #2940

merged 22 commits into from
May 6, 2022

Conversation

SteveMacenski
Copy link
Member

Fixing costmap region clearing logic

FixingTF lookups in controller server

Adding various improvements and new APIs to the Simple Commander Python API

Remove resizing of costmap on incremental update

Removing unused or unncessary methods and dependencies

Adding Theta* to default install list

Fix crashing on LC transition in BT navigator

Fixes crashing issue in AMCL when dynamically setting new maps

doisyg and others added 22 commits May 6, 2022 13:48
* Added call to controller action server with a path to follow
++ Added a new function  in robot_navigator,py
++ Added a launch script to test  function
++ Updated setup.py to include demo_follow_path.py

* Code refactoring

* Code refactoring

* Code refactoring for consistency

* Updated README.md

* Resolved executable conflict in setup.py for example_follow_path.py

* Code refactoring with  ament_flake8
* fix Typo

* add recoveries

* add docs

* added demo using backup and spin

* rename isNavigationComplete to isTaskComplete

* rename cancelNav to cancelTask

* add prints

* fix premature exit

* rename NavResult to TaskResult

* fix readme order

* fix import order

* renaming

* renaming
…2784)

* Fixing the issue #2781: raytraceLine with same start and end point no longer causes segmentation fault

* Some whitespace modifications to make the code pass release_test

* Add testcase for raytraceLine the same point

Co-authored-by: Alexey Merzlyakov <[email protected]>
* add option to specify navigator and localizer to wait for

* add docs for waituntilNav2Active

* wait for pose only for amcl

* revert order
* remove unused odometry smoother in bt navigator

Signed-off-by: Alberto Soragna <[email protected]>

* reorganize bt navigator to shared odom_smoother object with servers

Signed-off-by: Alberto Soragna <[email protected]>
* added options

* update docs
* fix empty address access on halt all actions

* fix unsafe declaration of parameters

* restore odom smoother

* fix styling issues

* add missing semicolumn
* Better Costmap Error Message

* PR Feedback
* Allow usage of std::string in searchAndGetParam()

Removed also old TODO related to legacy ROS API

* Fix
* Clean up action clients in nav2_simple_commander

* Add camelCase version

* Add docs
* Added mutex to prevent SEGFAULT on map change

* Removing redundant mutex

Co-authored-by: Antonio Marangi <[email protected]>
@SteveMacenski SteveMacenski merged commit f405c85 into galactic May 6, 2022
@SteveMacenski SteveMacenski deleted the galactic_sync6 branch May 6, 2022 21:03
@lopsided98
Copy link
Contributor

I think #2886 should have been included here, because otherwise the build fails for me.

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.