-
Notifications
You must be signed in to change notification settings - Fork 4
Multibeam Sonar Migration Template #15
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
base: ros2
Are you sure you want to change the base?
Conversation
for more information, see https://pre-commit.ci
…the dvl topic name from sdf
…rd-Down 2. Enabled water tracking mode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested a few more changes.
Additionally, please include the config script for ROS2 topic bridges when spawning the sensor model as discussed. Let me know if you need help with that!
gazebo/dave_gz_multibeam_sonar/multibeam_sonar/AxisAlignedPatch2.hh
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hmoyen thanks for making the changes! I have a few more suggestions.
I'm not sure why but the point cloud does not show up for me. Any idea why?
Can you also include the sonar plugin to the BlueROV (to the front bottom of the vehicle) for demo purposes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While this is correct, please follow the same format as in upload_robot.py for consistency.
Make use of PathJoinSubstitution
rather than the os.path.join
. Also, launch_setup
function is not required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making the changes now. I used launch_setup because most sensor models don’t have a config file, only the multibeam sonar models do. I couldn’t find a clean way to handle the missing file during launch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
About the point cloud, it's a difference between the topic names in the launch file and SDF model that is causing the error. I will correct it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay, makes sense! I have made some changes as well - pushing now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve added the BlueROV2 example, let me know if that's what you had in mind. I copied the bluerov2_heavy
model and attached the plugin into the robot. I also created a new demo world with some cylinders to help test and visualize the results. The documentation has also been updated : Wiki.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hmoyen Rather than creating a new robot model for this example, it would be better to attach the sensor plugin to the existing bluerov
, rexrov
, and bluerov2_heavy
as how the other sensors are attached. The purpose is to showcase the robots being spawned with full capability that includes all the important sensors on board.
Let me know if you need help with this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay! I will add to the bluerov2_heavy
model then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hmoyen Can you include an example of the sonar plugin added to BlueROV2 for demo purposes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay, makes sense! I have made some changes as well - pushing now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hmoyen Looks good to me!
One last thing we need to look at is:
"What happens if the machine doesn’t have CUDA installed? would it prevent from compiling with colcon build?"
In the
Is it failing to build on non CUDA platforms? |
Yes, the build is still failing on non CUDA platforms. But I found a fix. Currently, only |
@hmoyen Ready to merge! 🚀 |
This is to fastforward the effort the migration of the Muiltibeam Sonar plugin.
Notion note related to this is at https://yeongdocat.notion.site/Multibeam-Sonar-Plugin-Migration-Memo-3a98b778e87a419b9adf32a88d78814c?pvs=4
Example Launch Command with
ros_gz_bridge
to translate gz msg to ROS msgros_gz_bridge
is not updated for new sensors in new ros-gazebo pair yetExample Launch Command with
dave_sensor.launch.py
Progress (Check date of the update, I am updating this as I progress)
ros2 launch dave_demos dave_world.launch.py world_name:=dave_multibeam_sonar verbose:=true
ros2 launch dave_demos multibeam_sonar_standalone.launch.py x:=4.0 z:=0.5 Y:=3.14
point cloud
- RosGzPointCloud Not Loading gazebosim/ros_gz#549 (comment)
- Something is very wrong when trying to view point cloud at current state
- So many tutorials in
ros_gz_sim_demo
just doesn't work, butros2 launch ros_gz_sim_demos gpu_lidar_bridge.launch.py
works!- This uses GPULidar sensor. I would try remove all Acoustic Beam parts and hard insert GPU Lidar Code as it is to visualize point cloud correctly
Leftover
Float_V
is for