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

ergoCubGazeboV1 - joints l_shoulder_yaw and l_wrist_yaw joints go in hardware fault #60

Closed
xEnVrE opened this issue Feb 27, 2023 · 7 comments · Fixed by #61
Closed

ergoCubGazeboV1 - joints l_shoulder_yaw and l_wrist_yaw joints go in hardware fault #60

xEnVrE opened this issue Feb 27, 2023 · 7 comments · Fixed by #61
Assignees
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix

Comments

@xEnVrE
Copy link
Contributor

xEnVrE commented Feb 27, 2023

While testing the simulation using the ergoCubGazeboV1 model, I noticed that often the joints l_shoulder_yaw and l_wrist_yaw go in hardware fault, especially when the l_elbow joint is moved.

By playing a bit with the tuning of the PID in simulation, I noticed that the problem seems to disappear by removing the derivative action for joints l_shoulder_yaw and l_wrist_yaw - not saying this is the cause, maybe the derivative action is just emphasizing the effect caused by the real problem, e.g. wrongly scaled inertias, etc..

Is it ok to open a PR in order to change them in the file used by the urdf generator, i.e. https://github.com/icub-tech-iit/ergocub-software/blob/master/urdf/simmechanics/data/ergocub/conf/gazebo_ergocub_left_arm.ini? Otherwise, at the moment, the left arm is quite not usable.

Have the current values of the PID been inherited from iCub2.x simulated models or have they been tuned somehow?

Thanks

@Nicogene
Copy link
Member

Hi @xEnVrE!

Is it ok to open a PR in order to change them in the file used by the urdf generator, i.e. master/urdf/simmechanics/data/ergocub/conf/gazebo_ergocub_left_arm.ini? Otherwise, at the moment, the left arm is quite not usable.

Sure! For the time being we can fix this issue like you said.
Is this happening only on the left arm?
Because I would do the same change also on the right arm.

@Nicogene
Copy link
Member

I probably found out the problem, I will submit the fix soon @xEnVrE

@xEnVrE
Copy link
Contributor Author

xEnVrE commented Feb 28, 2023

Thanks @Nicogene

@Nicogene Nicogene self-assigned this Feb 28, 2023
@Nicogene Nicogene added domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix labels Feb 28, 2023
@lrapetti
Copy link
Contributor

Hi @Nicogene, I have updated and installed the latest master, but the issue is still occurring.

When launching this gazebo world: ergoCub_lifting_40.zip

I immediately get failure of left wrist and shoulder yaw:

[ERROR] |gazebo-yarp-plugins.plugins.GazeboYarpControlBoard| An hardware fault occurred on joint  2  torque too big! (  -2423.91  )
[ERROR] |gazebo-yarp-plugins.plugins.GazeboYarpControlBoard| An hardware fault occurred on joint  4  torque too big! (  -2251.08  )

@lrapetti
Copy link
Contributor

Hi @Nicogene, I have updated and installed the latest master, but the issue is still occurring.

When launching this gazebo world: ergoCub_lifting_40.zip

I immediately get failure of left wrist and shoulder yaw:

[ERROR] |gazebo-yarp-plugins.plugins.GazeboYarpControlBoard| An hardware fault occurred on joint  2  torque too big! (  -2423.91  )
[ERROR] |gazebo-yarp-plugins.plugins.GazeboYarpControlBoard| An hardware fault occurred on joint  4  torque too big! (  -2251.08  )

Removing the derivative gains as mentioned in #60 (comment) solves the issue also for this case.

@Nicogene let me know if you want me to open a new issue or re-open this one

@xEnVrE
Copy link
Contributor Author

xEnVrE commented Feb 28, 2023

I tested it again and for me the issue seems solved now without the need to disable the derivative action.

cc @Nicogene @lrapetti

@lrapetti
Copy link
Contributor

lrapetti commented Mar 1, 2023

I tested it again and for me the issue seems solved now without the need to disable the derivative action.

I will double check again if the corrected model was loaded in my test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants