osrf / vrx

Virtual RobotX (VRX) resources.
Apache License 2.0
421 stars 189 forks source link

Fix issue #641 #669

Closed caguero closed 1 year ago

caguero commented 1 year ago

See issue #641.

The problem was our well known Gazebo "feature" that eliminates fixed joints when converting from URDF to SDF. Then, the pinger link disappeared. I converted the fixed joint into a revolute joint with no movement to bypass the problem.

How to test it?

M1chaelM commented 1 year ago

@caguero This seems to be working but there is a little quirk where even though gz topic -l doesn't show /wamv/pingers/ping/range_bearing, ros2 topic list still includes it:

/clock
/parameter_events
/rosout
/tf
/tf_static
/vrx/contacts
/vrx/debug/wind/direction
/vrx/debug/wind/speed
/vrx/task/info
/wamv/pingers/pinger/range_bearing
/wamv/pingers/pinger/set_pinger_position
/wamv/pose
...

If I try to echo it I get nothing, so maybe it's ok?

caguero commented 1 year ago

@caguero This seems to be working but there is a little quirk where even though gz topic -l doesn't show /wamv/pingers/ping/range_bearing, ros2 topic list still includes it

I just tweaked the code to not advertise the ROS 2 topic if the pinger isn't present. 6bfcbbc