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
Added heron_control, changed heron_description to use with uuv_simulator #2
Conversation
heron_description/CHANGELOG.rst
Outdated
@@ -2,6 +2,10 @@ | |||
Changelog for package heron_description | |||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | |||
|
|||
0.4.0 (2018-06-26) |
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.
Remove this, it get automatically generated when you use catkin_generate_changelog
heron_description/package.xml
Outdated
@@ -1,7 +1,7 @@ | |||
<?xml version="1.0"?> | |||
<package> | |||
<name>heron_description</name> | |||
<version>0.3.0</version> | |||
<version>0.4.0</version> |
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.
Revert, this is done using catkin_prepare_release
heron_description/urdf/sensors.xacro
Outdated
<headingDrift>0.005</headingDrift> | ||
<headingGaussianNoise>0.005</headingGaussianNoise> | ||
</plugin> | ||
</gazebo> |
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.
Spacing is off.
heron_description/CMakeLists.txt
Outdated
@@ -1,9 +1,13 @@ | |||
cmake_minimum_required(VERSION 2.8.3) | |||
project(heron_description) | |||
|
|||
find_package(catkin REQUIRED COMPONENTS roslaunch) | |||
find_package(catkin REQUIRED COMPONENTS | |||
heron_msgs |
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.
Are heron_msgs
needed? I don't see them used any where.
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.
Should I put "roslaunch" back in?
heron_description/urdf/base.xacro
Outdated
<!-- Collision geometry mesh, usually in STL format (it is recommended to keep | ||
this geometry as simple as possible to improve the performance the physics engine | ||
regarding the computation of collision forces) --> | ||
<xacro:property name="collision_mesh_file" value="file://$(find heron_description)/meshes/heron_base.stl"/> |
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.
Does the simulator use the collision mesh to calculate buoyancy? If so, simplifying this mesh may go a long way for efficiency
@dniewinski any remaining concerns? |
LGTM |
@Nirzvi1 can you rebase? |
Deleted unnecessary comments Set frequency to 20Hz (the default update frequency of the um6 IMU).
* Added heron_control package * Deleted unnecessary comments from CMakeLists.txt * Updated robot_localization.yaml Deleted unnecessary comments Set frequency to 20Hz (the default update frequency of the um6 IMU). * Edited package.xml and moved heron_controller node from heron_control * Updated package.xml description
…arate sensor frame
No description provided.