-
Notifications
You must be signed in to change notification settings - Fork 52
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
lrmate200id_gazebo: add package to spawn model and position control on Gazebo simulator #20
base: indigo-devel
Are you sure you want to change the base?
lrmate200id_gazebo: add package to spawn model and position control on Gazebo simulator #20
Conversation
c48dd55
to
204f587
Compare
204f587
to
42faee3
Compare
42faee3
to
09f934f
Compare
747f69c
to
f714a8b
Compare
cmake_minimum_required(VERSION 2.8.3) | ||
project(fanuc_lrmate200id_gazebo) | ||
find_package(catkin REQUIRED) | ||
catkin_package() |
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.
Could you please add the required install()
commands here? See fanuc_m10ia_support/CMakeLists.txt for an example.
Thanks for the PR. Inertial values indeed are a bit too 'standard'; perhaps the cylinder approximates used in the If you address the issues I commented on above, we could merge the package. |
O, and try these for the masses (
|
Actually the robot blows up with the cylinder approximates and above weights. |
611e32f
to
9d24548
Compare
@gavanderhoorn I computed the inertial value with Meshlab, and I think it is closer to the true value. Also the robot looks stable on the Gazebo even with planed trajectory execution by MoveIt!. |
@wkentaro: Thanks for your efforts. I'm currently OoO. I'll take a look somewhere after the 15th. I'll let you know. |
Alright, thanks. 2016年8月6日土曜日、G.A. vd. Hoornnotifications@github.comさんは書きました:
和田 健太郎 / Kentaro Wada |
9d24548
to
af67e70
Compare
Note
Result