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

Service to reinitialize global localization #141

Closed
3 of 5 tasks
olmerg opened this issue Mar 17, 2023 · 1 comment · Fixed by #157
Closed
3 of 5 tasks

Service to reinitialize global localization #141

olmerg opened this issue Mar 17, 2023 · 1 comment · Fixed by #157
Assignees
Labels
cpp Related to C++ code enhancement New feature or request ros Related to ROS

Comments

@olmerg
Copy link
Collaborator

olmerg commented Mar 17, 2023

Description

This service make part of nav2_amcl which re-initialize the particle filter with a uniform distribution. This service interact with the parameter always_reset_initial_pose and the variable initial_pose_is_known_ which determine when to publish the estimated pose, the particles poses and the tf_broadcast. To make this possible it is required to divide map_callback, so here could be included the parameter first_map_only to reduce the tasks in 85.

To implement completely the functionality the initial_pose should be save over some states of lifecycle like on_cleanup and on_configure

Definition of done

  • create the service
  • create parameter always_reset_initial_pose
  • create parameter first_map_only
  • define when to publish data to ros with the flag initial_pose_is_known_
  • implement how to save the initial pose during restart the node.

Additional considerations

@olmerg olmerg added enhancement New feature or request cpp Related to C++ code ros Related to ROS labels Mar 17, 2023
@olmerg olmerg self-assigned this Mar 17, 2023
@olmerg
Copy link
Collaborator Author

olmerg commented Mar 17, 2023

Implementing first_map_only is a good moment to solve the problem of TODO(@ivanpauno ): Initialize later maps from last known pose, but it is required to be possible to set the map in the particle filter

@nahueespinosa nahueespinosa changed the title Reinitialize global localization Service Service to reinitialize global localization Mar 21, 2023
nahueespinosa pushed a commit that referenced this issue May 10, 2023
Closes to #141.

Signed-off-by: Olmer Garcia-Bedoya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp Related to C++ code enhancement New feature or request ros Related to ROS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant