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

Nav2 Sync 2 Nov 16 #10

Merged
merged 9 commits into from
Nov 20, 2023
Merged

Nav2 Sync 2 Nov 16 #10

merged 9 commits into from
Nov 20, 2023

Conversation

RBT22
Copy link

@RBT22 RBT22 commented Nov 16, 2023

  • Adjust the Variable types in Nav2_costmap_2d pkg in [nav2_humble]
  • Log if BT rate is exceeded (ros-planning#3909) (ros-planning#3913)
  • Update theta_star_planner.cpp (ros-planning#3918) (ros-planning#3922)
  • Fixing subtree issues with blackboard shared resources (3640)
  • Handle NaNs in AMCL beam sensor model (ros-planning#3929)
  • Fix Wrong Map Pointer (ros-planning#3311)
  • Fix NaN in Updated PathAlign (ros-planning#3943)
  • Fix for robot footprint collision in obstacles critic (ros-planning#3878

mergify bot and others added 9 commits November 16, 2023 11:26
…os-navigation#3897)

* Use mutex to protect costmap reads. (ros-navigation#3877)

* Use mutex to protect costmap reads.
Otherwise costmap can be read during a map update and return 0.

* Revert "Use mutex to protect costmap reads."

This reverts commit e16a44c.

* Lock costmap before running MPPI controller.

* Fix typo.

* Protect against costmap updates in MPP and RotationShim controllers.

---------

Co-authored-by: Leif Terry <[email protected]>
(cherry picked from commit a1c9fd5)

# Conflicts:
#	nav2_mppi_controller/src/controller.cpp

* fix merge conflict

---------

Co-authored-by: LeifHookedWireless <[email protected]>
Co-authored-by: Steve Macenski <[email protected]>
ros-navigation#3911) (ros-navigation#3916)

* Fixing subtree issues with blackboard shared resources (3640) (ros-navigation#3911)

* fixing subtree issues

* Update bt_action_server_impl.hpp

(cherry picked from commit 4b4465d)

# Conflicts:
#	nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server_impl.hpp

* Update bt_action_server_impl.hpp

---------

Co-authored-by: Steve Macenski <[email protected]>
…gation#3937)

* Handle NaNs in AMCL beam sensor model

Signed-off-by: Michel Hidalgo <[email protected]>

* Use proper isnan check

Signed-off-by: Michel Hidalgo <[email protected]>

---------

Signed-off-by: Michel Hidalgo <[email protected]>
(cherry picked from commit 06c3550)

Co-authored-by: Michel Hidalgo <[email protected]>
…s-navigation#3938)

Not assigning fixed map pointer to particle filter, using latest when resample.

(cherry picked from commit cc6f205)

Signed-off-by: Borong Yuan <[email protected]>

Signed-off-by: Borong Yuan <[email protected]>
(cherry picked from commit 85735ea)

Co-authored-by: Borong Yuan <[email protected]>
…#3878) (ros-navigation#3947)

* Inscribed/Circumscribed costs must be updated to take into account the
current shape of the robot.
Was previous only being called once in initialize().

* Add early return to avoid calculations if footprint has not changed.

* Only update radius if using footprint.
Add perf timers.

* Remove perf timers.

* Update comments.

---------

Co-authored-by: Leif Terry <[email protected]>
(cherry picked from commit 98af3b9)

Co-authored-by: LeifHookedWireless <[email protected]>
@RBT22 RBT22 changed the title Nav2 Sync 1 Nov 16 Nav2 Sync 2 Nov 16 Nov 16, 2023
@redvinaa redvinaa merged commit ed206e1 into humble Nov 20, 2023
@RBT22 RBT22 deleted the sync2 branch April 9, 2024 07:28
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.

2 participants