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

Improve joint simulation #40

Open
wants to merge 3 commits into
base: dev/noetic
Choose a base branch
from

Conversation

nickswalker
Copy link

Add gazebo_ros_control PID gains
Tuned manually to make movement via the teleop interface behave well
Wheels still unspecified because they cause the robot to collapse
Use effort joint interface for the gripper so that gripper contacts are well simulated

I have been able to pick up objects using these gains and a simulation step size of .0005. I still notice that the lift joint is sluggish on the way up and comes down with an implausible amount of force, which can cause the base to shake and shift position noticeably. It's possible that better tuning of the controller parameters will help, but it may also be the case that the simulated robot geometry/inertias need to be tweaked.

Closes #39

Add gazebo_ros_control PID gains
Tuned manually to make movement via the teleop interface behave well
Wheels still unspecified because they cause the robot to collapse
Use effort joint interface for the gripper so that gripper contacts are well simulated
Copy link
Contributor

@vatanaksoytezer vatanaksoytezer left a comment

Choose a reason for hiding this comment

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

@nickswalker thank you for the PR, this looks great! So, I've been testing this and happy with the gains so far. You are right about inertias as well, I don't know if you have hit that yet but if you try to grab a coke can or rather any object heavier than 200 grams when the arm is fully extended, the robot will fall over.

Having said that these gains do not play well with "default gazebo worlds" which has a step size of 0.001. I would prefer to tweak the gains a little more to ensure it plays well with default step size. I will look at what plays well, but if you have some gains tested if you can tweak it to include these number I will be more than happy to merge it.

Removes most of the instability in simulation behavior when lifting objects
Makes tipping behavior in simulation more plausible
I assume that the mass of an assembled base is at least this heavy, given that the robot is around 50lbs total
@nickswalker
Copy link
Author

The instability seems to have been from the arm direct position simulation. Here's the behavior with that controller using effort instead. I increased the mass of the base to 6kg to get the tipping behavior to look reasonable, but it looks like most of the robot's mass is not accounted for in the URDF. It'd be best if someone could measure this with a partially disassembled robot.

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.

Joint simulation is poor
2 participants