Skip to content
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

[WIP][Depends on #242]fix IOBplugin to enable use_velocity_feedback in kinetic #245

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Naoki-Hiraoka
Copy link
Contributor

@Naoki-Hiraoka Naoki-Hiraoka commented Dec 11, 2019

In kinetic,


if use_velocity_feedback is true, the robot blows off with roslaunch hrpsys_gazebo_general gazebo_samplerobot_no_controllers.launch.
samplebot_velocity

From http://answers.gazebosim.org/question/8371/kobukiturtlebot-doesnt-move-when-given-velocity-commands-in-gazebo-5/?answer=8377#post-id-8377 , I tried using joint->SetParam("vel") and joint->SetParam("max_force"). (e4b8b59)
However, the robot was not actuated.
samplebot_velocity1

Then, from http://gazebosim.org/tutorials?tut=set_velocity&cat= , I tried using joint->SetParam("vel") and joint->SetParam("fmax"). (4747e03)
Though the robot was actuated and not blew off, the robot was not controlled as intended.
samplebot_velocity2

For now, we have to set use_velocity_feedback to false in kinetic.

Copy link
Member

@k-okada k-okada left a comment

Choose a reason for hiding this comment

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

@Naoki-Hiraoka can we merge this? or is this only for kinetic and not need/work for melodic?

@Naoki-Hiraoka
Copy link
Contributor Author

This PR does NOT work on kinetic.
This PR is just a bug report , so we don't have to merge this.
I'm sorry for the confusion.

@pazeshun
Copy link
Collaborator

pazeshun commented May 4, 2020

@k-okada
In my understanding, this PR is a trial to use the same configuration as indigo (use_velocity_feedback: true) on kinetic and melodic.
Finally, @Naoki-Hiraoka gave up using the same configuration.
On #250 , he changed the configuration (use_velocity_feedback: false and adjusting gains).
@Naoki-Hiraoka Is my understanding correct?

@pazeshun
Copy link
Collaborator

pazeshun commented May 4, 2020

I created #253 to reconstruct old configuration.

@Naoki-Hiraoka
Copy link
Contributor Author

@k-okada
In my understanding, this PR is a trial to use the same configuration as indigo (use_velocity_feedback: true) on kinetic and melodic.
Finally, @Naoki-Hiraoka gave up using the same configuration.
On #250 , he changed the configuration (use_velocity_feedback: false and adjusting gains).
@Naoki-Hiraoka Is my understanding correct?

Yes.
We use use_velocity_feedback: false on kinetic and melodic now, and samplerobot can walk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants