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

ergoCub 1.1 S/N:001 – Torso Pitch OverCurrent #1793

Closed
carloscp3009 opened this issue Apr 15, 2024 · 20 comments
Closed

ergoCub 1.1 S/N:001 – Torso Pitch OverCurrent #1793

carloscp3009 opened this issue Apr 15, 2024 · 20 comments
Assignees
Labels
ergoCub 1.1 S/N:001 ergoCub1.1 platform

Comments

@carloscp3009
Copy link

Robot Name πŸ€–

ergoCub 1.1 S/N:001

Request/Failure description

During my experiments, the torso pitch motor turned off due to overcurrent.

Detailed context

I was conducting an experiment this morning, during the initial calibration there was a problem with the torso pitch.. I restarted the calibration and everything went well. Then, during the first attempt to make the robot walk the torso pitch went to hardware fault due to overcurrent just as shown in the attached material.

Screenshot from 2024-04-15 08-46-25

Attached you will find a video and the logger output.

Additional context

log_ergocub-torso_yarprobotinterface_3916.zip

torso_pitch_fault.mp4

How does it affect you?

I can not use the robot.

@github-actions github-actions bot changed the title Torso Pitch OverCurrent ergoCub 1.1 S/N:001 – Torso Pitch OverCurrent Apr 15, 2024
@github-actions github-actions bot added the ergoCub 1.1 S/N:001 ergoCub1.1 platform label Apr 15, 2024
@GiulioRomualdi
Copy link
Member

We tried several time the walking and the torso went always in overcurrent. I'm afraid that the motor was burned.

As far as I know, the temperature check was disabled for the last demo

cc @S-Dafarra @DanielePucci @valegagge @MSECode @maggia80

@S-Dafarra
Copy link

Unfortunately, we did not manage to revert the FW in time

@GiulioRomualdi
Copy link
Member

cc @evelyd @G-Cervettini @rob-mau

@GiulioRomualdi
Copy link
Member

Hi all, I was wondering which is the plan to fix this issue since it prevents the usage of the robot for most of the people. In case the motor is burned and there aren't other spare motors, we may think to add the pin to block the joint. If we go for this solution, we will need to modify the model (cc @Nicogene and @traversaro) and all the configuration file associated to the robot.

cc @DanielePucci

@SimoneMic
Copy link

Hello,
Is there any time estimation on how long the temporary fix will take to be put in place?
It's actually blocking a lot of activities.

Thank you so much!

@fbiggi
Copy link

fbiggi commented Apr 23, 2024

Ciao a tutti, mi chiedevo quale sia il piano per risolvere questo problema poichΓ© impedisce l'utilizzo del robot per la maggior parte delle persone. Nel caso in cui il motore sia bruciato e non ci siano altri motori di riserva, potremmo pensare di aggiungere il perno per bloccare lo snodo. Se scegliamo questa soluzione, dovremo modificare il modello (cc@NicogeneE@traversaro) e tutto il file di configurazione associato al robot.

cc@DanielePucci

Hi @GiulioRomualdi
The pin to lock the pitch was inserted due to a burnt motor
Please remind everyone who will be using the robot that this component is mounted

@AntonioConsilvio
Copy link
Contributor

Just to clarify, we inserted the torso pitch lock as we do not have the motors to replace the burnt one.

The motors have been ordered, when they arrive we will proceed with the replacement.

cc @GiulioRomualdi @SimoneMic

@SimoneMic
Copy link

Ok thanks, so we can use the robot now.
Have the config files and urdf been updated?

Thanks again

@AntonioConsilvio
Copy link
Contributor

Ok thanks, so we can use the robot now. Have the config files and urdf been updated?

Thanks again

The robot cannot be used at the moment because the lower arm is being upgraded.

When the upgrade is complete, the robot will be available!

@GiulioRomualdi
Copy link
Member

