Skip to content

Add sandwich boat model and multi-boat support#1

Merged
matthewheubach merged 4 commits intomainfrom
add_sandwich_model
May 16, 2022
Merged

Add sandwich boat model and multi-boat support#1
matthewheubach merged 4 commits intomainfrom
add_sandwich_model

Conversation

@crvogt
Copy link
Copy Markdown
Contributor

@crvogt crvogt commented May 5, 2022

Summary

First pass at sandwich boat model and multi-boat support. Uses the CoRa model as the foundation for the general structure.

Depends on vrx

To Test

roslaunch sandwich_gazebo sydney_sandwich.launch

This will launch two boat models. The joystick commands only affect the boat whose namespace is that of the default namespace args in the launch file.

Current Issues

  • Steering for sandwich and CoRa not as expected
  • Model is scaled up 10x as changing mass values causes the USV dynamics plugin to fail

@crvogt crvogt changed the title First pass at sandwich boat model and multi-boat support Add sandwich boat model and multi-boat support May 5, 2022
@crvogt crvogt marked this pull request as ready for review May 10, 2022 16:56
@bsb808
Copy link
Copy Markdown
Contributor

bsb808 commented May 13, 2022

I was able to run this PR using the setup described here: https://github.com/Field-Robotics-Lab/optical_swarm/wiki/workspace_setup

I'd like to have @matthewheubach do the review/approval of the PR. Good practice.

<visual>
<origin xyz="0 0 0" rpy="0 0 0" />
<geometry>
<mesh filename="file://$(find sandwich_description)/models/sandwich_base/sandwich.dae" scale="10 10 10"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note scale of 10x. See Issue #3

@matthewheubach
Copy link
Copy Markdown
Contributor

I was able to run the launch file as described. Except for known issues #2 and #3, I did not have any problems with the launch file or sandwich boat models.

@matthewheubach matthewheubach merged commit 1a3ac4a into main May 16, 2022
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.

3 participants