-
Notifications
You must be signed in to change notification settings - Fork 17
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
[Melodic] #38
[Melodic] #38
Conversation
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 can confirm that the plugin still works for kinetic - when using https://github.com/ipa320/cob_common/pull/271
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 see the following warning:
[ WARN] [1564736237.339278403, 0.469000000]: The default_robot_hw_sim plugin is using the Joint::SetPosition method without preserving the link velocity.
[ WARN] [1564736237.339305046, 0.469000000]: As a result, gravity will not be simulated correctly for your model.
[ WARN] [1564736237.339321186, 0.469000000]: Please set gazebo_pid parameters, switch to the VelocityJointInterface or EffortJointInterface, or upgrade to Gazebo 9.
[ WARN] [1564736237.339335329, 0.469000000]: For details, see https://github.com/ros-simulation/gazebo_ros_pkgs/issues/612
this might not necessarily be related to this PR, but I would like to investigate before merging
maybe we can do something like this: ros-simulation/gazebo_ros_pkgs#550 |
@benmaidel please re-test this branch with melodic simulation to help me verify the dual-distro compatibility...thx |
ref ipa320/care-o-bot#52
superseeds #37
I've removed the last commit from @ompugao concerning the pid interface because of @fmessmer comment #37 (review)
This should be compatible with kinetic and melodic if the
hardware_interface
prefix is used for all transmissions.