Once the robot is back to be sure we are able to run it with whole body dynamics we will need to use the latest master in whole body estimator (the one containing robotology/whole-body-estimators#188 ) and update the associated config file.

@traversaro
Copy link
Member

To clarify, the plan at this point is not to modify anymore the URDF thanks to the changes mentioned by @GiulioRomualdi . For some reason you need the URDF modified in your software @SimoneMic ?

@SimoneMic
Copy link

SimoneMic commented Apr 24, 2024

To clarify, the plan at this point is not to modify anymore the URDF thanks to the changes mentioned by @GiulioRomualdi . For some reason you need the URDF modified in your software @SimoneMic ?

I need the transform from the head to the feet. The TF, which is computed from the /joint_states topic, uses the robot URDF.
Furthermore, a lot of other software components(which are used for grasping, manipulation and perception) use as a reference frame the root_link of the robot.
Since the torso_pitch is a joint inside the kinematic chain, it impacts them all.

Maybe I am missing something: since the torso_pitch has been fixed in place with a pin, it will also be excluded from the startup (calibration, and so on), correct?
And it will not be enabled, right?
Or do we still use the robot in permanent HW fault?

@traversaro
Copy link
Member

Maybe I am missing something: since the torso_pitch has been fixed in place with a pin, it will also be excluded from the startup (calibration, and so on), correct?
And it will not be enabled, right?
Or do we still use the robot in permanent HW fault?

This is a good point, if it is present but in permanent HW fault, that should not be a problem as long as the encoder actually measure the correct value. However, if for some reason the joint is not available in the YARP controlboards or if it is present but reporting the wrong joint position, we can still try to avoid the expensive step of manually creating a custom URDF with the joint fixed to 10, by ensuring that torso_pitch remains available in /joint_states ROS 2 topic by adding a simple fakeMotionControl YARP device that contains a single joint torso_pitch and attach it to the remapper used to publish the /joint_states topic, i.e. in https://github.com/robotology/robots-configuration/blob/f3cb9d6c45de23966f6d99aae2f4ae984edcc44a/ergoCubSN001/wrappers/motorControl/alljoints-mc_remapper.xml .

@valegagge
Copy link
Member

Hi guys,
what I suppose is that we need to remove the joint from the calibrator list because we cannot use anymore. At this stage, the controlboard sends the joint position anyway but it is not calibrated.
If the AMO encoder is configured then the joint position is a not calibrated value, while if only motor encoder is configured the joint position is zero, and I guess it is the position of the joint pinned.

If my suppositions are true, the YARP control board will receive the position of the joint at zero without changing anything. The only problem I see with this solution is that if an application set a control mode to the entire body part it gets an error because it is impossible to change the control mode of an uncalibrated joint.

@traversaro
Copy link
Member

If the AMO encoder is configured then the joint position is a not calibrated value, while if only motor encoder is configured the joint position is zero, and I guess it is the position of the joint pinned.

As far as I remember the joint is pinned at 10.0 (deg) position.

@carloscp3009
Copy link
Author

@fbiggi FYI

@valegagge
Copy link
Member

@AntonioConsilvio please verify that on the robot the original 2foc fw is used instead of the one for the demo.

cc @fbiggi @Fabrizio69 @MSECode @S-Dafarra

@AntonioConsilvio
Copy link
Contributor

Hi! We replaced the burnt motor with a new one! The motor has been calibrated and tested, here the PR:

@valegagge, the robot has the original 2foc firmware in this moment.

Thank you @fbiggi @Gandoo @AntonioAzocar for your help with motor replacement!

@AntonioConsilvio AntonioConsilvio moved this from Triage to Review/QA in iCub Tech Support May 13, 2024
@S-Dafarra
Copy link

We tried to walk and it seems to be working fine!

@AntonioConsilvio
Copy link
Contributor

Thank you for the feedback! Closing!

@github-project-automation github-project-automation bot moved this from Review/QA to Done in iCub Tech Support May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ergoCub 1.1 S/N:001 ergoCub1.1 platform
Projects
Status: Done
Development

No branches or pull requests

8 participants