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

Publishing odometry for 6w_Drive plugin #16

Closed
wants to merge 1 commit into from

Conversation

l0g1x
Copy link
Contributor

@l0g1x l0g1x commented Dec 6, 2014

Uncommenting publish_odometry() function. Not sure if this was originally done on purpose or not.

…otprint->odom transform; should set this as a sdf parameter on next commit
@flabrosse
Copy link

Since 2014, the call to publish_odometry() is still commented out. Is there a reason for this? The odometry is calculated, why not publish it?

@qhdwight
Copy link

Any chance we can get this merged in? Would be super helpful. Although I'm not sure if the tf logic is correct, seems to not agree with the odometry publishing.

@Martin-Oehler
Copy link
Member

Martin-Oehler commented May 16, 2022

There are several problems with this code. The transform is directly published to tf instead of a separate topic. The tf frames are hardcoded. And most importantly, this just publishes the ground truth instead of using diff drive kinematics as one would maybe expect. Therefore, in its current form, this won't be merged.

If you are interested in the ground truth transform, you can use GazeboRosP3D.

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.

4 participants