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

Optionally disable map->odom tf transfrom #37

Closed
Thazz opened this issue May 6, 2021 · 3 comments
Closed

Optionally disable map->odom tf transfrom #37

Thazz opened this issue May 6, 2021 · 3 comments

Comments

@Thazz
Copy link

Thazz commented May 6, 2021

I would propose to add a parameter to optionally disable publishing of map->odom tf transfrom, such as tf_broadcast parameter in amcl package.

Use case for this is when one would be using robot_localization package for fusing multiple localization sources with Kalman filter (e.g localization with lama + bluetooth beacons or GPS). In this case map->odom is (or can be) published by robot_localization nodes and should be disabled for robot_localization input sources, i.e lama ros.

Edit: I see that using Kalman filter is also discussed in #5.

@eupedrosa
Copy link
Member

Hi @Thazz

Are you talking about a specific node, for example loc2d_ros, or should this feature be available in all nodes?

@Thazz
Copy link
Author

Thazz commented May 12, 2021

I think it only makes sense to implement it in loc2d_ros as it is used for localization.

For slam2d_ros it could be useful in case od lifelong mapping as discussed in #38, where this node would also be used for localization, but as I understand from discussion, LaMa is currently not suitable for this.

Another question, is it possible to estimate covariance matrix for localization, so that pose could be published as geometry_msgs::PoseWithCovarianceStamped message?

@eupedrosa
Copy link
Member

I think I can add the option to disable tf broadcasting for loc2d_ros.

Another question, is it possible to estimate covariance matrix for localization, so that pose could be published as geometry_msgs::PoseWithCovarianceStamped message?

This already exist for loc2d_ros

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

No branches or pull requests

2 participants