-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add 3D Lidar to iris for usage with Bonxai mapping example #63
Comments
FYI @tejalbarnwal |
Hi, I prefer the third approach, where we use a base SDF file (e.g., iris_ardupilot) and build on it to add specific sensors like 2D/3D LiDAR, depth cameras, or RGB camera to create different models like My only concern is ensuring consistency between the model passed via the launch argument and the model included in the world files (e.g., A potential improvement would be to enable the independent spawning of the iris model in the world, separate from other Gazebo components, to simplify integration. However, I am unsure how this could be implemented. |
Why don't we start with just the simple option 3 with manually ensured consistency. Did you want to take this on, or have me do it? |
I will implement it tonight after my job and submit a PR. |
Hi @Ryanf55 , Let me know your reviews for the PR. |
Background
As explained in this Bonxai demo, the ardupilot_gz
iris_lidar.launch.py
assumes a 2D lidar and it must be modified. https://github.com/tejalbarnwal/3D-planning-with-bonxaiWe'd like to add 3D lidar support that works out of the box without having to modify code here.
Approaches considered
iris_lidar.launch.py
toiris_2d_lidar.launch.py
and duplicate the code for 3D lidariris_with_lidar
toiris_with_2d_lidar
and duplicate toiris_with_3d_lidar
iris_with_lidar
and call itiris_ardupilot
, and then add two new modelsiris_with_2d_lidar
andiris_with_3d_lidar
that only add the lidar link and the lidar sensor, then have theiris_with_lidar.launch.py
pick which model based on an argumentHow to select 2d/3d lidar in launch file?
The text was updated successfully, but these errors were encountered: