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

R1 S/N:003 – Errors with R1 hands (ergomorphing) #1989

Open
fbrand-new opened this issue Dec 6, 2024 · 7 comments
Open

R1 S/N:003 – Errors with R1 hands (ergomorphing) #1989

fbrand-new opened this issue Dec 6, 2024 · 7 comments
Assignees

Comments

@fbrand-new
Copy link

Robot Name πŸ€–

R1 S/N:003

Request/Failure description

I just turned on the robot with the new hands calibrated by @MSECode and I have these errors:
Image
I do not know if it is safe to proceed further, or if I should turn off the roboto

@AntonioConsilvio @Fabrizio69 @MSECode

Detailed context

See above

Additional context

No response

How does it affect you?

No response

@MSECode
Copy link

MSECode commented Dec 6, 2024

Hi,
can u send me the configuration file of the calibrator and the one of the motion control, that I'll double check the calibration parameters and the PID values.
Thanks

@github-actions github-actions bot changed the title Errors with R1 hands (ergomorphing) R1 S/N:003 – Errors with R1 hands (ergomorphing) Dec 6, 2024
Copy link

github-actions bot commented Dec 6, 2024

⚠️ Detected missing mandatory information

Hi @fbrand-new πŸ‘‹πŸ»

Some of the following points need your attention.

You are required to:

  • enter the Robot Name.
  • give the issue a meaningful Title.
  • provide a Detailed Context (at least 100 characters).

Please, mark the points above as solved once done.

@fbrand-new
Copy link
Author

Update with the log files of the yri from inside and outside the docker on the base
log.txt
log_no_docker.txt

@mfussi66 mfussi66 self-assigned this Dec 10, 2024
@mfussi66
Copy link
Member

The OP issue regarding i2t limit seems a side-effect of the calibration procedure being unable to read the FAP data. The sensors read just fine and we can see it in the port /cer/left_hand/POS<2 or 4>:o if the service is enabled in the yarprobotinterface config file.

@maggia80
Copy link
Contributor

  • embobjPOS was not enabled in CCMake in the r1-base
  • POS services were not enabled in the yarprobotinterface.ini file

we still have issues in the right hand:

Image

@mfussi66
Copy link
Member

mfussi66 commented Dec 11, 2024

In order to run the calibration procedure correctly, as I talked with @MSECode , the POS service needs to be active before the calibrator. If the opposite happens, the operation fails.

In ergoCubSN002 the POS service starts right at the beginning of the yarprobotinterface file:

https://github.com/robotology/robots-configuration/blob/master/ergoCubSN002/ergocub.xml#L12-L22 .

Instead, in R1SN003 it starts later:

https://github.com/robotology/robots-configuration/blob/master/R1SN003/R1.xml#L36-L46

and therefore the time between the POS and the calibrator is not sufficient.

I'm not entirely sure how the yarprobotinterface works, but I'm afraid there is some kind of race condition in this scenario.

@MSECode
Copy link

MSECode commented Dec 11, 2024

In order to run the calibration procedure correctly, as I talked with @MSECode , the POS service needs to be active before the calibrator. If the opposite happens, the operation fails.

In ergoCubSN002 the POS service starts right at the beginning of the yarprobotinterface file:

https://github.com/robotology/robots-configuration/blob/master/ergoCubSN002/ergocub.xml#L12-L22 .

Instead, in R1SN003 it starts later:

https://github.com/robotology/robots-configuration/blob/master/R1SN003/R1.xml#L36-L46

and therefore the time between the POS and the calibrator is not sufficient.

I'm not entirely sure how the yarprobotinterface works, but I'm afraid there is some kind of race condition in this scenario.

Exactly, or at least that is what I've always seen on ergoCub and iCub robots, with the POS service running first. Thus, that can be said to be the standard use case. I'm not sure, anyways, if that is mandatory for having the whole system running correctly but what @mfussi66 said is totally correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Triage
Development

No branches or pull requests

5 participants