-
Notifications
You must be signed in to change notification settings - Fork 135
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
publish_tf doesn't seem to be used in the example fiducial_slam.launch #148
Comments
Hi, I'm currently trying to work on the same problem (ie publishing pose but not the tfs themselves). Can I ask, what did you change exactly? Did you just make the tf_publish_interval negative? Could you attach this code? You'd be doing me a huge favour and also I'm happy to help implementing it :) |
Sure can! I won't be able to get to it immediately today but I'll be
submitting a PR before the end of the day (CST).
…On Mon, Feb 18, 2019 at 5:54 AM elisabethwelburn ***@***.***> wrote:
Hi, I'm currently trying to work on the same problem (ie publishing pose
but not the tfs themselves). Can I ask, what did you change exactly? Did
you just make the tf_publish_interval negative? Could you attach this code?
You'd be doing me a huge favour and also I'm happy to help implementing it
:)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#148 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AmLdCqF30-L_bj2VN3EwyPGKl9GZgiQ2ks5vOpSAgaJpZM4amA_9>
.
|
Closed by #154 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I see there is a boolean rosparam called "publish_tf" which I'd assume turns off the map->odom publishing however it doesn't look like it's used at all. Is this what it was intended for but didn't get implemented?
I didn't see that rosparam until after I already wrote some of my own code to that checks if tf_publish_interval is negative, then doesn't publish tf and it's working fine, but I'd be happy to switch it to use the rosparam and make a PR addressing that.
The text was updated successfully, but these errors were encountered: